All of lore.kernel.org
 help / color / mirror / Atom feed
From: David Howells <dhowells@redhat.com>
To: Jens Axboe <axboe@kernel.dk>, Al Viro <viro@zeniv.linux.org.uk>,
	Christoph Hellwig <hch@infradead.org>
Cc: David Howells <dhowells@redhat.com>,
	Matthew Wilcox <willy@infradead.org>, Jan Kara <jack@suse.cz>,
	Jeff Layton <jlayton@kernel.org>,
	David Hildenbrand <david@redhat.com>,
	Jason Gunthorpe <jgg@nvidia.com>,
	Logan Gunthorpe <logang@deltatee.com>,
	Hillf Danton <hdanton@sina.com>,
	Christian Brauner <brauner@kernel.org>,
	Linus Torvalds <torvalds@linux-foundation.org>,
	linux-fsdevel@vger.kernel.org, linux-block@vger.kernel.org,
	linux-kernel@vger.kernel.org, linux-mm@kvack.org,
	Christoph Hellwig <hch@lst.de>,
	linux-erofs@lists.ozlabs.org, linux-ext4@vger.kernel.org,
	linux-xfs@vger.kernel.org
Subject: [PATCH v20 05/32] splice: Make splice from a DAX file use direct_splice_read()
Date: Fri, 19 May 2023 08:40:20 +0100	[thread overview]
Message-ID: <20230519074047.1739879-6-dhowells@redhat.com> (raw)
In-Reply-To: <20230519074047.1739879-1-dhowells@redhat.com>

Make a read splice from a DAX file go directly to direct_splice_read() to
do the reading as filemap_splice_read() is unlikely to find any pagecache
to splice.

I think this affects only erofs, Ext2, Ext4, fuse and XFS.

Signed-off-by: David Howells <dhowells@redhat.com>
cc: Christoph Hellwig <hch@lst.de>
cc: Al Viro <viro@zeniv.linux.org.uk>
cc: Jens Axboe <axboe@kernel.dk>
cc: linux-erofs@lists.ozlabs.org
cc: linux-ext4@vger.kernel.org
cc: linux-fsdevel@vger.kernel.org
cc: linux-xfs@vger.kernel.org
cc: linux-fsdevel@vger.kernel.org
cc: linux-block@vger.kernel.org
cc: linux-mm@kvack.org
---
 fs/splice.c | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/fs/splice.c b/fs/splice.c
index 1e0b7c7038b5..7b818b5b18d4 100644
--- a/fs/splice.c
+++ b/fs/splice.c
@@ -421,6 +421,11 @@ ssize_t generic_file_splice_read(struct file *in, loff_t *ppos,
 	struct kiocb kiocb;
 	int ret;
 
+#ifdef CONFIG_FS_DAX
+	if (IS_DAX(in->f_mapping->host))
+		return direct_splice_read(in, ppos, pipe, len, flags);
+#endif
+
 	iov_iter_pipe(&to, ITER_DEST, pipe, len);
 	init_sync_kiocb(&kiocb, in);
 	kiocb.ki_pos = *ppos;


WARNING: multiple messages have this Message-ID (diff)
From: David Howells <dhowells@redhat.com>
To: Jens Axboe <axboe@kernel.dk>, Al Viro <viro@zeniv.linux.org.uk>,
	Christoph Hellwig <hch@infradead.org>
Cc: linux-erofs@lists.ozlabs.org, linux-block@vger.kernel.org,
	Hillf Danton <hdanton@sina.com>, Jan Kara <jack@suse.cz>,
	linux-xfs@vger.kernel.org, David Hildenbrand <david@redhat.com>,
	Linus Torvalds <torvalds@linux-foundation.org>,
	Jeff Layton <jlayton@kernel.org>,
	Christian Brauner <brauner@kernel.org>,
	Matthew Wilcox <willy@infradead.org>,
	linux-kernel@vger.kernel.org, David Howells <dhowells@redhat.com>,
	linux-mm@kvack.org, Jason Gunthorpe <jgg@nvidia.com>,
	linux-fsdevel@vger.kernel.org, linux-ext4@vger.kernel.org,
	Logan Gunthorpe <logang@deltatee.com>,
	Christoph Hellwig <hch@lst.de>
Subject: [PATCH v20 05/32] splice: Make splice from a DAX file use direct_splice_read()
Date: Fri, 19 May 2023 08:40:20 +0100	[thread overview]
Message-ID: <20230519074047.1739879-6-dhowells@redhat.com> (raw)
In-Reply-To: <20230519074047.1739879-1-dhowells@redhat.com>

Make a read splice from a DAX file go directly to direct_splice_read() to
do the reading as filemap_splice_read() is unlikely to find any pagecache
to splice.

I think this affects only erofs, Ext2, Ext4, fuse and XFS.

Signed-off-by: David Howells <dhowells@redhat.com>
cc: Christoph Hellwig <hch@lst.de>
cc: Al Viro <viro@zeniv.linux.org.uk>
cc: Jens Axboe <axboe@kernel.dk>
cc: linux-erofs@lists.ozlabs.org
cc: linux-ext4@vger.kernel.org
cc: linux-fsdevel@vger.kernel.org
cc: linux-xfs@vger.kernel.org
cc: linux-fsdevel@vger.kernel.org
cc: linux-block@vger.kernel.org
cc: linux-mm@kvack.org
---
 fs/splice.c | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/fs/splice.c b/fs/splice.c
index 1e0b7c7038b5..7b818b5b18d4 100644
--- a/fs/splice.c
+++ b/fs/splice.c
@@ -421,6 +421,11 @@ ssize_t generic_file_splice_read(struct file *in, loff_t *ppos,
 	struct kiocb kiocb;
 	int ret;
 
+#ifdef CONFIG_FS_DAX
+	if (IS_DAX(in->f_mapping->host))
+		return direct_splice_read(in, ppos, pipe, len, flags);
+#endif
+
 	iov_iter_pipe(&to, ITER_DEST, pipe, len);
 	init_sync_kiocb(&kiocb, in);
 	kiocb.ki_pos = *ppos;


  parent reply	other threads:[~2023-05-19  7:42 UTC|newest]

Thread overview: 70+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-05-19  7:40 [PATCH v20 00/32] splice, block: Use page pinning and kill ITER_PIPE David Howells
2023-05-19  7:40 ` [PATCH v20 01/32] splice: Fix filemap of a blockdev David Howells
2023-05-19  8:06   ` Christoph Hellwig
2023-05-19  9:11   ` David Howells
2023-05-19  7:40 ` [PATCH v20 02/32] splice: Clean up direct_splice_read() a bit David Howells
2023-05-19  7:40 ` [PATCH v20 03/32] splice: Make direct_read_splice() limit to eof where appropriate David Howells
2023-05-19  8:09   ` Christoph Hellwig
2023-05-19  8:43   ` David Howells
2023-05-19  8:47     ` Christoph Hellwig
2023-05-19 22:27     ` David Howells
2023-05-20  3:54       ` Christoph Hellwig
2023-05-19 16:31   ` Linus Torvalds
2023-05-19 16:47   ` David Howells
2023-05-19 17:37     ` Linus Torvalds
2023-05-19  7:40 ` [PATCH v20 04/32] splice: Make do_splice_to() generic and export it David Howells
2023-05-19  7:40 ` David Howells [this message]
2023-05-19  7:40   ` [PATCH v20 05/32] splice: Make splice from a DAX file use direct_splice_read() David Howells
2023-05-19  8:10   ` Christoph Hellwig
2023-05-19  8:10     ` Christoph Hellwig
2023-05-19  8:48   ` David Howells
2023-05-19  8:48     ` David Howells
2023-05-19  7:40 ` [PATCH v20 06/32] shmem: Implement splice-read David Howells
2023-05-19  7:40 ` [PATCH v20 07/32] overlayfs: " David Howells
2023-05-19  7:40 ` [PATCH v20 08/32] coda: " David Howells
2023-05-19  7:40 ` [PATCH v20 09/32] tty, proc, kernfs, random: Use direct_splice_read() David Howells
2023-05-19  8:12   ` Christoph Hellwig
2023-05-19 16:22     ` Linus Torvalds
2023-05-19  7:40 ` [PATCH v20 10/32] net: Make sock_splice_read() use direct_splice_read() by default David Howells
2023-05-19  7:40 ` [PATCH v20 11/32] 9p: Add splice_read stub David Howells
2023-05-19  7:40 ` [PATCH v20 12/32] afs: Provide a splice-read stub David Howells
2023-05-19  7:40 ` [PATCH v20 13/32] ceph: " David Howells
2023-05-19  8:40   ` Xiubo Li
2023-05-19  9:24   ` David Howells
2023-05-22  1:53     ` Xiubo Li
2023-05-19  7:40 ` [PATCH v20 14/32] ecryptfs: " David Howells
2023-05-19  7:40 ` [PATCH v20 15/32] ext4: " David Howells
2023-05-19  7:40 ` [PATCH v20 16/32] f2fs: " David Howells
2023-05-19  7:40   ` [f2fs-dev] " David Howells
2023-05-19  7:40 ` [PATCH v20 17/32] nfs: " David Howells
2023-05-19  7:40 ` [PATCH v20 18/32] ntfs3: " David Howells
2023-05-19  7:40 ` [Ocfs2-devel] [PATCH v20 19/32] ocfs2: " David Howells via Ocfs2-devel
2023-05-19  7:40   ` David Howells
2023-05-19  7:40 ` [PATCH v20 20/32] orangefs: " David Howells
2023-05-19  7:40 ` [PATCH v20 21/32] xfs: " David Howells
2023-05-19  7:40 ` [PATCH v20 22/32] zonefs: " David Howells
2023-05-19  7:40 ` [PATCH v20 23/32] splice: Convert trace/seq to use direct_splice_read() David Howells
2023-05-22 14:29   ` Steven Rostedt
2023-05-22 14:50   ` David Howells
2023-05-22 17:42     ` Linus Torvalds
2023-05-22 18:38       ` Steven Rostedt
2023-05-19  7:40 ` [PATCH v20 24/32] splice: Do splice read from a file without using ITER_PIPE David Howells
2023-05-19  7:40 ` [PATCH v20 25/32] cifs: Use generic_file_splice_read() David Howells
2023-05-19  7:40 ` [PATCH v20 26/32] iov_iter: Kill ITER_PIPE David Howells
2023-05-19  7:40 ` [PATCH v20 27/32] iomap: Don't get an reference on ZERO_PAGE for direct I/O block zeroing David Howells
2023-05-19  7:40 ` [PATCH v20 28/32] block: Fix bio_flagged() so that gcc can better optimise it David Howells
2023-05-19  7:40 ` [PATCH v20 29/32] block: Replace BIO_NO_PAGE_REF with BIO_PAGE_REFFED with inverted logic David Howells
2023-05-20  1:26   ` Kent Overstreet
2023-05-20  3:56     ` Christoph Hellwig
2023-05-20  4:13       ` Kent Overstreet
2023-05-20  4:17         ` Christoph Hellwig
2023-05-20  5:52           ` Kent Overstreet
2023-05-20  8:40   ` David Howells
2023-05-19  7:40 ` [PATCH v20 30/32] block: Add BIO_PAGE_PINNED and associated infrastructure David Howells
2023-05-19  7:40 ` [PATCH v20 31/32] block: Convert bio_iov_iter_get_pages to use iov_iter_extract_pages David Howells
2023-05-19  7:40 ` [PATCH v20 32/32] block: convert bio_map_user_iov " David Howells
2023-05-19  7:49 ` [PATCH] iov_iter: Add automatic-alloc for ITER_BVEC and use in direct_splice_read() David Howells
2023-05-19  8:18   ` Christoph Hellwig
2023-05-19  8:06 ` [PATCH v20 00/32] splice, block: Use page pinning and kill ITER_PIPE Christoph Hellwig
2023-06-14  0:55 ` Mike Marshall
2023-06-16 11:38 ` David Howells

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20230519074047.1739879-6-dhowells@redhat.com \
    --to=dhowells@redhat.com \
    --cc=axboe@kernel.dk \
    --cc=brauner@kernel.org \
    --cc=david@redhat.com \
    --cc=hch@infradead.org \
    --cc=hch@lst.de \
    --cc=hdanton@sina.com \
    --cc=jack@suse.cz \
    --cc=jgg@nvidia.com \
    --cc=jlayton@kernel.org \
    --cc=linux-block@vger.kernel.org \
    --cc=linux-erofs@lists.ozlabs.org \
    --cc=linux-ext4@vger.kernel.org \
    --cc=linux-fsdevel@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-mm@kvack.org \
    --cc=linux-xfs@vger.kernel.org \
    --cc=logang@deltatee.com \
    --cc=torvalds@linux-foundation.org \
    --cc=viro@zeniv.linux.org.uk \
    --cc=willy@infradead.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.