On Tue, Jan 8, 2019 at 6:06 PM Chao Fan <[email protected]> wrote: > > On Mon, Jan 07, 2019 at 04:24:41PM +0800, Pingfan Liu wrote: > >Background about the defect of the current bottom-up allocation style, take > >the following scenario: > > | unmovable node | movable node | > > | kaslr-kernel |subtree of pgtable for phy<->virt | > > > >Although kaslr-kernel can avoid to stain the movable node. But the > >pgtable can still stain the movable node. That is a probability problem, > >with low probability, but still exist. This patch tries to eliminate the > >probability. With the previous patch, at the point of init_mem_mapping(), > >memblock allocator can work with the knowledge of acpi memory hotmovable > >info, and avoid to stain the movable node. As a result, > >memory_map_bottom_up() is not needed any more. > > > > Hi Pingfan, > > Tang Chen ever tried to do this before adding 'movable_node': > commit e8d1955258091e4c92d5a975ebd7fd8a98f5d30f > Author: Tang Chen <[email protected]> > Date: Fri Feb 22 16:33:44 2013 -0800 > > acpi, memory-hotplug: parse SRAT before memblock is ready > > Then, Lu Yinghai tried to do the similar job, you can see: > https://lwn.net/Articles/554854/ > for more information. Hope that can help you. > Thanks, It is a long thread, as my understanding, Tejun concerned about the early parsing of ACPI consumes memory from memblock allocator. If it is, then this should not happen in my series. Cc Tejun and Yinghai.
Regards, Pingfan > Thanks, > Chao Fan > > > > >Cc: Thomas Gleixner <[email protected]> > >Cc: Ingo Molnar <[email protected]> > >Cc: Borislav Petkov <[email protected]> > >Cc: "H. Peter Anvin" <[email protected]> > >Cc: Dave Hansen <[email protected]> > >Cc: Andy Lutomirski <[email protected]> > >Cc: Peter Zijlstra <[email protected]> > >Cc: "Rafael J. Wysocki" <[email protected]> > >Cc: Len Brown <[email protected]> > >Cc: [email protected] > > > >Pingfan Liu (4): > > acpi: change the topo of acpi_table_upgrade() > > x86/setup: parse acpi to get hotplug info before init_mem_mapping() > > x86/mm: set allowed range for memblock allocator > > x86/mm: remove bottom-up allocation style for x86_64 > > > > arch/arm64/kernel/setup.c | 2 +- > > arch/x86/kernel/setup.c | 17 ++++- > > arch/x86/mm/init.c | 154 > > +++++++--------------------------------------- > > arch/x86/mm/init_32.c | 123 ++++++++++++++++++++++++++++++++++++ > > arch/x86/mm/mm_internal.h | 7 +++ > > drivers/acpi/tables.c | 4 +- > > include/linux/acpi.h | 5 +- > > 7 files changed, 172 insertions(+), 140 deletions(-) > > > >-- > >2.7.4 > > > > > > > >
