FOLL_FORCE is really only for debugger access. As we unpin the pinned pages
using unpin_user_pages_dirty_lock(true), the assumption is that all these
pages are writable.

FOLL_FORCE in this case seems to be a legacy leftover. Let's just remove
it.

Cc: Dennis Dalessandro <dennis.dalessan...@cornelisnetworks.com>
Cc: Jason Gunthorpe <j...@ziepe.ca>
Cc: Leon Romanovsky <l...@kernel.org>
Signed-off-by: David Hildenbrand <da...@redhat.com>
---
 drivers/infiniband/hw/qib/qib_user_pages.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/infiniband/hw/qib/qib_user_pages.c 
b/drivers/infiniband/hw/qib/qib_user_pages.c
index f4b5f05058e4..f693bc753b6b 100644
--- a/drivers/infiniband/hw/qib/qib_user_pages.c
+++ b/drivers/infiniband/hw/qib/qib_user_pages.c
@@ -110,7 +110,7 @@ int qib_get_user_pages(unsigned long start_page, size_t 
num_pages,
        for (got = 0; got < num_pages; got += ret) {
                ret = pin_user_pages(start_page + got * PAGE_SIZE,
                                     num_pages - got,
-                                    FOLL_LONGTERM | FOLL_WRITE | FOLL_FORCE,
+                                    FOLL_LONGTERM | FOLL_WRITE,
                                     p + got, NULL);
                if (ret < 0) {
                        mmap_read_unlock(current->mm);
-- 
2.38.1

Reply via email to