On Wed, 19 Jan 2005 19:00:31 -0500, Jeff Garzik <jgarzik@xxxxxxxxx> wrote:
> Stephen Hemminger wrote:
> > This is the second public release of my new rewrite of the SysKonnect
> > Gigabit
> > Ethernet driver. This 0.3 version fixes bugs with link up/down and ethtool
> > phys_id support. It adds ethtool support for interrupt coalescing and pause
> > parameters. Performance is good, I am able to get 941 Mbit/sec receiving
> > (TCP using iperf). But obviously, the driver is still experimental.
> >
> > This driver doesn't support Yukon2 (yet), and there is report of problem
> > with D-Link card (Yukon-EC).
> >
> > The patch should work on 2.6.8 or later, I am testing with 2.6.11-rc1.
> > Also available as download from http://developer.osdl.org/shemminge/skge
<snip>
> > +static int skge_phys_id(struct net_device *dev, u32 data)
> > +{
> > + struct skge_port *skge = netdev_priv(dev);
> > +
> > + if(!data || data > (u32)(MAX_SCHEDULE_TIMEOUT / HZ))
> > + data = (u32)(MAX_SCHEDULE_TIMEOUT / HZ);
> > +
> > + /* start blinking */
> > + skge->blink_on = 1;
> > + mod_timer(&skge->led_blink, jiffies);
> > +
> > + set_current_state(TASK_INTERRUPTIBLE);
> > + schedule_timeout(data * HZ);
> > + del_timer_sync(&skge->led_blink);
Please consider using msleep_interruptible(data * 1000) here. The same
substitution has worked in other drivers that have used the exact same
code.
Thanks,
Nish
|