From: "Cédric Le Goater" <clg@kaod.org>
To: <linuxppc-dev@lists.ozlabs.org>
Cc: "Cédric Le Goater" <clg@kaod.org>
Subject: [PATCH 00/11] powerpc/xive: Improve diagnostic and activate StoreEOI on P10 PowerNV
Date: Fri, 5 Nov 2021 11:26:25 +0100 [thread overview]
Message-ID: <20211105102636.1016378-1-clg@kaod.org> (raw)
Hello,
This series tries to improve diagnostic support in the XIVE driver. It
adds pr_debug() primitives that can be activated at run-time and changes
the debugfs xive entry to expose more information :
/sys/kernel/debug/powerpc/xive/
├── eqs/
│ ├── cpu0
│ ├── cpu1
│ ├── cpu2
. .
│ └── cpu99
├── interrupts
├── ipis
├── save-restore
└── store-eoi
It also introduces 2 new kernel parameters :
xive.store-eoi=off to deactivate StoreEOI at boot but it still be
reactivated through sysfs.
doorbell=off to deactivate doorbells for CPU IPIs
and XIVE instead
The first is a workaround in case of a FW advertising the wrong
feature. It can be used to check performance also. The second is for
PowerVM development and tests when the LP-per-thread mode is
activated. Doorbells cannot be used in that case.
Finally, it activates StoreEOI support for the PowerNV platform. OPAL
for P10 has been released and we are free to use this extension.
Thanks,
C.
Cédric Le Goater (11):
powerpc/xive: Replace pr_devel() by pr_debug() to ease debug
powerpc/xive: Introduce an helper to print out interrupt
characteristics
powerpc/xive: Activate StoreEOI on P10
powerpc/xive: Introduce xive_core_debugfs_create()
powerpc/xive: Change the debugfs file 'xive' into a directory
powerpc/xive: Rename the 'cpus' debugfs file to 'ipis'
powerpc/xive: Add a debugfs file to dump EQs
powerpc/xive: Add a debugfs toggle for StoreEOI
powerpc/xive: Add a kernel parameter for StoreEOI
powerpc/xive: Add a debugfs toggle for save-restore
powerpc/smp: Add a doorbell=off kernel parameter
arch/powerpc/include/asm/dbell.h | 1 +
arch/powerpc/include/asm/opal-api.h | 1 +
arch/powerpc/sysdev/xive/xive-internal.h | 1 +
arch/powerpc/kernel/dbell.c | 17 ++
arch/powerpc/platforms/powernv/smp.c | 7 +-
arch/powerpc/platforms/pseries/smp.c | 3 +
arch/powerpc/sysdev/xive/common.c | 211 ++++++++++++------
arch/powerpc/sysdev/xive/native.c | 4 +-
arch/powerpc/sysdev/xive/spapr.c | 38 ++--
.../admin-guide/kernel-parameters.txt | 16 ++
10 files changed, 209 insertions(+), 90 deletions(-)
--
2.31.1
next reply other threads:[~2021-11-05 10:33 UTC|newest]
Thread overview: 19+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-11-05 10:26 Cédric Le Goater [this message]
2021-11-05 10:26 ` [PATCH 01/11] powerpc/xive: Replace pr_devel() by pr_debug() to ease debug Cédric Le Goater
2021-11-05 10:26 ` [PATCH 02/11] powerpc/xive: Introduce an helper to print out interrupt characteristics Cédric Le Goater
2021-11-05 10:26 ` [PATCH 03/11] powerpc/xive: Activate StoreEOI on P10 Cédric Le Goater
2021-11-05 10:26 ` [PATCH 04/11] powerpc/xive: Introduce xive_core_debugfs_create() Cédric Le Goater
2021-11-18 9:21 ` Michael Ellerman
2021-11-18 15:22 ` Cédric Le Goater
2021-11-05 10:26 ` [PATCH 05/11] powerpc/xive: Change the debugfs file 'xive' into a directory Cédric Le Goater
2021-11-05 10:26 ` [PATCH 06/11] powerpc/xive: Rename the 'cpus' debugfs file to 'ipis' Cédric Le Goater
2021-11-05 10:26 ` [PATCH 07/11] powerpc/xive: Add a debugfs file to dump EQs Cédric Le Goater
2021-11-05 10:26 ` [PATCH 08/11] powerpc/xive: Add a debugfs toggle for StoreEOI Cédric Le Goater
2021-11-05 10:26 ` [PATCH 09/11] powerpc/xive: Add a kernel parameter " Cédric Le Goater
2021-11-05 10:26 ` [PATCH 10/11] powerpc/xive: Add a debugfs toggle for save-restore Cédric Le Goater
2021-11-05 10:26 ` [PATCH 11/11] powerpc/smp: Add a doorbell=off kernel parameter Cédric Le Goater
2021-11-11 10:41 ` Michael Ellerman
2021-11-11 16:01 ` Cédric Le Goater
2021-11-18 9:24 ` Michael Ellerman
2021-11-18 15:26 ` Cédric Le Goater
2021-11-25 9:36 ` [PATCH 00/11] powerpc/xive: Improve diagnostic and activate StoreEOI on P10 PowerNV Michael Ellerman
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=20211105102636.1016378-1-clg@kaod.org \
--to=clg@kaod.org \
--cc=linuxppc-dev@lists.ozlabs.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).