| To: | Christoph Hellwig <hch@xxxxxx> |
|---|---|
| Subject: | Re: vfs/xfs: directio updates to ease COW handling V2 |
| From: | Jeff Moyer <jmoyer@xxxxxxxxxx> |
| Date: | Wed, 03 Feb 2016 14:43:02 -0500 |
| Cc: | linux-fsdevel@xxxxxxxxxxxxxxx, xfs@xxxxxxxxxxx, linux-ext4@xxxxxxxxxxxxxxx, ocfs2-devel@xxxxxxxxxxxxxx |
| Delivered-to: | xfs@xxxxxxxxxxx |
| In-reply-to: | <1454524816-11392-1-git-send-email-hch@xxxxxx> (Christoph Hellwig's message of "Wed, 3 Feb 2016 19:40:13 +0100") |
| References: | <1454524816-11392-1-git-send-email-hch@xxxxxx> |
| User-agent: | Gnus/5.13 (Gnus v5.13) Emacs/24.3 (gnu/linux) |
Hi, Christoph, Can you explain a bit what you mean by easing COW handling? Whenever I see COW referenced near DIO, my mind always turns to g_u_p vs. fork. Thanks! Jeff Christoph Hellwig <hch@xxxxxx> writes: > See http://www.infradead.org/rpr.html > > The first patch ensures ->end_io is always called for direct I/O requests > that pass it in, even if there was a zero length write, or if an error > occured. The existing users have been updated to ignore it, but XFS > will make use of it in the future, and a comment in ext4 suggests it > might be useful for it as well. > > The other two simplify the XFS direct I/O code. > > Changes since V1: > - allow ->end_io to return errors > - a comment spelling fix > > > -- > To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in > the body of a message to majordomo@xxxxxxxxxxxxxxx > More majordomo info at http://vger.kernel.org/majordomo-info.html |
| <Prev in Thread] | Current Thread | [Next in Thread> |
|---|---|---|
| ||
| Previous by Date: | [PATCH 1/3] direct-io: always call ->end_io if non-NULL, Christoph Hellwig |
|---|---|
| Next by Date: | Re: [PATCH 1/3] direct-io: always call ->end_io if non-NULL, Darrick J. Wong |
| Previous by Thread: | Re: [PATCH 1/3] direct-io: always call ->end_io if non-NULL, Darrick J. Wong |
| Next by Thread: | Re: vfs/xfs: directio updates to ease COW handling V2, Darrick J. Wong |
| Indexes: | [Date] [Thread] [Top] [All Lists] |