Attachment 833334 Details for Bug 1027451 – So close but yet so far

[patch] So close but yet so far

fsfreeze.patch (text/plain), 24.50 KB, created by Ben Marzinski on 2013-12-05 22:20:09 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Ben Marzinski

Created: 2013-12-05 22:20:09 UTC

Size: 24.50 KB

patch

obsolete

>---
> fs/gfs2/aops.c                   |    2 
> fs/gfs2/file.c                   |    2 
> fs/gfs2/glops.c                  |   46 +++++++++++--------
> fs/gfs2/glops.h                  |    2 
> fs/gfs2/incore.h                 |    9 ++-
> fs/gfs2/log.c                    |   93 +++++++++++++++++++++++++++------------
> fs/gfs2/log.h                    |   10 +++-
> fs/gfs2/ops_fstype.c             |   21 ++++++--
> fs/gfs2/quota.c                  |    2 
> fs/gfs2/recovery.c               |   19 +------
> fs/gfs2/rgrp.c                   |    2 
> fs/gfs2/super.c                  |   69 +++++++++++-----------------
> fs/gfs2/sys.c                    |    4 -
> fs/gfs2/trans.c                  |   27 ++---------
> include/uapi/linux/gfs2_ondisk.h |    2 
> 15 files changed, 168 insertions(+), 142 deletions(-)
>
>Index: gfs2-131111/fs/gfs2/glops.c
>===================================================================
>--- gfs2-131111.orig/fs/gfs2/glops.c
>+++ gfs2-131111/fs/gfs2/glops.c
>@@ -87,7 +87,9 @@ static void gfs2_ail_empty_gl(struct gfs
> 	if (!tr.tr_revokes)
> 		return;
> 
>-	/* A shortened, inline version of gfs2_trans_begin() */
>+	/* A shortened, inline version of gfs2_trans_begin()
>+         * tr->alloced is not set since the transaction structure is
>+         * on the stack */
> 	tr.tr_reserved = 1 + gfs2_struct2blk(sdp, tr.tr_revokes, sizeof(u64));
> 	tr.tr_ip = (unsigned long)__builtin_return_address(0);
> 	sb_start_intwrite(sdp->sd_vfs);
>@@ -98,7 +100,7 @@ static void gfs2_ail_empty_gl(struct gfs
> 	__gfs2_ail_flush(gl, 0, tr.tr_revokes);
> 
> 	gfs2_trans_end(sdp);
>-	gfs2_log_flush(sdp, NULL);
>+	gfs2_log_flush(sdp, NULL, NORMAL_FLUSH);
> }
> 
> void gfs2_ail_flush(struct gfs2_glock *gl, bool fsync)
>@@ -119,7 +121,7 @@ void gfs2_ail_flush(struct gfs2_glock *g
> 		return;
> 	__gfs2_ail_flush(gl, fsync, max_revokes);
> 	gfs2_trans_end(sdp);
>-	gfs2_log_flush(sdp, NULL);
>+	gfs2_log_flush(sdp, NULL, NORMAL_FLUSH);
> }
> 
> /**
>@@ -141,7 +143,7 @@ static void rgrp_go_sync(struct gfs2_glo
> 		return;
> 	GLOCK_BUG_ON(gl, gl->gl_state != LM_ST_EXCLUSIVE);
> 
>-	gfs2_log_flush(gl->gl_sbd, gl);
>+	gfs2_log_flush(gl->gl_sbd, gl, NORMAL_FLUSH);
> 	filemap_fdatawrite(metamapping);
> 	error = filemap_fdatawait(metamapping);
>         mapping_set_error(metamapping, error);
>@@ -199,7 +201,7 @@ static void inode_go_sync(struct gfs2_gl
> 
> 	GLOCK_BUG_ON(gl, gl->gl_state != LM_ST_EXCLUSIVE);
> 
>-	gfs2_log_flush(gl->gl_sbd, gl);
>+	gfs2_log_flush(gl->gl_sbd, gl, NORMAL_FLUSH);
> 	filemap_fdatawrite(metamapping);
> 	if (ip) {
> 		struct address_space *mapping = ip->i_inode.i_mapping;
>@@ -246,7 +248,7 @@ static void inode_go_inval(struct gfs2_g
> 	}
> 
> 	if (ip == GFS2_I(gl->gl_sbd->sd_rindex)) {
>-		gfs2_log_flush(gl->gl_sbd, NULL);
>+		gfs2_log_flush(gl->gl_sbd, NULL, NORMAL_FLUSH);
> 		gl->gl_sbd->sd_rindex_uptodate = 0;
> 	}
> 	if (ip && S_ISREG(ip->i_inode.i_mode))
>@@ -447,31 +449,39 @@ static int inode_go_dump(struct seq_file
> }
> 
> /**
>- * trans_go_sync - promote/demote the transaction glock
>+ * freeze_go_sync - promote/demote the freeze glock
>  * @gl: the glock
>  * @state: the requested state
>  * @flags:
>  *
>  */
> 
>-static void trans_go_sync(struct gfs2_glock *gl)
>+static void freeze_go_sync(struct gfs2_glock *gl)
> {
> 	struct gfs2_sbd *sdp = gl->gl_sbd;
>+	DEFINE_WAIT(wait);
> 
>-	if (gl->gl_state != LM_ST_UNLOCKED &&
>+	if (gl->gl_state == LM_ST_SHARED &&
> 	    test_bit(SDF_JOURNAL_LIVE, &sdp->sd_flags)) {
>-		gfs2_meta_syncfs(sdp);
>-		gfs2_log_shutdown(sdp);
>+		atomic_set(&sdp->sd_log_freeze, 1);
>+		wake_up(&sdp->sd_logd_waitq);
>+		do {
>+			prepare_to_wait(&sdp->sd_log_frozen_wait, &wait,
>+					TASK_UNINTERRUPTIBLE);
>+			if (atomic_read(&sdp->sd_log_freeze))
>+				io_schedule();
>+		} while(atomic_read(&sdp->sd_log_freeze));
>+        	finish_wait(&sdp->sd_log_frozen_wait, &wait);
> 	}
> }
> 
> /**
>- * trans_go_xmote_bh - After promoting/demoting the transaction glock
>+ * freeze_go_xmote_bh - After promoting/demoting the freeze glock
>  * @gl: the glock
>  *
>  */
> 
>-static int trans_go_xmote_bh(struct gfs2_glock *gl, struct gfs2_holder *gh)
>+static int freeze_go_xmote_bh(struct gfs2_glock *gl, struct gfs2_holder *gh)
> {
> 	struct gfs2_sbd *sdp = gl->gl_sbd;
> 	struct gfs2_inode *ip = GFS2_I(sdp->sd_jdesc->jd_inode);
>@@ -504,7 +514,7 @@ static int trans_go_xmote_bh(struct gfs2
>  * Always returns 0
>  */
> 
>-static int trans_go_demote_ok(const struct gfs2_glock *gl)
>+static int freeze_go_demote_ok(const struct gfs2_glock *gl)
> {
> 	return 0;
> }
>@@ -555,10 +565,10 @@ const struct gfs2_glock_operations gfs2_
> 	.go_flags = GLOF_ASPACE | GLOF_LVB,
> };
> 
>-const struct gfs2_glock_operations gfs2_trans_glops = {
>-	.go_sync = trans_go_sync,
>-	.go_xmote_bh = trans_go_xmote_bh,
>-	.go_demote_ok = trans_go_demote_ok,
>+const struct gfs2_glock_operations gfs2_freeze_glops = {
>+	.go_sync = freeze_go_sync,
>+	.go_xmote_bh = freeze_go_xmote_bh,
>+	.go_demote_ok = freeze_go_demote_ok,
> 	.go_type = LM_TYPE_NONDISK,
> };
> 
>Index: gfs2-131111/fs/gfs2/glops.h
>===================================================================
>--- gfs2-131111.orig/fs/gfs2/glops.h
>+++ gfs2-131111/fs/gfs2/glops.h
>@@ -15,7 +15,7 @@
> extern const struct gfs2_glock_operations gfs2_meta_glops;
> extern const struct gfs2_glock_operations gfs2_inode_glops;
> extern const struct gfs2_glock_operations gfs2_rgrp_glops;
>-extern const struct gfs2_glock_operations gfs2_trans_glops;
>+extern const struct gfs2_glock_operations gfs2_freeze_glops;
> extern const struct gfs2_glock_operations gfs2_iopen_glops;
> extern const struct gfs2_glock_operations gfs2_flock_glops;
> extern const struct gfs2_glock_operations gfs2_nondisk_glops;
>Index: gfs2-131111/fs/gfs2/incore.h
>===================================================================
>--- gfs2-131111.orig/fs/gfs2/incore.h
>+++ gfs2-131111/fs/gfs2/incore.h
>@@ -450,8 +450,7 @@ struct gfs2_trans {
> 	unsigned int tr_revokes;
> 	unsigned int tr_reserved;
> 
>-	struct gfs2_holder tr_t_gh;
>-
>+	int tr_alloced;
> 	int tr_touched;
> 	int tr_attached;
> 
>@@ -656,7 +655,7 @@ struct gfs2_sbd {
> 	struct lm_lockstruct sd_lockstruct;
> 	struct gfs2_holder sd_live_gh;
> 	struct gfs2_glock *sd_rename_gl;
>-	struct gfs2_glock *sd_trans_gl;
>+	struct gfs2_glock *sd_freeze_gl;
> 	wait_queue_head_t sd_glock_wait;
> 	atomic_t sd_glock_disposal;
> 	struct completion sd_locking_init;
>@@ -782,6 +781,10 @@ struct gfs2_sbd {
> 
> 	/* For quiescing the filesystem */
> 	struct gfs2_holder sd_freeze_gh;
>+	struct gfs2_holder sd_thaw_gh;
>+	atomic_t sd_log_freeze;
>+	atomic_t sd_started_freeze;
>+	wait_queue_head_t sd_log_frozen_wait;
> 
> 	char sd_fsname[GFS2_FSNAME_LEN];
> 	char sd_table_name[GFS2_FSNAME_LEN];
>Index: gfs2-131111/fs/gfs2/log.c
>===================================================================
>--- gfs2-131111.orig/fs/gfs2/log.c
>+++ gfs2-131111/fs/gfs2/log.c
>@@ -662,6 +662,43 @@ static void log_write_header(struct gfs2
> 		log_pull_tail(sdp, tail);
> }
> 
>+static int log_freeze(struct gfs2_sbd *sdp)
>+{
>+	int error = 0;
>+
>+	if (!atomic_read(&sdp->sd_started_freeze)) {
>+		printk(KERN_INFO "GFS2: freezing super\n");
>+		error = freeze_super(sdp->sd_vfs);
>+		printk(KERN_INFO "GFS2: done freezing super\n");
>+	}
>+	if (error) {
>+		printk(KERN_INFO "GFS2: couldn't freeze filesystem: %d\n",
>+		       error);
>+		gfs2_assert_withdraw(sdp, 0);
>+	}
>+	atomic_set(&sdp->sd_log_freeze, 0);
>+	wake_up(&sdp->sd_log_frozen_wait);
>+	error = gfs2_glock_nq_init(sdp->sd_freeze_gl, LM_ST_SHARED, 0,
>+				   &sdp->sd_thaw_gh);
>+	if (error) {
>+		printk(KERN_INFO "GFS2: couln't get freeze lock : %d\n", error);
>+		gfs2_assert_withdraw(sdp, 0);
>+	}
>+	else {
>+		gfs2_glock_dq_uninit(&sdp->sd_thaw_gh);
>+		if (!atomic_read(&sdp->sd_started_freeze)) {
>+			error = thaw_super(sdp->sd_vfs);
>+			if (error) {
>+				printk(KERN_INFO "GFS2: couldn't thaw filesystem: %d\n", error);
>+				gfs2_assert_withdraw(sdp, 0);
>+			}
>+		}
>+		else
>+			atomic_dec(&sdp->sd_started_freeze);
>+	}
>+	return error;
>+}
>+
> /**
>  * gfs2_log_flush - flush incore transaction(s)
>  * @sdp: the filesystem
>@@ -669,7 +706,8 @@ static void log_write_header(struct gfs2
>  *
>  */
> 
>-void gfs2_log_flush(struct gfs2_sbd *sdp, struct gfs2_glock *gl)
>+void gfs2_log_flush(struct gfs2_sbd *sdp, struct gfs2_glock *gl,
>+		    enum gfs2_flush_type type)
> {
> 	struct gfs2_trans *tr;
> 
>@@ -734,6 +772,29 @@ void gfs2_log_flush(struct gfs2_sbd *sdp
> 	}
> 	spin_unlock(&sdp->sd_ail_lock);
> 	gfs2_log_unlock(sdp);
>+
>+	if (atomic_read(&sdp->sd_log_freeze))
>+		type = FREEZE_FLUSH;
>+	if (type != NORMAL_FLUSH) {
>+		if (!sdp->sd_log_idle) {
>+			for (;;) {
>+				gfs2_ail1_start(sdp);
>+				gfs2_ail1_wait(sdp);
>+				if (gfs2_ail1_empty(sdp))
>+					break;
>+			}
>+			atomic_dec(&sdp->sd_log_blks_free); /* Adjust for unreserved buffer */
>+			trace_gfs2_log_blocks(sdp, -1);
>+			sdp->sd_log_flush_wrapped = 0;
>+			log_write_header(sdp, 0);
>+			sdp->sd_log_head = sdp->sd_log_flush_head;
>+		}
>+		if (type == SHUTDOWN_FLUSH || type == FREEZE_FLUSH)
>+			gfs2_log_shutdown(sdp);
>+		if (type == FREEZE_FLUSH)
>+			log_freeze(sdp);
>+	}
>+
> 	trace_gfs2_log_flush(sdp, 0);
> 	up_write(&sdp->sd_log_flush_lock);
> 
>@@ -764,7 +825,7 @@ static void log_refund(struct gfs2_sbd *
> 
> 	if (sdp->sd_log_tr == NULL &&
> 	    (tr->tr_num_buf_new || tr->tr_num_databuf_new)) {
>-		gfs2_assert_withdraw(sdp, tr->tr_t_gh.gh_gl);
>+		gfs2_assert_withdraw(sdp, tr->tr_alloced);
> 		sdp->sd_log_tr = tr;
> 		tr->tr_attached = 1;
> 	}
>@@ -804,8 +865,6 @@ void gfs2_log_commit(struct gfs2_sbd *sd
> 
> void gfs2_log_shutdown(struct gfs2_sbd *sdp)
> {
>-	down_write(&sdp->sd_log_flush_lock);
>-
> 	gfs2_assert_withdraw(sdp, !sdp->sd_log_blks_reserved);
> 	gfs2_assert_withdraw(sdp, !sdp->sd_log_num_buf);
> 	gfs2_assert_withdraw(sdp, !sdp->sd_log_num_revoke);
>@@ -818,33 +877,11 @@ void gfs2_log_shutdown(struct gfs2_sbd *
> 
> 	log_write_header(sdp, GFS2_LOG_HEAD_UNMOUNT);
> 
>-	gfs2_assert_warn(sdp, atomic_read(&sdp->sd_log_blks_free) == sdp->sd_jdesc->jd_blocks);
> 	gfs2_assert_warn(sdp, sdp->sd_log_head == sdp->sd_log_tail);
> 	gfs2_assert_warn(sdp, list_empty(&sdp->sd_ail2_list));
> 
> 	sdp->sd_log_head = sdp->sd_log_flush_head;
> 	sdp->sd_log_tail = sdp->sd_log_head;
>-
>-	up_write(&sdp->sd_log_flush_lock);
>-}
>-
>-
>-/**
>- * gfs2_meta_syncfs - sync all the buffers in a filesystem
>- * @sdp: the filesystem
>- *
>- */
>-
>-void gfs2_meta_syncfs(struct gfs2_sbd *sdp)
>-{
>-	gfs2_log_flush(sdp, NULL);
>-	for (;;) {
>-		gfs2_ail1_start(sdp);
>-		gfs2_ail1_wait(sdp);
>-		if (gfs2_ail1_empty(sdp))
>-			break;
>-	}
>-	gfs2_log_flush(sdp, NULL);
> }
> 
> static inline int gfs2_jrnl_flush_reqd(struct gfs2_sbd *sdp)
>@@ -876,14 +913,14 @@ int gfs2_logd(void *data)
> 
> 		if (gfs2_jrnl_flush_reqd(sdp) || t == 0) {
> 			gfs2_ail1_empty(sdp);
>-			gfs2_log_flush(sdp, NULL);
>+			gfs2_log_flush(sdp, NULL, NORMAL_FLUSH);
> 		}
> 
> 		if (gfs2_ail_flush_reqd(sdp)) {
> 			gfs2_ail1_start(sdp);
> 			gfs2_ail1_wait(sdp);
> 			gfs2_ail1_empty(sdp);
>-			gfs2_log_flush(sdp, NULL);
>+			gfs2_log_flush(sdp, NULL, NORMAL_FLUSH);
> 		}
> 
> 		if (!gfs2_ail_flush_reqd(sdp))
>Index: gfs2-131111/fs/gfs2/ops_fstype.c
>===================================================================
>--- gfs2-131111.orig/fs/gfs2/ops_fstype.c
>+++ gfs2-131111/fs/gfs2/ops_fstype.c
>@@ -115,6 +115,9 @@ static struct gfs2_sbd *init_sbd(struct
> 	init_rwsem(&sdp->sd_log_flush_lock);
> 	atomic_set(&sdp->sd_log_in_flight, 0);
> 	init_waitqueue_head(&sdp->sd_log_flush_wait);
>+	init_waitqueue_head(&sdp->sd_log_frozen_wait);
>+	atomic_set(&sdp->sd_log_freeze, 0);
>+	atomic_set(&sdp->sd_started_freeze, 0);
> 
> 	INIT_LIST_HEAD(&sdp->sd_revoke_list);
> 
>@@ -407,8 +410,8 @@ static int init_locking(struct gfs2_sbd
> 		goto fail_live;
> 	}
> 
>-	error = gfs2_glock_get(sdp, GFS2_TRANS_LOCK, &gfs2_trans_glops,
>-			       CREATE, &sdp->sd_trans_gl);
>+	error = gfs2_glock_get(sdp, GFS2_FREEZE_LOCK, &gfs2_freeze_glops,
>+			       CREATE, &sdp->sd_freeze_gl);
> 	if (error) {
> 		fs_err(sdp, "can't create transaction glock: %d\n", error);
> 		goto fail_rename;
>@@ -417,7 +420,7 @@ static int init_locking(struct gfs2_sbd
> 	return 0;
> 
> fail_trans:
>-	gfs2_glock_put(sdp->sd_trans_gl);
>+	gfs2_glock_put(sdp->sd_freeze_gl);
> fail_rename:
> 	gfs2_glock_put(sdp->sd_rename_gl);
> fail_live:
>@@ -802,7 +805,15 @@ static int init_journal(struct gfs2_sbd
> 	set_bit(SDF_JOURNAL_CHECKED, &sdp->sd_flags);
> 	gfs2_glock_dq_uninit(&ji_gh);
> 	jindex = 0;
>-
>+	if (!sdp->sd_args.ar_spectator) {
>+		error = gfs2_glock_nq_init(sdp->sd_freeze_gl, LM_ST_SHARED, 0,
>+					   &sdp->sd_thaw_gh);
>+		if (error) {
>+			fs_err(sdp, "can't acquire freeze glock: %d\n", error);
>+			goto fail_jinode_gh;
>+		}
>+	}
>+	gfs2_glock_dq_uninit(&sdp->sd_thaw_gh);
> 	return 0;
> 
> fail_jinode_gh:
>@@ -1457,7 +1468,7 @@ static void gfs2_kill_sb(struct super_bl
> 		return;
> 	}
> 
>-	gfs2_meta_syncfs(sdp);
>+	gfs2_log_flush(sdp, NULL, SYNC_FLUSH);
> 	dput(sdp->sd_root_dir);
> 	dput(sdp->sd_master_dir);
> 	sdp->sd_root_dir = NULL;
>Index: gfs2-131111/fs/gfs2/recovery.c
>===================================================================
>--- gfs2-131111.orig/fs/gfs2/recovery.c
>+++ gfs2-131111/fs/gfs2/recovery.c
>@@ -454,7 +454,7 @@ void gfs2_recover_func(struct work_struc
> 	struct gfs2_inode *ip = GFS2_I(jd->jd_inode);
> 	struct gfs2_sbd *sdp = GFS2_SB(jd->jd_inode);
> 	struct gfs2_log_header_host head;
>-	struct gfs2_holder j_gh, ji_gh, t_gh;
>+	struct gfs2_holder j_gh, ji_gh;
> 	unsigned long t;
> 	int ro = 0;
> 	unsigned int pass;
>@@ -508,14 +508,6 @@ void gfs2_recover_func(struct work_struc
> 
> 		t = jiffies;
> 
>-		/* Acquire a shared hold on the transaction lock */
>-
>-		error = gfs2_glock_nq_init(sdp->sd_trans_gl, LM_ST_SHARED,
>-					   LM_FLAG_NOEXP | LM_FLAG_PRIORITY |
>-					   GL_NOCACHE, &t_gh);
>-		if (error)
>-			goto fail_gunlock_ji;
>-
> 		if (test_bit(SDF_RORECOVERY, &sdp->sd_flags)) {
> 			ro = 1;
> 		} else if (test_bit(SDF_JOURNAL_CHECKED, &sdp->sd_flags)) {
>@@ -538,7 +530,7 @@ void gfs2_recover_func(struct work_struc
> 			fs_warn(sdp, "jid=%u: Can't replay: read-only block "
> 				"device\n", jd->jd_jid);
> 			error = -EROFS;
>-			goto fail_gunlock_tr;
>+			goto fail_gunlock_ji;
> 		}
> 
> 		fs_info(sdp, "jid=%u: Replaying journal...\n", jd->jd_jid);
>@@ -549,14 +541,13 @@ void gfs2_recover_func(struct work_struc
> 						   head.lh_blkno, pass);
> 			lops_after_scan(jd, error, pass);
> 			if (error)
>-				goto fail_gunlock_tr;
>+				goto fail_gunlock_ji;
> 		}
> 
> 		error = clean_journal(jd, &head);
> 		if (error)
>-			goto fail_gunlock_tr;
>+			goto fail_gunlock_ji;
> 
>-		gfs2_glock_dq_uninit(&t_gh);
> 		t = DIV_ROUND_UP(jiffies - t, HZ);
> 		fs_info(sdp, "jid=%u: Journal replayed in %lus\n",
> 			jd->jd_jid, t);
>@@ -572,8 +563,6 @@ void gfs2_recover_func(struct work_struc
> 	fs_info(sdp, "jid=%u: Done\n", jd->jd_jid);
> 	goto done;
> 
>-fail_gunlock_tr:
>-	gfs2_glock_dq_uninit(&t_gh);
> fail_gunlock_ji:
> 	if (jlocked) {
> 		gfs2_glock_dq_uninit(&ji_gh);
>Index: gfs2-131111/fs/gfs2/super.c
>===================================================================
>--- gfs2-131111.orig/fs/gfs2/super.c
>+++ gfs2-131111/fs/gfs2/super.c
>@@ -380,24 +380,18 @@ int gfs2_make_fs_rw(struct gfs2_sbd *sdp
> {
> 	struct gfs2_inode *ip = GFS2_I(sdp->sd_jdesc->jd_inode);
> 	struct gfs2_glock *j_gl = ip->i_gl;
>-	struct gfs2_holder t_gh;
> 	struct gfs2_log_header_host head;
> 	int error;
> 
>-	error = gfs2_glock_nq_init(sdp->sd_trans_gl, LM_ST_SHARED, 0, &t_gh);
>-	if (error)
>-		return error;
>-
> 	j_gl->gl_ops->go_inval(j_gl, DIO_METADATA);
> 
> 	error = gfs2_find_jhead(sdp->sd_jdesc, &head);
> 	if (error)
>-		goto fail;
>+		return error;
> 
> 	if (!(head.lh_flags & GFS2_LOG_HEAD_UNMOUNT)) {
> 		gfs2_consist(sdp);
>-		error = -EIO;
>-		goto fail;
>+		return -EIO;
> 	}
> 
> 	/*  Initialize some head of the log stuff  */
>@@ -406,19 +400,11 @@ int gfs2_make_fs_rw(struct gfs2_sbd *sdp
> 
> 	error = gfs2_quota_init(sdp);
> 	if (error)
>-		goto fail;
>+		return error;
> 
> 	set_bit(SDF_JOURNAL_LIVE, &sdp->sd_flags);
> 
>-	gfs2_glock_dq_uninit(&t_gh);
>-
> 	return 0;
>-
>-fail:
>-	t_gh.gh_flags |= GL_NOCACHE;
>-	gfs2_glock_dq_uninit(&t_gh);
>-
>-	return error;
> }
> 
> void gfs2_statfs_change_in(struct gfs2_statfs_change_host *sc, const void *buf)
>@@ -610,7 +596,7 @@ struct lfcc {
>  */
> 
> static int gfs2_lock_fs_check_clean(struct gfs2_sbd *sdp,
>-				    struct gfs2_holder *t_gh)
>+				    struct gfs2_holder *freeze_gh)
> {
> 	struct gfs2_inode *ip;
> 	struct gfs2_jdesc *jd;
>@@ -633,10 +619,9 @@ static int gfs2_lock_fs_check_clean(stru
> 		}
> 		list_add(&lfcc->list, &list);
> 	}
>-
>-	error = gfs2_glock_nq_init(sdp->sd_trans_gl, LM_ST_DEFERRED,
>-				   GL_NOCACHE, t_gh);
>-
>+	atomic_inc(&sdp->sd_started_freeze);
>+	error = gfs2_glock_nq_init(sdp->sd_freeze_gl, LM_ST_EXCLUSIVE,
>+				   GL_NOCACHE, freeze_gh);
> 	list_for_each_entry(jd, &sdp->sd_jindex_list, jd_list) {
> 		error = gfs2_jdesc_check(jd);
> 		if (error)
>@@ -651,7 +636,7 @@ static int gfs2_lock_fs_check_clean(stru
> 	}
> 
> 	if (error)
>-		gfs2_glock_dq_uninit(t_gh);
>+		gfs2_glock_dq_uninit(freeze_gh);
> 
> out:
> 	while (!list_empty(&list)) {
>@@ -717,7 +702,7 @@ static int gfs2_write_inode(struct inode
> 	int ret = 0;
> 
> 	if (wbc->sync_mode == WB_SYNC_ALL)
>-		gfs2_log_flush(GFS2_SB(inode), ip->i_gl);
>+		gfs2_log_flush(GFS2_SB(inode), ip->i_gl, NORMAL_FLUSH);
> 	if (bdi->dirty_exceeded)
> 		gfs2_ail1_flush(sdp, wbc);
> 	else
>@@ -797,29 +782,18 @@ out:
> 
> static int gfs2_make_fs_ro(struct gfs2_sbd *sdp)
> {
>-	struct gfs2_holder t_gh;
>-	int error;
>-
> 	flush_workqueue(gfs2_delete_workqueue);
> 	gfs2_quota_sync(sdp->sd_vfs, 0);
> 	gfs2_statfs_sync(sdp->sd_vfs, 0);
> 
>-	error = gfs2_glock_nq_init(sdp->sd_trans_gl, LM_ST_SHARED, GL_NOCACHE,
>-				   &t_gh);
>-	if (error && !test_bit(SDF_SHUTDOWN, &sdp->sd_flags))
>-		return error;
>-
>-	gfs2_meta_syncfs(sdp);
>-	gfs2_log_shutdown(sdp);
>+	gfs2_log_flush(sdp, NULL, SHUTDOWN_FLUSH);
>+	gfs2_assert_warn(sdp, atomic_read(&sdp->sd_log_blks_free) == sdp->sd_jdesc->jd_blocks);
> 
> 	clear_bit(SDF_JOURNAL_LIVE, &sdp->sd_flags);
> 
>-	if (t_gh.gh_gl)
>-		gfs2_glock_dq_uninit(&t_gh);
>-
> 	gfs2_quota_cleanup(sdp);
> 
>-	return error;
>+	return 0;
> }
> 
> static int gfs2_umount_recovery_wait(void *word)
>@@ -875,7 +849,7 @@ restart:
> 	iput(sdp->sd_quota_inode);
> 
> 	gfs2_glock_put(sdp->sd_rename_gl);
>-	gfs2_glock_put(sdp->sd_trans_gl);
>+	gfs2_glock_put(sdp->sd_freeze_gl);
> 
> 	if (!sdp->sd_args.ar_spectator) {
> 		gfs2_glock_dq_uninit(&sdp->sd_journal_gh);
>@@ -910,8 +884,8 @@ static int gfs2_sync_fs(struct super_blo
> 	struct gfs2_sbd *sdp = sb->s_fs_info;
> 
> 	gfs2_quota_sync(sb, -1);
>-	if (wait && sdp)
>-		gfs2_log_flush(sdp, NULL);
>+	if (wait && sdp && !atomic_read(&sdp->sd_log_freeze))
>+		gfs2_log_flush(sdp, NULL, NORMAL_FLUSH);
> 	return 0;
> }
> 
>@@ -926,6 +900,13 @@ static int gfs2_freeze(struct super_bloc
> 	struct gfs2_sbd *sdp = sb->s_fs_info;
> 	int error;
> 
>+	printk("GFS2: in gfs2_freeze\n");
>+	/* Just return if this is triggered by a freeze on another node
>+         * in the cluster */
>+	if (atomic_read(&sdp->sd_log_freeze))
>+		return 0;
>+	printk("GFS2: actually doing gfs2_freeze\n");
>+
> 	if (test_bit(SDF_SHUTDOWN, &sdp->sd_flags))
> 		return -EINVAL;
> 
>@@ -960,6 +941,10 @@ static int gfs2_unfreeze(struct super_bl
> {
> 	struct gfs2_sbd *sdp = sb->s_fs_info;
> 
>+	/* Just return if the freeze was started by another node in the
>+	 * cluster */
>+	if (!atomic_read(&sdp->sd_started_freeze))
>+		return 0;
> 	gfs2_glock_dq_uninit(&sdp->sd_freeze_gh);
> 	return 0;
> }
>@@ -1493,7 +1478,7 @@ static void gfs2_evict_inode(struct inod
> 	goto out_unlock;
> 
> out_truncate:
>-	gfs2_log_flush(sdp, ip->i_gl);
>+	gfs2_log_flush(sdp, ip->i_gl, NORMAL_FLUSH);
> 	if (test_bit(GLF_DIRTY, &ip->i_gl->gl_flags)) {
> 		struct address_space *metamapping = gfs2_glock2aspace(ip->i_gl);
> 		filemap_fdatawrite(metamapping);
>Index: gfs2-131111/fs/gfs2/sys.c
>===================================================================
>--- gfs2-131111.orig/fs/gfs2/sys.c
>+++ gfs2-131111/fs/gfs2/sys.c
>@@ -239,8 +239,8 @@ static ssize_t demote_rq_store(struct gf
> 
> 	if (gltype > LM_TYPE_JOURNAL)
> 		return -EINVAL;
>-	if (gltype == LM_TYPE_NONDISK && glnum == GFS2_TRANS_LOCK)
>-		glops = &gfs2_trans_glops;
>+	if (gltype == LM_TYPE_NONDISK && glnum == GFS2_FREEZE_LOCK)
>+		glops = &gfs2_freeze_glops;
> 	else
> 		glops = gfs2_glops_list[gltype];
> 	if (glops == NULL)
>Index: gfs2-131111/fs/gfs2/trans.c
>===================================================================
>--- gfs2-131111.orig/fs/gfs2/trans.c
>+++ gfs2-131111/fs/gfs2/trans.c
>@@ -46,32 +46,24 @@ int gfs2_trans_begin(struct gfs2_sbd *sd
> 	tr->tr_blocks = blocks;
> 	tr->tr_revokes = revokes;
> 	tr->tr_reserved = 1;
>+	tr->tr_alloced = 1;
> 	if (blocks)
> 		tr->tr_reserved += 6 + blocks;
> 	if (revokes)
> 		tr->tr_reserved += gfs2_struct2blk(sdp, revokes,
> 						   sizeof(u64));
> 	sb_start_intwrite(sdp->sd_vfs);
>-	gfs2_holder_init(sdp->sd_trans_gl, LM_ST_SHARED, 0, &tr->tr_t_gh);
>-
>-	error = gfs2_glock_nq(&tr->tr_t_gh);
>-	if (error)
>-		goto fail_holder_uninit;
> 
> 	error = gfs2_log_reserve(sdp, tr->tr_reserved);
> 	if (error)
>-		goto fail_gunlock;
>+		goto fail;
> 
> 	current->journal_info = tr;
> 
> 	return 0;
> 
>-fail_gunlock:
>-	gfs2_glock_dq(&tr->tr_t_gh);
>-
>-fail_holder_uninit:
>+fail:
> 	sb_end_intwrite(sdp->sd_vfs);
>-	gfs2_holder_uninit(&tr->tr_t_gh);
> 	kfree(tr);
> 
> 	return error;
>@@ -115,11 +107,8 @@ void gfs2_trans_end(struct gfs2_sbd *sdp
> 
> 	if (!tr->tr_touched) {
> 		gfs2_log_release(sdp, tr->tr_reserved);
>-		if (tr->tr_t_gh.gh_gl) {
>-			gfs2_glock_dq(&tr->tr_t_gh);
>-			gfs2_holder_uninit(&tr->tr_t_gh);
>+		if (tr->tr_alloced)
> 			kfree(tr);
>-		}
> 		sb_end_intwrite(sdp->sd_vfs);
> 		return;
> 	}
>@@ -133,16 +122,12 @@ void gfs2_trans_end(struct gfs2_sbd *sdp
> 		gfs2_print_trans(tr);
> 
> 	gfs2_log_commit(sdp, tr);
>-	if (tr->tr_t_gh.gh_gl) {
>-		gfs2_glock_dq(&tr->tr_t_gh);
>-		gfs2_holder_uninit(&tr->tr_t_gh);
>-		if (!tr->tr_attached)
>+	if (tr->tr_alloced && !tr->tr_attached)
> 			kfree(tr);
>-	}
> 	up_read(&sdp->sd_log_flush_lock);
> 
> 	if (sdp->sd_vfs->s_flags & MS_SYNCHRONOUS)
>-		gfs2_log_flush(sdp, NULL);
>+		gfs2_log_flush(sdp, NULL, NORMAL_FLUSH);
> 	sb_end_intwrite(sdp->sd_vfs);
> }
> 
>Index: gfs2-131111/include/uapi/linux/gfs2_ondisk.h
>===================================================================
>--- gfs2-131111.orig/include/uapi/linux/gfs2_ondisk.h
>+++ gfs2-131111/include/uapi/linux/gfs2_ondisk.h
>@@ -20,7 +20,7 @@
> 
> #define GFS2_MOUNT_LOCK		0
> #define GFS2_LIVE_LOCK		1
>-#define GFS2_TRANS_LOCK		2
>+#define GFS2_FREEZE_LOCK	2
> #define GFS2_RENAME_LOCK	3
> #define GFS2_CONTROL_LOCK	4
> #define GFS2_MOUNTED_LOCK	5
>Index: gfs2-131111/fs/gfs2/aops.c
>===================================================================
>--- gfs2-131111.orig/fs/gfs2/aops.c
>+++ gfs2-131111/fs/gfs2/aops.c
>@@ -371,7 +371,7 @@ static int gfs2_jdata_writepages(struct
> 
> 	ret = gfs2_write_cache_jdata(mapping, wbc);
> 	if (ret == 0 && wbc->sync_mode == WB_SYNC_ALL) {
>-		gfs2_log_flush(sdp, ip->i_gl);
>+		gfs2_log_flush(sdp, ip->i_gl, NORMAL_FLUSH);
> 		ret = gfs2_write_cache_jdata(mapping, wbc);
> 	}
> 	return ret;
>Index: gfs2-131111/fs/gfs2/file.c
>===================================================================
>--- gfs2-131111.orig/fs/gfs2/file.c
>+++ gfs2-131111/fs/gfs2/file.c
>@@ -256,7 +256,7 @@ static int do_gfs2_set_flags(struct file
> 	}
> 	if ((flags ^ new_flags) & GFS2_DIF_JDATA) {
> 		if (flags & GFS2_DIF_JDATA)
>-			gfs2_log_flush(sdp, ip->i_gl);
>+			gfs2_log_flush(sdp, ip->i_gl, NORMAL_FLUSH);
> 		error = filemap_fdatawrite(inode->i_mapping);
> 		if (error)
> 			goto out;
>Index: gfs2-131111/fs/gfs2/log.h
>===================================================================
>--- gfs2-131111.orig/fs/gfs2/log.h
>+++ gfs2-131111/fs/gfs2/log.h
>@@ -64,13 +64,19 @@ extern unsigned int gfs2_struct2blk(stru
> 			    unsigned int ssize);
> 
> extern int gfs2_log_reserve(struct gfs2_sbd *sdp, unsigned int blks);
>-extern void gfs2_log_flush(struct gfs2_sbd *sdp, struct gfs2_glock *gl);
>+enum gfs2_flush_type {
>+	NORMAL_FLUSH = 0,
>+	SYNC_FLUSH,
>+	SHUTDOWN_FLUSH,
>+	FREEZE_FLUSH
>+};
>+extern void gfs2_log_flush(struct gfs2_sbd *sdp, struct gfs2_glock *gl,
>+			   enum gfs2_flush_type type);
> extern void gfs2_log_commit(struct gfs2_sbd *sdp, struct gfs2_trans *trans);
> extern void gfs2_remove_from_ail(struct gfs2_bufdata *bd);
> extern void gfs2_ail1_flush(struct gfs2_sbd *sdp, struct writeback_control *wbc);
> 
> extern void gfs2_log_shutdown(struct gfs2_sbd *sdp);
>-extern void gfs2_meta_syncfs(struct gfs2_sbd *sdp);
> extern int gfs2_logd(void *data);
> extern void gfs2_add_revoke(struct gfs2_sbd *sdp, struct gfs2_bufdata *bd);
> extern void gfs2_write_revokes(struct gfs2_sbd *sdp);
>Index: gfs2-131111/fs/gfs2/quota.c
>===================================================================
>--- gfs2-131111.orig/fs/gfs2/quota.c
>+++ gfs2-131111/fs/gfs2/quota.c
>@@ -871,7 +871,7 @@ out:
> 		gfs2_glock_dq_uninit(&ghs[qx]);
> 	mutex_unlock(&ip->i_inode.i_mutex);
> 	kfree(ghs);
>-	gfs2_log_flush(ip->i_gl->gl_sbd, ip->i_gl);
>+	gfs2_log_flush(ip->i_gl->gl_sbd, ip->i_gl, NORMAL_FLUSH);
> 	return error;
> }
> 
>Index: gfs2-131111/fs/gfs2/rgrp.c
>===================================================================
>--- gfs2-131111.orig/fs/gfs2/rgrp.c
>+++ gfs2-131111/fs/gfs2/rgrp.c
>@@ -1943,7 +1943,7 @@ next_rgrp:
> 		}
> 		/* Flushing the log may release space */
> 		if (loops == 2)
>-			gfs2_log_flush(sdp, NULL);
>+			gfs2_log_flush(sdp, NULL, NORMAL_FLUSH);
> 	}
> 
> 	return -ENOSPC;

Actions: View | Diff

Attachments on bug 1027451: 820790 | 824779 | 833334 | 846826 | 888221 | 888975 | 891000