On Mon, 2006-07-31 at 14:02 -0700, David Miller wrote: > From: Dave Jones <[EMAIL PROTECTED]> > Date: Mon, 31 Jul 2006 16:50:04 -0400 > > > 2.6.18rc2-gitSomething on my firewall box just triggered this.. > > Lockdep is perhaps confused. > > > [515613.904945] swapper/0 is trying to acquire lock: > > [515613.931489] (&tbl->lock){-+-+}, at: [<c05b5d63>] neigh_lookup+0x50/0xaf > > [515613.964369] > > [515613.964373] but task is already holding lock: > > [515614.006550] (&skb_queue_lock_key){-+..}, at: [<c05b741c>] > > neigh_proxy_process+0x20/0xc2 > > The skb_queue_lock in question is &tbl->proxy_queue.lock > > > [515614.103459] the existing dependency chain (in reverse order) is: > > [515614.148752] > > [515614.148755] -> #2 (&skb_queue_lock_key){-+..}: > > [515614.188880] [<c043bf43>] lock_acquire+0x4b/0x6c > > [515614.215554] [<c06089a7>] _spin_lock_irqsave+0x22/0x32 > > [515614.243606] [<c05ac2e3>] skb_dequeue+0x12/0x43 > > [515614.269657] [<c05acffe>] skb_queue_purge+0x14/0x1b > > [515614.296565] [<c05b673e>] neigh_update+0x317/0x353 > > This is a different queue lock, namely &neigh->arp_queue.lock > > Like the ipv6 trace we got yesterday from Matt Domsche, lockdep > is aparently confusing two instances of the skb_queue_lock_key
we fixed lockdep to have this lock key to be per skb queue ... didn't you put that patch in rawhide Dave (J) ? - To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html