Received: by 2002:a05:6358:11c7:b0:104:8066:f915 with SMTP id i7csp5698117rwl; Tue, 11 Apr 2023 08:48:59 -0700 (PDT) X-Google-Smtp-Source: AKy350Y1ygaRl2DwmYqQL9wjgabYR6SawS1iYhkpjluAOdd9X+k6BDZerGsuvNM1PKmuiDWRfwfy X-Received: by 2002:a17:906:3958:b0:94a:4ec9:cf27 with SMTP id g24-20020a170906395800b0094a4ec9cf27mr7684655eje.20.1681228138672; Tue, 11 Apr 2023 08:48:58 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1681228138; cv=none; d=google.com; s=arc-20160816; b=Q0zb+FGa7Qd6aDNK7U7UmYMk15x/+t7W2Dcz+Agn8B256VEOu+RbVUAwIlqwdaK67G NrjMuq58JuS2ElYBfP4HEzVekQGrW7FzzK3XfyRhD2Vr71/fH1ZLtJ/ev9IQ/BRN4Xww ghLMYlf5gQNsjAeIDKctIvpdeyaO5NLObzCx5qo9bmdWZCGXFW8T9+IP+S3MKZVMPkJg 9u7FG/NfE6j19jEVCtoMU1iP/DIgHPgc+1QZSPZ7i6OXLYp3w8NFGiYko0KT9OIyvXh5 Ga3zZBsbC/GDYP/+fe6hPI/qNcmkeUJo0RAOw50dctRJjzyS2Um0ilkHsfl/U5B+tMrs KaPQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:in-reply-to:from :references:cc:to:content-language:subject:user-agent:mime-version :date:message-id; bh=vLyCWly/eJmtOXLrwaK5Z9qqBjnR4+fSEaIh4butYaY=; b=wbKtuKJDARZXBSXouRPdNC4ZuihEej7UNzBYtcE6JeEJOCW/nOSw/PcX0kHkj9g+Tu 2Pp1jjSCUV8yXwZbMwKm3qURi5OM2csgNNbX5jNItt4RFN1TCqNI+UhwIsOqPPqpnzHl 70rpv+gx0zo5j+rIBoUAVBl+uv2PaxLMcbd4RoKkKiAdT3rkcC9wyfESjNhNimnur3pa KD+6Ha0FlP4HpUB1qramip5zW7SFBJd8U6iky9Z1Xgl3Q4ClGnfVvNJnuKhGn7LdGUVy rxLKsVGbif91ieo617Mr3NJOvcKCu4dpsVeqpTjZYCOLUg9j7QhutBUasG5Bi+bAtuwH cRNw== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=arm.com Return-Path: Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id du15-20020a17090772cf00b0093e18eb2c07si1403713ejc.227.2023.04.11.08.48.33; Tue, 11 Apr 2023 08:48:58 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20; Authentication-Results: mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=arm.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S229955AbjDKPk5 (ORCPT + 99 others); Tue, 11 Apr 2023 11:40:57 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:59768 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229643AbjDKPk4 (ORCPT ); Tue, 11 Apr 2023 11:40:56 -0400 Received: from foss.arm.com (foss.arm.com [217.140.110.172]) by lindbergh.monkeyblade.net (Postfix) with ESMTP id DD18635B3; Tue, 11 Apr 2023 08:40:52 -0700 (PDT) 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 0CDB8FEC; Tue, 11 Apr 2023 08:41:37 -0700 (PDT) Received: from [10.57.20.231] (unknown [10.57.20.231]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id 897CC3F73F; Tue, 11 Apr 2023 08:40:49 -0700 (PDT) Message-ID: Date: Tue, 11 Apr 2023 17:40:47 +0200 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Thunderbird/102.9.0 Subject: Re: [PATCH 02/17] PM: EM: Find first CPU online while updating OPP efficiency Content-Language: en-US To: Lukasz Luba , linux-kernel@vger.kernel.org, linux-pm@vger.kernel.org, rafael@kernel.org Cc: dietmar.eggemann@arm.com, rui.zhang@intel.com, amit.kucheria@verdurent.com, amit.kachhap@gmail.com, daniel.lezcano@linaro.org, viresh.kumar@linaro.org, len.brown@intel.com, pavel@ucw.cz, ionela.voinescu@arm.com, rostedt@goodmis.org, mhiramat@kernel.org References: <20230314103357.26010-1-lukasz.luba@arm.com> <20230314103357.26010-3-lukasz.luba@arm.com> From: Pierre Gondois In-Reply-To: <20230314103357.26010-3-lukasz.luba@arm.com> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit X-Spam-Status: No, score=-4.5 required=5.0 tests=NICE_REPLY_A, RCVD_IN_DNSWL_MED,SPF_HELO_NONE,SPF_NONE autolearn=unavailable autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hello Lukasz, On 3/14/23 11:33, Lukasz Luba wrote: > The Energy Model might be updated at runtime and the energy efficiency > for each OPP may change. Thus, there is a need to update also the > cpufreq framework and make it aligned to the new values. In order to > do that, use a first online CPU from the Performance Domain. > > Signed-off-by: Lukasz Luba > --- > kernel/power/energy_model.c | 11 +++++++++-- > 1 file changed, 9 insertions(+), 2 deletions(-) > > diff --git a/kernel/power/energy_model.c b/kernel/power/energy_model.c > index 265d51a948d4..3d8d1fad00ac 100644 > --- a/kernel/power/energy_model.c > +++ b/kernel/power/energy_model.c > @@ -246,12 +246,19 @@ em_cpufreq_update_efficiencies(struct device *dev, struct em_perf_state *table) > struct em_perf_domain *pd = dev->em_pd; > struct cpufreq_policy *policy; > int found = 0; > - int i; > + int i, cpu; > > if (!_is_cpu_device(dev) || !pd) > return; Since dev is a CPU, I think it shouldn be possible to get the cpu id via 'dev->id'. If so the code below should not be necessary anymore. > > - policy = cpufreq_cpu_get(cpumask_first(em_span_cpus(pd))); > + /* Try to get a CPU which is online and in this PD */ > + cpu = cpumask_first_and(em_span_cpus(pd), cpu_active_mask); > + if (cpu >= nr_cpu_ids) { > + dev_warn(dev, "EM: No online CPU for CPUFreq policy\n"); > + return; > + } > + > + policy = cpufreq_cpu_get(cpu); > if (!policy) { > dev_warn(dev, "EM: Access to CPUFreq policy failed"); > return; Regards, Pierre