[libvirt] [PATCH v7 4/7] qemu: split qemuDomainMemoryStats into internal and external functions
Pavel Hrdina
phrdina at redhat.com
Wed Jul 27 10:21:21 UTC 2016
On Wed, Jul 13, 2016 at 01:42:15PM +0300, Derbyshev Dmitriy wrote:
> From: Derbyshev Dmitry <dderbyshev at virtuozzo.com>
>
> Is necessary to call it from other contexts, such as qemuDomainGetStatsBalloon.
>
> Signed-off-by: Derbyshev Dmitry <dderbyshev at virtuozzo.com>
> ---
> src/qemu/qemu_driver.c | 55 ++++++++++++++++++++++++++++++++------------------
> 1 file changed, 35 insertions(+), 20 deletions(-)
>
> diff --git a/src/qemu/qemu_driver.c b/src/qemu/qemu_driver.c
> index f8d9afe..6fa8d01 100644
> --- a/src/qemu/qemu_driver.c
> +++ b/src/qemu/qemu_driver.c
> @@ -10959,32 +10959,22 @@ qemuDomainGetInterfaceParameters(virDomainPtr dom,
> return ret;
> }
>
> +/* This functions assumes that job QEMU_JOB_QUERY is started by a caller */
> +
This was already pointed out, we don't put a new line between function and a
comment for that function.
> static int
> -qemuDomainMemoryStats(virDomainPtr dom,
> - virDomainMemoryStatPtr stats,
> - unsigned int nr_stats,
> - unsigned int flags)
> +qemuDomainMemoryStatsInternal(virQEMUDriverPtr driver,
> + virDomainObjPtr vm,
> + virDomainMemoryStatPtr stats,
> + unsigned int nr_stats)
> +
> {
> - virQEMUDriverPtr driver = dom->conn->privateData;
> - virDomainObjPtr vm;
> int ret = -1;
> long rss;
>
ACK
More information about the libvir-list
mailing list