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=-10.5 required=3.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,INCLUDES_CR_TRAILER,MAILING_LIST_MULTI,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 7C36BC433ED for ; Thu, 8 Apr 2021 13:29:51 +0000 (UTC) Received: from desiato.infradead.org (desiato.infradead.org [90.155.92.199]) (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 1A0FF6113A for ; Thu, 8 Apr 2021 13:29:51 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 1A0FF6113A 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+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=desiato.20200630; h=Sender:Content-Transfer-Encoding :Content-Type:List-Subscribe:List-Help:List-Post:List-Archive: List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:References:Message-ID: Subject:Cc: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=D7KF5ceDUdG4Dv5Jm27b/AJ4VfBz4rif//dKRWl5bA0=; b=IfqAXcITGykM/PgVRHM00a2x4 05NQkPGnYPfVhG1iPNCbS3ITsgBdnnjr1G8oSTxw1CGcUKbZmeK2JLQalHrQUJv9/3XSEifjvgChM SKRVL6TLW7YHUhyYcXAMISkLFirDuwsPBfsIsCXHBaAvL2fYiVoXcDuVuCjpEg4p4L35tAC5tvoaV 9QLdiG2Xr/BSwKvwYcvwsmplb2IQ6y1szp0hc6k/az61OLM/h1Y0ftkE/XW9mDhU5kiPH0qKG5Y0h fdbPUkOVN+pTYmohbI2mQdPhkPSKB36KdvEukUf9XO4xIMIHxWKrX8mFEcw1zGNyjTEePdDe41iO0 Klxbpl0TA==; Received: from localhost ([::1] helo=desiato.infradead.org) by desiato.infradead.org with esmtp (Exim 4.94 #2 (Red Hat Linux)) id 1lUUhn-0087uI-7Z; Thu, 08 Apr 2021 13:28:19 +0000 Received: from mail.kernel.org ([198.145.29.99]) by desiato.infradead.org with esmtps (Exim 4.94 #2 (Red Hat Linux)) id 1lUUhh-0087sk-4b for linux-arm-kernel@lists.infradead.org; Thu, 08 Apr 2021 13:28:15 +0000 Received: by mail.kernel.org (Postfix) with ESMTPSA id B3BB561107; Thu, 8 Apr 2021 13:28:08 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1617888490; bh=wkH0Q0JKhFFGHpi0excDmQI/Xn0hOyg48Ia2aLs0W14=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=L//1DccqA0sW9OR7Y58lptQc2UwfX8WREzKYBQMgr8+D/Ivol4Li85zKayJbxODCg snvD1dZp53fnbSPnZ8aoM1Ed5K4dC9wm5IxrxRUbUREdEmcQrYVj8PwwUNXZnjb13u HJfSr3K9ne7+PMcJqReG/i8TjxFmsDGW2CIh2yudJhejWPX+8BXyGkgnUvskVyU08s rf7Nh8FgKYN6YkjLYkgokniWI94zGY5EtEGIC7HcHmm07H9NQW/eaEOdqmX6gwpFo2 g+lFtTPSOwkWk6Ze/32Y4WzJ9NYYOyeuyk+yYqXMQrCoaRhGzlnSq3xcHCiEWTnZyB yT0G3xPSZLZUw== Date: Thu, 8 Apr 2021 14:28:05 +0100 From: Will Deacon To: John Garry Cc: Jonathan Cameron , "liuqi (BA)" , "mark.rutland@arm.com" , Zhangshaokun , Linuxarm , "linux-arm-kernel@lists.infradead.org" , "linux-kernel@vger.kernel.org" Subject: Re: [PATCH 1/2] drivers/perf: hisi: Add driver for HiSilicon PCIe PMU Message-ID: <20210408132805.GA17873@willie-the-truck> References: <1617788943-52722-1-git-send-email-liuqi115@huawei.com> <1617788943-52722-2-git-send-email-liuqi115@huawei.com> <20210407204004.GB16011@willie-the-truck> <20210408100157.000062f0@Huawei.com> <16a7a3f8-82b5-a113-57c0-a9d575c649bd@huawei.com> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <16a7a3f8-82b5-a113-57c0-a9d575c649bd@huawei.com> User-Agent: Mutt/1.10.1 (2018-07-13) X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20210408_142813_458981_803C06B7 X-CRM114-Status: GOOD ( 22.03 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org Hi John, On Thu, Apr 08, 2021 at 01:55:02PM +0100, John Garry wrote: > On 08/04/2021 10:01, Jonathan Cameron wrote: > > On Wed, 7 Apr 2021 21:40:05 +0100 > > Will Deacon wrote: > > > > > On Wed, Apr 07, 2021 at 05:49:02PM +0800, Qi Liu wrote: > > > > PCIe PMU Root Complex Integrated End Point(RCiEP) device is supported > > > > to sample bandwidth, latency, buffer occupation etc. > > > > > > > > Each PMU RCiEP device monitors multiple root ports, and each RCiEP is > > > > registered as a pmu in /sys/bus/event_source/devices, so users can > > > > select target PMU, and use filter to do further sets. > > > > > > > > Filtering options contains: > > > > event - select the event. > > > > subevent - select the subevent. > > > > port - select target root ports. Information of root ports > > > > are shown under sysfs. > > > > bdf - select requester_id of target EP device. > > > > trig_len - set trigger condition for starting event statistics. > > > > trigger_mode - set trigger mode. 0 means starting to statistic when > > > > bigger than trigger condition, and 1 means smaller. > > > > thr_len - set threshold for statistics. > > > > thr_mode - set threshold mode. 0 means count when bigger than > > > > threshold, and 1 means smaller. > > > > > > > > Reviewed-by: Jonathan Cameron > > > > > > Do you have a link to this review, please? > > > > Internal review, so drop the tag. > > > > Jonathan > > Hi Will, > > Are you implying that you would rather that any review for these drivers is > done in public on the lists? Absolutely! If I can see that you and/or Jonathan have given the thing a good going through, then it's a lot easier to merge the patches. But just having the tag doesn't help much, as I don't know whether it was a concerted review effort or a "yeah, this function is about what I thought, cheers" type of review. That's not to say internal patch review isn't a useful tool in some circumstances (e.g. somebody new to the kernel, confidential stuff, prototyping), but the vast majority of the time I'd say having the review on the public lists is the best bet. Will _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel