Re: [2.6.11-rc2] kernel BUG at fs/reiserfs/prints.c:362

[Vladimir Saveliev <vs@namesys.com>]

Hello

On Thu, 2005-01-27 at 14:26, Jan Kara wrote:
>   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.
> > > 
> > > rathamahata@dev rathamahata $ mount | grep /dev/sdb2
> > > /dev/sdb2 on /var/www type reiserfs (rw,noatime,nodiratime,data=writeback,grpquota,usrquota)
> > > rathamahata@dev rathamahata $
> > > 
> > > REISERFS: panic (device sdb2): journal_begin called without kernel lock held
> > 
> > Would you check whether this patch helps, please?
>   BTW: What are the exact rules where lock_kernel() should be held for
> reiserfs? Is there a doc somewhere? 
I do not think so.
Earlier reiserfs used to lock_kernel on entering and unlock on exit. The
reason is that reiserfs has no fine grain locking protecting access to
its data structures.
Since that time there could be introduced some minor improvements,
though.

> I suspect we might need the lock
> also for reiserfs_quota_read() (reiserfs_quota_write() should be
> already protected by your fix).
>   Hmm, I should also check ext2/ext3 whether it needs the lock...
> 
> 								Honza