From: Christian Loehle <christian.loehle@arm.com>
To: Andreas Ziegler <br025@umbiko.net>
Cc: Peter Zijlstra <peterz@infradead.org>,
Juri Lelli <juri.lelli@redhat.com>,
linux-kernel@vger.kernel.org,
Dietmar Eggemann <dietmar.eggemann@arm.com>,
John Stultz <jstultz@google.com>
Subject: Re: sched/deadline: Use revised wakeup rule for dl_server
Date: Fri, 8 May 2026 15:13:45 +0100 [thread overview]
Message-ID: <d21955c3-f788-493d-ae45-b756b7ebeeff@arm.com> (raw)
In-Reply-To: <97d9e04fd9d222f1a64f1ecfda8b81d7@umbiko.net>
On 5/8/26 13:06, Andreas Ziegler wrote:
> Hi Christian,
>
> On 2026-05-08 09:20, Christian Loehle wrote:
>> On 5/8/26 09:09, Andreas Ziegler wrote:
>>> Linux kernel version: 6.12
>>> CONFIG_PREEMPT_RT (w/ PREEMPT_RT patch applied)
>>> Architecture: aarch64
>>> Platform: Raspberry Pi 4
>>>
>>> Hi everyone,
>>>
>>> Commit d66792919d4f (sched/deadline: Use revised wakeup rule for dl_server) [1] introduced a marked degradation in scheduling latency for real-time tasks in the presence of heavy I/O load.
>>>
>>> --- a/kernel/sched/deadline.c
>>> +++ b/kernel/sched/deadline.c
>>> @@ -1079,7 +1079,7 @@ static void update_dl_entity(struct sched_dl_entity *dl_se)
>>> if (dl_time_before(dl_se->deadline, rq_clock(rq)) ||
>>> dl_entity_overflow(dl_se, rq_clock(rq))) {
>>>
>>> - if (unlikely(!dl_is_implicit(dl_se) &&
>>> + if (unlikely((!dl_is_implicit(dl_se) || dl_se->dl_defer) &&
>>> !dl_time_before(dl_se->deadline, rq_clock(rq)) &&
>>> !is_dl_boosted(dl_se))) {
>>> update_dl_revised_wakeup(dl_se, rq);
>>>
>>> This was observed using a modified version of Con Kolivas' interactivity benchmark [2]; kernel bisection eventually pointed to the above mentioned commit.
>>>
>>> Benchmark results before d66792919d4f:
>>>
>>> --- Benchmarking simulated cpu of Audio real time in the presence of simulated ---
>>> Load Latency +/- SD median max [100n] Desired CPU Deadlines met [%]
>>> None 76.6 +/- 8.3654 76 166
>>> Video 78.5 +/- 3.9433 78 107
>>> X 76.4 +/- 8.123 75 157
>>> Burn 72.0 +/- 6.4733 71 127
>>> Write 255.3 +/- 26.627 252 331
>>> Read 226.6 +/- 12.38 227 262
>>> Ring 84.2 +/- 6.6207 83 125
>>> Compile 225.3 +/- 23.949 222 328
>>>
>>> 136.8 +/- 78.462 331
>>>
>>> Benchmark results after d66792919d4f:
>>>
>>> --- Benchmarking simulated cpu of Audio real time in the presence of simulated ---
>>> Load Latency +/- SD median max [100n] Desired CPU Deadlines met [%]
>>> None 68.4 +/- 9.7864 67 169
>>> Video 74.4 +/- 3.724 74 97
>>> X 72.0 +/- 6.5681 71 129
>>> Burn 66.9 +/- 5.9059 66 117
>>> Write 9576.9 +/- 67639 250500418 98.1 98.1
>>> Read 209.3 +/- 11.018 209 267
>>> Ring 80.5 +/- 8.0993 78 125
>>> Compile 239.0 +/- 29.447 234 372
>>>
>>> 1298.4 +/- 24118 500418
>>>
>>> Reverting this commit obviously solves the issue for me. I have no idea why this issue appears exclusively with heavy write loads in the background.
>>>
>>> Is this a scheduler issue, or rather something in the background?
>>>
>>
>> Hi Andreas,
>> You're using cpufreq schedutil for your tests I'm assuming?
>> Is there a difference in cpufreq behavior (avg cpufreq or OPP residencies?)
>> Does the regression also happen on powersave/performance governor?
>
> Actually this is a very stripped-down system. The 'performance' cpufreq governor is the only one compiled in, the processor cores run on a fixed frequency. CONFIG_PM_OPP is not set.
That certainly makes the analysis easier.
I couldn't reproduce the issue so far on my system but it does seem like the dl server
would get potentially unbounded running time with very frequent
starting and stopping of the dlserver (which presumably happens because of
the writeback) reset the runtime, which then leads to your 25s observed latency.
Peter, how is the revised wakeup rule supposed to behave here?
> [snip]
next prev parent reply other threads:[~2026-05-08 14:13 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-05-08 8:09 sched/deadline: Use revised wakeup rule for dl_server Andreas Ziegler
2026-05-08 9:20 ` Christian Loehle
2026-05-08 12:06 ` Andreas Ziegler
2026-05-08 14:13 ` Christian Loehle [this message]
2026-05-09 11:42 ` Andreas Ziegler
2026-05-11 9:47 ` Christian Loehle
2026-05-11 12:37 ` Andreas Ziegler
2026-05-11 12:46 ` Juri Lelli
2026-05-11 14:13 ` Andreas Ziegler
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=d21955c3-f788-493d-ae45-b756b7ebeeff@arm.com \
--to=christian.loehle@arm.com \
--cc=br025@umbiko.net \
--cc=dietmar.eggemann@arm.com \
--cc=jstultz@google.com \
--cc=juri.lelli@redhat.com \
--cc=linux-kernel@vger.kernel.org \
--cc=peterz@infradead.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