[openib-general] Re: [PATCH]: was Re: LLTX and netif_stop_queue
Grant Grundler
iod00d at hp.com
Thu Jan 20 13:56:50 PST 2005
On Thu, Jan 20, 2005 at 01:34:20PM -0800, David S. Miller wrote:
> Even worse is that this breaks the acenic driver too because
> it does this:
>
> unsigned long maxjiff = jiffies + 3*HZ;
>
> if (time_before(jiffies, maxjiff)) {
> barrier();
> cpu_relax();
> goto restart;
> }
>
> in it's ->hard_start_xmit() routine.
>
> I was auditing spin_lock_irq() usage in ->hard_start_xmit()
> routines when I caught this.
>
> This one isn't impossible to fix though. We can replace the
> jiffies games with a udelay/mdelay and a "maxloop" variable.
> Any takers?
I saw the next mail suggesting to revert the changes because of more fallout.
But if/when acenic needs changes like those proposed above, I can implement
and test proposed changes if folks have the patience to wait a week or so.
acenic is no where on my "official HP supported NICs" list but I happen
to have the HW/infrastructure to test them.
grant
More information about the general
mailing list