Instead of creating another vm_fault structure, use the one passed to
__do_fault() for passing arguments into fault handler.

Reviewed-by: Ross Zwisler <ross.zwis...@linux.intel.com>
Signed-off-by: Jan Kara <j...@suse.cz>
---
 mm/memory.c | 25 ++++++++++---------------
 1 file changed, 10 insertions(+), 15 deletions(-)

diff --git a/mm/memory.c b/mm/memory.c
index 3b79eace8d23..8145dadb2645 100644
--- a/mm/memory.c
+++ b/mm/memory.c
@@ -2852,37 +2852,31 @@ static int __do_fault(struct vm_fault *vmf, struct page 
*cow_page,
                      struct page **page, void **entry)
 {
        struct vm_area_struct *vma = vmf->vma;
-       struct vm_fault vmf2;
        int ret;
 
-       vmf2.address = vmf->address;
-       vmf2.pgoff = vmf->pgoff;
-       vmf2.flags = vmf->flags;
-       vmf2.page = NULL;
-       vmf2.gfp_mask = __get_fault_gfp_mask(vma);
-       vmf2.cow_page = cow_page;
+       vmf->cow_page = cow_page;
 
-       ret = vma->vm_ops->fault(vma, &vmf2);
+       ret = vma->vm_ops->fault(vma, vmf);
        if (unlikely(ret & (VM_FAULT_ERROR | VM_FAULT_NOPAGE | VM_FAULT_RETRY)))
                return ret;
        if (ret & VM_FAULT_DAX_LOCKED) {
-               *entry = vmf2.entry;
+               *entry = vmf->entry;
                return ret;
        }
 
-       if (unlikely(PageHWPoison(vmf2.page))) {
+       if (unlikely(PageHWPoison(vmf->page))) {
                if (ret & VM_FAULT_LOCKED)
-                       unlock_page(vmf2.page);
-               put_page(vmf2.page);
+                       unlock_page(vmf->page);
+               put_page(vmf->page);
                return VM_FAULT_HWPOISON;
        }
 
        if (unlikely(!(ret & VM_FAULT_LOCKED)))
-               lock_page(vmf2.page);
+               lock_page(vmf->page);
        else
-               VM_BUG_ON_PAGE(!PageLocked(vmf2.page), vmf2.page);
+               VM_BUG_ON_PAGE(!PageLocked(vmf->page), vmf->page);
 
-       *page = vmf2.page;
+       *page = vmf->page;
        return ret;
 }
 
@@ -3579,6 +3573,7 @@ static int __handle_mm_fault(struct vm_area_struct *vma, 
unsigned long address,
                .address = address,
                .flags = flags,
                .pgoff = linear_page_index(vma, address),
+               .gfp_mask = __get_fault_gfp_mask(vma),
        };
        struct mm_struct *mm = vma->vm_mm;
        pgd_t *pgd;
-- 
2.6.6

_______________________________________________
Linux-nvdimm mailing list
Linux-nvdimm@lists.01.org
https://lists.01.org/mailman/listinfo/linux-nvdimm

Reply via email to