[Kimchi-devel] [PATCH][Kimchi] Fix memory value return when hotplug memory devs

[Rodrigo Trujillo]

There is an issue when Kimchi returns the value of memory. It is
returning the current memory value, instead of the total memory, which
includes the value of the memory devices added.
This patch fix this problem returning the total value from "dom.info".

Signed-off-by: Rodrigo Trujillo <rodrigo.trujillo at linux.vnet.ibm.com>
---
 model/vms.py | 12 +++++++++++-
 1 file changed, 11 insertions(+), 1 deletion(-)

diff --git a/model/vms.py b/model/vms.py
index e3bc266..396758d 100644
--- a/model/vms.py
+++ b/model/vms.py
@@ -1228,7 +1228,17 @@ class VMModel(object):
                 unit = 'KiB'
             memory = convert_data_size(val, unit, 'MiB')
         else:
-            memory = info[2] >> 10
+            # If guest is running we need information in real time, in order
+            # to get real value if memory devices were added:
+            #
+            #   Class Libvirt::Domain::Info
+            #     cpu_time[R]
+            #     max_mem[R]  -> current memory plus  memory devs (use it)
+            #     memory[R]   -> current memory only, does not include mem devs
+            #     nr_virt_cpu[R]
+            #     state[R]
+            #
+            memory = info[1] >> 10
 
         # assure there is no zombie process left
         for proc in self._serial_procs[:]:
-- 
2.1.0