[libvirt] [PATCH] Revert "LXC: show used memory as 0 when domain is not active"
Jim Fehlig
jfehlig at suse.com
Thu Aug 27 22:38:24 UTC 2015
Daniel P. Berrange wrote:
> On Tue, Aug 11, 2015 at 10:25:19AM +0200, Peter Krempa wrote:
>
>> On Tue, Aug 11, 2015 at 09:18:23 +0100, Daniel Berrange wrote:
>>
>>> On Mon, Aug 10, 2015 at 12:49:55PM -0600, Jim Fehlig wrote:
>>>
>>>> This reverts commit 1ce7c1d20cfd5afb26d2dbc88201085d52415d0e,
>>>> which introduced a significant semantic change to the
>>>> virDomainGetInfo() API. Additionally, the change was only
>>>> made to 2 of the 15 virt drivers.
>>>>
>>>> Conflicts:
>>>> src/qemu/qemu_driver.c
>>>>
>>>> Signed-off-by: Jim Fehlig <jfehlig at suse.com>
>>>> ---
>>>> src/lxc/lxc_driver.c | 2 +-
>>>> src/qemu/qemu_driver.c | 12 ++++++------
>>>> 2 files changed, 7 insertions(+), 7 deletions(-)
>>>>
>>> ACK
>>>
>>> I guess we should probably do this in the stable branch too, since I
>>> think it made it into the most recent release.
>>>
>> Erm, the change is out for a while now:
>>
>> git desc 1ce7c1d20cfd5afb26d2dbc88201085d52415d0e
>> v1.2.6-225-g1ce7c1d
>>
>
> Oh yes, so it is. I'm still inclined to say we should be reverting it as
> I think it is wrong. The change was based on the misleading field name
> shown by virsh. The info->memory field shows the current balloon target,
> and conceptually this should be equal to the max memory if the ballooon
> driver is not active. As such I think it should be equal to max memory
> if shutoff too.
>
Yikes, I forgot to commit this before leaving for travel/vacation. Is it
still ok to commit to master? And just to be clear, then cherry pick to
1.2.7-1.2.18 maint branches?
Regards,
Jim
More information about the libvir-list
mailing list