gfs2-utils: master - fsck.gfs2: remove duplicate designation during undo

Bob Peterson rpeterso at fedoraproject.org
Thu Jan 15 18:34:48 UTC 2015


Gitweb:        http://git.fedorahosted.org/git/?p=gfs2-utils.git;a=commitdiff;h=9aedcc0e2e1d1b801bcb0b7d33d1f6e104fd400a
Commit:        9aedcc0e2e1d1b801bcb0b7d33d1f6e104fd400a
Parent:        3a629eccf04f4f7ee960c3ed980352a3d022f8cd
Author:        Bob Peterson <rpeterso at redhat.com>
AuthorDate:    Fri Jan 9 07:35:46 2015 -0600
Committer:     Bob Peterson <rpeterso at redhat.com>
CommitterDate: Fri Jan 9 13:31:45 2015 -0600

fsck.gfs2: remove duplicate designation during undo

This patch fixes a problem whereby fsck.gfs2's pass1 would perform this
sequence of events:
1. Metadata block X is identified as being referenced from dinode D1.
2. Metadata block X is identified as being referenced from another dinode, D2,
   which makes it a duplicate reference, but so far, no serious errors were
   found for that dinode.
3. Dinode D2 is found later to be irreparably damaged, and needs to be removed.
When D2 is deleted, the duplicate reference from D2 is removed and block X is
not freed because D1 still references it. However, it's still marked as a
duplicate and requires processing in pass1b.

Later, pass1b resolves the duplicate and determine's there is really only one
reference remaining, so it makes the correct decision. However, it should not
be necessary. The "undo" functions should remove the duplicate reference if
(and only if) the only reference was from D2. Note, though, that if the
corruption is found later in the cycle (after "undo" is possible) the duplicate
reference MUST remain and be resolved by pass1b.
---
 gfs2/fsck/pass1.c |   12 +++++++++++-
 1 files changed, 11 insertions(+), 1 deletions(-)

diff --git a/gfs2/fsck/pass1.c b/gfs2/fsck/pass1.c
index a4ba04c..73b054c 100644
--- a/gfs2/fsck/pass1.c
+++ b/gfs2/fsck/pass1.c
@@ -364,6 +364,11 @@ static int undo_reference(struct gfs2_inode *ip, uint64_t block, int meta,
 				  "from another inode; not freeing.\n"),
 				(unsigned long long)block,
 				(unsigned long long)block);
+			if (dt->refs == 1) {
+				log_err(_("This was the only duplicate "
+					  "reference so far; removing it.\n"));
+				dup_delete(dt);
+			}
 			return 1;
 		}
 	}
@@ -1055,8 +1060,13 @@ static int rangecheck_block(struct gfs2_inode *ip, uint64_t block,
 			  (unsigned long long)ip->i_di.di_num.no_addr);
 		if ((*bad_pointers) <= BAD_POINTER_TOLERANCE)
 			return meta_is_good;
-		else
+		else {
+			log_debug(_("Inode 0x%llx bad pointer tolerance "
+				    "exceeded: block 0x%llx.\n"),
+				  (unsigned long long)ip->i_di.di_num.no_addr,
+				  (unsigned long long)block);
 			return meta_error; /* Exits check_metatree quicker */
+		}
 	}
 	return meta_is_good;
 }


More information about the cluster-commits mailing list