From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 5215 invoked by alias); 8 Sep 2009 18:20:48 -0000 Received: (qmail 5165 invoked by alias); 8 Sep 2009 18:20:47 -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: cluster: STABLE2 - GFS2 filesystem inconsistent after xfstests test suite run To: cluster-cvs-relay@redhat.com X-Project: Cluster Project X-Git-Module: cluster.git X-Git-Refname: refs/heads/STABLE2 X-Git-Reftype: branch X-Git-Oldrev: 1ffca8c9db71bd8c7609e0a3db0654fdf13571f0 X-Git-Newrev: 306d983bb92fefe5913f522a0f53cdb639620378 From: Bob Peterson Message-Id: <20090908182011.67C1012030A@lists.fedorahosted.org> Date: Tue, 08 Sep 2009 18:20:00 -0000 X-Scanned-By: MIMEDefang 2.67 on 10.5.11.18 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/msg00322.txt.bz2 Gitweb: http://git.fedorahosted.org/git/cluster.git?p=cluster.git;a=commitdiff;h=306d983bb92fefe5913f522a0f53cdb639620378 Commit: 306d983bb92fefe5913f522a0f53cdb639620378 Parent: 1ffca8c9db71bd8c7609e0a3db0654fdf13571f0 Author: Bob Peterson AuthorDate: Tue Sep 8 13:15:09 2009 -0500 Committer: Bob Peterson CommitterDate: Tue Sep 8 13:22:01 2009 -0500 GFS2 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#500483 --- gfs2/fsck/pass1.c | 1 - 1 files changed, 0 insertions(+), 1 deletions(-) diff --git a/gfs2/fsck/pass1.c b/gfs2/fsck/pass1.c index 89436dd..8a40ab6 100644 --- a/gfs2/fsck/pass1.c +++ b/gfs2/fsck/pass1.c @@ -388,7 +388,6 @@ static int finish_eattr_indir(struct gfs2_inode *ip, int leaf_pointers, so pass1c can check it. */ gfs2_block_mark(ip->i_sbd, bl, ip->i_di.di_num.no_addr, gfs2_eattr_block); - bc->ea_count++; if (!leaf_pointer_errors) return 0; log_err("Inode %lld (0x%llx) has recoverable indirect "