On Wed 06-12-17 22:52:37, Pravin Shedge wrote: > These duplicate includes have been found with scripts/checkincludes.pl but > they have been removed manually to avoid removing false positives. I only see this one but I can see this is a series of 45 patches. Is this really the best way to apply a change like this? Why not do it in a single patch? Other than that the patch looks correct. > Signed-off-by: Pravin Shedge <pravin.shedge4linux@xxxxxxxxx> > --- > mm/userfaultfd.c | 1 - > 1 file changed, 1 deletion(-) > > diff --git a/mm/userfaultfd.c b/mm/userfaultfd.c > index 8119270..39791b8 100644 > --- a/mm/userfaultfd.c > +++ b/mm/userfaultfd.c > @@ -16,7 +16,6 @@ > #include <linux/userfaultfd_k.h> > #include <linux/mmu_notifier.h> > #include <linux/hugetlb.h> > -#include <linux/pagemap.h> > #include <linux/shmem_fs.h> > #include <asm/tlbflush.h> > #include "internal.h" > -- > 2.7.4 > -- Michal Hocko SUSE Labs -- 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>