[PATCH] Re:[2.6.19-rc4] "possible recursive locking detected" inreiserfs_xattr_set
From: Srinivasa Ds
Date: Mon Nov 06 2006 - 04:12:02 EST
Tilman Schmidt wrote:
JFTR, these messages still appear with 2.6.19-rc4:
Nov 3 11:08:05 gx110 kernel: [ 1025.512025]
Nov 3 11:08:05 gx110 kernel: [ 1025.512031] =============================================
Nov 3 11:08:05 gx110 kernel: [ 1025.512053] [ INFO: possible recursive locking detected ]
Nov 3 11:08:05 gx110 kernel: [ 1025.512064] 2.6.19-rc4-noinitrd #1
Nov 3 11:08:05 gx110 kernel: [ 1025.512071] ---------------------------------------------
Nov 3 11:08:05 gx110 kernel: [ 1025.512080] kdm/3234 is trying to acquire lock:
Nov 3 11:08:05 gx110 kernel: [ 1025.512089] (&inode->i_mutex){--..}, at: [<c035aced>] mutex_lock+0x1c/0x1f
Nov 3 11:08:05 gx110 kernel: [ 1025.512124]
Nov 3 11:08:05 gx110 kernel: [ 1025.512126] but task is already holding lock:
Nov 3 11:08:05 gx110 kernel: [ 1025.512134] (&inode->i_mutex){--..}, at: [<c035aced>] mutex_lock+0x1c/0x1f
Nov 3 11:08:05 gx110 kernel: [ 1025.512152]
Nov 3 11:08:05 gx110 kernel: [ 1025.512155] other info that might help us debug this:
Nov 3 11:08:05 gx110 kernel: [ 1025.512165] 3 locks held by kdm/3234:
Nov 3 11:08:05 gx110 kernel: [ 1025.512172] #0: (&inode->i_mutex){--..}, at: [<c035aced>] mutex_lock+0x1c/0x1f
Nov 3 11:08:05 gx110 kernel: [ 1025.512192] #1: (&REISERFS_I(inode)->xattr_sem){----}, at: [<c01c3951>] reiserfs_acl_chmod+0xe1/0x180
Nov 3 11:08:05 gx110 kernel: [ 1025.512234] #2: (&REISERFS_SB(s)->xattr_dir_sem){----}, at: [<c01c3986>] reiserfs_acl_chmod+0x116/0x180
Nov 3 11:08:05 gx110 kernel: [ 1025.512257]
Nov 3 11:08:05 gx110 kernel: [ 1025.512260] stack backtrace:
Nov 3 11:08:05 gx110 kernel: [ 1025.512274] [<c0103dc4>] dump_trace+0x64/0x1cc
Nov 3 11:08:05 gx110 kernel: [ 1025.512300] [<c0103f45>] show_trace_log_lvl+0x19/0x2e
Nov 3 11:08:05 gx110 kernel: [ 1025.512317] [<c01042a2>] show_trace+0x12/0x14
Nov 3 11:08:05 gx110 kernel: [ 1025.512334] [<c01042bb>] dump_stack+0x17/0x19
Nov 3 11:08:05 gx110 kernel: [ 1025.512348] [<c012fdd9>] __lock_acquire+0x106/0x99c
Nov 3 11:08:05 gx110 kernel: [ 1025.512373] [<c0130930>] lock_acquire+0x5b/0x7b
Nov 3 11:08:05 gx110 kernel: [ 1025.512457] [<c035ab5d>] __mutex_lock_slowpath+0xc6/0x23a
Nov 3 11:08:05 gx110 kernel: [ 1025.512482] [<c035aced>] mutex_lock+0x1c/0x1f
Nov 3 11:08:05 gx110 kernel: [ 1025.512504] [<c01c29c5>] reiserfs_xattr_set+0xe4/0x2bf
Nov 3 11:08:05 gx110 kernel: [ 1025.512529] [<c01c33ef>] reiserfs_set_acl+0x18d/0x204
Nov 3 11:08:05 gx110 kernel: [ 1025.512553] [<c01c3994>] reiserfs_acl_chmod+0x124/0x180
Nov 3 11:08:05 gx110 kernel: [ 1025.512577] [<c01a3c41>] reiserfs_setattr+0x20b/0x243
Nov 3 11:08:05 gx110 kernel: [ 1025.512608] [<c017395b>] notify_change+0x135/0x2c2
Nov 3 11:08:05 gx110 kernel: [ 1025.512645] [<c015fbba>] sys_fchmodat+0xa0/0xca
Nov 3 11:08:05 gx110 kernel: [ 1025.512670] [<c015fc05>] sys_chmod+0x21/0x23
Nov 3 11:08:05 gx110 kernel: [ 1025.512691] [<c0102dfd>] sysenter_past_esp+0x56/0x8d
Nov 3 11:08:05 gx110 kernel: [ 1025.512718] DWARF2 unwinder stuck at sysenter_past_esp+0x56/0x8d
Nov 3 11:08:05 gx110 kernel: [ 1025.512733]
Nov 3 11:08:05 gx110 kernel: [ 1025.512746] Leftover inexact backtrace:
Nov 3 11:08:05 gx110 kernel: [ 1025.512750]
Nov 3 11:08:05 gx110 kernel: [ 1025.512771] =======================
The message appears when the first KDE session is started.
This patch should solve your problem. Please let me know your comments
for this patch.
Signed-off-by: Srinivasa DS <srinivasa@xxxxxxxxxx>
open.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
Index: linux-2.6.19-rc4/fs/open.c
===================================================================
--- linux-2.6.19-rc4.orig/fs/open.c 2006-10-31 09:07:36.000000000 +0530
+++ linux-2.6.19-rc4/fs/open.c 2006-11-06 14:28:19.000000000 +0530
@@ -549,7 +549,7 @@
if (IS_IMMUTABLE(inode) || IS_APPEND(inode))
goto dput_and_out;
- mutex_lock(&inode->i_mutex);
+ mutex_lock_nested(&inode->i_mutex, I_MUTEX_PARENT);
if (mode == (mode_t) -1)
mode = inode->i_mode;
newattrs.ia_mode = (mode & S_IALLUGO) | (inode->i_mode & ~S_IALLUGO);