Page MenuHomeFreeBSD

if_bnxt: Correcting the firmware package version parsing logic.
ClosedPublic

Authored by chandrakanth.patil_broadcom.com on Dec 7 2023, 12:27 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Jul 5, 8:56 PM
Unknown Object (File)
Thu, Jul 4, 1:05 AM
Unknown Object (File)
Wed, Jul 3, 2:20 PM
Unknown Object (File)
Wed, Jul 3, 3:47 AM
Unknown Object (File)
Fri, Jun 14, 1:08 AM
Unknown Object (File)
May 9 2024, 7:11 PM
Unknown Object (File)
May 9 2024, 3:37 AM
Unknown Object (File)
Apr 22 2024, 1:56 AM
Subscribers
None

Details

Summary

The firmware package version currently appears as "Unknown" through
the sysctl interface. The parsing logic for extracting the firmware
package version from the package log has been modified to ensure
compatibility with all controllers.

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

bnxt_sysctl.c
740 ↗(On Diff #131119)

So why not !isspace(*p) here? Might want to have somenote what the format of pkglog is....

So this looks fine, but maybe a comment would make it a bit better. It is just moving code, I understand, but restoring the 'tab deliminated field' comment in the copied code I think would address my objections and answer my question about isspace.
So unless the answer is quite a bit different than I think, and changes to address this should be fine.

This revision is now accepted and ready to land.Feb 7 2024, 7:42 PM