hfi1: get rid of pointless access_ok()
authorAl Viro <viro@zeniv.linux.org.uk>
Fri, 8 May 2020 01:39:29 +0000 (21:39 -0400)
committerAl Viro <viro@zeniv.linux.org.uk>
Fri, 29 May 2020 15:06:32 +0000 (11:06 -0400)
pin_user_pages_fast() doesn't need that from its caller.
NB: only reachable from ->ioctl(), and only under USER_DS

Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
drivers/infiniband/hw/hfi1/user_exp_rcv.c

index 4da03f82347492001c5e0fb49af5393087be9b4b..f81ca20f4b693e1597fc1cd10d62a21acdf05a2f 100644 (file)
@@ -206,13 +206,6 @@ static int pin_rcv_pages(struct hfi1_filedata *fd, struct tid_user_buf *tidbuf)
                return -EINVAL;
        }
 
-       /* Verify that access is OK for the user buffer */
-       if (!access_ok((void __user *)vaddr,
-                      npages * PAGE_SIZE)) {
-               dd_dev_err(dd, "Fail vaddr %p, %u pages, !access_ok\n",
-                          (void *)vaddr, npages);
-               return -EFAULT;
-       }
        /* Allocate the array of struct page pointers needed for pinning */
        pages = kcalloc(npages, sizeof(*pages), GFP_KERNEL);
        if (!pages)