On 09/28/10 12:08, Markus Armbruster wrote: > jes.soren...@redhat.com writes: >> diff --git a/qemu-monitor.hx b/qemu-monitor.hx >> index 49bcd8d..7f58fb2 100644 >> --- a/qemu-monitor.hx >> +++ b/qemu-monitor.hx >> @@ -1093,7 +1093,7 @@ EQMP >> >> { >> .name = "migrate_set_speed", >> - .args_type = "value:f", >> + .args_type = "value:o", >> .params = "value", >> .help = "set maximum speed (in bytes) for migrations", >> .user_print = monitor_user_noop, > > Doesn't this change the interpretation of "42" from 42 to (42 << 20)?
It was always so on the command line that a number without a specifier meant MB. If the monitor defaulted to bytes it will get ugly to support both defaults in common code. Cheers, Jes