mirror of
https://gitlab.com/qemu-project/qemu
synced 2024-11-02 21:32:52 +00:00
7653b44534
The various Intel CPU manuals claim that SGDT and SIDT can write either 24-bits or 32-bits depending upon the operand size, but this is incorrect. Not only do the Intel CPU manuals give contradictory information between processor revisions, but this information doesn't even match real-life behaviour. In fact, tests on real hardware show that the CPU always writes 32-bits for SGDT and SIDT, and this behaviour is required for at least OS/2 Warp and WFW 3.11 with Win32s to function correctly. Remove the masking applied due to the operand size for SGDT and SIDT so that the TCG behaviour matches the behaviour on real hardware. Signed-off-by: Mark Cave-Ayland <mark.cave-ayland@ilande.co.uk> Resolves: https://gitlab.com/qemu-project/qemu/-/issues/2198 -- MCA: Whilst I don't have a copy of OS/2 Warp handy, I've confirmed that this patch fixes the issue in WFW 3.11 with Win32s. For more technical information I highly recommend the excellent write-up at https://www.os2museum.com/wp/sgdtsidt-fiction-and-reality/. Message-ID: <20240419195147.434894-1-mark.cave-ayland@ilande.co.uk> Signed-off-by: Paolo Bonzini <pbonzini@redhat.com> |
||
|---|---|---|
| .. | ||
| hvf | ||
| kvm | ||
| nvmm | ||
| tcg | ||
| whpx | ||
| arch_dump.c | ||
| arch_memory_mapping.c | ||
| confidential-guest.c | ||
| confidential-guest.h | ||
| cpu-dump.c | ||
| cpu-internal.h | ||
| cpu-param.h | ||
| cpu-qom.h | ||
| cpu-sysemu.c | ||
| cpu.c | ||
| cpu.h | ||
| gdbstub.c | ||
| helper.c | ||
| helper.h | ||
| host-cpu.c | ||
| host-cpu.h | ||
| Kconfig | ||
| machine.c | ||
| meson.build | ||
| monitor.c | ||
| ops_sse.h | ||
| sev-sysemu-stub.c | ||
| sev.c | ||
| sev.h | ||
| svm.h | ||
| trace-events | ||
| trace.h | ||
| xsave_helper.c | ||