Page MenuHomeFreeBSD

ixgbe(4): Update shared code, add support for X552, fix bug
ClosedPublic

Authored by krzysztof.galazka_intel.com on Jan 12 2018, 4:46 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Apr 13, 1:41 PM
Unknown Object (File)
Mon, Apr 8, 5:51 PM
Unknown Object (File)
Feb 1 2024, 7:57 AM
Unknown Object (File)
Jan 12 2024, 5:53 PM
Unknown Object (File)
Jan 8 2024, 2:32 AM
Unknown Object (File)
Jan 8 2024, 2:29 AM
Unknown Object (File)
Jan 8 2024, 2:29 AM
Unknown Object (File)
Jan 8 2024, 2:29 AM

Details

Summary

This patch will:

  • Update ixgbe shared code
  • Add support for Intel(R) Ethernet Connection X552 1000BASE-T
  • Add error handling for link state check preventing VF from stopping traffic after changing PF's MTU value.

History of commits included in this revision may be found at https://github.com/intel-wired-ethernet/freebsd/commits/ixgbe-D13885

Test Plan

Compile-tested with minimal touch-testing. Respectfully request waiting for our Validation Team to perform a test pass before committing.

Diff Detail

Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 14454
Build 14601: arc lint + arc unit

Event Timeline

Let me know if you need me to commit this.

This revision is now accepted and ready to land.Jan 16 2018, 1:56 PM
skozlov requested changes to this revision.Jan 16 2018, 4:12 PM
skozlov added a subscriber: skozlov.

Blocking until Intel internal validation is complete.

This revision now requires changes to proceed.Jan 16 2018, 4:12 PM

ixv is totally broken. on a FreeBSD host with PCI_IOV, assigning an IP address to a "bare metal" ixv interface{F3125365} causes a kernel panic. On a Linux host, the VF hangs when assigned an IP address and requires a reboot of the VM to recover.

  • Fix kernel panic occuring when IP is assigned to ixv interface

This patch will fix kernel panic occuring when IP address is assigned
to ixv interface on FreeBSD host with PCI_IOV.

The above two patches resolve aforementioned kernel panic issue. Issue with VFs unable to pass traffic on FreeBSD host with PCI_IOV is under investigation.

piotr.pietruszewski_intel.com retitled this revision from ixgbe(4): Update shared code, add support for X552, fix bugs to ixgbe(4): Update shared code, add support for X552, fix bug.
piotr.pietruszewski_intel.com edited the summary of this revision. (Show Details)
  • Revert SRIOV initialization fix

SRIOV initialization fix is reverted at the request of Validation Team due to its incompleteness.

Rebased on top of HEAD with erj@ patch for ixv_if_init (r328365)

Unrelated, is there a use case for FreeBSD Bhyve here? If we had PF support on FreeBSD, could we use pass through to use VFs?

Unrelated, is there a use case for FreeBSD Bhyve here? If we had PF support on FreeBSD, could we use pass through to use VFs?

Yes, that'd be possible.

Unrelated, is there a use case for FreeBSD Bhyve here? If we had PF support on FreeBSD, could we use pass through to use VFs?

This patch originally was supposed to fix PCI_IOV support, but it introduced regressions. We thought it would be better to remove that portion of the patch until the code is complete and functional.

The validation exposed a memory leak for a specific traffic pattern. After additional investigation the same leak was found in iflib igb/em driver as well.
As that's the only issue discovered during testing, and, most probably, it's not caused by the changes in the patch - it was decided to merge the patch.

This revision is now accepted and ready to land.Mar 19 2018, 5:30 PM
erj added a subscriber: erj.

Committed in rS331224.

(Why do these never get automatically closed when I commit something?)