mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: Alexander Gordeev <agordeev@linux.ibm.com>
To: Mete Durlu <meted@linux.ibm.com>
Cc: Heiko Carstens <hca@linux.ibm.com>,
	Vasily Gorbik <gor@linux.ibm.com>,
	Sven Schnelle <svens@linux.ibm.com>,
	Anna-Maria Behnsen <anna-maria@linutronix.de>,
	Frederic Weisbecker <frederic@kernel.org>,
	Ingo Molnar <mingo@kernel.org>, Thomas Gleixner <tglx@kernel.org>,
	"Rafael J. Wysocki" <rafael@kernel.org>,
	Daniel Lezcano <daniel.lezcano@kernel.org>,
	Christian Loehle <christian.loehle@arm.com>,
	Christian Borntraeger <borntraeger@linux.ibm.com>,
	Ilya Leoshkevich <iii@linux.ibm.com>,
	linux-s390@vger.kernel.org, linux-kernel@vger.kernel.org,
	linux-pm@vger.kernel.org
Subject: Re: [PATCH v3 0/5] s390/idle: CPU idle driver
Date: Fri, 19 Jun 2026 11:29:20 +0200	[thread overview]
Message-ID: <e624cd2b-9b07-4e7e-bab3-0cf73e9f9db9-agordeev@linux.ibm.com> (raw)
In-Reply-To: <20260618-idledriver-v3-0-684061eecbcb@linux.ibm.com>

On Thu, Jun 18, 2026 at 02:00:14PM +0200, Mete Durlu wrote:
...
> This patch series introduces a CPU idle driver for s390 architecture
> that leverages the existing cpu idle infrastructure and idle governor
> to optimize idle state selection based on timer events and interrupt
> patterns.
> 
> - Implementation Overview
> -----------------------------------------------------------------------
> 
> The driver implements two idle states;
> 
> **Polling Idle (shallow state)**
> - Busy-loop implementation that prevents SIE-exit during short idle periods
> - Zero exit latency, optimized for latency-sensitive workloads
> - Maintains CPU with the guest, improving response times
> 
> **Enabled Wait (deep state)**
> - Traditional EW state that signals hypervisor availability
> - Higher exit latency but more efficient for longer idle periods
> 
> The idle governor dynamically selects between these states by monitoring
> next timer interrupts and system interrupt frequency, choosing the most
> appropriate idle state for each situation.
> 
> - Performance Benefits
> -----------------------------------------------------------------------
> 
> Benchmark results across LPAR, KVM, and z/VM guests demonstrate significant
> improvements for latency-sensitive and scheduler-intensive workloads:
> 
> ```
> Benchmark    | LPAR  | z/VM  | KVM   |
> -----------------------------------------
> cyclictest   | +45%  | +50%  | +30%  |
> hackbench    | +5%   | +12%  | -     |
> pgbench      | +2%   | +15%  | +2%   |
> uperf        | +50%* | +-5%  | +-5%  |
> ```
> 
> *Single-threaded uperf scenarios show 50-70% improvements on LPAR.
> Multi-threaded results show slight improvements overall.
> 
> Additionally, fio and ioping tests show latency distribution shifts toward
> lower values with reduced minimum and average latencies.
> 
> - Configuration
> -----------------------------------------------------------------------
> 
> Target residency and exit latency values are benchmark estimates. Due to
> s390's virtualized architecture, these values are non-deterministic.
> Idle state parameters are tuned per hypervisor type after benchmarks:
> 
> **LPAR:**
> - EW: 5us target residency, 5us exit latency
> 
> **KVM/z/VM:**
> - EW: 1us target residency, 1us exit latency
> 
> ---
> Mete Durlu (5):
>       s390/tick: Remove CIF_NOHZ_DELAY flag
>       tick: Remove arch_needs_cpu
>       s390: Enable TIF_POLLING_NRFLAG
>       s390/idle: Introduce cpuidle for s390
>       s390/configs: Enable cpuidle driver on s390
> 
>  MAINTAINERS                         |   8 +++
>  arch/s390/Kconfig                   |   5 ++
>  arch/s390/configs/debug_defconfig   |   2 +
>  arch/s390/configs/defconfig         |   2 +
>  arch/s390/include/asm/processor.h   |   4 --
>  arch/s390/include/asm/thread_info.h |   1 +
>  arch/s390/kernel/idle.c             |   1 -
>  arch/s390/kernel/irq.c              |   4 --
>  arch/s390/kernel/smp.c              |   1 -
>  drivers/cpuidle/Kconfig             |   5 ++
>  drivers/cpuidle/Kconfig.s390        |  11 ++++
>  drivers/cpuidle/Makefile            |   4 ++
>  drivers/cpuidle/cpuidle-s390.c      | 115 ++++++++++++++++++++++++++++++++++++
>  include/linux/tick.h                |   3 -
>  kernel/time/tick-sched.c            |   7 +--
>  15 files changed, 156 insertions(+), 17 deletions(-)
> ---
> base-commit: 2d3090a8aeb596a26935db0955d46c9a5db5c6ce
> change-id: 20260609-idledriver-217e2b2712a3
> 
> Best regards,
> -- 
> Mete Durlu <meted@linux.ibm.com>

Applied, thanks!

  parent reply	other threads:[~2026-06-19  9:29 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-06-18 12:00 Mete Durlu
2026-06-18 12:00 ` [PATCH v3 1/5] s390/tick: Remove CIF_NOHZ_DELAY flag Mete Durlu
2026-06-18 12:00 ` [PATCH v3 2/5] tick: Remove arch_needs_cpu Mete Durlu
2026-06-18 12:00 ` [PATCH v3 3/5] s390: Enable TIF_POLLING_NRFLAG Mete Durlu
2026-06-18 12:00 ` [PATCH v3 4/5] s390/idle: Introduce cpuidle for s390 Mete Durlu
2026-06-18 14:41   ` Christian Loehle
2026-06-18 14:47     ` Mete Durlu
2026-06-18 16:06       ` Christian Loehle
2026-06-19  8:01   ` Heiko Carstens
2026-06-18 12:00 ` [PATCH v3 5/5] s390/configs: Enable cpuidle driver on s390 Mete Durlu
2026-06-18 13:18   ` Christian Borntraeger
2026-06-19  9:29 ` Alexander Gordeev [this message]
2026-06-19 10:38   ` [PATCH v3 0/5] s390/idle: CPU idle driver Christian Loehle
2026-06-19 11:34     ` Alexander Gordeev
2026-06-22 17:07       ` 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=e624cd2b-9b07-4e7e-bab3-0cf73e9f9db9-agordeev@linux.ibm.com \
    --to=agordeev@linux.ibm.com \
    --cc=anna-maria@linutronix.de \
    --cc=borntraeger@linux.ibm.com \
    --cc=christian.loehle@arm.com \
    --cc=daniel.lezcano@kernel.org \
    --cc=frederic@kernel.org \
    --cc=gor@linux.ibm.com \
    --cc=hca@linux.ibm.com \
    --cc=iii@linux.ibm.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-pm@vger.kernel.org \
    --cc=linux-s390@vger.kernel.org \
    --cc=meted@linux.ibm.com \
    --cc=mingo@kernel.org \
    --cc=rafael@kernel.org \
    --cc=svens@linux.ibm.com \
    --cc=tglx@kernel.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

all inboxes | Powered by JetHome®