mirror of
https://github.com/xemu-project/xemu.git
synced 2024-11-23 19:49:43 +00:00
migration/postcopy: Update the bandwidth during postcopy
The recently added max-postcopy-bandwidth parameter is only read
at the transition from precopy->postcopy where as the older
max-bandwidth parameter updates the migration bandwidth when changed
even if the migration is already running.
Fix this discrepency so that:
a) You can change the bandwidth during postcopy by setting
max-postcopy-bandwidth
b) Changing max-bandwidth during postcopy has no effect
(it currently changes the postcopy bandwidth which isn't
expected).
Fixes: 7e555c6c
bz: https://bugzilla.redhat.com/show_bug.cgi?id=1686321
Signed-off-by: Dr. David Alan Gilbert <dgilbert@redhat.com>
Reviewed-by: Juan Quintela <quintela@redhat.com>
Signed-off-by: Juan Quintela <quintela@redhat.com>
This commit is contained in:
parent
5ed0deca41
commit
c38c1c142e
@ -1341,7 +1341,7 @@ static void migrate_params_apply(MigrateSetParameters *params, Error **errp)
|
|||||||
|
|
||||||
if (params->has_max_bandwidth) {
|
if (params->has_max_bandwidth) {
|
||||||
s->parameters.max_bandwidth = params->max_bandwidth;
|
s->parameters.max_bandwidth = params->max_bandwidth;
|
||||||
if (s->to_dst_file) {
|
if (s->to_dst_file && !migration_in_postcopy()) {
|
||||||
qemu_file_set_rate_limit(s->to_dst_file,
|
qemu_file_set_rate_limit(s->to_dst_file,
|
||||||
s->parameters.max_bandwidth / XFER_LIMIT_RATIO);
|
s->parameters.max_bandwidth / XFER_LIMIT_RATIO);
|
||||||
}
|
}
|
||||||
@ -1370,6 +1370,10 @@ static void migrate_params_apply(MigrateSetParameters *params, Error **errp)
|
|||||||
}
|
}
|
||||||
if (params->has_max_postcopy_bandwidth) {
|
if (params->has_max_postcopy_bandwidth) {
|
||||||
s->parameters.max_postcopy_bandwidth = params->max_postcopy_bandwidth;
|
s->parameters.max_postcopy_bandwidth = params->max_postcopy_bandwidth;
|
||||||
|
if (s->to_dst_file && migration_in_postcopy()) {
|
||||||
|
qemu_file_set_rate_limit(s->to_dst_file,
|
||||||
|
s->parameters.max_postcopy_bandwidth / XFER_LIMIT_RATIO);
|
||||||
|
}
|
||||||
}
|
}
|
||||||
if (params->has_max_cpu_throttle) {
|
if (params->has_max_cpu_throttle) {
|
||||||
s->parameters.max_cpu_throttle = params->max_cpu_throttle;
|
s->parameters.max_cpu_throttle = params->max_cpu_throttle;
|
||||||
|
Loading…
Reference in New Issue
Block a user