Commit 623ba664 authored by Bob Peterson's avatar Bob Peterson Committed by Andreas Gruenbacher

gfs2: When freezing gfs2, use GL_EXACT and not GL_NOCACHE

Before this patch, the freeze code in gfs2 specified GL_NOCACHE in
several places. That's wrong because we always want to know the state
of whether the file system is frozen.

There was also a problem with freeze/thaw transitioning the glock from
frozen (EX) to thawed (SH) because gfs2 will normally grant glocks in EX
to processes that request it in SH mode, unless GL_EXACT is specified.
Therefore, the freeze/thaw code, which tried to reacquire the glock in
SH mode would get the glock in EX mode, and miss the transition from EX
to SH. That made it think the thaw had completed normally, but since the
glock was still cached in EX, other nodes could not freeze again.

This patch removes the GL_NOCACHE flag to allow the freeze glock to be
cached. It also adds the GL_EXACT flag so the glock is fully transitioned
from EX to SH, thereby allowing future freeze operations.
Signed-off-by: default avatarBob Peterson <rpeterso@redhat.com>
parent b780cc61
...@@ -364,8 +364,8 @@ void gfs2_recover_func(struct work_struct *work) ...@@ -364,8 +364,8 @@ void gfs2_recover_func(struct work_struct *work)
/* Acquire a shared hold on the freeze lock */ /* Acquire a shared hold on the freeze lock */
error = gfs2_glock_nq_init(sdp->sd_freeze_gl, LM_ST_SHARED, error = gfs2_glock_nq_init(sdp->sd_freeze_gl, LM_ST_SHARED,
LM_FLAG_NOEXP | LM_FLAG_PRIORITY, LM_FLAG_NOEXP | LM_FLAG_PRIORITY |
&thaw_gh); GL_EXACT, &thaw_gh);
if (error) if (error)
goto fail_gunlock_ji; goto fail_gunlock_ji;
......
...@@ -167,7 +167,7 @@ int gfs2_make_fs_rw(struct gfs2_sbd *sdp) ...@@ -167,7 +167,7 @@ int gfs2_make_fs_rw(struct gfs2_sbd *sdp)
if (error) if (error)
return error; return error;
error = gfs2_glock_nq_init(sdp->sd_freeze_gl, LM_ST_SHARED, 0, error = gfs2_glock_nq_init(sdp->sd_freeze_gl, LM_ST_SHARED, GL_EXACT,
&freeze_gh); &freeze_gh);
if (error) if (error)
goto fail_threads; goto fail_threads;
...@@ -203,7 +203,6 @@ int gfs2_make_fs_rw(struct gfs2_sbd *sdp) ...@@ -203,7 +203,6 @@ int gfs2_make_fs_rw(struct gfs2_sbd *sdp)
return 0; return 0;
fail: fail:
freeze_gh.gh_flags |= GL_NOCACHE;
gfs2_glock_dq_uninit(&freeze_gh); gfs2_glock_dq_uninit(&freeze_gh);
fail_threads: fail_threads:
if (sdp->sd_quotad_process) if (sdp->sd_quotad_process)
...@@ -430,7 +429,7 @@ static int gfs2_lock_fs_check_clean(struct gfs2_sbd *sdp) ...@@ -430,7 +429,7 @@ static int gfs2_lock_fs_check_clean(struct gfs2_sbd *sdp)
} }
error = gfs2_glock_nq_init(sdp->sd_freeze_gl, LM_ST_EXCLUSIVE, error = gfs2_glock_nq_init(sdp->sd_freeze_gl, LM_ST_EXCLUSIVE,
GL_NOCACHE, &sdp->sd_freeze_gh); 0, &sdp->sd_freeze_gh);
if (error) if (error)
goto out; goto out;
...@@ -613,13 +612,14 @@ int gfs2_make_fs_ro(struct gfs2_sbd *sdp) ...@@ -613,13 +612,14 @@ int gfs2_make_fs_ro(struct gfs2_sbd *sdp)
!gfs2_glock_is_locked_by_me(sdp->sd_freeze_gl)) { !gfs2_glock_is_locked_by_me(sdp->sd_freeze_gl)) {
if (!log_write_allowed) { if (!log_write_allowed) {
error = gfs2_glock_nq_init(sdp->sd_freeze_gl, error = gfs2_glock_nq_init(sdp->sd_freeze_gl,
LM_ST_SHARED, GL_NOCACHE | LM_ST_SHARED,
LM_FLAG_TRY, &freeze_gh); LM_FLAG_TRY | GL_EXACT,
&freeze_gh);
if (error == GLR_TRYFAILED) if (error == GLR_TRYFAILED)
error = 0; error = 0;
} else { } else {
error = gfs2_glock_nq_init(sdp->sd_freeze_gl, error = gfs2_glock_nq_init(sdp->sd_freeze_gl,
LM_ST_SHARED, GL_NOCACHE, LM_ST_SHARED, GL_EXACT,
&freeze_gh); &freeze_gh);
if (error && !gfs2_withdrawn(sdp)) if (error && !gfs2_withdrawn(sdp))
return error; return error;
...@@ -761,7 +761,7 @@ void gfs2_freeze_func(struct work_struct *work) ...@@ -761,7 +761,7 @@ void gfs2_freeze_func(struct work_struct *work)
struct super_block *sb = sdp->sd_vfs; struct super_block *sb = sdp->sd_vfs;
atomic_inc(&sb->s_active); atomic_inc(&sb->s_active);
error = gfs2_glock_nq_init(sdp->sd_freeze_gl, LM_ST_SHARED, 0, error = gfs2_glock_nq_init(sdp->sd_freeze_gl, LM_ST_SHARED, GL_EXACT,
&freeze_gh); &freeze_gh);
if (error) { if (error) {
fs_info(sdp, "GFS2: couldn't get freeze lock : %d\n", error); fs_info(sdp, "GFS2: couldn't get freeze lock : %d\n", error);
...@@ -774,8 +774,6 @@ void gfs2_freeze_func(struct work_struct *work) ...@@ -774,8 +774,6 @@ void gfs2_freeze_func(struct work_struct *work)
error); error);
gfs2_assert_withdraw(sdp, 0); gfs2_assert_withdraw(sdp, 0);
} }
if (!test_bit(SDF_JOURNAL_LIVE, &sdp->sd_flags))
freeze_gh.gh_flags |= GL_NOCACHE;
gfs2_glock_dq_uninit(&freeze_gh); gfs2_glock_dq_uninit(&freeze_gh);
} }
deactivate_super(sb); deactivate_super(sb);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment