OSDN Git Service

vfio/migration: Reset bytes_transferred properly
authorAvihai Horon <avihaih@nvidia.com>
Wed, 28 Jun 2023 07:31:11 +0000 (10:31 +0300)
committerCédric Le Goater <clg@redhat.com>
Fri, 30 Jun 2023 04:02:51 +0000 (06:02 +0200)
commit808642a2f6407157f8cfb48ba6c2d28efd038d62
treeee25880b0b05a512dbae45f0be645fdd5e6d2c0a
parentc17408892319712c12357e5d1c6b305499c58c2a
vfio/migration: Reset bytes_transferred properly

Currently, VFIO bytes_transferred is not reset properly:
1. bytes_transferred is not reset after a VM snapshot (so a migration
   following a snapshot will report incorrect value).
2. bytes_transferred is a single counter for all VFIO devices, however
   upon migration failure it is reset multiple times, by each VFIO
   device.

Fix it by introducing a new function vfio_reset_bytes_transferred() and
calling it during migration and snapshot start.

Remove existing bytes_transferred reset in VFIO migration state
notifier, which is not needed anymore.

Fixes: 3710586caa5d ("qapi: Add VFIO devices migration stats in Migration stats")
Signed-off-by: Avihai Horon <avihaih@nvidia.com>
Reviewed-by: Cédric Le Goater <clg@redhat.com>
Reviewed-by: Alex Williamson <alex.williamson@redhat.com>
Signed-off-by: Cédric Le Goater <clg@redhat.com>
hw/vfio/migration.c
include/hw/vfio/vfio-common.h
migration/migration.c
migration/migration.h
migration/savevm.c
migration/target.c