Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1362078

Re: [PATCH v4 2/8] Documentation: arm: define DT cpu capacity bindings

Show all headers | View raw

On 03/21/2016 03:53 AM, Juri Lelli wrote:
> Hi Sai,
> 
> On 18/03/16 10:49, Sai Gurrappadi wrote:
>> Hi Juri,
>>
>> On 03/18/2016 07:24 AM, Juri Lelli wrote:
>>
>> <snip>
>>
>>> +
>>> +==========================================
>>> +2 - CPU capacity definition
>>> +==========================================
>>> +
>>> +CPU capacity is a number that provides the scheduler information about CPUs
>>> +heterogeneity. Such heterogeneity can come from micro-architectural differences
>>> +(e.g., ARM big.LITTLE systems) or maximum frequency at which CPUs can run
>>> +(e.g., SMP systems with multiple frequency domains). Heterogeneity in this
>>> +context is about differing performance characteristics; this binding tries to
>>> +capture a first-order approximation of the relative performance of CPUs.
>>
>> Any reason why this capacity number is not dynamically generated based on the
>> max frequency for each CPU? The DT property would then instead specify just
>> the micro-architectural differences between the CPU types.
>>
> 
> I'm not sure I clearly understand your question, so I'll try to
> reiterate it.
> 
> Are you asking why we don't dynamically profile the system, at boot for
> example, to get this number? Or do you ask why this number couldn't be
> only describing micro-arch differences (so, if I get it right, we should
> then multiply it by max freq to get the capacity of a CPU)?
> 
> We already played with the first option (please refer to v2 and v3), but
> we ended up agreeing that dynamic profiling adds overhead to the boot
> process (while a DT approach can provide information to speed up boot)
> and it is in general not repeatable/reliable (as numbers can vary from
> boot to boot for different reasons).
> 
> The second option I think can be feasible, but I'm not sure what we gain
> in practice. We will still need to specify a per-platform number, right?

I meant the second bit. We only need some per-platform fudge factor for
micro-architectural differences. Tying in the Fmax like this statically means
that we need to manually synchronize DVFS tables and this number which seems
unnecessary given that the kernel already has this info on boot.

> 
> Best,
> 
> - Juri
> 

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

[PATCH v4 0/8] CPUs capacity information for heterogeneous systems Juri Lelli <juri.lelli@arm.com> - 2016-03-18 15:30 +0100
  [PATCH v4 1/8] ARM: initialize cpu_scale to its default Juri Lelli <juri.lelli@arm.com> - 2016-03-18 15:30 +0100
  [PATCH v4 7/8] arm: add sysfs cpu_capacity attribute Juri Lelli <juri.lelli@arm.com> - 2016-03-18 15:30 +0100
  [PATCH v4 2/8] Documentation: arm: define DT cpu capacity bindings Juri Lelli <juri.lelli@arm.com> - 2016-03-18 15:40 +0100
    Re: [PATCH v4 2/8] Documentation: arm: define DT cpu capacity bindings Sai Gurrappadi <sgurrappadi@nvidia.com> - 2016-03-18 19:00 +0100
      Re: [PATCH v4 2/8] Documentation: arm: define DT cpu capacity  bindings Juri Lelli <juri.lelli@arm.com> - 2016-03-21 12:00 +0100
        Re: [PATCH v4 2/8] Documentation: arm: define DT cpu capacity bindings Vincent Guittot <vincent.guittot@linaro.org> - 2016-03-21 12:20 +0100
          Re: [PATCH v4 2/8] Documentation: arm: define DT cpu capacity  bindings Juri Lelli <juri.lelli@arm.com> - 2016-03-21 12:50 +0100
            Re: [PATCH v4 2/8] Documentation: arm: define DT cpu capacity  bindings Mark Brown <broonie@kernel.org> - 2016-03-21 13:20 +0100
              Re: [PATCH v4 2/8] Documentation: arm: define DT cpu capacity  bindings Juri Lelli <juri.lelli@arm.com> - 2016-03-21 18:30 +0100
                Re: [PATCH v4 2/8] Documentation: arm: define DT cpu capacity  bindings Mark Brown <broonie@kernel.org> - 2016-03-21 19:00 +0100
                Re: [PATCH v4 2/8] Documentation: arm: define DT cpu capacity  bindings Juri Lelli <juri.lelli@arm.com> - 2016-03-22 10:50 +0100
        Re: [PATCH v4 2/8] Documentation: arm: define DT cpu capacity bindings Sai Gurrappadi <sgurrappadi@nvidia.com> - 2016-03-21 20:30 +0100
    Re: [PATCH v4 2/8] Documentation: arm: define DT cpu capacity  bindings Rob Herring <robh@kernel.org> - 2016-03-20 02:20 +0100
      Re: [PATCH v4 2/8] Documentation: arm: define DT cpu capacity  bindings Juri Lelli <juri.lelli@arm.com> - 2016-03-21 12:40 +0100
  [PATCH v4 5/8] arm64: parse cpu capacity from DT Juri Lelli <juri.lelli@arm.com> - 2016-03-18 15:40 +0100

csiph-web