Received: by 2002:a05:6a10:16a7:0:0:0:0 with SMTP id gp39csp936946pxb; Thu, 5 Nov 2020 18:00:09 -0800 (PST) X-Google-Smtp-Source: ABdhPJwQC3WMmn8ssMZ209yM3AfIJR3vNzmlosGbdk0qvRASwg0uJFrYJV7oK7Cc95kvgQUtTFfO X-Received: by 2002:a17:906:d20e:: with SMTP id w14mr5145842ejz.479.1604628009487; Thu, 05 Nov 2020 18:00:09 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1604628009; cv=none; d=google.com; s=arc-20160816; b=hGF0S0m1A/RcimOkFA5sIujIcSucYWg7J9rrqo77Ackl5fdQU8yVEKaEjnyRzzioxd B5NLngDSCVOvulExc5+yyIEu28KVjcyK5n1NevpvxpIpqJ/PUKkvWLwBSi21SWcMACeG 551xgng2t3rx40fAIOoMzghSPi573r78VA1anpvVjZIYkrNcLDhuQYUQBx6dF3vghv7e UFyxSNe85XnrySVWLc1ABOOMzFOV28C2k7HkSjaxZOwcqgRui+Ra46tSP0/XoFuKZQJW TqrapHZRzur4THMrPfD7dvCl0KBCHmIHA/j+VoC8WxRruxm7+BjIZn+IwjuTrPdoCeii 7iXQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:thread-index:content-language :content-transfer-encoding:mime-version:message-id:date:subject :in-reply-to:references:cc:to:from:dkim-signature; bh=EQ2VgUbGgV+uFwLbFWhleNcbVstRS/kO5xtYuUuBq24=; b=PqT2S2n/eOO0OXW8zCWefjJBMuFawuXoAksRb8ouovZxW1wijg1s1jCqlN6YSGMI3d Ws2Yh7+5d8+RhQW3jMezA4Ft/JCNn9gXNIW7K7JoXOvn/J8pS1S/CH5BLQEU1V5qy+AS NvCYToViha1ApI3LU0B7yrRl0RwMeyu95Wz6izaDy3WEhrWkWuH3xBShYFTBby/tiXf9 3UyHpF8TrEILW4zNo/fEg/O/rJ80G+hb+6yMFVeIfNNt+YkyiO7OIgNdy0sHxTzDInzh swt2EiIMCEY8Zt8mOtj8GWxsNqx8kzLaCLa+uYn1mJL07bIRAGCTUy2Ic7CopDhU8WH1 koiw== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass (test mode) header.i=@telus.net header.s=neo header.b=YK1ZmjXW; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=telus.net Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id x3si2775054ede.137.2020.11.05.17.59.47; Thu, 05 Nov 2020 18:00:09 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; dkim=pass (test mode) header.i=@telus.net header.s=neo header.b=YK1ZmjXW; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=telus.net Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1732817AbgKFB6H (ORCPT + 99 others); Thu, 5 Nov 2020 20:58:07 -0500 Received: from cmta19.telus.net ([209.171.16.92]:32941 "EHLO cmta19.telus.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1730414AbgKFB6H (ORCPT ); Thu, 5 Nov 2020 20:58:07 -0500 X-Greylist: delayed 488 seconds by postgrey-1.27 at vger.kernel.org; Thu, 05 Nov 2020 20:58:06 EST Received: from dougxps ([173.180.45.3]) by cmsmtp with SMTP id aqt0kwPg7dLkEaqt1kZKc4; Thu, 05 Nov 2020 18:49:57 -0700 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=telus.net; s=neo; t=1604627397; bh=EQ2VgUbGgV+uFwLbFWhleNcbVstRS/kO5xtYuUuBq24=; h=From:To:Cc:References:In-Reply-To:Subject:Date; b=YK1ZmjXW5mhpM+I0VP6mno26vEh3v016ESNf5CQhoPBNDclhNCxwvPUaA+fGenvNl adS2nDJTZA95HanowQ0AsUG3mTaYDUTKwGO0DSHyF0/sFdiGVr4/RCxxjKxjSmGqO+ a1Tp767cLvRZUiKq6NEn20ubsopYSxrhMc+6iQGIwFwurAr4F5f+fs0S0KL9xacm8w 2h5u6RGWbW03BQi6PUCZCxNiyi81LYsNV/BDI8svtCS4RFpLb5qtzrAI5VypmyQ62O 6z6X/6VGReTjHTG6alMn58ZHA6sjwWaZpXqxAl/atDMq4s+BDvo6kTr6Vd+mVY1UkL +7OpSr5YQn+xw== X-Telus-Authed: none X-Authority-Analysis: v=2.4 cv=WPm64lgR c=1 sm=1 tr=0 ts=5fa4abc5 a=ZeVyObKPoMU90SgYCeSZ1g==:117 a=ZeVyObKPoMU90SgYCeSZ1g==:17 a=Pyq9K9CWowscuQLKlpiwfMBGOR0=:19 a=kj9zAlcOel0A:10 a=QyXUC8HyAAAA:8 a=gu6fZOg2AAAA:8 a=8THPvSkuJoZwLrQ9cBcA:9 a=CjuIK1q_8ugA:10 a=-FEs8UIgK8oA:10 a=NWVoK91CQyQA:10 a=2RSlZUUhi9gRBrsHwhhZ:22 From: "Doug Smythies" To: "'Rafael J. Wysocki'" , "'Linux PM'" Cc: "'Rafael J. Wysocki'" , "'Viresh Kumar'" , "'Srinivas Pandruvada'" , "'Zhang Rui'" , "'LKML'" References: <7417968.Ghue05m4RV@kreacher> <2233690.N3OVLkotou@kreacher> In-Reply-To: <2233690.N3OVLkotou@kreacher> Subject: RE: [PATCH 1/2] cpufreq: Introduce target min and max frequency hints Date: Thu, 5 Nov 2020 17:49:53 -0800 Message-ID: <001b01d6b3df$21165940$63430bc0$@net> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit X-Mailer: Microsoft Office Outlook 12.0 Content-Language: en-ca Thread-Index: AdazoQzOa64DtwxHQV2ivz0yvkdgTgAM4bQw X-CMAE-Envelope: MS4xfOnrGS/Thze5mEGyKkwj5SyTIuz7k6xioMBYgNxOS3r1kTiTTBoQHV/3fPgx5sDCTVmwN5AqW+TL2VAoRyjciQd389sWRFRcFPfuXto90COudkUsNIQB vWfHuai3mED8GwYSRWjD0E8wrb15q/bOUnvyb9D+A/3dOOxe1zRdLI7prRMmG4MXEB9GtbqquDdm7B4N7ifyfAeWX7yJxvgwDOj51M4f6DwSCqxcgslXJ5RD PdNvvTWhZSSiZPleL+BQp2s+tSY4eCvDVSaWhdn8sweclJA8mL/dEQxZ31lradvz5aOjac+CSnvqHFHctWcTF14X7rr1iU8dLqdY9W2b8TwioxLiQjlg9DCv q0R5+k50wfsIicLcLqICdWrKMt8YedaItQHEHZFMkzvlWfuhBKI= Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Rafael: Thank you for this patch set. I can not get the patch to apply. I was trying on top on 5.10-rc2, and have been unable to determine what other patches might need to be applied first. On 2020.11.05 10:24 Rafael J. Wysocki wrote: ... > > Signed-off-by: Rafael J. Wysocki > --- > drivers/cpufreq/cpufreq.c | 3 +++ > drivers/cpufreq/cpufreq_performance.c | 4 ++++ > drivers/cpufreq/cpufreq_powersave.c | 4 ++++ > include/linux/cpufreq.h | 16 ++++++++++++++++ I do not understand why this part says to look for 16 differences, but I can only find 2. > 4 files changed, 27 insertions(+) > > Index: linux-pm/include/linux/cpufreq.h > =================================================================== > --- linux-pm.orig/include/linux/cpufreq.h > +++ linux-pm/include/linux/cpufreq.h > @@ -63,6 +63,8 @@ struct cpufreq_policy { > > unsigned int min; /* in kHz */ > unsigned int max; /* in kHz */ > + unsigned int target_min; /* in kHz */ > + unsigned int target_max; /* in kHz */ > unsigned int cur; /* in kHz, only needed if cpufreq > * governors are used */ > unsigned int suspend_freq; /* freq to set during suspend */ > Index: linux-pm/drivers/cpufreq/cpufreq.c ... Anyway, I edited the patch, deleting the include/linux/cpufreq.h part, then it applied, as did patch 2 of 2. I edited include/linux/cpufreq.h manually. Issues with the powersave governor reported in [1] and [2] are fixed. Relevant part quoted and updated below: > In early September Doug wrote: >> powersave governor: >> acpi-cpufreq: good >> intel_cpufreq hwp: bad Now good, with this patch set. >> intel_cpufreq no hwp: good ... > For the powersave governor, this is what we have now: > > intel_cpufreq hwp == intel_pstate hwp > intel_cpufreq no hwp == acpi-cpufreq == always minimum freq > intel_pstate no hwp ~= acpi-cpufreq/ondemand ... > My expectation was/is: > > intel_cpufreq hwp == intel_cpufreq no hwp == acpi-cpufreq == always minimum freq And this is what we now have, with this patch set. > intel_pstate no hwp ~= acpi-cpufreq/ondemand > intel_pstate hwp == Unique. Say, extremely course version of ondemand. [1] https://marc.info/?l=linux-pm&m=159769839401767&w=2 [2] https://marc.info/?l=linux-pm&m=159943780220923&w=2 ... Doug