diff options
| author | Avihai Horon <avihaih@nvidia.com> | 2024-12-18 15:40:20 +0200 |
|---|---|---|
| committer | Cédric Le Goater <clg@redhat.com> | 2024-12-26 07:23:38 +0100 |
| commit | 7d5d9c8864f55983e5e9b9e6a8fd05440a80d48f (patch) | |
| tree | f2afe0606d35d282db3524916c741caa337f5db6 /system/dirtylimit.c | |
| parent | 1f21670ec0700575280bd6aa837eaa8d87fc98c1 (diff) | |
| download | qemu-7d5d9c8864f55983e5e9b9e6a8fd05440a80d48f.zip qemu-7d5d9c8864f55983e5e9b9e6a8fd05440a80d48f.tar.gz qemu-7d5d9c8864f55983e5e9b9e6a8fd05440a80d48f.tar.bz2 | |
system/dirtylimit: Don't use migration_is_active()
vcpu_dirty_rate_stat_collect() uses migration_is_active() to detect
whether migration is running or not, in order to get the correct dirty
rate period value.
However, recently there has been an effort to simplify the migration
status API and reduce it to a single migration_is_running() function.
To accommodate this, and since the same functionality can be achieved
with migration_is_running(), use it instead of migration_is_active().
Signed-off-by: Avihai Horon <avihaih@nvidia.com>
Reviewed-by: Hyman Huang <yong.huang@smartx.com>
Tested-by: Joao Martins <joao.m.martins@oracle.com>
Link: https://lore.kernel.org/r/20241218134022.21264-6-avihaih@nvidia.com
Signed-off-by: Cédric Le Goater <clg@redhat.com>
Diffstat (limited to 'system/dirtylimit.c')
| -rw-r--r-- | system/dirtylimit.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/system/dirtylimit.c b/system/dirtylimit.c index d94b994..7c07124 100644 --- a/system/dirtylimit.c +++ b/system/dirtylimit.c @@ -80,8 +80,7 @@ static void vcpu_dirty_rate_stat_collect(void) int i = 0; int64_t period = DIRTYLIMIT_CALC_TIME_MS; - if (migrate_dirty_limit() && - migration_is_active()) { + if (migrate_dirty_limit() && migration_is_running()) { period = migrate_vcpu_dirty_limit_period(); } |