Received: by 2002:a05:6a10:f347:0:0:0:0 with SMTP id d7csp4125890pxu; Mon, 30 Nov 2020 18:44:13 -0800 (PST) X-Google-Smtp-Source: ABdhPJz52ksaXxv7ekVon5huXmkeOkXuhgRj/6I3vSW17+d6iV1hlzLlCU/WVUEJyEGLqWh0KH8t X-Received: by 2002:a05:6402:1c90:: with SMTP id cy16mr867616edb.73.1606790653095; Mon, 30 Nov 2020 18:44:13 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1606790653; cv=none; d=google.com; s=arc-20160816; b=GMypREbpi9MpXiRpBTL2Pj2MkE6wMeHcP01INhQ/FZp6LIe/fkXA7i5QmdDZZ+BN6a i9kJjbJFmsawkoOO1+bgR291lvGGYtVaI/jsBIDVN08Z92p4Eb9OnCBCalLBlJ19+Kaf mTegrgCFM1oxl0D6xElgHTflAzjZUkZkxKQNhiJjxVlu7Wu70Hzyfzm59rKINitBl7NI 9E1zdTQv+a3NYoyAzT9KVEiLeNfuFHQicYyYrMydi7JNmasCD4iZACtgiv/hVbaWUce7 X/CSO1O7NizP9EGmaa/LBl72vWglBXyKekiJC6q3JA4g7siS2jqDoVOnF2wf0c8CjiO8 9Vrg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:mime-version :references:in-reply-to:message-id:subject:cc:to:from:date :dkim-signature; bh=yiGfjoCgHvWmU8/kkPQvAFx5oMe9S4/s2GoWmSvqlcw=; b=ZWxakvRzgNFvSIJhbKPeYUFm2lG9jkvRtOWUsR1VISjY3z1q8ydM/WsdiW+T5CNeu1 2mxVSIsyKT7VeR+wx45iN7K6MXf3XYQPSec/WMTGzEEJwBIed30aZaDCF5u7zHs+tMgD IMLkzPCLQmwuqKhEcA7hDFsXemL2MGzQFldMNXxYGIRtfKGUwTwGrHsEWd9ykkBXYUFl A8hxBOqx23rmIvw0yev09w75v4+iOryRh3JrhMfSku8ci1LreiVF4A8Xwz1zWbORZ1/S LdbVsmKJFEz1XhTLwfx3Kx/xfAnSsGzHSfK3xH/UzZjMp2fhKp5Rv651gmUbtHyyWeGY F8lg== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@kernel.org header.s=default header.b=yZObzgkb; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id w4si78033ejy.251.2020.11.30.18.43.50; Mon, 30 Nov 2020 18:44:13 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; dkim=pass header.i=@kernel.org header.s=default header.b=yZObzgkb; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726932AbgLAC3x (ORCPT + 99 others); Mon, 30 Nov 2020 21:29:53 -0500 Received: from mail.kernel.org ([198.145.29.99]:38370 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726198AbgLAC3x (ORCPT ); Mon, 30 Nov 2020 21:29:53 -0500 Received: from kicinski-fedora-pc1c0hjn.DHCP.thefacebook.com (unknown [163.114.132.7]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPSA id 59CBA20809; Tue, 1 Dec 2020 02:29:11 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=default; t=1606789752; bh=qmSgtrpGDoagvz4Citm0tPgCTGgHPEJNbRWr+dAB5pY=; h=Date:From:To:Cc:Subject:In-Reply-To:References:From; b=yZObzgkb5UzIRyJKwc8ErAvDLIqCbJN8w6VD1chkJCRbfZcZAfeEcZfbTiNcfU5fs X4BrJ9EktyeC6S/BhHluXIYDMLLBnZ+QlaegXz+e3lP4gv/bPewk+2hFhBe9UBm9Fy aZ6NROIMjHTKqO/14851z/NyGEsDYDOwD9ixcPgA= Date: Mon, 30 Nov 2020 18:29:10 -0800 From: Jakub Kicinski To: George Cherian Cc: , , , , , , , , , Subject: Re: [PATCHv5 net-next 2/3] octeontx2-af: Add devlink health reporters for NPA Message-ID: <20201130182910.49ea8c8c@kicinski-fedora-pc1c0hjn.DHCP.thefacebook.com> In-Reply-To: <20201126140251.963048-3-george.cherian@marvell.com> References: <20201126140251.963048-1-george.cherian@marvell.com> <20201126140251.963048-3-george.cherian@marvell.com> MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Thu, 26 Nov 2020 19:32:50 +0530 George Cherian wrote: > Add health reporters for RVU NPA block. > NPA Health reporters handle following HW event groups > - GENERAL events > - ERROR events > - RAS events > - RVU event > An event counter per event is maintained in SW. > > Output: > # devlink health > pci/0002:01:00.0: > reporter hw_npa > state healthy error 0 recover 0 > # devlink health dump show pci/0002:01:00.0 reporter hw_npa > NPA_AF_GENERAL: > Unmap PF Error: 0 > NIX: > 0: free disabled RX: 0 free disabled TX: 0 > 1: free disabled RX: 0 free disabled TX: 0 > Free Disabled for SSO: 0 > Free Disabled for TIM: 0 > Free Disabled for DPI: 0 > Free Disabled for AURA: 0 > Alloc Disabled for Resvd: 0 > NPA_AF_ERR: > Memory Fault on NPA_AQ_INST_S read: 0 > Memory Fault on NPA_AQ_RES_S write: 0 > AQ Doorbell Error: 0 > Poisoned data on NPA_AQ_INST_S read: 0 > Poisoned data on NPA_AQ_RES_S write: 0 > Poisoned data on HW context read: 0 > NPA_AF_RVU: > Unmap Slot Error: 0 You seem to have missed the feedback Saeed and I gave you on v2. Did you test this with the errors actually triggering? Devlink should store only one dump, are the counters not going to get out of sync unless something clears the dump every time it triggers?