Re: bk16 changes to cbq

["David S. Miller" <davem@xxxxxxxxxx>]

On 03 Jul 2004 09:36:50 -0400
jamal <hadi@xxxxxxxxxx> wrote:

> -               if (q->wd_expires && !netif_queue_stopped(sch->dev)) {
> +               if (q->wd_expires) {
>                         long delay = PSCHED_US2JIFFIE(q->wd_expires);
>                         if (delay <= 0)
>                                 delay = 1;
> 
> What i remember is this (4-5 years back) used to cure a bug - cant
> remember the details unfortunately, but Alexey may remember.
> I am hoping removal of the above line implies that those conditions dont
> exist anymore? 

The test is racy with drivers, on the very next line the
driver could take a TX completion interrupt and unplug the
queue invalidating the test entirely.

If the test proves wrong, that's OK because we'll try again
at the top level of packet queue dispatch.

There was a good explaination of Stephen's patch
on netdev when he posted it.