proxmox-mirrors/qemu-server
1
0
Fork 0
mirror of https://git.proxmox.com/git/qemu-server synced 2025-11-04 11:28:38 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

Use default values when memory is not set in vm.conf when migrating

Browse Source 
This fixes a "Use of uninitialized value in multiplication (*) "
warning when doing a migration
This commit is contained in:
Emmanuel Kasper 2017-07-03 13:58:14 +02:00 committed by Dietmar Maurer
parent 9fea7ca3cf
commit 171ed95c76
1 changed files with 4 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
PVE/QemuMigrate.pm
View File

@ -640,8 +640,9 @@ sub phase2 {
};
warn $@ if $@;
#set cachesize 10% of the total memory
my $cachesize = int($conf->{memory}*1048576/10);
# set cachesize to 10% of the total memory
my $memory = $conf->{memory} || $defaults->{memory};
my $cachesize = int($memory * 1048576 / 10);
$self->log('info', "set cachesize: $cachesize");
eval {
PVE::QemuServer::vm_mon_cmd_nocheck($vmid, "migrate-set-cache-size", value => int($cachesize));
@ -712,7 +713,7 @@ sub phase2 {
if ($stat->{status} eq 'completed') {
my $delay = time() - $start;
if ($delay > 0) {
my $mbps = sprintf "%.2f", $conf->{memory}/$delay;
my $mbps = sprintf "%.2f", $memory / $delay;
my $downtime = $stat->{downtime} || 0;
$self->log('info', "migration speed: $mbps MB/s - downtime $downtime ms");
}