From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 21153 invoked by alias); 4 Sep 2009 21:10:39 -0000 Received: (qmail 21147 invoked by alias); 4 Sep 2009 21:10:39 -0000 X-SWARE-Spam-Status: No, hits=-1.2 required=5.0 tests=AWL,BAYES_00,J_CHICKENPOX_41,SPF_HELO_PASS X-Spam-Status: No, hits=-1.2 required=5.0 tests=AWL,BAYES_00,J_CHICKENPOX_41,SPF_HELO_PASS X-Spam-Check-By: sourceware.org X-Spam-Checker-Version: SpamAssassin 3.2.5 (2008-06-10) on bastion2.fedora.phx.redhat.com Subject: gfs1-utils: master - GFS filesystem inconsistent after xfstests test suite run To: cluster-cvs-relay@redhat.com X-Project: Cluster Project X-Git-Module: gfs1-utils.git X-Git-Refname: refs/heads/master X-Git-Reftype: branch X-Git-Oldrev: 5a5e1d0bf3e030e38c050c29e8691d0e0db226db X-Git-Newrev: 68cffac61e089e0232fdfbf1fd3b17d47314b814 From: Bob Peterson Message-Id: <20090904211021.EF57A1201ED@lists.fedorahosted.org> Date: Fri, 04 Sep 2009 21:10:00 -0000 X-Scanned-By: MIMEDefang 2.67 on 10.5.11.12 Mailing-List: contact cluster-cvs-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Subscribe: List-Post: List-Help: , Sender: cluster-cvs-owner@sourceware.org X-SW-Source: 2009-q3/txt/msg00313.txt.bz2 Gitweb: http://git.fedorahosted.org/git/gfs1-utils.git?p=gfs1-utils.git;a=commitdiff;h=68cffac61e089e0232fdfbf1fd3b17d47314b814 Commit: 68cffac61e089e0232fdfbf1fd3b17d47314b814 Parent: 5a5e1d0bf3e030e38c050c29e8691d0e0db226db Author: Bob Peterson AuthorDate: Fri Sep 4 16:09:48 2009 -0500 Committer: Bob Peterson CommitterDate: Fri Sep 4 16:12:36 2009 -0500 GFS filesystem inconsistent after xfstests test suite run In pass1, an extra extended attribute block was being counted, causing improper block counts to be reported and set into the dinode. rhbz#521068 (crosswrite from GFS2) --- gfs/gfs_fsck/pass1.c | 1 - 1 files changed, 0 insertions(+), 1 deletions(-) diff --git a/gfs/gfs_fsck/pass1.c b/gfs/gfs_fsck/pass1.c index 8569db5..fb09db5 100644 --- a/gfs/gfs_fsck/pass1.c +++ b/gfs/gfs_fsck/pass1.c @@ -442,7 +442,6 @@ static int finish_eattr_indir(struct fsck_inode *ip, int leaf_pointers, /* Mark the inode as having an eattr in the block map so pass1c can check it. */ block_mark(ip->i_sbd->bl, ip->i_di.di_num.no_addr, eattr_block); - bc->ea_count++; if (!leaf_pointer_errors) return 0; log_err("Inode %lld has recoverable indirect "