On Wed, Dec 13, 2023 at 02:53:39PM -0700, Jerry Hoemann wrote: > ProLiants of vintage to have an iLO 5, no longer send watchdog NMI > as an IO CHECK. They are presented to hpwdt_pretimeout as NMI_UNKNOWN. > The preceding if statement rejects if !mynmi irrespective of value > of pretimeout making this if statement redundant. > > Signed-off-by: Jerry Hoemann <jerry.hoemann@xxxxxxx> Reviewed-by: Guenter Roeck <linux@xxxxxxxxxxxx> > --- > drivers/watchdog/hpwdt.c | 3 --- > 1 file changed, 3 deletions(-) > > diff --git a/drivers/watchdog/hpwdt.c b/drivers/watchdog/hpwdt.c > index 79ed1626d8ea..d5c0aa3ef069 100644 > --- a/drivers/watchdog/hpwdt.c > +++ b/drivers/watchdog/hpwdt.c > @@ -181,9 +181,6 @@ static int hpwdt_pretimeout(unsigned int ulReason, struct pt_regs *regs) > if (ulReason == NMI_UNKNOWN && !mynmi) > return NMI_DONE; > > - if (ilo5 && !pretimeout && !mynmi) > - return NMI_DONE; > - > if (kdumptimeout < 0) > hpwdt_stop(); > else if (kdumptimeout == 0) > -- > 2.41.0 >
