Re: [PATCH 3/3] xfs: borrow indirect blocks from freed extent when avail

xfs

Re: [PATCH 3/3] xfs: borrow indirect blocks from freed extent when avail

To:	Christoph Hellwig <hch@xxxxxxxxxxxxx>
Subject:	Re: [PATCH 3/3] xfs: borrow indirect blocks from freed extent when available
From:	Brian Foster <bfoster@xxxxxxxxxx>
Date:	Tue, 1 Mar 2016 08:11:03 -0500
Cc:	xfs@xxxxxxxxxxx
Delivered-to:	xfs@xxxxxxxxxxx
In-reply-to:	<20160301130018.GA28675@xxxxxxxxxxxxx>
References:	<1456756170-51248-1-git-send-email-bfoster@xxxxxxxxxx> <1456756170-51248-4-git-send-email-bfoster@xxxxxxxxxx> <20160301130018.GA28675@xxxxxxxxxxxxx>
User-agent:	Mutt/1.5.24 (2015-08-30)

On Tue, Mar 01, 2016 at 05:00:18AM -0800, Christoph Hellwig wrote:
> I've been wrapping my head around this since yersterday..
> 
> It looks reasonable, but I'd be much more comfortable if you could
> split this into two patches:
> 
>  - one to just factor out a helper to update the temp and temp2
>    values (great variable names while we're at it..  not your fault,
>    though)
>  - one to actually change the algorithm used.

Sure, will do.

Brian

[More with this subject...]

<Prev in Thread]	Current Thread	[Next in Thread>
Re: [PATCH 3/3] xfs: borrow indirect blocks from freed extent when available, Christoph Hellwig Re: [PATCH 3/3] xfs: borrow indirect blocks from freed extent when available, Brian Foster <=

Previous by Date:	Re: [PATCH RFC 1/3] xfs: debug mode forced buffered write failure, Brian Foster
Next by Date:	Re: [PATCH] xfs: remove impossible condition, Luis de Bethencourt
Previous by Thread:	Re: [PATCH 3/3] xfs: borrow indirect blocks from freed extent when available, Christoph Hellwig
Next by Thread:	Re: futher writeback updates, Christoph Hellwig
Indexes:	[Date] [Thread] [Top] [All Lists]