[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2] Move migrate_set_speed and migrate_set_downt
From: |
Daniel P. Berrange |
Subject: |
Re: [Qemu-devel] [PATCH v2] Move migrate_set_speed and migrate_set_downtime into migrate_set_parameter |
Date: |
Mon, 5 Sep 2016 16:56:02 +0100 |
User-agent: |
Mutt/1.7.0 (2016-08-17) |
On Mon, Sep 05, 2016 at 03:16:04PM +0200, Paolo Bonzini wrote:
>
>
> On 05/09/2016 15:09, Ashijeet Acharya wrote:
> >>> >> +
> >>> >> "compress-level:i?,compress-threads:i?,decompress-threads:i?,cpu-throttle-initial:i?,cpu-throttle-increment:i?,migrate-set-speed:o?,migrate-set-downtime:T?",
> >> >
> >> > Same here about the names. Also use "i" for QMP commands.
> > I think we will have to use "T" for downtime at-least otherwise you
> > cant use float values for setting seconds like "0.2" for example.
> > No issues using "i" with bandwidth though.
>
> Right, I should have mentioned that my remark was about the "o" (it
> caught my eye because of your change to hmp-commands.hx). Sorry!
I've always thought it a rather wierd to use fractional seconds for
downtime. It sort of made sense for HMP, but should really never have
been carried over into QMP. IMHO, we should make it just take integer
milliseconds for downtime with the new API.
Regards,
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 :|
- Re: [Qemu-devel] [PATCH v2] Move migrate_set_speed and migrate_set_downtime into migrate_set_parameter, (continued)
- Re: [Qemu-devel] [PATCH v2] Move migrate_set_speed and migrate_set_downtime into migrate_set_parameter, Ashijeet Acharya, 2016/09/05
- Re: [Qemu-devel] [PATCH v2] Move migrate_set_speed and migrate_set_downtime into migrate_set_parameter, Paolo Bonzini, 2016/09/05
- [Qemu-devel] [PATCH v3] Move migrate_set_speed and migrate_set_downtime into migrate_set_parameter, Ashijeet Acharya, 2016/09/05
- Re: [Qemu-devel] [PATCH v3] Move migrate_set_speed and migrate_set_downtime into migrate_set_parameter, Daniel P. Berrange, 2016/09/05
- Re: [Qemu-devel] [PATCH v3] Move migrate_set_speed and migrate_set_downtime into migrate_set_parameter, Ashijeet Acharya, 2016/09/05
- Re: [Qemu-devel] [PATCH v3] Move migrate_set_speed and migrate_set_downtime into migrate_set_parameter, Daniel P. Berrange, 2016/09/05
- Re: [Qemu-devel] [PATCH v3] Move migrate_set_speed and migrate_set_downtime into migrate_set_parameter, Ashijeet Acharya, 2016/09/05
- [Qemu-devel] [PATCH v3] Move migrate_set_speed and migrate_set_downtime into migrate_set_parameter (reversed logic), Ashijeet Acharya, 2016/09/05
- Re: [Qemu-devel] [PATCH v2] Move migrate_set_speed and migrate_set_downtime into migrate_set_parameter,
Daniel P. Berrange <=
- Re: [Qemu-devel] [PATCH v2] Move migrate_set_speed and migrate_set_downtime into migrate_set_parameter, Markus Armbruster, 2016/09/06
- Re: [Qemu-devel] [PATCH] Move migrate_set_speed and migrate_set_downtime into migrate_set_parameter, Juan Quintela, 2016/09/06