xfs
[Top] [All Lists]

Re: [PATCH 0/8 v4] xfs: get rid of xfs_cluster_write

To: Dave Chinner <david@xxxxxxxxxxxxx>
Subject: Re: [PATCH 0/8 v4] xfs: get rid of xfs_cluster_write
From: Christoph Hellwig <hch@xxxxxxxxxxxxx>
Date: Thu, 11 Feb 2016 07:13:01 -0800
Cc: Christoph Hellwig <hch@xxxxxxxxxxxxx>, xfs@xxxxxxxxxxx
Delivered-to: xfs@xxxxxxxxxxx
In-reply-to: <20160210212550.GM14668@dastard>
References: <1455094043-9694-1-git-send-email-david@xxxxxxxxxxxxx> <20160210182538.GA7481@xxxxxxxxxxxxx> <20160210212550.GM14668@dastard>
User-agent: Mutt/1.5.24 (2015-08-30)
On Thu, Feb 11, 2016 at 08:25:51AM +1100, Dave Chinner wrote:
> I really like the idea, especially how using a bioset encapsulates
> the ioend and binds the life cycle to the bio. It also removes a
> heap of code, too.
> 
> We really do need some form of chaining here, though. If we don't,
> then we'll be doing unwritten extent of set file size transactions
> for ever 1MB bio completion instead of once for however large
> writepages can build an ioend.
> 
> I think this is definitely worth pursuing - are you going to get any
> time to work on this in the next couple of months, Christoph (i.e.
> to target the 4.7 merge window)?

Yes, I've just started working on this and would love to get it off
my table ASAP.  I was doing this ontop of your initial writeback
changes, so let me know if I should go back to that version, or
you think the last two patches will be ready in time as well?

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