On Thu, Jan 23, 2020 at 09:22:10PM -0800, Darrick J. Wong wrote: > From: Darrick J. Wong <darrick.wong@xxxxxxxxxx> > > Stephen Rothwell reported an unused variable. Remove it. > > Fixes: 4bbb04abb4ee ("xfs: truncate should remove all blocks, not just to the end of the page cache") > Reported-by: Stephen Rothwell <sfr@xxxxxxxxxxxxxxxx> > Signed-off-by: Darrick J. Wong <darrick.wong@xxxxxxxxxx> self NAK, YueHaibing already sent this, which going through bash history I evidently misapplied and ... yeah. I've lost my marbles. :( --D > --- > fs/xfs/xfs_inode.c | 1 - > 1 file changed, 1 deletion(-) > > diff --git a/fs/xfs/xfs_inode.c b/fs/xfs/xfs_inode.c > index 1309f25c0d2b..1979a0055763 100644 > --- a/fs/xfs/xfs_inode.c > +++ b/fs/xfs/xfs_inode.c > @@ -1520,7 +1520,6 @@ xfs_itruncate_extents_flags( > xfs_fileoff_t first_unmap_block; > xfs_filblks_t unmap_len; > int error = 0; > - int done = 0; > > ASSERT(xfs_isilocked(ip, XFS_ILOCK_EXCL)); > ASSERT(!atomic_read(&VFS_I(ip)->i_count) ||