devicetree.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Peter Zijlstra <a.p.zijlstra@chello.nl>
To: Vincent Guittot <vincent.guittot@linaro.org>
Cc: linux-arm-kernel@lists.infradead.org,
	linux-kernel@vger.kernel.org, linaro-dev@lists.linaro.org,
	devicetree-discuss@lists.ozlabs.org, linux@arm.linux.org.uk,
	grant.likely@secretlab.ca, rob.herring@calxeda.com,
	yong.zhang0@gmail.com, namhyung@kernel.org,
	jean.pihet@newoldbits.com, mingo@kernel.org,
	Russell King <rmk@arm.linux.org.uk>
Subject: Re: [PATCH v4 0/5] ARM: topology: set the capacity of each cores for big.LITTLE
Date: Tue, 10 Jul 2012 13:27:02 +0200	[thread overview]
Message-ID: <1341919622.3462.124.camel@twins> (raw)
In-Reply-To: <1341826026-6504-1-git-send-email-vincent.guittot@linaro.org>

On Mon, 2012-07-09 at 11:27 +0200, Vincent Guittot wrote:
> This patchset creates an arch_scale_freq_power function for ARM, which is used 
> to set the relative capacity of each core of a big.LITTLE system. It also removes
> the broken power estimation of x86.
> 
> Modification since v3:
>  - Add comments
>  - Add optimization for SMP system
>  - Ensure that capacity of a CPU will be at most 1
> 
> Modification since v2:
>  - set_power_scale function becomes static
>  - Rework loop in update_siblings_masks
>  - Remove useless code in parse_dt_topology
> 
> Modification since v1:
>  - Add and update explanation about the use of the table and the range of the value 
>  - Remove the use of NR_CPUS and use nr_cpu_ids instead
>  - Remove broken power estimation of x86
> 
> Peter Zijlstra (1):
>   sched, x86: Remove broken power estimation

Note that this patch already made its way into tip as commit
bcae21d6e793a7047d38abc9ac0946c53733c1dd so it might be best to base
whatever tree this is supposed to fo into on something that includes
that.

> Vincent Guittot (4):
>   ARM: topology: Add arch_scale_freq_power function
>   ARM: topology: factorize the update of sibling masks
>   ARM: topology: Update cpu_power according to DT information
>   sched: cpu_power: enable ARCH_POWER

How would you like to proceed with these patches, I'm fine with them
going through the ARM tree..

In which case:

Acked-by: Peter Zijlstra <a.p.zijlstra@chello.nl>

Ingo, Russell, any other preferences?

  parent reply	other threads:[~2012-07-10 11:27 UTC|newest]

Thread overview: 22+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-07-09  9:27 [PATCH v4 0/5] ARM: topology: set the capacity of each cores for big.LITTLE Vincent Guittot
2012-07-09  9:27 ` [PATCH v4 1/5] ARM: topology: Add arch_scale_freq_power function Vincent Guittot
     [not found] ` <1341826026-6504-1-git-send-email-vincent.guittot-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>
2012-07-09  9:27   ` [PATCH v4 2/5] ARM: topology: factorize the update of sibling masks Vincent Guittot
2012-07-09  9:27   ` [PATCH v4 3/5] ARM: topology: Update cpu_power according to DT information Vincent Guittot
2012-07-09 10:55     ` Shilimkar, Santosh
2012-07-09 11:00       ` Peter Zijlstra
2012-07-09 11:04         ` Shilimkar, Santosh
     [not found]       ` <CAMQu2gydOEVK_fzmzkdrBwBfOq6eZG=0QfLUfFChWJu9unSn2A-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2012-07-09 12:32         ` Vincent Guittot
2012-07-09 13:00           ` Shilimkar, Santosh
2012-07-09 14:36             ` Vincent Guittot
     [not found]               ` <CAKfTPtCicLseyA4DnUwNWf7+jOmab4KLo8Yw=Rz1KPya+fE_-w-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2012-07-09 14:37                 ` Shilimkar, Santosh
2012-07-09 15:10                   ` Vincent Guittot
     [not found]                     ` <CAKfTPtCcxg5PNqjXrz9zYwE-DDAUtdbaUuuGpGqGzOFZrk3rnw-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2012-07-09 15:28                       ` Shilimkar, Santosh
     [not found]     ` <1341826026-6504-4-git-send-email-vincent.guittot-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>
2012-09-13 13:03       ` Dave Martin
2012-07-09  9:27   ` [PATCH v4 4/5] sched, x86: Remove broken power estimation Vincent Guittot
2012-07-09  9:27   ` [PATCH v4 5/5] sched: cpu_power: enable ARCH_POWER Vincent Guittot
2012-07-10 11:27 ` Peter Zijlstra [this message]
2012-07-10 12:35   ` [PATCH v4 0/5] ARM: topology: set the capacity of each cores for big.LITTLE Vincent Guittot
     [not found]     ` <CAKfTPtC0BcidJ2ga9c2fdWjrEsfXH48QtSnjv0YOCv7-k3K-Ug-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2012-07-10 13:42       ` Peter Zijlstra
2012-09-13  9:17         ` Vincent Guittot
     [not found]           ` <CAKfTPtCpZS=ALAjwqX5Wb0j5HjrMjv87MEefGC2O5_p7ENeY2w-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2012-09-13 12:07             ` Peter Zijlstra
2012-09-13 12:51               ` Vincent Guittot

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=1341919622.3462.124.camel@twins \
    --to=a.p.zijlstra@chello.nl \
    --cc=devicetree-discuss@lists.ozlabs.org \
    --cc=grant.likely@secretlab.ca \
    --cc=jean.pihet@newoldbits.com \
    --cc=linaro-dev@lists.linaro.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux@arm.linux.org.uk \
    --cc=mingo@kernel.org \
    --cc=namhyung@kernel.org \
    --cc=rmk@arm.linux.org.uk \
    --cc=rob.herring@calxeda.com \
    --cc=vincent.guittot@linaro.org \
    --cc=yong.zhang0@gmail.com \
    /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: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).