
* Run docker probe only if docker or podman are available The docker probe uses "sudo -n" which can cause an e-mail with a security warning each time when configure is run. Therefore run docker probe only if either docker or podman are available. That avoids the problematic "sudo -n" on build environments which have neither docker nor podman installed. Fixes: c4575b59155e2e00 ("configure: store container engine in config-host.mak") Signed-off-by: Stefan Weil <sw@weilnetz.de> Message-Id: <20221030083510.310584-1-sw@weilnetz.de> Signed-off-by: Alex Bennée <alex.bennee@linaro.org> Reviewed-by: Thomas Huth <thuth@redhat.com> Message-Id: <20221117172532.538149-2-alex.bennee@linaro.org> * tests/avocado/machine_aspeed.py: Reduce noise on the console for SDK tests The Aspeed SDK images are based on OpenBMC which starts a lot of services. The output noise on the console can break from time to time the test waiting for the logging prompt. Change the U-Boot bootargs variable to add "quiet" to the kernel command line and reduce the output volume. This also drops the test on the CPU id which was nice to have but not essential. Signed-off-by: Cédric Le Goater <clg@kaod.org> Message-Id: <20221104075347.370503-1-clg@kaod.org> Signed-off-by: Alex Bennée <alex.bennee@linaro.org> Message-Id: <20221117172532.538149-3-alex.bennee@linaro.org> * tests/docker: allow user to override check target This is useful when trying to bisect a particular failing test behind a docker run. For example: make docker-test-clang@fedora \ TARGET_LIST=arm-softmmu \ TEST_COMMAND="meson test qtest-arm/qos-test" \ J=9 V=1 Signed-off-by: Alex Bennée <alex.bennee@linaro.org> Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org> Message-Id: <20221117172532.538149-4-alex.bennee@linaro.org> * docs/devel: add a maintainers section to development process We don't currently have a clear place in the documentation to describe the roles and responsibilities of a maintainer. Lets create one so we can. I've moved a few small bits out of other files to try and keep everything in one place. Signed-off-by: Alex Bennée <alex.bennee@linaro.org> Reviewed-by: Stefan Hajnoczi <stefanha@redhat.com> Reviewed-by: Paolo Bonzini <pbonzini@redhat.com> Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org> Message-Id: <20221117172532.538149-5-alex.bennee@linaro.org> * docs/devel: make language a little less code centric We welcome all sorts of patches. Signed-off-by: Alex Bennée <alex.bennee@linaro.org> Reviewed-by: Stefan Hajnoczi <stefanha@redhat.com> Reviewed-by: Paolo Bonzini <pbonzini@redhat.com> Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org> Message-Id: <20221117172532.538149-6-alex.bennee@linaro.org> * docs/devel: simplify the minimal checklist The bullet points are quite long and contain process tips. Move those bits of the bullet to the relevant sections and link to them. Use a table for nicer formatting of the checklist. Signed-off-by: Alex Bennée <alex.bennee@linaro.org> Reviewed-by: Stefan Hajnoczi <stefanha@redhat.com> Reviewed-by: Paolo Bonzini <pbonzini@redhat.com> Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org> Message-Id: <20221117172532.538149-7-alex.bennee@linaro.org> * docs/devel: try and improve the language around patch review It is important that contributors take the review process seriously and we collaborate in a respectful way while avoiding personal attacks. Try and make this clear in the language. Signed-off-by: Alex Bennée <alex.bennee@linaro.org> Reviewed-by: Markus Armbruster <armbru@redhat.com> Reviewed-by: Stefan Hajnoczi <stefanha@redhat.com> Reviewed-by: Paolo Bonzini <pbonzini@redhat.com> Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org> Message-Id: <20221117172532.538149-8-alex.bennee@linaro.org> * tests/avocado: Raise timeout for boot_linux.py:BootLinuxPPC64.test_pseries_tcg On my machine, a debug build of QEMU takes about 260 seconds to complete this test, so with the current timeout value of 180 seconds it always times out. Double the timeout value to 360 so the test definitely has enough time to complete. Signed-off-by: Peter Maydell <peter.maydell@linaro.org> Signed-off-by: Alex Bennée <alex.bennee@linaro.org> Message-Id: <20221110142901.3832318-1-peter.maydell@linaro.org> Message-Id: <20221117172532.538149-9-alex.bennee@linaro.org> * tests/avocado: introduce alpine virt test for CI The boot_linux tests download and run a full cloud image boot and start a full distro. While the ability to test the full boot chain is worthwhile it is perhaps a little too heavy weight and causes issues in CI. Fix this by introducing a new alpine linux ISO boot in machine_aarch64_virt. This boots a fully loaded -cpu max with all the bells and whistles in 31s on my machine. A full debug build takes around 180s on my machine so we set a more generous timeout to cover that. We don't add a test for lesser GIC versions although there is some coverage for that already in the boot_xen.py tests. If we want to introduce more comprehensive testing we can do it with a custom kernel and initrd rather than a full distro boot. Signed-off-by: Alex Bennée <alex.bennee@linaro.org> Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org> Message-Id: <20221117172532.538149-10-alex.bennee@linaro.org> * tests/avocado: skip aarch64 cloud TCG tests in CI We now have a much lighter weight test in machine_aarch64_virt which tests the full boot chain in less time. Rename the tests while we are at it to make it clear it is a Fedora cloud image. Signed-off-by: Alex Bennée <alex.bennee@linaro.org> Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org> Message-Id: <20221117172532.538149-11-alex.bennee@linaro.org> * gitlab: integrate coverage report This should hopefully give is nice coverage information about what our tests (or at least the subset we are running) have hit. Ideally we would want a way to trigger coverage on tests likely to be affected by the current commit. Signed-off-by: Alex Bennée <alex.bennee@linaro.org> Acked-by: Stefan Hajnoczi <stefanha@redhat.com> Message-Id: <20221117172532.538149-12-alex.bennee@linaro.org> * vhost: mask VIRTIO_F_RING_RESET for vhost and vhost-user devices Commit 69e1c14aa2 ("virtio: core: vq reset feature negotation support") enabled VIRTIO_F_RING_RESET by default for all virtio devices. This feature is not currently emulated by QEMU, so for vhost and vhost-user devices we need to make sure it is supported by the offloaded device emulation (in-kernel or in another process). To do this we need to add VIRTIO_F_RING_RESET to the features bitmap passed to vhost_get_features(). This way it will be masked if the device does not support it. This issue was initially discovered with vhost-vsock and vhost-user-vsock, and then also tested with vhost-user-rng which confirmed the same issue. They fail when sending features through VHOST_SET_FEATURES ioctl or VHOST_USER_SET_FEATURES message, since VIRTIO_F_RING_RESET is negotiated by the guest (Linux >= v6.0), but not supported by the device. Fixes: 69e1c14aa2 ("virtio: core: vq reset feature negotation support") Resolves: https://gitlab.com/qemu-project/qemu/-/issues/1318 Signed-off-by: Stefano Garzarella <sgarzare@redhat.com> Message-Id: <20221121101101.29400-1-sgarzare@redhat.com> Reviewed-by: Michael S. Tsirkin <mst@redhat.com> Signed-off-by: Michael S. Tsirkin <mst@redhat.com> Reviewed-by: Stefan Hajnoczi <stefanha@redhat.com> Acked-by: Raphael Norwitz <raphael.norwitz@nutanix.com> Acked-by: Jason Wang <jasowang@redhat.com> * tests: acpi: whitelist DSDT before moving PRQx to _SB scope Signed-off-by: Igor Mammedov <imammedo@redhat.com> Message-Id: <20221121153613.3972225-2-imammedo@redhat.com> Reviewed-by: Michael S. Tsirkin <mst@redhat.com> Signed-off-by: Michael S. Tsirkin <mst@redhat.com> * acpi: x86: move RPQx field back to _SB scope Commit 47a373faa6b2 (acpi: pc/q35: drop ad-hoc PCI-ISA bridge AML routines and let bus ennumeration generate AML) moved ISA bridge AML generation to respective devices and was using aml_alias() to provide PRQx fields in _SB. scope. However, it turned out that SeaBIOS was not able to process Alias opcode when parsing DSDT, resulting in lack of keyboard during boot (SeaBIOS console, grub, FreeDOS). While fix for SeaBIOS is posted https://mail.coreboot.org/hyperkitty/list/seabios@seabios.org/thread/RGPL7HESH5U5JRLEO6FP77CZVHZK5J65/ fixed SeaBIOS might not make into QEMU-7.2 in time. Hence this workaround that puts PRQx back into _SB scope and gets rid of aliases in ISA bridge description, so DSDT will be parsable by broken SeaBIOS. That brings back hardcoded references to ISA bridge PCI0.S08.P40C/PCI0.SF8.PIRQ where middle part now is auto generated based on slot it's plugged in, but it should be fine as bridge initialization also hardcodes PCI address of the bridge so it can't ever move. Once QEMU tree has fixed SeaBIOS blob, we should be able to drop this part and revert back to alias based approach Reported-by: Volker Rümelin <vr_qemu@t-online.de> Signed-off-by: Igor Mammedov <imammedo@redhat.com> Message-Id: <20221121153613.3972225-3-imammedo@redhat.com> Reviewed-by: Michael S. Tsirkin <mst@redhat.com> Signed-off-by: Michael S. Tsirkin <mst@redhat.com> * tests: acpi: x86: update expected DSDT after moving PRQx fields in _SB scope Expected DSDT changes, pc: - Field (P40C, ByteAcc, NoLock, Preserve) + Scope (\_SB) { - PRQ0, 8, - PRQ1, 8, - PRQ2, 8, - PRQ3, 8 + Field (PCI0.S08.P40C, ByteAcc, NoLock, Preserve) + { + PRQ0, 8, + PRQ1, 8, + PRQ2, 8, + PRQ3, 8 + } } - Alias (PRQ0, \_SB.PRQ0) - Alias (PRQ1, \_SB.PRQ1) - Alias (PRQ2, \_SB.PRQ2) - Alias (PRQ3, \_SB.PRQ3) q35: - Field (PIRQ, ByteAcc, NoLock, Preserve) - { - PRQA, 8, - PRQB, 8, - PRQC, 8, - PRQD, 8, - Offset (0x08), - PRQE, 8, - PRQF, 8, - PRQG, 8, - PRQH, 8 + Scope (\_SB) + { + Field (PCI0.SF8.PIRQ, ByteAcc, NoLock, Preserve) + { + PRQA, 8, + PRQB, 8, + PRQC, 8, + PRQD, 8, + Offset (0x08), + PRQE, 8, + PRQF, 8, + PRQG, 8, + PRQH, 8 + } } - Alias (PRQA, \_SB.PRQA) - Alias (PRQB, \_SB.PRQB) - Alias (PRQC, \_SB.PRQC) - Alias (PRQD, \_SB.PRQD) - Alias (PRQE, \_SB.PRQE) - Alias (PRQF, \_SB.PRQF) - Alias (PRQG, \_SB.PRQG) - Alias (PRQH, \_SB.PRQH) Signed-off-by: Igor Mammedov <imammedo@redhat.com> Message-Id: <20221121153613.3972225-4-imammedo@redhat.com> Reviewed-by: Michael S. Tsirkin <mst@redhat.com> Signed-off-by: Michael S. Tsirkin <mst@redhat.com> * MAINTAINERS: add mst to list of biosbits maintainers Adding Michael's name to the list of bios bits maintainers so that all changes and fixes into biosbits framework can go through his tree and he is notified. Suggested-by: Michael S. Tsirkin <mst@redhat.com> Signed-off-by: Ani Sinha <ani@anisinha.ca> Message-Id: <20221111151138.36988-1-ani@anisinha.ca> Reviewed-by: Michael S. Tsirkin <mst@redhat.com> Signed-off-by: Michael S. Tsirkin <mst@redhat.com> * tests/avocado: configure acpi-bits to use avocado timeout Instead of using a hardcoded timeout, just rely on Avocado's built-in test case timeout. This helps avoid timeout issues on machines where 60 seconds is not sufficient. Signed-off-by: John Snow <jsnow@redhat.com> Message-Id: <20221115212759.3095751-1-jsnow@redhat.com> Reviewed-by: Michael S. Tsirkin <mst@redhat.com> Signed-off-by: Michael S. Tsirkin <mst@redhat.com> Reviewed-by: Ani Sinha <ani@anisinha.ca> * acpi/tests/avocado/bits: keep the work directory when BITS_DEBUG is set in env Debugging bits issue often involves running the QEMU command line manually outside of the avocado environment with the generated ISO. Hence, its inconvenient if the iso gets cleaned up after the test has finished. This change makes sure that the work directory is kept after the test finishes if the test is run with BITS_DEBUG=1 in the environment so that the iso is available for use with the QEMU command line. CC: Daniel P. Berrangé <berrange@redhat.com> Signed-off-by: Ani Sinha <ani@anisinha.ca> Message-Id: <20221117113630.543495-1-ani@anisinha.ca> Reviewed-by: Michael S. Tsirkin <mst@redhat.com> Signed-off-by: Michael S. Tsirkin <mst@redhat.com> * virtio: disable error for out of spec queue-enable Virtio 1.0 is pretty clear that features have to be negotiated before enabling VQs. Unfortunately Seabios ignored this ever since gaining 1.0 support (UEFI is ok). Comment the error out for now, and add a TODO. Fixes: 3c37f8b8d1 ("virtio: introduce virtio_queue_enable()") Cc: "Kangjie Xu" <kangjie.xu@linux.alibaba.com> Signed-off-by: Michael S. Tsirkin <mst@redhat.com> Message-Id: <20221121200339.362452-1-mst@redhat.com> * hw/loongarch: Add default stdout uart in fdt Add "chosen" subnode into LoongArch fdt, and set it's "stdout-path" prop to uart node. Signed-off-by: Xiaojuan Yang <yangxiaojuan@loongson.cn> Reviewed-by: Song Gao <gaosong@loongson.cn> Message-Id: <20221115114923.3372414-1-yangxiaojuan@loongson.cn> Signed-off-by: Song Gao <gaosong@loongson.cn> * hw/loongarch: Fix setprop_sized method in fdt rtc node. Fix setprop_sized method in fdt rtc node. Signed-off-by: Xiaojuan Yang <yangxiaojuan@loongson.cn> Reviewed-by: Richard Henderson <richard.henderson@linaro.org> Reviewed-by: Song Gao <gaosong@loongson.cn> Message-Id: <20221116040300.3459818-1-yangxiaojuan@loongson.cn> Signed-off-by: Song Gao <gaosong@loongson.cn> * hw/loongarch: Replace the value of uart info with macro Using macro to replace the value of uart info such as addr, size in acpi_build method. Signed-off-by: Xiaojuan Yang <yangxiaojuan@loongson.cn> Reviewed-by: Song Gao <gaosong@loongson.cn> Message-Id: <20221115115008.3372489-1-yangxiaojuan@loongson.cn> Signed-off-by: Song Gao <gaosong@loongson.cn> * target/arm: Don't do two-stage lookup if stage 2 is disabled In get_phys_addr_with_struct(), we call get_phys_addr_twostage() if the CPU supports EL2. However, we don't check here that stage 2 is actually enabled. Instead we only check that inside get_phys_addr_twostage() to skip stage 2 translation. This means that even if stage 2 is disabled we still tell the stage 1 lookup to do its page table walks via stage 2. This works by luck for normal CPU accesses, but it breaks for debug accesses, which are used by the disassembler and also by semihosting file reads and writes, because the debug case takes a different code path inside S1_ptw_translate(). This means that setups that use semihosting for file loads are broken (a regression since 7.1, introduced in recent ptw refactoring), and that sometimes disassembly in debug logs reports "unable to read memory" rather than showing the guest insns. Fix the bug by hoisting the "is stage 2 enabled?" check up to get_phys_addr_with_struct(), so that we handle S2 disabled the same way we do the "no EL2" case, with a simple single stage lookup. Reported-by: Jens Wiklander <jens.wiklander@linaro.org> Reviewed-by: Richard Henderson <richard.henderson@linaro.org> Signed-off-by: Peter Maydell <peter.maydell@linaro.org> Message-id: 20221121212404.1450382-1-peter.maydell@linaro.org * target/arm: Use signed quantity to represent VMSAv8-64 translation level The LPA2 extension implements 52-bit virtual addressing for 4k and 16k translation granules, and for the former, this means an additional level of translation is needed. This means we start counting at -1 instead of 0 when doing a walk, and so 'level' is now a signed quantity, and should be typed as such. So turn it from uint32_t into int32_t. This avoids a level of -1 getting misinterpreted as being >= 3, and terminating a page table walk prematurely with a bogus output address. Cc: Peter Maydell <peter.maydell@linaro.org> Cc: Philippe Mathieu-Daudé <f4bug@amsat.org> Cc: Richard Henderson <richard.henderson@linaro.org> Signed-off-by: Ard Biesheuvel <ardb@kernel.org> Reviewed-by: Peter Maydell <peter.maydell@linaro.org> Signed-off-by: Peter Maydell <peter.maydell@linaro.org> * Update VERSION for v7.2.0-rc2 Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com> * tests/avocado: Update the URLs of the advent calendar images The qemu-advent-calendar.org server will be decommissioned soon. I've mirrored the images that we use for the QEMU CI to gitlab, so update their URLs to point to the new location. Message-Id: <20221121102436.78635-1-thuth@redhat.com> Tested-by: Philippe Mathieu-Daudé <philmd@linaro.org> Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org> Signed-off-by: Thomas Huth <thuth@redhat.com> * tests/qtest: Decrease the amount of output from the qom-test The logs in the gitlab-CI have a size constraint, and sometimes we already hit this limit. The biggest part of the log then seems to be filled by the qom-test, so we should decrease the size of the output - which can be done easily by not printing the path for each property, since the path has already been logged at the beginning of each node that we handle here. However, if we omit the path, we should make sure to not recurse into child nodes in between, so that it is clear to which node each property belongs. Thus store the children and links in a temporary list and recurse only at the end of each node, when all properties have already been printed. Message-Id: <20221121194240.149268-1-thuth@redhat.com> Reviewed-by: Stefan Hajnoczi <stefanha@redhat.com> Signed-off-by: Thomas Huth <thuth@redhat.com> * tests/avocado: use new rootfs for orangepi test The old URL wasn't stable. I suspect the current URL will only be stable for a few months so maybe we need another strategy for hosting rootfs snapshots? Signed-off-by: Alex Bennée <alex.bennee@linaro.org> Message-Id: <20221118113309.1057790-1-alex.bennee@linaro.org> Signed-off-by: Thomas Huth <thuth@redhat.com> * Revert "usbredir: avoid queuing hello packet on snapshot restore" Run state is also in RUN_STATE_PRELAUNCH while "-S" is used. This reverts commit 0631d4b448454ae8a1ab091c447e3f71ab6e088a Signed-off-by: Joelle van Dyne <j@getutm.app> Reviewed-by: Ján Tomko <jtomko@redhat.com> The original commit broke the usage of usbredir with libvirt, which starts every domain with "-S". This workaround is no longer needed because the usbredir behavior has been fixed in the meantime: https://gitlab.freedesktop.org/spice/usbredir/-/merge_requests/61 Signed-off-by: Ján Tomko <jtomko@redhat.com> Message-Id: <1689cec3eadcea87255e390cb236033aca72e168.1669193161.git.jtomko@redhat.com> Signed-off-by: Gerd Hoffmann <kraxel@redhat.com> * gtk: disable GTK Clipboard with a new meson option The GTK Clipboard implementation may cause guest hangs. Therefore implement new configure switch: --enable-gtk-clipboard, as a meson option disabled by default, which warns in the help text about the experimental nature of the feature. Regenerate the meson build options to include it. The initialization of the clipboard is gtk.c, as well as the compilation of gtk-clipboard.c are now conditional on this new option to be set. Resolves: https://gitlab.com/qemu-project/qemu/-/issues/1150 Signed-off-by: Claudio Fontana <cfontana@suse.de> Acked-by: Gerd Hoffmann <kraxel@redhat.com> Reviewed-by: Jim Fehlig <jfehlig@suse.com> Message-Id: <20221121135538.14625-1-cfontana@suse.de> Signed-off-by: Gerd Hoffmann <kraxel@redhat.com> * hw/usb/hcd-xhci.c: spelling: tranfer Fixes: effaf5a240e03020f4ae953e10b764622c3e87cc Signed-off-by: Michael Tokarev <mjt@tls.msk.ru> Reviewed-by: Thomas Huth <thuth@redhat.com> Reviewed-by: Stefan Weil <sw@weilnetz.de> Message-Id: <20221105114851.306206-1-mjt@msgid.tls.msk.ru> Signed-off-by: Gerd Hoffmann <kraxel@redhat.com> * ui/gtk: prevent ui lock up when dpy_gl_update called again before current draw event occurs A warning, "qemu: warning: console: no gl-unblock within" followed by guest scanout lockup can happen if dpy_gl_update is called in a row and the second call is made before gd_draw_event scheduled by the first call is taking place. This is because draw call returns without decrementing gl_block ref count if the dmabuf was already submitted as shown below. (gd_gl_area_draw/gd_egl_draw) if (dmabuf) { if (!dmabuf->draw_submitted) { return; } else { dmabuf->draw_submitted = false; } } So it should not schedule any redundant draw event in case draw_submitted is already set in gd_egl_fluch/gd_gl_area_scanout_flush. Cc: Gerd Hoffmann <kraxel@redhat.com> Cc: Vivek Kasireddy <vivek.kasireddy@intel.com> Signed-off-by: Dongwon Kim <dongwon.kim@intel.com> Reviewed-by: Marc-André Lureau <marcandre.lureau@redhat.com> Message-Id: <20221021192315.9110-1-dongwon.kim@intel.com> Signed-off-by: Gerd Hoffmann <kraxel@redhat.com> * hw/usb/hcd-xhci: Reset the XHCIState with device_cold_reset() Currently the hcd-xhci-pci and hcd-xhci-sysbus devices, which are mostly wrappers around the TYPE_XHCI device, which is a direct subclass of TYPE_DEVICE. Since TYPE_DEVICE devices are not on any qbus and do not get automatically reset, the wrapper devices both reset the TYPE_XHCI device in their own reset functions. However, they do this using device_legacy_reset(), which will reset the device itself but not any bus it has. Switch to device_cold_reset(), which avoids using a deprecated function and also propagates reset along any child buses. Signed-off-by: Peter Maydell <peter.maydell@linaro.org> Message-Id: <20221014145423.2102706-1-peter.maydell@linaro.org> Signed-off-by: Gerd Hoffmann <kraxel@redhat.com> * hw/audio/intel-hda: don't reset codecs twice Currently the intel-hda device has a reset method which manually resets all the codecs by calling device_legacy_reset() on them. This means they get reset twice, once because child devices on a qbus get reset before the parent device's reset method is called, and then again because we're manually resetting them. Drop the manual reset call, and ensure that codecs are still reset when the guest does a reset via ICH6_GCTL_RESET by using device_cold_reset() (which resets all the devices on the qbus as well as the device itself) instead of a direct call to the reset function. This is a slight ordering change because the (only) codec reset now happens before the controller registers etc are reset, rather than once before and then once after, but the codec reset function hda_audio_reset() doesn't care. This lets us drop a use of device_legacy_reset(), which is deprecated. Signed-off-by: Peter Maydell <peter.maydell@linaro.org> Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org> Message-Id: <20221014142632.2092404-2-peter.maydell@linaro.org> Signed-off-by: Gerd Hoffmann <kraxel@redhat.com> * hw/audio/intel-hda: Drop unnecessary prototype The only use of intel_hda_reset() is after its definition, so we don't need to separately declare its prototype at the top of the file; drop the unnecessary line. Signed-off-by: Peter Maydell <peter.maydell@linaro.org> Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org> Message-Id: <20221014142632.2092404-3-peter.maydell@linaro.org> Signed-off-by: Gerd Hoffmann <kraxel@redhat.com> * add syx snapshot extras * it compiles! * virtiofsd: Add `sigreturn` to the seccomp whitelist The virtiofsd currently crashes on s390x. This is because of a `sigreturn` system call. See audit log below: type=SECCOMP msg=audit(1669382477.611:459): auid=4294967295 uid=0 gid=0 ses=4294967295 subj=system_u:system_r:virtd_t:s0-s0:c0.c1023 pid=6649 comm="virtiofsd" exe="/usr/libexec/virtiofsd" sig=31 arch=80000016 syscall=119 compat=0 ip=0x3fff15f748a code=0x80000000AUID="unset" UID="root" GID="root" ARCH=s390x SYSCALL=sigreturn Signed-off-by: Marc Hartmayer <mhartmay@linux.ibm.com> Reviewed-by: German Maglione <gmaglione@redhat.com> Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com> Message-Id: <20221125143946.27717-1-mhartmay@linux.ibm.com> * libvhost-user: Fix wrong type of argument to formatting function (reported by LGTM) Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org> Signed-off-by: Stefan Weil <sw@weilnetz.de> Message-Id: <20220422070144.1043697-2-sw@weilnetz.de> Signed-off-by: Laurent Vivier <laurent@vivier.eu> Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com> Message-Id: <20221126152507.283271-2-sw@weilnetz.de> * libvhost-user: Fix format strings Signed-off-by: Stefan Weil <sw@weilnetz.de> Reviewed-by: Marc-André Lureau <marcandre.lureau@redhat.com> Message-Id: <20220422070144.1043697-3-sw@weilnetz.de> Signed-off-by: Laurent Vivier <laurent@vivier.eu> Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com> Message-Id: <20221126152507.283271-3-sw@weilnetz.de> * libvhost-user: Fix two more format strings This fix is required for 32 bit hosts. The bug was detected by CI for arm-linux, but is also relevant for i386-linux. Reported-by: Stefan Hajnoczi <stefanha@gmail.com> Signed-off-by: Stefan Weil <sw@weilnetz.de> Reviewed-by: Stefan Hajnoczi <stefanha@redhat.com> Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com> Message-Id: <20221126152507.283271-4-sw@weilnetz.de> * libvhost-user: Add format attribute to local function vu_panic Signed-off-by: Stefan Weil <sw@weilnetz.de> Reviewed-by: Marc-André Lureau <marcandre.lureau@redhat.com> Message-Id: <20220422070144.1043697-4-sw@weilnetz.de> Signed-off-by: Laurent Vivier <laurent@vivier.eu> Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com> Message-Id: <20221126152507.283271-5-sw@weilnetz.de> * MAINTAINERS: Add subprojects/libvhost-user to section "vhost" Signed-off-by: Stefan Weil <sw@weilnetz.de> [Michael agreed to act as maintainer for libvhost-user via email in https://lore.kernel.org/qemu-devel/20221123015218-mutt-send-email-mst@kernel.org/. --Stefan] Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com> Message-Id: <20221126152507.283271-6-sw@weilnetz.de> * Add G_GNUC_PRINTF to function qemu_set_info_str and fix related issues With the G_GNUC_PRINTF function attribute the compiler detects two potential insecure format strings: ../../../net/stream.c:248:31: warning: format string is not a string literal (potentially insecure) [-Wformat-security] qemu_set_info_str(&s->nc, uri); ^~~ ../../../net/stream.c:322:31: warning: format string is not a string literal (potentially insecure) [-Wformat-security] qemu_set_info_str(&s->nc, uri); ^~~ There are also two other warnings: ../../../net/socket.c:182:35: warning: zero-length gnu_printf format string [-Wformat-zero-length] 182 | qemu_set_info_str(&s->nc, ""); | ^~ ../../../net/stream.c:170:35: warning: zero-length gnu_printf format string [-Wformat-zero-length] 170 | qemu_set_info_str(&s->nc, ""); Reviewed-by: Peter Maydell <peter.maydell@linaro.org> Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org> Signed-off-by: Stefan Weil <sw@weilnetz.de> Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com> Message-Id: <20221126152507.283271-7-sw@weilnetz.de> * del ramfile * update seabios source from 1.16.0 to 1.16.1 git shortlog rel-1.16.0..rel-1.16.1 =================================== Gerd Hoffmann (3): malloc: use variable for ZoneHigh size malloc: use large ZoneHigh when there is enough memory virtio-blk: use larger default request size Igor Mammedov (1): acpi: parse Alias object Volker Rümelin (2): pci: refactor the pci_config_*() functions reset: force standard PCI configuration access Xiaofei Lee (1): virtio-blk: Fix incorrect type conversion in virtio_blk_op() Xuan Zhuo (2): virtio-mmio: read/write the hi 32 features for mmio virtio: finalize features before using device Signed-off-by: Gerd Hoffmann <kraxel@redhat.com> * update seabios binaries to 1.16.1 Signed-off-by: Gerd Hoffmann <kraxel@redhat.com> * fix for non i386 archs * replay: Fix declaration of replay_read_next_clock Fixes the build with gcc 13: replay/replay-time.c:34:6: error: conflicting types for \ 'replay_read_next_clock' due to enum/integer mismatch; \ have 'void(ReplayClockKind)' [-Werror=enum-int-mismatch] 34 | void replay_read_next_clock(ReplayClockKind kind) | ^~~~~~~~~~~~~~~~~~~~~~ In file included from ../qemu/replay/replay-time.c:14: replay/replay-internal.h:139:6: note: previous declaration of \ 'replay_read_next_clock' with type 'void(unsigned int)' 139 | void replay_read_next_clock(unsigned int kind); | ^~~~~~~~~~~~~~~~~~~~~~ Fixes: 8eda206e090 ("replay: recording and replaying clock ticks") Signed-off-by: Richard Henderson <richard.henderson@linaro.org> Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org> Reviewed-by: Wilfred Mallawa <wilfred.mallawa@wdc.com> Reviewed-by: Pavel Dovgalyuk <Pavel.Dovgalyuk@ispras.ru> Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com> Message-Id: <20221129010547.284051-1-richard.henderson@linaro.org> * hw/display/qxl: Have qxl_log_command Return early if no log_cmd handler Only 3 command types are logged: no need to call qxl_phys2virt() for the other types. Using different cases will help to pass different structure sizes to qxl_phys2virt() in a pair of commits. Reviewed-by: Marc-André Lureau <marcandre.lureau@redhat.com> Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org> Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com> Message-Id: <20221128202741.4945-2-philmd@linaro.org> * hw/display/qxl: Document qxl_phys2virt() Reviewed-by: Marc-André Lureau <marcandre.lureau@redhat.com> Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org> Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com> Message-Id: <20221128202741.4945-3-philmd@linaro.org> * hw/display/qxl: Pass requested buffer size to qxl_phys2virt() Currently qxl_phys2virt() doesn't check for buffer overrun. In order to do so in the next commit, pass the buffer size as argument. For QXLCursor in qxl_render_cursor() -> qxl_cursor() we verify the size of the chunked data ahead, checking we can access 'sizeof(QXLCursor) + chunk->data_size' bytes. Since in the SPICE_CURSOR_TYPE_MONO case the cursor is assumed to fit in one chunk, no change are required. In SPICE_CURSOR_TYPE_ALPHA the ahead read is handled in qxl_unpack_chunks(). Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org> Acked-by: Gerd Hoffmann <kraxel@redhat.com> Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com> Message-Id: <20221128202741.4945-4-philmd@linaro.org> * hw/display/qxl: Avoid buffer overrun in qxl_phys2virt (CVE-2022-4144) Have qxl_get_check_slot_offset() return false if the requested buffer size does not fit within the slot memory region. Similarly qxl_phys2virt() now returns NULL in such case, and qxl_dirty_one_surface() aborts. This avoids buffer overrun in the host pointer returned by memory_region_get_ram_ptr(). Fixes: CVE-2022-4144 (out-of-bounds read) Reported-by: Wenxu Yin (@awxylitol) Resolves: https://gitlab.com/qemu-project/qemu/-/issues/1336 Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org> Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com> Message-Id: <20221128202741.4945-5-philmd@linaro.org> * hw/display/qxl: Assert memory slot fits in preallocated MemoryRegion Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org> Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com> Message-Id: <20221128202741.4945-6-philmd@linaro.org> * block-backend: avoid bdrv_unregister_buf() NULL pointer deref bdrv_*() APIs expect a valid BlockDriverState. Calling them with bs=NULL leads to undefined behavior. Jonathan Cameron reported this following NULL pointer dereference when a VM with a virtio-blk device and a memory-backend-file object is terminated: 1. qemu_cleanup() closes all drives, setting blk->root to NULL 2. qemu_cleanup() calls user_creatable_cleanup(), which results in a RAM block notifier callback because the memory-backend-file is destroyed. 3. blk_unregister_buf() is called by virtio-blk's BlockRamRegistrar notifier callback and undefined behavior occurs. Fixes: baf422684d73 ("virtio-blk: use BDRV_REQ_REGISTERED_BUF optimization hint") Co-authored-by: Jonathan Cameron <Jonathan.Cameron@huawei.com> Reviewed-by: Kevin Wolf <kwolf@redhat.com> Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org> Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com> Message-Id: <20221121211923.1993171-1-stefanha@redhat.com> * target/arm: Set TCGCPUOps.restore_state_to_opc for v7m This setting got missed, breaking v7m. Fixes: 56c6c98df85c ("target/arm: Convert to tcg_ops restore_state_to_opc") Resolves: https://gitlab.com/qemu-project/qemu/-/issues/1347 Signed-off-by: Richard Henderson <richard.henderson@linaro.org> Signed-off-by: Evgeny Ermakov <evgeny.v.ermakov@gmail.com> Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com> Message-Id: <20221129204146.550394-1-richard.henderson@linaro.org> * Update VERSION for v7.2.0-rc3 Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com> * hooks are now post mem access * tests/qtests: override "force-legacy" for gpio virtio-mmio tests The GPIO device is a VIRTIO_F_VERSION_1 devices but running with a legacy MMIO interface we miss out that feature bit causing confusion. For the GPIO test force the mmio bus to support non-legacy so we can properly test it. Signed-off-by: Alex Bennée <alex.bennee@linaro.org> Reviewed-by: Michael S. Tsirkin <mst@redhat.com> Resolves: https://gitlab.com/qemu-project/qemu/-/issues/1333 Message-Id: <20221130112439.2527228-2-alex.bennee@linaro.org> Acked-by: Thomas Huth <thuth@redhat.com> Signed-off-by: Michael S. Tsirkin <mst@redhat.com> * vhost: enable vrings in vhost_dev_start() for vhost-user devices Commit 02b61f38d3 ("hw/virtio: incorporate backend features in features") properly negotiates VHOST_USER_F_PROTOCOL_FEATURES with the vhost-user backend, but we forgot to enable vrings as specified in docs/interop/vhost-user.rst: If ``VHOST_USER_F_PROTOCOL_FEATURES`` has not been negotiated, the ring starts directly in the enabled state. If ``VHOST_USER_F_PROTOCOL_FEATURES`` has been negotiated, the ring is initialized in a disabled state and is enabled by ``VHOST_USER_SET_VRING_ENABLE`` with parameter 1. Some vhost-user front-ends already did this by calling vhost_ops.vhost_set_vring_enable() directly: - backends/cryptodev-vhost.c - hw/net/virtio-net.c - hw/virtio/vhost-user-gpio.c But most didn't do that, so we would leave the vrings disabled and some backends would not work. We observed this issue with the rust version of virtiofsd [1], which uses the event loop [2] provided by the vhost-user-backend crate where requests are not processed if vring is not enabled. Let's fix this issue by enabling the vrings in vhost_dev_start() for vhost-user front-ends that don't already do this directly. Same thing also in vhost_dev_stop() where we disable vrings. [1] https://gitlab.com/virtio-fs/virtiofsd [2] https://github.com/rust-vmm/vhost/blob/240fc2966/crates/vhost-user-backend/src/event_loop.rs#L217 Fixes: 02b61f38d3 ("hw/virtio: incorporate backend features in features") Reported-by: German Maglione <gmaglione@redhat.com> Tested-by: German Maglione <gmaglione@redhat.com> Signed-off-by: Stefano Garzarella <sgarzare@redhat.com> Acked-by: Raphael Norwitz <raphael.norwitz@nutanix.com> Message-Id: <20221123131630.52020-1-sgarzare@redhat.com> Signed-off-by: Alex Bennée <alex.bennee@linaro.org> Reviewed-by: Michael S. Tsirkin <mst@redhat.com> Message-Id: <20221130112439.2527228-3-alex.bennee@linaro.org> Signed-off-by: Michael S. Tsirkin <mst@redhat.com> * hw/virtio: add started_vu status field to vhost-user-gpio As per the fix to vhost-user-blk in f5b22d06fb (vhost: recheck dev state in the vhost_migration_log routine) we really should track the connection and starting separately. Signed-off-by: Alex Bennée <alex.bennee@linaro.org> Reviewed-by: Michael S. Tsirkin <mst@redhat.com> Message-Id: <20221130112439.2527228-4-alex.bennee@linaro.org> Signed-off-by: Michael S. Tsirkin <mst@redhat.com> * hw/virtio: generalise CHR_EVENT_CLOSED handling ..and use for both virtio-user-blk and virtio-user-gpio. This avoids the circular close by deferring shutdown due to disconnection until a later point. virtio-user-blk already had this mechanism in place so generalise it as a vhost-user helper function and use for both blk and gpio devices. While we are at it we also fix up vhost-user-gpio to re-establish the event handler after close down so we can reconnect later. Signed-off-by: Alex Bennée <alex.bennee@linaro.org> Reviewed-by: Michael S. Tsirkin <mst@redhat.com> Reviewed-by: Raphael Norwitz <raphael.norwitz@nutanix.com> Message-Id: <20221130112439.2527228-5-alex.bennee@linaro.org> Signed-off-by: Michael S. Tsirkin <mst@redhat.com> * include/hw: VM state takes precedence in virtio_device_should_start The VM status should always preempt the device status for these checks. This ensures the device is in the correct state when we suspend the VM prior to migrations. This restores the checks to the order they where in before the refactoring moved things around. While we are at it lets improve our documentation of the various fields involved and document the two functions. Fixes: 9f6bcfd99f (hw/virtio: move vm_running check to virtio_device_started) Fixes: 259d69c00b (hw/virtio: introduce virtio_device_should_start) Signed-off-by: Alex Bennée <alex.bennee@linaro.org> Tested-by: Christian Borntraeger <borntraeger@linux.ibm.com> Reviewed-by: Michael S. Tsirkin <mst@redhat.com> Message-Id: <20221130112439.2527228-6-alex.bennee@linaro.org> Signed-off-by: Michael S. Tsirkin <mst@redhat.com> * hw/nvme: fix aio cancel in format There are several bugs in the async cancel code for the Format command. Firstly, cancelling a format operation neglects to set iocb->ret as well as clearing the iocb->aiocb after cancelling the underlying aiocb which causes the aio callback to ignore the cancellation. Trivial fix. Secondly, and worse, because the request is queued up for posting to the CQ in a bottom half, if the cancellation is due to the submission queue being deleted (which calls blk_aio_cancel), the req structure is deallocated in nvme_del_sq prior to the bottom half being schedulued. Fix this by simply removing the bottom half, there is no reason to defer it anyway. Fixes: 3bcf26d3d619 ("hw/nvme: reimplement format nvm to allow cancellation") Reported-by: Jonathan Derrick <jonathan.derrick@linux.dev> Reviewed-by: Keith Busch <kbusch@kernel.org> Signed-off-by: Klaus Jensen <k.jensen@samsung.com> * hw/nvme: fix aio cancel in flush Make sure that iocb->aiocb is NULL'ed when cancelling. Fix a potential use-after-free by removing the bottom half and enqueuing the completion directly. Fixes: 38f4ac65ac88 ("hw/nvme: reimplement flush to allow cancellation") Reviewed-by: Keith Busch <kbusch@kernel.org> Signed-off-by: Klaus Jensen <k.jensen@samsung.com> * hw/nvme: fix aio cancel in zone reset If the zone reset operation is cancelled but the block unmap operation completes normally, the callback will continue resetting the next zone since it neglects to check iocb->ret which will have been set to -ECANCELED. Make sure that this is checked and bail out if an error is present. Secondly, fix a potential use-after-free by removing the bottom half and enqueuing the completion directly. Fixes: 63d96e4ffd71 ("hw/nvme: reimplement zone reset to allow cancellation") Reviewed-by: Keith Busch <kbusch@kernel.org> Signed-off-by: Klaus Jensen <k.jensen@samsung.com> * hw/nvme: fix aio cancel in dsm When the DSM operation is cancelled asynchronously, we set iocb->ret to -ECANCELED. However, the callback function only checks the return value of the completed aio, which may have completed succesfully prior to the cancellation and thus the callback ends up continuing the dsm operation instead of bailing out. Fix this. Secondly, fix a potential use-after-free by removing the bottom half and enqueuing the completion directly. Fixes: d7d1474fd85d ("hw/nvme: reimplement dsm to allow cancellation") Reviewed-by: Keith Busch <kbusch@kernel.org> Signed-off-by: Klaus Jensen <k.jensen@samsung.com> * hw/nvme: remove copy bh scheduling Fix a potential use-after-free by removing the bottom half and enqueuing the completion directly. Fixes: 796d20681d9b ("hw/nvme: reimplement the copy command to allow aio cancellation") Reviewed-by: Keith Busch <kbusch@kernel.org> Signed-off-by: Klaus Jensen <k.jensen@samsung.com> * target/i386: allow MMX instructions with CR4.OSFXSR=0 MMX state is saved/restored by FSAVE/FRSTOR so the instructions are not illegal opcodes even if CR4.OSFXSR=0. Make sure that validate_vex takes into account the prefix and only checks HF_OSFXSR_MASK in the presence of an SSE instruction. Fixes: 20581aadec5e ("target/i386: validate VEX prefixes via the instructions' exception classes", 2022-10-18) Resolves: https://gitlab.com/qemu-project/qemu/-/issues/1350 Reported-by: Helge Konetzka (@hejko on gitlab.com) Reviewed-by: Richard Henderson <richard.henderson@linaro.org> Signed-off-by: Paolo Bonzini <pbonzini@redhat.com> * target/i386: Always completely initialize TranslateFault In get_physical_address, the canonical address check failed to set TranslateFault.stage2, which resulted in an uninitialized read from the struct when reporting the fault in x86_cpu_tlb_fill. Adjust all error paths to use structure assignment so that the entire struct is always initialized. Reported-by: Daniel Hoffman <dhoff749@gmail.com> Fixes: 9bbcf372193a ("target/i386: Reorg GET_HPHYS") Signed-off-by: Richard Henderson <richard.henderson@linaro.org> Message-Id: <20221201074522.178498-1-richard.henderson@linaro.org> Resolves: https://gitlab.com/qemu-project/qemu/-/issues/1324 Signed-off-by: Paolo Bonzini <pbonzini@redhat.com> * hw/loongarch/virt: Add cfi01 pflash device Add cfi01 pflash device for LoongArch virt machine Signed-off-by: Xiaojuan Yang <yangxiaojuan@loongson.cn> Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org> Message-Id: <20221130100647.398565-1-yangxiaojuan@loongson.cn> Signed-off-by: Song Gao <gaosong@loongson.cn> * Sync pc on breakpoints * tests/qtest/migration-test: Fix unlink error and memory leaks When running the migration test compiled with Clang from Fedora 37 and sanitizers enabled, there is an error complaining about unlink(): ../tests/qtest/migration-test.c:1072:12: runtime error: null pointer passed as argument 1, which is declared to never be null /usr/include/unistd.h:858:48: note: nonnull attribute specified here SUMMARY: UndefinedBehaviorSanitizer: undefined-behavior ../tests/qtest/migration-test.c:1072:12 in (test program exited with status code 1) TAP parsing error: Too few tests run (expected 33, got 20) The data->clientcert and data->clientkey pointers can indeed be unset in some tests, so we have to check them before calling unlink() with those. While we're at it, I also noticed that the code is only freeing some but not all of the allocated strings in this function, and indeed, valgrind is also complaining about memory leaks here. So let's call g_free() on all allocated strings to avoid leaking memory here. Message-Id: <20221125083054.117504-1-thuth@redhat.com> Tested-by: Bin Meng <bmeng@tinylab.org> Reviewed-by: Daniel P. Berrangé <berrange@redhat.com> Reviewed-by: Juan Quintela <quintela@redhat.com> Signed-off-by: Thomas Huth <thuth@redhat.com> * target/s390x/tcg: Fix and improve the SACF instruction The SET ADDRESS SPACE CONTROL FAST instruction is not privileged, it can be used from problem space, too. Just the switching to the home address space is privileged and should still generate a privilege exception. This bug is e.g. causing programs like Java that use the "getcpu" vdso kernel function to crash (see https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=990417#26 ). While we're at it, also check if DAT is not enabled. In that case the instruction is supposed to generate a special operation exception. Resolves: https://gitlab.com/qemu-project/qemu/-/issues/655 Message-Id: <20221201184443.136355-1-thuth@redhat.com> Reviewed-by: Ilya Leoshkevich <iii@linux.ibm.com> Reviewed-by: David Hildenbrand <david@redhat.com> Reviewed-by: Richard Henderson <richard.henderson@linaro.org> Signed-off-by: Thomas Huth <thuth@redhat.com> * hw/display/next-fb: Fix comment typo Signed-off-by: Evgeny Ermakov <evgeny.v.ermakov@gmail.com> Message-Id: <20221125160849.23711-1-evgeny.v.ermakov@gmail.com> Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org> Reviewed-by: Peter Maydell <peter.maydell@linaro.org> Signed-off-by: Thomas Huth <thuth@redhat.com> * fix dev snapshots * working syx snaps * Revert "hw/loongarch/virt: Add cfi01 pflash device" This reverts commit 14dccc8ea6ece7ee63273144fb55e4770a05e0fd. Signed-off-by: Song Gao <gaosong@loongson.cn> Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com> Message-Id: <20221205113007.683505-1-gaosong@loongson.cn> * Update VERSION for v7.2.0-rc4 Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com> Signed-off-by: Stefan Weil <sw@weilnetz.de> Signed-off-by: Alex Bennée <alex.bennee@linaro.org> Signed-off-by: Cédric Le Goater <clg@kaod.org> Signed-off-by: Peter Maydell <peter.maydell@linaro.org> Signed-off-by: Stefano Garzarella <sgarzare@redhat.com> Signed-off-by: Michael S. Tsirkin <mst@redhat.com> Signed-off-by: Igor Mammedov <imammedo@redhat.com> Signed-off-by: Ani Sinha <ani@anisinha.ca> Signed-off-by: John Snow <jsnow@redhat.com> Signed-off-by: Xiaojuan Yang <yangxiaojuan@loongson.cn> Signed-off-by: Song Gao <gaosong@loongson.cn> Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com> Signed-off-by: Ard Biesheuvel <ardb@kernel.org> Signed-off-by: Thomas Huth <thuth@redhat.com> Signed-off-by: Ján Tomko <jtomko@redhat.com> Signed-off-by: Gerd Hoffmann <kraxel@redhat.com> Signed-off-by: Claudio Fontana <cfontana@suse.de> Signed-off-by: Michael Tokarev <mjt@tls.msk.ru> Signed-off-by: Dongwon Kim <dongwon.kim@intel.com> Signed-off-by: Marc Hartmayer <mhartmay@linux.ibm.com> Signed-off-by: Laurent Vivier <laurent@vivier.eu> Signed-off-by: Richard Henderson <richard.henderson@linaro.org> Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org> Signed-off-by: Evgeny Ermakov <evgeny.v.ermakov@gmail.com> Signed-off-by: Klaus Jensen <k.jensen@samsung.com> Signed-off-by: Paolo Bonzini <pbonzini@redhat.com> Co-authored-by: Stefan Weil <sw@weilnetz.de> Co-authored-by: Cédric Le Goater <clg@kaod.org> Co-authored-by: Alex Bennée <alex.bennee@linaro.org> Co-authored-by: Peter Maydell <peter.maydell@linaro.org> Co-authored-by: Stefano Garzarella <sgarzare@redhat.com> Co-authored-by: Igor Mammedov <imammedo@redhat.com> Co-authored-by: Ani Sinha <ani@anisinha.ca> Co-authored-by: John Snow <jsnow@redhat.com> Co-authored-by: Michael S. Tsirkin <mst@redhat.com> Co-authored-by: Xiaojuan Yang <yangxiaojuan@loongson.cn> Co-authored-by: Stefan Hajnoczi <stefanha@redhat.com> Co-authored-by: Ard Biesheuvel <ardb@kernel.org> Co-authored-by: Thomas Huth <thuth@redhat.com> Co-authored-by: Joelle van Dyne <j@getutm.app> Co-authored-by: Claudio Fontana <cfontana@suse.de> Co-authored-by: Michael Tokarev <mjt@tls.msk.ru> Co-authored-by: Dongwon Kim <dongwon.kim@intel.com> Co-authored-by: Marc Hartmayer <mhartmay@linux.ibm.com> Co-authored-by: Stefan Weil via <qemu-devel@nongnu.org> Co-authored-by: Gerd Hoffmann <kraxel@redhat.com> Co-authored-by: Richard Henderson <richard.henderson@linaro.org> Co-authored-by: Philippe Mathieu-Daudé <philmd@linaro.org> Co-authored-by: Jonathan Cameron <Jonathan.Cameron@huawei.com> Co-authored-by: Evgeny Ermakov <evgeny.v.ermakov@gmail.com> Co-authored-by: Klaus Jensen <k.jensen@samsung.com> Co-authored-by: Paolo Bonzini <pbonzini@redhat.com> Co-authored-by: Song Gao <gaosong@loongson.cn>
2637 lines
68 KiB
C
2637 lines
68 KiB
C
/*
|
|
* QEMU Block backends
|
|
*
|
|
* Copyright (C) 2014-2016 Red Hat, Inc.
|
|
*
|
|
* Authors:
|
|
* Markus Armbruster <armbru@redhat.com>,
|
|
*
|
|
* This work is licensed under the terms of the GNU LGPL, version 2.1
|
|
* or later. See the COPYING.LIB file in the top-level directory.
|
|
*/
|
|
|
|
#include "qemu/osdep.h"
|
|
#include "sysemu/block-backend.h"
|
|
#include "block/block_int.h"
|
|
#include "block/blockjob.h"
|
|
#include "block/coroutines.h"
|
|
#include "block/throttle-groups.h"
|
|
#include "hw/qdev-core.h"
|
|
#include "sysemu/blockdev.h"
|
|
#include "sysemu/runstate.h"
|
|
#include "sysemu/replay.h"
|
|
#include "qapi/error.h"
|
|
#include "qapi/qapi-events-block.h"
|
|
#include "qemu/id.h"
|
|
#include "qemu/main-loop.h"
|
|
#include "qemu/option.h"
|
|
#include "trace.h"
|
|
#include "migration/misc.h"
|
|
|
|
/* Number of coroutines to reserve per attached device model */
|
|
#define COROUTINE_POOL_RESERVATION 64
|
|
|
|
#define NOT_DONE 0x7fffffff /* used while emulated sync operation in progress */
|
|
|
|
static AioContext *blk_aiocb_get_aio_context(BlockAIOCB *acb);
|
|
|
|
typedef struct BlockBackendAioNotifier {
|
|
void (*attached_aio_context)(AioContext *new_context, void *opaque);
|
|
void (*detach_aio_context)(void *opaque);
|
|
void *opaque;
|
|
QLIST_ENTRY(BlockBackendAioNotifier) list;
|
|
} BlockBackendAioNotifier;
|
|
|
|
struct BlockBackend {
|
|
char *name;
|
|
int refcnt;
|
|
BdrvChild *root;
|
|
AioContext *ctx;
|
|
DriveInfo *legacy_dinfo; /* null unless created by drive_new() */
|
|
QTAILQ_ENTRY(BlockBackend) link; /* for block_backends */
|
|
QTAILQ_ENTRY(BlockBackend) monitor_link; /* for monitor_block_backends */
|
|
BlockBackendPublic public;
|
|
|
|
DeviceState *dev; /* attached device model, if any */
|
|
const BlockDevOps *dev_ops;
|
|
void *dev_opaque;
|
|
|
|
/* If the BDS tree is removed, some of its options are stored here (which
|
|
* can be used to restore those options in the new BDS on insert) */
|
|
BlockBackendRootState root_state;
|
|
|
|
bool enable_write_cache;
|
|
|
|
/* I/O stats (display with "info blockstats"). */
|
|
BlockAcctStats stats;
|
|
|
|
BlockdevOnError on_read_error, on_write_error;
|
|
bool iostatus_enabled;
|
|
BlockDeviceIoStatus iostatus;
|
|
|
|
uint64_t perm;
|
|
uint64_t shared_perm;
|
|
bool disable_perm;
|
|
|
|
bool allow_aio_context_change;
|
|
bool allow_write_beyond_eof;
|
|
|
|
/* Protected by BQL */
|
|
NotifierList remove_bs_notifiers, insert_bs_notifiers;
|
|
QLIST_HEAD(, BlockBackendAioNotifier) aio_notifiers;
|
|
|
|
int quiesce_counter;
|
|
CoQueue queued_requests;
|
|
bool disable_request_queuing;
|
|
|
|
VMChangeStateEntry *vmsh;
|
|
bool force_allow_inactivate;
|
|
|
|
/* Number of in-flight aio requests. BlockDriverState also counts
|
|
* in-flight requests but aio requests can exist even when blk->root is
|
|
* NULL, so we cannot rely on its counter for that case.
|
|
* Accessed with atomic ops.
|
|
*/
|
|
unsigned int in_flight;
|
|
};
|
|
|
|
typedef struct BlockBackendAIOCB {
|
|
BlockAIOCB common;
|
|
BlockBackend *blk;
|
|
int ret;
|
|
} BlockBackendAIOCB;
|
|
|
|
static const AIOCBInfo block_backend_aiocb_info = {
|
|
.get_aio_context = blk_aiocb_get_aio_context,
|
|
.aiocb_size = sizeof(BlockBackendAIOCB),
|
|
};
|
|
|
|
static void drive_info_del(DriveInfo *dinfo);
|
|
static BlockBackend *bdrv_first_blk(BlockDriverState *bs);
|
|
|
|
/* All BlockBackends. Protected by BQL. */
|
|
static QTAILQ_HEAD(, BlockBackend) block_backends =
|
|
QTAILQ_HEAD_INITIALIZER(block_backends);
|
|
|
|
/*
|
|
* All BlockBackends referenced by the monitor and which are iterated through by
|
|
* blk_next(). Protected by BQL.
|
|
*/
|
|
static QTAILQ_HEAD(, BlockBackend) monitor_block_backends =
|
|
QTAILQ_HEAD_INITIALIZER(monitor_block_backends);
|
|
|
|
static void blk_root_inherit_options(BdrvChildRole role, bool parent_is_format,
|
|
int *child_flags, QDict *child_options,
|
|
int parent_flags, QDict *parent_options)
|
|
{
|
|
/* We're not supposed to call this function for root nodes */
|
|
abort();
|
|
}
|
|
static void blk_root_drained_begin(BdrvChild *child);
|
|
static bool blk_root_drained_poll(BdrvChild *child);
|
|
static void blk_root_drained_end(BdrvChild *child, int *drained_end_counter);
|
|
|
|
static void blk_root_change_media(BdrvChild *child, bool load);
|
|
static void blk_root_resize(BdrvChild *child);
|
|
|
|
static bool blk_root_change_aio_ctx(BdrvChild *child, AioContext *ctx,
|
|
GHashTable *visited, Transaction *tran,
|
|
Error **errp);
|
|
|
|
static char *blk_root_get_parent_desc(BdrvChild *child)
|
|
{
|
|
BlockBackend *blk = child->opaque;
|
|
g_autofree char *dev_id = NULL;
|
|
|
|
if (blk->name) {
|
|
return g_strdup_printf("block device '%s'", blk->name);
|
|
}
|
|
|
|
dev_id = blk_get_attached_dev_id(blk);
|
|
if (*dev_id) {
|
|
return g_strdup_printf("block device '%s'", dev_id);
|
|
} else {
|
|
/* TODO Callback into the BB owner for something more detailed */
|
|
return g_strdup("an unnamed block device");
|
|
}
|
|
}
|
|
|
|
static const char *blk_root_get_name(BdrvChild *child)
|
|
{
|
|
return blk_name(child->opaque);
|
|
}
|
|
|
|
static void blk_vm_state_changed(void *opaque, bool running, RunState state)
|
|
{
|
|
Error *local_err = NULL;
|
|
BlockBackend *blk = opaque;
|
|
|
|
if (state == RUN_STATE_INMIGRATE) {
|
|
return;
|
|
}
|
|
|
|
qemu_del_vm_change_state_handler(blk->vmsh);
|
|
blk->vmsh = NULL;
|
|
blk_set_perm(blk, blk->perm, blk->shared_perm, &local_err);
|
|
if (local_err) {
|
|
error_report_err(local_err);
|
|
}
|
|
}
|
|
|
|
/*
|
|
* Notifies the user of the BlockBackend that migration has completed. qdev
|
|
* devices can tighten their permissions in response (specifically revoke
|
|
* shared write permissions that we needed for storage migration).
|
|
*
|
|
* If an error is returned, the VM cannot be allowed to be resumed.
|
|
*/
|
|
static void blk_root_activate(BdrvChild *child, Error **errp)
|
|
{
|
|
BlockBackend *blk = child->opaque;
|
|
Error *local_err = NULL;
|
|
uint64_t saved_shared_perm;
|
|
|
|
if (!blk->disable_perm) {
|
|
return;
|
|
}
|
|
|
|
blk->disable_perm = false;
|
|
|
|
/*
|
|
* blk->shared_perm contains the permissions we want to share once
|
|
* migration is really completely done. For now, we need to share
|
|
* all; but we also need to retain blk->shared_perm, which is
|
|
* overwritten by a successful blk_set_perm() call. Save it and
|
|
* restore it below.
|
|
*/
|
|
saved_shared_perm = blk->shared_perm;
|
|
|
|
blk_set_perm(blk, blk->perm, BLK_PERM_ALL, &local_err);
|
|
if (local_err) {
|
|
error_propagate(errp, local_err);
|
|
blk->disable_perm = true;
|
|
return;
|
|
}
|
|
blk->shared_perm = saved_shared_perm;
|
|
|
|
if (runstate_check(RUN_STATE_INMIGRATE)) {
|
|
/* Activation can happen when migration process is still active, for
|
|
* example when nbd_server_add is called during non-shared storage
|
|
* migration. Defer the shared_perm update to migration completion. */
|
|
if (!blk->vmsh) {
|
|
blk->vmsh = qemu_add_vm_change_state_handler(blk_vm_state_changed,
|
|
blk);
|
|
}
|
|
return;
|
|
}
|
|
|
|
blk_set_perm(blk, blk->perm, blk->shared_perm, &local_err);
|
|
if (local_err) {
|
|
error_propagate(errp, local_err);
|
|
blk->disable_perm = true;
|
|
return;
|
|
}
|
|
}
|
|
|
|
void blk_set_force_allow_inactivate(BlockBackend *blk)
|
|
{
|
|
GLOBAL_STATE_CODE();
|
|
blk->force_allow_inactivate = true;
|
|
}
|
|
|
|
static bool blk_can_inactivate(BlockBackend *blk)
|
|
{
|
|
/* If it is a guest device, inactivate is ok. */
|
|
if (blk->dev || blk_name(blk)[0]) {
|
|
return true;
|
|
}
|
|
|
|
/* Inactivating means no more writes to the image can be done,
|
|
* even if those writes would be changes invisible to the
|
|
* guest. For block job BBs that satisfy this, we can just allow
|
|
* it. This is the case for mirror job source, which is required
|
|
* by libvirt non-shared block migration. */
|
|
if (!(blk->perm & (BLK_PERM_WRITE | BLK_PERM_WRITE_UNCHANGED))) {
|
|
return true;
|
|
}
|
|
|
|
return blk->force_allow_inactivate;
|
|
}
|
|
|
|
static int blk_root_inactivate(BdrvChild *child)
|
|
{
|
|
BlockBackend *blk = child->opaque;
|
|
|
|
if (blk->disable_perm) {
|
|
return 0;
|
|
}
|
|
|
|
if (!blk_can_inactivate(blk)) {
|
|
return -EPERM;
|
|
}
|
|
|
|
blk->disable_perm = true;
|
|
if (blk->root) {
|
|
bdrv_child_try_set_perm(blk->root, 0, BLK_PERM_ALL, &error_abort);
|
|
}
|
|
|
|
return 0;
|
|
}
|
|
|
|
static void blk_root_attach(BdrvChild *child)
|
|
{
|
|
BlockBackend *blk = child->opaque;
|
|
BlockBackendAioNotifier *notifier;
|
|
|
|
trace_blk_root_attach(child, blk, child->bs);
|
|
|
|
QLIST_FOREACH(notifier, &blk->aio_notifiers, list) {
|
|
bdrv_add_aio_context_notifier(child->bs,
|
|
notifier->attached_aio_context,
|
|
notifier->detach_aio_context,
|
|
notifier->opaque);
|
|
}
|
|
}
|
|
|
|
static void blk_root_detach(BdrvChild *child)
|
|
{
|
|
BlockBackend *blk = child->opaque;
|
|
BlockBackendAioNotifier *notifier;
|
|
|
|
trace_blk_root_detach(child, blk, child->bs);
|
|
|
|
QLIST_FOREACH(notifier, &blk->aio_notifiers, list) {
|
|
bdrv_remove_aio_context_notifier(child->bs,
|
|
notifier->attached_aio_context,
|
|
notifier->detach_aio_context,
|
|
notifier->opaque);
|
|
}
|
|
}
|
|
|
|
static AioContext *blk_root_get_parent_aio_context(BdrvChild *c)
|
|
{
|
|
BlockBackend *blk = c->opaque;
|
|
IO_CODE();
|
|
|
|
return blk_get_aio_context(blk);
|
|
}
|
|
|
|
static const BdrvChildClass child_root = {
|
|
.inherit_options = blk_root_inherit_options,
|
|
|
|
.change_media = blk_root_change_media,
|
|
.resize = blk_root_resize,
|
|
.get_name = blk_root_get_name,
|
|
.get_parent_desc = blk_root_get_parent_desc,
|
|
|
|
.drained_begin = blk_root_drained_begin,
|
|
.drained_poll = blk_root_drained_poll,
|
|
.drained_end = blk_root_drained_end,
|
|
|
|
.activate = blk_root_activate,
|
|
.inactivate = blk_root_inactivate,
|
|
|
|
.attach = blk_root_attach,
|
|
.detach = blk_root_detach,
|
|
|
|
.change_aio_ctx = blk_root_change_aio_ctx,
|
|
|
|
.get_parent_aio_context = blk_root_get_parent_aio_context,
|
|
};
|
|
|
|
/*
|
|
* Create a new BlockBackend with a reference count of one.
|
|
*
|
|
* @perm is a bitmasks of BLK_PERM_* constants which describes the permissions
|
|
* to request for a block driver node that is attached to this BlockBackend.
|
|
* @shared_perm is a bitmask which describes which permissions may be granted
|
|
* to other users of the attached node.
|
|
* Both sets of permissions can be changed later using blk_set_perm().
|
|
*
|
|
* Return the new BlockBackend on success, null on failure.
|
|
*/
|
|
BlockBackend *blk_new(AioContext *ctx, uint64_t perm, uint64_t shared_perm)
|
|
{
|
|
BlockBackend *blk;
|
|
|
|
GLOBAL_STATE_CODE();
|
|
|
|
blk = g_new0(BlockBackend, 1);
|
|
blk->refcnt = 1;
|
|
blk->ctx = ctx;
|
|
blk->perm = perm;
|
|
blk->shared_perm = shared_perm;
|
|
blk_set_enable_write_cache(blk, true);
|
|
|
|
blk->on_read_error = BLOCKDEV_ON_ERROR_REPORT;
|
|
blk->on_write_error = BLOCKDEV_ON_ERROR_ENOSPC;
|
|
|
|
block_acct_init(&blk->stats);
|
|
|
|
qemu_co_queue_init(&blk->queued_requests);
|
|
notifier_list_init(&blk->remove_bs_notifiers);
|
|
notifier_list_init(&blk->insert_bs_notifiers);
|
|
QLIST_INIT(&blk->aio_notifiers);
|
|
|
|
QTAILQ_INSERT_TAIL(&block_backends, blk, link);
|
|
return blk;
|
|
}
|
|
|
|
/*
|
|
* Create a new BlockBackend connected to an existing BlockDriverState.
|
|
*
|
|
* @perm is a bitmasks of BLK_PERM_* constants which describes the
|
|
* permissions to request for @bs that is attached to this
|
|
* BlockBackend. @shared_perm is a bitmask which describes which
|
|
* permissions may be granted to other users of the attached node.
|
|
* Both sets of permissions can be changed later using blk_set_perm().
|
|
*
|
|
* Return the new BlockBackend on success, null on failure.
|
|
*/
|
|
BlockBackend *blk_new_with_bs(BlockDriverState *bs, uint64_t perm,
|
|
uint64_t shared_perm, Error **errp)
|
|
{
|
|
BlockBackend *blk = blk_new(bdrv_get_aio_context(bs), perm, shared_perm);
|
|
|
|
GLOBAL_STATE_CODE();
|
|
|
|
if (blk_insert_bs(blk, bs, errp) < 0) {
|
|
blk_unref(blk);
|
|
return NULL;
|
|
}
|
|
return blk;
|
|
}
|
|
|
|
/*
|
|
* Creates a new BlockBackend, opens a new BlockDriverState, and connects both.
|
|
* The new BlockBackend is in the main AioContext.
|
|
*
|
|
* Just as with bdrv_open(), after having called this function the reference to
|
|
* @options belongs to the block layer (even on failure).
|
|
*
|
|
* TODO: Remove @filename and @flags; it should be possible to specify a whole
|
|
* BDS tree just by specifying the @options QDict (or @reference,
|
|
* alternatively). At the time of adding this function, this is not possible,
|
|
* though, so callers of this function have to be able to specify @filename and
|
|
* @flags.
|
|
*/
|
|
BlockBackend *blk_new_open(const char *filename, const char *reference,
|
|
QDict *options, int flags, Error **errp)
|
|
{
|
|
BlockBackend *blk;
|
|
BlockDriverState *bs;
|
|
uint64_t perm = 0;
|
|
uint64_t shared = BLK_PERM_ALL;
|
|
|
|
GLOBAL_STATE_CODE();
|
|
|
|
/*
|
|
* blk_new_open() is mainly used in .bdrv_create implementations and the
|
|
* tools where sharing isn't a major concern because the BDS stays private
|
|
* and the file is generally not supposed to be used by a second process,
|
|
* so we just request permission according to the flags.
|
|
*
|
|
* The exceptions are xen_disk and blockdev_init(); in these cases, the
|
|
* caller of blk_new_open() doesn't make use of the permissions, but they
|
|
* shouldn't hurt either. We can still share everything here because the
|
|
* guest devices will add their own blockers if they can't share.
|
|
*/
|
|
if ((flags & BDRV_O_NO_IO) == 0) {
|
|
perm |= BLK_PERM_CONSISTENT_READ;
|
|
if (flags & BDRV_O_RDWR) {
|
|
perm |= BLK_PERM_WRITE;
|
|
}
|
|
}
|
|
if (flags & BDRV_O_RESIZE) {
|
|
perm |= BLK_PERM_RESIZE;
|
|
}
|
|
if (flags & BDRV_O_NO_SHARE) {
|
|
shared = BLK_PERM_CONSISTENT_READ | BLK_PERM_WRITE_UNCHANGED;
|
|
}
|
|
|
|
blk = blk_new(qemu_get_aio_context(), perm, shared);
|
|
bs = bdrv_open(filename, reference, options, flags, errp);
|
|
if (!bs) {
|
|
blk_unref(blk);
|
|
return NULL;
|
|
}
|
|
|
|
blk->root = bdrv_root_attach_child(bs, "root", &child_root,
|
|
BDRV_CHILD_FILTERED | BDRV_CHILD_PRIMARY,
|
|
perm, shared, blk, errp);
|
|
if (!blk->root) {
|
|
blk_unref(blk);
|
|
return NULL;
|
|
}
|
|
|
|
return blk;
|
|
}
|
|
|
|
static void blk_delete(BlockBackend *blk)
|
|
{
|
|
assert(!blk->refcnt);
|
|
assert(!blk->name);
|
|
assert(!blk->dev);
|
|
if (blk->public.throttle_group_member.throttle_state) {
|
|
blk_io_limits_disable(blk);
|
|
}
|
|
if (blk->root) {
|
|
blk_remove_bs(blk);
|
|
}
|
|
if (blk->vmsh) {
|
|
qemu_del_vm_change_state_handler(blk->vmsh);
|
|
blk->vmsh = NULL;
|
|
}
|
|
assert(QLIST_EMPTY(&blk->remove_bs_notifiers.notifiers));
|
|
assert(QLIST_EMPTY(&blk->insert_bs_notifiers.notifiers));
|
|
assert(QLIST_EMPTY(&blk->aio_notifiers));
|
|
QTAILQ_REMOVE(&block_backends, blk, link);
|
|
drive_info_del(blk->legacy_dinfo);
|
|
block_acct_cleanup(&blk->stats);
|
|
g_free(blk);
|
|
}
|
|
|
|
static void drive_info_del(DriveInfo *dinfo)
|
|
{
|
|
if (!dinfo) {
|
|
return;
|
|
}
|
|
qemu_opts_del(dinfo->opts);
|
|
g_free(dinfo);
|
|
}
|
|
|
|
int blk_get_refcnt(BlockBackend *blk)
|
|
{
|
|
GLOBAL_STATE_CODE();
|
|
return blk ? blk->refcnt : 0;
|
|
}
|
|
|
|
/*
|
|
* Increment @blk's reference count.
|
|
* @blk must not be null.
|
|
*/
|
|
void blk_ref(BlockBackend *blk)
|
|
{
|
|
assert(blk->refcnt > 0);
|
|
GLOBAL_STATE_CODE();
|
|
blk->refcnt++;
|
|
}
|
|
|
|
/*
|
|
* Decrement @blk's reference count.
|
|
* If this drops it to zero, destroy @blk.
|
|
* For convenience, do nothing if @blk is null.
|
|
*/
|
|
void blk_unref(BlockBackend *blk)
|
|
{
|
|
GLOBAL_STATE_CODE();
|
|
if (blk) {
|
|
assert(blk->refcnt > 0);
|
|
if (blk->refcnt > 1) {
|
|
blk->refcnt--;
|
|
} else {
|
|
blk_drain(blk);
|
|
/* blk_drain() cannot resurrect blk, nobody held a reference */
|
|
assert(blk->refcnt == 1);
|
|
blk->refcnt = 0;
|
|
blk_delete(blk);
|
|
}
|
|
}
|
|
}
|
|
|
|
/*
|
|
* Behaves similarly to blk_next() but iterates over all BlockBackends, even the
|
|
* ones which are hidden (i.e. are not referenced by the monitor).
|
|
*/
|
|
BlockBackend *blk_all_next(BlockBackend *blk)
|
|
{
|
|
GLOBAL_STATE_CODE();
|
|
return blk ? QTAILQ_NEXT(blk, link)
|
|
: QTAILQ_FIRST(&block_backends);
|
|
}
|
|
|
|
void blk_remove_all_bs(void)
|
|
{
|
|
BlockBackend *blk = NULL;
|
|
|
|
GLOBAL_STATE_CODE();
|
|
|
|
while ((blk = blk_all_next(blk)) != NULL) {
|
|
AioContext *ctx = blk_get_aio_context(blk);
|
|
|
|
aio_context_acquire(ctx);
|
|
if (blk->root) {
|
|
blk_remove_bs(blk);
|
|
}
|
|
aio_context_release(ctx);
|
|
}
|
|
}
|
|
|
|
/*
|
|
* Return the monitor-owned BlockBackend after @blk.
|
|
* If @blk is null, return the first one.
|
|
* Else, return @blk's next sibling, which may be null.
|
|
*
|
|
* To iterate over all BlockBackends, do
|
|
* for (blk = blk_next(NULL); blk; blk = blk_next(blk)) {
|
|
* ...
|
|
* }
|
|
*/
|
|
BlockBackend *blk_next(BlockBackend *blk)
|
|
{
|
|
GLOBAL_STATE_CODE();
|
|
return blk ? QTAILQ_NEXT(blk, monitor_link)
|
|
: QTAILQ_FIRST(&monitor_block_backends);
|
|
}
|
|
|
|
/* Iterates over all top-level BlockDriverStates, i.e. BDSs that are owned by
|
|
* the monitor or attached to a BlockBackend */
|
|
BlockDriverState *bdrv_next(BdrvNextIterator *it)
|
|
{
|
|
BlockDriverState *bs, *old_bs;
|
|
|
|
/* Must be called from the main loop */
|
|
assert(qemu_get_current_aio_context() == qemu_get_aio_context());
|
|
|
|
/* First, return all root nodes of BlockBackends. In order to avoid
|
|
* returning a BDS twice when multiple BBs refer to it, we only return it
|
|
* if the BB is the first one in the parent list of the BDS. */
|
|
if (it->phase == BDRV_NEXT_BACKEND_ROOTS) {
|
|
BlockBackend *old_blk = it->blk;
|
|
|
|
old_bs = old_blk ? blk_bs(old_blk) : NULL;
|
|
|
|
do {
|
|
it->blk = blk_all_next(it->blk);
|
|
bs = it->blk ? blk_bs(it->blk) : NULL;
|
|
} while (it->blk && (bs == NULL || bdrv_first_blk(bs) != it->blk));
|
|
|
|
if (it->blk) {
|
|
blk_ref(it->blk);
|
|
}
|
|
blk_unref(old_blk);
|
|
|
|
if (bs) {
|
|
bdrv_ref(bs);
|
|
bdrv_unref(old_bs);
|
|
return bs;
|
|
}
|
|
it->phase = BDRV_NEXT_MONITOR_OWNED;
|
|
} else {
|
|
old_bs = it->bs;
|
|
}
|
|
|
|
/* Then return the monitor-owned BDSes without a BB attached. Ignore all
|
|
* BDSes that are attached to a BlockBackend here; they have been handled
|
|
* by the above block already */
|
|
do {
|
|
it->bs = bdrv_next_monitor_owned(it->bs);
|
|
bs = it->bs;
|
|
} while (bs && bdrv_has_blk(bs));
|
|
|
|
if (bs) {
|
|
bdrv_ref(bs);
|
|
}
|
|
bdrv_unref(old_bs);
|
|
|
|
return bs;
|
|
}
|
|
|
|
static void bdrv_next_reset(BdrvNextIterator *it)
|
|
{
|
|
*it = (BdrvNextIterator) {
|
|
.phase = BDRV_NEXT_BACKEND_ROOTS,
|
|
};
|
|
}
|
|
|
|
BlockDriverState *bdrv_first(BdrvNextIterator *it)
|
|
{
|
|
GLOBAL_STATE_CODE();
|
|
bdrv_next_reset(it);
|
|
return bdrv_next(it);
|
|
}
|
|
|
|
/* Must be called when aborting a bdrv_next() iteration before
|
|
* bdrv_next() returns NULL */
|
|
void bdrv_next_cleanup(BdrvNextIterator *it)
|
|
{
|
|
/* Must be called from the main loop */
|
|
assert(qemu_get_current_aio_context() == qemu_get_aio_context());
|
|
|
|
if (it->phase == BDRV_NEXT_BACKEND_ROOTS) {
|
|
if (it->blk) {
|
|
bdrv_unref(blk_bs(it->blk));
|
|
blk_unref(it->blk);
|
|
}
|
|
} else {
|
|
bdrv_unref(it->bs);
|
|
}
|
|
|
|
bdrv_next_reset(it);
|
|
}
|
|
|
|
/*
|
|
* Add a BlockBackend into the list of backends referenced by the monitor, with
|
|
* the given @name acting as the handle for the monitor.
|
|
* Strictly for use by blockdev.c.
|
|
*
|
|
* @name must not be null or empty.
|
|
*
|
|
* Returns true on success and false on failure. In the latter case, an Error
|
|
* object is returned through @errp.
|
|
*/
|
|
bool monitor_add_blk(BlockBackend *blk, const char *name, Error **errp)
|
|
{
|
|
assert(!blk->name);
|
|
assert(name && name[0]);
|
|
GLOBAL_STATE_CODE();
|
|
|
|
if (!id_wellformed(name)) {
|
|
error_setg(errp, "Invalid device name");
|
|
return false;
|
|
}
|
|
if (blk_by_name(name)) {
|
|
error_setg(errp, "Device with id '%s' already exists", name);
|
|
return false;
|
|
}
|
|
if (bdrv_find_node(name)) {
|
|
error_setg(errp,
|
|
"Device name '%s' conflicts with an existing node name",
|
|
name);
|
|
return false;
|
|
}
|
|
|
|
blk->name = g_strdup(name);
|
|
QTAILQ_INSERT_TAIL(&monitor_block_backends, blk, monitor_link);
|
|
return true;
|
|
}
|
|
|
|
/*
|
|
* Remove a BlockBackend from the list of backends referenced by the monitor.
|
|
* Strictly for use by blockdev.c.
|
|
*/
|
|
void monitor_remove_blk(BlockBackend *blk)
|
|
{
|
|
GLOBAL_STATE_CODE();
|
|
|
|
if (!blk->name) {
|
|
return;
|
|
}
|
|
|
|
QTAILQ_REMOVE(&monitor_block_backends, blk, monitor_link);
|
|
g_free(blk->name);
|
|
blk->name = NULL;
|
|
}
|
|
|
|
/*
|
|
* Return @blk's name, a non-null string.
|
|
* Returns an empty string iff @blk is not referenced by the monitor.
|
|
*/
|
|
const char *blk_name(const BlockBackend *blk)
|
|
{
|
|
IO_CODE();
|
|
return blk->name ?: "";
|
|
}
|
|
|
|
/*
|
|
* Return the BlockBackend with name @name if it exists, else null.
|
|
* @name must not be null.
|
|
*/
|
|
BlockBackend *blk_by_name(const char *name)
|
|
{
|
|
BlockBackend *blk = NULL;
|
|
|
|
GLOBAL_STATE_CODE();
|
|
assert(name);
|
|
while ((blk = blk_next(blk)) != NULL) {
|
|
if (!strcmp(name, blk->name)) {
|
|
return blk;
|
|
}
|
|
}
|
|
return NULL;
|
|
}
|
|
|
|
/*
|
|
* Return the BlockDriverState attached to @blk if any, else null.
|
|
*/
|
|
BlockDriverState *blk_bs(BlockBackend *blk)
|
|
{
|
|
IO_CODE();
|
|
return blk->root ? blk->root->bs : NULL;
|
|
}
|
|
|
|
static BlockBackend *bdrv_first_blk(BlockDriverState *bs)
|
|
{
|
|
BdrvChild *child;
|
|
|
|
GLOBAL_STATE_CODE();
|
|
|
|
QLIST_FOREACH(child, &bs->parents, next_parent) {
|
|
if (child->klass == &child_root) {
|
|
return child->opaque;
|
|
}
|
|
}
|
|
|
|
return NULL;
|
|
}
|
|
|
|
/*
|
|
* Returns true if @bs has an associated BlockBackend.
|
|
*/
|
|
bool bdrv_has_blk(BlockDriverState *bs)
|
|
{
|
|
GLOBAL_STATE_CODE();
|
|
return bdrv_first_blk(bs) != NULL;
|
|
}
|
|
|
|
/*
|
|
* Returns true if @bs has only BlockBackends as parents.
|
|
*/
|
|
bool bdrv_is_root_node(BlockDriverState *bs)
|
|
{
|
|
BdrvChild *c;
|
|
|
|
GLOBAL_STATE_CODE();
|
|
QLIST_FOREACH(c, &bs->parents, next_parent) {
|
|
if (c->klass != &child_root) {
|
|
return false;
|
|
}
|
|
}
|
|
|
|
return true;
|
|
}
|
|
|
|
/*
|
|
* Return @blk's DriveInfo if any, else null.
|
|
*/
|
|
DriveInfo *blk_legacy_dinfo(BlockBackend *blk)
|
|
{
|
|
GLOBAL_STATE_CODE();
|
|
return blk->legacy_dinfo;
|
|
}
|
|
|
|
/*
|
|
* Set @blk's DriveInfo to @dinfo, and return it.
|
|
* @blk must not have a DriveInfo set already.
|
|
* No other BlockBackend may have the same DriveInfo set.
|
|
*/
|
|
DriveInfo *blk_set_legacy_dinfo(BlockBackend *blk, DriveInfo *dinfo)
|
|
{
|
|
assert(!blk->legacy_dinfo);
|
|
GLOBAL_STATE_CODE();
|
|
return blk->legacy_dinfo = dinfo;
|
|
}
|
|
|
|
/*
|
|
* Return the BlockBackend with DriveInfo @dinfo.
|
|
* It must exist.
|
|
*/
|
|
BlockBackend *blk_by_legacy_dinfo(DriveInfo *dinfo)
|
|
{
|
|
BlockBackend *blk = NULL;
|
|
GLOBAL_STATE_CODE();
|
|
|
|
while ((blk = blk_next(blk)) != NULL) {
|
|
if (blk->legacy_dinfo == dinfo) {
|
|
return blk;
|
|
}
|
|
}
|
|
abort();
|
|
}
|
|
|
|
/*
|
|
* Returns a pointer to the publicly accessible fields of @blk.
|
|
*/
|
|
BlockBackendPublic *blk_get_public(BlockBackend *blk)
|
|
{
|
|
GLOBAL_STATE_CODE();
|
|
return &blk->public;
|
|
}
|
|
|
|
/*
|
|
* Returns a BlockBackend given the associated @public fields.
|
|
*/
|
|
BlockBackend *blk_by_public(BlockBackendPublic *public)
|
|
{
|
|
GLOBAL_STATE_CODE();
|
|
return container_of(public, BlockBackend, public);
|
|
}
|
|
|
|
/*
|
|
* Disassociates the currently associated BlockDriverState from @blk.
|
|
*/
|
|
void blk_remove_bs(BlockBackend *blk)
|
|
{
|
|
ThrottleGroupMember *tgm = &blk->public.throttle_group_member;
|
|
BdrvChild *root;
|
|
|
|
GLOBAL_STATE_CODE();
|
|
|
|
notifier_list_notify(&blk->remove_bs_notifiers, blk);
|
|
if (tgm->throttle_state) {
|
|
BlockDriverState *bs = blk_bs(blk);
|
|
|
|
/*
|
|
* Take a ref in case blk_bs() changes across bdrv_drained_begin(), for
|
|
* example, if a temporary filter node is removed by a blockjob.
|
|
*/
|
|
bdrv_ref(bs);
|
|
bdrv_drained_begin(bs);
|
|
throttle_group_detach_aio_context(tgm);
|
|
throttle_group_attach_aio_context(tgm, qemu_get_aio_context());
|
|
bdrv_drained_end(bs);
|
|
bdrv_unref(bs);
|
|
}
|
|
|
|
blk_update_root_state(blk);
|
|
|
|
/* bdrv_root_unref_child() will cause blk->root to become stale and may
|
|
* switch to a completion coroutine later on. Let's drain all I/O here
|
|
* to avoid that and a potential QEMU crash.
|
|
*/
|
|
blk_drain(blk);
|
|
root = blk->root;
|
|
blk->root = NULL;
|
|
bdrv_root_unref_child(root);
|
|
}
|
|
|
|
/*
|
|
* Associates a new BlockDriverState with @blk.
|
|
*/
|
|
int blk_insert_bs(BlockBackend *blk, BlockDriverState *bs, Error **errp)
|
|
{
|
|
ThrottleGroupMember *tgm = &blk->public.throttle_group_member;
|
|
GLOBAL_STATE_CODE();
|
|
bdrv_ref(bs);
|
|
blk->root = bdrv_root_attach_child(bs, "root", &child_root,
|
|
BDRV_CHILD_FILTERED | BDRV_CHILD_PRIMARY,
|
|
blk->perm, blk->shared_perm,
|
|
blk, errp);
|
|
if (blk->root == NULL) {
|
|
return -EPERM;
|
|
}
|
|
|
|
notifier_list_notify(&blk->insert_bs_notifiers, blk);
|
|
if (tgm->throttle_state) {
|
|
throttle_group_detach_aio_context(tgm);
|
|
throttle_group_attach_aio_context(tgm, bdrv_get_aio_context(bs));
|
|
}
|
|
|
|
return 0;
|
|
}
|
|
|
|
/*
|
|
* Change BlockDriverState associated with @blk.
|
|
*/
|
|
int blk_replace_bs(BlockBackend *blk, BlockDriverState *new_bs, Error **errp)
|
|
{
|
|
GLOBAL_STATE_CODE();
|
|
return bdrv_replace_child_bs(blk->root, new_bs, errp);
|
|
}
|
|
|
|
/*
|
|
* Sets the permission bitmasks that the user of the BlockBackend needs.
|
|
*/
|
|
int blk_set_perm(BlockBackend *blk, uint64_t perm, uint64_t shared_perm,
|
|
Error **errp)
|
|
{
|
|
int ret;
|
|
GLOBAL_STATE_CODE();
|
|
|
|
if (blk->root && !blk->disable_perm) {
|
|
ret = bdrv_child_try_set_perm(blk->root, perm, shared_perm, errp);
|
|
if (ret < 0) {
|
|
return ret;
|
|
}
|
|
}
|
|
|
|
blk->perm = perm;
|
|
blk->shared_perm = shared_perm;
|
|
|
|
return 0;
|
|
}
|
|
|
|
void blk_get_perm(BlockBackend *blk, uint64_t *perm, uint64_t *shared_perm)
|
|
{
|
|
GLOBAL_STATE_CODE();
|
|
*perm = blk->perm;
|
|
*shared_perm = blk->shared_perm;
|
|
}
|
|
|
|
/*
|
|
* Attach device model @dev to @blk.
|
|
* Return 0 on success, -EBUSY when a device model is attached already.
|
|
*/
|
|
int blk_attach_dev(BlockBackend *blk, DeviceState *dev)
|
|
{
|
|
GLOBAL_STATE_CODE();
|
|
if (blk->dev) {
|
|
return -EBUSY;
|
|
}
|
|
|
|
/* While migration is still incoming, we don't need to apply the
|
|
* permissions of guest device BlockBackends. We might still have a block
|
|
* job or NBD server writing to the image for storage migration. */
|
|
if (runstate_check(RUN_STATE_INMIGRATE)) {
|
|
blk->disable_perm = true;
|
|
}
|
|
|
|
blk_ref(blk);
|
|
blk->dev = dev;
|
|
blk_iostatus_reset(blk);
|
|
|
|
return 0;
|
|
}
|
|
|
|
/*
|
|
* Detach device model @dev from @blk.
|
|
* @dev must be currently attached to @blk.
|
|
*/
|
|
void blk_detach_dev(BlockBackend *blk, DeviceState *dev)
|
|
{
|
|
assert(blk->dev == dev);
|
|
GLOBAL_STATE_CODE();
|
|
blk->dev = NULL;
|
|
blk->dev_ops = NULL;
|
|
blk->dev_opaque = NULL;
|
|
blk_set_perm(blk, 0, BLK_PERM_ALL, &error_abort);
|
|
blk_unref(blk);
|
|
}
|
|
|
|
/*
|
|
* Return the device model attached to @blk if any, else null.
|
|
*/
|
|
DeviceState *blk_get_attached_dev(BlockBackend *blk)
|
|
{
|
|
GLOBAL_STATE_CODE();
|
|
return blk->dev;
|
|
}
|
|
|
|
/* Return the qdev ID, or if no ID is assigned the QOM path, of the block
|
|
* device attached to the BlockBackend. */
|
|
char *blk_get_attached_dev_id(BlockBackend *blk)
|
|
{
|
|
DeviceState *dev = blk->dev;
|
|
IO_CODE();
|
|
|
|
if (!dev) {
|
|
return g_strdup("");
|
|
} else if (dev->id) {
|
|
return g_strdup(dev->id);
|
|
}
|
|
|
|
return object_get_canonical_path(OBJECT(dev)) ?: g_strdup("");
|
|
}
|
|
|
|
/*
|
|
* Return the BlockBackend which has the device model @dev attached if it
|
|
* exists, else null.
|
|
*
|
|
* @dev must not be null.
|
|
*/
|
|
BlockBackend *blk_by_dev(void *dev)
|
|
{
|
|
BlockBackend *blk = NULL;
|
|
|
|
GLOBAL_STATE_CODE();
|
|
|
|
assert(dev != NULL);
|
|
while ((blk = blk_all_next(blk)) != NULL) {
|
|
if (blk->dev == dev) {
|
|
return blk;
|
|
}
|
|
}
|
|
return NULL;
|
|
}
|
|
|
|
/*
|
|
* Set @blk's device model callbacks to @ops.
|
|
* @opaque is the opaque argument to pass to the callbacks.
|
|
* This is for use by device models.
|
|
*/
|
|
void blk_set_dev_ops(BlockBackend *blk, const BlockDevOps *ops,
|
|
void *opaque)
|
|
{
|
|
GLOBAL_STATE_CODE();
|
|
blk->dev_ops = ops;
|
|
blk->dev_opaque = opaque;
|
|
|
|
/* Are we currently quiesced? Should we enforce this right now? */
|
|
if (blk->quiesce_counter && ops && ops->drained_begin) {
|
|
ops->drained_begin(opaque);
|
|
}
|
|
}
|
|
|
|
/*
|
|
* Notify @blk's attached device model of media change.
|
|
*
|
|
* If @load is true, notify of media load. This action can fail, meaning that
|
|
* the medium cannot be loaded. @errp is set then.
|
|
*
|
|
* If @load is false, notify of media eject. This can never fail.
|
|
*
|
|
* Also send DEVICE_TRAY_MOVED events as appropriate.
|
|
*/
|
|
void blk_dev_change_media_cb(BlockBackend *blk, bool load, Error **errp)
|
|
{
|
|
GLOBAL_STATE_CODE();
|
|
if (blk->dev_ops && blk->dev_ops->change_media_cb) {
|
|
bool tray_was_open, tray_is_open;
|
|
Error *local_err = NULL;
|
|
|
|
tray_was_open = blk_dev_is_tray_open(blk);
|
|
blk->dev_ops->change_media_cb(blk->dev_opaque, load, &local_err);
|
|
if (local_err) {
|
|
assert(load == true);
|
|
error_propagate(errp, local_err);
|
|
return;
|
|
}
|
|
tray_is_open = blk_dev_is_tray_open(blk);
|
|
|
|
if (tray_was_open != tray_is_open) {
|
|
char *id = blk_get_attached_dev_id(blk);
|
|
qapi_event_send_device_tray_moved(blk_name(blk), id, tray_is_open);
|
|
g_free(id);
|
|
}
|
|
}
|
|
}
|
|
|
|
static void blk_root_change_media(BdrvChild *child, bool load)
|
|
{
|
|
blk_dev_change_media_cb(child->opaque, load, NULL);
|
|
}
|
|
|
|
/*
|
|
* Does @blk's attached device model have removable media?
|
|
* %true if no device model is attached.
|
|
*/
|
|
bool blk_dev_has_removable_media(BlockBackend *blk)
|
|
{
|
|
GLOBAL_STATE_CODE();
|
|
return !blk->dev || (blk->dev_ops && blk->dev_ops->change_media_cb);
|
|
}
|
|
|
|
/*
|
|
* Does @blk's attached device model have a tray?
|
|
*/
|
|
bool blk_dev_has_tray(BlockBackend *blk)
|
|
{
|
|
IO_CODE();
|
|
return blk->dev_ops && blk->dev_ops->is_tray_open;
|
|
}
|
|
|
|
/*
|
|
* Notify @blk's attached device model of a media eject request.
|
|
* If @force is true, the medium is about to be yanked out forcefully.
|
|
*/
|
|
void blk_dev_eject_request(BlockBackend *blk, bool force)
|
|
{
|
|
GLOBAL_STATE_CODE();
|
|
if (blk->dev_ops && blk->dev_ops->eject_request_cb) {
|
|
blk->dev_ops->eject_request_cb(blk->dev_opaque, force);
|
|
}
|
|
}
|
|
|
|
/*
|
|
* Does @blk's attached device model have a tray, and is it open?
|
|
*/
|
|
bool blk_dev_is_tray_open(BlockBackend *blk)
|
|
{
|
|
IO_CODE();
|
|
if (blk_dev_has_tray(blk)) {
|
|
return blk->dev_ops->is_tray_open(blk->dev_opaque);
|
|
}
|
|
return false;
|
|
}
|
|
|
|
/*
|
|
* Does @blk's attached device model have the medium locked?
|
|
* %false if the device model has no such lock.
|
|
*/
|
|
bool blk_dev_is_medium_locked(BlockBackend *blk)
|
|
{
|
|
GLOBAL_STATE_CODE();
|
|
if (blk->dev_ops && blk->dev_ops->is_medium_locked) {
|
|
return blk->dev_ops->is_medium_locked(blk->dev_opaque);
|
|
}
|
|
return false;
|
|
}
|
|
|
|
/*
|
|
* Notify @blk's attached device model of a backend size change.
|
|
*/
|
|
static void blk_root_resize(BdrvChild *child)
|
|
{
|
|
BlockBackend *blk = child->opaque;
|
|
|
|
if (blk->dev_ops && blk->dev_ops->resize_cb) {
|
|
blk->dev_ops->resize_cb(blk->dev_opaque);
|
|
}
|
|
}
|
|
|
|
void blk_iostatus_enable(BlockBackend *blk)
|
|
{
|
|
GLOBAL_STATE_CODE();
|
|
blk->iostatus_enabled = true;
|
|
blk->iostatus = BLOCK_DEVICE_IO_STATUS_OK;
|
|
}
|
|
|
|
/* The I/O status is only enabled if the drive explicitly
|
|
* enables it _and_ the VM is configured to stop on errors */
|
|
bool blk_iostatus_is_enabled(const BlockBackend *blk)
|
|
{
|
|
IO_CODE();
|
|
return (blk->iostatus_enabled &&
|
|
(blk->on_write_error == BLOCKDEV_ON_ERROR_ENOSPC ||
|
|
blk->on_write_error == BLOCKDEV_ON_ERROR_STOP ||
|
|
blk->on_read_error == BLOCKDEV_ON_ERROR_STOP));
|
|
}
|
|
|
|
BlockDeviceIoStatus blk_iostatus(const BlockBackend *blk)
|
|
{
|
|
GLOBAL_STATE_CODE();
|
|
return blk->iostatus;
|
|
}
|
|
|
|
void blk_iostatus_disable(BlockBackend *blk)
|
|
{
|
|
GLOBAL_STATE_CODE();
|
|
blk->iostatus_enabled = false;
|
|
}
|
|
|
|
void blk_iostatus_reset(BlockBackend *blk)
|
|
{
|
|
GLOBAL_STATE_CODE();
|
|
if (blk_iostatus_is_enabled(blk)) {
|
|
blk->iostatus = BLOCK_DEVICE_IO_STATUS_OK;
|
|
}
|
|
}
|
|
|
|
void blk_iostatus_set_err(BlockBackend *blk, int error)
|
|
{
|
|
IO_CODE();
|
|
assert(blk_iostatus_is_enabled(blk));
|
|
if (blk->iostatus == BLOCK_DEVICE_IO_STATUS_OK) {
|
|
blk->iostatus = error == ENOSPC ? BLOCK_DEVICE_IO_STATUS_NOSPACE :
|
|
BLOCK_DEVICE_IO_STATUS_FAILED;
|
|
}
|
|
}
|
|
|
|
void blk_set_allow_write_beyond_eof(BlockBackend *blk, bool allow)
|
|
{
|
|
IO_CODE();
|
|
blk->allow_write_beyond_eof = allow;
|
|
}
|
|
|
|
void blk_set_allow_aio_context_change(BlockBackend *blk, bool allow)
|
|
{
|
|
IO_CODE();
|
|
blk->allow_aio_context_change = allow;
|
|
}
|
|
|
|
void blk_set_disable_request_queuing(BlockBackend *blk, bool disable)
|
|
{
|
|
IO_CODE();
|
|
blk->disable_request_queuing = disable;
|
|
}
|
|
|
|
static int blk_check_byte_request(BlockBackend *blk, int64_t offset,
|
|
int64_t bytes)
|
|
{
|
|
int64_t len;
|
|
|
|
if (bytes < 0) {
|
|
return -EIO;
|
|
}
|
|
|
|
if (!blk_is_available(blk)) {
|
|
return -ENOMEDIUM;
|
|
}
|
|
|
|
if (offset < 0) {
|
|
return -EIO;
|
|
}
|
|
|
|
if (!blk->allow_write_beyond_eof) {
|
|
len = blk_getlength(blk);
|
|
if (len < 0) {
|
|
return len;
|
|
}
|
|
|
|
if (offset > len || len - offset < bytes) {
|
|
return -EIO;
|
|
}
|
|
}
|
|
|
|
return 0;
|
|
}
|
|
|
|
/* To be called between exactly one pair of blk_inc/dec_in_flight() */
|
|
static void coroutine_fn blk_wait_while_drained(BlockBackend *blk)
|
|
{
|
|
assert(blk->in_flight > 0);
|
|
|
|
if (blk->quiesce_counter && !blk->disable_request_queuing) {
|
|
blk_dec_in_flight(blk);
|
|
qemu_co_queue_wait(&blk->queued_requests, NULL);
|
|
blk_inc_in_flight(blk);
|
|
}
|
|
}
|
|
|
|
/* To be called between exactly one pair of blk_inc/dec_in_flight() */
|
|
static int coroutine_fn
|
|
blk_co_do_preadv_part(BlockBackend *blk, int64_t offset, int64_t bytes,
|
|
QEMUIOVector *qiov, size_t qiov_offset,
|
|
BdrvRequestFlags flags)
|
|
{
|
|
int ret;
|
|
BlockDriverState *bs;
|
|
IO_CODE();
|
|
|
|
blk_wait_while_drained(blk);
|
|
|
|
/* Call blk_bs() only after waiting, the graph may have changed */
|
|
bs = blk_bs(blk);
|
|
trace_blk_co_preadv(blk, bs, offset, bytes, flags);
|
|
|
|
ret = blk_check_byte_request(blk, offset, bytes);
|
|
if (ret < 0) {
|
|
return ret;
|
|
}
|
|
|
|
bdrv_inc_in_flight(bs);
|
|
|
|
/* throttling disk I/O */
|
|
if (blk->public.throttle_group_member.throttle_state) {
|
|
throttle_group_co_io_limits_intercept(&blk->public.throttle_group_member,
|
|
bytes, false);
|
|
}
|
|
|
|
ret = bdrv_co_preadv_part(blk->root, offset, bytes, qiov, qiov_offset,
|
|
flags);
|
|
bdrv_dec_in_flight(bs);
|
|
return ret;
|
|
}
|
|
|
|
int coroutine_fn blk_co_pread(BlockBackend *blk, int64_t offset, int64_t bytes,
|
|
void *buf, BdrvRequestFlags flags)
|
|
{
|
|
QEMUIOVector qiov = QEMU_IOVEC_INIT_BUF(qiov, buf, bytes);
|
|
IO_OR_GS_CODE();
|
|
|
|
assert(bytes <= SIZE_MAX);
|
|
|
|
return blk_co_preadv(blk, offset, bytes, &qiov, flags);
|
|
}
|
|
|
|
int coroutine_fn blk_co_preadv(BlockBackend *blk, int64_t offset,
|
|
int64_t bytes, QEMUIOVector *qiov,
|
|
BdrvRequestFlags flags)
|
|
{
|
|
int ret;
|
|
IO_OR_GS_CODE();
|
|
|
|
blk_inc_in_flight(blk);
|
|
ret = blk_co_do_preadv_part(blk, offset, bytes, qiov, 0, flags);
|
|
blk_dec_in_flight(blk);
|
|
|
|
return ret;
|
|
}
|
|
|
|
int coroutine_fn blk_co_preadv_part(BlockBackend *blk, int64_t offset,
|
|
int64_t bytes, QEMUIOVector *qiov,
|
|
size_t qiov_offset, BdrvRequestFlags flags)
|
|
{
|
|
int ret;
|
|
IO_OR_GS_CODE();
|
|
|
|
blk_inc_in_flight(blk);
|
|
ret = blk_co_do_preadv_part(blk, offset, bytes, qiov, qiov_offset, flags);
|
|
blk_dec_in_flight(blk);
|
|
|
|
return ret;
|
|
}
|
|
|
|
/* To be called between exactly one pair of blk_inc/dec_in_flight() */
|
|
static int coroutine_fn
|
|
blk_co_do_pwritev_part(BlockBackend *blk, int64_t offset, int64_t bytes,
|
|
QEMUIOVector *qiov, size_t qiov_offset,
|
|
BdrvRequestFlags flags)
|
|
{
|
|
int ret;
|
|
BlockDriverState *bs;
|
|
IO_CODE();
|
|
|
|
blk_wait_while_drained(blk);
|
|
|
|
/* Call blk_bs() only after waiting, the graph may have changed */
|
|
bs = blk_bs(blk);
|
|
trace_blk_co_pwritev(blk, bs, offset, bytes, flags);
|
|
|
|
ret = blk_check_byte_request(blk, offset, bytes);
|
|
if (ret < 0) {
|
|
return ret;
|
|
}
|
|
|
|
bdrv_inc_in_flight(bs);
|
|
/* throttling disk I/O */
|
|
if (blk->public.throttle_group_member.throttle_state) {
|
|
throttle_group_co_io_limits_intercept(&blk->public.throttle_group_member,
|
|
bytes, true);
|
|
}
|
|
|
|
if (!blk->enable_write_cache) {
|
|
flags |= BDRV_REQ_FUA;
|
|
}
|
|
|
|
ret = bdrv_co_pwritev_part(blk->root, offset, bytes, qiov, qiov_offset,
|
|
flags);
|
|
bdrv_dec_in_flight(bs);
|
|
return ret;
|
|
}
|
|
|
|
int coroutine_fn blk_co_pwritev_part(BlockBackend *blk, int64_t offset,
|
|
int64_t bytes,
|
|
QEMUIOVector *qiov, size_t qiov_offset,
|
|
BdrvRequestFlags flags)
|
|
{
|
|
int ret;
|
|
IO_OR_GS_CODE();
|
|
|
|
blk_inc_in_flight(blk);
|
|
ret = blk_co_do_pwritev_part(blk, offset, bytes, qiov, qiov_offset, flags);
|
|
blk_dec_in_flight(blk);
|
|
|
|
return ret;
|
|
}
|
|
|
|
int coroutine_fn blk_co_pwrite(BlockBackend *blk, int64_t offset, int64_t bytes,
|
|
const void *buf, BdrvRequestFlags flags)
|
|
{
|
|
QEMUIOVector qiov = QEMU_IOVEC_INIT_BUF(qiov, buf, bytes);
|
|
IO_OR_GS_CODE();
|
|
|
|
assert(bytes <= SIZE_MAX);
|
|
|
|
return blk_co_pwritev(blk, offset, bytes, &qiov, flags);
|
|
}
|
|
|
|
int coroutine_fn blk_co_pwritev(BlockBackend *blk, int64_t offset,
|
|
int64_t bytes, QEMUIOVector *qiov,
|
|
BdrvRequestFlags flags)
|
|
{
|
|
IO_OR_GS_CODE();
|
|
return blk_co_pwritev_part(blk, offset, bytes, qiov, 0, flags);
|
|
}
|
|
|
|
typedef struct BlkRwCo {
|
|
BlockBackend *blk;
|
|
int64_t offset;
|
|
void *iobuf;
|
|
int ret;
|
|
BdrvRequestFlags flags;
|
|
} BlkRwCo;
|
|
|
|
int blk_make_zero(BlockBackend *blk, BdrvRequestFlags flags)
|
|
{
|
|
GLOBAL_STATE_CODE();
|
|
return bdrv_make_zero(blk->root, flags);
|
|
}
|
|
|
|
void blk_inc_in_flight(BlockBackend *blk)
|
|
{
|
|
IO_CODE();
|
|
qatomic_inc(&blk->in_flight);
|
|
}
|
|
|
|
void blk_dec_in_flight(BlockBackend *blk)
|
|
{
|
|
IO_CODE();
|
|
qatomic_dec(&blk->in_flight);
|
|
aio_wait_kick();
|
|
}
|
|
|
|
static void error_callback_bh(void *opaque)
|
|
{
|
|
struct BlockBackendAIOCB *acb = opaque;
|
|
|
|
blk_dec_in_flight(acb->blk);
|
|
acb->common.cb(acb->common.opaque, acb->ret);
|
|
qemu_aio_unref(acb);
|
|
}
|
|
|
|
BlockAIOCB *blk_abort_aio_request(BlockBackend *blk,
|
|
BlockCompletionFunc *cb,
|
|
void *opaque, int ret)
|
|
{
|
|
struct BlockBackendAIOCB *acb;
|
|
IO_CODE();
|
|
|
|
blk_inc_in_flight(blk);
|
|
acb = blk_aio_get(&block_backend_aiocb_info, blk, cb, opaque);
|
|
acb->blk = blk;
|
|
acb->ret = ret;
|
|
|
|
replay_bh_schedule_oneshot_event(blk_get_aio_context(blk),
|
|
error_callback_bh, acb);
|
|
return &acb->common;
|
|
}
|
|
|
|
typedef struct BlkAioEmAIOCB {
|
|
BlockAIOCB common;
|
|
BlkRwCo rwco;
|
|
int64_t bytes;
|
|
bool has_returned;
|
|
} BlkAioEmAIOCB;
|
|
|
|
static AioContext *blk_aio_em_aiocb_get_aio_context(BlockAIOCB *acb_)
|
|
{
|
|
BlkAioEmAIOCB *acb = container_of(acb_, BlkAioEmAIOCB, common);
|
|
|
|
return blk_get_aio_context(acb->rwco.blk);
|
|
}
|
|
|
|
static const AIOCBInfo blk_aio_em_aiocb_info = {
|
|
.aiocb_size = sizeof(BlkAioEmAIOCB),
|
|
.get_aio_context = blk_aio_em_aiocb_get_aio_context,
|
|
};
|
|
|
|
static void blk_aio_complete(BlkAioEmAIOCB *acb)
|
|
{
|
|
if (acb->has_returned) {
|
|
acb->common.cb(acb->common.opaque, acb->rwco.ret);
|
|
blk_dec_in_flight(acb->rwco.blk);
|
|
qemu_aio_unref(acb);
|
|
}
|
|
}
|
|
|
|
static void blk_aio_complete_bh(void *opaque)
|
|
{
|
|
BlkAioEmAIOCB *acb = opaque;
|
|
assert(acb->has_returned);
|
|
blk_aio_complete(acb);
|
|
}
|
|
|
|
static BlockAIOCB *blk_aio_prwv(BlockBackend *blk, int64_t offset,
|
|
int64_t bytes,
|
|
void *iobuf, CoroutineEntry co_entry,
|
|
BdrvRequestFlags flags,
|
|
BlockCompletionFunc *cb, void *opaque)
|
|
{
|
|
BlkAioEmAIOCB *acb;
|
|
Coroutine *co;
|
|
|
|
blk_inc_in_flight(blk);
|
|
acb = blk_aio_get(&blk_aio_em_aiocb_info, blk, cb, opaque);
|
|
acb->rwco = (BlkRwCo) {
|
|
.blk = blk,
|
|
.offset = offset,
|
|
.iobuf = iobuf,
|
|
.flags = flags,
|
|
.ret = NOT_DONE,
|
|
};
|
|
acb->bytes = bytes;
|
|
acb->has_returned = false;
|
|
|
|
co = qemu_coroutine_create(co_entry, acb);
|
|
bdrv_coroutine_enter(blk_bs(blk), co);
|
|
|
|
acb->has_returned = true;
|
|
if (acb->rwco.ret != NOT_DONE) {
|
|
replay_bh_schedule_oneshot_event(blk_get_aio_context(blk),
|
|
blk_aio_complete_bh, acb);
|
|
}
|
|
|
|
return &acb->common;
|
|
}
|
|
|
|
static void coroutine_fn blk_aio_read_entry(void *opaque)
|
|
{
|
|
BlkAioEmAIOCB *acb = opaque;
|
|
BlkRwCo *rwco = &acb->rwco;
|
|
QEMUIOVector *qiov = rwco->iobuf;
|
|
|
|
assert(qiov->size == acb->bytes);
|
|
rwco->ret = blk_co_do_preadv_part(rwco->blk, rwco->offset, acb->bytes, qiov,
|
|
0, rwco->flags);
|
|
blk_aio_complete(acb);
|
|
}
|
|
|
|
static void coroutine_fn blk_aio_write_entry(void *opaque)
|
|
{
|
|
BlkAioEmAIOCB *acb = opaque;
|
|
BlkRwCo *rwco = &acb->rwco;
|
|
QEMUIOVector *qiov = rwco->iobuf;
|
|
|
|
assert(!qiov || qiov->size == acb->bytes);
|
|
rwco->ret = blk_co_do_pwritev_part(rwco->blk, rwco->offset, acb->bytes,
|
|
qiov, 0, rwco->flags);
|
|
blk_aio_complete(acb);
|
|
}
|
|
|
|
BlockAIOCB *blk_aio_pwrite_zeroes(BlockBackend *blk, int64_t offset,
|
|
int64_t bytes, BdrvRequestFlags flags,
|
|
BlockCompletionFunc *cb, void *opaque)
|
|
{
|
|
IO_CODE();
|
|
return blk_aio_prwv(blk, offset, bytes, NULL, blk_aio_write_entry,
|
|
flags | BDRV_REQ_ZERO_WRITE, cb, opaque);
|
|
}
|
|
|
|
int64_t blk_getlength(BlockBackend *blk)
|
|
{
|
|
IO_CODE();
|
|
if (!blk_is_available(blk)) {
|
|
return -ENOMEDIUM;
|
|
}
|
|
|
|
return bdrv_getlength(blk_bs(blk));
|
|
}
|
|
|
|
void blk_get_geometry(BlockBackend *blk, uint64_t *nb_sectors_ptr)
|
|
{
|
|
IO_CODE();
|
|
if (!blk_bs(blk)) {
|
|
*nb_sectors_ptr = 0;
|
|
} else {
|
|
bdrv_get_geometry(blk_bs(blk), nb_sectors_ptr);
|
|
}
|
|
}
|
|
|
|
int64_t blk_nb_sectors(BlockBackend *blk)
|
|
{
|
|
IO_CODE();
|
|
if (!blk_is_available(blk)) {
|
|
return -ENOMEDIUM;
|
|
}
|
|
|
|
return bdrv_nb_sectors(blk_bs(blk));
|
|
}
|
|
|
|
BlockAIOCB *blk_aio_preadv(BlockBackend *blk, int64_t offset,
|
|
QEMUIOVector *qiov, BdrvRequestFlags flags,
|
|
BlockCompletionFunc *cb, void *opaque)
|
|
{
|
|
IO_CODE();
|
|
assert((uint64_t)qiov->size <= INT64_MAX);
|
|
return blk_aio_prwv(blk, offset, qiov->size, qiov,
|
|
blk_aio_read_entry, flags, cb, opaque);
|
|
}
|
|
|
|
BlockAIOCB *blk_aio_pwritev(BlockBackend *blk, int64_t offset,
|
|
QEMUIOVector *qiov, BdrvRequestFlags flags,
|
|
BlockCompletionFunc *cb, void *opaque)
|
|
{
|
|
IO_CODE();
|
|
assert((uint64_t)qiov->size <= INT64_MAX);
|
|
return blk_aio_prwv(blk, offset, qiov->size, qiov,
|
|
blk_aio_write_entry, flags, cb, opaque);
|
|
}
|
|
|
|
void blk_aio_cancel(BlockAIOCB *acb)
|
|
{
|
|
GLOBAL_STATE_CODE();
|
|
bdrv_aio_cancel(acb);
|
|
}
|
|
|
|
void blk_aio_cancel_async(BlockAIOCB *acb)
|
|
{
|
|
IO_CODE();
|
|
bdrv_aio_cancel_async(acb);
|
|
}
|
|
|
|
/* To be called between exactly one pair of blk_inc/dec_in_flight() */
|
|
static int coroutine_fn
|
|
blk_co_do_ioctl(BlockBackend *blk, unsigned long int req, void *buf)
|
|
{
|
|
IO_CODE();
|
|
|
|
blk_wait_while_drained(blk);
|
|
|
|
if (!blk_is_available(blk)) {
|
|
return -ENOMEDIUM;
|
|
}
|
|
|
|
return bdrv_co_ioctl(blk_bs(blk), req, buf);
|
|
}
|
|
|
|
int coroutine_fn blk_co_ioctl(BlockBackend *blk, unsigned long int req,
|
|
void *buf)
|
|
{
|
|
int ret;
|
|
IO_OR_GS_CODE();
|
|
|
|
blk_inc_in_flight(blk);
|
|
ret = blk_co_do_ioctl(blk, req, buf);
|
|
blk_dec_in_flight(blk);
|
|
|
|
return ret;
|
|
}
|
|
|
|
static void coroutine_fn blk_aio_ioctl_entry(void *opaque)
|
|
{
|
|
BlkAioEmAIOCB *acb = opaque;
|
|
BlkRwCo *rwco = &acb->rwco;
|
|
|
|
rwco->ret = blk_co_do_ioctl(rwco->blk, rwco->offset, rwco->iobuf);
|
|
|
|
blk_aio_complete(acb);
|
|
}
|
|
|
|
BlockAIOCB *blk_aio_ioctl(BlockBackend *blk, unsigned long int req, void *buf,
|
|
BlockCompletionFunc *cb, void *opaque)
|
|
{
|
|
IO_CODE();
|
|
return blk_aio_prwv(blk, req, 0, buf, blk_aio_ioctl_entry, 0, cb, opaque);
|
|
}
|
|
|
|
/* To be called between exactly one pair of blk_inc/dec_in_flight() */
|
|
static int coroutine_fn
|
|
blk_co_do_pdiscard(BlockBackend *blk, int64_t offset, int64_t bytes)
|
|
{
|
|
int ret;
|
|
IO_CODE();
|
|
|
|
blk_wait_while_drained(blk);
|
|
|
|
ret = blk_check_byte_request(blk, offset, bytes);
|
|
if (ret < 0) {
|
|
return ret;
|
|
}
|
|
|
|
return bdrv_co_pdiscard(blk->root, offset, bytes);
|
|
}
|
|
|
|
static void coroutine_fn blk_aio_pdiscard_entry(void *opaque)
|
|
{
|
|
BlkAioEmAIOCB *acb = opaque;
|
|
BlkRwCo *rwco = &acb->rwco;
|
|
|
|
rwco->ret = blk_co_do_pdiscard(rwco->blk, rwco->offset, acb->bytes);
|
|
blk_aio_complete(acb);
|
|
}
|
|
|
|
BlockAIOCB *blk_aio_pdiscard(BlockBackend *blk,
|
|
int64_t offset, int64_t bytes,
|
|
BlockCompletionFunc *cb, void *opaque)
|
|
{
|
|
IO_CODE();
|
|
return blk_aio_prwv(blk, offset, bytes, NULL, blk_aio_pdiscard_entry, 0,
|
|
cb, opaque);
|
|
}
|
|
|
|
int coroutine_fn blk_co_pdiscard(BlockBackend *blk, int64_t offset,
|
|
int64_t bytes)
|
|
{
|
|
int ret;
|
|
IO_OR_GS_CODE();
|
|
|
|
blk_inc_in_flight(blk);
|
|
ret = blk_co_do_pdiscard(blk, offset, bytes);
|
|
blk_dec_in_flight(blk);
|
|
|
|
return ret;
|
|
}
|
|
|
|
/* To be called between exactly one pair of blk_inc/dec_in_flight() */
|
|
static int coroutine_fn blk_co_do_flush(BlockBackend *blk)
|
|
{
|
|
blk_wait_while_drained(blk);
|
|
IO_CODE();
|
|
|
|
if (!blk_is_available(blk)) {
|
|
return -ENOMEDIUM;
|
|
}
|
|
|
|
return bdrv_co_flush(blk_bs(blk));
|
|
}
|
|
|
|
static void coroutine_fn blk_aio_flush_entry(void *opaque)
|
|
{
|
|
BlkAioEmAIOCB *acb = opaque;
|
|
BlkRwCo *rwco = &acb->rwco;
|
|
|
|
rwco->ret = blk_co_do_flush(rwco->blk);
|
|
blk_aio_complete(acb);
|
|
}
|
|
|
|
BlockAIOCB *blk_aio_flush(BlockBackend *blk,
|
|
BlockCompletionFunc *cb, void *opaque)
|
|
{
|
|
IO_CODE();
|
|
return blk_aio_prwv(blk, 0, 0, NULL, blk_aio_flush_entry, 0, cb, opaque);
|
|
}
|
|
|
|
int coroutine_fn blk_co_flush(BlockBackend *blk)
|
|
{
|
|
int ret;
|
|
IO_OR_GS_CODE();
|
|
|
|
blk_inc_in_flight(blk);
|
|
ret = blk_co_do_flush(blk);
|
|
blk_dec_in_flight(blk);
|
|
|
|
return ret;
|
|
}
|
|
|
|
void blk_drain(BlockBackend *blk)
|
|
{
|
|
BlockDriverState *bs = blk_bs(blk);
|
|
GLOBAL_STATE_CODE();
|
|
|
|
if (bs) {
|
|
bdrv_ref(bs);
|
|
bdrv_drained_begin(bs);
|
|
}
|
|
|
|
/* We may have -ENOMEDIUM completions in flight */
|
|
AIO_WAIT_WHILE(blk_get_aio_context(blk),
|
|
qatomic_mb_read(&blk->in_flight) > 0);
|
|
|
|
if (bs) {
|
|
bdrv_drained_end(bs);
|
|
bdrv_unref(bs);
|
|
}
|
|
}
|
|
|
|
void blk_drain_all(void)
|
|
{
|
|
BlockBackend *blk = NULL;
|
|
|
|
GLOBAL_STATE_CODE();
|
|
|
|
bdrv_drain_all_begin();
|
|
|
|
while ((blk = blk_all_next(blk)) != NULL) {
|
|
AioContext *ctx = blk_get_aio_context(blk);
|
|
|
|
aio_context_acquire(ctx);
|
|
|
|
/* We may have -ENOMEDIUM completions in flight */
|
|
AIO_WAIT_WHILE(ctx, qatomic_mb_read(&blk->in_flight) > 0);
|
|
|
|
aio_context_release(ctx);
|
|
}
|
|
|
|
bdrv_drain_all_end();
|
|
}
|
|
|
|
void blk_set_on_error(BlockBackend *blk, BlockdevOnError on_read_error,
|
|
BlockdevOnError on_write_error)
|
|
{
|
|
GLOBAL_STATE_CODE();
|
|
blk->on_read_error = on_read_error;
|
|
blk->on_write_error = on_write_error;
|
|
}
|
|
|
|
BlockdevOnError blk_get_on_error(BlockBackend *blk, bool is_read)
|
|
{
|
|
IO_CODE();
|
|
return is_read ? blk->on_read_error : blk->on_write_error;
|
|
}
|
|
|
|
BlockErrorAction blk_get_error_action(BlockBackend *blk, bool is_read,
|
|
int error)
|
|
{
|
|
BlockdevOnError on_err = blk_get_on_error(blk, is_read);
|
|
IO_CODE();
|
|
|
|
switch (on_err) {
|
|
case BLOCKDEV_ON_ERROR_ENOSPC:
|
|
return (error == ENOSPC) ?
|
|
BLOCK_ERROR_ACTION_STOP : BLOCK_ERROR_ACTION_REPORT;
|
|
case BLOCKDEV_ON_ERROR_STOP:
|
|
return BLOCK_ERROR_ACTION_STOP;
|
|
case BLOCKDEV_ON_ERROR_REPORT:
|
|
return BLOCK_ERROR_ACTION_REPORT;
|
|
case BLOCKDEV_ON_ERROR_IGNORE:
|
|
return BLOCK_ERROR_ACTION_IGNORE;
|
|
case BLOCKDEV_ON_ERROR_AUTO:
|
|
default:
|
|
abort();
|
|
}
|
|
}
|
|
|
|
static void send_qmp_error_event(BlockBackend *blk,
|
|
BlockErrorAction action,
|
|
bool is_read, int error)
|
|
{
|
|
IoOperationType optype;
|
|
BlockDriverState *bs = blk_bs(blk);
|
|
|
|
optype = is_read ? IO_OPERATION_TYPE_READ : IO_OPERATION_TYPE_WRITE;
|
|
qapi_event_send_block_io_error(blk_name(blk), !!bs,
|
|
bs ? bdrv_get_node_name(bs) : NULL, optype,
|
|
action, blk_iostatus_is_enabled(blk),
|
|
error == ENOSPC, strerror(error));
|
|
}
|
|
|
|
/* This is done by device models because, while the block layer knows
|
|
* about the error, it does not know whether an operation comes from
|
|
* the device or the block layer (from a job, for example).
|
|
*/
|
|
void blk_error_action(BlockBackend *blk, BlockErrorAction action,
|
|
bool is_read, int error)
|
|
{
|
|
assert(error >= 0);
|
|
IO_CODE();
|
|
|
|
if (action == BLOCK_ERROR_ACTION_STOP) {
|
|
/* First set the iostatus, so that "info block" returns an iostatus
|
|
* that matches the events raised so far (an additional error iostatus
|
|
* is fine, but not a lost one).
|
|
*/
|
|
blk_iostatus_set_err(blk, error);
|
|
|
|
/* Then raise the request to stop the VM and the event.
|
|
* qemu_system_vmstop_request_prepare has two effects. First,
|
|
* it ensures that the STOP event always comes after the
|
|
* BLOCK_IO_ERROR event. Second, it ensures that even if management
|
|
* can observe the STOP event and do a "cont" before the STOP
|
|
* event is issued, the VM will not stop. In this case, vm_start()
|
|
* also ensures that the STOP/RESUME pair of events is emitted.
|
|
*/
|
|
qemu_system_vmstop_request_prepare();
|
|
send_qmp_error_event(blk, action, is_read, error);
|
|
qemu_system_vmstop_request(RUN_STATE_IO_ERROR);
|
|
} else {
|
|
send_qmp_error_event(blk, action, is_read, error);
|
|
}
|
|
}
|
|
|
|
/*
|
|
* Returns true if the BlockBackend can support taking write permissions
|
|
* (because its root node is not read-only).
|
|
*/
|
|
bool blk_supports_write_perm(BlockBackend *blk)
|
|
{
|
|
BlockDriverState *bs = blk_bs(blk);
|
|
GLOBAL_STATE_CODE();
|
|
|
|
if (bs) {
|
|
return !bdrv_is_read_only(bs);
|
|
} else {
|
|
return blk->root_state.open_flags & BDRV_O_RDWR;
|
|
}
|
|
}
|
|
|
|
/*
|
|
* Returns true if the BlockBackend can be written to in its current
|
|
* configuration (i.e. if write permission have been requested)
|
|
*/
|
|
bool blk_is_writable(BlockBackend *blk)
|
|
{
|
|
IO_CODE();
|
|
return blk->perm & BLK_PERM_WRITE;
|
|
}
|
|
|
|
bool blk_is_sg(BlockBackend *blk)
|
|
{
|
|
BlockDriverState *bs = blk_bs(blk);
|
|
GLOBAL_STATE_CODE();
|
|
|
|
if (!bs) {
|
|
return false;
|
|
}
|
|
|
|
return bdrv_is_sg(bs);
|
|
}
|
|
|
|
bool blk_enable_write_cache(BlockBackend *blk)
|
|
{
|
|
IO_CODE();
|
|
return blk->enable_write_cache;
|
|
}
|
|
|
|
void blk_set_enable_write_cache(BlockBackend *blk, bool wce)
|
|
{
|
|
IO_CODE();
|
|
blk->enable_write_cache = wce;
|
|
}
|
|
|
|
void blk_activate(BlockBackend *blk, Error **errp)
|
|
{
|
|
BlockDriverState *bs = blk_bs(blk);
|
|
GLOBAL_STATE_CODE();
|
|
|
|
if (!bs) {
|
|
error_setg(errp, "Device '%s' has no medium", blk->name);
|
|
return;
|
|
}
|
|
|
|
bdrv_activate(bs, errp);
|
|
}
|
|
|
|
bool blk_is_inserted(BlockBackend *blk)
|
|
{
|
|
BlockDriverState *bs = blk_bs(blk);
|
|
IO_CODE();
|
|
|
|
return bs && bdrv_is_inserted(bs);
|
|
}
|
|
|
|
bool blk_is_available(BlockBackend *blk)
|
|
{
|
|
IO_CODE();
|
|
return blk_is_inserted(blk) && !blk_dev_is_tray_open(blk);
|
|
}
|
|
|
|
void blk_lock_medium(BlockBackend *blk, bool locked)
|
|
{
|
|
BlockDriverState *bs = blk_bs(blk);
|
|
IO_CODE();
|
|
|
|
if (bs) {
|
|
bdrv_lock_medium(bs, locked);
|
|
}
|
|
}
|
|
|
|
void blk_eject(BlockBackend *blk, bool eject_flag)
|
|
{
|
|
BlockDriverState *bs = blk_bs(blk);
|
|
char *id;
|
|
IO_CODE();
|
|
|
|
if (bs) {
|
|
bdrv_eject(bs, eject_flag);
|
|
}
|
|
|
|
/* Whether or not we ejected on the backend,
|
|
* the frontend experienced a tray event. */
|
|
id = blk_get_attached_dev_id(blk);
|
|
qapi_event_send_device_tray_moved(blk_name(blk), id,
|
|
eject_flag);
|
|
g_free(id);
|
|
}
|
|
|
|
int blk_get_flags(BlockBackend *blk)
|
|
{
|
|
BlockDriverState *bs = blk_bs(blk);
|
|
GLOBAL_STATE_CODE();
|
|
|
|
if (bs) {
|
|
return bdrv_get_flags(bs);
|
|
} else {
|
|
return blk->root_state.open_flags;
|
|
}
|
|
}
|
|
|
|
/* Returns the minimum request alignment, in bytes; guaranteed nonzero */
|
|
uint32_t blk_get_request_alignment(BlockBackend *blk)
|
|
{
|
|
BlockDriverState *bs = blk_bs(blk);
|
|
IO_CODE();
|
|
return bs ? bs->bl.request_alignment : BDRV_SECTOR_SIZE;
|
|
}
|
|
|
|
/* Returns the maximum hardware transfer length, in bytes; guaranteed nonzero */
|
|
uint64_t blk_get_max_hw_transfer(BlockBackend *blk)
|
|
{
|
|
BlockDriverState *bs = blk_bs(blk);
|
|
uint64_t max = INT_MAX;
|
|
IO_CODE();
|
|
|
|
if (bs) {
|
|
max = MIN_NON_ZERO(max, bs->bl.max_hw_transfer);
|
|
max = MIN_NON_ZERO(max, bs->bl.max_transfer);
|
|
}
|
|
return ROUND_DOWN(max, blk_get_request_alignment(blk));
|
|
}
|
|
|
|
/* Returns the maximum transfer length, in bytes; guaranteed nonzero */
|
|
uint32_t blk_get_max_transfer(BlockBackend *blk)
|
|
{
|
|
BlockDriverState *bs = blk_bs(blk);
|
|
uint32_t max = INT_MAX;
|
|
IO_CODE();
|
|
|
|
if (bs) {
|
|
max = MIN_NON_ZERO(max, bs->bl.max_transfer);
|
|
}
|
|
return ROUND_DOWN(max, blk_get_request_alignment(blk));
|
|
}
|
|
|
|
int blk_get_max_hw_iov(BlockBackend *blk)
|
|
{
|
|
IO_CODE();
|
|
return MIN_NON_ZERO(blk->root->bs->bl.max_hw_iov,
|
|
blk->root->bs->bl.max_iov);
|
|
}
|
|
|
|
int blk_get_max_iov(BlockBackend *blk)
|
|
{
|
|
IO_CODE();
|
|
return blk->root->bs->bl.max_iov;
|
|
}
|
|
|
|
void *blk_try_blockalign(BlockBackend *blk, size_t size)
|
|
{
|
|
IO_CODE();
|
|
return qemu_try_blockalign(blk ? blk_bs(blk) : NULL, size);
|
|
}
|
|
|
|
void *blk_blockalign(BlockBackend *blk, size_t size)
|
|
{
|
|
IO_CODE();
|
|
return qemu_blockalign(blk ? blk_bs(blk) : NULL, size);
|
|
}
|
|
|
|
bool blk_op_is_blocked(BlockBackend *blk, BlockOpType op, Error **errp)
|
|
{
|
|
BlockDriverState *bs = blk_bs(blk);
|
|
GLOBAL_STATE_CODE();
|
|
|
|
if (!bs) {
|
|
return false;
|
|
}
|
|
|
|
return bdrv_op_is_blocked(bs, op, errp);
|
|
}
|
|
|
|
void blk_op_unblock(BlockBackend *blk, BlockOpType op, Error *reason)
|
|
{
|
|
BlockDriverState *bs = blk_bs(blk);
|
|
GLOBAL_STATE_CODE();
|
|
|
|
if (bs) {
|
|
bdrv_op_unblock(bs, op, reason);
|
|
}
|
|
}
|
|
|
|
void blk_op_block_all(BlockBackend *blk, Error *reason)
|
|
{
|
|
BlockDriverState *bs = blk_bs(blk);
|
|
GLOBAL_STATE_CODE();
|
|
|
|
if (bs) {
|
|
bdrv_op_block_all(bs, reason);
|
|
}
|
|
}
|
|
|
|
void blk_op_unblock_all(BlockBackend *blk, Error *reason)
|
|
{
|
|
BlockDriverState *bs = blk_bs(blk);
|
|
GLOBAL_STATE_CODE();
|
|
|
|
if (bs) {
|
|
bdrv_op_unblock_all(bs, reason);
|
|
}
|
|
}
|
|
|
|
AioContext *blk_get_aio_context(BlockBackend *blk)
|
|
{
|
|
BlockDriverState *bs = blk_bs(blk);
|
|
IO_CODE();
|
|
|
|
if (bs) {
|
|
AioContext *ctx = bdrv_get_aio_context(blk_bs(blk));
|
|
assert(ctx == blk->ctx);
|
|
}
|
|
|
|
return blk->ctx;
|
|
}
|
|
|
|
static AioContext *blk_aiocb_get_aio_context(BlockAIOCB *acb)
|
|
{
|
|
BlockBackendAIOCB *blk_acb = DO_UPCAST(BlockBackendAIOCB, common, acb);
|
|
return blk_get_aio_context(blk_acb->blk);
|
|
}
|
|
|
|
static int blk_do_set_aio_context(BlockBackend *blk, AioContext *new_context,
|
|
bool update_root_node, Error **errp)
|
|
{
|
|
BlockDriverState *bs = blk_bs(blk);
|
|
ThrottleGroupMember *tgm = &blk->public.throttle_group_member;
|
|
int ret;
|
|
|
|
if (bs) {
|
|
bdrv_ref(bs);
|
|
|
|
if (update_root_node) {
|
|
/*
|
|
* update_root_node MUST be false for blk_root_set_aio_ctx_commit(),
|
|
* as we are already in the commit function of a transaction.
|
|
*/
|
|
ret = bdrv_try_change_aio_context(bs, new_context, blk->root, errp);
|
|
if (ret < 0) {
|
|
bdrv_unref(bs);
|
|
return ret;
|
|
}
|
|
}
|
|
/*
|
|
* Make blk->ctx consistent with the root node before we invoke any
|
|
* other operations like drain that might inquire blk->ctx
|
|
*/
|
|
blk->ctx = new_context;
|
|
if (tgm->throttle_state) {
|
|
bdrv_drained_begin(bs);
|
|
throttle_group_detach_aio_context(tgm);
|
|
throttle_group_attach_aio_context(tgm, new_context);
|
|
bdrv_drained_end(bs);
|
|
}
|
|
|
|
bdrv_unref(bs);
|
|
} else {
|
|
blk->ctx = new_context;
|
|
}
|
|
|
|
return 0;
|
|
}
|
|
|
|
int blk_set_aio_context(BlockBackend *blk, AioContext *new_context,
|
|
Error **errp)
|
|
{
|
|
GLOBAL_STATE_CODE();
|
|
return blk_do_set_aio_context(blk, new_context, true, errp);
|
|
}
|
|
|
|
typedef struct BdrvStateBlkRootContext {
|
|
AioContext *new_ctx;
|
|
BlockBackend *blk;
|
|
} BdrvStateBlkRootContext;
|
|
|
|
static void blk_root_set_aio_ctx_commit(void *opaque)
|
|
{
|
|
BdrvStateBlkRootContext *s = opaque;
|
|
BlockBackend *blk = s->blk;
|
|
|
|
blk_do_set_aio_context(blk, s->new_ctx, false, &error_abort);
|
|
}
|
|
|
|
static TransactionActionDrv set_blk_root_context = {
|
|
.commit = blk_root_set_aio_ctx_commit,
|
|
.clean = g_free,
|
|
};
|
|
|
|
static bool blk_root_change_aio_ctx(BdrvChild *child, AioContext *ctx,
|
|
GHashTable *visited, Transaction *tran,
|
|
Error **errp)
|
|
{
|
|
BlockBackend *blk = child->opaque;
|
|
BdrvStateBlkRootContext *s;
|
|
|
|
if (!blk->allow_aio_context_change) {
|
|
/*
|
|
* Manually created BlockBackends (those with a name) that are not
|
|
* attached to anything can change their AioContext without updating
|
|
* their user; return an error for others.
|
|
*/
|
|
if (!blk->name || blk->dev) {
|
|
/* TODO Add BB name/QOM path */
|
|
error_setg(errp, "Cannot change iothread of active block backend");
|
|
return false;
|
|
}
|
|
}
|
|
|
|
s = g_new(BdrvStateBlkRootContext, 1);
|
|
*s = (BdrvStateBlkRootContext) {
|
|
.new_ctx = ctx,
|
|
.blk = blk,
|
|
};
|
|
|
|
tran_add(tran, &set_blk_root_context, s);
|
|
return true;
|
|
}
|
|
|
|
void blk_add_aio_context_notifier(BlockBackend *blk,
|
|
void (*attached_aio_context)(AioContext *new_context, void *opaque),
|
|
void (*detach_aio_context)(void *opaque), void *opaque)
|
|
{
|
|
BlockBackendAioNotifier *notifier;
|
|
BlockDriverState *bs = blk_bs(blk);
|
|
GLOBAL_STATE_CODE();
|
|
|
|
notifier = g_new(BlockBackendAioNotifier, 1);
|
|
notifier->attached_aio_context = attached_aio_context;
|
|
notifier->detach_aio_context = detach_aio_context;
|
|
notifier->opaque = opaque;
|
|
QLIST_INSERT_HEAD(&blk->aio_notifiers, notifier, list);
|
|
|
|
if (bs) {
|
|
bdrv_add_aio_context_notifier(bs, attached_aio_context,
|
|
detach_aio_context, opaque);
|
|
}
|
|
}
|
|
|
|
void blk_remove_aio_context_notifier(BlockBackend *blk,
|
|
void (*attached_aio_context)(AioContext *,
|
|
void *),
|
|
void (*detach_aio_context)(void *),
|
|
void *opaque)
|
|
{
|
|
BlockBackendAioNotifier *notifier;
|
|
BlockDriverState *bs = blk_bs(blk);
|
|
|
|
GLOBAL_STATE_CODE();
|
|
|
|
if (bs) {
|
|
bdrv_remove_aio_context_notifier(bs, attached_aio_context,
|
|
detach_aio_context, opaque);
|
|
}
|
|
|
|
QLIST_FOREACH(notifier, &blk->aio_notifiers, list) {
|
|
if (notifier->attached_aio_context == attached_aio_context &&
|
|
notifier->detach_aio_context == detach_aio_context &&
|
|
notifier->opaque == opaque) {
|
|
QLIST_REMOVE(notifier, list);
|
|
g_free(notifier);
|
|
return;
|
|
}
|
|
}
|
|
|
|
abort();
|
|
}
|
|
|
|
void blk_add_remove_bs_notifier(BlockBackend *blk, Notifier *notify)
|
|
{
|
|
GLOBAL_STATE_CODE();
|
|
notifier_list_add(&blk->remove_bs_notifiers, notify);
|
|
}
|
|
|
|
void blk_add_insert_bs_notifier(BlockBackend *blk, Notifier *notify)
|
|
{
|
|
GLOBAL_STATE_CODE();
|
|
notifier_list_add(&blk->insert_bs_notifiers, notify);
|
|
}
|
|
|
|
void blk_io_plug(BlockBackend *blk)
|
|
{
|
|
BlockDriverState *bs = blk_bs(blk);
|
|
IO_CODE();
|
|
|
|
if (bs) {
|
|
bdrv_io_plug(bs);
|
|
}
|
|
}
|
|
|
|
void blk_io_unplug(BlockBackend *blk)
|
|
{
|
|
BlockDriverState *bs = blk_bs(blk);
|
|
IO_CODE();
|
|
|
|
if (bs) {
|
|
bdrv_io_unplug(bs);
|
|
}
|
|
}
|
|
|
|
BlockAcctStats *blk_get_stats(BlockBackend *blk)
|
|
{
|
|
IO_CODE();
|
|
return &blk->stats;
|
|
}
|
|
|
|
void *blk_aio_get(const AIOCBInfo *aiocb_info, BlockBackend *blk,
|
|
BlockCompletionFunc *cb, void *opaque)
|
|
{
|
|
IO_CODE();
|
|
return qemu_aio_get(aiocb_info, blk_bs(blk), cb, opaque);
|
|
}
|
|
|
|
int coroutine_fn blk_co_pwrite_zeroes(BlockBackend *blk, int64_t offset,
|
|
int64_t bytes, BdrvRequestFlags flags)
|
|
{
|
|
IO_OR_GS_CODE();
|
|
return blk_co_pwritev(blk, offset, bytes, NULL,
|
|
flags | BDRV_REQ_ZERO_WRITE);
|
|
}
|
|
|
|
int coroutine_fn blk_co_pwrite_compressed(BlockBackend *blk, int64_t offset,
|
|
int64_t bytes, const void *buf)
|
|
{
|
|
QEMUIOVector qiov = QEMU_IOVEC_INIT_BUF(qiov, buf, bytes);
|
|
IO_OR_GS_CODE();
|
|
return blk_co_pwritev_part(blk, offset, bytes, &qiov, 0,
|
|
BDRV_REQ_WRITE_COMPRESSED);
|
|
}
|
|
|
|
int coroutine_fn blk_co_truncate(BlockBackend *blk, int64_t offset, bool exact,
|
|
PreallocMode prealloc, BdrvRequestFlags flags,
|
|
Error **errp)
|
|
{
|
|
IO_OR_GS_CODE();
|
|
if (!blk_is_available(blk)) {
|
|
error_setg(errp, "No medium inserted");
|
|
return -ENOMEDIUM;
|
|
}
|
|
|
|
return bdrv_co_truncate(blk->root, offset, exact, prealloc, flags, errp);
|
|
}
|
|
|
|
int blk_save_vmstate(BlockBackend *blk, const uint8_t *buf,
|
|
int64_t pos, int size)
|
|
{
|
|
int ret;
|
|
GLOBAL_STATE_CODE();
|
|
|
|
if (!blk_is_available(blk)) {
|
|
return -ENOMEDIUM;
|
|
}
|
|
|
|
ret = bdrv_save_vmstate(blk_bs(blk), buf, pos, size);
|
|
if (ret < 0) {
|
|
return ret;
|
|
}
|
|
|
|
if (ret == size && !blk->enable_write_cache) {
|
|
ret = bdrv_flush(blk_bs(blk));
|
|
}
|
|
|
|
return ret < 0 ? ret : size;
|
|
}
|
|
|
|
int blk_load_vmstate(BlockBackend *blk, uint8_t *buf, int64_t pos, int size)
|
|
{
|
|
GLOBAL_STATE_CODE();
|
|
if (!blk_is_available(blk)) {
|
|
return -ENOMEDIUM;
|
|
}
|
|
|
|
return bdrv_load_vmstate(blk_bs(blk), buf, pos, size);
|
|
}
|
|
|
|
int blk_probe_blocksizes(BlockBackend *blk, BlockSizes *bsz)
|
|
{
|
|
GLOBAL_STATE_CODE();
|
|
if (!blk_is_available(blk)) {
|
|
return -ENOMEDIUM;
|
|
}
|
|
|
|
return bdrv_probe_blocksizes(blk_bs(blk), bsz);
|
|
}
|
|
|
|
int blk_probe_geometry(BlockBackend *blk, HDGeometry *geo)
|
|
{
|
|
GLOBAL_STATE_CODE();
|
|
if (!blk_is_available(blk)) {
|
|
return -ENOMEDIUM;
|
|
}
|
|
|
|
return bdrv_probe_geometry(blk_bs(blk), geo);
|
|
}
|
|
|
|
/*
|
|
* Updates the BlockBackendRootState object with data from the currently
|
|
* attached BlockDriverState.
|
|
*/
|
|
void blk_update_root_state(BlockBackend *blk)
|
|
{
|
|
GLOBAL_STATE_CODE();
|
|
assert(blk->root);
|
|
|
|
blk->root_state.open_flags = blk->root->bs->open_flags;
|
|
blk->root_state.detect_zeroes = blk->root->bs->detect_zeroes;
|
|
}
|
|
|
|
/*
|
|
* Returns the detect-zeroes setting to be used for bdrv_open() of a
|
|
* BlockDriverState which is supposed to inherit the root state.
|
|
*/
|
|
bool blk_get_detect_zeroes_from_root_state(BlockBackend *blk)
|
|
{
|
|
GLOBAL_STATE_CODE();
|
|
return blk->root_state.detect_zeroes;
|
|
}
|
|
|
|
/*
|
|
* Returns the flags to be used for bdrv_open() of a BlockDriverState which is
|
|
* supposed to inherit the root state.
|
|
*/
|
|
int blk_get_open_flags_from_root_state(BlockBackend *blk)
|
|
{
|
|
GLOBAL_STATE_CODE();
|
|
return blk->root_state.open_flags;
|
|
}
|
|
|
|
BlockBackendRootState *blk_get_root_state(BlockBackend *blk)
|
|
{
|
|
GLOBAL_STATE_CODE();
|
|
return &blk->root_state;
|
|
}
|
|
|
|
int blk_commit_all(void)
|
|
{
|
|
BlockBackend *blk = NULL;
|
|
GLOBAL_STATE_CODE();
|
|
|
|
while ((blk = blk_all_next(blk)) != NULL) {
|
|
AioContext *aio_context = blk_get_aio_context(blk);
|
|
BlockDriverState *unfiltered_bs = bdrv_skip_filters(blk_bs(blk));
|
|
|
|
aio_context_acquire(aio_context);
|
|
if (blk_is_inserted(blk) && bdrv_cow_child(unfiltered_bs)) {
|
|
int ret;
|
|
|
|
ret = bdrv_commit(unfiltered_bs);
|
|
if (ret < 0) {
|
|
aio_context_release(aio_context);
|
|
return ret;
|
|
}
|
|
}
|
|
aio_context_release(aio_context);
|
|
}
|
|
return 0;
|
|
}
|
|
|
|
|
|
/* throttling disk I/O limits */
|
|
void blk_set_io_limits(BlockBackend *blk, ThrottleConfig *cfg)
|
|
{
|
|
GLOBAL_STATE_CODE();
|
|
throttle_group_config(&blk->public.throttle_group_member, cfg);
|
|
}
|
|
|
|
void blk_io_limits_disable(BlockBackend *blk)
|
|
{
|
|
BlockDriverState *bs = blk_bs(blk);
|
|
ThrottleGroupMember *tgm = &blk->public.throttle_group_member;
|
|
assert(tgm->throttle_state);
|
|
GLOBAL_STATE_CODE();
|
|
if (bs) {
|
|
bdrv_ref(bs);
|
|
bdrv_drained_begin(bs);
|
|
}
|
|
throttle_group_unregister_tgm(tgm);
|
|
if (bs) {
|
|
bdrv_drained_end(bs);
|
|
bdrv_unref(bs);
|
|
}
|
|
}
|
|
|
|
/* should be called before blk_set_io_limits if a limit is set */
|
|
void blk_io_limits_enable(BlockBackend *blk, const char *group)
|
|
{
|
|
assert(!blk->public.throttle_group_member.throttle_state);
|
|
GLOBAL_STATE_CODE();
|
|
throttle_group_register_tgm(&blk->public.throttle_group_member,
|
|
group, blk_get_aio_context(blk));
|
|
}
|
|
|
|
void blk_io_limits_update_group(BlockBackend *blk, const char *group)
|
|
{
|
|
GLOBAL_STATE_CODE();
|
|
/* this BB is not part of any group */
|
|
if (!blk->public.throttle_group_member.throttle_state) {
|
|
return;
|
|
}
|
|
|
|
/* this BB is a part of the same group than the one we want */
|
|
if (!g_strcmp0(throttle_group_get_name(&blk->public.throttle_group_member),
|
|
group)) {
|
|
return;
|
|
}
|
|
|
|
/* need to change the group this bs belong to */
|
|
blk_io_limits_disable(blk);
|
|
blk_io_limits_enable(blk, group);
|
|
}
|
|
|
|
static void blk_root_drained_begin(BdrvChild *child)
|
|
{
|
|
BlockBackend *blk = child->opaque;
|
|
ThrottleGroupMember *tgm = &blk->public.throttle_group_member;
|
|
|
|
if (++blk->quiesce_counter == 1) {
|
|
if (blk->dev_ops && blk->dev_ops->drained_begin) {
|
|
blk->dev_ops->drained_begin(blk->dev_opaque);
|
|
}
|
|
}
|
|
|
|
/* Note that blk->root may not be accessible here yet if we are just
|
|
* attaching to a BlockDriverState that is drained. Use child instead. */
|
|
|
|
if (qatomic_fetch_inc(&tgm->io_limits_disabled) == 0) {
|
|
throttle_group_restart_tgm(tgm);
|
|
}
|
|
}
|
|
|
|
static bool blk_root_drained_poll(BdrvChild *child)
|
|
{
|
|
BlockBackend *blk = child->opaque;
|
|
bool busy = false;
|
|
assert(blk->quiesce_counter);
|
|
|
|
if (blk->dev_ops && blk->dev_ops->drained_poll) {
|
|
busy = blk->dev_ops->drained_poll(blk->dev_opaque);
|
|
}
|
|
return busy || !!blk->in_flight;
|
|
}
|
|
|
|
static void blk_root_drained_end(BdrvChild *child, int *drained_end_counter)
|
|
{
|
|
BlockBackend *blk = child->opaque;
|
|
assert(blk->quiesce_counter);
|
|
|
|
assert(blk->public.throttle_group_member.io_limits_disabled);
|
|
qatomic_dec(&blk->public.throttle_group_member.io_limits_disabled);
|
|
|
|
if (--blk->quiesce_counter == 0) {
|
|
if (blk->dev_ops && blk->dev_ops->drained_end) {
|
|
blk->dev_ops->drained_end(blk->dev_opaque);
|
|
}
|
|
while (qemu_co_enter_next(&blk->queued_requests, NULL)) {
|
|
/* Resume all queued requests */
|
|
}
|
|
}
|
|
}
|
|
|
|
bool blk_register_buf(BlockBackend *blk, void *host, size_t size, Error **errp)
|
|
{
|
|
BlockDriverState *bs = blk_bs(blk);
|
|
|
|
GLOBAL_STATE_CODE();
|
|
|
|
if (bs) {
|
|
return bdrv_register_buf(bs, host, size, errp);
|
|
}
|
|
return true;
|
|
}
|
|
|
|
void blk_unregister_buf(BlockBackend *blk, void *host, size_t size)
|
|
{
|
|
BlockDriverState *bs = blk_bs(blk);
|
|
|
|
GLOBAL_STATE_CODE();
|
|
|
|
if (bs) {
|
|
bdrv_unregister_buf(bs, host, size);
|
|
}
|
|
}
|
|
|
|
int coroutine_fn blk_co_copy_range(BlockBackend *blk_in, int64_t off_in,
|
|
BlockBackend *blk_out, int64_t off_out,
|
|
int64_t bytes, BdrvRequestFlags read_flags,
|
|
BdrvRequestFlags write_flags)
|
|
{
|
|
int r;
|
|
IO_CODE();
|
|
|
|
r = blk_check_byte_request(blk_in, off_in, bytes);
|
|
if (r) {
|
|
return r;
|
|
}
|
|
r = blk_check_byte_request(blk_out, off_out, bytes);
|
|
if (r) {
|
|
return r;
|
|
}
|
|
return bdrv_co_copy_range(blk_in->root, off_in,
|
|
blk_out->root, off_out,
|
|
bytes, read_flags, write_flags);
|
|
}
|
|
|
|
const BdrvChild *blk_root(BlockBackend *blk)
|
|
{
|
|
GLOBAL_STATE_CODE();
|
|
return blk->root;
|
|
}
|
|
|
|
int blk_make_empty(BlockBackend *blk, Error **errp)
|
|
{
|
|
GLOBAL_STATE_CODE();
|
|
if (!blk_is_available(blk)) {
|
|
error_setg(errp, "No medium inserted");
|
|
return -ENOMEDIUM;
|
|
}
|
|
|
|
return bdrv_make_empty(blk->root, errp);
|
|
}
|