Re: [PATCH] powerpc: wire up memtest

[Michael Ellerman]

Christophe LEROY <christophe.leroy@xxxxxx> writes:
> Le 28/09/2018 Ã 05:41, Michael Ellerman a ÃcritÂ:
>> Christophe Leroy <christophe.leroy@xxxxxx> 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.leroy@xxxxxx>
>>> ---
>>>   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