Re: [PATCH v2] powerpc/fadump: when fadump is supported register the fadump sysfs files.
On Thu, 29 Aug 2019 10:58:16 +0530 Hari Bathini wrote: > On 28/08/19 10:57 PM, Michal Suchanek wrote: > > Currently it is not possible to distinguish the case when fadump is > > supported by firmware and disabled in kernel and completely unsupported > > using the kernel sysfs interface. User can investigate the devicetree > > but it is more reasonable to provide sysfs files in case we get some > > fadumpv2 in the future. > > > > With this patch sysfs files are available whenever fadump is supported > > by firmware. > > > > Signed-off-by: Michal Suchanek > > --- > > [...] > > > - if (!fw_dump.fadump_supported) { > > + if (!fw_dump.fadump_supported && fw_dump.fadump_enabled) { > > printk(KERN_ERR "Firmware-assisted dump is not supported on" > > " this hardware\n"); > > - return 0; > > } > > The above hunk is redundant with similar message already logged during > early boot in fadump_reserve_mem() function. I am not strongly against > this though. So... I see this: [0.00] debug: ignoring loglevel setting. [0.00] Firmware-assisted dump is not supported on this hardware [0.00] Reserving 256MB of memory at 128MB for crashkernel (System RAM: 8192MB) [0.00] Allocated 5832704 bytes for 2048 pacas at c7a8 [0.00] Page sizes from device-tree: [0.00] base_shift=12: shift=12, sllp=0x, avpnm=0x, tlbiel=1, penc=0 [0.00] base_shift=16: shift=16, sllp=0x0110, avpnm=0x, tlbiel=1, penc=1 [0.00] Page orders: linear mapping = 16, virtual = 16, io = 16, vmemmap = 16 [0.00] Using 1TB segments [0.00] Initializing hash mmu with SLB Clearly the second message is logged from the above code. The duplicate is capitalized: "Firmware-Assisted Dump is not supported on this hardware" and I don't see it logged. So if anything is duplicate that should be removed it is the message in fadump_reserve_mem(). It is not clear why that one is not seen, though. Thanks Michal
Re: [PATCH v2] powerpc/fadump: when fadump is supported register the fadump sysfs files.
On 28/08/19 10:57 PM, Michal Suchanek wrote: > Currently it is not possible to distinguish the case when fadump is > supported by firmware and disabled in kernel and completely unsupported > using the kernel sysfs interface. User can investigate the devicetree > but it is more reasonable to provide sysfs files in case we get some > fadumpv2 in the future. > > With this patch sysfs files are available whenever fadump is supported > by firmware. > > Signed-off-by: Michal Suchanek > --- [...] > - if (!fw_dump.fadump_supported) { > + if (!fw_dump.fadump_supported && fw_dump.fadump_enabled) { > printk(KERN_ERR "Firmware-assisted dump is not supported on" > " this hardware\n"); > - return 0; > } The above hunk is redundant with similar message already logged during early boot in fadump_reserve_mem() function. I am not strongly against this though. So... Acked-by: Hari Bathini
[PATCH v2] powerpc/fadump: when fadump is supported register the fadump sysfs files.
Currently it is not possible to distinguish the case when fadump is supported by firmware and disabled in kernel and completely unsupported using the kernel sysfs interface. User can investigate the devicetree but it is more reasonable to provide sysfs files in case we get some fadumpv2 in the future. With this patch sysfs files are available whenever fadump is supported by firmware. Signed-off-by: Michal Suchanek --- v2: move the sysfs initialization earlier to avoid condition nesting --- arch/powerpc/kernel/fadump.c | 15 +-- 1 file changed, 9 insertions(+), 6 deletions(-) diff --git a/arch/powerpc/kernel/fadump.c b/arch/powerpc/kernel/fadump.c index 4eab97292cc2..13741380b2f7 100644 --- a/arch/powerpc/kernel/fadump.c +++ b/arch/powerpc/kernel/fadump.c @@ -1671,16 +1671,20 @@ static void fadump_init_files(void) */ int __init setup_fadump(void) { - if (!fw_dump.fadump_enabled) - return 0; - - if (!fw_dump.fadump_supported) { + if (!fw_dump.fadump_supported && fw_dump.fadump_enabled) { printk(KERN_ERR "Firmware-assisted dump is not supported on" " this hardware\n"); - return 0; } + if (!fw_dump.fadump_supported) + return 0; + + fadump_init_files(); fadump_show_config(); + + if (!fw_dump.fadump_enabled) + return 1; + /* * If dump data is available then see if it is valid and prepare for * saving it to the disk. @@ -1696,7 +1700,6 @@ int __init setup_fadump(void) /* Initialize the kernel dump memory structure for FAD registration. */ else if (fw_dump.reserve_dump_area_size) init_fadump_mem_struct(&fdm, fw_dump.reserve_dump_area_start); - fadump_init_files(); return 1; } -- 2.22.0