On Wed, Mar 02, 2011 at 05:07:48PM +0900, Taku Izumi wrote:

This patch introduces a new libvirt API (virDomainSetMemoryFlags) and a flag (virDomainMemoryModFlags).

Signed-off-by: Taku Izumi <izumi.taku@jp.fujitsu.com> --- include/libvirt/libvirt.h.in | 10 ++++++ src/libvirt.c | 62 +++++++++++++++++++++++++++++++++++++++++++ src/libvirt_public.syms | 5 +++ 3 files changed, 77 insertions(+)

Index: libvirt-git/include/libvirt/libvirt.h.in =================================================================== --- libvirt-git.orig/include/libvirt/libvirt.h.in +++ libvirt-git/include/libvirt/libvirt.h.in @@ -780,6 +780,13 @@ int virDomainGetMemoryParameters(vir virMemoryParameterPtr params, int *nparams, unsigned int flags);

+/* Memory size modification flags. */ +typedef enum { + VIR_DOMAIN_MEM_LIVE = (1 << 0), /* affect active domain */ + VIR_DOMAIN_MEM_CONFIG = (1 << 1), /* affect next boot */ +} virDomainMemoryModFlags; + + /* * Dynamic control of domains */ @@ -795,6 +802,9 @@ int virDomainSetMaxM unsigned long memory); int virDomainSetMemory (virDomainPtr domain, unsigned long memory); +int virDomainSetMemoryFlags (virDomainPtr domain, + unsigned long memory, + unsigned int flags); int virDomainGetMaxVcpus (virDomainPtr domain); int virDomainGetSecurityLabel (virDomainPtr domain, virSecurityLabelPtr seclabel); Index: libvirt-git/src/libvirt_public.syms =================================================================== --- libvirt-git.orig/src/libvirt_public.syms +++ libvirt-git/src/libvirt_public.syms @@ -424,4 +424,9 @@ LIBVIRT_0.8.8 { virConnectGetSysinfo; } LIBVIRT_0.8.6;

+LIBVIRT_0.8.9 { + global: + virDomainSetMemoryFlags; +} LIBVIRT_0.8.8;

This bit will need re-basing since I added the 0.9.0 section to the file in a recent commit.

+ # .... define new API here using predicted next version number .... Index: libvirt-git/src/libvirt.c =================================================================== --- libvirt-git.orig/src/libvirt.c +++ libvirt-git/src/libvirt.c @@ -2846,6 +2846,68 @@ error: return -1; }

+/* + * virDomainSetMemoryFlags + * @domain: a domain object or NULL + * @memory: the memory size in kilobytes + * @flags: an OR'ed set of virDomainMemoryFlags + * + * Dynamically change the target amount of physical memory allocated to a + * domain. If domain is NULL, then this change the amount of memory reserved + * to Domain0 i.e. the domain where the application runs. + * This funcation may requires privileged access to the hypervisor. + * + * @flags must include VIR_DOMAIN_MEM_LIVE to affect a running + * domain (which may fail if domain is not active), or + * VIR_DOMAIN_MEM_CONFIG to affect the next boot via the XML + * description of the domain. Both flags may be set. + * + * Returns 0 in case of success, -1 in case of failure. + */ + +int +virDomainSetMemoryFlags(virDomainPtr domain, unsigned long memory, + unsigned int flags) +{ + virConnectPtr conn; + + VIR_DOMAIN_DEBUG(domain, "memory=%lu flags=%u", memory, flags); + + virResetLastError(); + + if (!VIR_IS_CONNECTED_DOMAIN(domain)) { + virLibDomainError(VIR_ERR_INVALID_DOMAIN, __FUNCTION__); + virDispatchError(NULL); + return -1; + } + + if (domain->conn->flags & VIR_CONNECT_RO) { + virLibDomainError(VIR_ERR_OPERATION_DENIED, __FUNCTION__); + goto error; + } + + if (memory < 4096 || + (flags & (VIR_DOMAIN_MEM_LIVE | VIR_DOMAIN_MEM_CONFIG)) == 0) { + virLibDomainError(VIR_ERR_INVALID_ARG, __FUNCTION__); + goto error; + } + + conn = domain->conn; + + if (conn->driver->domainSetMemoryFlags) { + int ret; + ret = conn->driver->domainSetMemoryFlags(domain, memory, flags); + if (ret < 0) + goto error; + return ret; + }

This bit is referencing a struct field that is only created in patch 2, so you should merge the contents of patch 2 into this patch. The goal is that everything should compile after each individual patch is applied The API proposal looks fine to me Regards, Daniel -- |: http://berrange.com -o- http://www.flickr.com/photos/dberrange/ :| |: http://libvirt.org -o- http://virt-manager.org :| |: http://autobuild.org -o- http://search.cpan.org/~danberr/ :| |: http://entangle-photo.org -o- http://live.gnome.org/gtk-vnc :|

On Wed, Mar 02, 2011 at 05:13:09PM +0900, Taku Izumi wrote:

This patch implements the code to address the new API in the qemu driver.

Signed-off-by: Taku Izumi <izumi.taku@jp.fujitsu.com> --- src/qemu/qemu_driver.c | 64 +++++++++++++++++++++++++++++++++++++++---------- 1 file changed, 51 insertions(+), 13 deletions(-)

Index: libvirt-git/src/qemu/qemu_driver.c =================================================================== --- libvirt-git.orig/src/qemu/qemu_driver.c +++ libvirt-git/src/qemu/qemu_driver.c @@ -1569,12 +1569,22 @@ cleanup: return ret; }

-static int qemudDomainSetMemory(virDomainPtr dom, unsigned long newmem) { +static int qemudDomainSetMemoryFlags(virDomainPtr dom, unsigned long newmem, + unsigned int flags) { struct qemud_driver *driver = dom->conn->privateData; qemuDomainObjPrivatePtr priv; virDomainObjPtr vm; + virDomainDefPtr persistentDef; int ret = -1, r;

+ virCheckFlags(VIR_DOMAIN_MEM_LIVE | + VIR_DOMAIN_MEM_CONFIG, -1); + + if ((flags & (VIR_DOMAIN_MEM_LIVE | VIR_DOMAIN_MEM_CONFIG)) == 0) { + qemuReportError(VIR_ERR_INVALID_ARG, + _("invalid flag combination: (0x%x)"), flags); + }

I think you forgot a 'return -1' statement just here.

+ qemuDriverLock(driver); vm = virDomainFindByUUID(&driver->domains, dom->uuid); qemuDriverUnlock(driver); @@ -1595,27 +1605,51 @@ static int qemudDomainSetMemory(virDomai if (qemuDomainObjBeginJob(vm) < 0) goto cleanup;

- if (!virDomainObjIsActive(vm)) { + if (!virDomainObjIsActive(vm) && (flags & VIR_DOMAIN_MEM_LIVE)) { qemuReportError(VIR_ERR_OPERATION_INVALID, "%s", _("domain is not running")); goto endjob; }

- priv = vm->privateData; - qemuDomainObjEnterMonitor(vm); - r = qemuMonitorSetBalloon(priv->mon, newmem); - qemuDomainObjExitMonitor(vm); - if (r < 0) + if (!vm->persistent && (flags & VIR_DOMAIN_MEM_CONFIG)) { + qemuReportError(VIR_ERR_OPERATION_INVALID, "%s", + _("cannot change persistent config of a transient domain")); goto endjob; + }

- /* Lack of balloon support is a fatal error */ - if (r == 0) { - qemuReportError(VIR_ERR_OPERATION_INVALID, - "%s", _("cannot set memory of an active domain")); + if (!(persistentDef = virDomainObjGetPersistentDef(driver->caps, vm))) goto endjob; + + switch (flags) { + case VIR_DOMAIN_MEM_CONFIG: + persistentDef->mem.cur_balloon = newmem; + ret = 0; + break; + + case VIR_DOMAIN_MEM_LIVE: + case VIR_DOMAIN_MEM_LIVE | VIR_DOMAIN_MEM_CONFIG: + priv = vm->privateData; + qemuDomainObjEnterMonitor(vm); + r = qemuMonitorSetBalloon(priv->mon, newmem); + qemuDomainObjExitMonitor(vm); + if (r < 0) + goto endjob; + /* Lack of balloon support is a fatal error */ + if (r == 0) { + qemuReportError(VIR_ERR_OPERATION_INVALID, + "%s", _("cannot set memory of an active domain")); + goto endjob; + } + if (flags & VIR_DOMAIN_MEM_CONFIG) + persistentDef->mem.cur_balloon = newmem; + ret = 0; + break; }

I think it is a little wierd to use a 'switch' statement for processing this. I'd just have a pair of 'if' statements eg if (flags & VIR_DOMAIN_MEM_LIVE) { .....do monitor stuff.... } if (flags & VIR_DOMAIN_MEM_CONFIG) { persistentDef->mem.cur_balloon = newmem; ret = virDomainSaveConfig(driver->configDir, persistentDef); } Regards, Daniel -- |: http://berrange.com -o- http://www.flickr.com/photos/dberrange/ :| |: http://libvirt.org -o- http://virt-manager.org :| |: http://autobuild.org -o- http://search.cpan.org/~danberr/ :| |: http://entangle-photo.org -o- http://live.gnome.org/gtk-vnc :|

On 03/02/2011 01:13 AM, Taku Izumi wrote:

This patch implements the code to address the new API in the qemu driver.

+ if (!vm->persistent && (flags & VIR_DOMAIN_MEM_CONFIG)) { + qemuReportError(VIR_ERR_OPERATION_INVALID, "%s", + _("cannot change persistent config of a transient domain")); goto endjob; + }

+ if (!(persistentDef = virDomainObjGetPersistentDef(driver->caps, vm))) goto endjob;

Doesn't that fail for a transient domain? That is, shouldn't the persistentDef only be set when flags & VIR_DOMAIN_MEM_CONFIG is requested and validated?

+ + switch (flags) { + case VIR_DOMAIN_MEM_CONFIG: + persistentDef->mem.cur_balloon = newmem; + ret = 0; + break; + + case VIR_DOMAIN_MEM_LIVE: + case VIR_DOMAIN_MEM_LIVE | VIR_DOMAIN_MEM_CONFIG: + priv = vm->privateData; + qemuDomainObjEnterMonitor(vm); + r = qemuMonitorSetBalloon(priv->mon, newmem);

Merge conflict with my audit patches. Plus Daniel's request to rearrange the switch statement into if statements.

+ qemuDomainObjExitMonitor(vm);

@@ -6789,7 +6827,7 @@ static virDriver qemuDriver = { qemudDomainGetMaxMemory, /* domainGetMaxMemory */ NULL, /* domainSetMaxMemory */ qemudDomainSetMemory, /* domainSetMemory */ - NULL, /* domainSetMemoryFlags */ + qemudDomainSetMemoryFlags, /* domainSetMemoryFlags */

Hmm, for other API additions, I have provided dummy wrappers for the new function anywhere the old function existed, that merely require the same flags as the old function would use. I'll probably propose that as a followup patch. Here's what I ended up with before pushing: diff --git i/src/qemu/qemu_driver.c w/src/qemu/qemu_driver.c index 2f15144..7921d98 100644 --- i/src/qemu/qemu_driver.c +++ w/src/qemu/qemu_driver.c @@ -1568,12 +1568,22 @@ cleanup: return ret; } -static int qemudDomainSetMemory(virDomainPtr dom, unsigned long newmem) { +static int qemudDomainSetMemoryFlags(virDomainPtr dom, unsigned long newmem, + unsigned int flags) { struct qemud_driver *driver = dom->conn->privateData; qemuDomainObjPrivatePtr priv; virDomainObjPtr vm; + virDomainDefPtr persistentDef = NULL; int ret = -1, r; + virCheckFlags(VIR_DOMAIN_MEM_LIVE | + VIR_DOMAIN_MEM_CONFIG, -1); + + if ((flags & (VIR_DOMAIN_MEM_LIVE | VIR_DOMAIN_MEM_CONFIG)) == 0) { + qemuReportError(VIR_ERR_INVALID_ARG, + _("invalid flag combination: (0x%x)"), flags); + } + qemuDriverLock(driver); vm = virDomainFindByUUID(&driver->domains, dom->uuid); qemuDriverUnlock(driver); @@ -1594,24 +1604,42 @@ static int qemudDomainSetMemory(virDomainPtr dom, unsigned long newmem) { if (qemuDomainObjBeginJob(vm) < 0) goto cleanup; - if (!virDomainObjIsActive(vm)) { + if (!virDomainObjIsActive(vm) && (flags & VIR_DOMAIN_MEM_LIVE)) { qemuReportError(VIR_ERR_OPERATION_INVALID, "%s", _("domain is not running")); goto endjob; } - priv = vm->privateData; - qemuDomainObjEnterMonitor(vm); - r = qemuMonitorSetBalloon(priv->mon, newmem); - qemuDomainObjExitMonitor(vm); - qemuAuditMemory(vm, vm->def->mem.cur_balloon, newmem, "update", r == 1); - if (r < 0) - goto endjob; + if (flags & VIR_DOMAIN_MEM_CONFIG) { + if (!vm->persistent) { + qemuReportError(VIR_ERR_OPERATION_INVALID, "%s", + _("cannot change persistent config of a transient domain")); + goto endjob; + } + if (!(persistentDef = virDomainObjGetPersistentDef(driver->caps, vm))) + goto endjob; + } - /* Lack of balloon support is a fatal error */ - if (r == 0) { - qemuReportError(VIR_ERR_OPERATION_INVALID, - "%s", _("cannot set memory of an active domain")); + if (flags & VIR_DOMAIN_MEM_LIVE) { + priv = vm->privateData; + qemuDomainObjEnterMonitor(vm); + r = qemuMonitorSetBalloon(priv->mon, newmem); + qemuDomainObjExitMonitor(vm); + qemuAuditMemory(vm, vm->def->mem.cur_balloon, newmem, "update", r == 1); + if (r < 0) + goto endjob; + + /* Lack of balloon support is a fatal error */ + if (r == 0) { + qemuReportError(VIR_ERR_OPERATION_INVALID, + "%s", _("cannot set memory of an active domain")); + goto endjob; + } + } + + if (flags& VIR_DOMAIN_MEM_CONFIG) { + persistentDef->mem.cur_balloon = newmem; + ret = virDomainSaveConfig(driver->configDir, persistentDef); goto endjob; } @@ -1626,6 +1654,10 @@ cleanup: return ret; } +static int qemudDomainSetMemory(virDomainPtr dom, unsigned long newmem) { + return qemudDomainSetMemoryFlags(dom, newmem, VIR_DOMAIN_MEM_LIVE); +} + static int qemudDomainGetInfo(virDomainPtr dom, virDomainInfoPtr info) { struct qemud_driver *driver = dom->conn->privateData; @@ -6854,7 +6886,7 @@ static virDriver qemuDriver = { qemudDomainGetMaxMemory, /* domainGetMaxMemory */ NULL, /* domainSetMaxMemory */ qemudDomainSetMemory, /* domainSetMemory */ - NULL, /* domainSetMemoryFlags */ + qemudDomainSetMemoryFlags, /* domainSetMemoryFlags */ qemudDomainGetInfo, /* domainGetInfo */ qemudDomainSave, /* domainSave */ qemudDomainRestore, /* domainRestore */ -- Eric Blake eblake@redhat.com +1-801-349-2682 Libvirt virtualization library http://libvirt.org

On 03/10/2011 03:53 PM, Eric Blake wrote:

* src/esx/esx_driver.c (esxDomainSetMemory): Move guts... (esxDomainSetMemoryFlags): ...here.

...

Hmm, for other API additions, I have provided dummy wrappers for the new function anywhere the old function existed, that merely require the same flags as the old function would use. I'll probably propose that as a followup patch.

Like so.

src/esx/esx_driver.c | 17 ++++++++++++++--- src/lxc/lxc_driver.c | 15 +++++++++++++-- src/test/test_driver.c | 19 ++++++++++++++++--- src/uml/uml_driver.c | 15 +++++++++++++-- src/vbox/vbox_tmpl.c | 15 +++++++++++++-- src/xen/xen_driver.c | 15 ++++++++++++++- 6 files changed, 83 insertions(+), 13 deletions(-)

diff --git a/src/esx/esx_driver.c b/src/esx/esx_driver.c index 37e104e..a2d8433 100644 --- a/src/esx/esx_driver.c +++ b/src/esx/esx_driver.c @@ -2111,7 +2111,8 @@ esxDomainSetMaxMemory(virDomainPtr domain, unsigned long memory)

static int -esxDomainSetMemory(virDomainPtr domain, unsigned long memory) +esxDomainSetMemoryFlags(virDomainPtr domain, unsigned long memory, + unsigned int flags)

I haven't revisited this patch since the rest of the setmem/setmaxmem changes went in, but we really should look at getting the rest of the hypervisors to support the new SetMemoryFlags API (even if only for the single flag combination that they previously supported). -- Eric Blake eblake@redhat.com +1-801-349-2682 Libvirt virtualization library http://libvirt.org

* src/Makefile.am (remote_protocol-structs): Flatten tabs. * src/remote_protocol-structs: Likewise. Also add a hint to emacs to make it easier to keep spaces in the file. ---

After fixing those, the patch still failed indentation in remote_protocol-structs. (Hmm - why do we create that file with TABs in the first place? I should fix that in a followup)

Here's the 'git diff -b' output, followed by the real patch. diff --git c/src/Makefile.am w/src/Makefile.am index 645119e..81f1a27 100644 --- c/src/Makefile.am +++ w/src/Makefile.am @@ -195,10 +195,14 @@ remote_protocol-structs: -e ' $$p =~ s!\t*/\*.*?\*/!!sg;' \ -e ' $$p =~ s!\s+\n!\n!sg;' \ -e ' $$p =~ s!\s+$$!!;' \ + -e ' $$p =~ s!\t! !g;' \ -e ' print "$$p\n";' \ -e ' $$n++;' \ -e ' }' \ -e '}' \ + -e 'BEGIN {' \ + -e ' print "/* -*- c -*- */\n";' \ + -e '}' \ -e 'END {' \ -e ' if ($$n < 300) {' \ -e ' warn "WARNING: your pdwtags program is too old\n";' \ diff --git c/src/remote_protocol-structs w/src/remote_protocol-structs index 6e131c8..726d0b0 100644 --- c/src/remote_protocol-structs +++ w/src/remote_protocol-structs @@ -1,3 +1,4 @@ +/* -*- c -*- */ struct remote_nonnull_domain { remote_nonnull_string name; remote_uuid uuid; src/Makefile.am | 4 + src/remote_protocol-structs | 1433 ++++++++++++++++++++++--------------------- 2 files changed, 721 insertions(+), 716 deletions(-) diff --git a/src/Makefile.am b/src/Makefile.am index 645119e..81f1a27 100644 --- a/src/Makefile.am +++ b/src/Makefile.am @@ -195,10 +195,14 @@ remote_protocol-structs: -e ' $$p =~ s!\t*/\*.*?\*/!!sg;' \ -e ' $$p =~ s!\s+\n!\n!sg;' \ -e ' $$p =~ s!\s+$$!!;' \ + -e ' $$p =~ s!\t! !g;' \ -e ' print "$$p\n";' \ -e ' $$n++;' \ -e ' }' \ -e '}' \ + -e 'BEGIN {' \ + -e ' print "/* -*- c -*- */\n";' \ + -e '}' \ -e 'END {' \ -e ' if ($$n < 300) {' \ -e ' warn "WARNING: your pdwtags program is too old\n";' \ diff --git a/src/remote_protocol-structs b/src/remote_protocol-structs index 6e131c8..726d0b0 100644 --- a/src/remote_protocol-structs +++ b/src/remote_protocol-structs @@ -1,1384 +1,1385 @@ +/* -*- c -*- */ struct remote_nonnull_domain { - remote_nonnull_string name; - remote_uuid uuid; - int id; + remote_nonnull_string name; + remote_uuid uuid; + int id; }; struct remote_nonnull_network { - remote_nonnull_string name; - remote_uuid uuid; + remote_nonnull_string name; + remote_uuid uuid; }; struct remote_nonnull_nwfilter { - remote_nonnull_string name; - remote_uuid uuid; + remote_nonnull_string name; + remote_uuid uuid; }; struct remote_nonnull_interface { - remote_nonnull_string name; - remote_nonnull_string mac; + remote_nonnull_string name; + remote_nonnull_string mac; }; struct remote_nonnull_storage_pool { - remote_nonnull_string name; - remote_uuid uuid; + remote_nonnull_string name; + remote_uuid uuid; }; struct remote_nonnull_storage_vol { - remote_nonnull_string pool; - remote_nonnull_string name; - remote_nonnull_string key; + remote_nonnull_string pool; + remote_nonnull_string name; + remote_nonnull_string key; }; struct remote_nonnull_node_device { - remote_nonnull_string name; + remote_nonnull_string name; }; struct remote_nonnull_secret { - remote_uuid uuid; - int usageType; - remote_nonnull_string usageID; + remote_uuid uuid; + int usageType; + remote_nonnull_string usageID; }; struct remote_nonnull_domain_snapshot { - remote_nonnull_string name; - remote_nonnull_domain domain; + remote_nonnull_string name; + remote_nonnull_domain domain; }; struct remote_error { - int code; - int domain; - remote_string message; - int level; - remote_domain dom; - remote_string str1; - remote_string str2; - remote_string str3; - int int1; - int int2; - remote_network net; + int code; + int domain; + remote_string message; + int level; + remote_domain dom; + remote_string str1; + remote_string str2; + remote_string str3; + int int1; + int int2; + remote_network net; }; struct remote_vcpu_info { - u_int number; - int state; - uint64_t cpu_time; - int cpu; + u_int number; + int state; + uint64_t cpu_time; + int cpu; }; struct remote_sched_param_value { - int type; - union { - int i; - u_int ui; - int64_t l; - uint64_t ul; - double d; - int b; - } remote_sched_param_value_u; + int type; + union { + int i; + u_int ui; + int64_t l; + uint64_t ul; + double d; + int b; + } remote_sched_param_value_u; }; struct remote_sched_param { - remote_nonnull_string field; - remote_sched_param_value value; + remote_nonnull_string field; + remote_sched_param_value value; }; struct remote_memory_param_value { - int type; - union { - int i; - u_int ui; - int64_t l; - uint64_t ul; - double d; - int b; - } remote_memory_param_value_u; + int type; + union { + int i; + u_int ui; + int64_t l; + uint64_t ul; + double d; + int b; + } remote_memory_param_value_u; }; struct remote_memory_param { - remote_nonnull_string field; - remote_memory_param_value value; + remote_nonnull_string field; + remote_memory_param_value value; }; struct remote_open_args { - remote_string name; - int flags; + remote_string name; + int flags; }; struct remote_supports_feature_args { - int feature; + int feature; }; struct remote_supports_feature_ret { - int supported; + int supported; }; struct remote_get_type_ret { - remote_nonnull_string type; + remote_nonnull_string type; }; struct remote_get_version_ret { - int64_t hv_ver; + int64_t hv_ver; }; struct remote_get_lib_version_ret { - int64_t lib_ver; + int64_t lib_ver; }; struct remote_get_hostname_ret { - remote_nonnull_string hostname; + remote_nonnull_string hostname; }; struct remote_get_sysinfo_args { - u_int flags; + u_int flags; }; struct remote_get_sysinfo_ret { - remote_nonnull_string sysinfo; + remote_nonnull_string sysinfo; }; struct remote_get_uri_ret { - remote_nonnull_string uri; + remote_nonnull_string uri; }; struct remote_get_max_vcpus_args { - remote_string type; + remote_string type; }; struct remote_get_max_vcpus_ret { - int max_vcpus; + int max_vcpus; }; struct remote_node_get_info_ret { - char model[32]; - int64_t memory; - int cpus; - int mhz; - int nodes; - int sockets; - int cores; - int threads; + char model[32]; + int64_t memory; + int cpus; + int mhz; + int nodes; + int sockets; + int cores; + int threads; }; struct remote_get_capabilities_ret { - remote_nonnull_string capabilities; + remote_nonnull_string capabilities; }; struct remote_node_get_cells_free_memory_args { - int startCell; - int maxCells; + int startCell; + int maxCells; }; struct remote_node_get_cells_free_memory_ret { - struct { - u_int freeMems_len; - int64_t * freeMems_val; - } freeMems; + struct { + u_int freeMems_len; + int64_t * freeMems_val; + } freeMems; }; struct remote_node_get_free_memory_ret { - int64_t freeMem; + int64_t freeMem; }; struct remote_domain_get_scheduler_type_args { - remote_nonnull_domain dom; + remote_nonnull_domain dom; }; struct remote_domain_get_scheduler_type_ret { - remote_nonnull_string type; - int nparams; + remote_nonnull_string type; + int nparams; }; struct remote_domain_get_scheduler_parameters_args { - remote_nonnull_domain dom; - int nparams; + remote_nonnull_domain dom; + int nparams; }; struct remote_domain_get_scheduler_parameters_ret { - struct { - u_int params_len; - remote_sched_param * params_val; - } params; + struct { + u_int params_len; + remote_sched_param * params_val; + } params; }; struct remote_domain_set_scheduler_parameters_args { - remote_nonnull_domain dom; - struct { - u_int params_len; - remote_sched_param * params_val; - } params; + remote_nonnull_domain dom; + struct { + u_int params_len; + remote_sched_param * params_val; + } params; }; struct remote_domain_set_memory_parameters_args { - remote_nonnull_domain dom; - struct { - u_int params_len; - remote_memory_param * params_val; - } params; - u_int flags; + remote_nonnull_domain dom; + struct { + u_int params_len; + remote_memory_param * params_val; + } params; + u_int flags; }; struct remote_domain_get_memory_parameters_args { - remote_nonnull_domain dom; - int nparams; - u_int flags; + remote_nonnull_domain dom; + int nparams; + u_int flags; }; struct remote_domain_get_memory_parameters_ret { - struct { - u_int params_len; - remote_memory_param * params_val; - } params; - int nparams; + struct { + u_int params_len; + remote_memory_param * params_val; + } params; + int nparams; }; struct remote_domain_block_stats_args { - remote_nonnull_domain dom; - remote_nonnull_string path; + remote_nonnull_domain dom; + remote_nonnull_string path; }; struct remote_domain_block_stats_ret { - int64_t rd_req; - int64_t rd_bytes; - int64_t wr_req; - int64_t wr_bytes; - int64_t errs; + int64_t rd_req; + int64_t rd_bytes; + int64_t wr_req; + int64_t wr_bytes; + int64_t errs; }; struct remote_domain_interface_stats_args { - remote_nonnull_domain dom; - remote_nonnull_string path; + remote_nonnull_domain dom; + remote_nonnull_string path; }; struct remote_domain_interface_stats_ret { - int64_t rx_bytes; - int64_t rx_packets; - int64_t rx_errs; - int64_t rx_drop; - int64_t tx_bytes; - int64_t tx_packets; - int64_t tx_errs; - int64_t tx_drop; + int64_t rx_bytes; + int64_t rx_packets; + int64_t rx_errs; + int64_t rx_drop; + int64_t tx_bytes; + int64_t tx_packets; + int64_t tx_errs; + int64_t tx_drop; }; struct remote_domain_memory_stats_args { - remote_nonnull_domain dom; - u_int maxStats; - u_int flags; + remote_nonnull_domain dom; + u_int maxStats; + u_int flags; }; struct remote_domain_memory_stat { - int tag; - uint64_t val; + int tag; + uint64_t val; }; struct remote_domain_memory_stats_ret { - struct { - u_int stats_len; - remote_domain_memory_stat * stats_val; - } stats; + struct { + u_int stats_len; + remote_domain_memory_stat * stats_val; + } stats; }; struct remote_domain_block_peek_args { - remote_nonnull_domain dom; - remote_nonnull_string path; - uint64_t offset; - u_int size; - u_int flags; + remote_nonnull_domain dom; + remote_nonnull_string path; + uint64_t offset; + u_int size; + u_int flags; }; struct remote_domain_block_peek_ret { - struct { - u_int buffer_len; - char * buffer_val; - } buffer; + struct { + u_int buffer_len; + char * buffer_val; + } buffer; }; struct remote_domain_memory_peek_args { - remote_nonnull_domain dom; - uint64_t offset; - u_int size; - u_int flags; + remote_nonnull_domain dom; + uint64_t offset; + u_int size; + u_int flags; }; struct remote_domain_memory_peek_ret { - struct { - u_int buffer_len; - char * buffer_val; - } buffer; + struct { + u_int buffer_len; + char * buffer_val; + } buffer; }; struct remote_domain_get_block_info_args { - remote_nonnull_domain dom; - remote_nonnull_string path; - u_int flags; + remote_nonnull_domain dom; + remote_nonnull_string path; + u_int flags; }; struct remote_domain_get_block_info_ret { - uint64_t allocation; - uint64_t capacity; - uint64_t physical; + uint64_t allocation; + uint64_t capacity; + uint64_t physical; }; struct remote_list_domains_args { - int maxids; + int maxids; }; struct remote_list_domains_ret { - struct { - u_int ids_len; - int * ids_val; - } ids; + struct { + u_int ids_len; + int * ids_val; + } ids; }; struct remote_num_of_domains_ret { - int num; + int num; }; struct remote_domain_create_xml_args { - remote_nonnull_string xml_desc; - int flags; + remote_nonnull_string xml_desc; + int flags; }; struct remote_domain_create_xml_ret { - remote_nonnull_domain dom; + remote_nonnull_domain dom; }; struct remote_domain_lookup_by_id_args { - int id; + int id; }; struct remote_domain_lookup_by_id_ret { - remote_nonnull_domain dom; + remote_nonnull_domain dom; }; struct remote_domain_lookup_by_uuid_args { - remote_uuid uuid; + remote_uuid uuid; }; struct remote_domain_lookup_by_uuid_ret { - remote_nonnull_domain dom; + remote_nonnull_domain dom; }; struct remote_domain_lookup_by_name_args { - remote_nonnull_string name; + remote_nonnull_string name; }; struct remote_domain_lookup_by_name_ret { - remote_nonnull_domain dom; + remote_nonnull_domain dom; }; struct remote_domain_suspend_args { - remote_nonnull_domain dom; + remote_nonnull_domain dom; }; struct remote_domain_resume_args { - remote_nonnull_domain dom; + remote_nonnull_domain dom; }; struct remote_domain_shutdown_args { - remote_nonnull_domain dom; + remote_nonnull_domain dom; }; struct remote_domain_reboot_args { - remote_nonnull_domain dom; - int flags; + remote_nonnull_domain dom; + int flags; }; struct remote_domain_destroy_args { - remote_nonnull_domain dom; + remote_nonnull_domain dom; }; struct remote_domain_get_os_type_args { - remote_nonnull_domain dom; + remote_nonnull_domain dom; }; struct remote_domain_get_os_type_ret { - remote_nonnull_string type; + remote_nonnull_string type; }; struct remote_domain_get_max_memory_args { - remote_nonnull_domain dom; + remote_nonnull_domain dom; }; struct remote_domain_get_max_memory_ret { - uint64_t memory; + uint64_t memory; }; struct remote_domain_set_max_memory_args { - remote_nonnull_domain dom; - uint64_t memory; + remote_nonnull_domain dom; + uint64_t memory; }; struct remote_domain_set_memory_args { - remote_nonnull_domain dom; - uint64_t memory; + remote_nonnull_domain dom; + uint64_t memory; }; struct remote_domain_set_memory_flags_args { - remote_nonnull_domain dom; - uint64_t memory; - u_int flags; + remote_nonnull_domain dom; + uint64_t memory; + u_int flags; }; struct remote_domain_get_info_args { - remote_nonnull_domain dom; + remote_nonnull_domain dom; }; struct remote_domain_get_info_ret { - u_char state; - uint64_t max_mem; - uint64_t memory; - u_short nr_virt_cpu; - uint64_t cpu_time; + u_char state; + uint64_t max_mem; + uint64_t memory; + u_short nr_virt_cpu; + uint64_t cpu_time; }; struct remote_domain_save_args { - remote_nonnull_domain dom; - remote_nonnull_string to; + remote_nonnull_domain dom; + remote_nonnull_string to; }; struct remote_domain_restore_args { - remote_nonnull_string from; + remote_nonnull_string from; }; struct remote_domain_core_dump_args { - remote_nonnull_domain dom; - remote_nonnull_string to; - int flags; + remote_nonnull_domain dom; + remote_nonnull_string to; + int flags; }; struct remote_domain_dump_xml_args { - remote_nonnull_domain dom; - int flags; + remote_nonnull_domain dom; + int flags; }; struct remote_domain_dump_xml_ret { - remote_nonnull_string xml; + remote_nonnull_string xml; }; struct remote_domain_migrate_prepare_args { - remote_string uri_in; - uint64_t flags; - remote_string dname; - uint64_t resource; + remote_string uri_in; + uint64_t flags; + remote_string dname; + uint64_t resource; }; struct remote_domain_migrate_prepare_ret { - struct { - u_int cookie_len; - char * cookie_val; - } cookie; - remote_string uri_out; + struct { + u_int cookie_len; + char * cookie_val; + } cookie; + remote_string uri_out; }; struct remote_domain_migrate_perform_args { - remote_nonnull_domain dom; - struct { - u_int cookie_len; - char * cookie_val; - } cookie; - remote_nonnull_string uri; - uint64_t flags; - remote_string dname; - uint64_t resource; + remote_nonnull_domain dom; + struct { + u_int cookie_len; + char * cookie_val; + } cookie; + remote_nonnull_string uri; + uint64_t flags; + remote_string dname; + uint64_t resource; }; struct remote_domain_migrate_finish_args { - remote_nonnull_string dname; - struct { - u_int cookie_len; - char * cookie_val; - } cookie; - remote_nonnull_string uri; - uint64_t flags; + remote_nonnull_string dname; + struct { + u_int cookie_len; + char * cookie_val; + } cookie; + remote_nonnull_string uri; + uint64_t flags; }; struct remote_domain_migrate_finish_ret { - remote_nonnull_domain ddom; + remote_nonnull_domain ddom; }; struct remote_domain_migrate_prepare2_args { - remote_string uri_in; - uint64_t flags; - remote_string dname; - uint64_t resource; - remote_nonnull_string dom_xml; + remote_string uri_in; + uint64_t flags; + remote_string dname; + uint64_t resource; + remote_nonnull_string dom_xml; }; struct remote_domain_migrate_prepare2_ret { - struct { - u_int cookie_len; - char * cookie_val; - } cookie; - remote_string uri_out; + struct { + u_int cookie_len; + char * cookie_val; + } cookie; + remote_string uri_out; }; struct remote_domain_migrate_finish2_args { - remote_nonnull_string dname; - struct { - u_int cookie_len; - char * cookie_val; - } cookie; - remote_nonnull_string uri; - uint64_t flags; - int retcode; + remote_nonnull_string dname; + struct { + u_int cookie_len; + char * cookie_val; + } cookie; + remote_nonnull_string uri; + uint64_t flags; + int retcode; }; struct remote_domain_migrate_finish2_ret { - remote_nonnull_domain ddom; + remote_nonnull_domain ddom; }; struct remote_list_defined_domains_args { - int maxnames; + int maxnames; }; struct remote_list_defined_domains_ret { - struct { - u_int names_len; - remote_nonnull_string * names_val; - } names; + struct { + u_int names_len; + remote_nonnull_string * names_val; + } names; }; struct remote_num_of_defined_domains_ret { - int num; + int num; }; struct remote_domain_create_args { - remote_nonnull_domain dom; + remote_nonnull_domain dom; }; struct remote_domain_create_with_flags_args { - remote_nonnull_domain dom; - u_int flags; + remote_nonnull_domain dom; + u_int flags; }; struct remote_domain_create_with_flags_ret { - remote_nonnull_domain dom; + remote_nonnull_domain dom; }; struct remote_domain_define_xml_args { - remote_nonnull_string xml; + remote_nonnull_string xml; }; struct remote_domain_define_xml_ret { - remote_nonnull_domain dom; + remote_nonnull_domain dom; }; struct remote_domain_undefine_args { - remote_nonnull_domain dom; + remote_nonnull_domain dom; }; struct remote_domain_set_vcpus_args { - remote_nonnull_domain dom; - int nvcpus; + remote_nonnull_domain dom; + int nvcpus; }; struct remote_domain_set_vcpus_flags_args { - remote_nonnull_domain dom; - u_int nvcpus; - u_int flags; + remote_nonnull_domain dom; + u_int nvcpus; + u_int flags; }; struct remote_domain_get_vcpus_flags_args { - remote_nonnull_domain dom; - u_int flags; + remote_nonnull_domain dom; + u_int flags; }; struct remote_domain_get_vcpus_flags_ret { - int num; + int num; }; struct remote_domain_pin_vcpu_args { - remote_nonnull_domain dom; - int vcpu; - struct { - u_int cpumap_len; - char * cpumap_val; - } cpumap; + remote_nonnull_domain dom; + int vcpu; + struct { + u_int cpumap_len; + char * cpumap_val; + } cpumap; }; struct remote_domain_get_vcpus_args { - remote_nonnull_domain dom; - int maxinfo; - int maplen; + remote_nonnull_domain dom; + int maxinfo; + int maplen; }; struct remote_domain_get_vcpus_ret { - struct { - u_int info_len; - remote_vcpu_info * info_val; - } info; - struct { - u_int cpumaps_len; - char * cpumaps_val; - } cpumaps; + struct { + u_int info_len; + remote_vcpu_info * info_val; + } info; + struct { + u_int cpumaps_len; + char * cpumaps_val; + } cpumaps; }; struct remote_domain_get_max_vcpus_args { - remote_nonnull_domain dom; + remote_nonnull_domain dom; }; struct remote_domain_get_max_vcpus_ret { - int num; + int num; }; struct remote_domain_get_security_label_args { - remote_nonnull_domain dom; + remote_nonnull_domain dom; }; struct remote_domain_get_security_label_ret { - struct { - u_int label_len; - char * label_val; - } label; - int enforcing; + struct { + u_int label_len; + char * label_val; + } label; + int enforcing; }; struct remote_node_get_security_model_ret { - struct { - u_int model_len; - char * model_val; - } model; - struct { - u_int doi_len; - char * doi_val; - } doi; + struct { + u_int model_len; + char * model_val; + } model; + struct { + u_int doi_len; + char * doi_val; + } doi; }; struct remote_domain_attach_device_args { - remote_nonnull_domain dom; - remote_nonnull_string xml; + remote_nonnull_domain dom; + remote_nonnull_string xml; }; struct remote_domain_attach_device_flags_args { - remote_nonnull_domain dom; - remote_nonnull_string xml; - u_int flags; + remote_nonnull_domain dom; + remote_nonnull_string xml; + u_int flags; }; struct remote_domain_detach_device_args { - remote_nonnull_domain dom; - remote_nonnull_string xml; + remote_nonnull_domain dom; + remote_nonnull_string xml; }; struct remote_domain_detach_device_flags_args { - remote_nonnull_domain dom; - remote_nonnull_string xml; - u_int flags; + remote_nonnull_domain dom; + remote_nonnull_string xml; + u_int flags; }; struct remote_domain_update_device_flags_args { - remote_nonnull_domain dom; - remote_nonnull_string xml; - u_int flags; + remote_nonnull_domain dom; + remote_nonnull_string xml; + u_int flags; }; struct remote_domain_get_autostart_args { - remote_nonnull_domain dom; + remote_nonnull_domain dom; }; struct remote_domain_get_autostart_ret { - int autostart; + int autostart; }; struct remote_domain_set_autostart_args { - remote_nonnull_domain dom; - int autostart; + remote_nonnull_domain dom; + int autostart; }; struct remote_num_of_networks_ret { - int num; + int num; }; struct remote_list_networks_args { - int maxnames; + int maxnames; }; struct remote_list_networks_ret { - struct { - u_int names_len; - remote_nonnull_string * names_val; - } names; + struct { + u_int names_len; + remote_nonnull_string * names_val; + } names; }; struct remote_num_of_defined_networks_ret { - int num; + int num; }; struct remote_list_defined_networks_args { - int maxnames; + int maxnames; }; struct remote_list_defined_networks_ret { - struct { - u_int names_len; - remote_nonnull_string * names_val; - } names; + struct { + u_int names_len; + remote_nonnull_string * names_val; + } names; }; struct remote_network_lookup_by_uuid_args { - remote_uuid uuid; + remote_uuid uuid; }; struct remote_network_lookup_by_uuid_ret { - remote_nonnull_network net; + remote_nonnull_network net; }; struct remote_network_lookup_by_name_args { - remote_nonnull_string name; + remote_nonnull_string name; }; struct remote_network_lookup_by_name_ret { - remote_nonnull_network net; + remote_nonnull_network net; }; struct remote_network_create_xml_args { - remote_nonnull_string xml; + remote_nonnull_string xml; }; struct remote_network_create_xml_ret { - remote_nonnull_network net; + remote_nonnull_network net; }; struct remote_network_define_xml_args { - remote_nonnull_string xml; + remote_nonnull_string xml; }; struct remote_network_define_xml_ret { - remote_nonnull_network net; + remote_nonnull_network net; }; struct remote_network_undefine_args { - remote_nonnull_network net; + remote_nonnull_network net; }; struct remote_network_create_args { - remote_nonnull_network net; + remote_nonnull_network net; }; struct remote_network_destroy_args { - remote_nonnull_network net; + remote_nonnull_network net; }; struct remote_network_dump_xml_args { - remote_nonnull_network net; - int flags; + remote_nonnull_network net; + int flags; }; struct remote_network_dump_xml_ret { - remote_nonnull_string xml; + remote_nonnull_string xml; }; struct remote_network_get_bridge_name_args { - remote_nonnull_network net; + remote_nonnull_network net; }; struct remote_network_get_bridge_name_ret { - remote_nonnull_string name; + remote_nonnull_string name; }; struct remote_network_get_autostart_args { - remote_nonnull_network net; + remote_nonnull_network net; }; struct remote_network_get_autostart_ret { - int autostart; + int autostart; }; struct remote_network_set_autostart_args { - remote_nonnull_network net; - int autostart; + remote_nonnull_network net; + int autostart; }; struct remote_num_of_nwfilters_ret { - int num; + int num; }; struct remote_list_nwfilters_args { - int maxnames; + int maxnames; }; struct remote_list_nwfilters_ret { - struct { - u_int names_len; - remote_nonnull_string * names_val; - } names; + struct { + u_int names_len; + remote_nonnull_string * names_val; + } names; }; struct remote_nwfilter_lookup_by_uuid_args { - remote_uuid uuid; + remote_uuid uuid; }; struct remote_nwfilter_lookup_by_uuid_ret { - remote_nonnull_nwfilter nwfilter; + remote_nonnull_nwfilter nwfilter; }; struct remote_nwfilter_lookup_by_name_args { - remote_nonnull_string name; + remote_nonnull_string name; }; struct remote_nwfilter_lookup_by_name_ret { - remote_nonnull_nwfilter nwfilter; + remote_nonnull_nwfilter nwfilter; }; struct remote_nwfilter_define_xml_args { - remote_nonnull_string xml; + remote_nonnull_string xml; }; struct remote_nwfilter_define_xml_ret { - remote_nonnull_nwfilter nwfilter; + remote_nonnull_nwfilter nwfilter; }; struct remote_nwfilter_undefine_args { - remote_nonnull_nwfilter nwfilter; + remote_nonnull_nwfilter nwfilter; }; struct remote_nwfilter_get_xml_desc_args { - remote_nonnull_nwfilter nwfilter; - int flags; + remote_nonnull_nwfilter nwfilter; + int flags; }; struct remote_nwfilter_get_xml_desc_ret { - remote_nonnull_string xml; + remote_nonnull_string xml; }; struct remote_num_of_interfaces_ret { - int num; + int num; }; struct remote_list_interfaces_args { - int maxnames; + int maxnames; }; struct remote_list_interfaces_ret { - struct { - u_int names_len; - remote_nonnull_string * names_val; - } names; + struct { + u_int names_len; + remote_nonnull_string * names_val; + } names; }; struct remote_num_of_defined_interfaces_ret { - int num; + int num; }; struct remote_list_defined_interfaces_args { - int maxnames; + int maxnames; }; struct remote_list_defined_interfaces_ret { - struct { - u_int names_len; - remote_nonnull_string * names_val; - } names; + struct { + u_int names_len; + remote_nonnull_string * names_val; + } names; }; struct remote_interface_lookup_by_name_args { - remote_nonnull_string name; + remote_nonnull_string name; }; struct remote_interface_lookup_by_name_ret { - remote_nonnull_interface iface; + remote_nonnull_interface iface; }; struct remote_interface_lookup_by_mac_string_args { - remote_nonnull_string mac; + remote_nonnull_string mac; }; struct remote_interface_lookup_by_mac_string_ret { - remote_nonnull_interface iface; + remote_nonnull_interface iface; }; struct remote_interface_get_xml_desc_args { - remote_nonnull_interface iface; - u_int flags; + remote_nonnull_interface iface; + u_int flags; }; struct remote_interface_get_xml_desc_ret { - remote_nonnull_string xml; + remote_nonnull_string xml; }; struct remote_interface_define_xml_args { - remote_nonnull_string xml; - u_int flags; + remote_nonnull_string xml; + u_int flags; }; struct remote_interface_define_xml_ret { - remote_nonnull_interface iface; + remote_nonnull_interface iface; }; struct remote_interface_undefine_args { - remote_nonnull_interface iface; + remote_nonnull_interface iface; }; struct remote_interface_create_args { - remote_nonnull_interface iface; - u_int flags; + remote_nonnull_interface iface; + u_int flags; }; struct remote_interface_destroy_args { - remote_nonnull_interface iface; - u_int flags; + remote_nonnull_interface iface; + u_int flags; }; struct remote_auth_list_ret { - struct { - u_int types_len; - remote_auth_type * types_val; - } types; + struct { + u_int types_len; + remote_auth_type * types_val; + } types; }; struct remote_auth_sasl_init_ret { - remote_nonnull_string mechlist; + remote_nonnull_string mechlist; }; struct remote_auth_sasl_start_args { - remote_nonnull_string mech; - int nil; - struct { - u_int data_len; - char * data_val; - } data; + remote_nonnull_string mech; + int nil; + struct { + u_int data_len; + char * data_val; + } data; }; struct remote_auth_sasl_start_ret { - int complete; - int nil; - struct { - u_int data_len; - char * data_val; - } data; + int complete; + int nil; + struct { + u_int data_len; + char * data_val; + } data; }; struct remote_auth_sasl_step_args { - int nil; - struct { - u_int data_len; - char * data_val; - } data; + int nil; + struct { + u_int data_len; + char * data_val; + } data; }; struct remote_auth_sasl_step_ret { - int complete; - int nil; - struct { - u_int data_len; - char * data_val; - } data; + int complete; + int nil; + struct { + u_int data_len; + char * data_val; + } data; }; struct remote_auth_polkit_ret { - int complete; + int complete; }; struct remote_num_of_storage_pools_ret { - int num; + int num; }; struct remote_list_storage_pools_args { - int maxnames; + int maxnames; }; struct remote_list_storage_pools_ret { - struct { - u_int names_len; - remote_nonnull_string * names_val; - } names; + struct { + u_int names_len; + remote_nonnull_string * names_val; + } names; }; struct remote_num_of_defined_storage_pools_ret { - int num; + int num; }; struct remote_list_defined_storage_pools_args { - int maxnames; + int maxnames; }; struct remote_list_defined_storage_pools_ret { - struct { - u_int names_len; - remote_nonnull_string * names_val; - } names; + struct { + u_int names_len; + remote_nonnull_string * names_val; + } names; }; struct remote_find_storage_pool_sources_args { - remote_nonnull_string type; - remote_string srcSpec; - u_int flags; + remote_nonnull_string type; + remote_string srcSpec; + u_int flags; }; struct remote_find_storage_pool_sources_ret { - remote_nonnull_string xml; + remote_nonnull_string xml; }; struct remote_storage_pool_lookup_by_uuid_args { - remote_uuid uuid; + remote_uuid uuid; }; struct remote_storage_pool_lookup_by_uuid_ret { - remote_nonnull_storage_pool pool; + remote_nonnull_storage_pool pool; }; struct remote_storage_pool_lookup_by_name_args { - remote_nonnull_string name; + remote_nonnull_string name; }; struct remote_storage_pool_lookup_by_name_ret { - remote_nonnull_storage_pool pool; + remote_nonnull_storage_pool pool; }; struct remote_storage_pool_lookup_by_volume_args { - remote_nonnull_storage_vol vol; + remote_nonnull_storage_vol vol; }; struct remote_storage_pool_lookup_by_volume_ret { - remote_nonnull_storage_pool pool; + remote_nonnull_storage_pool pool; }; struct remote_storage_pool_create_xml_args { - remote_nonnull_string xml; - u_int flags; + remote_nonnull_string xml; + u_int flags; }; struct remote_storage_pool_create_xml_ret { - remote_nonnull_storage_pool pool; + remote_nonnull_storage_pool pool; }; struct remote_storage_pool_define_xml_args { - remote_nonnull_string xml; - u_int flags; + remote_nonnull_string xml; + u_int flags; }; struct remote_storage_pool_define_xml_ret { - remote_nonnull_storage_pool pool; + remote_nonnull_storage_pool pool; }; struct remote_storage_pool_build_args { - remote_nonnull_storage_pool pool; - u_int flags; + remote_nonnull_storage_pool pool; + u_int flags; }; struct remote_storage_pool_undefine_args { - remote_nonnull_storage_pool pool; + remote_nonnull_storage_pool pool; }; struct remote_storage_pool_create_args { - remote_nonnull_storage_pool pool; - u_int flags; + remote_nonnull_storage_pool pool; + u_int flags; }; struct remote_storage_pool_destroy_args { - remote_nonnull_storage_pool pool; + remote_nonnull_storage_pool pool; }; struct remote_storage_pool_delete_args { - remote_nonnull_storage_pool pool; - u_int flags; + remote_nonnull_storage_pool pool; + u_int flags; }; struct remote_storage_pool_refresh_args { - remote_nonnull_storage_pool pool; - u_int flags; + remote_nonnull_storage_pool pool; + u_int flags; }; struct remote_storage_pool_dump_xml_args { - remote_nonnull_storage_pool pool; - u_int flags; + remote_nonnull_storage_pool pool; + u_int flags; }; struct remote_storage_pool_dump_xml_ret { - remote_nonnull_string xml; + remote_nonnull_string xml; }; struct remote_storage_pool_get_info_args { - remote_nonnull_storage_pool pool; + remote_nonnull_storage_pool pool; }; struct remote_storage_pool_get_info_ret { - u_char state; - uint64_t capacity; - uint64_t allocation; - uint64_t available; + u_char state; + uint64_t capacity; + uint64_t allocation; + uint64_t available; }; struct remote_storage_pool_get_autostart_args { - remote_nonnull_storage_pool pool; + remote_nonnull_storage_pool pool; }; struct remote_storage_pool_get_autostart_ret { - int autostart; + int autostart; }; struct remote_storage_pool_set_autostart_args { - remote_nonnull_storage_pool pool; - int autostart; + remote_nonnull_storage_pool pool; + int autostart; }; struct remote_storage_pool_num_of_volumes_args { - remote_nonnull_storage_pool pool; + remote_nonnull_storage_pool pool; }; struct remote_storage_pool_num_of_volumes_ret { - int num; + int num; }; struct remote_storage_pool_list_volumes_args { - remote_nonnull_storage_pool pool; - int maxnames; + remote_nonnull_storage_pool pool; + int maxnames; }; struct remote_storage_pool_list_volumes_ret { - struct { - u_int names_len; - remote_nonnull_string * names_val; - } names; + struct { + u_int names_len; + remote_nonnull_string * names_val; + } names; }; struct remote_storage_vol_lookup_by_name_args { - remote_nonnull_storage_pool pool; - remote_nonnull_string name; + remote_nonnull_storage_pool pool; + remote_nonnull_string name; }; struct remote_storage_vol_lookup_by_name_ret { - remote_nonnull_storage_vol vol; + remote_nonnull_storage_vol vol; }; struct remote_storage_vol_lookup_by_key_args { - remote_nonnull_string key; + remote_nonnull_string key; }; struct remote_storage_vol_lookup_by_key_ret { - remote_nonnull_storage_vol vol; + remote_nonnull_storage_vol vol; }; struct remote_storage_vol_lookup_by_path_args { - remote_nonnull_string path; + remote_nonnull_string path; }; struct remote_storage_vol_lookup_by_path_ret { - remote_nonnull_storage_vol vol; + remote_nonnull_storage_vol vol; }; struct remote_storage_vol_create_xml_args { - remote_nonnull_storage_pool pool; - remote_nonnull_string xml; - u_int flags; + remote_nonnull_storage_pool pool; + remote_nonnull_string xml; + u_int flags; }; struct remote_storage_vol_create_xml_ret { - remote_nonnull_storage_vol vol; + remote_nonnull_storage_vol vol; }; struct remote_storage_vol_create_xml_from_args { - remote_nonnull_storage_pool pool; - remote_nonnull_string xml; - remote_nonnull_storage_vol clonevol; - u_int flags; + remote_nonnull_storage_pool pool; + remote_nonnull_string xml; + remote_nonnull_storage_vol clonevol; + u_int flags; }; struct remote_storage_vol_create_xml_from_ret { - remote_nonnull_storage_vol vol; + remote_nonnull_storage_vol vol; }; struct remote_storage_vol_delete_args { - remote_nonnull_storage_vol vol; - u_int flags; + remote_nonnull_storage_vol vol; + u_int flags; }; struct remote_storage_vol_wipe_args { - remote_nonnull_storage_vol vol; - u_int flags; + remote_nonnull_storage_vol vol; + u_int flags; }; struct remote_storage_vol_dump_xml_args { - remote_nonnull_storage_vol vol; - u_int flags; + remote_nonnull_storage_vol vol; + u_int flags; }; struct remote_storage_vol_dump_xml_ret { - remote_nonnull_string xml; + remote_nonnull_string xml; }; struct remote_storage_vol_get_info_args { - remote_nonnull_storage_vol vol; + remote_nonnull_storage_vol vol; }; struct remote_storage_vol_get_info_ret { - char type; - uint64_t capacity; - uint64_t allocation; + char type; + uint64_t capacity; + uint64_t allocation; }; struct remote_storage_vol_get_path_args { - remote_nonnull_storage_vol vol; + remote_nonnull_storage_vol vol; }; struct remote_storage_vol_get_path_ret { - remote_nonnull_string name; + remote_nonnull_string name; }; struct remote_node_num_of_devices_args { - remote_string cap; - u_int flags; + remote_string cap; + u_int flags; }; struct remote_node_num_of_devices_ret { - int num; + int num; }; struct remote_node_list_devices_args { - remote_string cap; - int maxnames; - u_int flags; + remote_string cap; + int maxnames; + u_int flags; }; struct remote_node_list_devices_ret { - struct { - u_int names_len; - remote_nonnull_string * names_val; - } names; + struct { + u_int names_len; + remote_nonnull_string * names_val; + } names; }; struct remote_node_device_lookup_by_name_args { - remote_nonnull_string name; + remote_nonnull_string name; }; struct remote_node_device_lookup_by_name_ret { - remote_nonnull_node_device dev; + remote_nonnull_node_device dev; }; struct remote_node_device_dump_xml_args { - remote_nonnull_string name; - u_int flags; + remote_nonnull_string name; + u_int flags; }; struct remote_node_device_dump_xml_ret { - remote_nonnull_string xml; + remote_nonnull_string xml; }; struct remote_node_device_get_parent_args { - remote_nonnull_string name; + remote_nonnull_string name; }; struct remote_node_device_get_parent_ret { - remote_string parent; + remote_string parent; }; struct remote_node_device_num_of_caps_args { - remote_nonnull_string name; + remote_nonnull_string name; }; struct remote_node_device_num_of_caps_ret { - int num; + int num; }; struct remote_node_device_list_caps_args { - remote_nonnull_string name; - int maxnames; + remote_nonnull_string name; + int maxnames; }; struct remote_node_device_list_caps_ret { - struct { - u_int names_len; - remote_nonnull_string * names_val; - } names; + struct { + u_int names_len; + remote_nonnull_string * names_val; + } names; }; struct remote_node_device_dettach_args { - remote_nonnull_string name; + remote_nonnull_string name; }; struct remote_node_device_re_attach_args { - remote_nonnull_string name; + remote_nonnull_string name; }; struct remote_node_device_reset_args { - remote_nonnull_string name; + remote_nonnull_string name; }; struct remote_node_device_create_xml_args { - remote_nonnull_string xml_desc; - int flags; + remote_nonnull_string xml_desc; + int flags; }; struct remote_node_device_create_xml_ret { - remote_nonnull_node_device dev; + remote_nonnull_node_device dev; }; struct remote_node_device_destroy_args { - remote_nonnull_string name; + remote_nonnull_string name; }; struct remote_domain_events_register_ret { - int cb_registered; + int cb_registered; }; struct remote_domain_events_deregister_ret { - int cb_registered; + int cb_registered; }; struct remote_domain_event_lifecycle_msg { - remote_nonnull_domain dom; - int event; - int detail; + remote_nonnull_domain dom; + int event; + int detail; }; struct remote_domain_xml_from_native_args { - remote_nonnull_string nativeFormat; - remote_nonnull_string nativeConfig; - u_int flags; + remote_nonnull_string nativeFormat; + remote_nonnull_string nativeConfig; + u_int flags; }; struct remote_domain_xml_from_native_ret { - remote_nonnull_string domainXml; + remote_nonnull_string domainXml; }; struct remote_domain_xml_to_native_args { - remote_nonnull_string nativeFormat; - remote_nonnull_string domainXml; - u_int flags; + remote_nonnull_string nativeFormat; + remote_nonnull_string domainXml; + u_int flags; }; struct remote_domain_xml_to_native_ret { - remote_nonnull_string nativeConfig; + remote_nonnull_string nativeConfig; }; struct remote_num_of_secrets_ret { - int num; + int num; }; struct remote_list_secrets_args { - int maxuuids; + int maxuuids; }; struct remote_list_secrets_ret { - struct { - u_int uuids_len; - remote_nonnull_string * uuids_val; - } uuids; + struct { + u_int uuids_len; + remote_nonnull_string * uuids_val; + } uuids; }; struct remote_secret_lookup_by_uuid_args { - remote_uuid uuid; + remote_uuid uuid; }; struct remote_secret_lookup_by_uuid_ret { - remote_nonnull_secret secret; + remote_nonnull_secret secret; }; struct remote_secret_define_xml_args { - remote_nonnull_string xml; - u_int flags; + remote_nonnull_string xml; + u_int flags; }; struct remote_secret_define_xml_ret { - remote_nonnull_secret secret; + remote_nonnull_secret secret; }; struct remote_secret_get_xml_desc_args { - remote_nonnull_secret secret; - u_int flags; + remote_nonnull_secret secret; + u_int flags; }; struct remote_secret_get_xml_desc_ret { - remote_nonnull_string xml; + remote_nonnull_string xml; }; struct remote_secret_set_value_args { - remote_nonnull_secret secret; - struct { - u_int value_len; - char * value_val; - } value; - u_int flags; + remote_nonnull_secret secret; + struct { + u_int value_len; + char * value_val; + } value; + u_int flags; }; struct remote_secret_get_value_args { - remote_nonnull_secret secret; - u_int flags; + remote_nonnull_secret secret; + u_int flags; }; struct remote_secret_get_value_ret { - struct { - u_int value_len; - char * value_val; - } value; + struct { + u_int value_len; + char * value_val; + } value; }; struct remote_secret_undefine_args { - remote_nonnull_secret secret; + remote_nonnull_secret secret; }; struct remote_secret_lookup_by_usage_args { - int usageType; - remote_nonnull_string usageID; + int usageType; + remote_nonnull_string usageID; }; struct remote_secret_lookup_by_usage_ret { - remote_nonnull_secret secret; + remote_nonnull_secret secret; }; struct remote_domain_migrate_prepare_tunnel_args { - uint64_t flags; - remote_string dname; - uint64_t resource; - remote_nonnull_string dom_xml; + uint64_t flags; + remote_string dname; + uint64_t resource; + remote_nonnull_string dom_xml; }; struct remote_is_secure_ret { - int secure; + int secure; }; struct remote_domain_is_active_args { - remote_nonnull_domain dom; + remote_nonnull_domain dom; }; struct remote_domain_is_active_ret { - int active; + int active; }; struct remote_domain_is_persistent_args { - remote_nonnull_domain dom; + remote_nonnull_domain dom; }; struct remote_domain_is_persistent_ret { - int persistent; + int persistent; }; struct remote_domain_is_updated_args { - remote_nonnull_domain dom; + remote_nonnull_domain dom; }; struct remote_domain_is_updated_ret { - int updated; + int updated; }; struct remote_network_is_active_args { - remote_nonnull_network net; + remote_nonnull_network net; }; struct remote_network_is_active_ret { - int active; + int active; }; struct remote_network_is_persistent_args { - remote_nonnull_network net; + remote_nonnull_network net; }; struct remote_network_is_persistent_ret { - int persistent; + int persistent; }; struct remote_storage_pool_is_active_args { - remote_nonnull_storage_pool pool; + remote_nonnull_storage_pool pool; }; struct remote_storage_pool_is_active_ret { - int active; + int active; }; struct remote_storage_pool_is_persistent_args { - remote_nonnull_storage_pool pool; + remote_nonnull_storage_pool pool; }; struct remote_storage_pool_is_persistent_ret { - int persistent; + int persistent; }; struct remote_interface_is_active_args { - remote_nonnull_interface iface; + remote_nonnull_interface iface; }; struct remote_interface_is_active_ret { - int active; + int active; }; struct remote_cpu_compare_args { - remote_nonnull_string xml; - u_int flags; + remote_nonnull_string xml; + u_int flags; }; struct remote_cpu_compare_ret { - int result; + int result; }; struct remote_cpu_baseline_args { - struct { - u_int xmlCPUs_len; - remote_nonnull_string * xmlCPUs_val; - } xmlCPUs; - u_int flags; + struct { + u_int xmlCPUs_len; + remote_nonnull_string * xmlCPUs_val; + } xmlCPUs; + u_int flags; }; struct remote_cpu_baseline_ret { - remote_nonnull_string cpu; + remote_nonnull_string cpu; }; struct remote_domain_get_job_info_args { - remote_nonnull_domain dom; + remote_nonnull_domain dom; }; struct remote_domain_get_job_info_ret { - int type; - uint64_t timeElapsed; - uint64_t timeRemaining; - uint64_t dataTotal; - uint64_t dataProcessed; - uint64_t dataRemaining; - uint64_t memTotal; - uint64_t memProcessed; - uint64_t memRemaining; - uint64_t fileTotal; - uint64_t fileProcessed; - uint64_t fileRemaining; + int type; + uint64_t timeElapsed; + uint64_t timeRemaining; + uint64_t dataTotal; + uint64_t dataProcessed; + uint64_t dataRemaining; + uint64_t memTotal; + uint64_t memProcessed; + uint64_t memRemaining; + uint64_t fileTotal; + uint64_t fileProcessed; + uint64_t fileRemaining; }; struct remote_domain_abort_job_args { - remote_nonnull_domain dom; + remote_nonnull_domain dom; }; struct remote_domain_migrate_set_max_downtime_args { - remote_nonnull_domain dom; - uint64_t downtime; - u_int flags; + remote_nonnull_domain dom; + uint64_t downtime; + u_int flags; }; struct remote_domain_events_register_any_args { - int eventID; + int eventID; }; struct remote_domain_events_deregister_any_args { - int eventID; + int eventID; }; struct remote_domain_event_reboot_msg { - remote_nonnull_domain dom; + remote_nonnull_domain dom; }; struct remote_domain_event_rtc_change_msg { - remote_nonnull_domain dom; - int64_t offset; + remote_nonnull_domain dom; + int64_t offset; }; struct remote_domain_event_watchdog_msg { - remote_nonnull_domain dom; - int action; + remote_nonnull_domain dom; + int action; }; struct remote_domain_event_io_error_msg { - remote_nonnull_domain dom; - remote_nonnull_string srcPath; - remote_nonnull_string devAlias; - int action; + remote_nonnull_domain dom; + remote_nonnull_string srcPath; + remote_nonnull_string devAlias; + int action; }; struct remote_domain_event_io_error_reason_msg { - remote_nonnull_domain dom; - remote_nonnull_string srcPath; - remote_nonnull_string devAlias; - int action; - remote_nonnull_string reason; + remote_nonnull_domain dom; + remote_nonnull_string srcPath; + remote_nonnull_string devAlias; + int action; + remote_nonnull_string reason; }; struct remote_domain_event_graphics_address { - int family; - remote_nonnull_string node; - remote_nonnull_string service; + int family; + remote_nonnull_string node; + remote_nonnull_string service; }; struct remote_domain_event_graphics_identity { - remote_nonnull_string type; - remote_nonnull_string name; + remote_nonnull_string type; + remote_nonnull_string name; }; struct remote_domain_event_graphics_msg { - remote_nonnull_domain dom; - int phase; - remote_domain_event_graphics_address local; - remote_domain_event_graphics_address remote; - remote_nonnull_string authScheme; - struct { - u_int subject_len; - remote_domain_event_graphics_identity * subject_val; - } subject; + remote_nonnull_domain dom; + int phase; + remote_domain_event_graphics_address local; + remote_domain_event_graphics_address remote; + remote_nonnull_string authScheme; + struct { + u_int subject_len; + remote_domain_event_graphics_identity * subject_val; + } subject; }; struct remote_domain_managed_save_args { - remote_nonnull_domain dom; - u_int flags; + remote_nonnull_domain dom; + u_int flags; }; struct remote_domain_has_managed_save_image_args { - remote_nonnull_domain dom; - u_int flags; + remote_nonnull_domain dom; + u_int flags; }; struct remote_domain_has_managed_save_image_ret { - int ret; + int ret; }; struct remote_domain_managed_save_remove_args { - remote_nonnull_domain dom; - u_int flags; + remote_nonnull_domain dom; + u_int flags; }; struct remote_domain_snapshot_create_xml_args { - remote_nonnull_domain domain; - remote_nonnull_string xml_desc; - int flags; + remote_nonnull_domain domain; + remote_nonnull_string xml_desc; + int flags; }; struct remote_domain_snapshot_create_xml_ret { - remote_nonnull_domain_snapshot snap; + remote_nonnull_domain_snapshot snap; }; struct remote_domain_snapshot_dump_xml_args { - remote_nonnull_domain_snapshot snap; - int flags; + remote_nonnull_domain_snapshot snap; + int flags; }; struct remote_domain_snapshot_dump_xml_ret { - remote_nonnull_string xml; + remote_nonnull_string xml; }; struct remote_domain_snapshot_num_args { - remote_nonnull_domain domain; - int flags; + remote_nonnull_domain domain; + int flags; }; struct remote_domain_snapshot_num_ret { - int num; + int num; }; struct remote_domain_snapshot_list_names_args { - remote_nonnull_domain domain; - int nameslen; - int flags; + remote_nonnull_domain domain; + int nameslen; + int flags; }; struct remote_domain_snapshot_list_names_ret { - struct { - u_int names_len; - remote_nonnull_string * names_val; - } names; + struct { + u_int names_len; + remote_nonnull_string * names_val; + } names; }; struct remote_domain_snapshot_lookup_by_name_args { - remote_nonnull_domain domain; - remote_nonnull_string name; - int flags; + remote_nonnull_domain domain; + remote_nonnull_string name; + int flags; }; struct remote_domain_snapshot_lookup_by_name_ret { - remote_nonnull_domain_snapshot snap; + remote_nonnull_domain_snapshot snap; }; struct remote_domain_has_current_snapshot_args { - remote_nonnull_domain domain; - int flags; + remote_nonnull_domain domain; + int flags; }; struct remote_domain_has_current_snapshot_ret { - int result; + int result; }; struct remote_domain_snapshot_current_args { - remote_nonnull_domain domain; - int flags; + remote_nonnull_domain domain; + int flags; }; struct remote_domain_snapshot_current_ret { - remote_nonnull_domain_snapshot snap; + remote_nonnull_domain_snapshot snap; }; struct remote_domain_revert_to_snapshot_args { - remote_nonnull_domain_snapshot snap; - int flags; + remote_nonnull_domain_snapshot snap; + int flags; }; struct remote_domain_snapshot_delete_args { - remote_nonnull_domain_snapshot snap; - int flags; + remote_nonnull_domain_snapshot snap; + int flags; }; struct remote_domain_open_console_args { - remote_nonnull_domain domain; - remote_string devname; - u_int flags; + remote_nonnull_domain domain; + remote_string devname; + u_int flags; }; struct remote_message_header { - u_int prog; - u_int vers; - int proc; - remote_message_type type; - u_int serial; - remote_message_status status; + u_int prog; + u_int vers; + int proc; + remote_message_type type; + u_int serial; + remote_message_status status; }; -- 1.7.4

On Wed, Mar 02, 2011 at 05:13:39PM +0900, Taku Izumi wrote:

This patch adds the new options (--live and --config) to "virsh setmem" command. The behavior of above options is the same as that of "virsh setvcpus" and so on. That is, when the --config option is specified, a modification is effective for the persistent domain. Moreover we can modify the memory size of inactive domains as well as that of active domains.

Signed-off-by: Taku Izumi <izumi.taku@jp.fujitsu.com> --- tools/virsh.c | 16 ++++++++++++++-- 1 file changed, 14 insertions(+), 2 deletions(-)

Index: libvirt-git/tools/virsh.c =================================================================== --- libvirt-git.orig/tools/virsh.c +++ libvirt-git/tools/virsh.c @@ -2910,6 +2910,8 @@ static const vshCmdInfo info_setmem[] = static const vshCmdOptDef opts_setmem[] = { {"domain", VSH_OT_DATA, VSH_OFLAG_REQ, N_("domain name, id or uuid")}, {"kilobytes", VSH_OT_INT, VSH_OFLAG_REQ, N_("number of kilobytes of memory")}, + {"config", VSH_OT_BOOL, 0, N_("affect next boot")}, + {"live", VSH_OT_BOOL, 0, N_("affect running domain")}, {NULL, 0, 0, NULL} };

@@ -2920,6 +2922,10 @@ cmdSetmem(vshControl *ctl, const vshCmd virDomainInfo info; unsigned long kilobytes; int ret = TRUE; + int config = vshCommandOptBool(cmd, "config"); + int live = vshCommandOptBool(cmd, "live"); + int flags = ((config ? VIR_DOMAIN_MEM_CONFIG : 0) | + (live ? VIR_DOMAIN_MEM_LIVE : 0));

if (!vshConnectionUsability(ctl, ctl->conn)) return FALSE; @@ -2947,8 +2953,14 @@ cmdSetmem(vshControl *ctl, const vshCmd return FALSE; }

- if (virDomainSetMemory(dom, kilobytes) != 0) { - ret = FALSE; + if (!flags) { + if (virDomainSetMemory(dom, kilobytes) != 0) { + ret = FALSE; + }

Indentation went a little bit wrong here. Also '!flags' could in fact be '!flags || (flags == VIR_DOMAIN_MEM_LIVE)' since if only the --live flag was set, we can still use the original API call for greater backcompatibility.

+ } else { + if (virDomainSetMemoryFlags(dom, kilobytes, flags) < 0) { + ret = FALSE; + } }

Regards, Daniel -- |: http://berrange.com -o- http://www.flickr.com/photos/dberrange/ :| |: http://libvirt.org -o- http://virt-manager.org :| |: http://autobuild.org -o- http://search.cpan.org/~danberr/ :| |: http://entangle-photo.org -o- http://live.gnome.org/gtk-vnc :|