netdev
[Top] [All Lists]

Re: ACPI/HT or Packet Scheduler BUG?

To: Herbert Xu <herbert@xxxxxxxxxxxxxxxxxxx>
Subject: Re: ACPI/HT or Packet Scheduler BUG?
From: Patrick McHardy <kaber@xxxxxxxxx>
Date: Sun, 17 Apr 2005 19:46:16 +0200
Cc: Thomas Graf <tgraf@xxxxxxx>, Steven Rostedt <rostedt@xxxxxxxxxxx>, hadi@xxxxxxxxxx, netdev <netdev@xxxxxxxxxxx>, Tarhon-Onu Victor <mituc@xxxxxxxxxxxxxx>, kuznet@xxxxxxxxxxxxx, devik@xxxxxx, linux-kernel@xxxxxxxxxxxxxxx, "David S. Miller" <davem@xxxxxxxxxxxxx>
In-reply-to: <20050416111236.GA31550@xxxxxxxxxxxxxxxxxxx>
References: <Pine.LNX.4.61.0504081225510.27991@xxxxxxxxxxxxxxxxxxxxxxxx> <Pine.LNX.4.61.0504121526550.4822@xxxxxxxxxxxxxxxxxxxxxxxx> <Pine.LNX.4.61.0504141840420.13546@xxxxxxxxxxxxxxxxxxxxxxxx> <1113601029.4294.80.camel@xxxxxxxxxxxxxxxxxxxxx> <1113601446.17859.36.camel@xxxxxxxxxxxxxxxxxxxxx> <1113602052.4294.89.camel@xxxxxxxxxxxxxxxxxxxxx> <20050415225422.GF4114@xxxxxxxxxxxxxx> <20050416014906.GA3291@xxxxxxxxxxxxxxxxxxx> <20050416110639.GI4114@xxxxxxxxxxxxxx> <20050416111236.GA31550@xxxxxxxxxxxxxxxxxxx>
Sender: netdev-bounce@xxxxxxxxxxx
User-agent: Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv:1.7.6) Gecko/20050324 Debian/1.7.6-1
Herbert Xu wrote:
On Sat, Apr 16, 2005 at 01:06:39PM +0200, Thomas Graf wrote:

qdisc_destroy can still be invoked without qdisc_tree_lock via the
deletion of a class when it calls qdisc_destroy to destroy its
leaf qdisc.

Indeed.  Fortuantely HTB seems to be safe as it calls sch_tree_lock
which is another name for qdisc_tree_lock.  CBQ on the other hand
needs to have a little tweak.

HTB also needs to be fixed. Destruction is usually defered by the
refcnt until ->put(), htb_put() doesn't lock the tree. Same for
HFSC and CBQ.

Regards
Patrick

<Prev in Thread] Current Thread [Next in Thread>