From: Juergen Gross <jgross@suse.com>
To: linux-kernel@vger.kernel.org, x86@kernel.org
Cc: Juergen Gross <jgross@suse.com>,
Boris Ostrovsky <boris.ostrovsky@oracle.com>,
Thomas Gleixner <tglx@linutronix.de>,
Ingo Molnar <mingo@redhat.com>, Borislav Petkov <bp@alien8.de>,
Dave Hansen <dave.hansen@linux.intel.com>,
"H. Peter Anvin" <hpa@zytor.com>,
Josh Poimboeuf <jpoimboe@kernel.org>,
Peter Zijlstra <peterz@infradead.org>,
xen-devel@lists.xenproject.org
Subject: [PATCH v4 15/21] x86/xen: Drop xen_irq_ops
Date: Thu, 27 Nov 2025 08:08:38 +0100 [thread overview]
Message-ID: <20251127070844.21919-16-jgross@suse.com> (raw)
In-Reply-To: <20251127070844.21919-1-jgross@suse.com>
Instead of having a pre-filled array xen_irq_ops for Xen PV paravirt
functions, drop the array and assign each element individually.
This is in preparation of reducing the paravirt include hell by
splitting paravirt.h into multiple more fine grained header files,
which will in turn require to split up the pv_ops vector as well.
Dropping the pre-filled array makes life easier for objtool to
detect missing initializers in multiple pv_ops_ arrays.
Signed-off-by: Juergen Gross <jgross@suse.com>
---
V2:
- new patch
---
arch/x86/xen/irq.c | 20 +++++++-------------
tools/objtool/check.c | 1 -
2 files changed, 7 insertions(+), 14 deletions(-)
diff --git a/arch/x86/xen/irq.c b/arch/x86/xen/irq.c
index 39982f955cfe..d8678c3d3971 100644
--- a/arch/x86/xen/irq.c
+++ b/arch/x86/xen/irq.c
@@ -40,20 +40,14 @@ static void xen_halt(void)
xen_safe_halt();
}
-static const typeof(pv_ops) xen_irq_ops __initconst = {
- .irq = {
- /* Initial interrupt flag handling only called while interrupts off. */
- .save_fl = __PV_IS_CALLEE_SAVE(paravirt_ret0),
- .irq_disable = __PV_IS_CALLEE_SAVE(paravirt_nop),
- .irq_enable = __PV_IS_CALLEE_SAVE(BUG_func),
-
- .safe_halt = xen_safe_halt,
- .halt = xen_halt,
- },
-};
-
void __init xen_init_irq_ops(void)
{
- pv_ops.irq = xen_irq_ops.irq;
+ /* Initial interrupt flag handling only called while interrupts off. */
+ pv_ops.irq.save_fl = __PV_IS_CALLEE_SAVE(paravirt_ret0);
+ pv_ops.irq.irq_disable = __PV_IS_CALLEE_SAVE(paravirt_nop);
+ pv_ops.irq.irq_enable = __PV_IS_CALLEE_SAVE(BUG_func);
+ pv_ops.irq.safe_halt = xen_safe_halt;
+ pv_ops.irq.halt = xen_halt;
+
x86_init.irqs.intr_init = xen_init_IRQ;
}
diff --git a/tools/objtool/check.c b/tools/objtool/check.c
index 9004fbc06769..3fd551c080ee 100644
--- a/tools/objtool/check.c
+++ b/tools/objtool/check.c
@@ -594,7 +594,6 @@ static int init_pv_ops(struct objtool_file *file)
static const char *pv_ops_tables[] = {
"pv_ops",
"xen_cpu_ops",
- "xen_irq_ops",
"xen_mmu_ops",
NULL,
};
--
2.51.0
next prev parent reply other threads:[~2025-11-27 7:10 UTC|newest]
Thread overview: 29+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-11-27 7:08 [PATCH v4 00/21] paravirt: cleanup and reorg Juergen Gross
2025-11-27 7:08 ` [PATCH v4 01/21] x86/paravirt: Remove not needed includes of paravirt.h Juergen Gross
2025-11-27 7:08 ` [PATCH v4 02/21] x86/paravirt: Remove some unneeded struct declarations Juergen Gross
2025-11-27 7:08 ` [PATCH v4 03/21] x86/paravirt: Remove PARAVIRT_DEBUG config option Juergen Gross
2025-11-27 7:08 ` [PATCH v4 04/21] x86/paravirt: Move thunk macros to paravirt_types.h Juergen Gross
2025-11-27 7:08 ` [PATCH v4 05/21] paravirt: Remove asm/paravirt_api_clock.h Juergen Gross
2025-11-27 7:08 ` [PATCH v4 06/21] sched: Move clock related paravirt code to kernel/sched Juergen Gross
2025-11-27 7:08 ` [PATCH v4 07/21] arm/paravirt: Use common code for paravirt_steal_clock() Juergen Gross
2025-11-27 7:08 ` [PATCH v4 08/21] arm64/paravirt: " Juergen Gross
2025-11-27 7:08 ` [PATCH v4 09/21] loongarch/paravirt: " Juergen Gross
2025-11-27 7:08 ` [PATCH v4 10/21] riscv/paravirt: " Juergen Gross
2025-11-27 7:08 ` [PATCH v4 11/21] x86/paravirt: " Juergen Gross
2025-11-27 7:08 ` [PATCH v4 12/21] x86/paravirt: Move paravirt_sched_clock() related code into tsc.c Juergen Gross
2025-11-27 7:08 ` [PATCH v4 13/21] x86/paravirt: Introduce new paravirt-base.h header Juergen Gross
2025-11-27 7:08 ` [PATCH v4 14/21] x86/paravirt: Move pv_native_*() prototypes to paravirt.c Juergen Gross
2025-11-27 7:08 ` Juergen Gross [this message]
2025-12-15 19:10 ` [PATCH v4 15/21] x86/xen: Drop xen_irq_ops Boris Ostrovsky
2025-11-27 7:08 ` [PATCH v4 16/21] x86/xen: Drop xen_cpu_ops Juergen Gross
2025-12-15 19:14 ` Boris Ostrovsky
2025-11-27 7:08 ` [PATCH v4 17/21] x86/xen: Drop xen_mmu_ops Juergen Gross
2025-12-15 19:16 ` Boris Ostrovsky
2025-11-27 7:08 ` [PATCH v4 18/21] objtool: Allow multiple pv_ops arrays Juergen Gross
2025-11-27 7:08 ` [PATCH v4 19/21] x86/paravirt: Allow pv-calls outside paravirt.h Juergen Gross
2025-11-27 7:08 ` [PATCH v4 20/21] x86/paravirt: Specify pv_ops array in paravirt macros Juergen Gross
2025-11-27 7:08 ` [PATCH v4 21/21] x86/pvlocks: Move paravirt spinlock functions into own header Juergen Gross
2025-11-27 9:24 ` kernel test robot
2025-11-27 9:24 ` kernel test robot
2025-12-15 8:27 ` [PATCH v4 00/21] paravirt: cleanup and reorg Juergen Gross
2026-02-20 4:10 ` patchwork-bot+linux-riscv
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=20251127070844.21919-16-jgross@suse.com \
--to=jgross@suse.com \
--cc=boris.ostrovsky@oracle.com \
--cc=bp@alien8.de \
--cc=dave.hansen@linux.intel.com \
--cc=hpa@zytor.com \
--cc=jpoimboe@kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=mingo@redhat.com \
--cc=peterz@infradead.org \
--cc=tglx@linutronix.de \
--cc=x86@kernel.org \
--cc=xen-devel@lists.xenproject.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