Re: ix(4) and Unsupported SFP+ Module

2014-12-05 Thread Jonathan Gray
On Fri, Dec 05, 2014 at 11:24:30AM +0100, Gabriel Linder wrote:
 Hi,
 
 On a 5.6-release I have an ix card which refuses to work with unsupported 
 SFP+ modules, saying this : ix0 at pci1 dev 0 function 0 Intel 82599 rev 
 0x01Unsupported SFP+ Module.
 
 However, this seems to be an artificial limitation of the ix driver : the 
 following diff (against sys.tar.gz as of 5.6-release) allows my SFP+ and it 
 works fine so far.
 
 The patch was made in a hurry, so I probably have cut more than I should... 
 Is there a simpler/better way to fix this ?

This is already fixed in -current with rev 1.12:

http://cvsweb.openbsd.org/cgi-bin/cvsweb/src/sys/dev/pci/ixgbe_phy.c.diff?r1=1.11r2=1.12f=h

revision 1.12
date: 2014/08/25 14:26:25;  author: mikeb;  state: Exp;  lines: +6 -26;  
commitid: 8deZACRIBvIf8lft;
We have never limited the definition of supported SPF modules
to the vendor/make whitelist maintained by Intel so there's no
reason to start doing it now.

When syncing the driver to the FreeBSD codebase I have decided
to take this chunk as is but it appears that it breaks cheap
chinese SFP+ fiber optics modules that we all love.  And while
there's still a lot of places where we check for the vendor
OUI, most of these checks are not necessary.

Issue reported and fix tested by Tony Sarendal.  Thanks!



Re: ix(4) and Unsupported SFP+ Module

2014-12-05 Thread Mike Belopuhov
On Fri, Dec 05, 2014 at 11:24 +0100, Gabriel Linder wrote:
 Hi,
 
 On a 5.6-release I have an ix card which refuses to work with unsupported 
 SFP+ modules, saying this : ix0 at pci1 dev 0 function 0 Intel 82599 rev 
 0x01Unsupported SFP+ Module.
 
 However, this seems to be an artificial limitation of the ix driver : the 
 following diff (against sys.tar.gz as of 5.6-release) allows my SFP+ and it 
 works fine so far.
 
 The patch was made in a hurry, so I probably have cut more than I should... 
 Is there a simpler/better way to fix this ?
 
 

I think this is roughly what I have recently committed to -current.
But hey, thanks for taking your time for looking into this!



Re: ix(4) and Unsupported SFP+ Module

2014-12-05 Thread Gabriel Linder
On 12/05/14 13:08, Mike Belopuhov wrote:
 On Fri, Dec 05, 2014 at 11:24 +0100, Gabriel Linder wrote:
 Hi,

 On a 5.6-release I have an ix card which refuses to work with unsupported 
 SFP+ modules, saying this : ix0 at pci1 dev 0 function 0 Intel 82599 rev 
 0x01Unsupported SFP+ Module.

 However, this seems to be an artificial limitation of the ix driver : the 
 following diff (against sys.tar.gz as of 5.6-release) allows my SFP+ and it 
 works fine so far.

 The patch was made in a hurry, so I probably have cut more than I should... 
 Is there a simpler/better way to fix this ?


 I think this is roughly what I have recently committed to -current.
 But hey, thanks for taking your time for looking into this!


Thanks you Mike and Jonathan for answering, I will backport your diff then :)