Re: [PATCH v3 3/3] infiniband/mm: convert put_page() to put_user_page*()

From: Jason Gunthorpe
Date: Mon Oct 08 2018 - 16:56:07 EST


On Mon, Oct 08, 2018 at 01:37:35PM -0700, John Hubbard wrote:
> On 10/8/18 12:42 PM, Jason Gunthorpe wrote:
> > On Fri, Oct 05, 2018 at 07:49:49PM -0700, john.hubbard@xxxxxxxxx wrote:
> >> From: John Hubbard <jhubbard@xxxxxxxxxx>
> [...]
> >> drivers/infiniband/core/umem.c | 7 ++++---
> >> drivers/infiniband/core/umem_odp.c | 2 +-
> >> drivers/infiniband/hw/hfi1/user_pages.c | 11 ++++-------
> >> drivers/infiniband/hw/mthca/mthca_memfree.c | 6 +++---
> >> drivers/infiniband/hw/qib/qib_user_pages.c | 11 ++++-------
> >> drivers/infiniband/hw/qib/qib_user_sdma.c | 8 ++++----
> >> drivers/infiniband/hw/usnic/usnic_uiom.c | 7 ++++---
> >> 7 files changed, 24 insertions(+), 28 deletions(-)
> >
> > I have no issues with this, do you want this series to go through the
> > rdma tree? Otherwise:
> >
> > Acked-by: Jason Gunthorpe <jgg@xxxxxxxxxxxx>
> >
>
> The RDMA tree seems like a good path for this, yes, glad you suggested
> that.
>
> I'll post a v4 with the comment fix and the recent reviewed-by's, which
> should be ready for that. It's based on today's linux.git tree at the
> moment, but let me know if I should re-apply it to the RDMA tree.

I'm unclear who needs to ack the MM sections for us to take it to
RDMA?

Otherwise it is no problem..

Jason