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

Groups > linux.kernel > #1330057

Re: [PATCH v3 0/6] CPUs capacity information for heterogeneous systems

Show all headers | View raw

Hi Steve,

On 08/02/16 15:59, Steve Muckle wrote:
> Hi Juri,
> 
> On 02/03/2016 03:59 AM, Juri Lelli wrote:
> >  v1: DT + sysfs [1]
> > 
> >  v2: Dynamic profiling at boot [2]
> > 
> > Third version of this patchset proposes what seems to be the solution we agreed
> > upon (see [2] for reference) to the problem of how do we init CPUs original
> > capacity: we run a bogus benchmark (stealing int_sqrt from lib/ we run that in
> > a loop to perform some integer computation, better benchmarks are welcome)
> > on the first cpu of each frequency domain (assuming no u-arch differences
> > inside domains), measure time to complete a fixed number of iterations and then
> > normalize results to SCHED_CAPACITY_SCALE (1024). This time around we also
> > added a boot time parameter to disable profiling at boot (as it can be time
> > consuming) and sysfs attributes with which default values can be overwritten.
> > The proposed solution is basically putting together bits of v1 and v2 that are
> > considered valuable and acceptable for mainline.
> 
> I'm still concerned that there's no way to obtain optimal boot time on a
> heterogeneous system. Either the dynamic benchmarking is enabled, adding
> 1 sec, or the benchmarking is skipped, and task distribution on the
> heterogeneous CPUs is determined by the platform's CPU numbering and
> chance, potentially impacting performance nondeterministically until
> userspace sets the correct capacity values via sysfs.
> 
> I believe you tested the impact on boot time of using equal capacity
> values and saw little difference. I'm wondering though, what was the CPU
> numbering on that target?
> 

My targets (Juno and TC2) had big cluster on 1,2 and little on the
remaining cpus. Why do you think this might matter?

Anyway, IMHO boot time performance is not what we are targeting here, so
I wouldn't be too worried about this particular point.

Best,

- Juri

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

Thread

[PATCH v3 0/6] CPUs capacity information for heterogeneous systems Juri Lelli <juri.lelli@arm.com> - 2016-02-03 13:00 +0100
  [PATCH v3 4/6] arm64: Enable dynamic CPU capacity initialization Juri Lelli <juri.lelli@arm.com> - 2016-02-03 13:00 +0100
    Re: [PATCH v3 4/6] arm64: Enable dynamic CPU capacity initialization Dietmar Eggemann <dietmar.eggemann@arm.com> - 2016-02-08 13:30 +0100
      Re: [PATCH v3 4/6] arm64: Enable dynamic CPU capacity initialization Mark Brown <broonie@kernel.org> - 2016-02-08 14:20 +0100
        Re: [PATCH v3 4/6] arm64: Enable dynamic CPU capacity initialization Dietmar Eggemann <dietmar.eggemann@arm.com> - 2016-02-08 14:50 +0100
  [PATCH v3 6/6] arm64: add sysfs cpu_capacity attribute Juri Lelli <juri.lelli@arm.com> - 2016-02-03 13:10 +0100
    Re: [PATCH v3 6/6] arm64: add sysfs cpu_capacity attribute Dietmar Eggemann <dietmar.eggemann@arm.com> - 2016-02-05 18:20 +0100
      Re: [PATCH v3 6/6] arm64: add sysfs cpu_capacity attribute Juri Lelli <juri.lelli@arm.com> - 2016-02-05 18:50 +0100
  [PATCH v3 5/6] arm: add sysfs cpu_capacity attribute Juri Lelli <juri.lelli@arm.com> - 2016-02-03 13:10 +0100
  [PATCH v3 3/6] arm: Enable dynamic CPU capacity initialization Juri Lelli <juri.lelli@arm.com> - 2016-02-03 13:10 +0100
  Re: [PATCH v3 0/6] CPUs capacity information for heterogeneous  systems Steve Muckle <steve.muckle@linaro.org> - 2016-02-09 01:10 +0100
    Re: [PATCH v3 0/6] CPUs capacity information for heterogeneous  systems Juri Lelli <juri.lelli@arm.com> - 2016-02-09 11:40 +0100
      Re: [PATCH v3 0/6] CPUs capacity information for heterogeneous  systems Steve Muckle <steve.muckle@linaro.org> - 2016-02-09 18:40 +0100
        Re: [PATCH v3 0/6] CPUs capacity information for heterogeneous  systems Juri Lelli <juri.lelli@arm.com> - 2016-02-09 18:50 +0100

csiph-web