Re: cvs commit: src/sys/conf options src/sys/dev/netif/bge Makefile if_bge.c if_bgereg.h

["Devon H. O'Dell" <dodell@xxxxxxxxxxxxxxx>]

On Mon, Aug 22, 2005 at 01:57:15PM -0700, Devon H. O'Dell wrote:
> On Mon, Aug 22, 2005 at 09:52:47PM +0100, Hiten Pandya wrote:
> > Joerg Sonnenberger wrote:
> > >joerg       2005/08/22 11:29:52 PDT
> > >
> > >DragonFly src repository
> > >
> > >  Modified files:
> > >    sys/conf             options 
> > >    sys/dev/netif/bge    Makefile if_bge.c if_bgereg.h 
> > >  Log:
> > >  Create a kernel option BGE_FAKE_AUTONEG for IBM/Intel blade servers,
> > >  which should make the DNLK switch module work.  IBM/Intel blade server
> > >  with Intel or AD switch modules should work without the kernel options
> > >  
> > >  Obtained-from: FreeBSD
> > >  Submitted-by: Sepherosa Ziehau <sepherosa@xxxxxxxxx>
> > 
> > Please do not make this a kernel option, instead turn it into a sysctl.  I 
> > do not like the idea of recompiling the kernel just so that some part of a 
> > driver available in GENERIC can be made working.
> > 
> > It's non-intuitive.  We need to be dynamic. :-)
> > 
> > 				-Hiten
> 
> Having this doesn't help because it has problems with the autonegotiation
> for the switch and will not communicate any more without the code. You can't
> renegotiate. Hence the change.
> 
> The change was made because we discovered this problem with our blades here
> at OMS.

I should note that this change is necessary because the switch is broken, not
the driver...
 
> -Devon