| To: | "David S. Miller" <davem@xxxxxxxxxx> |
|---|---|
| Subject: | Re: [PATCH 2.6 1/5]: Fix locking in __qdisc_destroy rcu-callback |
| From: | Patrick McHardy <kaber@xxxxxxxxx> |
| Date: | Wed, 04 Aug 2004 21:53:28 +0200 |
| Cc: | netdev@xxxxxxxxxxx |
| In-reply-to: | <20040804093328.4ee56d30.davem@xxxxxxxxxx> |
| References: | <410FAD44.7020503@xxxxxxxxx> <20040804093328.4ee56d30.davem@xxxxxxxxxx> |
| Sender: | netdev-bounce@xxxxxxxxxxx |
| User-agent: | Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.6) Gecko/20040413 Debian/1.6-5 |
David S. Miller wrote: On Tue, 03 Aug 2004 17:20:36 +0200 Patrick McHardy <kaber@xxxxxxxxx> wrote:I'm not sure if RCU callback can be scheduled while the kernel is running in process context, so this may be unneccessary.RCU callbacks run via tasklets, and thus in softirq context, thus your locking changes to _bh() are indeed necessary. I meant because in my understanding of RCU the quintescent state is not reached while the CPU is running kernel code, so the callback can't be scheduled. A previously scheduled callback should finish before further kernel code is run in process context. Regards Patrick |
| <Prev in Thread] | Current Thread | [Next in Thread> |
|---|---|---|
| ||
| Previous by Date: | Re: [PATCH] ibmtr: fix init (devinit) section usage, Randy.Dunlap |
|---|---|
| Next by Date: | Re: [PATCH 2.6 3/5]: Use double-linked list for dev->qdisc_list, Patrick McHardy |
| Previous by Thread: | Re: [PATCH 2.6 1/5]: Fix locking in __qdisc_destroy rcu-callback, David S. Miller |
| Next by Thread: | [PATCH 2.6] (1/3) add ethtool ioctl forwarding to vlan, Stephen Hemminger |
| Indexes: | [Date] [Thread] [Top] [All Lists] |