[PATCH v33 00/14] add kdump support
David Woodhouse
dwmw2 at infradead.org
Tue Mar 21 02:42:23 PDT 2017
On Tue, 2017-03-21 at 16:34 +0900, AKASHI Takahiro wrote:
> Yes, it is intentional. I removed 'offline' code in my v14 (2016/3/4).
> As you assumed, I'd expect 'online' status of all CPUs to be kept
> unchanged in the core dump.
I wonder if it would be better to take a *copy* of it and put it back
after we're done taking the CPUs down? As things stand, we now have
*three* different methods of taking down all the CPUs... and *none* of
them allow a platform to override it with an NMI-based or STONITH-based
method, which seems like something of an oversight.
> If you can agree, I would like to modify this disputed warning code to:
>
> + BUG_ON(!in_kexec_crash && (stuck_cpus || (num_online_cpus() > 1)));
> + WARN(in_kexec_crash && (stuck_cpus || smp_crash_stop_failed()),
> + "Some CPUs may be stale, kdump will be unreliable.\n");
That works; thanks.
FWIW I'm currently blaming my platform's firmware for my sporadic
crash-on-CPU#1 failures. If your testing includes crashes on non-boot
CPUs (perhaps using the sysrq hack I posted) and it reliably passes for
you, then let's ignore that for now.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: smime.p7s
Type: application/x-pkcs7-signature
Size: 4938 bytes
Desc: not available
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20170321/3266da88/attachment-0001.bin>
More information about the linux-arm-kernel
mailing list