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.shutemov@xxxxxxxxxxxxxxx> -- Kirill A. Shutemov -- To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html