From: Mohamed Mediouni <mohamed@unpredictable.fr>
To: qemu-devel@nongnu.org
Cc: "Michael S. Tsirkin" <mst@redhat.com>,
Pedro Barbuda <pbarbuda@microsoft.com>,
Mohamed Mediouni <mohamed@unpredictable.fr>,
Paolo Bonzini <pbonzini@redhat.com>,
Zhao Liu <zhao1.liu@intel.com>,
Roman Bolshakov <rbolshakov@ddn.com>,
Wei Liu <wei.liu@kernel.org>,
Phil Dennis-Jordan <phil@philjordan.eu>
Subject: [PATCH v11 14/15] hw/intc: apic: disallow APIC reads when disabled
Date: Mon, 13 Apr 2026 18:52:16 +0200 [thread overview]
Message-ID: <20260413165217.47105-15-mohamed@unpredictable.fr> (raw)
In-Reply-To: <20260413165217.47105-1-mohamed@unpredictable.fr>
!APICBASE_ENABLE + attempting to read xAPIC registers is not an allowed combination.
And neither is x2APIC enabled + attempting to read xAPIC registers
Signed-off-by: Mohamed Mediouni <mohamed@unpredictable.fr>
---
hw/intc/apic.c | 9 +++++++++
1 file changed, 9 insertions(+)
diff --git a/hw/intc/apic.c b/hw/intc/apic.c
index 8766ed00b9..c09dddef39 100644
--- a/hw/intc/apic.c
+++ b/hw/intc/apic.c
@@ -875,6 +875,15 @@ static uint64_t apic_mem_read(void *opaque, hwaddr addr, unsigned size)
return -1;
}
+ /* if the xAPIC is disabled, return early. */
+ if (!(s->apicbase & MSR_IA32_APICBASE_ENABLE)) {
+ return 0xffffffff;
+ }
+
+ if (s->apicbase & MSR_IA32_APICBASE_EXTD) {
+ return 0xffffffff;
+ }
+
index = (addr >> 4) & 0xff;
apic_register_read(s, index, &val);
--
2.50.1 (Apple Git-155)
next prev parent reply other threads:[~2026-04-13 16:55 UTC|newest]
Thread overview: 16+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-04-13 16:52 [PATCH v11 00/15] whpx: i386: bug fixes, feature probing and CPUID Mohamed Mediouni
2026-04-13 16:52 ` [PATCH v11 01/15] target/i386: emulate: include name of unhandled instruction Mohamed Mediouni
2026-04-13 16:52 ` [PATCH v11 02/15] whpx: i386: x2apic emulation Mohamed Mediouni
2026-04-13 16:52 ` [PATCH v11 03/15] whpx: i386: wire up feature probing Mohamed Mediouni
2026-04-13 16:52 ` [PATCH v11 04/15] whpx: i386: disable TbFlushHypercalls for emulated LAPIC Mohamed Mediouni
2026-04-13 16:52 ` [PATCH v11 05/15] whpx: i386: enable x2apic by default for user-mode LAPIC Mohamed Mediouni
2026-04-13 16:52 ` [PATCH v11 06/15] whpx: i386: reintroduce enlightenments for Windows 10 Mohamed Mediouni
2026-04-13 16:52 ` [PATCH v11 07/15] whpx: i386: introduce proper cpuid support Mohamed Mediouni
2026-04-13 16:52 ` [PATCH v11 08/15] whpx: i386: kernel-irqchip=off fixes Mohamed Mediouni
2026-04-13 16:52 ` [PATCH v11 09/15] whpx: i386: use WHvX64RegisterCr8 only when kernel-irqchip=off Mohamed Mediouni
2026-04-13 16:52 ` [PATCH v11 10/15] whpx: i386: disable kernel-irqchip on Windows 10 when PIC enabled Mohamed Mediouni
2026-04-13 16:52 ` [PATCH v11 11/15] whpx: i386: IO port fast path cleanup Mohamed Mediouni
2026-04-13 16:52 ` [PATCH v11 12/15] whpx: i386: disable enlightenments and LAPIC for isapc Mohamed Mediouni
2026-04-13 16:52 ` [PATCH v11 13/15] whpx: i386: interrupt priority support Mohamed Mediouni
2026-04-13 16:52 ` Mohamed Mediouni [this message]
2026-04-13 16:52 ` [PATCH v11 15/15] whpx: i386: fix CPUID[1:EDX].APIC reporting Mohamed Mediouni
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=20260413165217.47105-15-mohamed@unpredictable.fr \
--to=mohamed@unpredictable.fr \
--cc=mst@redhat.com \
--cc=pbarbuda@microsoft.com \
--cc=pbonzini@redhat.com \
--cc=phil@philjordan.eu \
--cc=qemu-devel@nongnu.org \
--cc=rbolshakov@ddn.com \
--cc=wei.liu@kernel.org \
--cc=zhao1.liu@intel.com \
/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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.