Received: by 2002:a05:7412:8d10:b0:f3:1519:9f41 with SMTP id bj16csp5617177rdb; Wed, 13 Dec 2023 14:14:29 -0800 (PST) X-Google-Smtp-Source: AGHT+IFHWgGpJkuE6gnf8hEeI8VD0OM7W8wwbWAMPTFBqsU0qfClN6mT0SbkZ0m5r96zeaGEIXs1 X-Received: by 2002:a05:6a00:10d1:b0:6ce:5fc9:8d8d with SMTP id d17-20020a056a0010d100b006ce5fc98d8dmr5320486pfu.14.1702505668872; Wed, 13 Dec 2023 14:14:28 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1702505668; cv=none; d=google.com; s=arc-20160816; b=TMD4xMTdk2ckjm1RKzMy5ncrHZO49KwAn1MHMB92C1PXmGU5JOFTIsXyZGDgXg2Qoi PbkyIhCvIdavN9gtYIhhOUHbqm4wVI9wjS0bjw+GjZbRESOiranuqSc7bylDmUPzsKkv ZD0Gb1yb7Ep6XLysOSekFnDJAf3QymM2IN+WcQZ8aVuLv8D9lSERPtz4Hznp2TceRm3J ypl1YwUbeB5GK2UDOIBo7wMOeEL9FyV2x8yZB8SvX2UmnFXxAbPX7ubYCvt+ExCJ1COm dMTwWlc269dx3miVC500tBnV5Q/IUQ927AshIz0jeGDNbqC4ZHMlb5GzIMNaybqquScb IZLA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:in-reply-to:content-disposition:mime-version :references:message-id:subject:cc:to:from:date:sender:dkim-signature; bh=kUOnuYPNNImTr05uNyh+XqTTfSm5CyCumIyly8xyq6A=; fh=AI573epPix8xVJ7mSnV11X9WsevjguBzbe/w0Vx967E=; b=AO+pr4XeOygrzGOYurJBYZqKtNdr9D6wsdiIAiGBM8wV/Y+ZSyyFYiwrANO31Lgr04 RNzd/NV3SGQKnqcvwWdQpqkW7i8mCk6u4PyKoX20l5rOz+Br6QC+W8wnz+/Zkfeyczp1 94FK/uJKacm0eKJH0NxLaWJ+Agr85p/M7j+yyAGS7+4I52XW8PI6yBUGV1bxUwLhpnb0 jCfqemqtkhMqD0AkgZsFLPkKWs9VFpGjGwAyG6isXKUiXA08krr7pdRVHjda2yI+a6D7 zR4OlUEtkJfAsrOH8n7g8O3mr0VZiuFHZFEQ8wu8YHsGdszvd6AA2FI89CHFvBVGImCz 60xQ== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@gmail.com header.s=20230601 header.b=SNdirChu; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::3:2 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from agentk.vger.email (agentk.vger.email. [2620:137:e000::3:2]) by mx.google.com with ESMTPS id y16-20020a056a00191000b006cbbacf7aaasi10365750pfi.29.2023.12.13.14.14.28 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 13 Dec 2023 14:14:28 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::3:2 as permitted sender) client-ip=2620:137:e000::3:2; Authentication-Results: mx.google.com; dkim=pass header.i=@gmail.com header.s=20230601 header.b=SNdirChu; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::3:2 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: from out1.vger.email (depot.vger.email [IPv6:2620:137:e000::3:0]) by agentk.vger.email (Postfix) with ESMTP id 0074E80C4794; Wed, 13 Dec 2023 14:14:25 -0800 (PST) X-Virus-Status: Clean X-Virus-Scanned: clamav-milter 0.103.11 at agentk.vger.email Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S235372AbjLMWOI (ORCPT + 99 others); Wed, 13 Dec 2023 17:14:08 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:34268 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229929AbjLMWOF (ORCPT ); Wed, 13 Dec 2023 17:14:05 -0500 Received: from mail-ot1-x32a.google.com (mail-ot1-x32a.google.com [IPv6:2607:f8b0:4864:20::32a]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 644ABAC; Wed, 13 Dec 2023 14:14:11 -0800 (PST) Received: by mail-ot1-x32a.google.com with SMTP id 46e09a7af769-6d9dadc3dc0so5760104a34.1; Wed, 13 Dec 2023 14:14:11 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20230601; t=1702505650; x=1703110450; darn=vger.kernel.org; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:sender:from:to:cc:subject:date:message-id :reply-to; bh=kUOnuYPNNImTr05uNyh+XqTTfSm5CyCumIyly8xyq6A=; b=SNdirChuLu8QJ8oubJL7Uzg18dHyT+gupDes1MJ2PjkvwkCjfPM8u3KXHKfIjsIOUZ zQmk0JXYv+9Km/zsBIE4V4Rew60l3KNSwu8NGPCsWjnHYFg5rfoJwIfPUuL6XYyztJkv iDkexcRAUxtRBz92h/j6JxAfLOir9NPjx3F223osoQv2cJtobfl6kW1rwR686QkJ1C2T klCMO/Jkg0r3y7u+pKcvGmTrcDXzeVc/ayxK+GjvbBWClDhRW5kT6OWpG0IZLfiOsTlX RNCJYXp2JofSO6QasWWnstbaAN+aH3FV7dHxHTtdBot6TjReiVf8QOTaZUEanMBQ6z0K +Rwg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1702505650; x=1703110450; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:sender:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=kUOnuYPNNImTr05uNyh+XqTTfSm5CyCumIyly8xyq6A=; b=E4OmmJ2buWrSCV6R9SXXbHRdUmYislda3hZeXf3MOQ9k3bcUJO2gfhRue9kMyyBwXj 2j6O+J/xrLF2tuvJlXrnooKX9iymtUueWdnjlz5SpQBimhLKRYJI7lolA1CJ4rp9duFJ f0AC61C35KbXlHMalGAz2DWF2cAIFyrBcbbqaeFsLwmZbTI0wvyLBGKJguLefh+EzDiw AjY5UBLS/1LZlRg+D5VdM7K8Ic0DitAPBdpCDcBl9Orv+fDRLOJv2xoARKXSw4MNlV6O kX8+FzT9B1zB5sZBMjLTPO0dniEjg5IV0NmLkgHGknZLAKqXZXKbqiszT2OwVec3TMUb jrlw== X-Gm-Message-State: AOJu0YzEvUPpmJGhECZZeGaD5jcYV9pi31+X9S5Lyr+WjiGEjJkgpaZI QoONluCRiEVEvxJ94xbl+KCdcJFx/6M= X-Received: by 2002:a9d:7f8c:0:b0:6d9:f4a3:aece with SMTP id t12-20020a9d7f8c000000b006d9f4a3aecemr8913972otp.49.1702505650512; Wed, 13 Dec 2023 14:14:10 -0800 (PST) Received: from server.roeck-us.net ([2600:1700:e321:62f0:329c:23ff:fee3:9d7c]) by smtp.gmail.com with ESMTPSA id c11-20020a630d0b000000b005c2185be2basm10368437pgl.54.2023.12.13.14.14.09 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 13 Dec 2023 14:14:09 -0800 (PST) Sender: Guenter Roeck Date: Wed, 13 Dec 2023 14:14:08 -0800 From: Guenter Roeck To: Jerry Hoemann Cc: wim@linux-watchdog.org, linux-watchdog@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH v2 1/3] watchdog/hpwdt: Only claim UNKNOWN NMI if from iLO Message-ID: <9c7c6ba4-2140-4d1c-b2de-705b9f6b9e2e@roeck-us.net> References: <20231213215340.495734-1-jerry.hoemann@hpe.com> <20231213215340.495734-2-jerry.hoemann@hpe.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20231213215340.495734-2-jerry.hoemann@hpe.com> X-Spam-Status: No, score=-0.8 required=5.0 tests=DKIM_SIGNED,DKIM_VALID, HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,SPF_HELO_NONE, SPF_PASS,T_SCC_BODY_TEXT_LINE autolearn=unavailable autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on agentk.vger.email Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org X-Greylist: Sender passed SPF test, not delayed by milter-greylist-4.6.4 (agentk.vger.email [0.0.0.0]); Wed, 13 Dec 2023 14:14:26 -0800 (PST) On Wed, Dec 13, 2023 at 02:53:38PM -0700, Jerry Hoemann wrote: > 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: 62290a5c194b ("watchdog: hpwdt: Claim NMIs generated by iLO5") > Signed-off-by: Jerry Hoemann Reviewed-by: Guenter Roeck > --- > drivers/watchdog/hpwdt.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/watchdog/hpwdt.c b/drivers/watchdog/hpwdt.c > index f79f932bca14..79ed1626d8ea 100644 > --- a/drivers/watchdog/hpwdt.c > +++ b/drivers/watchdog/hpwdt.c > @@ -178,7 +178,7 @@ static int hpwdt_pretimeout(unsigned int ulReason, struct pt_regs *regs) > "3. OA Forward Progress Log\n" > "4. iLO Event Log"; > > - if (ilo5 && ulReason == NMI_UNKNOWN && !mynmi) > + if (ulReason == NMI_UNKNOWN && !mynmi) > return NMI_DONE; > > if (ilo5 && !pretimeout && !mynmi) > -- > 2.41.0 >