Skip to content

Commit f557180

Browse files
author
Joseph Szczypek
committed
watchdog/hpwdt: Only claim UNKNOWN NMI if from iLO
JIRA: https://issues.redhat.com/browse/RHEL-104148 Upstream Status: git://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git commit dced0b3 Author: Jerry Hoemann <jerry.hoemann@hpe.com> Date: Wed Dec 13 14:53:38 2023 -0700 watchdog/hpwdt: Only claim UNKNOWN NMI if from iLO Avoid unnecessary crashes by claiming only NMIs that are due to ERROR signalling or generated by the hpwdt hardware device. The code does this, but only for iLO5. The intent was to preserve legacy, Gen9 and earlier, semantics of using hpwdt for error containtment as hardware/firmware would signal fatal IO errors as an NMI with the expectation of hpwdt crashing the system. Howerver, these IO errors should be received by hpwdt as an NMI_IO_CHECK. So the test is overly permissive and should not be limited to only ilo5. We need to enable this protection for future iLOs not matching the current PCI IDs. Fixes: 62290a5 ("watchdog: hpwdt: Claim NMIs generated by iLO5") Signed-off-by: Jerry Hoemann <jerry.hoemann@hpe.com> Reviewed-by: Guenter Roeck <linux@roeck-us.net> Link: https://lore.kernel.org/r/20231213215340.495734-2-jerry.hoemann@hpe.com Signed-off-by: Guenter Roeck <linux@roeck-us.net> Signed-off-by: Wim Van Sebroeck <wim@linux-watchdog.org> Signed-off-by: Joseph Szczypek <jszczype@redhat.com>
1 parent f537aee commit f557180

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

drivers/watchdog/hpwdt.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -178,7 +178,7 @@ static int hpwdt_pretimeout(unsigned int ulReason, struct pt_regs *regs)
178178
"3. OA Forward Progress Log\n"
179179
"4. iLO Event Log";
180180

181-
if (ilo5 && ulReason == NMI_UNKNOWN && !mynmi)
181+
if (ulReason == NMI_UNKNOWN && !mynmi)
182182
return NMI_DONE;
183183

184184
if (ilo5 && !pretimeout && !mynmi)

0 commit comments

Comments
 (0)