linux-pm.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: "Doug Smythies" <dsmythies@telus.net>
To: "'Christian Loehle'" <christian.loehle@arm.com>
Cc: "'Reka Norman'" <rekanorman@chromium.org>,
	<daniel.lezcano@linaro.org>, <linux-pm@vger.kernel.org>,
	"Doug Smythies" <dsmythies@telus.net>,
	"'Rafael J. Wysocki'" <rafael@kernel.org>
Subject: RE: Regression in TEO cpuidle governor between 6.6 and 6.12
Date: Wed, 19 Nov 2025 14:43:34 -0800	[thread overview]
Message-ID: <003c01dc59a5$f0d97010$d28c5030$@telus.net> (raw)
In-Reply-To: <ca45366d-4c85-4802-8a35-886a6f69d10d@arm.com>

Hi Christian,

This email is about testing as opposed to about the subject and patches.
Specifically, the " adrestia" test.

On 2025.11.13 07:43 Christian wrote:
...
> Forgot to post the full results, anyway as expected with mtdblock (a very slow
> / low frequent wakeup scenario) the impact becomes clearly visible.
> Still hopeful that the more conservative approach will be acceptable!
...
> test       gov        i     score  %change    idles  idle_miss  miss_rt   belows   aboves
...
> adrestia   teo-m      0         8    0     103680         21      0.000         19          2   <<<<
> adrestia   teo-m      1        12   50     104026         31      0.000         16         15
> adrestia   teo-m      2        12   50     104063         32      0.000         24          8
> adrestia   teo-m      3        12   50     104768         44      0.000         22         22
> adrestia   teo-m      4        12   50     104388         18      0.000         16          2

> adrestia   teo-patch  0        12   50     103096         67      0.001         65          2
> adrestia   teo-patch  1        12   50     103128         57      0.001         49          8
> adrestia   teo-patch  2         8    0     102812         78      0.001         72          6    <<<<
> adrestia   teo-patch  3        12   50     102990         60      0.001         55          5
> adrestia   teo-patch  4        12   50     103168         24      0.000         24          0
...

I am curious about the significant variations between iterations for your
adrestia test. And what does "score" mean for this test?

I don't know if we are running the exact same test [1], but my "score" means
wakeup cost (periodic, 20us), and for the 6.18-rc4 kernel is around 2.7 uSec,
independent of the various patches or teo verses menu governors.
My results are very repeatable.
With kernel 6.13 I was getting 3.3 uSec, but I suspect the differences
have more to do with other kernel changes than any idle governor changes.

My typical testing command line is:

./adrestia -a 20 -l 2000000 -t 500 wakeup-periodic

Where:
-a      arrival time (microseconds) << Only used for periodic test, sleep time. Actually arrival rate.
-l      number of loops
-L      list tests  << wakeup-periodic and wakeup-single
-s      service time (microseconds) << This has not been implemented.
-t      thread count

Those run parameters give a reasonable amount of idle time using
both idle states 0 and 1.

[1] It is likely we are not running the exact same test, since I converted the
source code I use from uSec to nSec. Notes to myself from a couple of
years ago suggest that I am using the version from [2] and that the version
from [3] does not compile (at least for me).

[2] https://github.com/notcarbide/adrestia
[3] https://github.com/mfleming/adrestia

... Doug



  parent reply	other threads:[~2025-11-19 22:43 UTC|newest]

Thread overview: 48+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-11-04  3:36 Regression in TEO cpuidle governor between 6.6 and 6.12 Reka Norman
2025-11-04  9:03 ` Christian Loehle
2025-11-04 23:24   ` Christian Loehle
2025-11-05  6:22     ` Reka Norman
2025-11-05 22:47       ` Christian Loehle
2025-11-06 15:21         ` Rafael J. Wysocki
2025-11-05 20:48     ` Rafael J. Wysocki
2025-11-06 11:13       ` Christian Loehle
2025-11-06 20:33         ` Rafael J. Wysocki
2025-11-07  3:28           ` Reka Norman
2025-11-07 11:35             ` Rafael J. Wysocki
2025-11-09 20:35               ` Christian Loehle
2025-11-10  6:10                 ` Reka Norman
2025-11-10 12:06                   ` Christian Loehle
2025-11-11  4:23                     ` Reka Norman
2025-11-11 10:00                       ` Christian Loehle
2025-11-11 10:48                         ` Christian Loehle
2025-11-11 11:48                           ` Rafael J. Wysocki
2025-11-11 17:20                             ` Christian Loehle
2025-11-12  9:51                               ` Christian Loehle
2025-11-12 13:40                                 ` Rafael J. Wysocki
2025-11-12 14:14                                   ` Christian Loehle
2025-11-12 14:23                                     ` Rafael J. Wysocki
2025-11-12 13:32                               ` Rafael J. Wysocki
2025-11-12 14:03                                 ` Christian Loehle
2025-11-12 14:16                                   ` Rafael J. Wysocki
2025-11-12 18:33                                     ` Christian Loehle
2025-11-13 15:43                                       ` Christian Loehle
2025-11-13 16:56                                         ` [PATCH v1] cpuidle: governors: teo: Rework the handling of tick wakeups Rafael J. Wysocki
2025-11-14  4:05                                           ` Reka Norman
2025-11-14  8:33                                             ` Christian Loehle
2025-11-14 14:15                                               ` Rafael J. Wysocki
2025-11-14 14:49                                                 ` Christian Loehle
2025-11-14 15:13                                                   ` Rafael J. Wysocki
2025-11-17  5:44                                                 ` Reka Norman
2025-11-17  5:14                                               ` Reka Norman
2025-11-17  8:45                                                 ` Christian Loehle
2025-11-17 16:29                                                   ` Rafael J. Wysocki
2025-11-18  0:19                                                   ` Reka Norman
2025-11-14  9:20                                             ` Christian Loehle
2025-11-17  5:41                                               ` Reka Norman
2025-11-14 14:47                                           ` Christian Loehle
2025-11-14 15:12                                             ` Rafael J. Wysocki
2025-11-14 15:33                                               ` Christian Loehle
2025-11-19 22:43                                         ` Doug Smythies [this message]
2025-11-13  5:13                                 ` Regression in TEO cpuidle governor between 6.6 and 6.12 Reka Norman
2025-11-13 10:52                                   ` Rafael J. Wysocki
2025-11-10 20:51                   ` Rafael J. Wysocki

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='003c01dc59a5$f0d97010$d28c5030$@telus.net' \
    --to=dsmythies@telus.net \
    --cc=christian.loehle@arm.com \
    --cc=daniel.lezcano@linaro.org \
    --cc=linux-pm@vger.kernel.org \
    --cc=rafael@kernel.org \
    --cc=rekanorman@chromium.org \
    /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).