Re: [PATCH 2/2] mm/page_alloc: DEBUG_VM checks for free_list placement of CMA and RESERVE pages

From: Vlastimil Babka
Date: Wed May 14 2014 - 05:01:31 EST


On 05/14/2014 07:19 AM, Hugh Dickins wrote:
On Tue, 13 May 2014, Sasha Levin wrote:
On 05/05/2014 11:50 AM, Vlastimil Babka wrote:
So in the end this VM_DEBUG check probably cannot work anymore for MIGRATE_RESERVE, only for CMA. I'm not sure if it's worth keeping it only for CMA, what are the CMA guys' opinions on that?

The way I understood it is that this patch is wrong, but it's still
alive in -mm. Should it still be there?

I agree that it should be dropped. I did not follow the discussion,
but mmotm soon gives me BUG at mm/page_alloc.c:1242 under swapping load.

Yes, I have already asked for dropping, and updating message of PATCH 1/2 at http://marc.info/?l=linux-mm&m=139947475413079&w=2

Vlastimil

Hugh


--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/