Jeff Garzik wrote:

Andrew Morton wrote:

quoted

Jeff Garzik wrote:

quoted

...
Also, a further question:  do you have access to the slave struct
net_device?  If so, just test netif_carrier_ok(slave_dev) and avoid all
that ioctl calling if it returns non-zero.

Make that "avoid all that ioctl calling from interrupt context", which
is a bug.  Of the box-killing variety ;)

Indeed.  /me looks at the bond_check_dev_link callers more closely and
shudders.

That wants fixing...

Note that netif_carrier_ok() can indeed be checked in interrupt context.
  And if someone wants to send me patches converting more drivers to use
netif_carrier_{on,off}, I would be very happy :)

That would be best.

I'm so slack.  I received the below two years ago; Nelson has
added netif_carrier_foo support to 3c59x.c.  As Jeff says: patches
solicited.

-------- Original Message --------
Subject: Re: netlink
Date: Wed, 14 Jun 2000 11:19:33 +0800 (SST)
From: Nelson <redacted>
To: Andrew Morton <redacted>
References: [off-list ref]

hi Andrew,

The modified 3c59x.c is attached.

for your easy reference, these lines in the attached driver 
was modified:

69-72:		notes on what are added
121-123:	defined the time to expiry of the vertex_timer
		as TX_EXPIRE. i have set the value as 1*HZ.
		the original value should be 60*HZ. you may
		set to any value you feel is good ;p you r rite
		abt the 1 sec part since reading the MII management
		registers is time consuming.
1122:		used TX_EXPIRE instead
1335, 1399, 1428:		
		set next_tick to TX_EXPIRE
1351:		added calls to netif_carrier_on()
1356:		added calls to netif_carrier_off()
1367-1371:	added checking of link state

do let me know if there are any discrepencies.
thanx.