On Wed, Jun 29, 2016 at 12:05:09PM +0200, Vlastimil Babka wrote: > On 06/29/2016 10:12 AM, Joonsoo Kim wrote: > >>@@ -1035,8 +1034,12 @@ static void isolate_freepages(struct com > >> continue; > >> > >> /* Found a block suitable for isolating free pages from. */ > >>- isolate_freepages_block(cc, &isolate_start_pfn, > >>- block_end_pfn, freelist, false); > >>+ isolated = isolate_freepages_block(cc, &isolate_start_pfn, > >>+ block_end_pfn, freelist, false); > >>+ /* If isolation failed early, do not continue needlessly */ > >>+ if (!isolated && isolate_start_pfn < block_end_pfn && > >>+ cc->nr_migratepages > cc->nr_freepages) > >>+ break; > > > >Hello, David. > > > >Minchan found the bug on this patch. > > Was it reported in some mail to linux-mm before the patch was sent > to Linus, or just now? Unfortunately, we found it yesterday and discussed it in offline. Thanks. -- To unsubscribe from this list: send the line "unsubscribe stable" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html