On Mon, Nov 14, 2016 at 06:12:34PM +0100, Christoph Hellwig wrote: > Signed-off-by: Christoph Hellwig <hch@xxxxxx> > --- Reviewed-by: Brian Foster <bfoster@xxxxxxxxxx> > fs/xfs/libxfs/xfs_bmap.c | 14 ++++++-------- > 1 file changed, 6 insertions(+), 8 deletions(-) > > diff --git a/fs/xfs/libxfs/xfs_bmap.c b/fs/xfs/libxfs/xfs_bmap.c > index 98f490b..1a0fee4 100644 > --- a/fs/xfs/libxfs/xfs_bmap.c > +++ b/fs/xfs/libxfs/xfs_bmap.c > @@ -4145,12 +4145,11 @@ xfs_bmapi_read( > struct xfs_mount *mp = ip->i_mount; > struct xfs_ifork *ifp; > struct xfs_bmbt_irec got; > - struct xfs_bmbt_irec prev; > xfs_fileoff_t obno; > xfs_fileoff_t end; > - xfs_extnum_t lastx; > + xfs_extnum_t idx; > int error; > - int eof; > + bool eof = false; > int n = 0; > int whichfork = xfs_bmapi_whichfork(flags); > > @@ -4190,7 +4189,8 @@ xfs_bmapi_read( > return error; > } > > - xfs_bmap_search_extents(ip, bno, whichfork, &eof, &lastx, &got, &prev); > + if (!xfs_iext_lookup_extent(ip, ifp, bno, &idx, &got)) > + eof = true; > end = bno + len; > obno = bno; > > @@ -4221,10 +4221,8 @@ xfs_bmapi_read( > break; > > /* Else go on to the next record. */ > - if (++lastx < xfs_iext_count(ifp)) > - xfs_bmbt_get_all(xfs_iext_get_ext(ifp, lastx), &got); > - else > - eof = 1; > + if (!xfs_iext_get_extent(ifp, ++idx, &got)) > + eof = true; > } > *nmap = n; > return 0; > -- > 2.1.4 > > -- > To unsubscribe from this list: send the line "unsubscribe linux-xfs" in > the body of a message to majordomo@xxxxxxxxxxxxxxx > More majordomo info at http://vger.kernel.org/majordomo-info.html -- To unsubscribe from this list: send the line "unsubscribe linux-xfs" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html