[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20080328103809.GB23039@gondor.apana.org.au>
Date: Fri, 28 Mar 2008 18:38:09 +0800
From: Herbert Xu <herbert@...dor.apana.org.au>
To: Matheos Worku <Matheos.Worku@....COM>
Cc: David Miller <davem@...emloft.net>, jesse.brandeburg@...el.com,
jarkao2@...il.com, netdev@...r.kernel.org, hadi@...erus.ca
Subject: Re: 2.6.24 BUG: soft lockup - CPU#X
On Thu, Mar 27, 2008 at 06:58:18PM -0700, Matheos Worku wrote:
>
> In general, while the TX serialization improves performance in terms to
> lock contention, wouldn't it reduce throughput since only one guy is
> doing the actual TX at any given time. Wondering if it would be
> worthwhile to have an enable/disable option specially for multi queue TX.
Unless you have a truly lockless driver where two CPUs can transmit
at the same time then regardless of what we do only one CPU can
transmit at a time.
Given that premise, we might as well let one CPU transmit as much
as possible since moving to another CPU after each packet is going
to bounce a lot more than just the spin lock and that is going to
be expensive.
Of course getting stuck in that loop forever when there is other
work to be done (such as running the watchdog :) is bad. But
hopefully that's what this patch will fix.
Thanks,
--
Visit Openswan at http://www.openswan.org/
Email: Herbert Xu ~{PmV>HI~} <herbert@...dor.apana.org.au>
Home Page: http://gondor.apana.org.au/~herbert/
PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Powered by blists - more mailing lists