On 1/10/19 9:12 PM, Pingfan Liu wrote: > This patch identifies the point where memblock alloc start. It has no > functional. It has no functional ... what? Effects? > - memblock_set_current_limit(ISA_END_ADDRESS); > - e820__memblock_setup(); > - > reserve_bios_regions(); > > if (efi_enabled(EFI_MEMMAP)) { > @@ -1113,6 +1087,8 @@ void __init setup_arch(char **cmdline_p) > efi_reserve_boot_services(); > } > > + memblock_set_current_limit(0, ISA_END_ADDRESS, false); > + e820__memblock_setup(); It looks like you changed the arguments passed to memblock_set_current_limit(). How can this even compile? Did you mean that this patch is not functional?