gfs2-utils: master - fsck.gfs2: Move pass5 to immediately follow pass1
by Bob Peterson
Gitweb: http://git.fedorahosted.org/git/?p=gfs2-utils.git;a=commitdiff;h=4519c9ca...
Commit: 4519c9ca6f8e85713989a4faf113f6468d5a1152
Parent: 3d9879c4f7e27cad55e30b054ce570e601cf431a
Author: Bob Peterson <rpeterso(a)redhat.com>
AuthorDate: Mon Mar 14 15:32:08 2016 -0400
Committer: Bob Peterson <rpeterso(a)redhat.com>
CommitterDate: Fri Apr 15 08:56:53 2016 -0400
fsck.gfs2: Move pass5 to immediately follow pass1
The purpose of pass1 is to make sure all bits are represented
properly for inodes. After that, the bitmaps should be kept in
sync by subsequent passes. The purpose of the old pass5 was really
to free any blocks that were not linked somehow to dinodes.
This patch moves that reconciliation to immediately follow pass1.
That allows us to free the bitmap memory sooner, making subsequent
passes use the rgrp bitmaps for their work. The purpose is to
reduce the memory footprint of fsck.gfs2.
Signed-off-by: Bob Peterson <rpeterso(a)redhat.com>
---
gfs2/fsck/main.c | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/gfs2/fsck/main.c b/gfs2/fsck/main.c
index 08343ca..8f2c733 100644
--- a/gfs2/fsck/main.c
+++ b/gfs2/fsck/main.c
@@ -236,11 +236,11 @@ struct fsck_pass {
static const struct fsck_pass passes[] = {
{ .name = "pass1", .f = pass1 },
+ { .name = "reconcile_bitmaps", .f = pass5 },
{ .name = "pass1b", .f = pass1b },
{ .name = "pass2", .f = pass2 },
{ .name = "pass3", .f = pass3 },
{ .name = "pass4", .f = pass4 },
- { .name = "pass5", .f = pass5 },
{ .name = "check_statfs", .f = check_statfs },
{ .name = NULL, }
};
8 years