Message ID | 20240206094151epcms2p373ed7f50efa332765e14bff2b5a2abe2@epcms2p3 |
---|---|
State | New |
Headers |
Return-Path: <linux-kernel+bounces-54614-ouuuleilei=gmail.com@vger.kernel.org> Delivered-To: ouuuleilei@gmail.com Received: by 2002:a05:7301:168b:b0:106:860b:bbdd with SMTP id ma11csp1425534dyb; Tue, 6 Feb 2024 01:42:13 -0800 (PST) X-Google-Smtp-Source: AGHT+IGNH+8Qdjwaq33Un15ljj8zlW+F7yNOU9XxtPqs2sqq/S0ECcwLp2XASa5Q/L5HCzyiwPTr X-Received: by 2002:a25:860f:0:b0:dc6:ce0e:ad47 with SMTP id y15-20020a25860f000000b00dc6ce0ead47mr1166373ybk.56.1707212533641; Tue, 06 Feb 2024 01:42:13 -0800 (PST) ARC-Seal: i=2; a=rsa-sha256; t=1707212533; cv=pass; d=google.com; s=arc-20160816; b=P/gNsLtiRl/fkXW/nUFQtADPfHK0WPXcFa5+703pEvP/UFh+Yg6dOWYtAvCRvNe/lT dZSmnwGSu/iqMzYItc5goHWiuUFQqrfpurJz9/ocW38yWBG34tJzNZrrifoQpMyfHqVB uupA21TpLvBrMcSR7oNG2KTeZ/0A7Bng3LOlFKqcOpl0QaMm4O8qGHuAdL7o2Dw6JVZf wEm3cro1EKdxc/8ZpEiR2IcKXpR2mQvTrVpcilZCGUjBmj3FICulNv3hMYB1lZbkm48P TwjhJoFHuIRu0Ce+Omhap0tti9gvlAL8N9blmgQatN2oLJm5wnOaXkXS2+ZxhaFzyVwD Td+w== ARC-Message-Signature: i=2; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=references:dlp-filter:cms-type:content-transfer-encoding:date :message-id:cc:to:from:sender:reply-to:subject:mime-version :list-unsubscribe:list-subscribe:list-id:precedence:dkim-signature :dkim-filter; bh=6FS5OxjDvcI+9OytrSkqO4qI4whSWvgqrs/WRChPmI8=; fh=81ZH2cTyH0DO89529OEyd3c+h4qFtpW1d/vOi9POok0=; b=uUb/jXS6UHu52cUxvyMgwFuO0PQIVBx9ZFTdlswL+AE6d69cmxAJEbgMEIO6quFFDr f8nS5A63lGYDrr13M4XyIPMEGcAfxceVJR1a+0NlC9mhmeLV+LFispksPqs2l1gGIMFw jUaNNWGuU0kPiP2nG3l7a3UyTLfoxviNBSPgOULJERlQJ6um0fsicj5wj1DenFaN6cxA phNYWhexLOPcTGj7/D6WUaP37xDT711DerBExGZTM5A6UBM+bGi6kxvz5qpHlkanZNUB ADoL2YvopDH3gKX59hBSwwBDdf58/XtZZF4JxniGMOoHZhM7UYA3TgJSTqt3g37h93ch nN4g==; dara=google.com ARC-Authentication-Results: i=2; mx.google.com; dkim=pass header.i=@samsung.com header.s=mail20170921 header.b=pOX4kbbn; arc=pass (i=1 spf=pass spfdomain=samsung.com dkim=pass dkdomain=samsung.com dmarc=pass fromdomain=samsung.com); spf=pass (google.com: domain of linux-kernel+bounces-54614-ouuuleilei=gmail.com@vger.kernel.org designates 147.75.199.223 as permitted sender) smtp.mailfrom="linux-kernel+bounces-54614-ouuuleilei=gmail.com@vger.kernel.org"; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=samsung.com X-Forwarded-Encrypted: i=1; AJvYcCVAd/Z70sIvDoi+AbWRtQvpWKf1aJznjggNgdZ3HbCxw9JT5E8ft08jotxLo0o/XFULpoJBEzweR7Zyn/j52IA6Ikg48w== Received: from ny.mirrors.kernel.org (ny.mirrors.kernel.org. [147.75.199.223]) by mx.google.com with ESMTPS id e13-20020ac84e4d000000b00429c0f18707si1917316qtw.574.2024.02.06.01.42.13 for <ouuuleilei@gmail.com> (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 06 Feb 2024 01:42:13 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel+bounces-54614-ouuuleilei=gmail.com@vger.kernel.org designates 147.75.199.223 as permitted sender) client-ip=147.75.199.223; Authentication-Results: mx.google.com; dkim=pass header.i=@samsung.com header.s=mail20170921 header.b=pOX4kbbn; arc=pass (i=1 spf=pass spfdomain=samsung.com dkim=pass dkdomain=samsung.com dmarc=pass fromdomain=samsung.com); spf=pass (google.com: domain of linux-kernel+bounces-54614-ouuuleilei=gmail.com@vger.kernel.org designates 147.75.199.223 as permitted sender) smtp.mailfrom="linux-kernel+bounces-54614-ouuuleilei=gmail.com@vger.kernel.org"; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=samsung.com Received: from smtp.subspace.kernel.org (wormhole.subspace.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ny.mirrors.kernel.org (Postfix) with ESMTPS id 6EA951C2241B for <ouuuleilei@gmail.com>; Tue, 6 Feb 2024 09:42:13 +0000 (UTC) Received: from localhost.localdomain (localhost.localdomain [127.0.0.1]) by smtp.subspace.kernel.org (Postfix) with ESMTP id 3FD6A12D163; Tue, 6 Feb 2024 09:42:01 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=samsung.com header.i=@samsung.com header.b="pOX4kbbn" Received: from mailout3.samsung.com (mailout3.samsung.com [203.254.224.33]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 62CD912D14D for <linux-kernel@vger.kernel.org>; Tue, 6 Feb 2024 09:41:55 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=203.254.224.33 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1707212518; cv=none; b=tje2NanUqNfp44g/FRwGSB0oDR58kXw+7SAyR6JAKPjhNzlcRRZ1OQjmbEh40p8RwIlQ1AYf9hXbCQ+Es9oyQJlDXkqmX+/pz8ofeFg2fMfSHykHhMVLa7SRMs8Lm2wZFKzwWSPIjtBovAbFBbpVpt9CQ/GoJl3uhii5z5lba1A= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1707212518; c=relaxed/simple; bh=bzedo6w2sVKvO39Zdi8zshHDEowHD0bVTtf5g9U2YNw=; h=Mime-Version:Subject:From:To:CC:Message-ID:Date:Content-Type: References; b=BjWAAB2aPG7iBei1T1fKQ1q7yTLrAv+9u2dj+oHdK3MAkYdt0Q7z4g3pAM29b/wNCZ/YJA9Yx+bKJYkL2FV77lQLqpnWd8kOxDi/TKvl/CtUTH8BM6O9vMQMd2t/RpdhRXAB+eOrwj4B1atxcPgRYC+HrEQRm2Vt3ZIzAgINpRw= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=samsung.com; spf=pass smtp.mailfrom=samsung.com; dkim=pass (1024-bit key) header.d=samsung.com header.i=@samsung.com header.b=pOX4kbbn; arc=none smtp.client-ip=203.254.224.33 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=samsung.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=samsung.com Received: from epcas2p2.samsung.com (unknown [182.195.41.54]) by mailout3.samsung.com (KnoxPortal) with ESMTP id 20240206094153epoutp03c0cdfa70d79b71163340ad0b71e0da5f~xPLxTiVkS0707607076epoutp03O for <linux-kernel@vger.kernel.org>; Tue, 6 Feb 2024 09:41:53 +0000 (GMT) DKIM-Filter: OpenDKIM Filter v2.11.0 mailout3.samsung.com 20240206094153epoutp03c0cdfa70d79b71163340ad0b71e0da5f~xPLxTiVkS0707607076epoutp03O DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=samsung.com; s=mail20170921; t=1707212513; bh=6FS5OxjDvcI+9OytrSkqO4qI4whSWvgqrs/WRChPmI8=; h=Subject:Reply-To:From:To:CC:Date:References:From; b=pOX4kbbnINMQqLbPbVq/0LTNtB8CdIpT6bBXAYxd3z6ZirCy5Ofeodrqk2jzRWpWY D+7TTB+mBr+zpxADU/URzhlcoFmJiJZTOPKJJ4mxPNSLcFJV5lKoHsK3SQM+llBac1 SA4IbjjS4ySmZRiRyJ0UYEbJe9pOUgi8SZN0Zy/E= Received: from epsnrtp3.localdomain (unknown [182.195.42.164]) by epcas2p1.samsung.com (KnoxPortal) with ESMTP id 20240206094152epcas2p1c921e056f2588e840065ede5f46a109b~xPLwqOpLg3006430064epcas2p1N; Tue, 6 Feb 2024 09:41:52 +0000 (GMT) Received: from epsmgec2p1-new.samsung.com (unknown [182.195.36.98]) by epsnrtp3.localdomain (Postfix) with ESMTP id 4TTdYr0Z5Hz4x9Q0; Tue, 6 Feb 2024 09:41:52 +0000 (GMT) X-AuditID: b6c32a4d-743ff70000004a32-3c-65c1fedf9e0a Received: from epcas2p3.samsung.com ( [182.195.41.55]) by epsmgec2p1-new.samsung.com (Symantec Messaging Gateway) with SMTP id 85.0A.18994.FDEF1C56; Tue, 6 Feb 2024 18:41:51 +0900 (KST) Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: <linux-kernel.vger.kernel.org> List-Subscribe: <mailto:linux-kernel+subscribe@vger.kernel.org> List-Unsubscribe: <mailto:linux-kernel+unsubscribe@vger.kernel.org> Mime-Version: 1.0 Subject: [PATCH] perf: CXL: fix mismatched cpmu event opcode Reply-To: hj96.nam@samsung.com Sender: Hojin Nam <hj96.nam@samsung.com> From: Hojin Nam <hj96.nam@samsung.com> To: "linux-cxl@vger.kernel.org" <linux-cxl@vger.kernel.org>, "jonathan.cameron@huawei.com" <jonathan.cameron@huawei.com> CC: Wonjae Lee <wj28.lee@samsung.com>, KyungSan Kim <ks0204.kim@samsung.com>, "linux-arm-kernel@lists.infradead.org" <linux-arm-kernel@lists.infradead.org>, "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>, "will@kernel.org" <will@kernel.org>, "mark.rutland@arm.com" <mark.rutland@arm.com> X-Priority: 3 X-Content-Kind-Code: NORMAL X-CPGS-Detection: blocking_info_exchange X-Drm-Type: N,general X-Msg-Generator: Mail X-Msg-Type: PERSONAL X-Reply-Demand: N Message-ID: <20240206094151epcms2p373ed7f50efa332765e14bff2b5a2abe2@epcms2p3> Date: Tue, 06 Feb 2024 18:41:51 +0900 X-CMS-MailID: 20240206094151epcms2p373ed7f50efa332765e14bff2b5a2abe2 Content-Transfer-Encoding: 7bit Content-Type: text/plain; charset="utf-8" X-Sendblock-Type: AUTO_CONFIDENTIAL CMS-TYPE: 102P X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFprIJsWRmVeSWpSXmKPExsWy7bCmue79fwdTDZqOW1usWniNzeLoHg6L TY+vsVqcn3WKxeLyrjlsFkuvX2SyaLljarHx/js2Bw6PNfPWMHq0HHnL6rFpVSebx+Yl9R59 W1YxenzeJBfAFpVtk5GamJJapJCal5yfkpmXbqvkHRzvHG9qZmCoa2hpYa6kkJeYm2qr5OIT oOuWmQN0jZJCWWJOKVAoILG4WEnfzqYov7QkVSEjv7jEVim1ICWnwLxArzgxt7g0L10vL7XE ytDAwMgUqDAhO2P5pENMBRc5K3btnMPewLiRvYuRk0NCwERi67bnrF2MXBxCAnsYJRb/usjY xcjBwSsgKPF3hzBIjbCAjURT/y4WEFtIQE5iz4nzLBBxDYm13w+DzWETUJNYeP8SM4gtIlAl cf39UyaQmcwC25gk/j5cygixjFdiRvtTFghbWmL78q1QcQ2JH8t6mSFsUYmbq9+yw9jvj82H qhGRaL13FqpGUOLBz91QcSmJT3c3QcWLJWbvXwYVL5BY8WMR1C59icbr79lAbF4BX4k379Yy gdgsAqoS87esg+p1kbg38Q1YDbOAvMT2t3OYQeHALKApsX6XPogpIaAsceQWC0QFn0TH4b/s MF/tmPeECcJWktjf0Qo1UULizonLUNd4SEz5eJwZEoSBEt2r/7JOYFSYhQjoWUj2zkLYu4CR eRWjVGpBcW56arJRgaFuXmo5PH6T83M3MYLTppbvDsbX6//qHWJk4mA8xCjBwawkwmu240Cq EG9KYmVValF+fFFpTmrxIUZToK8nMkuJJucDE3deSbyhiaWBiZmZobmRqYG5kjjvvda5KUIC 6YklqdmpqQWpRTB9TBycUg1MWRnH91pJdKzUqT1xROz8tLd8Gxuyf/1L3/QnqfdsZfDfR6Y3 fbxmL7o5b3ZxetNUx6agg27xsy/9zL2bdEqk/fg/mwXs7UuYmVxnXdhUc9vtQ9fHaxqiNu6x Mx4qn3957dWfH8r6pytFFqc3vt0YbrTCZcLXm7y8lgG6QbaHnhSEB94vD/EzmWIq8XKCGdPp Tvmd7P/aJK2naJzbzP1qvaFolajKpFATvv4JXqdiqk80OS6QtDIy+R206q3dloS3xgLmysXC YZ37+T/73P+3+M/hnInz5U0vxxl/dM4uuDvJ/8u+qZyiCxuenjd9rV2990zIU9m1d/kmdyes a06e81y7574p99w1ya3fH5mbKrEUZyQaajEXFScCAG87ZjMkBAAA DLP-Filter: Pass X-CFilter-Loop: Reflected X-CMS-RootMailID: 20240206094151epcms2p373ed7f50efa332765e14bff2b5a2abe2 References: <CGME20240206094151epcms2p373ed7f50efa332765e14bff2b5a2abe2@epcms2p3> X-getmail-retrieved-from-mailbox: INBOX X-GMAIL-THRID: 1790142089656954206 X-GMAIL-MSGID: 1790142089656954206 |
Series |
perf: CXL: fix mismatched cpmu event opcode
|
|
Commit Message
Hojin Nam
Feb. 6, 2024, 9:41 a.m. UTC
S2M NDR BI-ConflictAck opcode is described as 4 in the CXL
3.0 specification. However, it is defined as 3 in macro definition.
Signed-off-by: Hojin Nam <hj96.nam@samsung.com>
---
drivers/perf/cxl_pmu.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
--
2.34.1
Comments
On Tue, 06 Feb 2024 18:41:51 +0900 Hojin Nam <hj96.nam@samsung.com> wrote: > S2M NDR BI-ConflictAck opcode is described as 4 in the CXL > 3.0 specification. However, it is defined as 3 in macro definition. > Please provide a reference to a specific spec section. Ideally from r3.1 as that's the easily available version today. I think this is Table 3-50 S2M NDR Opcodes in r3.1 Looks like s2m_ndr_cmpm should be there for bit 3 so we should probably add that as part of the fix. > Signed-off-by: Hojin Nam <hj96.nam@samsung.com> > --- > drivers/perf/cxl_pmu.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/perf/cxl_pmu.c b/drivers/perf/cxl_pmu.c > index 365d964b0f6a..bc0d414a6aff 100644 > --- a/drivers/perf/cxl_pmu.c > +++ b/drivers/perf/cxl_pmu.c > @@ -419,7 +419,7 @@ static struct attribute *cxl_pmu_event_attrs[] = { > CXL_PMU_EVENT_CXL_ATTR(s2m_ndr_cmp, CXL_PMU_GID_S2M_NDR, BIT(0)), > CXL_PMU_EVENT_CXL_ATTR(s2m_ndr_cmps, CXL_PMU_GID_S2M_NDR, BIT(1)), > CXL_PMU_EVENT_CXL_ATTR(s2m_ndr_cmpe, CXL_PMU_GID_S2M_NDR, BIT(2)), > - CXL_PMU_EVENT_CXL_ATTR(s2m_ndr_biconflictack, CXL_PMU_GID_S2M_NDR, BIT(3)), > + CXL_PMU_EVENT_CXL_ATTR(s2m_ndr_biconflictack, CXL_PMU_GID_S2M_NDR, BIT(4)), > /* CXL rev 3.0 Table 3-46 S2M DRS opcodes */ > CXL_PMU_EVENT_CXL_ATTR(s2m_drs_memdata, CXL_PMU_GID_S2M_DRS, BIT(0)), > CXL_PMU_EVENT_CXL_ATTR(s2m_drs_memdatanxm, CXL_PMU_GID_S2M_DRS, BIT(1)), > -- > 2.34.1
On Tue, 6 Feb 2024 10:21:32 +0000 Jonathan Cameron <Jonathan.Cameron@Huawei.com> wrote: > On Tue, 06 Feb 2024 18:41:51 +0900 > Hojin Nam <hj96.nam@samsung.com> wrote: > > > S2M NDR BI-ConflictAck opcode is described as 4 in the CXL > > 3.0 specification. However, it is defined as 3 in macro definition. > > > Please provide a reference to a specific spec section. > Ideally from r3.1 as that's the easily available version today. > I think this is Table 3-50 S2M NDR Opcodes in r3.1 > > Looks like s2m_ndr_cmpm should be there for bit 3 so we should probably > add that as part of the fix. > Also, please add a Fixes tag. > > > Signed-off-by: Hojin Nam <hj96.nam@samsung.com> > > --- > > drivers/perf/cxl_pmu.c | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/drivers/perf/cxl_pmu.c b/drivers/perf/cxl_pmu.c > > index 365d964b0f6a..bc0d414a6aff 100644 > > --- a/drivers/perf/cxl_pmu.c > > +++ b/drivers/perf/cxl_pmu.c > > @@ -419,7 +419,7 @@ static struct attribute *cxl_pmu_event_attrs[] = { > > CXL_PMU_EVENT_CXL_ATTR(s2m_ndr_cmp, CXL_PMU_GID_S2M_NDR, BIT(0)), > > CXL_PMU_EVENT_CXL_ATTR(s2m_ndr_cmps, CXL_PMU_GID_S2M_NDR, BIT(1)), > > CXL_PMU_EVENT_CXL_ATTR(s2m_ndr_cmpe, CXL_PMU_GID_S2M_NDR, BIT(2)), > > - CXL_PMU_EVENT_CXL_ATTR(s2m_ndr_biconflictack, CXL_PMU_GID_S2M_NDR, BIT(3)), > > + CXL_PMU_EVENT_CXL_ATTR(s2m_ndr_biconflictack, CXL_PMU_GID_S2M_NDR, BIT(4)), > > /* CXL rev 3.0 Table 3-46 S2M DRS opcodes */ > > CXL_PMU_EVENT_CXL_ATTR(s2m_drs_memdata, CXL_PMU_GID_S2M_DRS, BIT(0)), > > CXL_PMU_EVENT_CXL_ATTR(s2m_drs_memdatanxm, CXL_PMU_GID_S2M_DRS, BIT(1)), > > -- > > 2.34.1 > >
diff --git a/drivers/perf/cxl_pmu.c b/drivers/perf/cxl_pmu.c index 365d964b0f6a..bc0d414a6aff 100644 --- a/drivers/perf/cxl_pmu.c +++ b/drivers/perf/cxl_pmu.c @@ -419,7 +419,7 @@ static struct attribute *cxl_pmu_event_attrs[] = { CXL_PMU_EVENT_CXL_ATTR(s2m_ndr_cmp, CXL_PMU_GID_S2M_NDR, BIT(0)), CXL_PMU_EVENT_CXL_ATTR(s2m_ndr_cmps, CXL_PMU_GID_S2M_NDR, BIT(1)), CXL_PMU_EVENT_CXL_ATTR(s2m_ndr_cmpe, CXL_PMU_GID_S2M_NDR, BIT(2)), - CXL_PMU_EVENT_CXL_ATTR(s2m_ndr_biconflictack, CXL_PMU_GID_S2M_NDR, BIT(3)), + CXL_PMU_EVENT_CXL_ATTR(s2m_ndr_biconflictack, CXL_PMU_GID_S2M_NDR, BIT(4)), /* CXL rev 3.0 Table 3-46 S2M DRS opcodes */ CXL_PMU_EVENT_CXL_ATTR(s2m_drs_memdata, CXL_PMU_GID_S2M_DRS, BIT(0)), CXL_PMU_EVENT_CXL_ATTR(s2m_drs_memdatanxm, CXL_PMU_GID_S2M_DRS, BIT(1)),