On Thu, Nov 17, 2016 at 22:24 +0100, Mike Belopuhov wrote: > On Wed, Nov 16, 2016 at 23:04 +0100, Mike Belopuhov wrote: > > Hi, > > > > I've done a massive update of our ix(4) driver that brings > > support for X550 family of controllers including those > > integrated into new Xeon chips as well as QSFP support for > > X520 (82599) but this needs thorough testing. If you're > > using Intel 10Gb controllers, please make sure that you > > either (or both!) test the complete diff found at this URL: > > http://gir.theapt.org/~mike/ixgbe.diff or next few snapshots > > that will (hopefully) contain bits of this monster diff. > > > > To test the monster diff, make sure that you are running a > > recent snapshot and your kernel source code is up-to-date, > > then reset a few files to the specified revisions and > > remove the support file for X550: > > > > % pwd > > /usr/src > > % cvs up -r1.326 sys/dev/pci/files.pci > > % cvs up -r1.133 sys/dev/pci/if_ix.c > > % cvs up -r1.14 sys/dev/pci/ixgbe.c > > % cvs up -r1.23 sys/dev/pci/ixgbe.h > > % cvs up -r1.11 sys/dev/pci/ixgbe_82598.c > > % cvs up -r1.12 sys/dev/pci/ixgbe_82599.c > > % cvs up -r1.13 sys/dev/pci/ixgbe_phy.c > > % cvs up -r1.22 sys/dev/pci/ixgbe_type.h > > % cvs up -r1.4 sys/dev/pci/ixgbe_x540.c > > % rm -f sys/dev/pci/ixgbe_x550.c > > > > To verify that files have been reset: > > > > % pwd > > /usr/src > > % fgrep "//T1" sys/dev/pci/CVS/Entries > > /files.pci/1.326/Mon Sep 12 09:45:53 2016//T1.326 > > /if_ix.c/1.133/Thu Oct 27 05:00:50 2016//T1.133 > > /ixgbe.c/1.14/Wed Nov 26 17:03:52 2014//T1.14 > > /ixgbe.h/1.23/Tue Oct 4 09:24:02 2016//T1.23 > > /ixgbe_82598.c/1.11/Mon Aug 5 19:58:06 2013//T1.11 > > /ixgbe_82599.c/1.12/Fri May 1 04:15:00 2015//T1.12 > > /ixgbe_phy.c/1.13/Fri May 1 04:15:00 2015//T1.13 > > /ixgbe_type.h/1.22/Wed Nov 16 21:53:57 2016//T1.22 > > /ixgbe_x540.c/1.4/Wed May 20 14:34:27 2015//T1.4 > > > > And then test and apply the diff: > > > > % pwd > > /usr/src > > % patch -Csp0 </tmp/ixgbe.diff && patch -sp0 </tmp/ixgbe.diff > > > > Make sure to reset files every time the source tree gets updated. > > > > Cheers, > > Mike > > As of today and file revisions below, most of the boilerplate > code is now committed. I've tried very hard to not introduce > any noticeable changes in behavior so far. > > if_ix.c/1.133 > ixgbe_82598.c/1.15 > ixgbe_x550.c/1.1 > ixgbe.c/1.19 > ixgbe.h/1.26 > ixgbe_82599.c/1.16 > ixgbe_x540.c/1.9 > ixgbe_type.h/1.29 > ixgbe_phy.c/1.18 > > Remaining bits are below. I'll start picking at low hanging fruits, > but won't mind another pair of eyes. >
I've just realised I forgot to commit one small bit that is also not part of this diff that might fail the kernel compile. I'll post an update in about 12 hours.
