Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S272591AbTHEI2O (ORCPT ); Tue, 5 Aug 2003 04:28:14 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S272592AbTHEI2O (ORCPT ); Tue, 5 Aug 2003 04:28:14 -0400 Received: from angband.namesys.com ([212.16.7.85]:41119 "EHLO angband.namesys.com") by vger.kernel.org with ESMTP id S272591AbTHEI2I (ORCPT ); Tue, 5 Aug 2003 04:28:08 -0400 Date: Tue, 5 Aug 2003 12:28:07 +0400 From: Oleg Drokin To: Andrey Borzenkov Cc: linux-kernel@vger.kernel.org Subject: Re: 2.6.0-test2: resiserfs BUG on Alt-SysRq-U Message-ID: <20030805082807.GB14521@namesys.com> References: <200308042056.15413.arvidjaar@mail.ru> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <200308042056.15413.arvidjaar@mail.ru> User-Agent: Mutt/1.4i Sender: linux-kernel-owner@vger.kernel.org X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2105 Lines: 76 Hello! On Mon, Aug 04, 2003 at 08:56:15PM +0400, Andrey Borzenkov wrote: > this has been around since 2.5.75 at least and may be before it as well. > kernel BUG at fs/reiserfs/journal.c:409! Hm, indeed. So they are calling ->remount() without lock_kernel these days. The patch below should help, please verify. Thank you. Bye, Oleg ===== fs/reiserfs/super.c 1.66 vs edited ===== --- 1.66/fs/reiserfs/super.c Sat Jun 21 00:16:06 2003 +++ edited/fs/reiserfs/super.c Tue Aug 5 12:22:10 2003 @@ -761,6 +761,7 @@ if (!reiserfs_parse_options(s, arg, &mount_options, &blocks, NULL)) return -EINVAL; + reiserfs_write_lock(s); handle_attrs(s); /* Add options that are safe here */ @@ -778,17 +779,22 @@ if(blocks) { int rc = reiserfs_resize(s, blocks); - if (rc != 0) + if (rc != 0) { + reiserfs_write_unlock(s); return rc; + } } if (*mount_flags & MS_RDONLY) { /* remount read-only */ - if (s->s_flags & MS_RDONLY) + if (s->s_flags & MS_RDONLY) { /* it is read-only already */ + reiserfs_write_unlock(s); return 0; + } /* try to remount file system with read-only permissions */ if (sb_umount_state(rs) == REISERFS_VALID_FS || REISERFS_SB(s)->s_mount_state != REISERFS_VALID_FS) { + reiserfs_write_unlock(s); return 0; } @@ -800,8 +806,10 @@ s->s_dirt = 0; } else { /* remount read-write */ - if (!(s->s_flags & MS_RDONLY)) + if (!(s->s_flags & MS_RDONLY)) { + reiserfs_write_unlock(s); return 0; /* We are read-write already */ + } REISERFS_SB(s)->s_mount_state = sb_umount_state(rs) ; s->s_flags &= ~MS_RDONLY ; /* now it is safe to call journal_begin */ @@ -824,6 +832,7 @@ if (!( *mount_flags & MS_RDONLY ) ) finish_unfinished( s ); + reiserfs_write_unlock(s); return 0; } - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/