From: John Hubbard <jhubbard@xxxxxxxxxx> Commit 63630f9a8d72 ("mm/gup: add make_dirty arg to put_user_pages_dirty_lock()") removed the only user of the local variable p. Finish the job by removing p as well. Fixes: 63630f9a8d72 ("mm/gup: add make_dirty arg to put_user_pages_dirty_lock()") Signed-off-by: John Hubbard <jhubbard@xxxxxxxxxx> --- Hi, This fixes the warning. Ideally this should be merged with the commit that it fixes, if that's still possible. Andrew, would you also like a fixed version of this patch posted as a new version of the 3-patch set that it came with? thanks, John Hubbard NVIDIA drivers/infiniband/sw/siw/siw_mem.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/drivers/infiniband/sw/siw/siw_mem.c b/drivers/infiniband/sw/siw/siw_mem.c index 358d440efa11..ab83a9cec562 100644 --- a/drivers/infiniband/sw/siw/siw_mem.c +++ b/drivers/infiniband/sw/siw/siw_mem.c @@ -63,8 +63,6 @@ struct siw_mem *siw_mem_id2obj(struct siw_device *sdev, int stag_index) static void siw_free_plist(struct siw_page_chunk *chunk, int num_pages, bool dirty) { - struct page **p = chunk->plist; - put_user_pages_dirty_lock(chunk->plist, num_pages, dirty); } -- 2.22.0