On Fri, Nov 04, 2016 at 05:25:01AM +0100, Jan Kara wrote: > static int do_cow_fault(struct vm_fault *vmf) > { > struct vm_area_struct *vma = vmf->vma; > - struct page *fault_page, *new_page; > - void *fault_entry; > + struct page *new_page;
Why not get rid of new_page too? Otherwise makes sense: Acked-by: Kirill A. Shutemov <kirill.shute...@linux.intel.com> -- Kirill A. Shutemov _______________________________________________ Linux-nvdimm mailing list Linux-nvdimm@lists.01.org https://lists.01.org/mailman/listinfo/linux-nvdimm