PR/52820 CVS commit: [netbsd-8] src/sys/dev/pci/ixgbe

["Soren Jacobsen" <snj%netbsd.org@localhost>]

The following reply was made to PR kern/52820; it has been noted by GNATS.

From: "Soren Jacobsen" <snj%netbsd.org@localhost>
To: gnats-bugs%gnats.NetBSD.org@localhost
Cc: 
Subject: PR/52820 CVS commit: [netbsd-8] src/sys/dev/pci/ixgbe
Date: Sat, 13 Jan 2018 21:40:02 +0000

 Module Name:	src
 Committed By:	snj
 Date:		Sat Jan 13 21:40:01 UTC 2018
 
 Modified Files:
 	src/sys/dev/pci/ixgbe [netbsd-8]: ixgbe.c ixv.c
 
 Log Message:
 Pull up following revision(s) (requested by msaitoh in ticket #490):
 	sys/dev/pci/ixgbe/ixgbe.c: revision 1.116-1.119
 	sys/dev/pci/ixgbe/ixv.c: revision 1.76-1.77
 Don't panic when resource shortage occured. Fixes PR#52820 reported by
 kardel@:
  - Don't use if_free() because ixgbe(4) don't use if_alloc().
  - Move location of {ixgbe,ixv}_setup_interface() call at a position that
    any error don't occur. One of the reason is that it should be. Another
    reason is that it's hard to call ether_ifdetach() and if_detach() when
    cold == 1 (because of pserialize_perform, xc_wait, timing of domaininit
    and maybe more).
 --
 Don't panic when resource shortage occured. Like ixgbe.c rev. 1.116.
  - Move location of {ixgbe,ixv}_setup_interface() call at a position that
    any error don't occur. One of the reason is that it should be. Another
    reason is that it's hard to call ether_ifdetach() and if_detach() when
    cold == 1 (because of pserialize_perform, xc_wait, timing of domaininit
    and maybe more).
 --
 Fix panic when only link interrupt can't be established.
 --
 - If MSI can't be used on some environment, fallback to INTx correctly.
 - Use single vector MSI when number of CPU is 1 to save interrupt slot.
 --
 Fallback from MSI-X to MSI or INTx if MSI-X setup failed.
 
 
 To generate a diff of this commit:
 cvs rdiff -u -r1.88.2.7 -r1.88.2.8 src/sys/dev/pci/ixgbe/ixgbe.c
 cvs rdiff -u -r1.56.2.4 -r1.56.2.5 src/sys/dev/pci/ixgbe/ixv.c
 
 Please note that diffs are not public domain; they are subject to the
 copyright notices on the relevant files.