ix(4): Add support for firmware logging for E610 adapters
AcceptedPublic
Actions

Authored by Yogesh.Bhosale_intel.com on Nov 28 2025, 6:19 AM.

Details

Reviewers

ziaee
kgalazka
kbowling
erj

Group Reviewers

Restricted Owners Package

(Owns No Changed Paths)

Summary

This is part 3 of the support for the new Intel Ethernet E610
family of devices

The ix driver now enables firmware logging on Intel E610 devices
for debugging with Customer Support. Logs are enabled by default
and generated in binary format that requires decoding by support
teams. The collected data is firmware and hardware related for
debugging purposes only.

When the driver loads, it creates a fw_log sysctl node under the
debug section. Events are organized into categories (modules) for
targeted logging, and users can adjust verbosity levels as needed.

This adds sysctl support for the firmware logging feature and
updates the ix(4) manual page with documentation.

Signed-off-by: Yogesh Bhosale <yogesh.bhosale@intel.com>
Co-developed-by: Krzysztof Galazka <krzysztof.galazka@intel.com>

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 68914
Build 65797: arc lint + arc unit

Event Timeline

Yogesh.Bhosale_intel.com created this revision.Nov 28 2025, 6:19 AM

Herald added subscribers: ziaee, ae, imp. · View Herald TranscriptNov 28 2025, 6:19 AM

Yogesh.Bhosale_intel.com requested review of this revision.Nov 28 2025, 6:19 AM

Harbormaster completed remote builds in B68914: Diff 167230.Nov 28 2025, 6:19 AM

Yogesh.Bhosale_intel.com added reviewers: Restricted Owners Package, ziaee, kgalazka.Nov 28 2025, 6:33 AM

Yogesh.Bhosale_intel.com added reviewers: kbowling, erj.

No comment on the implementation, but the manual looks good to me. Thanks Yogesh!

share/man/man4/ix.4
163–194	For alphabetization.

This revision is now accepted and ready to land.Dec 1 2025, 9:58 PM

Oops, I forgot: Please make sure to wrap commit messages at 72 columns!

In D53973#1234193, @ziaee wrote:

No comment on the implementation, but the manual looks good to me. Thanks Yogesh!

Thanks!

In D53973#1234270, @ziaee wrote:

Oops, I forgot: Please make sure to wrap commit messages at 72 columns!

Done!

Just a cursory look on my end, approving for @kgalazka

@kbowling @kgalazka this seems to have been missed

@emaste We had a miscommunication with QA team, but it is finally being tested. I'll commit the patch as soon as I get confirmation that we didn't break anything while porting from out-of-tree.

Sounds good, thanks.

I commented on this review because srgmgr is now on a regular basis looking for phabricator reviews that appear to be stalled (e.g. accepted but not landed) and this one appeared in the list.