Re: [merged] mm-compaction-abort-free-scanner-if-split-fails.patch removed from -mm tree

[Joonsoo Kim]

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.