From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-8.5 required=3.0 tests=DKIMWL_WL_HIGH,DKIM_SIGNED, DKIM_VALID,INCLUDES_PATCH,MAILING_LIST_MULTI,SIGNED_OFF_BY,SPF_HELO_NONE, SPF_PASS,URIBL_BLOCKED,USER_AGENT_SANE_1 autolearn=unavailable autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id B7460C3A59E for ; Wed, 21 Aug 2019 16:53:52 +0000 (UTC) Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id 8931A22DD3 for ; Wed, 21 Aug 2019 16:53:52 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=lists.infradead.org header.i=@lists.infradead.org header.b="BIF7alrv"; dkim=fail reason="signature verification failed" (1024-bit key) header.d=kernel.org header.i=@kernel.org header.b="mfA3TYZ5" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 8931A22DD3 Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=kernel.org Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-arm-kernel-bounces+infradead-linux-arm-kernel=archiver.kernel.org@lists.infradead.org DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20170209; h=Sender: Content-Transfer-Encoding:Content-Type:Cc:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:References: Message-ID:Subject:To:From:Date:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=XM/2mZSgEfBKw4DQVHtf1lsGWHiDk3UTeDlsqPjh0bI=; b=BIF7alrvvBfhko 5L1RmNbzv4q194Dg0IkaPCaIoxTbABA46HNIcdZ0U+OCdgZD4+WNR4cjRdD8fxNlyLMI1qBT4cdqI hZwvVqCK3TjuE5kdIM0qs4OLs6pPRowVcGD8OtsL9lwpYC0jljXBv9z7kr6PP5yzj1VKIS8xmgJ5R TL5ygpIzTQk7+8Eh8noPa2IgjaZI1RkuQaYR/nOcePFD3/Z6KxXDypOEWfRqx/zyWxnGaJTMWPvja 9qLGFvLst6bHTylvAp4sAQGBMIop0fESvO+krPJoJXLtPR0RjypmMNv0IKDAPwP4chIwFUUdGefvp puUMnfDzpSgYxHhDRvSQ==; Received: from localhost ([127.0.0.1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.92 #3 (Red Hat Linux)) id 1i0Trr-0003JN-SA; Wed, 21 Aug 2019 16:53:51 +0000 Received: from mail.kernel.org ([198.145.29.99]) by bombadil.infradead.org with esmtps (Exim 4.92 #3 (Red Hat Linux)) id 1i0Tro-0003IA-BT for linux-arm-kernel@lists.infradead.org; Wed, 21 Aug 2019 16:53:49 +0000 Received: from willie-the-truck (236.31.169.217.in-addr.arpa [217.169.31.236]) (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 9699122DD3; Wed, 21 Aug 2019 16:53:42 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=default; t=1566406424; bh=8EpYXOIJQ7TePwZ/ZCJnzDEkdpUgKmzzxBX02HTYDWc=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=mfA3TYZ5mReb6a41G+2FUsWZ4XDr3vM5MuLTd+xih2gFnd/B+p3syzRto7GLy1yCP yNbNLlyJluiuGn/UDS0ljvkiKAsdh75khbDmuBybOByPBaXAfA3M/PM56Bd93ckdf7 Uz4JeqWNifNtxOjNgPEvRWiDgS+Ymy2V+CFJCsZE= Date: Wed, 21 Aug 2019 17:53:39 +0100 From: Will Deacon To: Mark Rutland Subject: Re: [PATCH v3 2/2] drivers/perf: Add CCPI2 PMU support in ThunderX2 UNCORE driver. Message-ID: <20190821165339.7gu4rxkvdjcr4mta@willie-the-truck> References: <1563873380-2003-1-git-send-email-gkulkarni@marvell.com> <1563873380-2003-3-git-send-email-gkulkarni@marvell.com> <20190812120125.GA50712@lakrids.cambridge.arm.com> <20190813110345.GD866@lakrids.cambridge.arm.com> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <20190813110345.GD866@lakrids.cambridge.arm.com> User-Agent: NeoMutt/20170113 (1.7.2) X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20190821_095348_427672_B3F44F4F X-CRM114-Status: GOOD ( 24.98 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Ganapatrao Kulkarni , "corbet@lwn.net" , Jan Glauber , "linux-doc@vger.kernel.org" , "linux-kernel@vger.kernel.org" , Robert Richter , Ganapatrao Kulkarni , Jayachandran Chandrasekharan Nair , "linux-arm-kernel@lists.infradead.org" Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+infradead-linux-arm-kernel=archiver.kernel.org@lists.infradead.org On Tue, Aug 13, 2019 at 12:03:45PM +0100, Mark Rutland wrote: > On Tue, Aug 13, 2019 at 04:25:15PM +0530, Ganapatrao Kulkarni wrote: > > On Mon, Aug 12, 2019 at 5:31 PM Mark Rutland wrote: > > > > > > On Tue, Jul 23, 2019 at 09:16:28AM +0000, Ganapatrao Kulkarni wrote: > > > > CCPI2 is a low-latency high-bandwidth serial interface for connecting > > > > ThunderX2 processors. This patch adds support to capture CCPI2 perf events. > > > > > > It would be worth pointing out in the commit message how the CCPI2 > > > counters differ from the others. I realise you have that in the body of > > > patch 1, but it's critical information when reviewing this patch... > > > > Ok, I will add in next version. > > > > > > > > > > > Signed-off-by: Ganapatrao Kulkarni > > > > --- > > > > drivers/perf/thunderx2_pmu.c | 248 ++++++++++++++++++++++++++++++----- > > > > 1 file changed, 214 insertions(+), 34 deletions(-) > > > > > > > > diff --git a/drivers/perf/thunderx2_pmu.c b/drivers/perf/thunderx2_pmu.c > > > > index 43d76c85da56..a4e1273eafa3 100644 > > > > --- a/drivers/perf/thunderx2_pmu.c > > > > +++ b/drivers/perf/thunderx2_pmu.c > > > > @@ -17,22 +17,31 @@ > > > > */ > > > > > > > > #define TX2_PMU_MAX_COUNTERS 4 > > > > > > Shouldn't this be 8 now? > > > > It is kept unchanged to 4(as suggested by Will), which is same for > > both L3 and DMC. > > For CCPI2 this macro is not used. > > Hmmm.... > > I disagree with that suggestion given that this also affects the > active_counters bitmap size (and thus it is not correctly sized as of > this patch), and it doesn't really save us much. > > I think it would be better to bump this to 8 and always update the > events array, even though it will be unused for CCPI2. That's less > surprising, needs fewer special-cases, and we can use the hrtimer > function pointer alone to determine if we need to do any hrtimer work. tbf, my complaint was actually about some macros applying to the whole PMU whilst others refer only to DMC/L3C and this not being apparent from the naming: https://lkml.org/lkml/2019/6/27/250 so I'm fine having TX2_PMU_DMC_L3C_MAX_COUNTERS and TX2_PMU_CCPI2_MAX_COUNTERS, but that sort of naming needs to be consistent unless the macro/definition really applies to both. That fed the suggestion that GET_EVENTID could be generic and switch on the event type internally instead of at the caller. Will _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel