[ofa-general] Re: Continue of "defer skb_orphan() until irqs enabled"

Eli Cohen eli at dev.mellanox.co.il
Sun Sep 28 04:39:45 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.
> 

We'll integrate this into ofed 1.4 and monitor this through our
regression system.



More information about the general mailing list