Re: [Qemu-devel] [PATCH 0/4] virtio-balloon: assorted fixes

[Stefan Hajnoczi <stefanha@gmail.com>]

[-- Attachment #1: Type: text/plain, Size: 1294 bytes --]

On Thu, Aug 18, 2016 at 09:27:50PM +0300, Roman Kagan wrote:
> This patchset addresses a few problems discovered when analyzing aborts
> of (an older version of) QEMU with backported commit
> afd9096eb1882f23929f5b5c177898ed231bac66 "virtio: error out if guest
> exceeds virtqueue size".  Those problems are present in master, too,
> except that they don't trigger an abort and are thus not as easy to
> notice.
> 
> 
> Roman Kagan (4):
>   virtio: assert on ->inuse underflow
>   virtio-balloon: make stats virtqueue length 1
>   virtio-balloon: don't restart stats timer in callback
>   virtio-balloon: keep collecting stats on save/restore
> 
> Cc: "Michael S. Tsirkin" <mst@redhat.com>
> 
>  hw/virtio/virtio-balloon.c         | 49 +++++++++++++++++++++-----------------
>  hw/virtio/virtio.c                 |  3 ++-
>  include/hw/virtio/virtio-balloon.h |  1 +
>  3 files changed, 30 insertions(+), 23 deletions(-)

Please see these related patches on the mailing list:

1. [PATCH v3] balloon: Fix failure of updating guest memory status

   This addresses the virtio-balloon stats virtqueue hang after
   migration.

2. [PATCH 0/2] virtio: fix VirtQueue->inuse field

   Fixes to update virtio.c in code paths where it currently becomes
   inconsistent.

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 473 bytes --]