Hi, On Wed, Jun 7, 2023 at 8:26 AM Petr Mladek <pmladek@xxxxxxxx> wrote: > > The HAVE_ prefix means that the code could be enabled. Add another > variable for HAVE_HARDLOCKUP_DETECTOR_SPARC64 without this prefix. > It will be set when it should be built. It will make it compatible > with the other hardlockup detectors. > > Before, it is far from obvious that the SPARC64 variant is actually used: > > $> make ARCH=sparc64 defconfig > $> grep HARDLOCKUP_DETECTOR .config > CONFIG_HAVE_HARDLOCKUP_DETECTOR_BUDDY=y > CONFIG_HAVE_HARDLOCKUP_DETECTOR_SPARC64=y > > After, it is more clear: > > $> make ARCH=sparc64 defconfig > $> grep HARDLOCKUP_DETECTOR .config > CONFIG_HAVE_HARDLOCKUP_DETECTOR_BUDDY=y > CONFIG_HAVE_HARDLOCKUP_DETECTOR_SPARC64=y > CONFIG_HARDLOCKUP_DETECTOR_SPARC64=y > > Signed-off-by: Petr Mladek <pmladek@xxxxxxxx> > --- > arch/sparc/Kconfig.debug | 10 +++++++++- > include/linux/nmi.h | 4 ++-- > kernel/watchdog.c | 2 +- > lib/Kconfig.debug | 2 +- > 4 files changed, 13 insertions(+), 5 deletions(-) Reviewed-by: Douglas Anderson <dianders@xxxxxxxxxxxx>
