[Qemu-devel] [PATCH v2 0/2] virtio: migration fixes for memory region cache

Stefan Hajnoczi posted 2 patches 7 years, 1 month ago
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20170222163734.13104-1-stefanha@redhat.com
Test checkpatch passed
Test docker passed
Test s390x passed
hw/virtio/virtio.c | 26 ++++++++++++++++++++------
1 file changed, 20 insertions(+), 6 deletions(-)
[Qemu-devel] [PATCH v2 0/2] virtio: migration fixes for memory region cache
Posted by Stefan Hajnoczi 7 years, 1 month ago
v2:
 * Avoid early region cache initialization in virtio_load() with wrong ring
   addresses for VIRTIO-1 in Patch 2 [Cornelia]

Bug fixes for the recently added memory region cache in virtio.  This series
fixes live migration, see patches for details.

Stefan Hajnoczi (2):
  virtio: invalidate memory in vring_set_avail_event()
  virtio: add missing region cache init in virtio_load()

 hw/virtio/virtio.c | 26 ++++++++++++++++++++------
 1 file changed, 20 insertions(+), 6 deletions(-)

-- 
2.9.3


Re: [Qemu-devel] [PATCH v2 0/2] virtio: migration fixes for memory region cache
Posted by Auger Eric 7 years, 1 month ago
Hi Stephan,

On 22/02/2017 17:37, Stefan Hajnoczi wrote:
> v2:
>  * Avoid early region cache initialization in virtio_load() with wrong ring
>    addresses for VIRTIO-1 in Patch 2 [Cornelia]
> 
> Bug fixes for the recently added memory region cache in virtio.  This series
> fixes live migration, see patches for details.
> 
> Stefan Hajnoczi (2):
>   virtio: invalidate memory in vring_set_avail_event()
>   virtio: add missing region cache init in virtio_load()
> 
>  hw/virtio/virtio.c | 26 ++++++++++++++++++++------
>  1 file changed, 20 insertions(+), 6 deletions(-)
> 

For me this fixes virtio-net migration issue caused by "virtio: use
VRingMemoryRegionCaches for avail and used rings".

Tested-by: Eric Auger <eric.auger@redhat.com>

Thanks

Eric

Re: [Qemu-devel] [PATCH v2 0/2] virtio: migration fixes for memory region cache
Posted by Stefan Hajnoczi 7 years ago
On Wed, Feb 22, 2017 at 04:37:32PM +0000, Stefan Hajnoczi wrote:
> v2:
>  * Avoid early region cache initialization in virtio_load() with wrong ring
>    addresses for VIRTIO-1 in Patch 2 [Cornelia]
> 
> Bug fixes for the recently added memory region cache in virtio.  This series
> fixes live migration, see patches for details.
> 
> Stefan Hajnoczi (2):
>   virtio: invalidate memory in vring_set_avail_event()
>   virtio: add missing region cache init in virtio_load()
> 
>  hw/virtio/virtio.c | 26 ++++++++++++++++++++------
>  1 file changed, 20 insertions(+), 6 deletions(-)

Michael: Ping?