From mboxrd@z Thu Jan 1 00:00:00 1970 From: Steven Whitehouse Date: Fri, 12 Aug 2011 10:18:51 +0100 Subject: [Cluster-devel] [Patch 08/44] fsck.gfs2: eliminate vestigial buffer_head variable in check_leaf In-Reply-To: <2022993666.544648.1313096465827.JavaMail.root@zmail06.collab.prod.int.phx2.redhat.com> References: <2022993666.544648.1313096465827.JavaMail.root@zmail06.collab.prod.int.phx2.redhat.com> Message-ID: <1313140731.2704.24.camel@menhir> List-Id: To: cluster-devel.redhat.com MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Likewise this one, Steve. On Thu, 2011-08-11 at 17:01 -0400, Bob Peterson wrote: > >From dfa63a3b56e71b8607098cb02e5162fc01aa8bab Mon Sep 17 00:00:00 2001 > From: Bob Peterson > Date: Mon, 8 Aug 2011 13:28:08 -0500 > Subject: [PATCH 08/44] fsck.gfs2: eliminate vestigial buffer_head variable in > check_leaf > > This patch eliminates a variable bh from all the check_leaf metawalk > functions because it is no longer referenced. > > rhbzs#675723 > --- > gfs2/fsck/metawalk.c | 10 ++++------ > gfs2/fsck/metawalk.h | 5 ++--- > gfs2/fsck/pass1.c | 14 ++++++-------- > 3 files changed, 12 insertions(+), 17 deletions(-) > > diff --git a/gfs2/fsck/metawalk.c b/gfs2/fsck/metawalk.c > index 5d0afa5..ea1774a 100644 > --- a/gfs2/fsck/metawalk.c > +++ b/gfs2/fsck/metawalk.c > @@ -686,7 +686,7 @@ static int check_leaf_blks(struct gfs2_inode *ip, struct metawalk_fxns *pass) > } > gfs2_leaf_in(&leaf, lbh); > if (pass->check_leaf) > - error = pass->check_leaf(ip, leaf_no, lbh, > + error = pass->check_leaf(ip, leaf_no, > pass->private); > > /* > @@ -1462,10 +1462,9 @@ int delete_metadata(struct gfs2_inode *ip, uint64_t block, > return delete_block_if_notdup(ip, block, bh, _("metadata"), private); > } > > -int delete_leaf(struct gfs2_inode *ip, uint64_t block, > - struct gfs2_buffer_head *bh, void *private) > +int delete_leaf(struct gfs2_inode *ip, uint64_t block, void *private) > { > - return delete_block_if_notdup(ip, block, &bh, _("leaf"), private); > + return delete_block_if_notdup(ip, block, NULL, _("leaf"), private); > } > > int delete_data(struct gfs2_inode *ip, uint64_t block, void *private) > @@ -1528,8 +1527,7 @@ static int alloc_data(struct gfs2_inode *ip, uint64_t block, void *private) > return 0; > } > > -static int alloc_leaf(struct gfs2_inode *ip, uint64_t block, > - struct gfs2_buffer_head *bh, void *private) > +static int alloc_leaf(struct gfs2_inode *ip, uint64_t block, void *private) > { > uint8_t q; > > diff --git a/gfs2/fsck/metawalk.h b/gfs2/fsck/metawalk.h > index c1e61fb..ea023b6 100644 > --- a/gfs2/fsck/metawalk.h > +++ b/gfs2/fsck/metawalk.h > @@ -20,8 +20,7 @@ extern int delete_block(struct gfs2_inode *ip, uint64_t block, > void *private); > extern int delete_metadata(struct gfs2_inode *ip, uint64_t block, > struct gfs2_buffer_head **bh, int h, void *private); > -extern int delete_leaf(struct gfs2_inode *ip, uint64_t block, > - struct gfs2_buffer_head *bh, void *private); > +extern int delete_leaf(struct gfs2_inode *ip, uint64_t block, void *private); > extern int delete_data(struct gfs2_inode *ip, uint64_t block, void *private); > extern int delete_eattr_indir(struct gfs2_inode *ip, uint64_t block, uint64_t parent, > struct gfs2_buffer_head **bh, void *private); > @@ -60,7 +59,7 @@ extern struct gfs2_inode *fsck_system_inode(struct gfs2_sbd *sdp, > struct metawalk_fxns { > void *private; > int (*check_leaf) (struct gfs2_inode *ip, uint64_t block, > - struct gfs2_buffer_head *bh, void *private); > + void *private); > int (*check_metalist) (struct gfs2_inode *ip, uint64_t block, > struct gfs2_buffer_head **bh, int h, > void *private); > diff --git a/gfs2/fsck/pass1.c b/gfs2/fsck/pass1.c > index 30d6b3c..f0e7277 100644 > --- a/gfs2/fsck/pass1.c > +++ b/gfs2/fsck/pass1.c > @@ -34,8 +34,7 @@ struct block_count { > uint64_t ea_count; > }; > > -static int leaf(struct gfs2_inode *ip, uint64_t block, > - struct gfs2_buffer_head *bh, void *private); > +static int leaf(struct gfs2_inode *ip, uint64_t block, void *private); > static int check_metalist(struct gfs2_inode *ip, uint64_t block, > struct gfs2_buffer_head **bh, int h, void *private); > static int undo_check_metalist(struct gfs2_inode *ip, uint64_t block, > @@ -66,7 +65,7 @@ static int invalidate_metadata(struct gfs2_inode *ip, uint64_t block, > struct gfs2_buffer_head **bh, int h, > void *private); > static int invalidate_leaf(struct gfs2_inode *ip, uint64_t block, > - struct gfs2_buffer_head *bh, void *private); > + void *private); > static int invalidate_data(struct gfs2_inode *ip, uint64_t block, > void *private); > static int invalidate_eattr_indir(struct gfs2_inode *ip, uint64_t block, > @@ -200,8 +199,7 @@ struct metawalk_fxns sysdir_fxns = { > .check_dentry = resuscitate_dentry, > }; > > -static int leaf(struct gfs2_inode *ip, uint64_t block, > - struct gfs2_buffer_head *bh, void *private) > +static int leaf(struct gfs2_inode *ip, uint64_t block, void *private) > { > struct block_count *bc = (struct block_count *) private; > > @@ -856,7 +854,7 @@ static int invalidate_metadata(struct gfs2_inode *ip, uint64_t block, > } > > static int invalidate_leaf(struct gfs2_inode *ip, uint64_t block, > - struct gfs2_buffer_head *bh, void *private) > + void *private) > { > return mark_block_invalid(ip, block, ref_as_meta, _("leaf")); > } > @@ -939,9 +937,9 @@ static int rangecheck_metadata(struct gfs2_inode *ip, uint64_t block, > } > > static int rangecheck_leaf(struct gfs2_inode *ip, uint64_t block, > - struct gfs2_buffer_head *bh, void *private) > + void *private) > { > - return rangecheck_block(ip, block, &bh, _("leaf"), private); > + return rangecheck_block(ip, block, NULL, _("leaf"), private); > } > > static int rangecheck_data(struct gfs2_inode *ip, uint64_t block,