From: john.hubbard@gmail.com To: Andrew Morton <akpm@linux-foundation.org> Cc: "Alexander Viro" <viro@zeniv.linux.org.uk>, "Björn Töpel" <bjorn.topel@intel.com>, "Boaz Harrosh" <boaz@plexistor.com>, "Christoph Hellwig" <hch@lst.de>, "Daniel Vetter" <daniel@ffwll.ch>, "Dan Williams" <dan.j.williams@intel.com>, "Dave Chinner" <david@fromorbit.com>, "David Airlie" <airlied@linux.ie>, "David S . Miller" <davem@davemloft.net>, "Ilya Dryomov" <idryomov@gmail.com>, "Jan Kara" <jack@suse.cz>, "Jason Gunthorpe" <jgg@ziepe.ca>, "Jens Axboe" <axboe@kernel.dk>, "Jérôme Glisse" <jglisse@redhat.com>, "Johannes Thumshirn" <jthumshirn@suse.de>, "Magnus Karlsson" <magnus.karlsson@intel.com>, "Matthew Wilcox" <willy@infradead.org>, "Miklos Szeredi" <miklos@szeredi.hu>, "Ming Lei" <ming.lei@redhat.com>, "Sage Weil" <sage@redhat.com>, "Santosh Shilimkar" <santosh.shilimkar@oracle.com>, "Yan Zheng" <zyan@redhat.com>, netdev@vger.kernel.org, dri-devel@lists.freedesktop.org, linux-mm@kvack.org, linux-rdma@vger.kernel.org, bpf@vger.kernel.org, LKML <linux-kernel@vger.kernel.org>, "John Hubbard" <jhubbard@nvidia.com> Subject: [PATCH 0/3] introduce __put_user_pages(), convert a few call sites Date: Mon, 22 Jul 2019 15:34:12 -0700 [thread overview] Message-ID: <20190722223415.13269-1-jhubbard@nvidia.com> (raw) From: John Hubbard <jhubbard@nvidia.com> As discussed in [1] just now, this adds a more capable variation of put_user_pages() to the API set, and uses it to simplify both the main implementation, and (especially) the call sites. Thanks to Christoph for the simplifying ideas, and Matthew for (again) recommending an enum in the API. Matthew, I seem to recall you asked for enums before this, so I'm sorry it took until now for me to add them. :) The new __put_user_pages() takes an enum that handles the various combinations of needing to call set_page_dirty() or set_page_dirty_lock(), before calling put_user_page(). I'm using the same CC list as in [1], even though IB is no longer included in the series. That's everyone can see what the end result turns out to be. Notes about the remaining patches to come: There are about 50+ patches in my tree [2], and I'll be sending out the remaining ones in a few more groups: * The block/bio related changes (Jerome mostly wrote those, but I've had to move stuff around extensively, and add a little code) * mm/ changes * other subsystem patches * an RFC that shows the current state of the tracking patch set. That can only be applied after all call sites are converted, but it's good to get an early look at it. This is part a tree-wide conversion, as described in commit fc1d8e7cca2d ("mm: introduce put_user_page*(), placeholder versions"). [1] https://lore.kernel.org/r/20190722093355.GB29538@lst.de [2] https://github.com/johnhubbard/linux/tree/gup_dma_core John Hubbard (3): mm/gup: introduce __put_user_pages() drivers/gpu/drm/via: convert put_page() to put_user_page*() net/xdp: convert put_page() to put_user_page*() drivers/gpu/drm/via/via_dmablit.c | 11 +-- include/linux/mm.h | 58 +++++++++++- mm/gup.c | 149 +++++++++++++++--------------- net/xdp/xdp_umem.c | 9 +- 4 files changed, 135 insertions(+), 92 deletions(-) -- 2.22.0
WARNING: multiple messages have this Message-ID
From: john.hubbard@gmail.com To: Andrew Morton <akpm@linux-foundation.org> Cc: "Alexander Viro" <viro@zeniv.linux.org.uk>, "Björn Töpel" <bjorn.topel@intel.com>, "Boaz Harrosh" <boaz@plexistor.com>, "Christoph Hellwig" <hch@lst.de>, "Daniel Vetter" <daniel@ffwll.ch>, "Dan Williams" <dan.j.williams@intel.com>, "Dave Chinner" <david@fromorbit.com>, "David Airlie" <airlied@linux.ie>, "David S . Miller" <davem@davemloft.net>, "Ilya Dryomov" <idryomov@gmail.com>, "Jan Kara" <jack@suse.cz>, "Jason Gunthorpe" <jgg@ziepe.ca>, "Jens Axboe" <axboe@kernel.dk>, "Jérôme Glisse" <jglisse@redhat.com>, "Johannes Thumshirn" <jthumshirn@suse.de>, "Magnus Karlsson" <magnus.karlsson@intel.com>, "Matthew Wilcox" <willy@infradead.org>, "Miklos Szeredi" <miklos@szeredi.hu>, "Ming Lei" <ming.lei@redhat.com>, "Sage Weil" <sage@redhat.com> Subject: [PATCH 0/3] introduce __put_user_pages(), convert a few call sites Date: Mon, 22 Jul 2019 15:34:12 -0700 [thread overview] Message-ID: <20190722223415.13269-1-jhubbard@nvidia.com> (raw) From: John Hubbard <jhubbard@nvidia.com> As discussed in [1] just now, this adds a more capable variation of put_user_pages() to the API set, and uses it to simplify both the main implementation, and (especially) the call sites. Thanks to Christoph for the simplifying ideas, and Matthew for (again) recommending an enum in the API. Matthew, I seem to recall you asked for enums before this, so I'm sorry it took until now for me to add them. :) The new __put_user_pages() takes an enum that handles the various combinations of needing to call set_page_dirty() or set_page_dirty_lock(), before calling put_user_page(). I'm using the same CC list as in [1], even though IB is no longer included in the series. That's everyone can see what the end result turns out to be. Notes about the remaining patches to come: There are about 50+ patches in my tree [2], and I'll be sending out the remaining ones in a few more groups: * The block/bio related changes (Jerome mostly wrote those, but I've had to move stuff around extensively, and add a little code) * mm/ changes * other subsystem patches * an RFC that shows the current state of the tracking patch set. That can only be applied after all call sites are converted, but it's good to get an early look at it. This is part a tree-wide conversion, as described in commit fc1d8e7cca2d ("mm: introduce put_user_page*(), placeholder versions"). [1] https://lore.kernel.org/r/20190722093355.GB29538@lst.de [2] https://github.com/johnhubbard/linux/tree/gup_dma_core John Hubbard (3): mm/gup: introduce __put_user_pages() drivers/gpu/drm/via: convert put_page() to put_user_page*() net/xdp: convert put_page() to put_user_page*() drivers/gpu/drm/via/via_dmablit.c | 11 +-- include/linux/mm.h | 58 +++++++++++- mm/gup.c | 149 +++++++++++++++--------------- net/xdp/xdp_umem.c | 9 +- 4 files changed, 135 insertions(+), 92 deletions(-) -- 2.22.0
next reply other threads:[~2019-07-22 22:34 UTC|newest] Thread overview: 23+ messages / expand[flat|nested] mbox.gz Atom feed top 2019-07-22 22:34 john.hubbard [this message] 2019-07-22 22:34 ` john.hubbard 2019-07-22 22:34 ` [PATCH 1/3] mm/gup: introduce __put_user_pages() john.hubbard 2019-07-22 22:34 ` john.hubbard 2019-07-23 5:53 ` Christoph Hellwig 2019-07-23 6:33 ` John Hubbard 2019-07-23 6:33 ` John Hubbard 2019-07-23 15:36 ` Christoph Hellwig 2019-07-23 15:36 ` Christoph Hellwig 2019-07-22 22:34 ` [PATCH 2/3] drivers/gpu/drm/via: convert put_page() to put_user_page*() john.hubbard 2019-07-22 22:34 ` john.hubbard 2019-07-22 22:34 ` [PATCH 3/3] net/xdp: " john.hubbard 2019-07-22 22:34 ` john.hubbard 2019-07-23 0:25 ` Ira Weiny 2019-07-23 0:25 ` Ira Weiny 2019-07-23 4:41 ` John Hubbard 2019-07-23 4:41 ` John Hubbard 2019-07-23 12:47 ` Jason Gunthorpe 2019-07-23 12:47 ` Jason Gunthorpe 2019-07-23 18:06 ` Ira Weiny 2019-07-23 18:06 ` Ira Weiny 2019-07-23 23:24 ` John Hubbard 2019-07-23 23:24 ` 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=20190722223415.13269-1-jhubbard@nvidia.com \ --to=john.hubbard@gmail.com \ --cc=airlied@linux.ie \ --cc=akpm@linux-foundation.org \ --cc=axboe@kernel.dk \ --cc=bjorn.topel@intel.com \ --cc=boaz@plexistor.com \ --cc=bpf@vger.kernel.org \ --cc=dan.j.williams@intel.com \ --cc=daniel@ffwll.ch \ --cc=davem@davemloft.net \ --cc=david@fromorbit.com \ --cc=dri-devel@lists.freedesktop.org \ --cc=hch@lst.de \ --cc=idryomov@gmail.com \ --cc=jack@suse.cz \ --cc=jgg@ziepe.ca \ --cc=jglisse@redhat.com \ --cc=jhubbard@nvidia.com \ --cc=jthumshirn@suse.de \ --cc=linux-kernel@vger.kernel.org \ --cc=linux-mm@kvack.org \ --cc=linux-rdma@vger.kernel.org \ --cc=magnus.karlsson@intel.com \ --cc=miklos@szeredi.hu \ --cc=ming.lei@redhat.com \ --cc=netdev@vger.kernel.org \ --cc=sage@redhat.com \ --cc=santosh.shilimkar@oracle.com \ --cc=viro@zeniv.linux.org.uk \ --cc=willy@infradead.org \ --cc=zyan@redhat.com \ --subject='Re: [PATCH 0/3] introduce __put_user_pages(), convert a few call sites' \ /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
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.