Re: [PATCH v2] mm: hugetlb: add arch hook for clearing page flagsbefore entering pool

[Will Deacon]

On Thu, Aug 23, 2012 at 06:11:56PM +0100, Michal Hocko wrote:
> On Thu 23-08-12 17:37:13, Will Deacon wrote:
> > The core page allocator ensures that page flags are zeroed when freeing
> > pages via free_pages_check. A number of architectures (ARM, PPC, MIPS)
> > rely on this property to treat new pages as dirty with respect to the
> > data cache and perform the appropriate flushing before mapping the pages
> > into userspace.
> > 
> > This can lead to cache synchronisation problems when using hugepages,
> > since the allocator keeps its own pool of pages above the usual page
> > allocator and does not reset the page flags when freeing a page into
> > the pool.
> > 
> > This patch adds a new architecture hook, arch_clear_hugepage_flags, so
> > that architectures which rely on the page flags being in a particular
> > state for fresh allocations can adjust the flags accordingly when a
> > page is freed into the pool.
> > 
> > Cc: Michal Hocko <mhocko@xxxxxxx>
> > Signed-off-by: Will Deacon <will.deacon@xxxxxxx>
> 
> Looks good to me
> Reviewed-by: Michal Hocko <mhocko@xxxxxxx>

Cheers Michal. Next step: start posting the ARM code!

Will
--
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/