cluster: RHEL6 - fsck.gfs2: pass1c counts percentage backward

Bob Peterson rpeterso at fedoraproject.org
Tue Jan 24 20:46:04 UTC 2012


Gitweb:        http://git.fedorahosted.org/git/cluster.git?p=cluster.git;a=commitdiff;h=68b32af3f105ca6a373328e66029ea80b8e8630b
Commit:        68b32af3f105ca6a373328e66029ea80b8e8630b
Parent:        165021e9be8503dd8ffc37edd2da71d1aa994df0
Author:        Bob Peterson <rpeterso at redhat.com>
AuthorDate:    Fri Jan 6 14:20:26 2012 -0600
Committer:     Bob Peterson <rpeterso at redhat.com>
CommitterDate: Fri Jan 20 08:26:27 2012 -0600

fsck.gfs2: pass1c counts percentage backward

In testing I noticed that pass1c, which checks all extended
attributes, counted percent-complete backwards.  It's better to
count forward because disk block seeks will be more efficient.
This patch changes the special_list functions of block_list to
add items as "prev" so that they're processed in block order.

rhbz#675723
---
 gfs2/libgfs2/block_list.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/gfs2/libgfs2/block_list.c b/gfs2/libgfs2/block_list.c
index 65003b0..554a1ef 100644
--- a/gfs2/libgfs2/block_list.c
+++ b/gfs2/libgfs2/block_list.c
@@ -42,7 +42,7 @@ void gfs2_special_add(struct special_blocks *blocklist, uint64_t block)
 	if (b) {
 		memset(b, 0, sizeof(*b));
 		b->block = block;
-		osi_list_add(&b->list, &blocklist->list);
+		osi_list_add_prev(&b->list, &blocklist->list);
 	}
 }
 


More information about the cluster-commits mailing list