On 27/05/2025 19.20, Thomas Huth wrote:
On 12/05/2025 10.39, Philippe Mathieu-Daudé wrote:
The "fallback" property was only used by the hw_compat_2_5[] array,
as 'fallback=144'. We removed all machines using that array, lets
remove ISA floppy drive 'fallback' property, manually setting the
default value in isabus_fdc_realize().
Signed-off-by: Philippe Mathieu-Daudé <phi...@linaro.org>
Reviewed-by: Daniel P. Berrangé <berra...@redhat.com>
Reviewed-by: Zhao Liu <zhao1....@intel.com>
---
hw/block/fdc-isa.c | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
FWIW, this needs a fixup for iotest 172:
https://gitlab.com/thuth/qemu/-/jobs/10166450223#L466
FYI, since I was testing this series anyway, I'll fix up this patch with:
diff --git a/tests/qemu-iotests/172.out b/tests/qemu-iotests/172.out
--- a/tests/qemu-iotests/172.out
+++ b/tests/qemu-iotests/172.out
@@ -14,7 +14,6 @@ Testing:
dma = 2 (0x2)
fdtypeA = "auto"
fdtypeB = "auto"
- fallback = "288"
bus: floppy-bus.0
type floppy-bus
dev: floppy, id ""
@@ -43,7 +42,6 @@ Testing: -fda TEST_DIR/t.qcow2
dma = 2 (0x2)
fdtypeA = "auto"
fdtypeB = "auto"
- fallback = "288"
bus: floppy-bus.0
type floppy-bus
dev: floppy, id ""
@@ -79,7 +77,6 @@ Testing: -fdb TEST_DIR/t.qcow2
dma = 2 (0x2)
fdtypeA = "auto"
fdtypeB = "auto"
- fallback = "288"
bus: floppy-bus.0
type floppy-bus
dev: floppy, id ""
etc.
and I will queue this series (without the "Remove X86CPU::check_cpuid field"
patch as mentioned by Xiaoyao Li), unless Paolo or another x86 maintainer
wants to do this instead (please let me know!).
Thomas
