Hello
On Thu, 2005-01-27 at 10:24, Sergey S. Kostyliov wrote:
> Hello all,
>
> Here is a BUG() I've just hited on quota enabled reiserfs disk.
>
> [EMAIL PROTECTED] rathamahata $ mount | grep /dev/sdb2
> /dev/sdb2 on /var/www type reiserfs
> (rw,noatime,nodiratime,data=writeback,grpquota,usrquota)
> [EMAIL PROTECTED] rathamahata $
>
> REISERFS: panic (device sdb2): journal_begin called without kernel lock held
Would you check whether this patch helps, please?
> ------------[ cut here ]------------
> kernel BUG at fs/reiserfs/prints.c:362!
> invalid operand: 0000 [#1]
> PREEMPT SMP
> Modules linked in:
> CPU: 0
> EIP: 0060:[<c019e9fb>] Not tainted VLI
> EFLAGS: 00010296 (2.6.11-rc2)
> EIP is at reiserfs_panic+0x4b/0x80
> eax: 00000050 ebx: c02b75b2 ecx: f7fe4270 edx: f1034e38
> esi: f7543600 edi: f7543770 ebp: 000000f2 esp: f1034e34
> ds: 007b es: 007b ss: 0068
> Process qmail-local (pid: 10803, threadinfo=f1034000 task=cc440a60)
> Stack: c02bbb00 f7543770 c03b89c0 f7543600 f1034ecc f89af000 c01a8a0e f7543600
> c02bcf20 c02b7db8 c01ac186 f74b66c0 000000f2 f1034ecc c0167403 f74b6900
> c02f2740 fffffff4 f74b6960 00295e98 c01679dc 00295e98 f74b6090 f74b66c0
> Call Trace:
> [<c01a8a0e>] reiserfs_check_lock_depth+0x2e/0x30
> [<c01ac186>] do_journal_begin_r+0x26/0x2d0
> [<c0167403>] d_alloc+0x133/0x180
> [<c01679dc>] __d_lookup+0x11c/0x130
> [<c01ac641>] journal_begin+0x61/0xf0
> [<c019d445>] reiserfs_dquot_initialize+0x25/0x60
> [<c015db2d>] link_path_walk+0x48d/0xd20
> [<c015d1c6>] permission+0x76/0xa0
> [<c015ea98>] vfs_create+0xc8/0x110
> [<c015f2cf>] open_namei+0x57f/0x5d0
> [<c014fcfd>] filp_open+0x2d/0x60
> [<c0150014>] get_unused_fd+0x94/0xc0
> [<c015cff7>] getname+0x67/0xb0
> [<c015012c>] sys_open+0x3c/0x80
> [<c01026b1>] sysenter_past_esp+0x52/0x75
> Code: 8d be 70 01 00 00 e8 a5 fd ff ff c7 04 24 00 bb 2b c0 85 f6 89 d8 0f 45
> c7 ba c0 893b c0 89 54 24 08 89 44 24 04 e8 25 9d f7 ff <0f> 0b 6a 01 1a 7b
> 2b c0 c7 04 24 40 bb 2b c0 85 f6 be c0 89 3b
> <0>REISERFS: panic (device sdb2): journal_begin called without kernel lock
> held
> ------------[ cut here ]------------
fs/reiserfs/super.c | 12 ++++++++++++
1 files changed, 12 insertions(+)
diff -puN fs/reiserfs/super.c~reiserfs-quota-add-missing-lock_kernel
fs/reiserfs/super.c
--- linux-2.6.11-rc2/fs/reiserfs/super.c~reiserfs-quota-add-missing-lock_kernel
2005-01-27 13:09:35.101322376 +0300
+++ linux-2.6.11-rc2-root/fs/reiserfs/super.c 2005-01-27 13:15:19.608949256
+0300
@@ -1835,9 +1835,11 @@ static int reiserfs_dquot_initialize(str
int ret;
/* We may create quota structure so we need to reserve enough blocks */
+ reiserfs_write_lock(inode->i_sb);
journal_begin(&th, inode->i_sb, 2*REISERFS_QUOTA_INIT_BLOCKS);
ret = dquot_initialize(inode, type);
journal_end(&th, inode->i_sb, 2*REISERFS_QUOTA_INIT_BLOCKS);
+ reiserfs_write_unlock(inode->i_sb);
return ret;
}
@@ -1847,9 +1849,11 @@ static int reiserfs_dquot_drop(struct in
int ret;
/* We may delete quota structure so we need to reserve enough blocks */
+ reiserfs_write_lock(inode->i_sb);
journal_begin(&th, inode->i_sb, 2*REISERFS_QUOTA_INIT_BLOCKS);
ret = dquot_drop(inode);
journal_end(&th, inode->i_sb, 2*REISERFS_QUOTA_INIT_BLOCKS);
+ reiserfs_write_unlock(inode->i_sb);
return ret;
}
@@ -1858,9 +1862,11 @@ static int reiserfs_write_dquot(struct d
struct reiserfs_transaction_handle th;
int ret;
+ reiserfs_write_lock(dquot->dq_sb);
journal_begin(&th, dquot->dq_sb, REISERFS_QUOTA_TRANS_BLOCKS);
ret = dquot_commit(dquot);
journal_end(&th, dquot->dq_sb, REISERFS_QUOTA_TRANS_BLOCKS);
+ reiserfs_write_unlock(dquot->dq_sb);
return ret;
}
@@ -1869,9 +1875,11 @@ static int reiserfs_acquire_dquot(struct
struct reiserfs_transaction_handle th;
int ret;
+ reiserfs_write_lock(dquot->dq_sb);
journal_begin(&th, dquot->dq_sb, REISERFS_QUOTA_INIT_BLOCKS);
ret = dquot_acquire(dquot);
journal_end(&th, dquot->dq_sb, REISERFS_QUOTA_INIT_BLOCKS);
+ reiserfs_write_unlock(dquot->dq_sb);
return ret;
}
@@ -1880,9 +1888,11 @@ static int reiserfs_release_dquot(struct
struct reiserfs_transaction_handle th;
int ret;
+ reiserfs_write_lock(dquot->dq_sb);
journal_begin(&th, dquot->dq_sb, REISERFS_QUOTA_INIT_BLOCKS);
ret = dquot_release(dquot);
journal_end(&th, dquot->dq_sb, REISERFS_QUOTA_INIT_BLOCKS);
+ reiserfs_write_unlock(dquot->dq_sb);
return ret;
}
@@ -1904,9 +1914,11 @@ static int reiserfs_write_info(struct su
int ret;
/* Data block + inode block */
+ reiserfs_write_lock(sb);
journal_begin(&th, sb, 2);
ret = dquot_commit_info(sb, type);
journal_end(&th, sb, 2);
+ reiserfs_write_unlock(sb);
return ret;
}
_
