[Top][All Lists]

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

Re: [Qemu-devel] [PATCH 05/10] qga: introduce three guest memory block c

From: zhanghailiang
Subject: Re: [Qemu-devel] [PATCH 05/10] qga: introduce three guest memory block commmands with stubs
Date: Wed, 25 Feb 2015 10:46:37 +0800
User-agent: Mozilla/5.0 (Windows NT 6.1; rv:31.0) Gecko/20100101 Thunderbird/31.1.1

On 2015/2/18 2:10, Michael Roth wrote:
Quoting Eric Blake (2015-02-17 09:26:12)
On 02/16/2015 08:14 PM, Michael Roth wrote:
From: zhanghailiang <address@hidden>

Introduce three new guest commands:
guest-get-memory-blocks, guest-set-memory-blocks, guest-get-memory-block-size.

Sorry for the late review, but I think guest-get-memory-block-size is
the wrong command to add.

+# @guest-get-memory-block-size:
+# Get the the size (in bytes) of a memory block in guest.
+# It is the unit of memory block online/offline operation (also called Logical
+# Memory Hotplug).
+# Returns: memory block size in bytes.
+# Since 2.3
+{ 'command': 'guest-get-memory-block-size',
+  'returns': 'int' }

Any QAPI command that returns a bare int instead of a dictionary is
non-extensible, and therefore of suspect design.  I think it would be
better to have:

{ 'command': 'guest-get-memory-block-info',
   'returns': { 'size': 'int' } }

to allow for future extension.

It seems like a reasonable suggestion to me. I can change it in my
tree if there are no objections. zhanghailiang?

OK, you are free to do that, please. Thanks ;)

Eric Blake   eblake redhat com    +1-919-301-3266
Libvirt virtualization library http://libvirt.org

reply via email to

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