diff options
author | David Teigland <teigland@redhat.com> | 2012-01-09 14:40:06 -0500 |
---|---|---|
committer | Steven Whitehouse <swhiteho@redhat.com> | 2012-01-11 04:23:40 -0500 |
commit | e8ca5cc571a60339491f8c273a01093096ff8704 (patch) | |
tree | e5133a5600113e8c2849675ab96dce0e6521aa09 /fs/gfs2/recovery.c | |
parent | 49528b4e479195e5db4fe51fcd5ddd97901efc16 (diff) |
GFS2: let spectator mount do read only recovery
Previously, a spectator mount would not even attempt to do
journal recovery for a failed node. This meant that if all
mounted nodes were spectators, everyone would be stuck after
a node failed, all waiting for recovery to be performed.
This is unnecessary since the failed node had a clean journal.
Instead, allow a spectator mount to do a partial "read only"
recovery, which means it will check if the failed journal is
clean, and if so, report a successful recovery. If the failed
journal is not clean, it reports that journal recovery failed.
This makes it work the same as a read only mount on a read only
block device.
Signed-off-by: David Teigland <teigland@redhat.com>
Signed-off-by: Steven Whitehouse <swhiteho@redhat.com>
Diffstat (limited to 'fs/gfs2/recovery.c')
-rw-r--r-- | fs/gfs2/recovery.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/fs/gfs2/recovery.c b/fs/gfs2/recovery.c index af49e8f432fe..80701d1566a1 100644 --- a/fs/gfs2/recovery.c +++ b/fs/gfs2/recovery.c | |||
@@ -516,7 +516,9 @@ void gfs2_recover_func(struct work_struct *work) | |||
516 | if (error) | 516 | if (error) |
517 | goto fail_gunlock_ji; | 517 | goto fail_gunlock_ji; |
518 | 518 | ||
519 | if (test_bit(SDF_JOURNAL_CHECKED, &sdp->sd_flags)) { | 519 | if (test_bit(SDF_RORECOVERY, &sdp->sd_flags)) { |
520 | ro = 1; | ||
521 | } else if (test_bit(SDF_JOURNAL_CHECKED, &sdp->sd_flags)) { | ||
520 | if (!test_bit(SDF_JOURNAL_LIVE, &sdp->sd_flags)) | 522 | if (!test_bit(SDF_JOURNAL_LIVE, &sdp->sd_flags)) |
521 | ro = 1; | 523 | ro = 1; |
522 | } else { | 524 | } else { |