From: Dietmar Eggemann <dietmar.eggemann@arm.com> To: Vincent Guittot <vincent.guittot@linaro.org> Cc: linux-kernel <linux-kernel@vger.kernel.org>, LAK <linux-arm-kernel@lists.infradead.org>, "devicetree@vger.kernel.org" <devicetree@vger.kernel.org>, linux-samsung-soc <linux-samsung-soc@vger.kernel.org>, linux-renesas-soc@vger.kernel.org, Russell King <linux@armlinux.org.uk>, Rob Herring <robh+dt@kernel.org>, Mark Rutland <mark.rutland@arm.com>, Kukjin Kim <kgene@kernel.org>, Krzysztof Kozlowski <krzk@kernel.org>, Juri Lelli <juri.lelli@arm.com> Subject: Re: [PATCH 1/4] arm: topology: remove cpu_efficiency Date: Wed, 6 Sep 2017 12:43:31 +0100 [thread overview] Message-ID: <303d3f7b-5d64-e13a-c4f9-dd575958cafa@arm.com> (raw) In-Reply-To: <CAKfTPtBXkAyAc1aqqqEn6Yt14_Y7LFp3oySY-=G3ZTuG-DR_nA@mail.gmail.com> Hi Vincent, On 04/09/17 08:49, Vincent Guittot wrote: > Hi Dietmar, > > Removing cpu effificiency table looks good to me. Nevertheless, i have > some comments below for this patch. Thanks for the review! > On 30 August 2017 at 16:41, Dietmar Eggemann <dietmar.eggemann@arm.com> wrote: >> Remove the 'cpu_efficiency/clock-frequency dt property' based solution >> to set cpu capacity which was only working for Cortex-A15/A7 arm >> big.LITTLE systems. >> >> I.e. the 'capacity-dmips-mhz' based solution is now the only one. It is >> shared between arm and arm64 and works for every big.LITTLE system no >> matter which core types it consists of. >> >> Cc: Russell King <linux@arm.linux.org.uk> >> Cc: Vincent Guittot <vincent.guittot@linaro.org> >> Cc: Juri Lelli <juri.lelli@arm.com> >> Signed-off-by: Dietmar Eggemann <dietmar.eggemann@arm.com> >> --- >> arch/arm/kernel/topology.c | 113 ++------------------------------------------- >> 1 file changed, 3 insertions(+), 110 deletions(-) [...] >> @@ -115,73 +70,13 @@ static void __init parse_dt_topology(void) >> of_node_put(cn); >> continue; > > AFAICT, this continue is now useless as it was there to skipe the cpu > table efficiency method You're right ... will remove it. [...] >> - if (cap_from_dt) >> - topology_normalize_cpu_scale(); > > Why have you moved the call to topology_normalize_cpu_scale() from > parse_dt_topology() to update_cpu_capacity() ? Didn't move it ? It's still called from parse_dt_topology(). > You should keep it in parse_dt_topology() as itis part of the dt > parsing sequence Yes, this should be the case. [...] >> -/* >> - * Look for a customed capacity of a CPU in the cpu_capacity table during the >> - * boot. The update of all CPUs is in O(n^2) for heteregeneous system but the >> - * function returns directly for SMP system. >> - */ >> -static void update_cpu_capacity(unsigned int cpu) >> -{ >> - if (!cpu_capacity(cpu) || cap_from_dt) >> - return; >> - >> - topology_set_cpu_scale(cpu, cpu_capacity(cpu) / middle_capacity); >> - >> - pr_info("CPU%u: update cpu_capacity %lu\n", >> - cpu, topology_get_cpu_scale(NULL, cpu)); >> + topology_normalize_cpu_scale(); >> } > > You can probably just removed update_cpu_capacity() I did remove update_cpu_capacity(). Maybe the patch layout is confusing? [...]
WARNING: multiple messages have this Message-ID (diff)
From: dietmar.eggemann@arm.com (Dietmar Eggemann) To: linux-arm-kernel@lists.infradead.org Subject: [PATCH 1/4] arm: topology: remove cpu_efficiency Date: Wed, 6 Sep 2017 12:43:31 +0100 [thread overview] Message-ID: <303d3f7b-5d64-e13a-c4f9-dd575958cafa@arm.com> (raw) In-Reply-To: <CAKfTPtBXkAyAc1aqqqEn6Yt14_Y7LFp3oySY-=G3ZTuG-DR_nA@mail.gmail.com> Hi Vincent, On 04/09/17 08:49, Vincent Guittot wrote: > Hi Dietmar, > > Removing cpu effificiency table looks good to me. Nevertheless, i have > some comments below for this patch. Thanks for the review! > On 30 August 2017 at 16:41, Dietmar Eggemann <dietmar.eggemann@arm.com> wrote: >> Remove the 'cpu_efficiency/clock-frequency dt property' based solution >> to set cpu capacity which was only working for Cortex-A15/A7 arm >> big.LITTLE systems. >> >> I.e. the 'capacity-dmips-mhz' based solution is now the only one. It is >> shared between arm and arm64 and works for every big.LITTLE system no >> matter which core types it consists of. >> >> Cc: Russell King <linux@arm.linux.org.uk> >> Cc: Vincent Guittot <vincent.guittot@linaro.org> >> Cc: Juri Lelli <juri.lelli@arm.com> >> Signed-off-by: Dietmar Eggemann <dietmar.eggemann@arm.com> >> --- >> arch/arm/kernel/topology.c | 113 ++------------------------------------------- >> 1 file changed, 3 insertions(+), 110 deletions(-) [...] >> @@ -115,73 +70,13 @@ static void __init parse_dt_topology(void) >> of_node_put(cn); >> continue; > > AFAICT, this continue is now useless as it was there to skipe the cpu > table efficiency method You're right ... will remove it. [...] >> - if (cap_from_dt) >> - topology_normalize_cpu_scale(); > > Why have you moved the call to topology_normalize_cpu_scale() from > parse_dt_topology() to update_cpu_capacity() ? Didn't move it ? It's still called from parse_dt_topology(). > You should keep it in parse_dt_topology() as itis part of the dt > parsing sequence Yes, this should be the case. [...] >> -/* >> - * Look for a customed capacity of a CPU in the cpu_capacity table during the >> - * boot. The update of all CPUs is in O(n^2) for heteregeneous system but the >> - * function returns directly for SMP system. >> - */ >> -static void update_cpu_capacity(unsigned int cpu) >> -{ >> - if (!cpu_capacity(cpu) || cap_from_dt) >> - return; >> - >> - topology_set_cpu_scale(cpu, cpu_capacity(cpu) / middle_capacity); >> - >> - pr_info("CPU%u: update cpu_capacity %lu\n", >> - cpu, topology_get_cpu_scale(NULL, cpu)); >> + topology_normalize_cpu_scale(); >> } > > You can probably just removed update_cpu_capacity() I did remove update_cpu_capacity(). Maybe the patch layout is confusing? [...]
next prev parent reply other threads:[~2017-09-06 11:43 UTC|newest] Thread overview: 40+ messages / expand[flat|nested] mbox.gz Atom feed top 2017-08-30 14:41 [PATCH 0/4] arm: remove cpu_efficiency Dietmar Eggemann 2017-08-30 14:41 ` Dietmar Eggemann 2017-08-30 14:41 ` [PATCH 1/4] arm: topology: " Dietmar Eggemann 2017-08-30 14:41 ` Dietmar Eggemann 2017-09-04 7:49 ` Vincent Guittot 2017-09-04 7:49 ` Vincent Guittot 2017-09-04 7:49 ` Vincent Guittot 2017-09-06 11:43 ` Dietmar Eggemann [this message] 2017-09-06 11:43 ` Dietmar Eggemann 2017-09-06 12:40 ` Vincent Guittot 2017-09-06 12:40 ` Vincent Guittot 2017-09-06 12:40 ` Vincent Guittot 2017-09-07 10:41 ` Dietmar Eggemann 2017-09-07 10:41 ` Dietmar Eggemann 2017-08-30 14:41 ` [PATCH 2/4] arm: dts: exynos: add exynos5420 cpu capacity-dmips-mhz information Dietmar Eggemann 2017-08-30 14:41 ` Dietmar Eggemann 2017-08-30 14:41 ` Dietmar Eggemann 2017-08-30 20:26 ` Krzysztof Kozlowski 2017-08-30 20:26 ` Krzysztof Kozlowski 2017-08-30 20:26 ` Krzysztof Kozlowski 2017-08-31 10:36 ` Dietmar Eggemann 2017-08-31 10:36 ` Dietmar Eggemann 2017-09-03 19:56 ` Krzysztof Kozlowski 2017-09-03 19:56 ` Krzysztof Kozlowski 2017-09-06 11:47 ` Dietmar Eggemann 2017-09-06 11:47 ` Dietmar Eggemann 2017-09-06 11:47 ` Dietmar Eggemann 2017-09-17 7:37 ` Krzysztof Kozlowski 2017-09-17 7:37 ` Krzysztof Kozlowski 2017-08-30 14:41 ` [PATCH 3/4] arm: dts: exynos: add exynos5422 " Dietmar Eggemann 2017-08-30 14:41 ` Dietmar Eggemann 2017-08-30 14:41 ` Dietmar Eggemann 2017-09-17 7:37 ` Krzysztof Kozlowski 2017-09-17 7:37 ` Krzysztof Kozlowski 2017-08-30 14:41 ` [PATCH 4/4] arm: dts: r8a7790: add " Dietmar Eggemann 2017-08-30 14:41 ` Dietmar Eggemann 2017-09-18 7:39 ` Simon Horman 2017-09-18 7:39 ` Simon Horman 2017-10-09 17:55 ` Dietmar Eggemann 2017-10-09 17:55 ` Dietmar Eggemann
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=303d3f7b-5d64-e13a-c4f9-dd575958cafa@arm.com \ --to=dietmar.eggemann@arm.com \ --cc=devicetree@vger.kernel.org \ --cc=juri.lelli@arm.com \ --cc=kgene@kernel.org \ --cc=krzk@kernel.org \ --cc=linux-arm-kernel@lists.infradead.org \ --cc=linux-kernel@vger.kernel.org \ --cc=linux-renesas-soc@vger.kernel.org \ --cc=linux-samsung-soc@vger.kernel.org \ --cc=linux@armlinux.org.uk \ --cc=mark.rutland@arm.com \ --cc=robh+dt@kernel.org \ --cc=vincent.guittot@linaro.org \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.