summary refs log tree commit diff stats
path: root/hw/display/vhost-user-gpu-pci.c
diff options
context:
space:
mode:
authorMarkus Armbruster <armbru@redhat.com>2019-06-04 17:12:50 +0200
committerPaolo Bonzini <pbonzini@redhat.com>2019-06-05 16:15:25 +0200
commit9ea18ed25a36527167e9676f25d983df5e7f76e6 (patch)
tree1cfbbf08c6d662dae2ecacded4224b8b70878b1e /hw/display/vhost-user-gpu-pci.c
parentc87759ce876a7a0b17c2bf4f0b964bd51f0ee871 (diff)
downloadfocaccia-qemu-9ea18ed25a36527167e9676f25d983df5e7f76e6.tar.gz
focaccia-qemu-9ea18ed25a36527167e9676f25d983df5e7f76e6.zip
vl: Fix -drive / -blockdev persistent reservation management
qemu-system-FOO's main() acts on command line arguments in its own
idiosyncratic order.  There's not much method to its madness.
Whenever we find a case where one kind of command line argument needs
to refer to something created for another kind later, we rejigger the
order.

Recent commit cda4aa9a5a "vl: Create block backends before setting
machine properties" was such a rejigger.  Block backends are now
created before "delayed" objects.  This broke persistent reservation
management.  Reproducer:

    $ qemu-system-x86_64 -object pr-manager-helper,id=pr-helper0,path=/tmp/pr-helper0.sock-drive -drive file=/dev/mapper/crypt,file.pr-manager=pr-helper0,format=raw,if=none,id=drive-scsi0-0-0-2
    qemu-system-x86_64: -drive file=/dev/mapper/crypt,file.pr-manager=pr-helper0,format=raw,if=none,id=drive-scsi0-0-0-2: No persistent reservation manager with id 'pr-helper0'

The delayed pr-manager-helper object is created too late for use by
-drive or -blockdev.  Normal objects are still created in time.

pr-manager-helper has always been a delayed object (commit 7c9e527659
"scsi, file-posix: add support for persistent reservation
management").  Turns out there's no real reason for that.  Make it a
normal object.

Fixes: cda4aa9a5a08777cf13e164c0543bd4888b8adce
Signed-off-by: Markus Armbruster <armbru@redhat.com>
Message-Id: <20190604151251.9903-2-armbru@redhat.com>
Reviewed-by: Michal Privoznik <mprivozn@redhat.com>
Cc: qemu-stable@nongnu.org
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
Diffstat (limited to 'hw/display/vhost-user-gpu-pci.c')
0 files changed, 0 insertions, 0 deletions