On Wed 16-11-16 01:10:01, Kirill A. Shutemov wrote: > 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? OK, I did that as well. > Otherwise makes sense: > > Acked-by: Kirill A. Shutemov <kirill.shutemov@xxxxxxxxxxxxxxx> Thanks! Honza -- Jan Kara <jack@xxxxxxxx> SUSE Labs, CR -- 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