Yes - thank you very much. It compiles .. haven't tested yet.
I certainly don't understand this fix. It seems to work by not letting us
miss a code path that should unqueue a skb by not queuing it in the first
place, which kind of hurts my head (but I don't grok network code at all).
The unqueue is still missable but never gets an opportunity to be missed
and hurt. I get a vague sensation that this may be a kind of reentrancy
problem, but not much more.
> *** linux-2.0.35/net/bridge/br.c Tue Aug 26 20:05:34 1997
> --- linux/net/bridge/br.c Sat Oct 17 16:04:22 1998
> ***************
> *** 921,930 ****
> skb->pkt_bridged = IS_BRIDGED;
> skb->arp = 1; /* do not resolve... */
> skb->h.raw = skb->data + ETH_HLEN;
> ! save_flags(flags);
> ! cli();
> ! skb_queue_tail(dev->buffs, skb);
> ! restore_flags(flags);
> return(0);
> }
>
> --- 921,927 ----
> skb->pkt_bridged = IS_BRIDGED;
> skb->arp = 1; /* do not resolve... */
> skb->h.raw = skb->data + ETH_HLEN;
> ! dev_queue_xmit(skb, dev, SOPRI_INTERACTIVE);
> return(0);
> }
>
> ***************
> *** 977,986 ****
> skb->pkt_bridged = IS_BRIDGED;
> skb->arp = 1; /* do not resolve... */
> skb->h.raw = skb->data + ETH_HLEN;
> ! save_flags(flags);
> ! cli();
> ! skb_queue_tail(dev->buffs, skb);
> ! restore_flags(flags);
> return(0);
> }
>
> --- 974,981 ----
> skb->pkt_bridged = IS_BRIDGED;
> skb->arp = 1; /* do not resolve... */
> skb->h.raw = skb->data + ETH_HLEN;
> !
> ! dev_queue_xmit(skb, dev, SOPRI_INTERACTIVE);
> return(0);
> }
>
> HTH
>
> - andreas
>
> --
> Andreas Haumer | email: andreas@xss.co.at | PGP key available
> *x Software + Systeme | phone: +43.1.6001508 | on request.
> Buchengasse 67/8 | +43.664.3004449 |
> A-1100 Vienna, Austria | fax: +43.1.6001507 | AH327-RIPE
Peter
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.rutgers.edu
Please read the FAQ at http://www.tux.org/lkml/