On Fri, 2006-23-06 at 13:35 +1000, Herbert Xu wrote: 
> On Thu, Jun 22, 2006 at 08:52:17PM -0400, jamal wrote:
> > 
> > It does feel like the qdisc_is_running though is now a replacement
> > for the need for dev->txlock which existed to protect multi-cpus from
> > entering the device transmit path. Is that unintended side effect?
> > i.e why would dev->txlock be needed anymore in that path?
> 
> It's not totally redundant yet since you can set tx_queue_len to zero.
> It also still protects against the asynchronous paths that take xmit_lock.

The tx_timeout? 

> However, it wouldn't be a bad idea to see if there is a way to reduce the
> number of locks required on the xmit path to one.

It also seems to be there - dont know why it took me so long to see
this. I am actually gut-feeling now that we may get better performance
with LLTX drivers with this approach;-> talk about a flip-flop.

The qualification for LLTX seems to be for a driver to have a
private tx lock which is the case of about every ethernet driver out
there.

cheers,
jamal


-
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to