aboutsummaryrefslogtreecommitdiffstats
path: root/fs/ocfs2/ocfs2.h
diff options
context:
space:
mode:
authorXue jiufei <xuejiufei@huawei.com>2014-06-04 19:06:13 -0400
committerLinus Torvalds <torvalds@linux-foundation.org>2014-06-04 19:53:54 -0400
commita9e9acaeb0a981a6dfa54b32dd756103aeefa6a7 (patch)
tree132b64a3d1e5c79a76c903a58f4f335869f87f5f /fs/ocfs2/ocfs2.h
parentc253ed1f6fd68df8542efed2d6b7656c3324f3e4 (diff)
ocfs2: fix umount hang while shutting down truncate log
Revert commit 75f82eaa502c ("ocfs2: fix NULL pointer dereference when dismount and ocfs2rec simultaneously") because it may cause a umount hang while shutting down the truncate log. fix NULL pointer dereference when dismount and ocfs2rec simultaneously The situation is as followes: ocfs2_dismout_volume -> ocfs2_recovery_exit -> free osb->recovery_map -> ocfs2_truncate_shutdown -> lock global bitmap inode -> ocfs2_wait_for_recovery -> check whether osb->recovery_map->rm_used is zero Because osb->recovery_map is already freed, rm_used can be any other values, so it may yield umount hang. To prevent NULL pointer dereference while getting sys_root_inode, we use a osb_tl_disable flag to disable schedule osb_truncate_log_wq after truncate log shutdown. Signed-off-by: joyce.xue <xuejiufei@huawei.com> Cc: Mark Fasheh <mfasheh@suse.com> Cc: Joel Becker <jlbec@evilplan.org> Signed-off-by: Andrew Morton <akpm@linux-foundation.org> Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'fs/ocfs2/ocfs2.h')
-rw-r--r--fs/ocfs2/ocfs2.h1
1 files changed, 1 insertions, 0 deletions
diff --git a/fs/ocfs2/ocfs2.h b/fs/ocfs2/ocfs2.h
index 8d64a97a9d5e..bbec539230fd 100644
--- a/fs/ocfs2/ocfs2.h
+++ b/fs/ocfs2/ocfs2.h
@@ -422,6 +422,7 @@ struct ocfs2_super
422 struct inode *osb_tl_inode; 422 struct inode *osb_tl_inode;
423 struct buffer_head *osb_tl_bh; 423 struct buffer_head *osb_tl_bh;
424 struct delayed_work osb_truncate_log_wq; 424 struct delayed_work osb_truncate_log_wq;
425 atomic_t osb_tl_disable;
425 /* 426 /*
426 * How many clusters in our truncate log. 427 * How many clusters in our truncate log.
427 * It must be protected by osb_tl_inode->i_mutex. 428 * It must be protected by osb_tl_inode->i_mutex.