xfs
[Top] [All Lists]

Re: [PATCH] xfs: preallocation transactions do not need to be synchronou

To: Dave Chinner <david@xxxxxxxxxxxxx>
Subject: Re: [PATCH] xfs: preallocation transactions do not need to be synchronous
From: Christoph Hellwig <hch@xxxxxxxxxxxxx>
Date: Thu, 17 Mar 2011 10:23:13 -0400
Cc: xfs@xxxxxxxxxxx
In-reply-to: <20110310073634.GA25374@xxxxxxxxxxxxx>
References: <1299713852-7663-1-git-send-email-david@xxxxxxxxxxxxx> <20110310073634.GA25374@xxxxxxxxxxxxx>
User-agent: Mutt/1.5.21 (2010-09-15)
On Thu, Mar 10, 2011 at 02:36:34AM -0500, Christoph Hellwig wrote:
> I had the same patch in my queue, but it turns out even that isn't
> enough for good performance when used fallocate in the fast path (e.g.
> recent Samba).  I'll send a more comprehensive optimization that
> includes this soon.
> 
> I also refactored the whole area to make the various flags more
> sensible.

Given that 2.6.38 is out now and I won't have my series ready in time
I'm fine with putting your minimal version in for now, I'll port mine
on top.

Reviewed-by: Christoph Hellwig <hch@xxxxxx>

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