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 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 smtp.lore.kernel.org (Postfix) with ESMTPS id 2DF00C47DB3 for ; Wed, 31 Jan 2024 11:29:15 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; 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=nhfPHIG50zsVEfYP7npWZXwbge9kkTI7KJzu2XS2K6M=; b=xzS/R9Bgw/+ytn ZZZ4RR9LAtge+mxp2/oKfG8dZN4J9a3ceAbSuTuWpyPD5x2cOvUJMggTO27Ki2xbKQ4cTTnQj/Em0 hYtKu4YHEYthLMQ9RNnEoSenfCsAyvg3PP/n2TYPSp0GMrvQvUCBV0R0vIWXmV47XuJO4ItZLyQXb ZzOm60+YWujSPECh5p5XkZPXkoivwr0O2Te4/w7ygKVD4Qxq4IE6z//OD3dJoDtrHI6VXMudnvP4C bUfZJBfv6fBZBQMCHIBPis+cADdnzYVU3j/O8FacAeuxCmdReRgY90KwiyjXg/7krB67pa3Bo/rU3 QSaKw8ZaCSAR3TvWZZoQ==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.97.1 #2 (Red Hat Linux)) id 1rV8m8-000000035va-4AhL; Wed, 31 Jan 2024 11:29:04 +0000 Received: from foss.arm.com ([217.140.110.172]) by bombadil.infradead.org with esmtp (Exim 4.97.1 #2 (Red Hat Linux)) id 1rV8m6-000000035uE-1Z3q for linux-arm-kernel@lists.infradead.org; Wed, 31 Jan 2024 11:29:03 +0000 Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.121.207.14]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id ABF94DA7; Wed, 31 Jan 2024 03:29:43 -0800 (PST) Received: from bogus (unknown [10.57.78.35]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id 7FF973F762; Wed, 31 Jan 2024 03:28:57 -0800 (PST) Date: Wed, 31 Jan 2024 11:28:54 +0000 From: Sudeep Holla To: Cristian Marussi Cc: Sibi Sankar , rafael@kernel.org, Sudeep Holla , viresh.kumar@linaro.org, morten.rasmussen@arm.com, dietmar.eggemann@arm.com, lukasz.luba@arm.com, linux-arm-kernel@lists.infradead.org, linux-pm@vger.kernel.org, linux-kernel@vger.kernel.org, quic_mdtipton@quicinc.com, linux-arm-msm@vger.kernel.org Subject: Re: [PATCH V2 1/4] firmware: arm_scmi: Add perf_notify_support interface Message-ID: <20240131112854.24euh6c6xgljsj6c@bogus> References: <20240117104116.2055349-1-quic_sibis@quicinc.com> <20240117104116.2055349-2-quic_sibis@quicinc.com> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20240131_032902_478417_22AAD430 X-CRM114-Status: GOOD ( 18.19 ) 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 On Mon, Jan 29, 2024 at 05:33:42PM +0000, Cristian Marussi wrote: > On Mon, Jan 29, 2024 at 03:50:20PM +0000, Cristian Marussi wrote: > > On Wed, Jan 17, 2024 at 04:11:13PM +0530, Sibi Sankar wrote: > > > Add a new perf_notify_support interface to the existing perf_ops to export > > > info regarding limit/level change notification support. > > > > > > > Hi Sibi, > > > > as I mentioned previously, in order not to add a needless stream of SCMI > > Perf accessors I posted this: > > > > https://lore.kernel.org/linux-arm-kernel/20240129151002.1215333-1-cristian.marussi@arm.com/T/#u > > > > to expose all the Perf domains infos via the usual info_get(), similarly > > to how other SCMI protocols do already. > > > > I think that reworking this series on that, you can certainly drop this patch and just > > check the _notify booleans on the retrieved domain info. > > Sorry, but hold on with this change, I will probably post an updated version > my patch above. > As discussed in private, I would prefer to avoid exposing all the internals to the users of SCMI perf. At the same time may we can do better if we can check the availability of notification as part of notification enablement from the SCMI driver, I need to think the details yet. -- Regards, Sudeep _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel