From: Minchan Kim <minchan.kim@gmail.com>
To: linux kernel mailing list <linux-kernel@vger.kernel.org>
Cc: Greg Kroah-Hartman <gregkh@suse.de>,
Alan Stern <stern@rowland.harvard.edu>,
Andrew Morton <akpm@linux-foundation.org>
Subject: [BUG] usb_set_device_state
Date: Sun, 6 Feb 2011 02:15:04 +0900 [thread overview]
Message-ID: <20110205171504.GA1044@barrios-desktop> (raw)
I got following message during booting mmotm-02-04.
sky2 0000:04:00.0: PCI INT A -> GSI 16 (level, low) -> IRQ 16
sky2 0000:04:00.0: setting latency timer to 64
sky2 0000:04:00.0: Yukon-2 EC Ultra chip revision 2
sky2 0000:04:00.0: irq 44 for MSI/MSI-X
sky2 0000:04:00.0: eth0: addr 00:1a:4d:65:01:51
BUG: sleeping function called from invalid context at kernel/mutex.c:278
in_atomic(): 0, irqs_disabled(): 1, pid: 21, name: khubd
3 locks held by khubd/21:
#0: (&__lockdep_no_validate__){+.+.+.}, at: [<c047e2dd>] hub_thread+0xcd/0xf10
#1: (&__lockdep_no_validate__){+.+.+.}, at: [<c0419a82>] device_attach+0x22/0x90
#2: (device_state_lock){......}, at: [<c047c4c4>] usb_set_device_state+0x24/0xf0
irq event stamp: 6264
hardirqs last enabled at (6263): [<c05d7adf>] __mutex_unlock_slowpath+0xaf/0x140
hardirqs last disabled at (6264): [<c05d9196>] _raw_spin_lock_irqsave+0x26/0x70
softirqs last enabled at (6252): [<c015d137>] del_timer_sync+0x57/0xa0
softirqs last disabled at (6250): [<c015d0f4>] del_timer_sync+0x14/0xa0
Pid: 21, comm: khubd Not tainted 2.6.38-rc3-mm1+ #1
Call Trace:
[<c015d0f4>] ? del_timer_sync+0x14/0xa0
[<c013cc29>] ? __might_sleep+0xe9/0x120
[<c05d80f6>] ? mutex_lock_nested+0x26/0x50
[<c038272f>] ? do_raw_spin_lock+0x3f/0x140
[<c0289b98>] ? sysfs_get_dirent+0x28/0x70
[<c028b3f1>] ? sysfs_merge_group+0x21/0xc0
[<c041dd75>] ? wakeup_sysfs_add+0x15/0x20
[<c041fdf5>] ? device_set_wakeup_capable+0x45/0x80
[<c047c578>] ? usb_set_device_state+0xd8/0xf0
[<c0484556>] ? usb_set_configuration+0x366/0x680
[<c028a381>] ? sysfs_do_create_link+0xb1/0x1e0
[<c048c93b>] ? generic_probe+0x3b/0x90
[<c028a4e7>] ? sysfs_create_link+0x17/0x20
[<c0485849>] ? usb_probe_device+0x29/0x50
[<c0419822>] ? driver_probe_device+0x82/0x190
[<c05d98a2>] ? _raw_spin_unlock+0x22/0x30
[<c0419a09>] ? __device_attach+0x49/0x60
[<c0418af3>] ? bus_for_each_drv+0x53/0x80
[<c0419ada>] ? device_attach+0x7a/0x90
[<c04199c0>] ? __device_attach+0x0/0x60
[<c0418915>] ? bus_probe_device+0x25/0x40
[<c04174c5>] ? device_add+0x5a5/0x670
[<c047c100>] ? usb_hub_init+0xa0/0xb0
[<c042182c>] ? pm_runtime_forbid+0x4c/0x60
[<c047dee0>] ? usb_new_device+0xe0/0x120
[<c018336b>] ? trace_hardirqs_on+0xb/0x10
[<c047e904>] ? hub_thread+0x6f4/0xf10
[<c016d7a0>] ? autoremove_wake_function+0x0/0x50
[<c047e210>] ? hub_thread+0x0/0xf10
[<c016d384>] ? kthread+0x84/0x90
[<c016d300>] ? kthread+0x0/0x90
[<c0103606>] ? kernel_thread_helper+0x6/0x10
-
It seems usb_set_device_state disable irq by spin_lock_irqsave.
But it calls device_set_wakeup_capable which ends up calling sysfs_merge_group.
It calls mutex_lock. :(
Is it already known BUG?
--
Kind regards,
Minchan Kim
next reply other threads:[~2011-02-05 17:15 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-02-05 17:15 Minchan Kim [this message]
2011-02-05 18:25 ` [BUG] usb_set_device_state Greg KH
2011-02-06 13:02 ` Minchan Kim
2011-02-05 20:11 ` Alan Stern
2011-02-05 23:06 ` Rafael J. Wysocki
2011-02-06 3:19 ` Alan Stern
2011-02-06 12:01 ` Rafael J. Wysocki
2011-02-06 12:03 ` Rafael J. Wysocki
2011-02-06 12:53 ` Minchan Kim
2011-02-06 13:01 ` Minchan Kim
2011-02-06 13:08 ` Rafael J. Wysocki
2011-02-06 15:23 ` Alan Stern
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20110205171504.GA1044@barrios-desktop \
--to=minchan.kim@gmail.com \
--cc=akpm@linux-foundation.org \
--cc=gregkh@suse.de \
--cc=linux-kernel@vger.kernel.org \
--cc=stern@rowland.harvard.edu \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.