Christophe LEROY <christophe.le...@c-s.fr> writes: > Le 28/09/2018 à 05:41, Michael Ellerman a écrit : >> Christophe Leroy <christophe.le...@c-s.fr> writes: >>> Add call to early_memtest() so that kernel compiled with >>> CONFIG_MEMTEST really perform memtest at startup when requested >>> via 'memtest' boot parameter. >>> >>> Signed-off-by: Christophe Leroy <christophe.le...@c-s.fr> >>> --- >>> arch/powerpc/kernel/setup-common.c | 3 +++ >>> 1 file changed, 3 insertions(+) >>> >>> diff --git a/arch/powerpc/kernel/setup-common.c >>> b/arch/powerpc/kernel/setup-common.c >>> index 93fa0c99681e..904b728eb20d 100644 >>> --- a/arch/powerpc/kernel/setup-common.c >>> +++ b/arch/powerpc/kernel/setup-common.c >>> @@ -33,6 +33,7 @@ >>> #include <linux/serial_8250.h> >>> #include <linux/percpu.h> >>> #include <linux/memblock.h> >>> +#include <linux/bootmem.h> >>> #include <linux/of_platform.h> >>> #include <linux/hugetlb.h> >>> #include <asm/debugfs.h> >>> @@ -917,6 +918,8 @@ void __init setup_arch(char **cmdline_p) >>> /* Parse memory topology */ >>> mem_topology_setup(); >>> >>> + early_memtest(min_low_pfn << PAGE_SHIFT, max_low_pfn << PAGE_SHIFT); >> >> On a ppc64le VM this boils down to early_memtest(0, 0) for me. >> >> I think it's too early, we don't set up max_low_pfn until >> initmem_init(). >> >> If I move it after initmem_init() then it does something more useful: > > Ok. On my 8xx max_low_pfn is set in mem_topology_setup(). > > Moving the test afte initmem_init() still works on the 8xx so I'll do that.
Great, thanks. cheers
