| To: | Patrick McHardy <kaber@xxxxxxxxx> |
|---|---|
| Subject: | Re: [PATCH] rtnl_unlock/lock in sch_api.c |
| From: | "Catalin(ux aka Dino) BOIE" <util@xxxxxxxxxxxxxxx> |
| Date: | Tue, 29 Mar 2005 01:16:52 +0300 (EEST) |
| Cc: | netdev@xxxxxxxxxxx, davem@xxxxxxxxxxxxx |
| In-reply-to: | <424819B8.8000201@xxxxxxxxx> |
| References: | <Pine.LNX.4.62.0503281720430.20453@xxxxxxxxxxxxxxxxxxx> <424819B8.8000201@xxxxxxxxx> |
| Sender: | netdev-bounce@xxxxxxxxxxx |
On Mon, 28 Mar 2005, Patrick McHardy wrote: Catalin(ux aka Dino) BOIE wrote:Hello! Trying to load a custom module (same for teql but I didn't tried it) whan the qdisc module is not loaded, makes tc hang.This is because qdisc_create aquires rtnl_sem and then tries to load a module that tries to register_netdev (that tries to aquire the same rtnl_sem).Applying this patch makes the problem go away.You open a race by dropping the lock and not replaying the request after acquiring it again. This is Dave's original patch, please simply fix this one up so it applies again. Regards Patrick Patrick, What to fix in Dave's patch? Seems ok to me. Thanks! --- Catalin(ux aka Dino) BOIE catab at deuroconsult.ro http://kernel.umbrella.ro/ |
| <Prev in Thread] | Current Thread | [Next in Thread> |
|---|---|---|
| ||
| Previous by Date: | Re: gcc confused about static inline. - route.c, Francois Romieu |
|---|---|
| Next by Date: | Re: [PATCH] rtnl_unlock/lock in sch_api.c, Patrick McHardy |
| Previous by Thread: | Re: [PATCH] rtnl_unlock/lock in sch_api.c, Patrick McHardy |
| Next by Thread: | Re: [PATCH] rtnl_unlock/lock in sch_api.c, Patrick McHardy |
| Indexes: | [Date] [Thread] [Top] [All Lists] |