From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jan Kara Subject: Re: [PATCH 2/4] mm: introduce put_user_page(), placeholder version Date: Wed, 3 Oct 2018 18:22:37 +0200 Message-ID: <20181003162237.GH24030@quack2.suse.cz> References: <20180928053949.5381-1-jhubbard@nvidia.com> <20180928053949.5381-4-jhubbard@nvidia.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: <20180928053949.5381-4-jhubbard@nvidia.com> Sender: linux-kernel-owner@vger.kernel.org To: john.hubbard@gmail.com Cc: Matthew Wilcox , Michal Hocko , Christopher Lameter , Jason Gunthorpe , Dan Williams , Jan Kara , Al Viro , linux-mm@kvack.org, LKML , linux-rdma , linux-fsdevel@vger.kernel.org, John Hubbard List-Id: linux-rdma@vger.kernel.org On Thu 27-09-18 22:39:48, john.hubbard@gmail.com wrote: > From: John Hubbard > > Introduces put_user_page(), which simply calls put_page(). > This provides a way to update all get_user_pages*() callers, > so that they call put_user_page(), instead of put_page(). > > Also adds release_user_pages(), a drop-in replacement for > release_pages(). This is intended to be easily grep-able, > for later performance improvements, since release_user_pages > is not batched like release_pages() is, and is significantly > slower. A small nit but can we maybe call this put_user_pages() for symmetry with put_user_page()? I don't really care too much but it would look natural to me. Honza -- Jan Kara SUSE Labs, CR