]> git.proxmox.com Git - qemu-server.git/commitdiff
correctly handle undef values when calling qemu_block_set_io_throttle
authorDietmar Maurer <dietmar@proxmox.com>
Wed, 12 Jun 2013 05:08:02 +0000 (07:08 +0200)
committerDietmar Maurer <dietmar@proxmox.com>
Wed, 12 Jun 2013 05:08:02 +0000 (07:08 +0200)
PVE/API2/Qemu.pm
PVE/QemuServer.pm

index 6651d16723b26a3f71993f95888279ad2fcbd619..076356c7f266ee9bddc0e71ada73a3d8588f6bbe 100644 (file)
@@ -766,9 +766,13 @@ my $vmconfig_update_disk = sub {
                &$safe_num_ne($drive->{iops}, $old_drive->{iops}) ||
                &$safe_num_ne($drive->{iops_rd}, $old_drive->{iops_rd}) ||
                &$safe_num_ne($drive->{iops_wr}, $old_drive->{iops_wr})) {
-               PVE::QemuServer::qemu_block_set_io_throttle($vmid,"drive-$opt", $drive->{mbps}*1024*1024,
-                                                          $drive->{mbps_rd}*1024*1024, $drive->{mbps_wr}*1024*1024,
-                                                          $drive->{iops}, $drive->{iops_rd}, $drive->{iops_wr})
+               PVE::QemuServer::qemu_block_set_io_throttle($vmid,"drive-$opt", 
+                                                          ($drive->{mbps} || 0)*1024*1024,
+                                                          ($drive->{mbps_rd} || 0)*1024*1024, 
+                                                          ($drive->{mbps_wr} || 0)*1024*1024,
+                                                          $drive->{iops} || 0, 
+                                                          $drive->{iops_rd} || 0, 
+                                                          $drive->{iops_wr} || 0)
                   if !PVE::QemuServer::drive_is_cdrom($drive);
             }
        }
index d669d6bea7d2149cf976ddd9eccde48c47d86da4..d2b942fec0d8364361639b611bfec3a834a66bb9 100644 (file)
@@ -2803,13 +2803,6 @@ sub qemu_block_set_io_throttle {
 
     return if !check_running($vmid) ;
 
-    $bps = 0 if !$bps;
-    $bps_rd = 0 if !$bps_rd;
-    $bps_wr = 0 if !$bps_wr;
-    $iops = 0 if !$iops;
-    $iops_rd = 0 if !$iops_rd;
-    $iops_wr = 0 if !$iops_wr;
-
     vm_mon_cmd($vmid, "block_set_io_throttle", device => $deviceid, bps => int($bps), bps_rd => int($bps_rd), bps_wr => int($bps_wr), iops => int($iops), iops_rd => int($iops_rd), iops_wr => int($iops_wr));
 
 }