qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [PATCH 05/17] monitor: Fix do_info_balloon() output


From: Anthony Liguori
Subject: Re: [Qemu-devel] [PATCH 05/17] monitor: Fix do_info_balloon() output
Date: Mon, 07 Dec 2009 13:41:08 -0600
User-agent: Thunderbird 2.0.0.23 (X11/20090825)

Luiz Capitulino wrote:
New monitor commands should always return values in bytes.

Signed-off-by: Luiz Capitulino <address@hidden>
---
 monitor.c |   12 +++++++++---
 1 files changed, 9 insertions(+), 3 deletions(-)

diff --git a/monitor.c b/monitor.c
index 5bf32f0..8cedfa9 100644
--- a/monitor.c
+++ b/monitor.c
@@ -1889,12 +1889,18 @@ static void do_balloon(Monitor *mon, const QDict 
*qdict, QObject **ret_data)
static void monitor_print_balloon(Monitor *mon, const QObject *data)
 {
-    monitor_printf(mon, "balloon: actual=%d\n",
-                                     (int)qint_get_int(qobject_to_qint(data)));
+    monitor_printf(mon, "balloon: actual=%" PRId64 "\n",
+                        qint_get_int(qobject_to_qint(data)) >> 20);
 }
/**
  * do_info_balloon(): Balloon information
+ *
+ * Return a QInt with current ballooning value.
+ *
+ * Example:
+ *
+ * 1073741824
  */
 static void do_info_balloon(Monitor *mon, QObject **ret_data)
 {
@@ -1906,7 +1912,7 @@ static void do_info_balloon(Monitor *mon, QObject 
**ret_data)
     else if (actual == 0)
         qemu_error_new(QERR_DEVICE_NOT_ACTIVE, "balloon");
     else
-        *ret_data = QOBJECT(qint_from_int((int)(actual >> 20)));
+        *ret_data = QOBJECT(qint_from_int(actual));
 }

Returning a dictionary would allow us to extend this information more easily.

Regards,

Anthony Liguori




reply via email to

[Prev in Thread] Current Thread [Next in Thread]