diff options
| author | Kirti Wankhede <kwankhede@nvidia.com> | 2021-06-09 00:07:21 +0530 |
|---|---|---|
| committer | Alex Williamson <alex.williamson@redhat.com> | 2021-06-18 09:10:35 -0600 |
| commit | d742d064c1f541ac8ad6541e248a97f2dc502721 (patch) | |
| tree | cb2ca8b424f9ff3f1c6bed536aeb66028ca4d613 | |
| parent | 22fca190e25b10761925bb1eeadeda07aabf3c26 (diff) | |
vfio/migration: Correct device state from vmstate change for savevm case
Set _SAVING flag for device state from vmstate change handler when it
gets called from savevm.
Currently State transition savevm/suspend is seen as:
_RUNNING -> _STOP -> Stop-and-copy -> _STOP
State transition savevm/suspend should be:
_RUNNING -> Stop-and-copy -> _STOP
State transition from _RUNNING to _STOP occurs from
vfio_vmstate_change() where when vmstate changes from running to
!running, _RUNNING flag is reset but at the same time when
vfio_vmstate_change() is called for RUN_STATE_SAVE_VM, _SAVING bit
should be set.
Reported by: Yishai Hadas <yishaih@nvidia.com>
Signed-off-by: Kirti Wankhede <kwankhede@nvidia.com>
Message-Id: <1623177441-27496-1-git-send-email-kwankhede@nvidia.com>
Signed-off-by: Alex Williamson <alex.williamson@redhat.com>
| -rw-r--r-- | hw/vfio/migration.c | 11 |
1 files changed, 10 insertions, 1 deletions
diff --git a/hw/vfio/migration.c b/hw/vfio/migration.c index ef397ebe6c..82f654afb6 100644 --- a/hw/vfio/migration.c +++ b/hw/vfio/migration.c @@ -724,7 +724,16 @@ static void vfio_vmstate_change(void *opaque, bool running, RunState state) * _RUNNING bit */ mask = ~VFIO_DEVICE_STATE_RUNNING; - value = 0; + + /* + * When VM state transition to stop for savevm command, device should + * start saving data. + */ + if (state == RUN_STATE_SAVE_VM) { + value = VFIO_DEVICE_STATE_SAVING; + } else { + value = 0; + } } ret = vfio_migration_set_state(vbasedev, mask, value); |