[Top] [All Lists]

Re: [PATCH 4/5] xfs: simplify the fallocate path

To: Dave Chinner <david@xxxxxxxxxxxxx>
Subject: Re: [PATCH 4/5] xfs: simplify the fallocate path
From: Christoph Hellwig <hch@xxxxxxxxxxxxx>
Date: Mon, 14 Oct 2013 00:30:47 -0700
Cc: Christoph Hellwig <hch@xxxxxxxxxxxxx>, xfs@xxxxxxxxxxx
Delivered-to: xfs@xxxxxxxxxxx
In-reply-to: <20131014050424.GI4446@dastard>
References: <20131012075503.370510641@xxxxxxxxxxxxxxxxxxxxxx> <20131012075640.467511096@xxxxxxxxxxxxxxxxxxxxxx> <20131014050424.GI4446@dastard>
User-agent: Mutt/1.5.21 (2010-09-15)
On Mon, Oct 14, 2013 at 04:04:24PM +1100, Dave Chinner wrote:
> Seems a bit clunky to do all this work when we've got to repeat most
> of it when when we call xfs_setattr_size() if the size has changed.
> Any thoughts on how we might reduce to a single transaction?

I tried that, but the helper becomes a complete mess of flag values for the
possible inode modifications.  You also pointed this out the last time
around.  If anyone can come up with a saner helper than I tired feel
free to send a patch on top.

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