Re: [PATCH v2] sched/fair: Make PELT signal more accurate

[Joel Fernandes]

Hi Mike,

On Sat, Aug 19, 2017 at 11:00 PM, Mike Galbraith <efault@xxxxxx> wrote:
> On Sat, 2017-08-19 at 10:58 -0700, Joel Fernandes wrote:
>>
>> > Ok, I gotta ask:  In order to fix what?  What exactly does the small
>> > but existent overhead increase buy us other than an ever so slightly
>> > different chart?  What is your motivation to care about a microscopic
>> > change in signal shape?
>>
>> I wouldn't call the change "microscopic", its about 2% absolute which
>> comes down to 1% with this change (if you count in relative terms, its
>> higher and you can see the bump as the signal rises).
>> If you look at the first chart at [1] at 3.74, that's not microscopic
>> at all to me.
>
> Whether that 0.02->0.01 is viewed as significant by you or I matters
> not at all, for it to be significant, it must have measurable effect.
>
> Where is the consumer which benefits from precision improvement.. of
> the instantaneously wildly inaccurate average.  Where is the non-zero
> return on investment?  If the chart is the entire product, no sale.

One thing I want to mention is the overhead shows up only in the one
unixbench test that this is sensitive to (and even in that the
overhead is around 0.005, its within the noise for large number of
runs), for the other tests I ran like hackbench, there wasn't any
overhead at all. I am not saying that patch solves a major issue that
exists that I know off, but its just an observation while studying the
code but I'd argue its still an improvement that's harmless and
worthwhile to have.

Anyway the study is here for any one to see in the the future and it
was indeed useful to me, to learn more about the code. I am Ok with
dropping this patch if the general feeling is the inaccurate average
is Ok.

thanks for your review,

-Joel

>
>         -Mike
>
> --
> You received this message because you are subscribed to the Google Groups "kernel-team" group.
> To unsubscribe from this group and stop receiving emails from it, send an email to kernel-team+unsubscribe@xxxxxxxxxxxx
>