Gitweb: http://git.fedorahosted.org/git/?p=gfs2-utils.git;a=commitdiff;h=4519c9ca6f8... Commit: 4519c9ca6f8e85713989a4faf113f6468d5a1152 Parent: 3d9879c4f7e27cad55e30b054ce570e601cf431a Author: Bob Peterson rpeterso@redhat.com AuthorDate: Mon Mar 14 15:32:08 2016 -0400 Committer: Bob Peterson rpeterso@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@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, } };