lkml.org 
[lkml]   [2018]   [Oct]   [16]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [RFC PATCH 0/7] Introduce thermal pressure

* Thara Gopinath <thara.gopinath@linaro.org> wrote:

> >> Regarding testing, basic build, boot and sanity testing have been
> >> performed on hikey960 mainline kernel with debian file system.
> >> Further aobench (An occlusion renderer for benchmarking realworld
> >> floating point performance) showed the following results on hikey960
> >> with debain.
> >>
> >> Result Standard Standard
> >> (Time secs) Error Deviation
> >> Hikey 960 - no thermal pressure applied 138.67 6.52 11.52%
> >> Hikey 960 - thermal pressure applied 122.37 5.78 11.57%
> >
> > Wow, +13% speedup, impressive! We definitely want this outcome.
> >
> > I'm wondering what happens if we do not track and decay the thermal
> > load at all at the PELT level, but instantaneously decrease/increase
> > effective CPU capacity in reaction to thermal events we receive from
> > the CPU.
>
> The problem with instantaneous update is that sometimes thermal events
> happen at a much faster pace than cpu_capacity is updated in the
> scheduler. This means that at the moment when scheduler uses the
> value, it might not be correct anymore.

Let me offer a different interpretation: if we average throttling events
then we create a 'smooth' average of 'true CPU capacity' that doesn't
fluctuate much. This allows more stable yet asymmetric task placement if
the thermal characteristics of the different cores is different
(asymmetric). This, compared to instantaneous updates, would reduce
unnecessary task migrations between cores.

Is that accurate?

If the thermal characteristics of the cores is roughly symmetric and the
measured CPU-intense load itself is symmetric as well, then I have
trouble seeing why reacting to thermal events should make any difference
at all.

Are there any inherent asymmetries in the thermal properties of the
cores, or in the benchmarked workload itself?

Thanks,

Ingo

\
 
 \ /
  Last update: 2018-10-16 09:34    [W:0.124 / U:6.224 seconds]
©2003-2018 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site