[libvirt] [PATCH 1/2] BlockJob: Bandwidth parameter is in MB when using text monitor

[Daniel P. Berrange]

On Wed, Aug 31, 2011 at 05:15:43PM -0500, Adam Litke wrote:
> Due to an unfortunate precedent in qemu, the units for the bandwidth parameter
> to block_job_set_speed are different between the text monitor and the qmp
> monitor.  While the qmp monitor uses bytes/s, the text monitor expects MB/s.
> 
> Correct the units for the text interface.
> 
> Signed-off-by: Adam Litke <agl at us.ibm.com>
> ---
>  src/qemu/qemu_monitor_text.c |    3 +--
>  1 files changed, 1 insertions(+), 2 deletions(-)
> 
> diff --git a/src/qemu/qemu_monitor_text.c b/src/qemu/qemu_monitor_text.c
> index f37c98c..854ee7f 100644
> --- a/src/qemu/qemu_monitor_text.c
> +++ b/src/qemu/qemu_monitor_text.c
> @@ -3067,8 +3067,7 @@ int qemuMonitorTextBlockJob(qemuMonitorPtr mon,
>          ret = virAsprintf(&cmd, "%s", cmd_name);
>      } else if (mode == BLOCK_JOB_SPEED) {
>          cmd_name = "block_job_set_speed";
> -        ret = virAsprintf(&cmd, "%s %s %llu", cmd_name, device,
> -                          bandwidth * 1024ULL * 1024ULL);
> +        ret = virAsprintf(&cmd, "%s %s %luM", cmd_name, device, bandwidth);
>      } else if (mode == BLOCK_JOB_PULL) {
>          cmd_name = "block_stream";
>          ret = virAsprintf(&cmd, "%s %s", cmd_name, device);

ACK


Daniel
-- 
|: http://berrange.com      -o-    http://www.flickr.com/photos/dberrange/ :|
|: http://libvirt.org              -o-             http://virt-manager.org :|
|: http://autobuild.org       -o-         http://search.cpan.org/~danberr/ :|
|: http://entangle-photo.org       -o-       http://live.gnome.org/gtk-vnc :|