Re: [PATCH] mm,hugetlb,migration: don't migrate kernelcore hugepages

From: Michal Hocko
Date: Tue Oct 03 2017 - 03:10:31 EST


On Tue 03-10-17 07:42:25, Alexandru Moise wrote:
> On Mon, Oct 02, 2017 at 06:15:00PM +0200, Michal Hocko wrote:
[...]
> > I really fail to see why kernel vs. movable zones play any role here.
> > Zones should be mostly an implementation detail which userspace
> > shouldn't really care about.
>
> Ok, the whole zone approach is a bad idea. Do you think that there's
> any value at all to trying to make hugepages un-movable at all?

I am not aware of any usecase, to be honest.

> Should
> the hugepages_treat_as_movable sysctl die and just make hugepages movable
> by default?

I think that hugepages_treat_as_movable is just a historical relict from
the time when hugetlb pages were not movable but the main purpose of
the movable zone was different back at the time. Just to clarifiy, the
original intention of the zone was to prevent memory fragmentation and
as hugetlb pages are not fragmenting memory because they are long lived
and contiguous, it was acceptable to use the zone. The purpose of the
zone has changed towards a migratability guarantee since then but the
knob has stayed behind. I think we should just remove it.
--
Michal Hocko
SUSE Labs