xfs
[Top] [All Lists]

Re: [PATCH 1/6] xfs: optimize AGFL refills

To: Alex Elder <aelder@xxxxxxx>
Subject: Re: [PATCH 1/6] xfs: optimize AGFL refills
From: Christoph Hellwig <hch@xxxxxxxxxxxxx>
Date: Wed, 23 Mar 2011 08:16:13 -0400
Cc: Christoph Hellwig <hch@xxxxxxxxxxxxx>, xfs@xxxxxxxxxxx
In-reply-to: <1300833051.2875.275.camel@doink>
References: <20110322195550.260682574@xxxxxxxxxxxxxxxxxxxxxx> <20110322200137.280301318@xxxxxxxxxxxxxxxxxxxxxx> <1300833051.2875.275.camel@doink>
User-agent: Mutt/1.5.21 (2010-09-15)
On Tue, Mar 22, 2011 at 05:30:51PM -0500, Alex Elder wrote:
> On Tue, 2011-03-22 at 15:55 -0400, Christoph Hellwig wrote:
> > Avoid forcing out busy extent when moving blocks from/to the AGFL.  We
> 
> Without thinking a bit about it, it wasn't clear why doing
> this was OK to do.  It would be nice to record a one-sentence
> justification of why we don't need to force busy extents out
> in this case.

I've added a sentence, but it seems rather pointless.

>  achieve this

fixed.

> > the callers that need it, and by moving the busy list insert from
> > xfs_free_ag_extent extent which is used both by AGFL refills and real
>                      ^^^^^ drop this word

fixed.

<Prev in Thread] Current Thread [Next in Thread>