[ofa-general] Re: Continue of "defer skb_orphan() until irqs enabled"
akepner at sgi.com
akepner at sgi.com
Fri Sep 26 14:21:11 PDT 2008
On Fri, Sep 26, 2008 at 01:19:00PM -0700, Roland Dreier wrote:
> How about this? Instead of trying to rely on some complicated and
> fragile reasoning about when some race might occur, let's just do what
> we want to do anyway and get rid of LLTX. We change from priv->tx_lock
> (taken with IRQ disabling) to netif_tx_lock (taken on with
> BH-disabling). And then we can keep the skb_orphan in the place it is,
> since our xmit routine runs with IRQs enabled.
> ...
Thanks for doing this work, Roland.
We'll give it a test asap. (Actually, we'll need to port the patch
to OFED 1.3.1 first, since that's what we're using/shipping.)
--
Arthur
More information about the general
mailing list