Page MenuHomeFreeBSD

ixgbe(4): Update HEAD (p3) to 3.2.12-k
ClosedPublic

Authored by cramerj_intel.com on Jun 16 2017, 1:31 AM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Nov 15, 12:09 AM
Unknown Object (File)
Sun, Nov 10, 5:11 AM
Unknown Object (File)
Thu, Nov 7, 6:57 PM
Unknown Object (File)
Thu, Nov 7, 9:02 AM
Unknown Object (File)
Thu, Nov 7, 6:33 AM
Unknown Object (File)
Wed, Nov 6, 9:33 PM
Unknown Object (File)
Wed, Nov 6, 2:35 PM
Unknown Object (File)
Tue, Nov 5, 10:01 PM

Details

Summary

Update to ixgbe including:

  • Support for X550EM devices.
  • Support for Bypass adapters.
  • Flow Director code moved to separate files
  • SR-IOV code moved to separate files
  • Netmap code moved to separate files

This does NOT contain iflib support. Review D10293 (with iflib support) will be abandoned and a new review will be created to add iflib support.

Test Plan

Work-in-progress. Respectfully request waiting for Jeff's team to perform a validation pass before committing.

Diff Detail

Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 10159
Build 10582: arc lint + arc unit

Event Timeline

  • Fixed compilation issue with unused variables.
  • Fixed compilation error with ixv when compiled dynamically.
  • Removed "Frame Limit" flag as it seems unnecessary.
  • Fixed a load error when trying to load ixv dynamically.
  • Fixed unresolved symbol issue with ixv.
This revision is now accepted and ready to land.Jun 29 2017, 11:50 PM

We need to hold off on committing this. We are investigating some issues that were discovered today.

This revision now requires changes to proceed.Jul 1 2017, 2:54 AM
This revision is now accepted and ready to land.Jul 7 2017, 11:15 PM

In the future when you find yourselves implementing synchronization primitives like swfw_sync or your hand rolled locks in if_bypass.c please make sure you get the attention and sign off from @markj, @jhb, or myself.

sys/dev/ixgbe/if_bypass.c
48

This lock needs to be replaced.

Also, can we stop massively refactoring the drivers until after the iflib version goes in to HEAD? And in general, there's a lot of non-productive function shuffling that goes on in Intel drivers that makes it a lot more work for downstream users maintaining their own branches.

sys/dev/ixgbe/ixgbe_fdir.h
45

Has this code ever been tested with IXGBE_FDIR enabled? This leads to duplicate symbols.