"Dr. David Alan Gilbert" <dgilb...@redhat.com> writes: > * Markus Armbruster (arm...@redhat.com) wrote: >> The previous commit made it unsigned in QMP. Switch HMP's args_type >> from 'M' to 'o'. Loses support for expressions (QEMU pocket >> calculator), gains support for units other than mebibytes. Negative >> values are no longer accepted and interpreted modulo 2^64. Instead, >> values between 2^63 and 2^64-1 are now accepted. >> >> Signed-off-by: Markus Armbruster <arm...@redhat.com> >> --- >> hmp-commands.hx | 2 +- >> hmp.c | 4 ++-- >> 2 files changed, 3 insertions(+), 3 deletions(-) >> >> diff --git a/hmp-commands.hx b/hmp-commands.hx >> index 1941e19..46ce79c 100644 >> --- a/hmp-commands.hx >> +++ b/hmp-commands.hx >> @@ -1433,7 +1433,7 @@ ETEXI >> >> { >> .name = "balloon", >> - .args_type = "value:M", >> + .args_type = "value:o", >> .params = "target", >> .help = "request VM to change its memory allocation (in MB)", >> .cmd = hmp_balloon, >> diff --git a/hmp.c b/hmp.c >> index 4556045..1932a11 100644 >> --- a/hmp.c >> +++ b/hmp.c >> @@ -781,7 +781,7 @@ void hmp_info_balloon(Monitor *mon, const QDict *qdict) >> return; >> } >> >> - monitor_printf(mon, "balloon: actual=%" PRIu64 "\n", info->actual >> >> 20); >> + monitor_printf(mon, "balloon: actual=%" PRId64 "\n", info->actual >> >> 20); > > That looks like a partial reversion of the last patch ?
Accident, will fix, thanks! [...]