[GFS2] kernel panic mounting volume
authorBob Peterson <rpeterso@redhat.com>
Mon, 9 Jun 2008 17:08:23 +0000 (12:08 -0500)
committerSteven Whitehouse <swhiteho@redhat.com>
Fri, 27 Jun 2008 08:39:41 +0000 (09:39 +0100)
commit9171f5a991e7613cbee816874ad8c9515dcab50f
tree6fd5df32583c059344f7ec4b9436b68edecd61c0
parent01b7c7ae88a6376c508b35a22bb61e04cb1b37f0
[GFS2] kernel panic mounting volume

This patch fixes Red Hat bugzilla bug 450156.

This started with a not-too-improbable mount failure because the
locking protocol was never set back to its proper "lock_dlm" after the
system was rebooted in the middle of a gfs2_fsck.  That left a
(purposely) invalid locking protocol in the superblock, which caused an
error when the file system was mounted the next time.

When there's an error mounting, vfs calls DQUOT_OFF, which calls
vfs_quota_off which calls gfs2_sync_fs.  Next, gfs2_sync_fs calls
gfs2_log_flush passing s_fs_info.  But due to the error, s_fs_info
had been previously set to NULL, and so we have the kernel oops.

My solution in this patch is to test for the NULL value before passing
it.  I tested this patch and it fixes the problem.

Signed-off-by: Bob Peterson <rpeterso@redhat.com>
Signed-off-by: Steven Whitehouse <swhiteho@redhat.com>
fs/gfs2/ops_super.c