[PATCH 3/4] xfs: reset buffer pointers before freeing them

[Dave Chinner]

On Thu, Apr 21, 2011 at 12:52:03AM -0400, Christoph Hellwig wrote:
> >  		ealign = round_down(end_block, sectbb);
> >  		if (j == 0 && (start_block + endcount > ealign)) {
> > -			offset = XFS_BUF_PTR(bp);
> > -			balign = BBTOB(ealign - start_block);
> > -			error = XFS_BUF_SET_PTR(bp, offset + balign,
> > -						BBTOB(sectbb));
> > +			balign = ealign - start_block;
> > +			offset = XFS_BUF_PTR(bp) + BBTOB(balign);
> > +			error = xlog_bread_offset(log, ealign, sectbb,
> > +							bp, offset);
> 
> I'd remove the use of balign entirely here.  The first use of this
> variable earlier in the functions is for something entirely different,
> so it's rather confusing.  (I only looked into that because I remember
> some align variable beeing used later on, but that was elsewhere)

Ok, will do.

> Otherwise looks good,
> 
> Reviewed-by: Christoph Hellwig <hch at lst.de>

Thanks.

-- 
Dave Chinner
david at fromorbit.com