From: Magnus Kulke <magnuskulke@linux.microsoft.com>
To: Michael Tokarev <mjt@tls.msk.ru>
Cc: qemu-devel@nongnu.org, "Magnus Kulke" <magnuskulke@microsoft.com>,
"Doru Blânzeanu" <dblanzeanu@linux.microsoft.com>,
"Mohamed Mediouni" <mohamed@unpredictable.fr>,
"Wei Liu" <wei.liu@kernel.org>, "Wei Liu" <liuwe@microsoft.com>,
"Paolo Bonzini" <pbonzini@redhat.com>,
qemu-stable <qemu-stable@nongnu.org>
Subject: Re: [PATCH] target/i386/mshv: Fix segment regression in MMIO emu
Date: Sat, 25 Apr 2026 10:10:14 +0200 [thread overview]
Message-ID: <aex25q67R3TxXTzN@example.com> (raw)
In-Reply-To: <24a2d178-b674-4355-870e-415c259658e3@tls.msk.ru>
On Sat, Apr 25, 2026 at 12:35:27AM +0300, Michael Tokarev wrote:
> On 10.04.2026 17:26, Magnus Kulke wrote:
> > When the segmentation code has been reworked, there is now an
> > unconditional call to emul_ops->read_segment_descriptor(). The MSHV impl
> > was delegating this to x86_read_segement_descriptor(), which read from
> > the GDT in guest memory. This fails for selector.idx == 0 and when no
> > GDT is set up (which is the case in real mode).
> >
> > In the fix we change the MSHV impl to fill segment descriptor from
> > SegmentCache, that was populated from the hypervisor by mshv_load_regs()
> > before instruction emulation.
> >
> > Fixes: 09442d98ab (target/i386: emulate: segmentation rework)
> >
> > Signed-off-by: Magnus Kulke <magnuskulke@linux.microsoft.com>
> > ---
> > target/i386/mshv/mshv-cpu.c | 39 ++++++++++++++++++++++++++++++-------
> > 1 file changed, 32 insertions(+), 7 deletions(-)
>
> This looks like a qemu-stable material (11.0.x).
>
> Please let me know if it isn't.
>
> Thanks,
>
> /mjt
I think so, yes. real mode mmio-emu will not work, which affects pretty
much all (x86) workloads for mshv.
best,
magnus
prev parent reply other threads:[~2026-04-25 8:11 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-04-10 14:26 [PATCH] target/i386/mshv: Fix segment regression in MMIO emu Magnus Kulke
2026-04-10 16:19 ` Mohamed Mediouni
2026-04-11 12:26 ` Paolo Bonzini
2026-04-24 21:35 ` Michael Tokarev
2026-04-25 8:10 ` Magnus Kulke [this message]
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=aex25q67R3TxXTzN@example.com \
--to=magnuskulke@linux.microsoft.com \
--cc=dblanzeanu@linux.microsoft.com \
--cc=liuwe@microsoft.com \
--cc=magnuskulke@microsoft.com \
--cc=mjt@tls.msk.ru \
--cc=mohamed@unpredictable.fr \
--cc=pbonzini@redhat.com \
--cc=qemu-devel@nongnu.org \
--cc=qemu-stable@nongnu.org \
--cc=wei.liu@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 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.