| To: | Patrick McHardy <kaber@xxxxxxxxx> |
|---|---|
| Subject: | Re: PROBLEM: IProute hangs after running traffic shaping scripts |
| From: | Thomas Graf <tgraf@xxxxxxx> |
| Date: | Mon, 8 Nov 2004 14:54:31 +0100 |
| Cc: | Szymon Miotk <spam@xxxxxxxxxxxxx>, netdev@xxxxxxxxxxx |
| In-reply-to: | <418ECE85.9090203@xxxxxxxxx> |
| References: | <418B4C7C.8000402@xxxxxxxxxxxxx> <418EA032.7050507@xxxxxxxxx> <418ECE85.9090203@xxxxxxxxx> |
| Sender: | netdev-bounce@xxxxxxxxxxx |
> + /* unlink inner qdiscs from dev->qdisc_list immediately */ > + if (qdisc->ops->cl_ops != NULL) > + list_add(&qdisc->list, &cql); I think you should extend the above to: if (qdisc->ops->cl_ops != NULL && !list_empty(&qdisc->list)) Otherwise you might unlink entries just added before the rcu callback and it prevents an unneeded attempt to delete qdiscs for inner classful qdiscs Otherwise the patch looks very good. |
| Previous by Date: | Re: Bug in ipv6_ifa_notify?, YOSHIFUJI Hideaki / 吉藤英明 |
|---|---|
| Next by Date: | Re: linux 2.6.9: r8169: eth0: PCI error (status: 0x8404). Device disabled., Eamonn Hamilton |
| Previous by Thread: | Re: PROBLEM: IProute hangs after running traffic shaping scripts, Patrick McHardy |
| Next by Thread: | Re: PROBLEM: IProute hangs after running traffic shaping scripts, Patrick McHardy |
| Indexes: | [Date] [Thread] [Top] [All Lists] |