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)
Sun, Feb 9, 7:12 PM
Unknown Object (File)
Tue, Jan 28, 6:34 AM
Unknown Object (File)
Mon, Jan 27, 6:58 AM
Unknown Object (File)
Fri, Jan 24, 6:22 AM
Unknown Object (File)
Thu, Jan 23, 6:43 PM
Unknown Object (File)
Jan 16 2025, 10:35 AM
Unknown Object (File)
Jan 9 2025, 2:31 AM
Unknown Object (File)
Jan 8 2025, 5:41 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