From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mx1.redhat.com ([209.132.183.28]:56696 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S933811AbdEOJDR (ORCPT ); Mon, 15 May 2017 05:03:17 -0400 Date: Mon, 15 May 2017 11:03:08 +0200 From: Carlos Maiolino Subject: Re: [PATCH] xfs: fix warnings about unused stack variables Message-ID: <20170515090308.cyrkuoyy5mm3snma@eorzea.usersys.redhat.com> References: <20170512005352.GF4519@birch.djwong.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20170512005352.GF4519@birch.djwong.org> Sender: linux-xfs-owner@vger.kernel.org List-ID: List-Id: xfs To: "Darrick J. Wong" Cc: xfs , Christoph Hellwig On Thu, May 11, 2017 at 05:53:52PM -0700, Darrick J. Wong wrote: > Reduce stack usage and get rid of compiler warnings by eliminating > unused variables. The variables were eliminated during one of > Christoph's cleanups for 4.12. > > Signed-off-by: Darrick J. Wong > --- > fs/xfs/libxfs/xfs_bmap.c | 2 -- > 1 file changed, 2 deletions(-) Looks fine Reviewed-by: Carlos Maiolino > > diff --git a/fs/xfs/libxfs/xfs_bmap.c b/fs/xfs/libxfs/xfs_bmap.c > index 8adb91b..a7048eaf 100644 > --- a/fs/xfs/libxfs/xfs_bmap.c > +++ b/fs/xfs/libxfs/xfs_bmap.c > @@ -1280,7 +1280,6 @@ xfs_bmap_read_extents( > xfs_bmbt_rec_t *frp; > xfs_fsblock_t nextbno; > xfs_extnum_t num_recs; > - xfs_extnum_t start; > > num_recs = xfs_btree_get_numrecs(block); > if (unlikely(i + num_recs > room)) { > @@ -1303,7 +1302,6 @@ xfs_bmap_read_extents( > * Copy records into the extent records. > */ > frp = XFS_BMBT_REC_ADDR(mp, block, 1); > - start = i; > for (j = 0; j < num_recs; j++, i++, frp++) { > xfs_bmbt_rec_host_t *trp = xfs_iext_get_ext(ifp, i); > trp->l0 = be64_to_cpu(frp->l0); > -- > To unsubscribe from this list: send the line "unsubscribe linux-xfs" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html -- Carlos