From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932078AbWFFMua (ORCPT ); Tue, 6 Jun 2006 08:50:30 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S932150AbWFFMua (ORCPT ); Tue, 6 Jun 2006 08:50:30 -0400 Received: from hp3.statik.TU-Cottbus.De ([141.43.120.68]:11496 "EHLO hp3.statik.tu-cottbus.de") by vger.kernel.org with ESMTP id S932078AbWFFMu3 (ORCPT ); Tue, 6 Jun 2006 08:50:29 -0400 Message-ID: <4485798B.4030007@s5r6.in-berlin.de> Date: Tue, 06 Jun 2006 14:48:11 +0200 From: Stefan Richter User-Agent: Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.7.12) Gecko/20050915 X-Accept-Language: de, en MIME-Version: 1.0 To: Jiri Slaby CC: Valdis.Kletnieks@vt.edu, Andrew Morton , arjan@infradead.org, mingo@redhat.com, linux-kernel@vger.kernel.org, linux1394-devel@lists.sourceforge.net Subject: Re: 2.6.17-rc5-mm3-lockdep - References: <200606060250.k562oCrA004583@turing-police.cc.vt.edu> <44852819.2080503@gmail.com> In-Reply-To: <44852819.2080503@gmail.com> Content-Type: text/plain; charset=us-ascii Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org X-Mailing-List: linux-kernel@vger.kernel.org Jiri Slaby wrote at lkml: > Valdis.Kletnieks@vt.edu napsal(a): >> It's living longer before it throws a complaint - we're making it out of >> rc.sysinit and into rc5.d ;) This time we were in an 'id' command from this: >> >> test `id -u` = 0 || exit 4 >> >> in either S11mcstrans or S18auditd. Looks like the Firewire (which I >> don't actually use for anything) threw an IRQ at an inopportune time? >> >> (Obviously I stress different code paths than Arjan or Ingo. But if >> I did the same things they did, it wouldn't be interesting.. ;) >> >> [ 464.687000] ( id-2700 |#0): new 569737200 us user-latency. >> [ 464.687000] stopped custom tracer. >> [ 464.687000] >> [ 464.687000] ============================ >> [ 464.687000] [ BUG: illegal lock usage! ] >> [ 464.687000] ---------------------------- >> [ 464.687000] illegal {in-hardirq-W} -> {hardirq-on-W} usage. >> [ 464.687000] id/2700 [HC0[0]:SC0[1]:HE1:SE0] takes: >> [ 464.687000] (&list->lock){++..}, at: [] unix_stream_connect+0x334/0x408 >> [ 464.687000] {in-hardirq-W} state was registered at: >> [ 464.687000] [] lockdep_acquire+0x67/0x7f >> [ 464.687000] [] _spin_lock_irqsave+0x30/0x3f >> [ 464.687000] [] skb_dequeue+0x18/0x49 >> [ 464.687000] [] hpsb_bus_reset+0x5e/0xa2 [ieee1394] >> [ 464.687000] [] ohci_irq_handler+0x370/0x726 [ohci1394] >> [ 464.687000] [] handle_IRQ_event+0x1d/0x52 >> [ 464.687000] [] handle_level_irq+0x97/0xe3 >> [ 464.687000] [] do_IRQ+0x8b/0xaf >> [ 464.687000] irq event stamp: 2964 >> [ 464.687000] hardirqs last enabled at (2963): [] _spin_unlock_irqrestore+0x3b/0x6d >> [ 464.687000] hardirqs last disabled at (2962): [] _spin_lock_irqsave+0x14/0x3f >> [ 464.687000] softirqs last enabled at (2956): [] __do_softirq+0x9d/0xa5 >> [ 464.687000] softirqs last disabled at (2964): [] _spin_lock_bh+0x10/0x3a >> [ 464.687000] >> [ 464.687000] other info that might help us debug this: >> [ 464.687000] 1 locks held by id/2700: >> [ 464.687000] #0: (&u->lock){--..}, at: [] unix_stream_connect+0xe8/0x408 >> [ 464.687000] >> [ 464.687000] stack backtrace: >> [ 464.687000] [] show_trace_log_lvl+0x64/0x125 >> [ 464.687000] [] show_trace+0x1b/0x20 >> [ 464.687000] [] dump_stack+0x1f/0x24 >> [ 464.687000] [] print_usage_bug+0x1a8/0x1b4 >> [ 464.687000] [] mark_lock+0x2ba/0x4e5 >> [ 464.687000] [] __lockdep_acquire+0x476/0xa91 >> [ 464.687000] [] lockdep_acquire+0x67/0x7f >> [ 464.687000] [] _spin_lock_bh+0x2c/0x3a >> [ 464.687000] [] unix_stream_connect+0x334/0x408 >> [ 464.687000] [] sys_connect+0x6e/0xa3 >> [ 464.687000] [] sys_socketcall+0x96/0x190 >> [ 464.687000] [] sysenter_past_esp+0x63/0xa1 >> > That one would be corrected now: > http://lkml.org/lkml/2006/6/5/100 Do you think so? Looks different to me: skb_dequeue is obviously called from ieee1394's packet handling. The patch from 2006/6/5 addresses ieee1394's management of protocol driver callbacks. (added Cc: linux1394-devel) -- Stefan Richter -=====-=-==- -==- ---=- http://arcgraph.de/sr/