diff options
author | David Woodhouse <dwmw@amazon.co.uk> | 2023-10-20 18:00:18 +0100 |
---|---|---|
committer | David Woodhouse <dwmw@amazon.co.uk> | 2023-11-06 10:03:45 +0000 |
commit | a1c1082908dde4867b1ac55f546bea0c17d52318 (patch) | |
tree | b971b9d6dd328ab8b4d06067ba4c7924d29797ba | |
parent | debc995e883b05c2fd02fb797a61ab1328e5bae2 (diff) | |
download | qemu-a1c1082908dde4867b1ac55f546bea0c17d52318.zip qemu-a1c1082908dde4867b1ac55f546bea0c17d52318.tar.gz qemu-a1c1082908dde4867b1ac55f546bea0c17d52318.tar.bz2 |
hw/xen: use correct default protocol for xen-block on x86
Even on x86_64 the default protocol is the x86-32 one if the guest doesn't
specifically ask for x86-64.
Cc: qemu-stable@nongnu.org
Fixes: b6af8926fb85 ("xen: add implementations of xen-block connect and disconnect functions...")
Signed-off-by: David Woodhouse <dwmw@amazon.co.uk>
Reviewed-by: Paul Durrant <paul@xen.org>
-rw-r--r-- | hw/block/xen-block.c | 10 |
1 files changed, 7 insertions, 3 deletions
diff --git a/hw/block/xen-block.c b/hw/block/xen-block.c index a07cd7e..bfa5396 100644 --- a/hw/block/xen-block.c +++ b/hw/block/xen-block.c @@ -115,9 +115,13 @@ static void xen_block_connect(XenDevice *xendev, Error **errp) return; } - if (xen_device_frontend_scanf(xendev, "protocol", "%ms", - &str) != 1) { - protocol = BLKIF_PROTOCOL_NATIVE; + if (xen_device_frontend_scanf(xendev, "protocol", "%ms", &str) != 1) { + /* x86 defaults to the 32-bit protocol even for 64-bit guests. */ + if (object_dynamic_cast(OBJECT(qdev_get_machine()), "x86-machine")) { + protocol = BLKIF_PROTOCOL_X86_32; + } else { + protocol = BLKIF_PROTOCOL_NATIVE; + } } else { if (strcmp(str, XEN_IO_PROTO_ABI_X86_32) == 0) { protocol = BLKIF_PROTOCOL_X86_32; |