From: Christoph Hellwig <hch@infradead.org> To: John Hubbard <jhubbard@nvidia.com> Cc: Christoph Hellwig <hch@infradead.org>, john.hubbard@gmail.com, Andrew Morton <akpm@linux-foundation.org>, Alexander Viro <viro@zeniv.linux.org.uk>, Anna Schumaker <anna.schumaker@netapp.com>, "David S . Miller" <davem@davemloft.net>, Dominique Martinet <asmadeus@codewreck.org>, Eric Van Hensbergen <ericvh@gmail.com>, Jason Gunthorpe <jgg@ziepe.ca>, Jason Wang <jasowang@redhat.com>, Jens Axboe <axboe@kernel.dk>, Latchesar Ionkov <lucho@ionkov.net>, "Michael S . Tsirkin" <mst@redhat.com>, Miklos Szeredi <miklos@szeredi.hu>, Trond Myklebust <trond.myklebust@hammerspace.com>, Christoph Hellwig <hch@lst.de>, Matthew Wilcox <willy@infradead.org>, linux-mm@kvack.org, LKML <linux-kernel@vger.kernel.org>, ceph-devel@vger.kernel.org, kvm@vger.kernel.org, linux-block@vger.kernel.org, linux-cifs@vger.kernel.org, linux-fsdevel@vger.kernel.org, linux-nfs@vger.kernel.org, linux-rdma@vger.kernel.org, netdev@vger.kernel.org, samba-technical@lists.samba.org, v9fs-developer@lists.sourceforge.net, virtualization@lists.linux-foundation.org Subject: Re: [PATCH 00/12] block/bio, fs: convert put_page() to put_user_page*() Date: Tue, 6 Aug 2019 23:34:48 -0700 [thread overview] Message-ID: <20190807063448.GA6002@infradead.org> (raw) In-Reply-To: <c35aa2bf-c830-9e57-78ca-9ce6fb6cb53b@nvidia.com> On Mon, Aug 05, 2019 at 03:54:35PM -0700, John Hubbard wrote: > On 7/23/19 11:17 PM, Christoph Hellwig wrote: > > On Tue, Jul 23, 2019 at 09:25:06PM -0700, john.hubbard@gmail.com wrote: > >> * Store, in the iov_iter, a "came from gup (get_user_pages)" parameter. > >> Then, use the new iov_iter_get_pages_use_gup() to retrieve it when > >> it is time to release the pages. That allows choosing between put_page() > >> and put_user_page*(). > >> > >> * Pass in one more piece of information to bio_release_pages: a "from_gup" > >> parameter. Similar use as above. > >> > >> * Change the block layer, and several file systems, to use > >> put_user_page*(). > > > > I think we can do this in a simple and better way. We have 5 ITER_* > > types. Of those ITER_DISCARD as the name suggests never uses pages, so > > we can skip handling it. ITER_PIPE is rejected іn the direct I/O path, > > which leaves us with three. > > > > Hi Christoph, > > Are you working on anything like this? I was hoping I could steer you towards it. But if you don't want to do it yourself I'll add it to my ever growing todo list. > Or on the put_user_bvec() idea? I have a prototype from two month ago: http://git.infradead.org/users/hch/misc.git/shortlog/refs/heads/gup-bvec but that only survived the most basic testing, so it'll need more work, which I'm not sure when I'll find time for.
WARNING: multiple messages have this Message-ID (diff)
From: Christoph Hellwig <hch@infradead.org> To: John Hubbard <jhubbard@nvidia.com> Cc: Christoph Hellwig <hch@infradead.org>, john.hubbard@gmail.com, Andrew Morton <akpm@linux-foundation.org>, Alexander Viro <viro@zeniv.linux.org.uk>, Anna Schumaker <anna.schumaker@netapp.com>, "David S . Miller" <davem@davemloft.net>, Dominique Martinet <asmadeus@codewreck.org>, Eric Van Hensbergen <ericvh@gmail.com>, Jason Gunthorpe <jgg@ziepe.ca>, Jason Wang <jasowang@redhat.com>, Jens Axboe <axboe@kernel.dk>, Latchesar Ionkov <lucho@ionkov.net>, "Michael S . Tsirkin" <mst@redhat.com>, Miklos Szeredi <miklos@szeredi.hu>, Trond Myklebust <trond.myklebust@hammerspace.com>, Christoph Hellwig <hch@lst.de>, Matthew Wilcox <willy@infradead.org>, linux-mm@kvack.org, LKML <linux-kernel@vger.kernel.org>, ceph-devel@vger.kernel.org, kvm@vger.kernel.org, linux-block@v Subject: Re: [PATCH 00/12] block/bio, fs: convert put_page() to put_user_page*() Date: Tue, 6 Aug 2019 23:34:48 -0700 [thread overview] Message-ID: <20190807063448.GA6002@infradead.org> (raw) In-Reply-To: <c35aa2bf-c830-9e57-78ca-9ce6fb6cb53b@nvidia.com> On Mon, Aug 05, 2019 at 03:54:35PM -0700, John Hubbard wrote: > On 7/23/19 11:17 PM, Christoph Hellwig wrote: > > On Tue, Jul 23, 2019 at 09:25:06PM -0700, john.hubbard@gmail.com wrote: > >> * Store, in the iov_iter, a "came from gup (get_user_pages)" parameter. > >> Then, use the new iov_iter_get_pages_use_gup() to retrieve it when > >> it is time to release the pages. That allows choosing between put_page() > >> and put_user_page*(). > >> > >> * Pass in one more piece of information to bio_release_pages: a "from_gup" > >> parameter. Similar use as above. > >> > >> * Change the block layer, and several file systems, to use > >> put_user_page*(). > > > > I think we can do this in a simple and better way. We have 5 ITER_* > > types. Of those ITER_DISCARD as the name suggests never uses pages, so > > we can skip handling it. ITER_PIPE is rejected іn the direct I/O path, > > which leaves us with three. > > > > Hi Christoph, > > Are you working on anything like this? I was hoping I could steer you towards it. But if you don't want to do it yourself I'll add it to my ever growing todo list. > Or on the put_user_bvec() idea? I have a prototype from two month ago: http://git.infradead.org/users/hch/misc.git/shortlog/refs/heads/gup-bvec but that only survived the most basic testing, so it'll need more work, which I'm not sure when I'll find time for.
next prev parent reply other threads:[~2019-08-07 6:35 UTC|newest] Thread overview: 75+ messages / expand[flat|nested] mbox.gz Atom feed top 2019-07-24 4:25 [PATCH 00/12] block/bio, fs: convert put_page() to put_user_page*() john.hubbard 2019-07-24 4:25 ` john.hubbard 2019-07-24 4:25 ` [PATCH 01/12] mm/gup: add make_dirty arg to put_user_pages_dirty_lock() john.hubbard 2019-07-24 4:25 ` john.hubbard 2019-07-24 4:25 ` john.hubbard 2019-07-24 4:25 ` [PATCH 02/12] iov_iter: add helper to test if an iter would use GUP v2 john.hubbard 2019-07-24 4:25 ` john.hubbard 2019-07-24 4:25 ` john.hubbard 2019-07-24 4:25 ` [PATCH 03/12] block: bio_release_pages: use flags arg instead of bool john.hubbard 2019-07-24 4:25 ` john.hubbard 2019-07-24 5:30 ` Christoph Hellwig 2019-07-24 5:30 ` Christoph Hellwig 2019-07-24 5:30 ` Christoph Hellwig 2019-07-29 20:57 ` Jerome Glisse 2019-07-29 20:57 ` Jerome Glisse 2019-07-30 10:25 ` Christoph Hellwig 2019-07-30 10:25 ` Christoph Hellwig 2019-07-30 15:57 ` Jerome Glisse 2019-07-30 15:57 ` Jerome Glisse 2019-07-30 15:57 ` Jerome Glisse 2019-08-01 8:20 ` Christoph Hellwig 2019-08-01 8:20 ` Christoph Hellwig 2019-08-01 8:20 ` Christoph Hellwig 2019-07-30 10:25 ` Christoph Hellwig 2019-07-29 20:57 ` Jerome Glisse 2019-07-24 4:25 ` john.hubbard 2019-07-24 4:25 ` [PATCH 04/12] block: bio_release_pages: convert put_page() to put_user_page*() john.hubbard 2019-07-24 4:25 ` john.hubbard 2019-07-24 4:25 ` john.hubbard 2019-07-24 4:25 ` [PATCH 05/12] block_dev: " john.hubbard 2019-07-24 4:25 ` john.hubbard 2019-07-24 4:25 ` john.hubbard 2019-07-24 4:25 ` [PATCH 06/12] fs/nfs: " john.hubbard 2019-07-24 4:25 ` john.hubbard 2019-07-24 4:25 ` john.hubbard 2019-07-24 4:25 ` [PATCH 07/12] vhost-scsi: " john.hubbard 2019-07-24 4:25 ` john.hubbard 2019-07-24 4:34 ` John Hubbard 2019-07-24 4:34 ` John Hubbard 2019-07-24 4:34 ` John Hubbard 2019-07-24 8:07 ` Michael S. Tsirkin 2019-07-24 8:07 ` Michael S. Tsirkin 2019-07-24 4:25 ` john.hubbard 2019-07-24 4:25 ` [PATCH 08/12] fs/cifs: " john.hubbard 2019-07-24 4:25 ` john.hubbard 2019-07-24 4:25 ` john.hubbard 2019-07-24 4:25 ` [PATCH 09/12] fs/fuse: " john.hubbard 2019-07-24 4:25 ` john.hubbard 2019-07-24 4:25 ` john.hubbard 2019-07-24 4:25 ` [PATCH 10/12] fs/ceph: " john.hubbard 2019-07-24 4:25 ` john.hubbard 2019-07-24 4:25 ` john.hubbard 2019-07-24 4:25 ` [PATCH 11/12] 9p/net: " john.hubbard 2019-07-24 4:25 ` john.hubbard 2019-07-24 4:25 ` john.hubbard 2019-07-24 4:25 ` [PATCH 12/12] fs/ceph: fix a build warning: returning a value from void function john.hubbard 2019-07-24 4:25 ` john.hubbard 2019-07-24 4:25 ` john.hubbard 2019-07-24 6:17 ` [PATCH 00/12] block/bio, fs: convert put_page() to put_user_page*() Christoph Hellwig 2019-07-24 6:17 ` Christoph Hellwig 2019-07-24 6:17 ` Christoph Hellwig 2019-07-24 23:23 ` John Hubbard 2019-07-24 23:23 ` John Hubbard 2019-08-05 22:54 ` John Hubbard 2019-08-05 22:54 ` John Hubbard 2019-08-07 6:34 ` Christoph Hellwig 2019-08-07 6:34 ` Christoph Hellwig [this message] 2019-08-07 6:34 ` Christoph Hellwig 2019-08-07 6:38 ` John Hubbard 2019-08-07 6:38 ` John Hubbard 2019-07-25 0:41 ` Bob Liu 2019-07-25 0:41 ` Bob Liu 2019-07-26 1:24 ` John Hubbard 2019-07-26 1:24 ` John Hubbard 2019-07-24 4:25 john.hubbard
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=20190807063448.GA6002@infradead.org \ --to=hch@infradead.org \ --cc=akpm@linux-foundation.org \ --cc=anna.schumaker@netapp.com \ --cc=asmadeus@codewreck.org \ --cc=axboe@kernel.dk \ --cc=ceph-devel@vger.kernel.org \ --cc=davem@davemloft.net \ --cc=ericvh@gmail.com \ --cc=hch@lst.de \ --cc=jasowang@redhat.com \ --cc=jgg@ziepe.ca \ --cc=jhubbard@nvidia.com \ --cc=john.hubbard@gmail.com \ --cc=kvm@vger.kernel.org \ --cc=linux-block@vger.kernel.org \ --cc=linux-cifs@vger.kernel.org \ --cc=linux-fsdevel@vger.kernel.org \ --cc=linux-kernel@vger.kernel.org \ --cc=linux-mm@kvack.org \ --cc=linux-nfs@vger.kernel.org \ --cc=linux-rdma@vger.kernel.org \ --cc=lucho@ionkov.net \ --cc=miklos@szeredi.hu \ --cc=mst@redhat.com \ --cc=netdev@vger.kernel.org \ --cc=samba-technical@lists.samba.org \ --cc=trond.myklebust@hammerspace.com \ --cc=v9fs-developer@lists.sourceforge.net \ --cc=viro@zeniv.linux.org.uk \ --cc=virtualization@lists.linux-foundation.org \ --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: linkBe 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.