Commit 7e339128 authored by Andrea Arcangeli's avatar Andrea Arcangeli Committed by Linus Torvalds

mm: gup: use get_user_pages_unlocked

This allows those get_user_pages calls to pass FAULT_FLAG_ALLOW_RETRY to
the page fault in order to release the mmap_sem during the I/O.
Signed-off-by: default avatarAndrea Arcangeli <aarcange@redhat.com>
Reviewed-by: default avatarKirill A. Shutemov <kirill.shutemov@linux.intel.com>
Cc: Andres Lagar-Cavilla <andreslc@google.com>
Cc: Peter Feiner <pfeiner@google.com>
Signed-off-by: default avatarAndrew Morton <akpm@linux-foundation.org>
Signed-off-by: default avatarLinus Torvalds <torvalds@linux-foundation.org>
parent a7b78075
...@@ -124,10 +124,8 @@ int ivtv_udma_setup(struct ivtv *itv, unsigned long ivtv_dest_addr, ...@@ -124,10 +124,8 @@ int ivtv_udma_setup(struct ivtv *itv, unsigned long ivtv_dest_addr,
} }
/* Get user pages for DMA Xfer */ /* Get user pages for DMA Xfer */
down_read(&current->mm->mmap_sem); err = get_user_pages_unlocked(current, current->mm,
err = get_user_pages(current, current->mm, user_dma.uaddr, user_dma.page_count, 0, 1, dma->map);
user_dma.uaddr, user_dma.page_count, 0, 1, dma->map, NULL);
up_read(&current->mm->mmap_sem);
if (user_dma.page_count != err) { if (user_dma.page_count != err) {
IVTV_DEBUG_WARN("failed to map user pages, returned %d instead of %d\n", IVTV_DEBUG_WARN("failed to map user pages, returned %d instead of %d\n",
......
...@@ -4551,18 +4551,15 @@ static int sgl_map_user_pages(struct st_buffer *STbp, ...@@ -4551,18 +4551,15 @@ static int sgl_map_user_pages(struct st_buffer *STbp,
return -ENOMEM; return -ENOMEM;
/* Try to fault in all of the necessary pages */ /* Try to fault in all of the necessary pages */
down_read(&current->mm->mmap_sem);
/* rw==READ means read from drive, write into memory area */ /* rw==READ means read from drive, write into memory area */
res = get_user_pages( res = get_user_pages_unlocked(
current, current,
current->mm, current->mm,
uaddr, uaddr,
nr_pages, nr_pages,
rw == READ, rw == READ,
0, /* don't force */ 0, /* don't force */
pages, pages);
NULL);
up_read(&current->mm->mmap_sem);
/* Errors and no page mapped should return here */ /* Errors and no page mapped should return here */
if (res < nr_pages) if (res < nr_pages)
......
...@@ -686,10 +686,8 @@ static ssize_t pvr2fb_write(struct fb_info *info, const char *buf, ...@@ -686,10 +686,8 @@ static ssize_t pvr2fb_write(struct fb_info *info, const char *buf,
if (!pages) if (!pages)
return -ENOMEM; return -ENOMEM;
down_read(&current->mm->mmap_sem); ret = get_user_pages_unlocked(current, current->mm, (unsigned long)buf,
ret = get_user_pages(current, current->mm, (unsigned long)buf, nr_pages, WRITE, 0, pages);
nr_pages, WRITE, 0, pages, NULL);
up_read(&current->mm->mmap_sem);
if (ret < nr_pages) { if (ret < nr_pages) {
nr_pages = ret; nr_pages = ret;
......
...@@ -99,11 +99,8 @@ static int process_vm_rw_single_vec(unsigned long addr, ...@@ -99,11 +99,8 @@ static int process_vm_rw_single_vec(unsigned long addr,
size_t bytes; size_t bytes;
/* Get the pages we're interested in */ /* Get the pages we're interested in */
down_read(&mm->mmap_sem); pages = get_user_pages_unlocked(task, mm, pa, pages,
pages = get_user_pages(task, mm, pa, pages, vm_write, 0, process_pages);
vm_write, 0, process_pages, NULL);
up_read(&mm->mmap_sem);
if (pages <= 0) if (pages <= 0)
return -EFAULT; return -EFAULT;
......
...@@ -23,17 +23,15 @@ struct page **ceph_get_direct_page_vector(const void __user *data, ...@@ -23,17 +23,15 @@ struct page **ceph_get_direct_page_vector(const void __user *data,
if (!pages) if (!pages)
return ERR_PTR(-ENOMEM); return ERR_PTR(-ENOMEM);
down_read(&current->mm->mmap_sem);
while (got < num_pages) { while (got < num_pages) {
rc = get_user_pages(current, current->mm, rc = get_user_pages_unlocked(current, current->mm,
(unsigned long)data + ((unsigned long)got * PAGE_SIZE), (unsigned long)data + ((unsigned long)got * PAGE_SIZE),
num_pages - got, write_page, 0, pages + got, NULL); num_pages - got, write_page, 0, pages + got);
if (rc < 0) if (rc < 0)
break; break;
BUG_ON(rc == 0); BUG_ON(rc == 0);
got += rc; got += rc;
} }
up_read(&current->mm->mmap_sem);
if (rc < 0) if (rc < 0)
goto fail; goto fail;
return pages; return pages;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment