Skip to content
Snippets Groups Projects
  1. Aug 01, 2022
  2. Jul 30, 2022
    • Richard Henderson's avatar
      Merge tag 'pull-la-20220729' of https://gitlab.com/rth7680/qemu into staging · 3916603e
      Richard Henderson authored
      Rename ls7a to virt, when it's board not chipset related.
      
      # -----BEGIN PGP SIGNATURE-----
      #
      # iQFRBAABCgA7FiEEekgeeIaLTbaoWgXAZN846K9+IV8FAmLkfO8dHHJpY2hhcmQu
      # aGVuZGVyc29uQGxpbmFyby5vcmcACgkQZN846K9+IV9UvAgAud1jhWBalaON0be6
      # tC3UMB2Xe5Dzgm5yiLC7EspHci/HB/kSqbeXY436/hbU9iBXGEZkuTeQ1BX41Aq8
      # D8LBzFAr35uySD5wfZbDdpefCvuBiDcb1SMpNXLC4I3zJj0Euj96j/IewIeJfGrc
      # 0ZkJSq4jAOuPaU0NB1+Wmb9UsoMWhHQQOcIdz8ZpR0hjuU8yz7xAEGQosJNh/Acq
      # Fdm6jDCOH4KY+uw/6dKF9poeSqpBDz3rCLicNNk6D+btDQybb2NzaVHE5ApLGRbW
      # T0MnOf1ERoWTubAbJasKR/ODCt6Jby3kC9lZFsfOAqKjRXMYL/HexdJcM2UqKE9W
      # E0aFjQ==
      # =c3v3
      # -----END PGP SIGNATURE-----
      # gpg: Signature made Fri 29 Jul 2022 05:35:59 PM PDT
      # gpg:                using RSA key 7A481E78868B4DB6A85A05C064DF38E8AF7E215F
      # gpg:                issuer "richard.henderson@linaro.org"
      # gpg: Good signature from "Richard Henderson <richard.henderson@linaro.org>" [ultimate]
      
      * tag 'pull-la-20220729' of https://gitlab.com/rth7680/qemu
      
      :
        hw/loongarch: Change macro name 'LS7A_XXX' to 'VIRT_XXX'
        hw/loongarch: Rename file 'loongson3.XXX' to 'virt.XXX'
      
      Signed-off-by: default avatarRichard Henderson <richard.henderson@linaro.org>
      3916603e
  3. Jul 29, 2022
  4. Jul 28, 2022
    • Paolo Bonzini's avatar
      configure: pass correct cflags to container-based cross compilers · ebc55f52
      Paolo Bonzini authored
      
      probe_target_compiler returns nonempty $target_cc for installed toolchains
      and $container_cross_cc for container-based toolchains.  In both cases
      however the flags (coming from $cross_cc_cflags_${target_arch}) must be
      in $target_cflags.
      
      Therefore, do not clear them prior to returning from probe_target_compiler.
      
      Reported-by: default avatarTaylor Simpson <tsimpson@quicinc.com>
      Fixes: 92e288fc ("build: try both native and cross compilers", 2022-07-08)
      Signed-off-by: default avatarPaolo Bonzini <pbonzini@redhat.com>
      ebc55f52
    • Cornelia Huck's avatar
      kvm: don't use perror() without useful errno · 47c182fe
      Cornelia Huck authored
      
      perror() is designed to append the decoded errno value to a
      string. This, however, only makes sense if we called something that
      actually sets errno prior to that.
      
      For the callers that check for split irqchip support that is not the
      case, and we end up with confusing error messages that end in
      "success". Use error_report() instead.
      
      Signed-off-by: default avatarCornelia Huck <cohuck@redhat.com>
      Message-Id: <20220728142446.438177-1-cohuck@redhat.com>
      Signed-off-by: default avatarPaolo Bonzini <pbonzini@redhat.com>
      47c182fe
    • Richard Henderson's avatar
      configure: Fix ppc container_cross_cc substitution · 705c881f
      Richard Henderson authored
      
      When moving this code out of probe_target_compiler(), we failed to adjust
      the variable in which the target is located, resulting in e.g.
      powerpc64-linux-user-linux-gnu-gcc-10
      
      Fixes: cd362def ("tests/tcg: merge configure.sh back into main configure script")
      Signed-off-by: default avatarRichard Henderson <richard.henderson@linaro.org>
      Message-Id: <20220728183901.1290113-1-richard.henderson@linaro.org>
      Signed-off-by: default avatarPaolo Bonzini <pbonzini@redhat.com>
      705c881f
    • Richard Henderson's avatar
      Merge tag 'pull-ppc-20220728' of https://gitlab.com/danielhb/qemu into staging · cc42559a
      Richard Henderson authored
      ppc patch queue for 2022-07-28:
      
      Short queue with 2 Coverity fixes and one fix of the
      'wait' insns that is causing hangs if the guest kernel uses
      the most up to date wait opcode.
      
      - target/ppc:
        - implement new wait variants to fix guest hang when using the new opcode
      - ppc440_uc: initialize length passed to cpu_physical_memory_map()
      - spapr_nvdimm: check if spapr_drc_index() returns NULL
      
      # -----BEGIN PGP SIGNATURE-----
      #
      # iHUEABYKAB0WIQQX6/+ZI9AYAK8oOBk82cqW3gMxZAUCYuK8VgAKCRA82cqW3gMx
      # ZOc7AQDPMsFY9NHNqJ3O0MiX4Qoy8IGUreZ9dzZSS3zT1nxtEAD+Lwl0/aGO+dk+
      # +NiIO80A5Agy/0g8PHie4qR3EqHEnwA=
      # =Q4eR
      # -----END PGP SIGNATURE-----
      # gpg: Signature made Thu 28 Jul 2022 09:41:58 AM PDT
      # gpg:                using EDDSA key 17EBFF9923D01800AF2838193CD9CA96DE033164
      # gpg: Good signature from "Daniel Henrique Barboza <danielhb413@gmail.com>" [unknown]
      # gpg: WARNING: This key is not certified with a trusted signature!
      # gpg:          There is no indication that the signature belongs to the owner.
      # Primary key fingerprint: 17EB FF99 23D0 1800 AF28  3819 3CD9 CA96 DE03 3164
      
      * tag 'pull-ppc-20220728' of https://gitlab.com/danielhb/qemu
      
      :
        target/ppc: Implement new wait variants
        hw/ppc/ppc440_uc: Initialize length passed to cpu_physical_memory_map()
        hw/ppc: check if spapr_drc_index() returns NULL in spapr_nvdimm.c
      
      Signed-off-by: default avatarRichard Henderson <richard.henderson@linaro.org>
      cc42559a
    • Nicholas Piggin's avatar
      target/ppc: Implement new wait variants · 0c9717ff
      Nicholas Piggin authored
      ISA v2.06 adds new variations of wait, specified by the WC field. These
      are not all compatible with the prior wait implementation, because they
      add additional conditions that cause the processor to resume, which can
      cause software to hang or run very slowly.
      
      At this moment, with the current wait implementation and a pseries guest
      using mainline kernel with new wait upcodes [1], QEMU hangs during boot if
      more than one CPU is present:
      
       qemu-system-ppc64 -M pseries,x-vof=on -cpu POWER10 -smp 2 -nographic
      -kernel zImage.pseries -no-reboot
      
      QEMU will exit (as there's no filesystem) if the test "passes", or hang
      during boot if it hits the bug.
      
      ISA v3.0 changed the wait opcode and removed the new variants (retaining
      the WC field but making non-zero values reserved).
      
      ISA v3.1 added new WC values to the new wait opcode, and added a PL
      field.
      
      This patch implements the new wait encoding and supports WC variants
      with no-op implementations, which provides basic correctness as
      explained in comments.
      
      [1] https://lore.kernel.org/all/20220720132132.903462-1-npiggin@gmail.com/
      
      
      
      Signed-off-by: default avatarNicholas Piggin <npiggin@gmail.com>
      Reviewed-by: default avatarVíctor Colombo <victor.colombo@eldorado.org.br>
      Tested-by: default avatarJoel Stanley <joel@jms.id.au>
      Reviewed-by: default avatarDaniel Henrique Barboza <danielhb413@gmail.com>
      Message-Id: <20220720133352.904263-1-npiggin@gmail.com>
      [danielhb: added information about the bug being fixed]
      Signed-off-by: default avatarDaniel Henrique Barboza <danielhb413@gmail.com>
      0c9717ff
    • Claudio Fontana's avatar
      stubs: update replay-tools to match replay.h types · bb7e03cb
      Claudio Fontana authored
      
      detected with GCC 13 [-Werror=enum-int-mismatch]
      
      Solves Issue #1096.
      
      Signed-off-by: default avatarClaudio Fontana <cfontana@suse.de>
      Cc: Pavel Dovgalyuk <pavel.dovgalyuk@ispras.ru>
      Reviewed-by: default avatarThomas Huth <thuth@redhat.com>
      Message-Id: <20220704075832.31537-1-cfontana@suse.de>
      Signed-off-by: default avatarPaolo Bonzini <pbonzini@redhat.com>
      bb7e03cb
    • Paolo Bonzini's avatar
      vga: fix incorrect line height in 640x200x2 mode · 37e7b867
      Paolo Bonzini authored
      When in CGA modes, QEMU wants to ignore the maximum scan field (bits 0..4) of
      the maximum scan length register in the CRTC.  It is not clear why this is
      needed---for example, Bochs ignores bit 7 instead.  The issue is that the
      CGA modes are not detected correctly, and in particular mode 6 results in
      multi_scan==3 according to how SeaBIOS programs it.  The right way to check
      for CGA graphics modes is to check whether bit 13 of the address is special
      cased by the CRT controller to achieve line interleaving, i.e. whether bit 0
      of the CRTC mode control register is clear.
      
      Resolves: https://gitlab.com/qemu-project/qemu/-/issues/1020
      
      
      Reported-by: default avatarKorneliusz Osmenda <korneliuszo@gmail.com>
      Signed-off-by: default avatarPaolo Bonzini <pbonzini@redhat.com>
      37e7b867
    • Peter Maydell's avatar
      hw/ppc/ppc440_uc: Initialize length passed to cpu_physical_memory_map() · eda3f17b
      Peter Maydell authored
      
      In dcr_write_dma(), there is code that uses cpu_physical_memory_map()
      to implement a DMA transfer.  That function takes a 'plen' argument,
      which points to a hwaddr which is used for both input and output: the
      caller must set it to the size of the range it wants to map, and on
      return it is updated to the actual length mapped. The dcr_write_dma()
      code fails to initialize rlen and wlen, so will end up mapping an
      unpredictable amount of memory.
      
      Initialize the length values correctly, and check that we managed to
      map the entire range before using the fast-path memmove().
      
      This was spotted by Coverity, which points out that we never
      initialized the variables before using them.
      
      Fixes: Coverity CID 1487137, 1487150
      Signed-off-by: default avatarPeter Maydell <peter.maydell@linaro.org>
      Reviewed-by: default avatarRichard Henderson <richard.henderson@linaro.org>
      Message-Id: <20220726182341.1888115-2-peter.maydell@linaro.org>
      Signed-off-by: default avatarDaniel Henrique Barboza <danielhb413@gmail.com>
      eda3f17b
    • Daniel Henrique Barboza's avatar
      hw/ppc: check if spapr_drc_index() returns NULL in spapr_nvdimm.c · edccf661
      Daniel Henrique Barboza authored
      
      spapr_nvdimm_flush_completion_cb() and flush_worker_cb() are using the
      DRC object returned by spapr_drc_index() without checking it for NULL.
      In this case we would be dereferencing a NULL pointer when doing
      SPAPR_NVDIMM(drc->dev) and PC_DIMM(drc->dev).
      
      This can happen if, during a scm_flush(), the DRC object is wrongly
      freed/released (e.g. a bug in another part of the code).
      spapr_drc_index() would then return NULL in the callbacks.
      
      Fixes: Coverity CID 1487108, 1487178
      Reviewed-by: default avatarGreg Kurz <groug@kaod.org>
      Message-Id: <20220409200856.283076-2-danielhb413@gmail.com>
      Signed-off-by: default avatarDaniel Henrique Barboza <danielhb413@gmail.com>
      edccf661
    • Richard Henderson's avatar
      Merge tag 'pull-riscv-to-apply-20220728' of github.com:alistair23/qemu into staging · a17001c4
      Richard Henderson authored
      
      Sixth RISC-V PR for QEMU 7.1
      
      This is a PR to go in for RC1. It fixes a segfault that occurs
      when using multiple sockets on the RISC-V virt board. It also
      includes a small fix to allow both Zmmul and M extensions.
      
      * Allow both Zmmul and M extension
      * Fix multi-socket plic configuraiton
      
      # -----BEGIN PGP SIGNATURE-----
      #
      # iQEzBAABCAAdFiEE9sSsRtSTSGjTuM6PIeENKd+XcFQFAmLh33AACgkQIeENKd+X
      # cFROBQf/QFxHsIX9clpAkHmK220efQ3rjHZtdCqQoCeRZp2EytFS9KZ6iae/BM9r
      # 3Z8cZci38kxjqTzsYJLj46yNO3AxHoFsDH41yWTMOsxjVWVlno/06R/C1B4Ek37N
      # kZXWKHzqfQvZRJIUAjKfVxaLtw9xRI9xYqWxVngdYSoW3HWHHz5UmA6fFoJ29QiZ
      # SKEgxhakrqhvN9GMm1aWGkLN10uD5lFWOBMYdqMVcWq48XSP3Df5FU2Xk0sfegXq
      # EqbIYKJL/Q6koyvmdpQz7VmtMAGjMTcmozEH8oN/MuCk7MCLmbloWVl+LF39SeTH
      # 3amapiJBtYBOwaNZUpb5TZkv/bEDIw==
      # =ip1R
      # -----END PGP SIGNATURE-----
      # gpg: Signature made Wed 27 Jul 2022 05:59:28 PM PDT
      # gpg:                using RSA key F6C4AC46D4934868D3B8CE8F21E10D29DF977054
      # gpg: Good signature from "Alistair Francis <alistair@alistair23.me>" [undefined]
      # gpg: WARNING: This key is not certified with a trusted signature!
      # gpg:          There is no indication that the signature belongs to the owner.
      # Primary key fingerprint: F6C4 AC46 D493 4868 D3B8  CE8F 21E1 0D29 DF97 7054
      
      * tag 'pull-riscv-to-apply-20220728' of github.com:alistair23/qemu:
        hw/intc: sifive_plic: Fix multi-socket plic configuraiton
        RISC-V: Allow both Zmmul and M
      
      Signed-off-by: default avatarRichard Henderson <richard.henderson@linaro.org>
      a17001c4
    • Paolo Bonzini's avatar
      ui: dbus-display requires CONFIG_GBM · d82b11f6
      Paolo Bonzini authored
      Without CONFIG_GBM, compiling dbus-display fails with
      
      ../ui/dbus.c: In function ‘dbus_create_context’:
      ../ui/dbus.c:47:20: error: ‘qemu_egl_rn_ctx’ undeclared (first use in this function); did you mean ‘qemu_egl_init_ctx’?
         47 |                    qemu_egl_rn_ctx);
            |                    ^~~~~~~~~~~~~~~
            |                    qemu_egl_init_ctx
      ../ui/dbus.c:47:20: note: each undeclared identifier is reported only once for each function it appears in
      
      and many other similar errors, because include/ui/egl-helpers.h only has
      these declaration if gbm is found on the system.
      
      Resolves: https://gitlab.com/qemu-project/qemu/-/issues/1108
      
      
      Reviewed-by: default avatarMarc-André Lureau <marcandre.lureau@redhat.com>
      Reviewed-by: default avatarDaniel P. Berrangé <berrange@redhat.com>
      Signed-off-by: default avatarPaolo Bonzini <pbonzini@redhat.com>
      d82b11f6
    • Richard Henderson's avatar
      Merge tag 'pull-block-2022-07-27' of https://gitlab.com/vsementsov/qemu into staging · 3e4abe2c
      Richard Henderson authored
      Block: fix parallels block driver
      
      # -----BEGIN PGP SIGNATURE-----
      #
      # iQIzBAABCgAdFiEEi5wmzbL9FHyIDoahVh8kwfGfefsFAmLhi0cACgkQVh8kwfGf
      # efs1Dg/7BwbaJu5uZEGhz3+KBRPk5kdYKX60bOojac27pBTVo4OyiP7QFzBt4c6+
      # 4yftT2vD7yTyzKANlmIYBvmjoIEw6eB09gJ5/mnUKgxTAS+thKo0e2v1zdncliy+
      # h9SEYRT3RhlePJYSssZx8lW4gfCG2JZi5xSjfqbG50X7I8RgDtMmcj7EUwkvCkaI
      # WL3iZIuYPxkfFwbQ/6xVmwc6uE97tWom9Z0iyEgFIhtFGlrgV3zJrDJ2CbOXIbi+
      # 9c2j4zmnMUZLwtdT2CFwyvO03iU8eMJxqnt4aSyByOAd/rqko+ugHeE53eZkND0Q
      # ci4bFq9XjgxOSsIqHXemIEUnuExhMuw5i7dtwR8w7K5Kwc88/44GTUgCZrPnBLx2
      # smGX0g7BiCpNYXA8DkquOsUQf8cS67M3rjdTB6SiMo0KuQHe5O0RDQAwu7f+hnTw
      # vEyo8dk4xGqUvqYcOpLLBHDis1lghWwseC5gB/M6Q+KqvDF4WDpIWwPLfR1phJ0L
      # kA1M9QO+NAcUtLEuT7N22QU8LMTxAX/hSYpR5Jrt5g3R26h7w7VZEvJbpQaytXTY
      # VhDVWAGg9Xn3oxGTEyVqGP3Avik9OeoK9gCFiIyTEOL1jfEXqOsX8V6QkpaKP6c+
      # WXWiWfV9A9D7O556Z92hUeDuWhQKb2w1dry2e7DPeSWiUmgHtyY=
      # =QfFj
      # -----END PGP SIGNATURE-----
      # gpg: Signature made Wed 27 Jul 2022 12:00:23 PM PDT
      # gpg:                using RSA key 8B9C26CDB2FD147C880E86A1561F24C1F19F79FB
      # gpg: Good signature from "Vladimir Sementsov-Ogievskiy <vsementsov@yandex-team.ru>" [unknown]
      # gpg:                 aka "Vladimir Sementsov-Ogievskiy <vsementsov@virtuozzo.com>" [unknown]
      # gpg: WARNING: This key is not certified with a trusted signature!
      # gpg:          There is no indication that the signature belongs to the owner.
      # Primary key fingerprint: 8B9C 26CD B2FD 147C 880E  86A1 561F 24C1 F19F 79FB
      
      * tag 'pull-block-2022-07-27' of https://gitlab.com/vsementsov/qemu
      
      :
        iotests/131: Add parallels regression test
        block/parallels: Fix buffer-based write call
      
      Signed-off-by: default avatarRichard Henderson <richard.henderson@linaro.org>
      3e4abe2c
  5. Jul 27, 2022
  6. Jul 26, 2022
    • Richard Henderson's avatar
      Merge tag 'for_upstream' of git://git.kernel.org/pub/scm/virt/kvm/mst/qemu into staging · e5b6555f
      Richard Henderson authored
      
      pc,virtio: fixes
      
      Several fixes. From now on, regression fixes only.
      
      Signed-off-by: default avatarMichael S. Tsirkin <mst@redhat.com>
      
      # -----BEGIN PGP SIGNATURE-----
      #
      # iQFDBAABCAAtFiEEXQn9CHHI+FuUyooNKB8NuNKNVGkFAmLgQr8PHG1zdEByZWRo
      # YXQuY29tAAoJECgfDbjSjVRpGUUIAKtNhrnKopGm4LlRpx8zN3Jc1Jo0nb648gaM
      # Oyi+Pl8+hpESUhaWN10XDk38/QuPQfIFeR2ZhfYjFTRlZE+n3X9LVlwL8ejjP8KH
      # AcWm78Ff/SLA45aMKMmw74pvEDNsoPYTp7TrfeIej5ub8BIXr8+8pqDdIR9WwtWO
      # PbhLNXkTT2yLEs6jCVT4/dyh7zivSkrY7G/RVmtUaFe3PgY8fdW2z3+Txz7UIMgw
      # CQoGuAucCO5ToBbs2CbT0V5yxY6G5VO6Qd8g0PzDW4M6GsY/Xr5QCnyJe0jTW0d6
      # Dcc7UZFAzGNzyQCxHCic9xwTO+ZcJPJlH5TwknunxOb9xwCx4Qs=
      # =zN41
      # -----END PGP SIGNATURE-----
      # gpg: Signature made Tue 26 Jul 2022 12:38:39 PM PDT
      # gpg:                using RSA key 5D09FD0871C8F85B94CA8A0D281F0DB8D28D5469
      # gpg:                issuer "mst@redhat.com"
      # gpg: Good signature from "Michael S. Tsirkin <mst@kernel.org>" [undefined]
      # gpg:                 aka "Michael S. Tsirkin <mst@redhat.com>" [undefined]
      # gpg: WARNING: This key is not certified with a trusted signature!
      # gpg:          There is no indication that the signature belongs to the owner.
      # Primary key fingerprint: 0270 606B 6F3C DF3D 0B17  0970 C350 3912 AFBE 8E67
      #      Subkey fingerprint: 5D09 FD08 71C8 F85B 94CA  8A0D 281F 0DB8 D28D 5469
      
      * tag 'for_upstream' of git://git.kernel.org/pub/scm/virt/kvm/mst/qemu
      
      :
        hw/virtio/virtio-iommu: Enforce power-of-two notify for both MAP and UNMAP
        i386/pc: restrict AMD only enforcing of 1Tb hole to new machine type
        i386/pc: relocate 4g start to 1T where applicable
        i386/pc: bounds check phys-bits against max used GPA
        i386/pc: factor out device_memory base/size to helper
        i386/pc: handle unitialized mr in pc_get_cxl_range_end()
        i386/pc: factor out cxl range start to helper
        i386/pc: factor out cxl range end to helper
        i386/pc: factor out above-4g end to an helper
        i386/pc: pass pci_hole64_size to pc_memory_init()
        i386/pc: create pci-host qdev prior to pc_memory_init()
        hw/i386: add 4g boundary start to X86MachineState
        hw/cxl: Fix size of constant in interleave granularity function.
        hw/i386/pc: Always place CXL Memory Regions after device_memory
        hw/machine: Clear out left over CXL related pointer from move of state handling to machines.
        acpi/nvdimm: Define trace events for NVDIMM and substitute nvdimm_debug()
      
      Signed-off-by: default avatarRichard Henderson <richard.henderson@linaro.org>
      e5b6555f
    • Jean-Philippe Brucker's avatar
      hw/virtio/virtio-iommu: Enforce power-of-two notify for both MAP and UNMAP · 0522be9a
      Jean-Philippe Brucker authored
      
      Currently we only enforce power-of-two mappings (required by the QEMU
      notifier) for UNMAP requests. A MAP request not aligned on a
      power-of-two may be successfully handled by VFIO, and then the
      corresponding UNMAP notify will fail because it will attempt to split
      that mapping. Ensure MAP and UNMAP notifications are consistent.
      
      Fixes: dde3f08b ("virtio-iommu: Handle non power of 2 range invalidations")
      Reported-by: default avatarTina Zhang <tina.zhang@intel.com>
      Signed-off-by: default avatarJean-Philippe Brucker <jean-philippe@linaro.org>
      Message-Id: <20220718135636.338264-1-jean-philippe@linaro.org>
      Tested-by: default avatarTina Zhang <tina.zhang@intel.com>
      Reviewed-by: default avatarMichael S. Tsirkin <mst@redhat.com>
      Signed-off-by: default avatarMichael S. Tsirkin <mst@redhat.com>
      0522be9a
    • Hanna Reitz's avatar
      iotests/131: Add parallels regression test · 0c2cb382
      Hanna Reitz authored
      
      Test an allocating write to a parallels image that has a backing node.
      Before HEAD^, doing so used to give me a failed assertion (when the
      backing node contains only `42` bytes; the results varies with the value
      chosen, for `0` bytes, for example, all I get is EIO).
      
      Signed-off-by: default avatarHanna Reitz <hreitz@redhat.com>
      Reviewed-by: default avatarDenis V. Lunev <den@openvz.org>
      Reviewed-by: default avatarVladimir Sementsov-Ogievskiy <vsementsov@yandex-team.ru>
      Message-Id: <20220714132801.72464-3-hreitz@redhat.com>
      Signed-off-by: default avatarVladimir Sementsov-Ogievskiy <vsementsov@yandex-team.ru>
      0c2cb382
    • Hanna Reitz's avatar
      block/parallels: Fix buffer-based write call · eba088f9
      Hanna Reitz authored
      
      Commit a4072543 has changed the I/O here
      from working on a local one-element I/O vector to just using the buffer
      directly (using the bdrv_co_pread()/bdrv_co_pwrite() helper functions
      introduced shortly before).
      
      However, it only changed the bdrv_co_preadv() call to bdrv_co_pread() -
      the subsequent bdrv_co_pwritev() call stayed this way, and so still
      expects a QEMUIOVector pointer instead of a plain buffer.  We must
      change that to be a bdrv_co_pwrite() call.
      
      Fixes: a4072543 ("block/parallels: use buffer-based io")
      Signed-off-by: default avatarHanna Reitz <hreitz@redhat.com>
      Reviewed-by: default avatarDenis V. Lunev <den@openvz.org>
      Reviewed-by: default avatarVladimir Sementsov-Ogievskiy <vsementsov@yandex-team.ru>
      Message-Id: <20220714132801.72464-2-hreitz@redhat.com>
      Signed-off-by: default avatarVladimir Sementsov-Ogievskiy <vsementsov@yandex-team.ru>
      eba088f9
Loading