[PULL 0/5] Misc fixes, for thread-pool, xen, and xen-emulate

Anthony PERARD via posted 5 patches 9 months ago
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20230801094038.11026-1-anthony.perard@citrix.com
Maintainers: Peter Maydell <peter.maydell@linaro.org>, Stefano Stabellini <sstabellini@kernel.org>, Anthony Perard <anthony.perard@citrix.com>, Paul Durrant <paul@xen.org>, Kevin Wolf <kwolf@redhat.com>, Hanna Reitz <hreitz@redhat.com>, David Woodhouse <dwmw2@infradead.org>, "Michael S. Tsirkin" <mst@redhat.com>, Marcel Apfelbaum <marcel.apfelbaum@gmail.com>, Paolo Bonzini <pbonzini@redhat.com>, Richard Henderson <richard.henderson@linaro.org>, Eduardo Habkost <eduardo@habkost.net>
hw/arm/xen_arm.c                |  4 ++--
hw/block/xen-block.c            | 11 ++++++-----
hw/i386/kvm/xenstore_impl.c     | 12 +++++++++++-
hw/i386/xen/xen-hvm.c           |  4 ++--
hw/i386/xen/xen_platform.c      |  7 ++++---
hw/xen/xen-hvm-common.c         |  8 ++++----
include/hw/xen/xen-hvm-common.h |  2 +-
util/thread-pool.c              |  2 +-
8 files changed, 31 insertions(+), 19 deletions(-)
[PULL 0/5] Misc fixes, for thread-pool, xen, and xen-emulate
Posted by Anthony PERARD via 9 months ago
The following changes since commit 802341823f1720511dd5cf53ae40285f7978c61b:

  Merge tag 'pull-tcg-20230731' of https://gitlab.com/rth7680/qemu into staging (2023-07-31 14:02:51 -0700)

are available in the Git repository at:

  https://xenbits.xen.org/git-http/people/aperard/qemu-dm.git tags/pull-xen-20230801

for you to fetch changes up to 856ca10f9ce1fcffeab18546b36a64f79017c905:

  xen-platform: do full PCI reset during unplug of IDE devices (2023-08-01 10:22:33 +0100)

----------------------------------------------------------------
Misc fixes, for thread-pool, xen, and xen-emulate

* fix an access to `request_cond` QemuCond in thread-pool
* fix issue with PCI devices when unplugging IDE devices in Xen guest
* several fixes for issues pointed out by Coverity

----------------------------------------------------------------
Anthony PERARD (2):
      xen-block: Avoid leaks on new error path
      thread-pool: signal "request_cond" while locked

David Woodhouse (1):
      hw/xen: Clarify (lack of) error handling in transaction_commit()

Olaf Hering (1):
      xen-platform: do full PCI reset during unplug of IDE devices

Peter Maydell (1):
      xen: Don't pass MemoryListener around by value

 hw/arm/xen_arm.c                |  4 ++--
 hw/block/xen-block.c            | 11 ++++++-----
 hw/i386/kvm/xenstore_impl.c     | 12 +++++++++++-
 hw/i386/xen/xen-hvm.c           |  4 ++--
 hw/i386/xen/xen_platform.c      |  7 ++++---
 hw/xen/xen-hvm-common.c         |  8 ++++----
 include/hw/xen/xen-hvm-common.h |  2 +-
 util/thread-pool.c              |  2 +-
 8 files changed, 31 insertions(+), 19 deletions(-)
Re: [PULL 0/5] Misc fixes, for thread-pool, xen, and xen-emulate
Posted by Michael Tokarev 9 months ago
01.08.2023 12:40, Anthony PERARD via wrote:
...
> Anthony PERARD (2):
>        xen-block: Avoid leaks on new error path
>        thread-pool: signal "request_cond" while locked
> 
> David Woodhouse (1):
>        hw/xen: Clarify (lack of) error handling in transaction_commit()
> 
> Olaf Hering (1):
>        xen-platform: do full PCI reset during unplug of IDE devices
> 
> Peter Maydell (1):
>        xen: Don't pass MemoryListener around by value

 From the descriptions, it looks like

         xen-block: Avoid leaks on new error path
         thread-pool: signal "request_cond" while locked
         xen-platform: do full PCI reset during unplug of IDE devices

should be applied to -stable.

More, it looks like all 3 should be applied to v7.2.x series too,
including the PCI reset one, which is a bit problematic for 7.2
because it lacks dfa6ba6baeebe "Replace use of qdev_reset_all()
with device_cold_reset()", but that change was mechanical.

Olaf, can you please take a look at
https://gitlab.com/mjt0k/qemu/-/commit/645d74c366c8c6334bf9a3403f780adefe4e410f
(from https://gitlab.com/mjt0k/qemu/-/commits/staging-7.2/ )
and check if it makes sense?

Or if whole thing makes no sense to backport to stable, please
let me know as well.

Thanks!

/mjt
Re: [PULL 0/5] Misc fixes, for thread-pool, xen, and xen-emulate
Posted by Olaf Hering 9 months ago
Wed, 2 Aug 2023 18:18:01 +0300 Michael Tokarev <mjt@tls.msk.ru>:

> Or if whole thing makes no sense to backport to stable, please
> let me know as well.

The xen-platform change does not need to be backported IMHO.


Olaf
Re: [PULL 0/5] Misc fixes, for thread-pool, xen, and xen-emulate
Posted by Richard Henderson 9 months ago
On 8/1/23 02:40, Anthony PERARD via wrote:
> The following changes since commit 802341823f1720511dd5cf53ae40285f7978c61b:
> 
>    Merge tag 'pull-tcg-20230731' ofhttps://gitlab.com/rth7680/qemu  into staging (2023-07-31 14:02:51 -0700)
> 
> are available in the Git repository at:
> 
>    https://xenbits.xen.org/git-http/people/aperard/qemu-dm.git  tags/pull-xen-20230801
> 
> for you to fetch changes up to 856ca10f9ce1fcffeab18546b36a64f79017c905:
> 
>    xen-platform: do full PCI reset during unplug of IDE devices (2023-08-01 10:22:33 +0100)
> 
> ----------------------------------------------------------------
> Misc fixes, for thread-pool, xen, and xen-emulate
> 
> * fix an access to `request_cond` QemuCond in thread-pool
> * fix issue with PCI devices when unplugging IDE devices in Xen guest
> * several fixes for issues pointed out by Coverity

Applied, thanks.  Please update https://wiki.qemu.org/ChangeLog/8.1 as appropriate.


r~