diff options
author | Srinivas Eeda <srinivas.eeda@oracle.com> | 2009-03-06 17:21:46 -0500 |
---|---|---|
committer | Mark Fasheh <mfasheh@suse.com> | 2009-04-03 14:39:26 -0400 |
commit | 9140db04ef185f934acf2b1b15b3dd5e6a6bfc22 (patch) | |
tree | 354dbc0351195ae2b6ea5f7dfbf68a99307c3d69 /fs/ocfs2/super.c | |
parent | 1fca3a05ef2823830925dfb66711d6d920265a8d (diff) |
ocfs2: recover orphans in offline slots during recovery and mount
During recovery, a node recovers orphans in it's slot and the dead node(s). But
if the dead nodes were holding orphans in offline slots, they will be left
unrecovered.
If the dead node is the last one to die and is holding orphans in other slots
and is the first one to mount, then it only recovers it's own slot, which
leaves orphans in offline slots.
This patch queues complete_recovery to clean orphans for all offline slots
during mount and node recovery.
Signed-off-by: Srinivas Eeda <srinivas.eeda@oracle.com>
Acked-by: Joel Becker <joel.becker@oracle.com>
Signed-off-by: Mark Fasheh <mfasheh@suse.com>
Diffstat (limited to 'fs/ocfs2/super.c')
-rw-r--r-- | fs/ocfs2/super.c | 6 |
1 files changed, 6 insertions, 0 deletions
diff --git a/fs/ocfs2/super.c b/fs/ocfs2/super.c index 53892d874ffc..79ff8d9d37e0 100644 --- a/fs/ocfs2/super.c +++ b/fs/ocfs2/super.c | |||
@@ -2312,6 +2312,12 @@ static int ocfs2_check_volume(struct ocfs2_super *osb) | |||
2312 | * lock, and it's marked as dirty, set the bit in the recover | 2312 | * lock, and it's marked as dirty, set the bit in the recover |
2313 | * map and launch a recovery thread for it. */ | 2313 | * map and launch a recovery thread for it. */ |
2314 | status = ocfs2_mark_dead_nodes(osb); | 2314 | status = ocfs2_mark_dead_nodes(osb); |
2315 | if (status < 0) { | ||
2316 | mlog_errno(status); | ||
2317 | goto finally; | ||
2318 | } | ||
2319 | |||
2320 | status = ocfs2_compute_replay_slots(osb); | ||
2315 | if (status < 0) | 2321 | if (status < 0) |
2316 | mlog_errno(status); | 2322 | mlog_errno(status); |
2317 | 2323 | ||