From mboxrd@z Thu Jan 1 00:00:00 1970 From: Eric Dumazet Subject: Re: UDP/bnx2: inconsistent lock state (Re: linux-next: Tree for October 30) Date: Thu, 30 Oct 2008 14:22:30 +0100 Message-ID: <4909B516.30406@cosmosbay.com> References: Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: QUOTED-PRINTABLE Cc: mchan@broadcom.com, linux-next@vger.kernel.org, Netdev To: Alexander Beregalov Return-path: Received: from gw1.cosmosbay.com ([86.65.150.130]:48032 "EHLO gw1.cosmosbay.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753800AbYJ3NWn convert rfc822-to-8bit (ORCPT ); Thu, 30 Oct 2008 09:22:43 -0400 In-Reply-To: Sender: netdev-owner@vger.kernel.org List-ID: Alexander Beregalov a =C3=A9crit : > 2008/10/30 Alexander Beregalov : >> bnx2 0000:03:00.0: irq 38 for MSI/MSI-X >> bnx2: eth0: using MSI >> bnx2: eth0 NIC Copper Link is Up, 1000 Mbps full duplex, receive & >> transmit flow control ON >> >> [ INFO: inconsistent lock state ] >> 2.6.28-rc2-next-20081030 #4 >> --------------------------------- >> inconsistent {softirq-on-W} -> {in-softirq-W} usage. >> swapper/0 [HC0[0]:SC1[1]:HE1:SE0] takes: >> (&table->hash[i].lock){-+..}, at: [] >> __udp4_lib_rcv+0x2a5/0x640 >> {softirq-on-W} state was registered at: >> [] __lock_acquire+0x2fd/0xc72 >> [] lock_acquire+0x85/0xa9 >> [] _spin_lock+0x3c/0x70 >> [] udp_lib_unhash+0x2b/0x90 >> [] sk_common_release+0x2f/0x83 >> [] udp_lib_close+0x9/0xb >> [] inet_release+0x58/0x5f >> [] sock_release+0x20/0xc1 >> [] sock_close+0x34/0x3a >> [] __fput+0xc5/0x164 >> [] fput+0x15/0x17 >> [] filp_close+0x67/0x72 >> [] put_files_struct+0x74/0xc8 >> [] exit_files+0x47/0x4f >> [] do_exit+0x27a/0x8b7 >> [] do_group_exit+0x7f/0xaf >> [] sys_exit_group+0x12/0x14 >> [] system_call_fastpath+0x16/0x1b >> [] 0xffffffffffffffff >> irq event stamp: 56806 >> hardirqs last enabled at (56806): [] >> _local_bh_enable_ip+0xba/0xe3 >> hardirqs last disabled at (56805): [] >> _local_bh_enable_ip+0x54/0xe3 >> softirqs last enabled at (56786): [] __do_softirq= +0xf8/0x104 >> softirqs last disabled at (56791): [] call_softirq= +0x1c/0x28 >> >> other info that might help us debug this: >> no locks held by swapper/0. >> >> stack backtrace: >> Pid: 0, comm: swapper Not tainted 2.6.28-rc2-next-20081030 #4 >> Call Trace: >> [] valid_state+0x179/0x18c >> [] mark_lock+0x14d/0x37a >> [] __lock_acquire+0x282/0xc72 >> [] ? rt_intern_hash+0x442/0x45c >> [] ? local_bh_enable_ip+0x9/0xb >> [] lock_acquire+0x85/0xa9 >> [] ? __udp4_lib_rcv+0x2a5/0x640 >> [] _spin_lock+0x3c/0x70 >> [] ? __udp4_lib_rcv+0x2a5/0x640 >> [] __udp4_lib_rcv+0x2a5/0x640 >> [] udp_rcv+0x15/0x17 >> [] ip_local_deliver+0xad/0x13d >> [] ip_rcv+0x4b7/0x4ef >> [] netif_receive_skb+0x213/0x23a >> [] bnx2_poll_work+0x92e/0xaa5 >> [] ? getnstimeofday+0x3a/0x96 >> [] ? do_settimeofday+0x7f/0x131 >> [] ? update_wall_time+0x38e/0x40f >> [] ? mark_lock+0x1c/0x37a >> [] ? __rcu_read_lock+0xa0/0xaf >> [] bnx2_poll+0x128/0x20d >> [] net_rx_action+0xd2/0x1f4 >> [] __do_softirq+0x70/0x104 >> [] call_softirq+0x1c/0x28 >> [] do_softirq+0x39/0x8a >> [] irq_exit+0x45/0xa2 >> [] do_IRQ+0x16a/0x19c >> [] ret_from_intr+0x0/0xf >> [] ? mwait_idle+0x3e/0x48 >> [] ? mwait_idle+0x35/0x48 >> [] ? cpu_idle+0x59/0xc2 >> [] ? start_secondary+0x263/0x26b >> >=20 > Similar issue on sparc/sunhme >=20 > Yes, its a generic problem, I replied to your previous mail with a prop= er fix Thank you