Re: 2.6.33-rc3 -- INFO: possible recursive locking -- (s_active){++++.+}, at: [<c10d2941>] sysfs_hash_and_remove+0x3d/0x4f

From: Eric W. Biederman
Date: Wed Jan 13 2010 - 13:38:18 EST

Next message: Joe Perches: "[PATCH] scripts/get_maintainer.pl: Add --sections, print entirematched subsystem"
Previous message: tip-bot for Brian Gerst: "[tip:x86/asm] x86: Merge show_regs()"
In reply to: Eric W. Biederman: "Re: 2.6.33-rc3 -- INFO: possible recursive locking -- (s_active){++++.+}, at: [<c10d2941>] sysfs_hash_and_remove+0x3d/0x4f"
Next in thread: Miles Lane: "Re: 2.6.33-rc3 -- INFO: possible recursive locking -- (s_active){++++.+}, at: [<c10d2941>] sysfs_hash_and_remove+0x3d/0x4f"
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

Miles Lane <miles.lane@xxxxxxxxx> writes:

> On Mon, Jan 11, 2010 at 7:32 PM, Eric W. Biederman <ebiederm@xxxxxxxxxxxx>
> wrote:
>
> Tejun Heo <tj@xxxxxxxxxx> writes:
>
> > On 01/11/2010 11:26 AM, Eric W. Biederman wrote:
> >> From: Eric W. Biederman <ebiederm@xxxxxxxxxxxx>
> >> Date: Sun, 10 Jan 2010 18:13:35 -0800
> >> Subject: [PATCH] sysfs: Add support for lockdep subclasses to s_active
> >>
> >> We have apparently valid cases where the code for a sysfs attribute
> >> removes other sysfs attributes. ÂWithout support for subclasses
> >> lockdep flags a possible recursive lock problem as it figures
> >> the first sysfs attribute could be attempting to remove itself.
> >>
> >> By adding support for sysfs subclasses we can teach lockdep to
> >> distinguish between different types of sysfs attributes and not
> >> get confused.
> >>
> >> Signed-off-by: Eric W. Biederman <ebiederm@xxxxxxxxxxxx>
> >
> > Acked-by: Tejun Heo <tj@xxxxxxxxxx>
>
> Now if I can just get a Tested-by Âthis patch will be all set ;)
>
>
> Not sure if this is related to your patch.Â It doesn't look like it.
> When I tried to suspend, I got the following:

There doesn't appear to be any sysfs involvement in any of those call
traces. Going farther the failure mode of the races I have been trying
to warn about is a silent deadlock. I don't have a clue what would
have triggered this.

Eric

>
> PM: Syncing filesystems ... d? spin_lock_irq+0x8/0xa
> Â [<c105c3ea>] ? lock_acquire+0xa3/0xcd
> Â [<c116eb75>] ? do_raw_spin_lock+0x60/0x122
> Â [<c1041d0a>] ? spin_lock_irq+0x8/0xa
> Â [<c12f6347>] ? _raw_spin_unlock_irq+0x22/0x57
> Â [<c105b09d>] ? trace_hardirqs_on_caller+0x119/0x141
> Â [<c105b0d0>] ? trace_hardirqs_on+0xb/0xd
> Â [<c12f6370>] ? _raw_spin_unlock_irq+0x4b/0x57
> Â [<c1041d14>] ? spin_unlock_irq+0x8/0xa
> Â [<c1041edc>] ? sigprocmask+0xbe/0xc6
> Â [<c10c3b1b>] ? sys_ppoll+0xe1/0xf8
> Â [<c100321c>] ? sysenter_exit+0xf/0x1f
> Â [<c1002cdd>] do_notify_resume+0x23/0x62
> Â [<c12f6a3c>] work_notifysig+0x13/0x1b
> cfg80211ÂÂÂÂÂ S 00000227Â 2956Â 3441ÂÂÂÂÂ 2 0x00000000
> Â f3774f44 00000086 43681739 00000227 f38d15a0 f38d1854 c1536980 f3774f08
> Â c1531ef0 c1536980 f38d1854 c1536980 c105b0d0 f3774f20 c12f6318 f3824a00
> Â 435e6f3a 00000227 f38d15a0 f3774f44 c104b9ff 00000286 00000000 c2c0d4c0
> Call Trace:
> Â [<c105b0d0>] ? trace_hardirqs_on+0xb/0xd
> Â [<c12f6318>] ? _raw_spin_unlock_irqrestore+0x74/0x81
> Â [<c104b9ff>] ? prepare_to_wait+0x4e/0x55
> Â [<c1047e63>] worker_thread+0xa1/0x289
> Â [<f856f4d2>] ? wdev_cleanup_work+0x0/0xa5 [cfg80211]
> Â [<c104b810>] ? autoremove_wake_function+0x0/0x2f
> Â [<c1047dc2>] ? worker_thread+0x0/0x289
> Â [<c104b4aa>] kthread+0x6a/0x6f
> Â [<c104b440>] ? kthread+0x0/0x6f
> Â [<c1003742>] kernel_thread_helper+0x6/0x1a
> hd-audio0ÂÂÂÂ S 00000005Â 3276Â 3478ÂÂÂÂÂ 2 0x00000000
> Â f2e93f44 00000086 4120e090 00000005 f38d2070 f38d2324 c1536980 f2e93f08
> Â c1531ef0 c1536980 f38d2324 c1536980 c105b0d0 fffef111 c2c05ef0 00000000
> Â 41207fac 00000005 f38d2070 f2e93f44 00000000 00000000 00000000 c2c0d5c0
> Call Trace:
> Â [<c105b0d0>] ? trace_hardirqs_on+0xb/0xd
> Â [<c1047e63>] worker_thread+0xa1/0x289
> Â [<c12f6321>] ? _raw_spin_unlock_irqrestore+0x7d/0x81
> Â [<c104b810>] ? autoremove_wake_function+0x0/0x2f
> Â [<c102758b>] ? complete+0x34/0x3e
> Â [<c1047dc2>] ? worker_thread+0x0/0x289
> Â [<c104b4aa>] kthread+0x6a/0x6f
> Â [<c104b440>] ? kthread+0x0/0x6f
> Â [<c1003742>] kernel_thread_helper+0x6/0x1a
> phy0ÂÂÂÂÂÂÂÂÂ S 00000227Â 2396Â 3480ÂÂÂÂÂ 2 0x00000000
> Â f2e81f44 00000086 41ecea10 00000227 f38d0ad0 f38d0d84 c1536980 f2e81f08
> Â c1531ef0 c1536980 f38d0d84 c1536980 c105b0d0 f2e81f20 c12f6318 00000000
> Â 41e8dbc7 00000227 f38d0ad0 f2e81f44 c104b9ff 00000286 00000000 c2c0d640
> Call Trace:
> Â [<c105b0d0>] ? trace_hardirqs_on+0xb/0xd
> Â [<c12f6318>] ? _raw_spin_unlock_irqrestore+0x74/0x81
> Â [<c104b9ff>] ? prepare_to_wait+0x4e/0x55
> Â [<c1047e63>] worker_thread+0xa1/0x289
> Â [<f86a54e7>] ? ieee80211_sta_work+0x0/0xcd6 [mac80211]
> Â [<c104b810>] ? autoremove_wake_function+0x0/0x2f
> Â [<c1047dc2>] ? worker_thread+0x0/0x289
> Â [<c104b4aa>] kthread+0x6a/0x6f
> Â [<c104b440>] ? kthread+0x0/0x6f
> Â [<c1003742>] kernel_thread_helper+0x6/0x1a
> flush-7:0ÂÂÂÂ S 0000022dÂ 1924Â 3574ÂÂÂÂÂ 2 0x00000000
> Â c5e17f14 00000086 9ec1c11b 0000022d 000802be f7166ed4 c1536980 c158cdc0
> Â c1531ef0 c1536980 f7166ed4 c1536980 c12f6318 0007fd57 c2c05ef0 00000000
> Â 9ec12b74 0000022d f7166c20 00000000 00000000 00000000 00000000 c5e17f20
> Call Trace:
> Â [<c12f6318>] ? _raw_spin_unlock_irqrestore+0x74/0x81
> Â [<c12f4b60>] schedule_timeout+0x85/0xba
> Â [<c103eda1>] ? process_timeout+0x0/0xa
> Â [<c12f4bd8>] schedule_timeout_interruptible+0x15/0x17
> Â [<c10ce1dd>] bdi_writeback_task+0x65/0x84
> Â [<c109f684>] ? bdi_start_fn+0x0/0xa4
> Â [<c109f6d2>] bdi_start_fn+0x4e/0xa4
> Â [<c109f684>] ? bdi_start_fn+0x0/0xa4
> Â [<c104b4aa>] kthread+0x6a/0x6f
> Â [<c104b440>] ? kthread+0x0/0x6f
> Â [<c1003742>] kernel_thread_helper+0x6/0x1a
> flush-8:0ÂÂÂÂ S 0000022dÂ 2240Â 3575ÂÂÂÂÂ 2 0x00000000
> Â c59b8f14 00000086 3f62054c 0000022d 0008012e f7166404 c1536980 f70b0000
> Â c1531ef0 c1536980 f7166404 c1536980 c12f6318 0007f038 c2e05ef0 00000000
> Â 3f615a75 0000022d f7166150 00000001 00000000 00000000 00000000 c59b8f20
> Call Trace:
> Â [<c12f6318>] ? _raw_spin_unlock_irqrestore+0x74/0x81
> Â [<c12f4b60>] schedule_timeout+0x85/0xba
> Â [<c103eda1>] ? process_timeout+0x0/0xa
> Â [<c12f4bd8>] schedule_timeout_interruptible+0x15/0x17
> Â [<c10ce1dd>] bdi_writeback_task+0x65/0x84
> Â [<c109f684>] ? bdi_start_fn+0x0/0xa4
> Â [<c109f6d2>] bdi_start_fn+0x4e/0xa4
> Â [<c109f684>] ? bdi_start_fn+0x0/0xa4
> Â [<c104b4aa>] kthread+0x6a/0x6f
> Â [<c104b440>] ? kthread+0x0/0x6f
> Â [<c1003742>] kernel_thread_helper+0x6/0x1a
> ddÂÂÂÂÂÂÂÂÂÂÂ D 00000228Â 1928Â 3597ÂÂÂÂÂ 1 0x00000000
> Â c5a06e5c 00000086 fea63f68 00000228 c1052065 f3ff0d84 c1536980 00000000
> Â c1531ef0 c1536980 f3ff0d84 c1536980 00000086 00000007 f3ff0ad0 f3bdc780
> Â fea60661 00000228 f3ff0ad0 f6512604 00000000 c5a06e50 c105b0d0 f3ff0ad0
> Call Trace:
> Â [<c1052065>] ? refrigerator+0x64/0xbb
> Â [<c105b0d0>] ? trace_hardirqs_on+0xb/0xd
> Â [<c10520a9>] refrigerator+0xa8/0xbb
> Â [<c1042aa8>] try_to_freeze+0x23/0x2a
> Â [<c1044426>] get_signal_to_deliver+0x4c/0x37f
> Â [<c1002649>] do_signal+0x5a/0x6cb
> Â [<c10eae6e>] ? spin_lock+0x8/0xa
> Â [<c105c3ea>] ? lock_acquire+0xa3/0xcd
> Â [<c116eb75>] ? do_raw_spin_lock+0x60/0x122
> Â [<c10eae6e>] ? spin_lock+0x8/0xa
> Â [<c116eb10>] ? do_raw_spin_unlock+0x8f/0x94
> Â [<c12f6298>] ? _raw_spin_unlock+0x33/0x3f
> Â [<c10b6d7a>] ? fsnotify_access+0x4f/0x5a
> Â [<c10eb0e9>] ? proc_reg_read+0x0/0x84
> Â [<c10b763c>] ? vfs_read+0x9d/0xdc
> Â [<c1002cdd>] do_notify_resume+0x23/0x62
> Â [<c12f6a3c>] work_notifysig+0x13/0x1b
> rsyslogdÂÂÂÂÂ D 00000228Â 2036Â 3607ÂÂÂÂÂ 1 0x00000000
> Â c59e2e5c 00000086 fea6deea 00000228 00000039 f7160d84 c1536980 c59e2e18
> Â c1531ef0 c1536980 f7160d84 c1536980 f7160e18 00000007 f7160ad0 f3bdc780
> Â fea657db 00000228 f7160ad0 f38f9584 00000000 c59e2e50 c105b0d0 f7160ad0
> Call Trace:
> Â [<c105b0d0>] ? trace_hardirqs_on+0xb/0xd
> Â [<c10520a9>] refrigerator+0xa8/0xbb
> Â [<c1042aa8>] try_to_freeze+0x23/0x2a
> Â [<c1044426>] get_signal_to_deliver+0x4c/0x37f
> Â [<c1002649>] do_signal+0x5a/0x6cb
> Â [<c105c597>] ? lock_release_non_nested+0x86/0x1d8
> Â [<c10a0fda>] ? might_fault+0x4e/0x88
> Â [<c105c3ea>] ? lock_acquire+0xa3/0xcd
> Â [<c10a0fda>] ? might_fault+0x4e/0x88
> Â [<c10a0fda>] ? might_fault+0x4e/0x88
> Â [<c10a100f>] ? might_fault+0x83/0x88
> Â [<c116bad6>] ? copy_to_user+0x2b/0x57
> Â [<c10c29d7>] ? poll_select_copy_remaining+0xb1/0xcd
> Â [<c10c3660>] ? sys_select+0x7a/0x84
> Â [<c1002cdd>] do_notify_resume+0x23/0x62
> Â [<c12f6a3c>] work_notifysig+0x13/0x1b
> rsyslogdÂÂÂÂÂ D 00000228Â 1988Â 3617ÂÂÂÂÂ 1 0x00000000
> Â c51a9e5c 00000086 fea768ee 00000228 c1052065 f3ff38c4 c1536980 00001719
> Â c1531ef0 c1536980 f3ff38c4 c1536980 c12ff364 0007ea75 c2c05ef0 f3891680
> Â fea768ee 00000228 f3ff3610 00000078 00000000 00000000 00001719 f3ff3610
> Call Trace:
> Â [<c1052065>] ? refrigerator+0x64/0xbb
> Â [<c10520a9>] refrigerator+0xa8/0xbb
> Â [<c1042aa8>] try_to_freeze+0x23/0x2a
> Â [<c1044426>] get_signal_to_deliver+0x4c/0x37f
> Â [<c1002649>] do_signal+0x5a/0x6cb
> Â [<c1027954>] ? cpuacct_charge+0x51/0x58
> Â [<c102df9c>] ? set_next_entity+0x1e/0x89
> Â [<c12f6347>] ? _raw_spin_unlock_irq+0x22/0x57
> Â [<c105b09d>] ? trace_hardirqs_on_caller+0x119/0x141
> Â [<c105b0d0>] ? trace_hardirqs_on+0xb/0xd
> Â [<c1025f06>] ? need_resched+0x14/0x1e
> Â [<c12f4909>] ? schedule+0x808/0x826
> Â [<c10c3660>] ? sys_select+0x7a/0x84
> Â [<c1002cdd>] do_notify_resume+0x23/0x62
> Â [<c12f6a3c>] work_notifysig+0x13/0x1b
> rsyslogdÂÂÂÂÂ D 00000228Â 2844Â 3618ÂÂÂÂÂ 1 0x00000000
> Â c5007e5c 00000086 fed9e135 00000228 c1052065 f3ff2df4 c1536980 00000000
> Â c1531ef0 c1536980 f3ff2df4 c1536980 00000086 00000007 f3ff2b40 f3bdc780
> Â fea6bc0e 00000228 f3ff2b40 f38f9584 00000000 c5007e50 c105b0d0 f3ff2b40
> Call Trace:
> Â [<c1052065>] ? refrigerator+0x64/0xbb
> Â [<c105b0d0>] ? trace_hardirqs_on+0xb/0xd
> Â [<c10520a9>] refrigerator+0xa8/0xbb
> Â [<c1042aa8>] try_to_freeze+0x23/0x2a
> Â [<c1044426>] get_signal_to_deliver+0x4c/0x37f
> Â [<c10bd749>] ? pipe_read+0x290/0x2db
> Â [<c1002649>] do_signal+0x5a/0x6cb
> Â [<c10b701e>] ? do_sync_read+0x87/0xc2
> Â [<c1147456>] ? security_file_permission+0xf/0x11
> Â [<c10b70f1>] ? rw_verify_area+0x98/0xbb
> Â [<c10b6f97>] ? do_sync_read+0x0/0xc2
> Â [<c10b761c>] ? vfs_read+0x7d/0xdc
> Â [<c10b6ca5>] ? fput_light+0xc/0xe
> Â [<c1002cdd>] do_notify_resume+0x23/0x62
> Â [<c12f6a3c>] work_notifysig+0x13/0x1b
> rsyslogdÂÂÂÂÂ D 00000228Â 2392Â 5563ÂÂÂÂÂ 1 0x00000000
> Â c5253e5c 00000086 feda3b6a 00000228 c1052065 e6426ed4 c1536980 00000000
> Â c1531ef0 c1536980 e6426ed4 c1536980 00000086 00000007 e6426c20 f3bdc280
> Â fea6eb78 00000228 e6426c20 f38f9584 00000000 c5253e50 c105b0d0 e6426c20
> Call Trace:
> Â [<c1052065>] ? refrigerator+0x64/0xbb
> Â [<c105b0d0>] ? trace_hardirqs_on+0xb/0xd
> Â [<c10520a9>] refrigerator+0xa8/0xbb
> Â [<c1042aa8>] try_to_freeze+0x23/0x2a
> Â [<c1044426>] get_signal_to_deliver+0x4c/0x37f
> Â [<c1002649>] do_signal+0x5a/0x6cb
> Â [<c105f9fa>] ? do_futex+0x77/0x690
> Â [<c10a0fda>] ? might_fault+0x4e/0x88
> Â [<c105c3ea>] ? lock_acquire+0xa3/0xcd
> Â [<c10a0fda>] ? might_fault+0x4e/0x88
> Â [<c1052e19>] ? timekeeping_get_ns+0x11/0x49
> Â [<c1053a28>] ? ktime_get+0x58/0x95
> Â [<c10600f4>] ? sys_futex+0xe1/0xf5
> Â [<c100321c>] ? sysenter_exit+0xf/0x1f
> Â [<c1002cdd>] do_notify_resume+0x23/0x62
> Â [<c12f6a3c>] work_notifysig+0x13/0x1b
> dbus-daemonÂÂ D 00000228Â 1872Â 3637ÂÂÂÂÂ 1 0x00000000
> Â c5562e5c 00000086 fedcc7a9 00000228 c1052065 f6465934 c1536980 00000000
> Â c1531ef0 c1536980 f6465934 c1536980 00000086 00000007 f6465680 f3bdd400
> Â fea71ea3 00000228 f6465680 f38f8504 00000000 c5562e50 c105b0d0 f6465680
> Call Trace:
> Â [<c1052065>] ? refrigerator+0x64/0xbb
> Â [<c105b0d0>] ? trace_hardirqs_on+0xb/0xd
> Â [<c10520a9>] refrigerator+0xa8/0xbb
> Â [<c1042aa8>] try_to_freeze+0x23/0x2a
> Â [<c1044426>] get_signal_to_deliver+0x4c/0x37f
> Â [<c1002649>] do_signal+0x5a/0x6cb
> Â [<c1002cdd>] do_notify_resume+0x23/0x62
> Â [<c12f6a3c>] work_notifysig+0x13/0x1b
> gdm-binaryÂÂÂ D 00000228Â 2036Â 3641ÂÂÂÂÂ 1 0x00000000
> Â c5005e5c 00000086 fedd34de 00000228 c1052065 f3ff4e64 c1536980 00000000
> Â c1531ef0 c1536980 f3ff4e64 c1536980 00000086 00000007 f3ff4bb0 f3825180
> Â fea7dfdb 00000228 f3ff4bb0 f6513104 00000000 c5005e50 c105b0d0 f3ff4bb0
> Call Trace:
> Â [<c1052065>] ? refrigerator+0x64/0xbb
> Â [<c105b0d0>] ? trace_hardirqs_on+0xb/0xd
> Â [<c10520a9>] refrigerator+0xa8/0xbb
> Â [<c1042aa8>] try_to_freeze+0x23/0x2a
> Â [<c1044426>] get_signal_to_deliver+0x4c/0x37f
> Â [<c1002649>] do_signal+0x5a/0x6cb
> Â [<c104b87e>] ? spin_unlock_irqrestore+0x8/0xa
> Â [<c104b927>] ? finish_wait+0x44/0x49
> Â [<c105c597>] ? lock_release_non_nested+0x86/0x1d8
> Â [<c10a0fda>] ? might_fault+0x4e/0x88
> Â [<c105c3ea>] ? lock_acquire+0xa3/0xcd
> Â [<c10a0fda>] ? might_fault+0x4e/0x88
> Â [<c10a0fda>] ? might_fault+0x4e/0x88
> Â [<c10a100f>] ? might_fault+0x83/0x88
> Â [<c1002cdd>] do_notify_resume+0x23/0x62
> Â [<c12f6a3c>] work_notifysig+0x13/0x1b
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/

Next message: Joe Perches: "[PATCH] scripts/get_maintainer.pl: Add --sections, print entirematched subsystem"
Previous message: tip-bot for Brian Gerst: "[tip:x86/asm] x86: Merge show_regs()"
In reply to: Eric W. Biederman: "Re: 2.6.33-rc3 -- INFO: possible recursive locking -- (s_active){++++.+}, at: [<c10d2941>] sysfs_hash_and_remove+0x3d/0x4f"
Next in thread: Miles Lane: "Re: 2.6.33-rc3 -- INFO: possible recursive locking -- (s_active){++++.+}, at: [<c10d2941>] sysfs_hash_and_remove+0x3d/0x4f"
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]