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 3247DE7718B for ; Thu, 19 Dec 2024 10:38:53 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:List-Subscribe:List-Help :List-Post:List-Archive:List-Unsubscribe:List-Id:In-Reply-To:Content-Type: MIME-Version:References:Message-ID:Subject:Cc:To:From:Date:Reply-To: Content-Transfer-Encoding:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=Ceh++z+WqXGM46INc/W89OSTJw8DjpL2idRVtq4UZjM=; b=qbSDMV3xVcAxZTdUzNc/v9XZ4X +2xvHSyT8ZY+eFeqtPxe555MIHI4zsAMv/fuugS3eZGjvHDl7RpHZKBEP+Aw1WqMM1hHedyRs62il 2agVn3zDKQ6qbJinjaWnJTxdCsewpOQivu/370jO8PEb0WqWrDMRw2FTTkAsPXzgYw0eYjm7ptAyF QoD9enXf7wVH81zFt6UnpiXmFe0Co0tk5KgPOPIOAXaKn1gup2Trbyy+6nINM18sagG2A3mX1WEpm PXYMLaUNqJhUiqz0DFA5mv+F8BaS+C7JmMGVjQ1Y2Yde5qQ5OrUpcEg/FOFBVoPDNAH2tdn6tmGnj cNuBBZqQ==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.98 #2 (Red Hat Linux)) id 1tODvU-00000001YuY-1UH3; Thu, 19 Dec 2024 10:38:40 +0000 Received: from nyc.source.kernel.org ([147.75.193.91]) by bombadil.infradead.org with esmtps (Exim 4.98 #2 (Red Hat Linux)) id 1tODuO-00000001Ynm-0N68 for linux-arm-kernel@lists.infradead.org; Thu, 19 Dec 2024 10:37:33 +0000 Received: from smtp.kernel.org (transwarp.subspace.kernel.org [100.75.92.58]) by nyc.source.kernel.org (Postfix) with ESMTP id E52EAA42073; Thu, 19 Dec 2024 10:35:39 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 184B0C4CED0; Thu, 19 Dec 2024 10:37:30 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1734604650; bh=6NtTS3sFBKTuhe+YrFYJjIB7PeX5OysvL/IIYf8a964=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=nSzDAU+HqyERLsxyx3tZBNRur3PX7/DH8uv9Fpzuq4hQlJ9wE2jv4yLtUgfx5rnDI +L0RvXFLDuUdFu7tUinxgfzFVWB/a1EXCdy3t8JOafqaeMLXgJB9CKLT3aUVO3Uc57 weaGduU7CW427tXzwZkbaokDGw1tRh3RnTnPTMaoOWzyBy12FYH/0KM8kTLdiUDuXV h22oGMfLRMWV/iYrINSxXPv+M8pqgQZHwC55zVMKxBxSwXtSMydib5gjudjCt5yikj rB2W8QWckzJNiITHvlA+jXKLa/zH8loNuhGgJNbhH+IOjMmu4+fnFXbjz0o+JDAUp2 o8dEXEQJtmo5Q== Received: from johan by xi.lan with local (Exim 4.97.1) (envelope-from ) id 1tODuQ-000000001mQ-2Z6I; Thu, 19 Dec 2024 11:37:34 +0100 Date: Thu, 19 Dec 2024 11:37:34 +0100 From: Johan Hovold To: Sibi Sankar Cc: Cristian Marussi , sudeep.holla@arm.com, andersson@kernel.org, konrad.dybcio@linaro.org, robh+dt@kernel.org, krzysztof.kozlowski+dt@linaro.org, linux-kernel@vger.kernel.org, linux-arm-msm@vger.kernel.org, devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, quic_rgottimu@quicinc.com, quic_kshivnan@quicinc.com, conor+dt@kernel.org, arm-scmi@vger.kernel.org, Ettore Chimenti Subject: Re: [PATCH V4 0/5] arm_scmi: vendors: Qualcomm Generic Vendor Extensions Message-ID: References: <20241007061023.1978380-1-quic_sibis@quicinc.com> <8d42682b-0fa7-3962-da12-728cfe64903b@quicinc.com> <132d1404-9009-9fb5-1fb9-63eca03ce9fc@quicinc.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <132d1404-9009-9fb5-1fb9-63eca03ce9fc@quicinc.com> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20241219_023732_261175_7640D835 X-CRM114-Status: GOOD ( 22.92 ) 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: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On Tue, Dec 17, 2024 at 05:19:25PM +0530, Sibi Sankar wrote: > On 12/5/24 21:22, Johan Hovold wrote: > > On Thu, Dec 05, 2024 at 04:26:55PM +0530, Sibi Sankar wrote: > >> On 11/22/24 14:07, Johan Hovold wrote: > > > >>> I have a Lenovo ThinkPad T14s set up now so I gave this series a spin > >>> there too, and there I do *not* see the above mentioned -EOPNOSUPP error > >>> and the memlat driver probes successfully. > >>> > >>> On the other hand, this series seems to have no effect on a kernel > >>> compilation benchmark. Is that expected? > >> > >> I can have a look at your tree. But memlat in general > >> depends on the cpu frequency when your benchmarks max > >> the cpu's the ddr/llcc are scaled accordingly by it. > > > > A kernel compilation should max out the CPU frequency on all cores. Answering my own question here; bwmon should scale the buses for benchmarks like kernel compilations so I guess the non-existing impact of memlat is expected here. Ettore helped me run some further benchmarks, including cachebench, but also saw no positive (or negative) effect with this series running on an X1E CRD (with recent firmware). Do you have any suggestions of benchmarks to run where the effect of memlat should show up? What have you been using for testing? I did measure a possibly slightly higher (idle) power consumption with memlat, but I guess that is also expected given the intended more aggressive ramping of the bus clocks. These are the branches (and configs; johan_defconfig) we've used for testing: https://github.com/jhovold/linux/tree/wip/x1e80100-6.13-rc3 https://github.com/jhovold/linux/tree/wip/x1e80100-6.13-rc3-memlat > >>> And does this mean that you should stick with the uppercase "MEMLAT" > >>> string after all? The firmware on my CRD is not the latest one, but I am > >>> using the latest available firmware for the T14s. > >> > >> We should stick with "memlat" if we run into a device in the > >> wild that doesn't support "MEMLAT" > > > > Ok. So the updated firmware supports both strings? > > Sry for the delay, was out sick. Yes the updated firmware supports both > strings. No worries, hope you're feeling better. I noticed that the firmware on the T14s indeed accepts both strings. Johan