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, send a message with 'unsubscribe linux-mm' in the body to majordomo@xxxxxxxxx. For more info on Linux MM, see: http://www.linux-mm.org/ . Don't email: <a href=mailto:"dont@xxxxxxxxx"> email@xxxxxxxxx </a>