| Message ID | 20190802022005.5117-27-jhubbard@nvidia.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | put_user_pages(): miscellaneous call sites | expand |
On Thu, Aug 01, 2019 at 07:19:57PM -0700, john.hubbard@gmail.com wrote: > From: John Hubbard <jhubbard@nvidia.com> > > For pages that were retained via get_user_pages*(), release those pages > via the new put_user_page*() routines, instead of via put_page() or > release_pages(). > > This is part a tree-wide conversion, as described in commit fc1d8e7cca2d > ("mm: introduce put_user_page*(), placeholder versions"). > > Cc: Dan Carpenter <dan.carpenter@oracle.com> > Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org> > Cc: Keith Busch <keith.busch@intel.com> > Cc: Kirill A. Shutemov <kirill.shutemov@linux.intel.com> > Cc: Michael S. Tsirkin <mst@redhat.com> > Cc: YueHaibing <yuehaibing@huawei.com> > Signed-off-by: John Hubbard <jhubbard@nvidia.com> Looks fine. Reviewed-by: Keith Busch <keith.busch@intel.com> > mm/gup_benchmark.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/mm/gup_benchmark.c b/mm/gup_benchmark.c > index 7dd602d7f8db..515ac8eeb6ee 100644 > --- a/mm/gup_benchmark.c > +++ b/mm/gup_benchmark.c > @@ -79,7 +79,7 @@ static int __gup_benchmark_ioctl(unsigned int cmd, > for (i = 0; i < nr_pages; i++) { > if (!pages[i]) > break; > - put_page(pages[i]); > + put_user_page(pages[i]); > } > end_time = ktime_get(); > gup->put_delta_usec = ktime_us_delta(end_time, start_time); > --
diff --git a/mm/gup_benchmark.c b/mm/gup_benchmark.c index 7dd602d7f8db..515ac8eeb6ee 100644 --- a/mm/gup_benchmark.c +++ b/mm/gup_benchmark.c @@ -79,7 +79,7 @@ static int __gup_benchmark_ioctl(unsigned int cmd, for (i = 0; i < nr_pages; i++) { if (!pages[i]) break; - put_page(pages[i]); + put_user_page(pages[i]); } end_time = ktime_get(); gup->put_delta_usec = ktime_us_delta(end_time, start_time);