| 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> |
|---|---|---|
| ||
| Previous by Date: | Re: block allocations for the refcount btree, Brian Foster |
|---|---|
| Next by Date: | Re: [PATCH 2/8] xfs: remove xfs_cancel_ioend, Christoph Hellwig |
| Previous by Thread: | Re: [PATCH 0/8 v4] xfs: get rid of xfs_cluster_write, Dave Chinner |
| Next by Thread: | Re: [PATCH 0/8 v4] xfs: get rid of xfs_cluster_write, Dave Chinner |
| Indexes: | [Date] [Thread] [Top] [All Lists] |