<!DOCTYPE html>
<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=UTF-8">
</head>
<body>
<div class="moz-cite-prefix">On 09/09/25 2:32 am, Mukesh Kumar
Chaurasiya wrote:<br>
</div>
<blockquote type="cite"
cite="mid:20250908210235.137300-2-mchauras@linux.ibm.com">
<pre wrap="" class="moz-quote-pre">Adding support for the generic irq entry/exit handling for PowerPC. The
goal is to bring PowerPC in line with other architectures that already
use the common irq entry infrastructure, reducing duplicated code and
making it easier to share future changes in entry/exit paths.
This is slightly tested on ppc64le.
The performance benchmarks from perf bench basic syscall are below:
| Metric | W/O Generic Framework | With Generic Framework | Improvement |
| ---------- | --------------------- | ---------------------- | ----------- |
| Total time | 0.885 [sec] | 0.880 [sec] | ~0.56% |
| usecs/op | 0.088518 | 0.088005 | ~0.58% |
| ops/sec | 1,12,97,086 | 1,13,62,977 | ~0.58% |
Thats close to 0.6% improvement with this.
Changelog:
V1 -> V2: Support added for irq with generic framework.
Mukesh Kumar Chaurasiya (8):
powerpc: rename arch_irq_disabled_regs
powerpc: Prepare to build with generic entry/exit framework
powerpc: introduce arch_enter_from_user_mode
powerpc: Introduce syscall exit arch functions
powerpc: add exit_flags field in pt_regs
powerpc: Prepare for IRQ entry exit
powerpc: Enable IRQ generic entry/exit path.
powerpc: Enable Generic Entry/Exit for syscalls.
arch/powerpc/Kconfig | 2 +
arch/powerpc/include/asm/entry-common.h | 550 ++++++++++++++++++++++++
arch/powerpc/include/asm/hw_irq.h | 4 +-
arch/powerpc/include/asm/interrupt.h | 393 +++--------------
arch/powerpc/include/asm/ptrace.h | 2 +
arch/powerpc/include/asm/stacktrace.h | 8 +
arch/powerpc/include/asm/syscall.h | 5 +
arch/powerpc/include/asm/thread_info.h | 1 +
arch/powerpc/include/uapi/asm/ptrace.h | 14 +-
arch/powerpc/kernel/asm-offsets.c | 1 +
arch/powerpc/kernel/interrupt.c | 251 ++---------
arch/powerpc/kernel/interrupt_64.S | 2 -
arch/powerpc/kernel/ptrace/ptrace.c | 142 +-----
arch/powerpc/kernel/signal.c | 8 +
arch/powerpc/kernel/syscall.c | 119 +----
arch/powerpc/kernel/traps.c | 2 +-
arch/powerpc/kernel/watchdog.c | 2 +-
arch/powerpc/perf/core-book3s.c | 2 +-
18 files changed, 698 insertions(+), 810 deletions(-)
create mode 100644 arch/powerpc/include/asm/entry-common.h
</pre>
</blockquote>
<p>Hi,</p>
<p><br>
</p>
<p>I have reviewed and tested the generic IRQ entry/exist patch
series. Below are my observations:</p>
<p><br>
Test Coverage<br>
• Successfully ran LTP (specially syscall) and entire LTP
test suite, without observing any regressions or issues related to
the implementation.</p>
<p><br>
System Configuration<br>
• CPUs: 640<br>
• Kernel: v6.17.0-rc6+<br>
• Processor mode: Shared (uncapped)</p>
<p><br>
Performance Evaluation<br>
• Conducted benchmarking using perf bench syscall basic -l.<br>
• No functional regressions observed, and results were
consistent with expectations.</p>
<p><br>
</p>
<p>The performance benchmarks from perf bench basic syscall are
below:</p>
<p><span
style="color: rgb(209, 210, 211); font-family: Monaco, Menlo, Consolas, "Courier New", monospace; font-size: 12px; font-style: normal; font-variant-ligatures: none; font-variant-caps: normal; font-weight: 400; letter-spacing: normal; orphans: 2; text-align: left; text-indent: 0px; text-transform: none; widows: 2; word-spacing: 0px; -webkit-text-stroke-width: 0px; white-space: pre-wrap; background-color: rgba(232, 232, 232, 0.04); text-decoration-thickness: initial; text-decoration-style: initial; text-decoration-color: initial; display: inline !important; float: none;">Loops = 100,000
| Metric | W/O Generic Framework | With Generic Framework | Improvement |
|----------|-----------------------:|-----------------------:|------------:|
| usecs/op | 0.124562 | 0.124253 | ~0.25% |
| ops/sec | 8,028,471 | 8,048,158 | ~0.25% |
Loops = 1,000,000
| Metric | W/O Generic Framework | With Generic Framework | Improvement |
|----------|-----------------------:|-----------------------:|------------:|
| usecs/op | 0.125389 | 0.124374 | ~0.81% |
| ops/sec | 7,977,511 | 8,040,330 | ~0.79% |
Loops = 10,000,000
| Metric | W/O Generic Framework | With Generic Framework | Improvement |
|----------|-----------------------:|-----------------------:|------------:|
| usecs/op | 0.124626 | 0.123928 | ~0.56% |
| ops/sec | 8,024,058 | 8,069,182 | ~0.56% |
**Overall (aggregated across all runs)**
| Metric | W/O Generic Framework | With Generic Framework | Improvement |
| ---------- | ---------------------:|-----------------------:|------------:|
| Total time | 1.384 [sec] | 1.376 [sec] | ~0.58% |
| usecs/op | 0.124694 | 0.123971 | ~0.58% |
| ops/sec | 8,019,904 | 8,066,393 | ~0.58% |</span><br>
<br>
Here with this benchmarking we can see the improvement close to
0.6%.<br>
<br>
Overall, the patch series works as intended in my testing.</p>
<p><br>
</p>
<p>Please add below tag for the patch set.</p>
<p><br>
Tested-by: Samir M <a class="moz-txt-link-rfc2396E" href="mailto:samir@linux.vnet.ibm.com"><samir@linux.vnet.ibm.com></a></p>
<p>Thank You !!</p>
<p><br>
</p>
</body>
</html>