Unifying units in memory setting and reporting

[Martin Sivak]

Hi,

we have an issue with reporting data over xml-rpc. It supports only 32bit integers and we ran into the limitation in https://bugzilla.redhat.com/show_bug.cgi?id=974917

It is probably caused by us using KiB units when reporting balloon memory (the limit is 2047 GiB which is high but reachable).

According to my understanding of the sources we already use MiB when creating the VM and so I propose we unify that and use megabytes in reporting as well. This will probably require change to all parts: VDSM, MoM and engine.

I have created a bug to track it: https://bugzilla.redhat.com/show_bug.cgi?id=975945

Regarding implementation, please do not be fooled by the variable names balloon_max/balloon_cur actually hold the maximum and current amount of memory available to the VM. We (me and Doron) were confused by it this morning while investigating for another SLA feature regarding minimal guaranteed memory.

Regards

--
Martin Sivák
msivak at redhat.com
Red Hat Czech
RHEV-M SLA / Brno, CZ