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

From: John Hubbard
Date: Mon Oct 08 2018 - 16:59:07 EST


On 10/8/18 1:56 PM, Jason Gunthorpe wrote:
> 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..
>

It needs Andrew Morton (+CC) and preferably also Michal Hocko (already on CC).

thanks,
--
John Hubbard
NVIDIA