Thread (7 messages) flat view 7 messages, 4 authors, 2007-07-16

Re: [Bugme-new] [Bug 8736] New: New TC deadlock scenario

From: David Miller <davem@davemloft.net>
Date: 2007-07-15 03:49:49

From: Ranko Zivojnovic <redacted>
Date: Sat, 14 Jul 2007 19:45:35 +0300
On Sat, 2007-07-14 at 17:43 +0200, Patrick McHardy wrote:
quoted
Patrick McHardy wrote:
quoted
Andrew Morton wrote:
quoted
quoted
http://bugzilla.kernel.org/show_bug.cgi?id=8736

Here is another scenario I bumped onto - qdisc_watchdog_cancel() and
qdisc_restart() deadlock.

[...]
DEADLOCK!


Good catch.

Please try reverting commit 1936502d00ae6c2aa3931c42f6cf54afaba094f2,
that should fix it.

Ranko, did you get a chance to test this? I've attached the patch
since it doesn't revert cleanly ..
I have the commit reverted on my tree and have been testing it along
with the other changes to qdisc code I'm currently doing and it does
rectify the problem.

Acked-by: Ranko Zivojnovic <redacted>
Applied, thanks everyone.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help