- Devel - libvirt List Archives

[libvirt] [PATCHv2] Configure native vlan modes on Open vSwitch ports
by james robson 19 Apr '13

19 Apr '13

This patch adds functionality to allow libvirt to configure the 'native-tagged' and 'native-untagged' modes on openvswitch networks. v2 changes: Fix problems reported by Eric Blake --- diff --git a/docs/formatdomain.html.in b/docs/formatdomain.html.in index ffcc33e..a5054cc 100644 --- a/docs/formatdomain.html.in +++ b/docs/formatdomain.html.in @@ -3209,6 +3209,12 @@ qemu-kvm -net nic,model=? /dev/null <parameters interfaceid='09b11c53-8b5c-4eeb-8f00-d84eaa0aaa4f'/> </virtualport> </interface> + <interface type='bridge'> + <vlan trunk='yes' native_mode='tagged' native_tag='123'> + <tag id='42'/> + </vlan> + ... + </interface> <devices> ...</pre> @@ -3234,7 +3240,17 @@ qemu-kvm -net nic,model=? /dev/null attribute <code>trunk='yes'</code> can be added to the toplevel vlan element. - + + For network connections using openvswitch it is possible to + configure the 'native-tagged' and 'native-untagged' vlan modes + (Since 1.0.3). This uses the optional + <code>native_mode</code> and <code>native_tag</code> attributes + on the <code><vlan></code> element: <code>native_mode</code> + may be set to 'tagged' or 'untagged', <code>native_tag</code> + sets the id of the native vlan. Setting a native vlan implies + this is a trunk port, so <code>trunk='yes'</code> will be added if not + explicitly set. + <h5><a name="elementLink">Modifying virtual link state</a></h5> <pre> ... diff --git a/docs/formatnetwork.html.in b/docs/formatnetwork.html.in index 7b42529..68c562b 100644 --- a/docs/formatnetwork.html.in +++ b/docs/formatnetwork.html.in @@ -396,6 +396,12 @@ <parameters interfaceid='09b11c53-8b5c-4eeb-8f00-d84eaa0aaa4f'/> </virtualport> </interface> + <interface type='bridge'> + <vlan trunk='yes' native_mode='tagged' native_tag='123'> + <tag id='42'/> + </vlan> + ... + </interface> <devices> ...</pre> diff --git a/docs/schemas/networkcommon.rng b/docs/schemas/networkcommon.rng index 51ff759..4696f43 100644 --- a/docs/schemas/networkcommon.rng +++ b/docs/schemas/networkcommon.rng @@ -197,6 +197,21 @@ <value>yes</value> </attribute> </optional> + <optional> + <attribute name="native_mode"> + <choice> + <value>tagged</value> + <value>untagged</value> + </choice> + </attribute> + </optional> + <optional> + <attribute name="native_tag"> + <data type="unsignedInt"> + <param name="maxInclusive">4095</param> + </data> + </attribute> + </optional> <oneOrMore> <element name="tag"> <attribute name="id"> diff --git a/src/conf/netdev_vlan_conf.c b/src/conf/netdev_vlan_conf.c index 13ba8c6..618eb4c 100644 --- a/src/conf/netdev_vlan_conf.c +++ b/src/conf/netdev_vlan_conf.c @@ -17,6 +17,7 @@ * * Authors: * Laine Stump <laine(a)redhat.com> + * James Robson <jrobson(a)websense.com> */ #include <config.h> @@ -33,6 +34,8 @@ virNetDevVlanParse(xmlNodePtr node, xmlXPathContextPtr ctxt, virNetDevVlanPtr de int ret = -1; xmlNodePtr save = ctxt->node; const char *trunk = NULL; + const char *nativeMode; + unsigned int nativeTag; xmlNodePtr *tagNodes = NULL; int nTags, ii; @@ -73,16 +76,44 @@ virNetDevVlanParse(xmlNodePtr node, xmlXPathContextPtr ctxt, virNetDevVlanPtr de def->nTags = nTags; - /* now that we know how many tags there are, look for an explicit - * trunk setting. - */ - if (nTags > 1) - def->trunk = true; + def->nativeMode = 0; + def->nativeTag = 0; ctxt->node = node; + if ((nativeMode = virXPathString("string(./@native_mode)", ctxt)) ! = NULL) { + if (STRCASENEQ(nativeMode, "tagged") && STRCASENEQ(nativeMode, "untagged")) { + virReportError(VIR_ERR_XML_ERROR, + _("invalid \"native_mode='%s'\" in <vlan> - " + "native_mode must be 'tagged' or 'untagged'"), nativeMode); + goto error; + } + if (virXPathUInt("string(./@native_tag)", ctxt, &nativeTag) < 0) { + virReportError(VIR_ERR_XML_ERROR, "%s", + _("missing or invalid native_tag attribute")); + goto error; + } + if (nativeTag > 4095) { + virReportError(VIR_ERR_XML_ERROR, + _("native_tag %u too large (maximum 4095)"), nativeTag); + goto error; + } + def->nativeMode = STRCASEEQ(nativeMode, "tagged") ? 1 : 2; + def->nativeTag = nativeTag; + } + + /* def->trunk will be set to true if: + * "trunk='yes'" is set in xml + * a native-* vlan mode has been set + * >1 tag has been set */ if ((trunk = virXPathString("string(./@trunk)", ctxt)) != NULL) { def->trunk = STRCASEEQ(trunk, "yes"); if (!def->trunk) { + if (def->nativeMode > 0) { + virReportError(VIR_ERR_XML_ERROR, "%s", + _("invalid configuration in <vlan> - \"trunk='no'\" is " + "not allowed with native_mode")); + goto error; + } if (nTags > 1) { virReportError(VIR_ERR_XML_ERROR, _("invalid \"trunk='%s'\" in <vlan> - trunk='yes' " @@ -97,6 +128,8 @@ virNetDevVlanParse(xmlNodePtr node, xmlXPathContextPtr ctxt, virNetDevVlanPtr de goto error; } } + } else if (nTags > 1 || def->nativeMode > 0) { + def->trunk = true; } ret = 0; @@ -122,8 +155,11 @@ virNetDevVlanFormat(virNetDevVlanPtr def, virBufferPtr buf) _("missing vlan tag data")); return -1; } - - virBufferAsprintf(buf, "<vlan%s>\n", def->trunk ? " trunk='yes'" : ""); + if (def->nativeMode == 0) { + virBufferAsprintf(buf, "<vlan%s>\n", def->trunk ? " trunk='yes'" : ""); + } else { + virBufferAsprintf(buf, "<vlan trunk='yes' native_mode='%s' native_tag='%u'>\n", def->nativeMode == 1 ? "tagged" : "untagged", def->nativeTag); + } for (ii = 0; ii < def->nTags; ii++) { virBufferAsprintf(buf, " <tag id='%u'/>\n", def->tag[ii]); } diff --git a/src/util/virnetdevopenvswitch.c b/src/util/virnetdevopenvswitch.c index 4fe077a..87122b0 100644 --- a/src/util/virnetdevopenvswitch.c +++ b/src/util/virnetdevopenvswitch.c @@ -19,6 +19,7 @@ * Dan Wendlandt <dan(a)nicira.com> * Kyle Mestery <kmestery(a)cisco.com> * Ansis Atteka <aatteka(a)nicira.com> + * James Robson <jrobson(a)websense.com> */ #include <config.h> @@ -108,9 +109,13 @@ int virNetDevOpenvswitchAddPort(const char *brname, const char *ifname, virCommandAddArgList(cmd, "--timeout=5", "--", "--may-exist", "add-port", brname, ifname, NULL); - if (virBufferUse(&buf) != 0) + if (virBufferUse(&buf) != 0) { + if (virtVlan->nativeMode > 0) { + virCommandAddArgFormat(cmd, "vlan_mode=%s", virtVlan->nativeMode == 1 ? "native-tagged" : "native-untagged"); + virCommandAddArgFormat(cmd, "tag=%d", virtVlan->nativeTag); + } virCommandAddArgList(cmd, virBufferCurrentContent(&buf), NULL); - + } if (ovsport->profileID[0] == '\0') { virCommandAddArgList(cmd, "--", "set", "Interface", ifname, attachedmac_ex_id, diff --git a/src/util/virnetdevvlan.c b/src/util/virnetdevvlan.c index 2fe2017..298673d 100644 --- a/src/util/virnetdevvlan.c +++ b/src/util/virnetdevvlan.c @@ -17,6 +17,7 @@ * * Authors: * Laine Stump <laine(a)redhat.com> + * James Robson <jrobson(a)websense.com> */ #include <config.h> @@ -33,6 +34,8 @@ virNetDevVlanClear(virNetDevVlanPtr vlan) { VIR_FREE(vlan->tag); vlan->nTags = 0; + vlan->nativeMode = 0; + vlan->nativeTag = -1; } void @@ -54,7 +57,9 @@ virNetDevVlanEqual(const virNetDevVlanPtr a, const virNetDevVlanPtr b) return false; if (a->trunk != b->trunk || - a->nTags != b->nTags) { + a->nTags != b->nTags || + a->nativeMode != b->nativeMode || + a->nativeTag != b->nativeTag) { return false; } @@ -89,6 +94,8 @@ virNetDevVlanCopy(virNetDevVlanPtr dst, const virNetDevVlanPtr src) dst->trunk = src->trunk; dst->nTags = src->nTags; + dst->nativeMode = src->nativeMode; + dst->nativeTag = src->nativeTag; memcpy(dst->tag, src->tag, src->nTags * sizeof(*src->tag)); return 0; } diff --git a/src/util/virnetdevvlan.h b/src/util/virnetdevvlan.h index c6b16ef..f0f78f0 100644 --- a/src/util/virnetdevvlan.h +++ b/src/util/virnetdevvlan.h @@ -17,6 +17,7 @@ * * Authors: * Laine Stump <laine(a)redhat.com> + * James Robson <jrobson(a)websense.com> */ #ifndef __VIR_NETDEV_VLAN_H__ @@ -25,6 +26,8 @@ typedef struct _virNetDevVlan virNetDevVlan; typedef virNetDevVlan *virNetDevVlanPtr; struct _virNetDevVlan { + short int nativeMode; /* 0=off, 1=tagged, 2=untagged */ + unsigned int nativeTag; bool trunk; /* true if this is a trunk */ int nTags; /* number of tags in array */ unsigned int *tag; /* pointer to array of tags */ diff --git a/tests/networkxml2xmlin/openvswitch-net.xml b/tests/networkxml2xmlin/openvswitch-net.xml index a3d82b1..93c49d5 100644 --- a/tests/networkxml2xmlin/openvswitch-net.xml +++ b/tests/networkxml2xmlin/openvswitch-net.xml @@ -21,4 +21,13 @@ <parameters profileid='alice-profile'/> </virtualport> </portgroup> + <portgroup name='tagged'> + <vlan native_mode='tagged' native_tag='123'> + <tag id='555'/> + <tag id='444'/> + </vlan> + <virtualport> + <parameters profileid='tagged-profile'/> + </virtualport> + </portgroup> </network> diff --git a/tests/networkxml2xmlout/openvswitch-net.xml b/tests/networkxml2xmlout/openvswitch-net.xml index a3d82b1..ab3d797 100644 --- a/tests/networkxml2xmlout/openvswitch-net.xml +++ b/tests/networkxml2xmlout/openvswitch-net.xml @@ -21,4 +21,13 @@ <parameters profileid='alice-profile'/> </virtualport> </portgroup> + <portgroup name='tagged'> + <vlan trunk='yes' native_mode='tagged' native_tag='123'> + <tag id='555'/> + <tag id='444'/> + </vlan> + <virtualport> + <parameters profileid='tagged-profile'/> + </virtualport> + </portgroup> </network> Protected by Websense Hosted Email Security -- www.websense.com

2 2

[libvirt] [libvirt-glib] vapi: Get rid of redundant LibvirtGObject-1.0.metadata
by Zeeshan Ali (Khattak) 19 Apr '13

19 Apr '13

From: "Zeeshan Ali (Khattak)" <zeeshanak(a)gnome.org> Achieve the same through GIR annotations, just like we already do for libvirt-glib. --- libvirt-gobject/libvirt-gobject-main.c | 8 ++++++-- vapi/LibvirtGObject-1.0.metadata | 2 -- vapi/Makefile.am | 2 +- 3 files changed, 7 insertions(+), 5 deletions(-) delete mode 100644 vapi/LibvirtGObject-1.0.metadata diff --git a/libvirt-gobject/libvirt-gobject-main.c b/libvirt-gobject/libvirt-gobject-main.c index 31d2514..f9f6e2d 100644 --- a/libvirt-gobject/libvirt-gobject-main.c +++ b/libvirt-gobject/libvirt-gobject-main.c @@ -33,7 +33,9 @@ /** * gvir_init_object: * @argc: (inout): pointer to application's argc - * @argv: (inout) (array length=argc) (allow-none): pointer to application's argv + * @argv: (array length=argc) (inout) (allow-none) (transfer none): Address of the + * <parameter>argv</parameter> parameter of main(), or %NULL. Any options + * understood by GTK+ are stripped before return. */ void gvir_init_object(int *argc, char ***argv) @@ -58,7 +60,9 @@ static void gvir_log_handler(const gchar *log_domain G_GNUC_UNUSED, /** * gvir_init_object_check: * @argc: (inout): pointer to application's argc - * @argv: (inout) (array length=argc) (allow-none): pointer to application's argv + * @argv: (array length=argc) (inout) (allow-none) (transfer none): Address of the + * <parameter>argv</parameter> parameter of main(), or %NULL. Any options + * understood by GTK+ are stripped before return. * @err: pointer to a #GError to which a message will be posted on error */ gboolean gvir_init_object_check(int *argc, diff --git a/vapi/LibvirtGObject-1.0.metadata b/vapi/LibvirtGObject-1.0.metadata deleted file mode 100644 index 90e5197..0000000 --- a/vapi/LibvirtGObject-1.0.metadata +++ /dev/null @@ -1,2 +0,0 @@ -init_object.argv unowned -init_object_check.argv unowned diff --git a/vapi/Makefile.am b/vapi/Makefile.am index 2ecc3e3..0154104 100644 --- a/vapi/Makefile.am +++ b/vapi/Makefile.am @@ -44,4 +44,4 @@ CLEANFILES = \ libvirt-gobject-1.0.vapi \ $(NULL) -EXTRA_DIST = LibvirtGObject-1.0.metadata libvirt-gobject-1.0.deps +EXTRA_DIST = libvirt-gobject-1.0.deps -- 1.8.1.4

2 1

[libvirt] [libvirt-designer 0/9] Handle more devices in GVirDesignerDomain
by Christophe Fergeau 19 Apr '13

19 Apr '13

This series builds on the previous one (the one adding gvir_designer_domain_get_supported_devices()). It adds methods to add more devices disks/network interfaces to GVirDesignDomain. Some parts like 'Automatically add SPICE channel with SPICE graphics' may not be what we want, but I've tried to make the external API as easy as possible. Christophe

3 13

[libvirt] Improve disk bus type generation
by Christophe Fergeau 19 Apr '13

19 Apr '13

This is mostly an excuse to add a _get_supported_devices() method used in fallback cases (when the deployment does not specify a disk driver to be used) as this will be useful later. The list of supported devices is generated with - devices supported both by the OS and the platform (hypervisor) - devices for which we have a OS driver, and which are supported by the platform When doing this work, I realized a few things were missing from libosinfo: - to properly guess the bus to use for a disk (sata, ide, scsi, ...) from a libosinfo device, we'd need to have the PCI subclass in libosinfo. This will probably need to be added at some point - something that would be nice for easier fallback when the deployment does not specify a device would be a way for OS/platforms to indicate that their preferred disk device is virtio-disk, then sata device XXX, ... I also think deployments would be more flexible if a deployment meant for qemu-1.2.0 could be used with qemu-1.3.0. Currently, osinfo_db_find_deployment does not follow 'upgrades' relations on platforms, but I think it would be more convenient if it did. Christophe

2 7

[libvirt] [libvirt-designer PATCHv2] Add support for floppies and CDROMs
by Christophe Fergeau 19 Apr '13

19 Apr '13

This mirrors disk_file/disk_device API so that it's possible to add CDROMs and floppies to a GVirDesignerDomain. This also adds the corresponding -C/-F options to virtxml --- Here is a reworked version of my patch to add support for CDROMs which does not override the 'format' parameter this time. Christophe examples/virtxml.c | 108 +++++++++++++++++++++--- libvirt-designer/libvirt-designer-domain.c | 127 +++++++++++++++++++++++++++++ libvirt-designer/libvirt-designer-domain.h | 14 ++++ libvirt-designer/libvirt-designer.sym | 4 + 4 files changed, 240 insertions(+), 13 deletions(-) diff --git a/examples/virtxml.c b/examples/virtxml.c index 49a7b4e..d4c5967 100644 --- a/examples/virtxml.c +++ b/examples/virtxml.c @@ -33,7 +33,9 @@ #include <unistd.h> #include <glib/gprintf.h> +GList *cdrom_str_list = NULL; GList *disk_str_list = NULL; +GList *floppy_str_list = NULL; GList *iface_str_list = NULL; OsinfoDb *db = NULL; @@ -173,16 +175,17 @@ cleanup: return TRUE; } + static void -add_disk(gpointer data, - gpointer user_data) +add_disk_generic(GVirDesignerDomain *domain, + const char *path, + GVirConfigDomainDiskGuestDeviceType type) { - GVirDesignerDomain *domain = (GVirDesignerDomain *) user_data; GVirConfigDomainDisk *disk; - char *path = (char *) data; char *format = NULL; struct stat buf; GError *error = NULL; + gboolean is_device; format = strchr(path, ','); if (format) { @@ -195,12 +198,30 @@ add_disk(gpointer data, exit(EXIT_FAILURE); } - if (!stat(path, &buf) && - !S_ISREG(buf.st_mode)) { - disk = gvir_designer_domain_add_disk_device(domain, path, &error); - } else { - disk = gvir_designer_domain_add_disk_file(domain, path, format, &error); + is_device = (!stat(path, &buf) && !S_ISREG(buf.st_mode)); + switch(type) { + case GVIR_CONFIG_DOMAIN_DISK_GUEST_DEVICE_CDROM: + if (is_device) + disk = gvir_designer_domain_add_cdrom_device(domain, path, &error); + else + disk = gvir_designer_domain_add_cdrom_file(domain, path, format, &error); + break; + case GVIR_CONFIG_DOMAIN_DISK_GUEST_DEVICE_DISK: + if (is_device) + disk = gvir_designer_domain_add_disk_device(domain, path, &error); + else + disk = gvir_designer_domain_add_disk_file(domain, path, format, &error); + break; + case GVIR_CONFIG_DOMAIN_DISK_GUEST_DEVICE_FLOPPY: + if (is_device) + disk = gvir_designer_domain_add_floppy_device(domain, path, &error); + else + disk = gvir_designer_domain_add_floppy_file(domain, path, format, &error); + break; + default: + g_return_if_reached(); } + if (disk) g_object_unref(G_OBJECT(disk)); @@ -210,6 +231,42 @@ add_disk(gpointer data, } } + +static void add_cdrom(gpointer data, gpointer user_data) +{ + add_disk_generic(GVIR_DESIGNER_DOMAIN(user_data), + (const char *)data, + GVIR_CONFIG_DOMAIN_DISK_GUEST_DEVICE_CDROM); +} + + +static void add_disk(gpointer data, gpointer user_data) +{ + add_disk_generic(GVIR_DESIGNER_DOMAIN(user_data), + (const char *)data, + GVIR_CONFIG_DOMAIN_DISK_GUEST_DEVICE_DISK); +} + + +static void add_floppy(gpointer data, gpointer user_data) +{ + add_disk_generic(GVIR_DESIGNER_DOMAIN(user_data), + (const char *)data, + GVIR_CONFIG_DOMAIN_DISK_GUEST_DEVICE_FLOPPY); +} + + +static gboolean +add_cdrom_str(const gchar *option_name, + const gchar *value, + gpointer data, + GError **error) +{ + cdrom_str_list = g_list_append(cdrom_str_list, g_strdup(value)); + return TRUE; +} + + static gboolean add_disk_str(const gchar *option_name, const gchar *value, @@ -220,6 +277,18 @@ add_disk_str(const gchar *option_name, return TRUE; } + +static gboolean +add_floppy_str(const gchar *option_name, + const gchar *value, + gpointer data, + GError **error) +{ + floppy_str_list = g_list_append(floppy_str_list, g_strdup(value)); + return TRUE; +} + + static void add_iface(gpointer data, gpointer user_data) @@ -357,7 +426,7 @@ cleanup: } static OsinfoOs * -guess_os_from_disk(GList *disk_list) +guess_os_from_cdrom(GList *disk_list) { OsinfoOs *ret = NULL; GList *list_it = NULL; @@ -512,8 +581,12 @@ main(int argc, char *argv[]) "set hypervisor under which domain will be running", "PLATFORM"}, {"architecture", 'a', 0, G_OPTION_ARG_STRING, &arch_str, "set domain architecture", "ARCH"}, + {"cdrom", 'C', 0, G_OPTION_ARG_CALLBACK, add_cdrom_str, + "add CDROM to domain with PATH being source and FORMAT its format", "PATH[,FORMAT]"}, {"disk", 'd', 0, G_OPTION_ARG_CALLBACK, add_disk_str, "add disk to domain with PATH being source and FORMAT its format", "PATH[,FORMAT]"}, + {"floppy", 'F', 0, G_OPTION_ARG_CALLBACK, add_floppy_str, + "add floppy to domain with PATH being source and FORMAT its format", "PATH[,FORMAT]"}, {"interface", 'i', 0, G_OPTION_ARG_CALLBACK, add_iface_str, "add interface with NETWORK source. Possible ARGs: mac, link={up,down}", "NETWORK[,ARG=VAL]"}, {"resources", 'r', 0, G_OPTION_ARG_STRING, &resources_str, @@ -543,7 +616,7 @@ main(int argc, char *argv[]) if (!os) os = find_os_by_short_id(os_str); } else { - os = guess_os_from_disk(disk_str_list); + os = guess_os_from_cdrom(cdrom_str_list); } if (!os) { @@ -593,8 +666,12 @@ main(int argc, char *argv[]) NULL); } + g_list_foreach(cdrom_str_list, add_cdrom, domain); + g_list_foreach(disk_str_list, add_disk, domain); + g_list_foreach(floppy_str_list, add_floppy, domain); + g_list_foreach(iface_str_list, add_iface, domain); config = gvir_designer_domain_get_config(domain); @@ -683,6 +760,11 @@ is and ID which can be obtained via I<--list-platform>. Set domain's architecture +=item -C PATH[,FORMAT] --cdrom=PATH[,FORMAT] + +Add I<PATH> as a CDROM to the domain. To specify its format (e.g. raw, +qcow2, phy) use I<FORMAT>. + =item -d PATH[,FORMAT] --disk=PATH[,FORMAT] Add I<PATH> as a disk to the domain. To specify its format (e.g. raw, @@ -712,12 +794,12 @@ platform. Usually, the platform is guessed from the connection URI. Domain with Fedora 17 from locally stored ISO and one NIC with mac 00:11:22:33:44:55 and link set down: - # virtxml -d Fedora-17-x86_64-Live-KDE.iso \ + # virtxml -C Fedora-17-x86_64-Live-KDE.iso \ -i default,mac=00:11:22:33:44:55,link=down To add multiple devices just use appropriate argument multiple times: - # virtxml -d /tmp/Fedora-17-x86_64-Live-KDE.iso,raw \ + # virtxml -d /tmp/home.img,qcow2 \ -d /var/lib/libvirt/images/f17.img,qcow2 \ -i default,mac=00:11:22:33:44:55,link=down \ -i blue_network \ diff --git a/libvirt-designer/libvirt-designer-domain.c b/libvirt-designer/libvirt-designer-domain.c index 0d47d3c..2226d39 100644 --- a/libvirt-designer/libvirt-designer-domain.c +++ b/libvirt-designer/libvirt-designer-domain.c @@ -841,6 +841,7 @@ gvir_designer_domain_next_disk_target(GVirDesignerDomain *design, static GVirConfigDomainDisk * gvir_designer_domain_add_disk_full(GVirDesignerDomain *design, GVirConfigDomainDiskType type, + GVirConfigDomainDiskGuestDeviceType guest_type, const char *path, const char *format, gchar *target, @@ -960,6 +961,7 @@ GVirConfigDomainDisk *gvir_designer_domain_add_disk_file(GVirDesignerDomain *des ret = gvir_designer_domain_add_disk_full(design, GVIR_CONFIG_DOMAIN_DISK_FILE, + GVIR_CONFIG_DOMAIN_DISK_GUEST_DEVICE_DISK, filepath, format, NULL, @@ -989,6 +991,7 @@ GVirConfigDomainDisk *gvir_designer_domain_add_disk_device(GVirDesignerDomain *d ret = gvir_designer_domain_add_disk_full(design, GVIR_CONFIG_DOMAIN_DISK_BLOCK, + GVIR_CONFIG_DOMAIN_DISK_GUEST_DEVICE_DISK, devpath, "raw", NULL, @@ -996,6 +999,130 @@ GVirConfigDomainDisk *gvir_designer_domain_add_disk_device(GVirDesignerDomain *d return ret; } +/** + * gvir_designer_domain_add_cdrom_file: + * @design: (transfer none): the domain designer instance + * @filepath: (transfer none): the path to a file + * @format: (transfer none): file format + * @error: return location for a #GError, or NULL + * + * Add a new disk to the domain. + * + * Returns: (transfer full): the pointer to new cdrom. + * If something fails NULL is returned and @error is set. + */ +GVirConfigDomainDisk *gvir_designer_domain_add_cdrom_file(GVirDesignerDomain *design, + const char *filepath, + const char *format, + GError **error) +{ + g_return_val_if_fail(GVIR_DESIGNER_IS_DOMAIN(design), NULL); + + GVirConfigDomainDisk *ret = NULL; + + ret = gvir_designer_domain_add_disk_full(design, + GVIR_CONFIG_DOMAIN_DISK_FILE, + GVIR_CONFIG_DOMAIN_DISK_GUEST_DEVICE_CDROM, + filepath, + format, + NULL, + error); + return ret; +} + + +/** + * gvir_designer_domain_add_cdrom_device: + * @design: (transfer none): the domain designer instance + * @devpath: (transfer none): path to the device + * @error: return location for a #GError, or NULL + * + * Add given device as a new cdrom to the domain designer instance. + * + * Returns: (transfer full): the pointer to the new cdrom. + * If something fails NULL is returned and @error is set. + */ +GVirConfigDomainDisk *gvir_designer_domain_add_cdrom_device(GVirDesignerDomain *design, + const char *devpath, + GError **error) +{ + g_return_val_if_fail(GVIR_DESIGNER_IS_DOMAIN(design), NULL); + + GVirConfigDomainDisk *ret = NULL; + + ret = gvir_designer_domain_add_disk_full(design, + GVIR_CONFIG_DOMAIN_DISK_BLOCK, + GVIR_CONFIG_DOMAIN_DISK_GUEST_DEVICE_CDROM, + devpath, + "raw", + NULL, + error); + return ret; +} + + +/** + * gvir_designer_domain_add_floppy_file: + * @design: (transfer none): the domain designer instance + * @filepath: (transfer none): the path to a file + * @format: (transfer none): file format + * @error: return location for a #GError, or NULL + * + * Add a new disk to the domain. + * + * Returns: (transfer full): the pointer to new floppy. + * If something fails NULL is returned and @error is set. + */ +GVirConfigDomainDisk *gvir_designer_domain_add_floppy_file(GVirDesignerDomain *design, + const char *filepath, + const char *format, + GError **error) +{ + g_return_val_if_fail(GVIR_DESIGNER_IS_DOMAIN(design), NULL); + + GVirConfigDomainDisk *ret = NULL; + + ret = gvir_designer_domain_add_disk_full(design, + GVIR_CONFIG_DOMAIN_DISK_FILE, + GVIR_CONFIG_DOMAIN_DISK_GUEST_DEVICE_FLOPPY, + filepath, + format, + NULL, + error); + return ret; +} + + +/** + * gvir_designer_domain_add_floppy_device: + * @design: (transfer none): the domain designer instance + * @devpath: (transfer none): path to the device + * @error: return location for a #GError, or NULL + * + * Add given device as a new floppy to the domain designer instance. + * + * Returns: (transfer full): the pointer to the new floppy. + * If something fails NULL is returned and @error is set. + */ +GVirConfigDomainDisk *gvir_designer_domain_add_floppy_device(GVirDesignerDomain *design, + const char *devpath, + GError **error) +{ + g_return_val_if_fail(GVIR_DESIGNER_IS_DOMAIN(design), NULL); + + GVirConfigDomainDisk *ret = NULL; + + ret = gvir_designer_domain_add_disk_full(design, + GVIR_CONFIG_DOMAIN_DISK_BLOCK, + GVIR_CONFIG_DOMAIN_DISK_GUEST_DEVICE_FLOPPY, + devpath, + "raw", + NULL, + error); + return ret; +} + + static const gchar * gvir_designer_domain_get_preferred_nic_model(GVirDesignerDomain *design, diff --git a/libvirt-designer/libvirt-designer-domain.h b/libvirt-designer/libvirt-designer-domain.h index 4d68ff6..99c280b 100644 --- a/libvirt-designer/libvirt-designer-domain.h +++ b/libvirt-designer/libvirt-designer-domain.h @@ -99,6 +99,13 @@ gboolean gvir_designer_domain_setup_container_full(GVirDesignerDomain *design, const char *arch, GError **error); +GVirConfigDomainDisk *gvir_designer_domain_add_cdrom_file(GVirDesignerDomain *design, + const char *filepath, + const char *format, + GError **error); +GVirConfigDomainDisk *gvir_designer_domain_add_cdrom_device(GVirDesignerDomain *design, + const char *devpath, + GError **error); GVirConfigDomainDisk *gvir_designer_domain_add_disk_file(GVirDesignerDomain *design, const char *filepath, const char *format, @@ -106,6 +113,13 @@ GVirConfigDomainDisk *gvir_designer_domain_add_disk_file(GVirDesignerDomain *des GVirConfigDomainDisk *gvir_designer_domain_add_disk_device(GVirDesignerDomain *design, const char *devpath, GError **error); +GVirConfigDomainDisk *gvir_designer_domain_add_floppy_file(GVirDesignerDomain *design, + const char *filepath, + const char *format, + GError **error); +GVirConfigDomainDisk *gvir_designer_domain_add_floppy_device(GVirDesignerDomain *design, + const char *devpath, + GError **error); GVirConfigDomainInterface *gvir_designer_domain_add_interface_network(GVirDesignerDomain *design, const char *network, diff --git a/libvirt-designer/libvirt-designer.sym b/libvirt-designer/libvirt-designer.sym index 79db09f..7a24f2c 100644 --- a/libvirt-designer/libvirt-designer.sym +++ b/libvirt-designer/libvirt-designer.sym @@ -10,8 +10,12 @@ LIBVIRT_DESIGNER_0.0.1 { gvir_designer_domain_get_platform; gvir_designer_domain_get_capabilities; + gvir_designer_domain_add_cdrom_file; + gvir_designer_domain_add_cdrom_device; gvir_designer_domain_add_disk_file; gvir_designer_domain_add_disk_device; + gvir_designer_domain_add_floppy_file; + gvir_designer_domain_add_floppy_device; gvir_designer_domain_add_interface_network; gvir_designer_domain_setup_resources; gvir_designer_domain_resources_get_type; -- 1.8.1.4

2 2

Re: [libvirt] [PATCH 2/3][ocaml] Add event callback implementation based on virConnectDomainEventRegisterAny
by David Scott 18 Apr '13

18 Apr '13

Hi, I just spotted a flaw in my OCaml event callback patch. I was hoping to ensure the ocaml bindings build against libvirt 0.9.1 and later. For functions, I remembered to include the usual symbol detection magic. However I forgot that there are also some enums which were added later than 0.9.1 i.e. all of these: On Wed, Apr 17, 2013 at 11:16 AM, David Scott <scott.dj(a)gmail.com> wrote: [snip] > + case VIR_DOMAIN_EVENT_ID_IO_ERROR_REASON: > + cb = VIR_DOMAIN_EVENT_CALLBACK(s_s_i_s_callback); > + break; > + case VIR_DOMAIN_EVENT_ID_CONTROL_ERROR: > + cb = VIR_DOMAIN_EVENT_CALLBACK(u_callback); > + break; > + case VIR_DOMAIN_EVENT_ID_BLOCK_JOB: > + cb = VIR_DOMAIN_EVENT_CALLBACK(s_i_i_callback); > + break; > + case VIR_DOMAIN_EVENT_ID_DISK_CHANGE: > + cb = VIR_DOMAIN_EVENT_CALLBACK(s_s_s_i_callback); > + break; > + case VIR_DOMAIN_EVENT_ID_TRAY_CHANGE: > + cb = VIR_DOMAIN_EVENT_CALLBACK(s_i_callback); > + break; > + case VIR_DOMAIN_EVENT_ID_PMWAKEUP: > + cb = VIR_DOMAIN_EVENT_CALLBACK(i_callback); > + break; > + case VIR_DOMAIN_EVENT_ID_PMSUSPEND: > + cb = VIR_DOMAIN_EVENT_CALLBACK(i_callback); > + break; > + case VIR_DOMAIN_EVENT_ID_BALLOON_CHANGE: > + cb = VIR_DOMAIN_EVENT_CALLBACK(i64_callback); > + break; > + case VIR_DOMAIN_EVENT_ID_PMSUSPEND_DISK: > + cb = VIR_DOMAIN_EVENT_CALLBACK(i_callback); > + break; > These currently cause the build to break against older libvirts. I'll rework this and resubmit once I've verified it definitely does work against an older version. Sorry for the noise! Cheers, Dave

1 0

[libvirt] [PATCH] Change default resource partition to /machine
by Daniel P. Berrange 18 Apr '13

18 Apr '13

From: "Daniel P. Berrange" <berrange(a)redhat.com> After discussions with systemd developers it was decided that a better default policy for resource partitions is to have 3 default partitions at the top level /system - system services /machine - virtual machines / containers /user - user login session This ensures that the default policy isolates guest from user login sessions & system services, so a mis-behaving guest can't consume 100% of CPU usage if other things are contending for it. Thus we change the default partition from /system to /machine Signed-off-by: Daniel P. Berrange <berrange(a)redhat.com> --- src/lxc/lxc_cgroup.c | 2 +- src/qemu/qemu_cgroup.c | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/src/lxc/lxc_cgroup.c b/src/lxc/lxc_cgroup.c index 0a43b61..7311489 100644 --- a/src/lxc/lxc_cgroup.c +++ b/src/lxc/lxc_cgroup.c @@ -537,7 +537,7 @@ virCgroupPtr virLXCCgroupCreate(virDomainDefPtr def, bool startup) goto cleanup; } - if (!(res->partition = strdup("/system"))) { + if (!(res->partition = strdup("/machine"))) { virReportOOMError(); VIR_FREE(res); goto cleanup; diff --git a/src/qemu/qemu_cgroup.c b/src/qemu/qemu_cgroup.c index 9a7bffb..3a58f24 100644 --- a/src/qemu/qemu_cgroup.c +++ b/src/qemu/qemu_cgroup.c @@ -236,7 +236,7 @@ int qemuInitCgroup(virQEMUDriverPtr driver, goto cleanup; } - if (!(res->partition = strdup("/system"))) { + if (!(res->partition = strdup("/machine"))) { virReportOOMError(); VIR_FREE(res); goto cleanup; -- 1.7.11.7

3 2

[libvirt] [PATCH 0/3] Speeding up the wait for session daemon
by Martin Kletzander 18 Apr '13

18 Apr '13

When the session daemon is spawned and ends with an error before the spawning process notices, it becomes waiting for Godot (fortunately with a timeout of about 20 seconds). With this series, there is pidfile created for the spawned daemon and the pid is being checked for to make sure we are not wasting precious time. Resolves: https://bugzilla.redhat.com/show_bug.cgi?id=927369 Martin Kletzander (3): Add missing error reporting Move daemonPidFilePath into virutil.c Speed up waiting for the session daemon daemon/libvirtd.c | 43 +++---------------------------------------- src/libvirt_private.syms | 1 + src/rpc/virnetsocket.c | 41 +++++++++++++++++++++++++++++++++++------ src/util/virutil.c | 45 ++++++++++++++++++++++++++++++++++++++++++++- src/util/virutil.h | 4 ++++ 5 files changed, 87 insertions(+), 47 deletions(-) -- 1.8.1.5

3 8

[libvirt] Add USER support to virt-sandbox-service
by dwalsh＠redhat.com 18 Apr '13

18 Apr '13

This patch set is adding support for UID/GID/USERNAME/USERDIR for use with openshift containers [sandbox PATCH 1/6] Add UID/GID support for use with interactive [sandbox PATCH 2/6] We should not turn on the sanbox shell by [sandbox PATCH 3/6] Only create the destination directory if it does [sandbox PATCH 4/6] Add support for --homedir and --username setting [sandbox PATCH 5/6] Add support for --uid, --gid, --username, [sandbox PATCH 6/6] Update man page to document current

2 12

[libvirt] [PATCH 0/2] Clean up and fix handling of --holdtime in the virsh send-key command
by Peter Krempa 18 Apr '13

18 Apr '13

Peter Krempa (2): virsh-domain: Clean up cmdSendKey virsh-domain: Report errors on invalid --holdtime value for cmdSendKey tools/virsh-domain.c | 14 ++++++++------ 1 file changed, 8 insertions(+), 6 deletions(-) -- 1.8.1.5

2 5

[libvirt] [PATCH] Reusing the result of virArchFromHost instead of calling it multiple times
by Tal Kain 18 Apr '13

18 Apr '13

Signed-off-by: Tal Kain <tal.kain(a)ravellosystems.com> --- src/qemu/qemu_capabilities.c | 13 +++++++------ 1 file changed, 7 insertions(+), 6 deletions(-) diff --git a/src/qemu/qemu_capabilities.c b/src/qemu/qemu_capabilities.c index 50712b0..b235059 100644 --- a/src/qemu/qemu_capabilities.c +++ b/src/qemu/qemu_capabilities.c @@ -860,8 +860,8 @@ virCapsPtr virQEMUCapsInit(virQEMUCapsCachePtr cache) { virCapsPtr caps; int i; - - if ((caps = virCapabilitiesNew(virArchFromHost(), + virArch hostarch = virArchFromHost(); + if ((caps = virCapabilitiesNew(hostarch, 1, 1)) == NULL) goto error; @@ -874,7 +874,7 @@ virCapsPtr virQEMUCapsInit(virQEMUCapsCachePtr cache) VIR_WARN("Failed to query host NUMA topology, disabling NUMA capabilities"); } - if (virQEMUCapsInitCPU(caps, virArchFromHost()) < 0) + if (virQEMUCapsInitCPU(caps, hostarch) < 0) VIR_WARN("Failed to get host CPU"); /* Add the power management features of the host */ @@ -891,7 +891,7 @@ virCapsPtr virQEMUCapsInit(virQEMUCapsCachePtr cache) */ for (i = 0 ; i < VIR_ARCH_LAST ; i++) if (virQEMUCapsInitGuest(caps, cache, - virArchFromHost(), + hostarch, i) < 0) goto error; @@ -1639,13 +1639,14 @@ int virQEMUCapsGetDefaultVersion(virCapsPtr caps, if (*version > 0) return 0; + virArch hostarch = virArchFromHost(); if ((binary = virCapabilitiesDefaultGuestEmulator(caps, "hvm", - virArchFromHost(), + hostarch, "qemu")) == NULL) { virReportError(VIR_ERR_INTERNAL_ERROR, _("Cannot find suitable emulator for %s"), - virArchToString(virArchFromHost())); + virArchToString(hostarch)); return -1; } -- 1.7.9.5

3 5

[libvirt] libvirt-1.0.3 vs qemu-1.4
by Yin Olivia-R63875 18 Apr '13

18 Apr '13

Hi guys, I used to go through libvirt (1.0.3) use cases with qemu-1.0 + linux-3.0.43. Recently I tried Linux-3.8.4 + qemu-1.4. But it can't start qemu domain. --------------------------------- root@p1022ds:~# virsh version Compiled against library: libvirt 1.0.3 Using library: libvirt 1.0.3 Using API: QEMU 1.0.3 Running hypervisor: QEMU 1.4.0 root@p1022ds:~# cat test.args /usr/bin/qemu-system-ppc -name test -M mpc8544ds -enable-kvm -m 256 -nographic -kernel /media/ram/uImage -initrd /media/ram/ramdisk -append "root=/dev/ram rw console=ttyS0,115200" -serial pty root@p1022ds:~# virsh domxml-from-native qemu-argv test.args >test.xml root@p1022ds:~# virsh define test.xml Domain test defined from test.xml root@p1022ds:~# virsh start test error: Failed to start domain test error: internal error no assigned pty for device charserial0 --------------------------------- I don't know if it depends on qemu version or kernel config. Do you have any idea about this issue? Best Regards, Olivia

1 0

[libvirt] [PATCH] Switch to a more extensible annotation system for RPC protocols
by Daniel P. Berrange 18 Apr '13

18 Apr '13

From: "Daniel P. Berrange" <berrange(a)redhat.com> Currently the RPC protocol files can contain annotations after the protocol enum eg REMOTE_PROC_DOMAIN_SNAPSHOT_LIST_CHILDREN_NAMES = 247, /* autogen autogen priority:high */ This is not very extensible as the number of annotations grows. Change it to use /** * @generate: both * @priority: high */ REMOTE_PROC_DOMAIN_SNAPSHOT_LIST_CHILDREN_NAMES = 247, Signed-off-by: Daniel P. Berrange <berrange(a)redhat.com> --- src/locking/lock_protocol.x | 68 +- src/remote/lxc_protocol.x | 39 +- src/remote/qemu_protocol.x | 51 +- src/remote/remote_protocol.x | 1832 +++++++++++++++++++++++++++++++++--------- src/rpc/gendispatch.pl | 67 +- 5 files changed, 1638 insertions(+), 419 deletions(-) diff --git a/src/locking/lock_protocol.x b/src/locking/lock_protocol.x index 5f40f9a..aef928e 100644 --- a/src/locking/lock_protocol.x +++ b/src/locking/lock_protocol.x @@ -77,19 +77,63 @@ const VIR_LOCK_SPACE_PROTOCOL_PROGRAM = 0xEA7BEEF; const VIR_LOCK_SPACE_PROTOCOL_PROGRAM_VERSION = 1; enum virLockSpaceProtocolProcedure { - /* Each function must have a two-word comment. The first word is - * whether remote_generator.pl handles daemon, the second whether - * it handles src/remote. Additional flags can be specified after a - * pipe. + /* Each function must be preceeded by a comment providing one or + * more annotations: + * + * - @generate: none|client|server|both + * + * Whether to generate the dispatch stubs for the server + * and/or client code. + * + * - @readstream: paramnumber + * - @writestream: paramnumber + * + * The @readstream or @writestream annotations let daemon and src/remote + * create a stream. The direction is defined from the src/remote point + * of view. A readstream transfers data from daemon to src/remote. The + * <paramnumber> specifies at which offset the stream parameter is inserted + * in the function parameter list. + * + * - @priority: low|high + * + * Each API that might eventually access hypervisor's monitor (and thus + * block) MUST fall into low priority. However, there are some exceptions + * to this rule, e.g. domainDestroy. Other APIs MAY be marked as high + * priority. If in doubt, it's safe to choose low. Low is taken as default, + * and thus can be left out. */ - VIR_LOCK_SPACE_PROTOCOL_PROC_REGISTER = 1, /* skipgen skipgen */ - VIR_LOCK_SPACE_PROTOCOL_PROC_RESTRICT = 2, /* skipgen skipgen */ - VIR_LOCK_SPACE_PROTOCOL_PROC_NEW = 3, /* skipgen skipgen */ - VIR_LOCK_SPACE_PROTOCOL_PROC_CREATE_RESOURCE = 4, /* skipgen skipgen */ - VIR_LOCK_SPACE_PROTOCOL_PROC_DELETE_RESOURCE = 5, /* skipgen skipgen */ + /** + * @generate: none + */ + VIR_LOCK_SPACE_PROTOCOL_PROC_REGISTER = 1, + /** + * @generate: none + */ + VIR_LOCK_SPACE_PROTOCOL_PROC_RESTRICT = 2, + /** + * @generate: none + */ + VIR_LOCK_SPACE_PROTOCOL_PROC_NEW = 3, + /** + * @generate: none + */ + VIR_LOCK_SPACE_PROTOCOL_PROC_CREATE_RESOURCE = 4, + /** + * @generate: none + */ + VIR_LOCK_SPACE_PROTOCOL_PROC_DELETE_RESOURCE = 5, - VIR_LOCK_SPACE_PROTOCOL_PROC_ACQUIRE_RESOURCE = 6, /* skipgen skipgen */ - VIR_LOCK_SPACE_PROTOCOL_PROC_RELEASE_RESOURCE = 7, /* skipgen skipgen */ + /** + * @generate: none + */ + VIR_LOCK_SPACE_PROTOCOL_PROC_ACQUIRE_RESOURCE = 6, + /** + * @generate: none + */ + VIR_LOCK_SPACE_PROTOCOL_PROC_RELEASE_RESOURCE = 7, - VIR_LOCK_SPACE_PROTOCOL_PROC_CREATE_LOCKSPACE = 8 /* skipgen skipgen */ + /** + * @generate: none + */ + VIR_LOCK_SPACE_PROTOCOL_PROC_CREATE_LOCKSPACE = 8 }; diff --git a/src/remote/lxc_protocol.x b/src/remote/lxc_protocol.x index 1453f89..020fcba 100644 --- a/src/remote/lxc_protocol.x +++ b/src/remote/lxc_protocol.x @@ -37,13 +37,34 @@ const LXC_PROGRAM = 0x00068000; const LXC_PROTOCOL_VERSION = 1; enum lxc_procedure { - /* Each function must have a three-word comment. The first word is - * whether gendispatch.pl handles daemon, the second whether - * it handles src/remote. - * The last argument describes priority of API. There are two accepted - * values: low, high; Each API that might eventually access hypervisor's - * monitor (and thus block) MUST fall into low priority. However, there - * are some exceptions to this rule, e.g. domainDestroy. Other APIs MAY - * be marked as high priority. If in doubt, it's safe to choose low. */ - LXC_PROC_DOMAIN_OPEN_NAMESPACE = 1 /* skipgen skipgen priority:low */ + /* Each function must be preceeded by a comment providing one or + * more annotations: + * + * - @generate: none|client|server|both + * + * Whether to generate the dispatch stubs for the server + * and/or client code. + * + * - @readstream: paramnumber + * - @writestream: paramnumber + * + * The @readstream or @writestream annotations let daemon and src/remote + * create a stream. The direction is defined from the src/remote point + * of view. A readstream transfers data from daemon to src/remote. The + * <paramnumber> specifies at which offset the stream parameter is inserted + * in the function parameter list. + * + * - @priority: low|high + * + * Each API that might eventually access hypervisor's monitor (and thus + * block) MUST fall into low priority. However, there are some exceptions + * to this rule, e.g. domainDestroy. Other APIs MAY be marked as high + * priority. If in doubt, it's safe to choose low. Low is taken as default, + * and thus can be left out. + */ + /** + * @generate: none + * @priority: low + */ + LXC_PROC_DOMAIN_OPEN_NAMESPACE = 1 }; diff --git a/src/remote/qemu_protocol.x b/src/remote/qemu_protocol.x index d555cbb..e284180 100644 --- a/src/remote/qemu_protocol.x +++ b/src/remote/qemu_protocol.x @@ -63,15 +63,44 @@ const QEMU_PROGRAM = 0x20008087; const QEMU_PROTOCOL_VERSION = 1; enum qemu_procedure { - /* Each function must have a three-word comment. The first word is - * whether gendispatch.pl handles daemon, the second whether - * it handles src/remote. - * The last argument describes priority of API. There are two accepted - * values: low, high; Each API that might eventually access hypervisor's - * monitor (and thus block) MUST fall into low priority. However, there - * are some exceptions to this rule, e.g. domainDestroy. Other APIs MAY - * be marked as high priority. If in doubt, it's safe to choose low. */ - QEMU_PROC_MONITOR_COMMAND = 1, /* skipgen skipgen priority:low */ - QEMU_PROC_DOMAIN_ATTACH = 2, /* autogen autogen priority:low */ - QEMU_PROC_DOMAIN_AGENT_COMMAND = 3 /* autogen autogen priority:low */ + /* Each function must be preceeded by a comment providing one or + * more annotations: + * + * - @generate: none|client|server|both + * + * Whether to generate the dispatch stubs for the server + * and/or client code. + * + * - @readstream: paramnumber + * - @writestream: paramnumber + * + * The @readstream or @writestream annotations let daemon and src/remote + * create a stream. The direction is defined from the src/remote point + * of view. A readstream transfers data from daemon to src/remote. The + * <paramnumber> specifies at which offset the stream parameter is inserted + * in the function parameter list. + * + * - @priority: low|high + * + * Each API that might eventually access hypervisor's monitor (and thus + * block) MUST fall into low priority. However, there are some exceptions + * to this rule, e.g. domainDestroy. Other APIs MAY be marked as high + * priority. If in doubt, it's safe to choose low. Low is taken as default, + * and thus can be left out. + */ + /** + * @generate: none + * @priority: low + */ + QEMU_PROC_MONITOR_COMMAND = 1, + /** + * @generate: both + * @priority: low + */ + QEMU_PROC_DOMAIN_ATTACH = 2, + /** + * @generate: both + * @priority: low + */ + QEMU_PROC_DOMAIN_AGENT_COMMAND = 3 }; diff --git a/src/remote/remote_protocol.x b/src/remote/remote_protocol.x index d384225..d1f5a4f 100644 --- a/src/remote/remote_protocol.x +++ b/src/remote/remote_protocol.x @@ -1921,7 +1921,7 @@ struct remote_node_device_destroy_args { }; -/** +/* * Events Register/Deregister: * It would seem rpcgen does not like both args, and ret * to be null. It will not generate the prototype otherwise. @@ -2744,370 +2744,1478 @@ const REMOTE_PROGRAM = 0x20008086; const REMOTE_PROTOCOL_VERSION = 1; enum remote_procedure { - /* Each function must have a three-word comment. The first word is - * whether gendispatch.pl handles daemon, the second whether - * it handles src/remote. Additional flags can be specified after a - * pipe. - * The last argument describes priority of API. There are two accepted - * values: low, high; Each API that might eventually access hypervisor's - * monitor (and thus block) MUST fall into low priority. However, there - * are some exceptions to this rule, e.g. domainDestroy. Other APIs MAY - * be marked as high priority. If in doubt, it's safe to choose low. - * Low is taken as default, and thus can be left out. + /* Each function must be preceeded by a comment providing one or + * more annotations: + * + * - @generate: none|client|server|both + * + * Whether to generate the dispatch stubs for the server + * and/or client code. + * + * - @readstream: paramnumber + * - @writestream: paramnumber * - * The (readstream|writestream)@<offset> flag lets daemon and src/remote - * create a stream. The direction is defined from the src/remote point - * of view. A readstream transfers data from daemon to src/remote. The - * <offset> specifies at which offset the stream parameter is inserted - * in the function parameter list. */ - REMOTE_PROC_OPEN = 1, /* skipgen skipgen priority:high */ - REMOTE_PROC_CLOSE = 2, /* skipgen skipgen priority:high */ - REMOTE_PROC_GET_TYPE = 3, /* autogen skipgen priority:high */ - REMOTE_PROC_GET_VERSION = 4, /* autogen autogen priority:high */ - REMOTE_PROC_GET_MAX_VCPUS = 5, /* autogen autogen priority:high */ - REMOTE_PROC_NODE_GET_INFO = 6, /* autogen autogen priority:high */ - REMOTE_PROC_GET_CAPABILITIES = 7, /* autogen autogen */ - REMOTE_PROC_DOMAIN_ATTACH_DEVICE = 8, /* autogen autogen */ - REMOTE_PROC_DOMAIN_CREATE = 9, /* autogen skipgen */ - REMOTE_PROC_DOMAIN_CREATE_XML = 10, /* autogen autogen */ - - REMOTE_PROC_DOMAIN_DEFINE_XML = 11, /* autogen autogen priority:high */ - REMOTE_PROC_DOMAIN_DESTROY = 12, /* autogen autogen priority:high */ - REMOTE_PROC_DOMAIN_DETACH_DEVICE = 13, /* autogen autogen */ - REMOTE_PROC_DOMAIN_GET_XML_DESC = 14, /* autogen autogen */ - REMOTE_PROC_DOMAIN_GET_AUTOSTART = 15, /* autogen autogen priority:high */ - REMOTE_PROC_DOMAIN_GET_INFO = 16, /* autogen autogen */ - REMOTE_PROC_DOMAIN_GET_MAX_MEMORY = 17, /* autogen autogen priority:high */ - REMOTE_PROC_DOMAIN_GET_MAX_VCPUS = 18, /* autogen autogen priority:high */ - REMOTE_PROC_DOMAIN_GET_OS_TYPE = 19, /* autogen autogen priority:high */ - REMOTE_PROC_DOMAIN_GET_VCPUS = 20, /* skipgen skipgen priority:high */ - - REMOTE_PROC_LIST_DEFINED_DOMAINS = 21, /* autogen autogen priority:high */ - REMOTE_PROC_DOMAIN_LOOKUP_BY_ID = 22, /* autogen autogen priority:high */ - REMOTE_PROC_DOMAIN_LOOKUP_BY_NAME = 23, /* autogen autogen priority:high */ - REMOTE_PROC_DOMAIN_LOOKUP_BY_UUID = 24, /* autogen autogen priority:high */ - REMOTE_PROC_NUM_OF_DEFINED_DOMAINS = 25, /* autogen autogen priority:high */ - REMOTE_PROC_DOMAIN_PIN_VCPU = 26, /* autogen autogen */ - REMOTE_PROC_DOMAIN_REBOOT = 27, /* autogen autogen */ - REMOTE_PROC_DOMAIN_RESUME = 28, /* autogen autogen */ - REMOTE_PROC_DOMAIN_SET_AUTOSTART = 29, /* autogen autogen priority:high */ - REMOTE_PROC_DOMAIN_SET_MAX_MEMORY = 30, /* autogen autogen priority:high */ - - REMOTE_PROC_DOMAIN_SET_MEMORY = 31, /* autogen autogen */ - REMOTE_PROC_DOMAIN_SET_VCPUS = 32, /* autogen autogen */ - REMOTE_PROC_DOMAIN_SHUTDOWN = 33, /* autogen autogen */ - REMOTE_PROC_DOMAIN_SUSPEND = 34, /* autogen autogen */ - REMOTE_PROC_DOMAIN_UNDEFINE = 35, /* autogen autogen priority:high */ - REMOTE_PROC_LIST_DEFINED_NETWORKS = 36, /* autogen autogen priority:high */ - REMOTE_PROC_LIST_DOMAINS = 37, /* autogen skipgen priority:high */ - REMOTE_PROC_LIST_NETWORKS = 38, /* autogen autogen priority:high */ - REMOTE_PROC_NETWORK_CREATE = 39, /* autogen autogen */ - REMOTE_PROC_NETWORK_CREATE_XML = 40, /* autogen autogen */ - - REMOTE_PROC_NETWORK_DEFINE_XML = 41, /* autogen autogen priority:high */ - REMOTE_PROC_NETWORK_DESTROY = 42, /* autogen autogen priority:high */ - REMOTE_PROC_NETWORK_GET_XML_DESC = 43, /* autogen autogen priority:high */ - REMOTE_PROC_NETWORK_GET_AUTOSTART = 44, /* autogen autogen priority:high */ - REMOTE_PROC_NETWORK_GET_BRIDGE_NAME = 45, /* autogen autogen priority:high */ - REMOTE_PROC_NETWORK_LOOKUP_BY_NAME = 46, /* autogen autogen priority:high */ - REMOTE_PROC_NETWORK_LOOKUP_BY_UUID = 47, /* autogen autogen priority:high */ - REMOTE_PROC_NETWORK_SET_AUTOSTART = 48, /* autogen autogen priority:high */ - REMOTE_PROC_NETWORK_UNDEFINE = 49, /* autogen autogen priority:high */ - REMOTE_PROC_NUM_OF_DEFINED_NETWORKS = 50, /* autogen autogen priority:high */ - - REMOTE_PROC_NUM_OF_DOMAINS = 51, /* autogen autogen priority:high */ - REMOTE_PROC_NUM_OF_NETWORKS = 52, /* autogen autogen priority:high */ - REMOTE_PROC_DOMAIN_CORE_DUMP = 53, /* autogen autogen */ - REMOTE_PROC_DOMAIN_RESTORE = 54, /* autogen autogen */ - REMOTE_PROC_DOMAIN_SAVE = 55, /* autogen autogen */ - REMOTE_PROC_DOMAIN_GET_SCHEDULER_TYPE = 56, /* skipgen skipgen */ - REMOTE_PROC_DOMAIN_GET_SCHEDULER_PARAMETERS = 57, /* skipgen autogen */ - REMOTE_PROC_DOMAIN_SET_SCHEDULER_PARAMETERS = 58, /* autogen autogen */ - REMOTE_PROC_GET_HOSTNAME = 59, /* autogen autogen priority:high */ - REMOTE_PROC_SUPPORTS_FEATURE = 60, /* skipgen autogen priority:high */ - - REMOTE_PROC_DOMAIN_MIGRATE_PREPARE = 61, /* skipgen skipgen */ - REMOTE_PROC_DOMAIN_MIGRATE_PERFORM = 62, /* autogen autogen */ - REMOTE_PROC_DOMAIN_MIGRATE_FINISH = 63, /* autogen autogen */ - REMOTE_PROC_DOMAIN_BLOCK_STATS = 64, /* autogen autogen */ - REMOTE_PROC_DOMAIN_INTERFACE_STATS = 65, /* autogen autogen priority:high */ - REMOTE_PROC_AUTH_LIST = 66, /* skipgen skipgen priority:high */ - REMOTE_PROC_AUTH_SASL_INIT = 67, /* skipgen skipgen priority:high */ - REMOTE_PROC_AUTH_SASL_START = 68, /* skipgen skipgen priority:high */ - REMOTE_PROC_AUTH_SASL_STEP = 69, /* skipgen skipgen priority:high */ - REMOTE_PROC_AUTH_POLKIT = 70, /* skipgen skipgen priority:high */ - - REMOTE_PROC_NUM_OF_STORAGE_POOLS = 71, /* autogen autogen priority:high */ - REMOTE_PROC_LIST_STORAGE_POOLS = 72, /* autogen autogen priority:high */ - REMOTE_PROC_NUM_OF_DEFINED_STORAGE_POOLS = 73, /* autogen autogen priority:high */ - REMOTE_PROC_LIST_DEFINED_STORAGE_POOLS = 74, /* autogen autogen priority:high */ - REMOTE_PROC_FIND_STORAGE_POOL_SOURCES = 75, /* autogen skipgen */ - REMOTE_PROC_STORAGE_POOL_CREATE_XML = 76, /* autogen autogen */ - REMOTE_PROC_STORAGE_POOL_DEFINE_XML = 77, /* autogen autogen priority:high */ - REMOTE_PROC_STORAGE_POOL_CREATE = 78, /* autogen autogen */ - REMOTE_PROC_STORAGE_POOL_BUILD = 79, /* autogen autogen */ - REMOTE_PROC_STORAGE_POOL_DESTROY = 80, /* autogen autogen priority:high */ - - REMOTE_PROC_STORAGE_POOL_DELETE = 81, /* autogen autogen */ - REMOTE_PROC_STORAGE_POOL_UNDEFINE = 82, /* autogen autogen priority:high */ - REMOTE_PROC_STORAGE_POOL_REFRESH = 83, /* autogen autogen */ - REMOTE_PROC_STORAGE_POOL_LOOKUP_BY_NAME = 84, /* autogen autogen priority:high */ - REMOTE_PROC_STORAGE_POOL_LOOKUP_BY_UUID = 85, /* autogen autogen priority:high */ - REMOTE_PROC_STORAGE_POOL_LOOKUP_BY_VOLUME = 86, /* autogen autogen priority:high */ - REMOTE_PROC_STORAGE_POOL_GET_INFO = 87, /* autogen autogen priority:high */ - REMOTE_PROC_STORAGE_POOL_GET_XML_DESC = 88, /* autogen autogen priority:high */ - REMOTE_PROC_STORAGE_POOL_GET_AUTOSTART = 89, /* autogen autogen priority:high */ - REMOTE_PROC_STORAGE_POOL_SET_AUTOSTART = 90, /* autogen autogen priority:high */ - - REMOTE_PROC_STORAGE_POOL_NUM_OF_VOLUMES = 91, /* autogen autogen priority:high */ - REMOTE_PROC_STORAGE_POOL_LIST_VOLUMES = 92, /* autogen autogen priority:high */ - REMOTE_PROC_STORAGE_VOL_CREATE_XML = 93, /* autogen autogen */ - REMOTE_PROC_STORAGE_VOL_DELETE = 94, /* autogen autogen */ - REMOTE_PROC_STORAGE_VOL_LOOKUP_BY_NAME = 95, /* autogen autogen priority:high */ - REMOTE_PROC_STORAGE_VOL_LOOKUP_BY_KEY = 96, /* autogen autogen priority:high */ - REMOTE_PROC_STORAGE_VOL_LOOKUP_BY_PATH = 97, /* autogen autogen priority:high */ - REMOTE_PROC_STORAGE_VOL_GET_INFO = 98, /* autogen autogen priority:high */ - REMOTE_PROC_STORAGE_VOL_GET_XML_DESC = 99, /* autogen autogen priority:high */ - REMOTE_PROC_STORAGE_VOL_GET_PATH = 100, /* autogen autogen priority:high */ - - REMOTE_PROC_NODE_GET_CELLS_FREE_MEMORY = 101, /* autogen skipgen priority:high */ - REMOTE_PROC_NODE_GET_FREE_MEMORY = 102, /* autogen autogen priority:high */ - REMOTE_PROC_DOMAIN_BLOCK_PEEK = 103, /* skipgen skipgen */ - REMOTE_PROC_DOMAIN_MEMORY_PEEK = 104, /* skipgen skipgen */ - REMOTE_PROC_DOMAIN_EVENTS_REGISTER = 105, /* skipgen skipgen priority:high */ - REMOTE_PROC_DOMAIN_EVENTS_DEREGISTER = 106, /* skipgen skipgen priority:high */ - REMOTE_PROC_DOMAIN_EVENT_LIFECYCLE = 107, /* autogen autogen */ - REMOTE_PROC_DOMAIN_MIGRATE_PREPARE2 = 108, /* skipgen skipgen */ - REMOTE_PROC_DOMAIN_MIGRATE_FINISH2 = 109, /* autogen autogen */ - REMOTE_PROC_GET_URI = 110, /* autogen skipgen priority:high */ - - REMOTE_PROC_NODE_NUM_OF_DEVICES = 111, /* autogen autogen priority:high */ - REMOTE_PROC_NODE_LIST_DEVICES = 112, /* autogen autogen priority:high */ - REMOTE_PROC_NODE_DEVICE_LOOKUP_BY_NAME = 113, /* autogen autogen priority:high */ - REMOTE_PROC_NODE_DEVICE_GET_XML_DESC = 114, /* autogen autogen */ - REMOTE_PROC_NODE_DEVICE_GET_PARENT = 115, /* skipgen autogen priority:high */ - REMOTE_PROC_NODE_DEVICE_NUM_OF_CAPS = 116, /* autogen autogen priority:high */ - REMOTE_PROC_NODE_DEVICE_LIST_CAPS = 117, /* autogen autogen priority:high */ - REMOTE_PROC_NODE_DEVICE_DETTACH = 118, /* autogen skipgen */ - REMOTE_PROC_NODE_DEVICE_RE_ATTACH = 119, /* autogen skipgen */ - REMOTE_PROC_NODE_DEVICE_RESET = 120, /* autogen skipgen */ - - REMOTE_PROC_DOMAIN_GET_SECURITY_LABEL = 121, /* skipgen skipgen priority:high */ - REMOTE_PROC_NODE_GET_SECURITY_MODEL = 122, /* skipgen skipgen priority:high */ - REMOTE_PROC_NODE_DEVICE_CREATE_XML = 123, /* autogen autogen */ - REMOTE_PROC_NODE_DEVICE_DESTROY = 124, /* autogen autogen priority:high */ - REMOTE_PROC_STORAGE_VOL_CREATE_XML_FROM = 125, /* autogen autogen */ - REMOTE_PROC_NUM_OF_INTERFACES = 126, /* autogen autogen priority:high */ - REMOTE_PROC_LIST_INTERFACES = 127, /* autogen autogen priority:high */ - REMOTE_PROC_INTERFACE_LOOKUP_BY_NAME = 128, /* autogen autogen priority:high */ - REMOTE_PROC_INTERFACE_LOOKUP_BY_MAC_STRING = 129, /* autogen autogen priority:high */ - REMOTE_PROC_INTERFACE_GET_XML_DESC = 130, /* autogen autogen */ - - REMOTE_PROC_INTERFACE_DEFINE_XML = 131, /* autogen autogen priority:high */ - REMOTE_PROC_INTERFACE_UNDEFINE = 132, /* autogen autogen priority:high */ - REMOTE_PROC_INTERFACE_CREATE = 133, /* autogen autogen */ - REMOTE_PROC_INTERFACE_DESTROY = 134, /* autogen autogen priority:high */ - REMOTE_PROC_DOMAIN_XML_FROM_NATIVE = 135, /* autogen autogen */ - REMOTE_PROC_DOMAIN_XML_TO_NATIVE = 136, /* autogen autogen */ - REMOTE_PROC_NUM_OF_DEFINED_INTERFACES = 137, /* autogen autogen priority:high */ - REMOTE_PROC_LIST_DEFINED_INTERFACES = 138, /* autogen autogen priority:high */ - REMOTE_PROC_NUM_OF_SECRETS = 139, /* autogen autogen priority:high */ - REMOTE_PROC_LIST_SECRETS = 140, /* autogen autogen priority:high */ - - REMOTE_PROC_SECRET_LOOKUP_BY_UUID = 141, /* autogen autogen priority:high */ - REMOTE_PROC_SECRET_DEFINE_XML = 142, /* autogen autogen priority:high */ - REMOTE_PROC_SECRET_GET_XML_DESC = 143, /* autogen autogen priority:high */ - REMOTE_PROC_SECRET_SET_VALUE = 144, /* autogen autogen priority:high */ - REMOTE_PROC_SECRET_GET_VALUE = 145, /* skipgen skipgen priority:high */ - REMOTE_PROC_SECRET_UNDEFINE = 146, /* autogen autogen priority:high */ - REMOTE_PROC_SECRET_LOOKUP_BY_USAGE = 147, /* autogen autogen priority:high */ - REMOTE_PROC_DOMAIN_MIGRATE_PREPARE_TUNNEL = 148, /* autogen autogen | writestream@1 */ - REMOTE_PROC_IS_SECURE = 149, /* autogen skipgen priority:high */ - REMOTE_PROC_DOMAIN_IS_ACTIVE = 150, /* autogen autogen priority:high */ - - REMOTE_PROC_DOMAIN_IS_PERSISTENT = 151, /* autogen autogen priority:high */ - REMOTE_PROC_NETWORK_IS_ACTIVE = 152, /* autogen autogen priority:high */ - REMOTE_PROC_NETWORK_IS_PERSISTENT = 153, /* autogen autogen priority:high */ - REMOTE_PROC_STORAGE_POOL_IS_ACTIVE = 154, /* autogen autogen priority:high */ - REMOTE_PROC_STORAGE_POOL_IS_PERSISTENT = 155, /* autogen autogen priority:high */ - REMOTE_PROC_INTERFACE_IS_ACTIVE = 156, /* autogen autogen priority:high */ - REMOTE_PROC_GET_LIB_VERSION = 157, /* autogen autogen priority:high */ - REMOTE_PROC_CPU_COMPARE = 158, /* autogen autogen priority:high */ - REMOTE_PROC_DOMAIN_MEMORY_STATS = 159, /* skipgen skipgen */ - REMOTE_PROC_DOMAIN_ATTACH_DEVICE_FLAGS = 160, /* autogen autogen */ - - REMOTE_PROC_DOMAIN_DETACH_DEVICE_FLAGS = 161, /* autogen autogen */ - REMOTE_PROC_CPU_BASELINE = 162, /* autogen autogen */ - REMOTE_PROC_DOMAIN_GET_JOB_INFO = 163, /* autogen autogen */ - REMOTE_PROC_DOMAIN_ABORT_JOB = 164, /* autogen autogen */ - REMOTE_PROC_STORAGE_VOL_WIPE = 165, /* autogen autogen */ - REMOTE_PROC_DOMAIN_MIGRATE_SET_MAX_DOWNTIME = 166, /* autogen autogen */ - REMOTE_PROC_DOMAIN_EVENTS_REGISTER_ANY = 167, /* skipgen skipgen priority:high */ - REMOTE_PROC_DOMAIN_EVENTS_DEREGISTER_ANY = 168, /* skipgen skipgen priority:high */ - REMOTE_PROC_DOMAIN_EVENT_REBOOT = 169, /* autogen autogen */ - REMOTE_PROC_DOMAIN_EVENT_RTC_CHANGE = 170, /* autogen autogen */ - - REMOTE_PROC_DOMAIN_EVENT_WATCHDOG = 171, /* autogen autogen */ - REMOTE_PROC_DOMAIN_EVENT_IO_ERROR = 172, /* autogen autogen */ - REMOTE_PROC_DOMAIN_EVENT_GRAPHICS = 173, /* autogen autogen */ - REMOTE_PROC_DOMAIN_UPDATE_DEVICE_FLAGS = 174, /* autogen autogen */ - REMOTE_PROC_NWFILTER_LOOKUP_BY_NAME = 175, /* autogen autogen priority:high */ - REMOTE_PROC_NWFILTER_LOOKUP_BY_UUID = 176, /* autogen autogen priority:high */ - REMOTE_PROC_NWFILTER_GET_XML_DESC = 177, /* autogen autogen priority:high */ - REMOTE_PROC_NUM_OF_NWFILTERS = 178, /* autogen autogen priority:high */ - REMOTE_PROC_LIST_NWFILTERS = 179, /* autogen autogen priority:high */ - REMOTE_PROC_NWFILTER_DEFINE_XML = 180, /* autogen autogen priority:high */ - - REMOTE_PROC_NWFILTER_UNDEFINE = 181, /* autogen autogen priority:high */ - REMOTE_PROC_DOMAIN_MANAGED_SAVE = 182, /* autogen autogen */ - REMOTE_PROC_DOMAIN_HAS_MANAGED_SAVE_IMAGE = 183, /* autogen autogen */ - REMOTE_PROC_DOMAIN_MANAGED_SAVE_REMOVE = 184, /* autogen autogen */ - REMOTE_PROC_DOMAIN_SNAPSHOT_CREATE_XML = 185, /* autogen autogen */ - REMOTE_PROC_DOMAIN_SNAPSHOT_GET_XML_DESC = 186, /* autogen autogen priority:high */ - REMOTE_PROC_DOMAIN_SNAPSHOT_NUM = 187, /* autogen autogen priority:high */ - REMOTE_PROC_DOMAIN_SNAPSHOT_LIST_NAMES = 188, /* autogen autogen priority:high */ - REMOTE_PROC_DOMAIN_SNAPSHOT_LOOKUP_BY_NAME = 189, /* autogen autogen priority:high */ - REMOTE_PROC_DOMAIN_HAS_CURRENT_SNAPSHOT = 190, /* autogen autogen */ - - REMOTE_PROC_DOMAIN_SNAPSHOT_CURRENT = 191, /* autogen autogen */ - REMOTE_PROC_DOMAIN_REVERT_TO_SNAPSHOT = 192, /* autogen autogen */ - REMOTE_PROC_DOMAIN_SNAPSHOT_DELETE = 193, /* autogen autogen */ - REMOTE_PROC_DOMAIN_GET_BLOCK_INFO = 194, /* autogen autogen */ - REMOTE_PROC_DOMAIN_EVENT_IO_ERROR_REASON = 195, /* autogen autogen */ - REMOTE_PROC_DOMAIN_CREATE_WITH_FLAGS = 196, /* autogen autogen */ - REMOTE_PROC_DOMAIN_SET_MEMORY_PARAMETERS = 197, /* autogen autogen */ - REMOTE_PROC_DOMAIN_GET_MEMORY_PARAMETERS = 198, /* skipgen skipgen */ - REMOTE_PROC_DOMAIN_SET_VCPUS_FLAGS = 199, /* autogen autogen */ - REMOTE_PROC_DOMAIN_GET_VCPUS_FLAGS = 200, /* autogen autogen */ - - REMOTE_PROC_DOMAIN_OPEN_CONSOLE = 201, /* autogen autogen | readstream@2 */ - REMOTE_PROC_DOMAIN_IS_UPDATED = 202, /* autogen autogen priority:high */ - REMOTE_PROC_GET_SYSINFO = 203, /* autogen autogen priority:high */ - REMOTE_PROC_DOMAIN_SET_MEMORY_FLAGS = 204, /* autogen autogen */ - REMOTE_PROC_DOMAIN_SET_BLKIO_PARAMETERS = 205, /* autogen autogen */ - REMOTE_PROC_DOMAIN_GET_BLKIO_PARAMETERS = 206, /* skipgen skipgen */ - REMOTE_PROC_DOMAIN_MIGRATE_SET_MAX_SPEED = 207, /* autogen autogen */ - REMOTE_PROC_STORAGE_VOL_UPLOAD = 208, /* autogen autogen | writestream@1 */ - REMOTE_PROC_STORAGE_VOL_DOWNLOAD = 209, /* autogen autogen | readstream@1 */ - REMOTE_PROC_DOMAIN_INJECT_NMI = 210, /* autogen autogen */ - - REMOTE_PROC_DOMAIN_SCREENSHOT = 211, /* autogen autogen | readstream@1 */ - REMOTE_PROC_DOMAIN_GET_STATE = 212, /* skipgen skipgen priority:high */ - REMOTE_PROC_DOMAIN_MIGRATE_BEGIN3 = 213, /* skipgen skipgen */ - REMOTE_PROC_DOMAIN_MIGRATE_PREPARE3 = 214, /* skipgen skipgen */ - REMOTE_PROC_DOMAIN_MIGRATE_PREPARE_TUNNEL3 = 215, /* autogen skipgen | writestream@1 */ - REMOTE_PROC_DOMAIN_MIGRATE_PERFORM3 = 216, /* skipgen skipgen */ - REMOTE_PROC_DOMAIN_MIGRATE_FINISH3 = 217, /* skipgen skipgen */ - REMOTE_PROC_DOMAIN_MIGRATE_CONFIRM3 = 218, /* skipgen skipgen */ - REMOTE_PROC_DOMAIN_SET_SCHEDULER_PARAMETERS_FLAGS = 219, /* autogen autogen */ - REMOTE_PROC_INTERFACE_CHANGE_BEGIN = 220, /* autogen autogen */ - - REMOTE_PROC_INTERFACE_CHANGE_COMMIT = 221, /* autogen autogen */ - REMOTE_PROC_INTERFACE_CHANGE_ROLLBACK = 222, /* autogen autogen */ - REMOTE_PROC_DOMAIN_GET_SCHEDULER_PARAMETERS_FLAGS = 223, /* skipgen autogen */ - REMOTE_PROC_DOMAIN_EVENT_CONTROL_ERROR = 224, /* skipgen skipgen */ - REMOTE_PROC_DOMAIN_PIN_VCPU_FLAGS = 225, /* autogen autogen */ - REMOTE_PROC_DOMAIN_SEND_KEY = 226, /* autogen autogen */ - REMOTE_PROC_NODE_GET_CPU_STATS = 227, /* skipgen skipgen priority:high */ - REMOTE_PROC_NODE_GET_MEMORY_STATS = 228, /* skipgen skipgen priority:high */ - REMOTE_PROC_DOMAIN_GET_CONTROL_INFO = 229, /* autogen autogen priority:high */ - REMOTE_PROC_DOMAIN_GET_VCPU_PIN_INFO = 230, /* skipgen skipgen */ - - REMOTE_PROC_DOMAIN_UNDEFINE_FLAGS = 231, /* autogen autogen priority:high */ - REMOTE_PROC_DOMAIN_SAVE_FLAGS = 232, /* autogen autogen */ - REMOTE_PROC_DOMAIN_RESTORE_FLAGS = 233, /* autogen autogen */ - REMOTE_PROC_DOMAIN_DESTROY_FLAGS = 234, /* autogen autogen priority:high */ - REMOTE_PROC_DOMAIN_SAVE_IMAGE_GET_XML_DESC = 235, /* autogen autogen priority:high */ - REMOTE_PROC_DOMAIN_SAVE_IMAGE_DEFINE_XML = 236, /* autogen autogen priority:high */ - REMOTE_PROC_DOMAIN_BLOCK_JOB_ABORT = 237, /* autogen autogen */ - REMOTE_PROC_DOMAIN_GET_BLOCK_JOB_INFO = 238, /* skipgen skipgen */ - REMOTE_PROC_DOMAIN_BLOCK_JOB_SET_SPEED = 239, /* autogen autogen */ - REMOTE_PROC_DOMAIN_BLOCK_PULL = 240, /* autogen autogen */ - - REMOTE_PROC_DOMAIN_EVENT_BLOCK_JOB = 241, /* skipgen skipgen */ - REMOTE_PROC_DOMAIN_MIGRATE_GET_MAX_SPEED = 242, /* autogen autogen */ - REMOTE_PROC_DOMAIN_BLOCK_STATS_FLAGS = 243, /* skipgen skipgen */ - REMOTE_PROC_DOMAIN_SNAPSHOT_GET_PARENT = 244, /* autogen autogen priority:high */ - REMOTE_PROC_DOMAIN_RESET = 245, /* autogen autogen */ - REMOTE_PROC_DOMAIN_SNAPSHOT_NUM_CHILDREN = 246, /* autogen autogen priority:high */ - REMOTE_PROC_DOMAIN_SNAPSHOT_LIST_CHILDREN_NAMES = 247, /* autogen autogen priority:high */ - REMOTE_PROC_DOMAIN_EVENT_DISK_CHANGE = 248, /* skipgen skipgen */ - REMOTE_PROC_DOMAIN_OPEN_GRAPHICS = 249, /* skipgen skipgen */ - REMOTE_PROC_NODE_SUSPEND_FOR_DURATION = 250, /* autogen autogen */ - - REMOTE_PROC_DOMAIN_BLOCK_RESIZE = 251, /* autogen autogen */ - REMOTE_PROC_DOMAIN_SET_BLOCK_IO_TUNE = 252, /* autogen autogen */ - REMOTE_PROC_DOMAIN_GET_BLOCK_IO_TUNE = 253, /* skipgen skipgen */ - REMOTE_PROC_DOMAIN_SET_NUMA_PARAMETERS = 254, /* autogen autogen */ - REMOTE_PROC_DOMAIN_GET_NUMA_PARAMETERS = 255, /* skipgen skipgen */ - REMOTE_PROC_DOMAIN_SET_INTERFACE_PARAMETERS = 256, /* autogen autogen */ - REMOTE_PROC_DOMAIN_GET_INTERFACE_PARAMETERS = 257, /* skipgen skipgen */ - REMOTE_PROC_DOMAIN_SHUTDOWN_FLAGS = 258, /* autogen autogen */ - REMOTE_PROC_STORAGE_VOL_WIPE_PATTERN = 259, /* autogen autogen */ - REMOTE_PROC_STORAGE_VOL_RESIZE = 260, /* autogen autogen */ - - REMOTE_PROC_DOMAIN_PM_SUSPEND_FOR_DURATION = 261, /* autogen autogen */ - REMOTE_PROC_DOMAIN_GET_CPU_STATS = 262, /* skipgen skipgen */ - REMOTE_PROC_DOMAIN_GET_DISK_ERRORS = 263, /* skipgen skipgen */ - REMOTE_PROC_DOMAIN_SET_METADATA = 264, /* autogen autogen */ - REMOTE_PROC_DOMAIN_GET_METADATA = 265, /* autogen autogen */ - REMOTE_PROC_DOMAIN_BLOCK_REBASE = 266, /* autogen autogen */ - REMOTE_PROC_DOMAIN_PM_WAKEUP = 267, /* autogen autogen */ - REMOTE_PROC_DOMAIN_EVENT_TRAY_CHANGE = 268, /* autogen autogen */ - REMOTE_PROC_DOMAIN_EVENT_PMWAKEUP = 269, /* autogen autogen */ - REMOTE_PROC_DOMAIN_EVENT_PMSUSPEND = 270, /* autogen autogen */ - - REMOTE_PROC_DOMAIN_SNAPSHOT_IS_CURRENT = 271, /* autogen autogen */ - REMOTE_PROC_DOMAIN_SNAPSHOT_HAS_METADATA = 272, /* autogen autogen */ - REMOTE_PROC_CONNECT_LIST_ALL_DOMAINS = 273, /* skipgen skipgen priority:high */ - REMOTE_PROC_DOMAIN_LIST_ALL_SNAPSHOTS = 274, /* skipgen skipgen priority:high */ - REMOTE_PROC_DOMAIN_SNAPSHOT_LIST_ALL_CHILDREN = 275, /* skipgen skipgen priority:high */ - REMOTE_PROC_DOMAIN_EVENT_BALLOON_CHANGE = 276, /* autogen autogen */ - REMOTE_PROC_DOMAIN_GET_HOSTNAME = 277, /* autogen autogen */ - REMOTE_PROC_DOMAIN_GET_SECURITY_LABEL_LIST = 278, /* skipgen skipgen priority:high */ - REMOTE_PROC_DOMAIN_PIN_EMULATOR = 279, /* skipgen skipgen */ - REMOTE_PROC_DOMAIN_GET_EMULATOR_PIN_INFO = 280, /* skipgen skipgen */ - - REMOTE_PROC_CONNECT_LIST_ALL_STORAGE_POOLS = 281, /* skipgen skipgen priority:high */ - REMOTE_PROC_STORAGE_POOL_LIST_ALL_VOLUMES = 282, /* skipgen skipgen priority:high */ - REMOTE_PROC_CONNECT_LIST_ALL_NETWORKS = 283, /* skipgen skipgen priority:high */ - REMOTE_PROC_CONNECT_LIST_ALL_INTERFACES = 284, /* skipgen skipgen priority:high */ - REMOTE_PROC_CONNECT_LIST_ALL_NODE_DEVICES = 285, /* skipgen skipgen priority:high */ - REMOTE_PROC_CONNECT_LIST_ALL_NWFILTERS = 286, /* skipgen skipgen priority:high */ - REMOTE_PROC_CONNECT_LIST_ALL_SECRETS = 287, /* skipgen skipgen priority:high */ - REMOTE_PROC_NODE_SET_MEMORY_PARAMETERS = 288, /* autogen autogen */ - REMOTE_PROC_NODE_GET_MEMORY_PARAMETERS = 289, /* skipgen skipgen */ - REMOTE_PROC_DOMAIN_BLOCK_COMMIT = 290, /* autogen autogen */ - - REMOTE_PROC_NETWORK_UPDATE = 291, /* autogen autogen priority:high */ - REMOTE_PROC_DOMAIN_EVENT_PMSUSPEND_DISK = 292, /* autogen autogen */ - REMOTE_PROC_NODE_GET_CPU_MAP = 293, /* skipgen skipgen */ - REMOTE_PROC_DOMAIN_FSTRIM = 294, /* autogen autogen */ - REMOTE_PROC_DOMAIN_SEND_PROCESS_SIGNAL = 295, /* autogen autogen */ - REMOTE_PROC_DOMAIN_OPEN_CHANNEL = 296, /* autogen autogen | readstream@2 */ - REMOTE_PROC_NODE_DEVICE_LOOKUP_SCSI_HOST_BY_WWN = 297, /* autogen autogen priority:high */ - REMOTE_PROC_DOMAIN_GET_JOB_STATS = 298, /* skipgen skipgen */ - REMOTE_PROC_DOMAIN_MIGRATE_GET_COMPRESSION_CACHE = 299, /* autogen autogen */ - REMOTE_PROC_DOMAIN_MIGRATE_SET_COMPRESSION_CACHE = 300 /* autogen autogen */ + * The @readstream or @writestream annotations let daemon and src/remote + * create a stream. The direction is defined from the src/remote point + * of view. A readstream transfers data from daemon to src/remote. The + * <paramnumber> specifies at which offset the stream parameter is inserted + * in the function parameter list. + * + * - @priority: low|high + * + * Each API that might eventually access hypervisor's monitor (and thus + * block) MUST fall into low priority. However, there are some exceptions + * to this rule, e.g. domainDestroy. Other APIs MAY be marked as high + * priority. If in doubt, it's safe to choose low. Low is taken as default, + * and thus can be left out. + */ + + /** + * @generate: none + * @priority: high + */ + REMOTE_PROC_OPEN = 1, + /** + * @generate: none + * @priority: high + */ + REMOTE_PROC_CLOSE = 2, + /** + * @generate: server + * @priority: high + */ + REMOTE_PROC_GET_TYPE = 3, + /** + * @generate: both + * @priority: high + */ + REMOTE_PROC_GET_VERSION = 4, + /** + * @generate: both + * @priority: high + */ + REMOTE_PROC_GET_MAX_VCPUS = 5, + /** + * @generate: both + * @priority: high + */ + REMOTE_PROC_NODE_GET_INFO = 6, + /** + * @generate: both + */ + REMOTE_PROC_GET_CAPABILITIES = 7, + /** + * @generate: both + */ + REMOTE_PROC_DOMAIN_ATTACH_DEVICE = 8, + /** + * @generate: server + */ + REMOTE_PROC_DOMAIN_CREATE = 9, + /** + * @generate: both + */ + REMOTE_PROC_DOMAIN_CREATE_XML = 10, + + + + /** + * @generate: both + * @priority: high + */ + REMOTE_PROC_DOMAIN_DEFINE_XML = 11, + /** + * @generate: both + * @priority: high + */ + REMOTE_PROC_DOMAIN_DESTROY = 12, + /** + * @generate: both + */ + REMOTE_PROC_DOMAIN_DETACH_DEVICE = 13, + /** + * @generate: both + */ + REMOTE_PROC_DOMAIN_GET_XML_DESC = 14, + /** + * @generate: both + * @priority: high + */ + REMOTE_PROC_DOMAIN_GET_AUTOSTART = 15, + /** + * @generate: both + */ + REMOTE_PROC_DOMAIN_GET_INFO = 16, + /** + * @generate: both + * @priority: high + */ + REMOTE_PROC_DOMAIN_GET_MAX_MEMORY = 17, + /** + * @generate: both + * @priority: high + */ + REMOTE_PROC_DOMAIN_GET_MAX_VCPUS = 18, + /** + * @generate: both + * @priority: high + */ + REMOTE_PROC_DOMAIN_GET_OS_TYPE = 19, + /** + * @generate: none + * @priority: high + */ + REMOTE_PROC_DOMAIN_GET_VCPUS = 20, + + + + /** + * @generate: both + * @priority: high + */ + REMOTE_PROC_LIST_DEFINED_DOMAINS = 21, + /** + * @generate: both + * @priority: high + */ + REMOTE_PROC_DOMAIN_LOOKUP_BY_ID = 22, + /** + * @generate: both + * @priority: high + */ + REMOTE_PROC_DOMAIN_LOOKUP_BY_NAME = 23, + /** + * @generate: both + * @priority: high + */ + REMOTE_PROC_DOMAIN_LOOKUP_BY_UUID = 24, + /** + * @generate: both + * @priority: high + */ + REMOTE_PROC_NUM_OF_DEFINED_DOMAINS = 25, + /** + * @generate: both + */ + REMOTE_PROC_DOMAIN_PIN_VCPU = 26, + /** + * @generate: both + */ + REMOTE_PROC_DOMAIN_REBOOT = 27, + /** + * @generate: both + */ + REMOTE_PROC_DOMAIN_RESUME = 28, + /** + * @generate: both + * @priority: high + */ + REMOTE_PROC_DOMAIN_SET_AUTOSTART = 29, + /** + * @generate: both + * @priority: high + */ + REMOTE_PROC_DOMAIN_SET_MAX_MEMORY = 30, + + + + /** + * @generate: both + */ + REMOTE_PROC_DOMAIN_SET_MEMORY = 31, + /** + * @generate: both + */ + REMOTE_PROC_DOMAIN_SET_VCPUS = 32, + /** + * @generate: both + */ + REMOTE_PROC_DOMAIN_SHUTDOWN = 33, + /** + * @generate: both + */ + REMOTE_PROC_DOMAIN_SUSPEND = 34, + /** + * @generate: both + * @priority: high + */ + REMOTE_PROC_DOMAIN_UNDEFINE = 35, + /** + * @generate: both + * @priority: high + */ + REMOTE_PROC_LIST_DEFINED_NETWORKS = 36, + /** + * @generate: server + * @priority: high + */ + REMOTE_PROC_LIST_DOMAINS = 37, + /** + * @generate: both + * @priority: high + */ + REMOTE_PROC_LIST_NETWORKS = 38, + /** + * @generate: both + */ + REMOTE_PROC_NETWORK_CREATE = 39, + /** + * @generate: both + */ + REMOTE_PROC_NETWORK_CREATE_XML = 40, + + + + /** + * @generate: both + * @priority: high + */ + REMOTE_PROC_NETWORK_DEFINE_XML = 41, + /** + * @generate: both + * @priority: high + */ + REMOTE_PROC_NETWORK_DESTROY = 42, + /** + * @generate: both + * @priority: high + */ + REMOTE_PROC_NETWORK_GET_XML_DESC = 43, + /** + * @generate: both + * @priority: high + */ + REMOTE_PROC_NETWORK_GET_AUTOSTART = 44, + /** + * @generate: both + * @priority: high + */ + REMOTE_PROC_NETWORK_GET_BRIDGE_NAME = 45, + /** + * @generate: both + * @priority: high + */ + REMOTE_PROC_NETWORK_LOOKUP_BY_NAME = 46, + /** + * @generate: both + * @priority: high + */ + REMOTE_PROC_NETWORK_LOOKUP_BY_UUID = 47, + /** + * @generate: both + * @priority: high + */ + REMOTE_PROC_NETWORK_SET_AUTOSTART = 48, + /** + * @generate: both + * @priority: high + */ + REMOTE_PROC_NETWORK_UNDEFINE = 49, + /** + * @generate: both + * @priority: high + */ + REMOTE_PROC_NUM_OF_DEFINED_NETWORKS = 50, + + + + /** + * @generate: both + * @priority: high + */ + REMOTE_PROC_NUM_OF_DOMAINS = 51, + /** + * @generate: both + * @priority: high + */ + REMOTE_PROC_NUM_OF_NETWORKS = 52, + /** + * @generate: both + */ + REMOTE_PROC_DOMAIN_CORE_DUMP = 53, + /** + * @generate: both + */ + REMOTE_PROC_DOMAIN_RESTORE = 54, + /** + * @generate: both + */ + REMOTE_PROC_DOMAIN_SAVE = 55, + /** + * @generate: none + */ + REMOTE_PROC_DOMAIN_GET_SCHEDULER_TYPE = 56, + /** + * @generate: client + */ + REMOTE_PROC_DOMAIN_GET_SCHEDULER_PARAMETERS = 57, + /** + * @generate: both + */ + REMOTE_PROC_DOMAIN_SET_SCHEDULER_PARAMETERS = 58, + /** + * @generate: both + * @priority: high + */ + REMOTE_PROC_GET_HOSTNAME = 59, + /** + * @generate: client + * @priority: high + */ + REMOTE_PROC_SUPPORTS_FEATURE = 60, + + + + /** + * @generate: none + */ + REMOTE_PROC_DOMAIN_MIGRATE_PREPARE = 61, + /** + * @generate: both + */ + REMOTE_PROC_DOMAIN_MIGRATE_PERFORM = 62, + /** + * @generate: both + */ + REMOTE_PROC_DOMAIN_MIGRATE_FINISH = 63, + /** + * @generate: both + */ + REMOTE_PROC_DOMAIN_BLOCK_STATS = 64, + /** + * @generate: both + * @priority: high + */ + REMOTE_PROC_DOMAIN_INTERFACE_STATS = 65, + /** + * @generate: none + * @priority: high + */ + REMOTE_PROC_AUTH_LIST = 66, + /** + * @generate: none + * @priority: high + */ + REMOTE_PROC_AUTH_SASL_INIT = 67, + /** + * @generate: none + * @priority: high + */ + REMOTE_PROC_AUTH_SASL_START = 68, + /** + * @generate: none + * @priority: high + */ + REMOTE_PROC_AUTH_SASL_STEP = 69, + /** + * @generate: none + * @priority: high + */ + REMOTE_PROC_AUTH_POLKIT = 70, + + + + /** + * @generate: both + * @priority: high + */ + REMOTE_PROC_NUM_OF_STORAGE_POOLS = 71, + /** + * @generate: both + * @priority: high + */ + REMOTE_PROC_LIST_STORAGE_POOLS = 72, + /** + * @generate: both + * @priority: high + */ + REMOTE_PROC_NUM_OF_DEFINED_STORAGE_POOLS = 73, + /** + * @generate: both + * @priority: high + */ + REMOTE_PROC_LIST_DEFINED_STORAGE_POOLS = 74, + /** + * @generate: server + */ + REMOTE_PROC_FIND_STORAGE_POOL_SOURCES = 75, + /** + * @generate: both + */ + REMOTE_PROC_STORAGE_POOL_CREATE_XML = 76, + /** + * @generate: both + * @priority: high + */ + REMOTE_PROC_STORAGE_POOL_DEFINE_XML = 77, + /** + * @generate: both + */ + REMOTE_PROC_STORAGE_POOL_CREATE = 78, + /** + * @generate: both + */ + REMOTE_PROC_STORAGE_POOL_BUILD = 79, + /** + * @generate: both + * @priority: high + */ + REMOTE_PROC_STORAGE_POOL_DESTROY = 80, + + + + /** + * @generate: both + */ + REMOTE_PROC_STORAGE_POOL_DELETE = 81, + /** + * @generate: both + * @priority: high + */ + REMOTE_PROC_STORAGE_POOL_UNDEFINE = 82, + /** + * @generate: both + */ + REMOTE_PROC_STORAGE_POOL_REFRESH = 83, + /** + * @generate: both + * @priority: high + */ + REMOTE_PROC_STORAGE_POOL_LOOKUP_BY_NAME = 84, + /** + * @generate: both + * @priority: high + */ + REMOTE_PROC_STORAGE_POOL_LOOKUP_BY_UUID = 85, + /** + * @generate: both + * @priority: high + */ + REMOTE_PROC_STORAGE_POOL_LOOKUP_BY_VOLUME = 86, + /** + * @generate: both + * @priority: high + */ + REMOTE_PROC_STORAGE_POOL_GET_INFO = 87, + /** + * @generate: both + * @priority: high + */ + REMOTE_PROC_STORAGE_POOL_GET_XML_DESC = 88, + /** + * @generate: both + * @priority: high + */ + REMOTE_PROC_STORAGE_POOL_GET_AUTOSTART = 89, + /** + * @generate: both + * @priority: high + */ + REMOTE_PROC_STORAGE_POOL_SET_AUTOSTART = 90, + + + + /** + * @generate: both + * @priority: high + */ + REMOTE_PROC_STORAGE_POOL_NUM_OF_VOLUMES = 91, + /** + * @generate: both + * @priority: high + */ + REMOTE_PROC_STORAGE_POOL_LIST_VOLUMES = 92, + /** + * @generate: both + */ + REMOTE_PROC_STORAGE_VOL_CREATE_XML = 93, + /** + * @generate: both + */ + REMOTE_PROC_STORAGE_VOL_DELETE = 94, + /** + * @generate: both + * @priority: high + */ + REMOTE_PROC_STORAGE_VOL_LOOKUP_BY_NAME = 95, + /** + * @generate: both + * @priority: high + */ + REMOTE_PROC_STORAGE_VOL_LOOKUP_BY_KEY = 96, + /** + * @generate: both + * @priority: high + */ + REMOTE_PROC_STORAGE_VOL_LOOKUP_BY_PATH = 97, + /** + * @generate: both + * @priority: high + */ + REMOTE_PROC_STORAGE_VOL_GET_INFO = 98, + /** + * @generate: both + * @priority: high + */ + REMOTE_PROC_STORAGE_VOL_GET_XML_DESC = 99, + /** + * @generate: both + * @priority: high + */ + REMOTE_PROC_STORAGE_VOL_GET_PATH = 100, + + + + /** + * @generate: server + * @priority: high + */ + REMOTE_PROC_NODE_GET_CELLS_FREE_MEMORY = 101, + /** + * @generate: both + * @priority: high + */ + REMOTE_PROC_NODE_GET_FREE_MEMORY = 102, + /** + * @generate: none + */ + REMOTE_PROC_DOMAIN_BLOCK_PEEK = 103, + /** + * @generate: none + */ + REMOTE_PROC_DOMAIN_MEMORY_PEEK = 104, + /** + * @generate: none + * @priority: high + */ + REMOTE_PROC_DOMAIN_EVENTS_REGISTER = 105, + /** + * @generate: none + * @priority: high + */ + REMOTE_PROC_DOMAIN_EVENTS_DEREGISTER = 106, + /** + * @generate: both + */ + REMOTE_PROC_DOMAIN_EVENT_LIFECYCLE = 107, + /** + * @generate: none + */ + REMOTE_PROC_DOMAIN_MIGRATE_PREPARE2 = 108, + /** + * @generate: both + */ + REMOTE_PROC_DOMAIN_MIGRATE_FINISH2 = 109, + /** + * @generate: server + * @priority: high + */ + REMOTE_PROC_GET_URI = 110, + + + + /** + * @generate: both + * @priority: high + */ + REMOTE_PROC_NODE_NUM_OF_DEVICES = 111, + /** + * @generate: both + * @priority: high + */ + REMOTE_PROC_NODE_LIST_DEVICES = 112, + /** + * @generate: both + * @priority: high + */ + REMOTE_PROC_NODE_DEVICE_LOOKUP_BY_NAME = 113, + /** + * @generate: both + */ + REMOTE_PROC_NODE_DEVICE_GET_XML_DESC = 114, + /** + * @generate: client + * @priority: high + */ + REMOTE_PROC_NODE_DEVICE_GET_PARENT = 115, + /** + * @generate: both + * @priority: high + */ + REMOTE_PROC_NODE_DEVICE_NUM_OF_CAPS = 116, + /** + * @generate: both + * @priority: high + */ + REMOTE_PROC_NODE_DEVICE_LIST_CAPS = 117, + /** + * @generate: server + */ + REMOTE_PROC_NODE_DEVICE_DETTACH = 118, + /** + * @generate: server + */ + REMOTE_PROC_NODE_DEVICE_RE_ATTACH = 119, + /** + * @generate: server + */ + REMOTE_PROC_NODE_DEVICE_RESET = 120, + + + + /** + * @generate: none + * @priority: high + */ + REMOTE_PROC_DOMAIN_GET_SECURITY_LABEL = 121, + /** + * @generate: none + * @priority: high + */ + REMOTE_PROC_NODE_GET_SECURITY_MODEL = 122, + /** + * @generate: both + */ + REMOTE_PROC_NODE_DEVICE_CREATE_XML = 123, + /** + * @generate: both + * @priority: high + */ + REMOTE_PROC_NODE_DEVICE_DESTROY = 124, + /** + * @generate: both + */ + REMOTE_PROC_STORAGE_VOL_CREATE_XML_FROM = 125, + /** + * @generate: both + * @priority: high + */ + REMOTE_PROC_NUM_OF_INTERFACES = 126, + /** + * @generate: both + * @priority: high + */ + REMOTE_PROC_LIST_INTERFACES = 127, + /** + * @generate: both + * @priority: high + */ + REMOTE_PROC_INTERFACE_LOOKUP_BY_NAME = 128, + /** + * @generate: both + * @priority: high + */ + REMOTE_PROC_INTERFACE_LOOKUP_BY_MAC_STRING = 129, + /** + * @generate: both + */ + REMOTE_PROC_INTERFACE_GET_XML_DESC = 130, + + + + /** + * @generate: both + * @priority: high + */ + REMOTE_PROC_INTERFACE_DEFINE_XML = 131, + /** + * @generate: both + * @priority: high + */ + REMOTE_PROC_INTERFACE_UNDEFINE = 132, + /** + * @generate: both + */ + REMOTE_PROC_INTERFACE_CREATE = 133, + /** + * @generate: both + * @priority: high + */ + REMOTE_PROC_INTERFACE_DESTROY = 134, + /** + * @generate: both + */ + REMOTE_PROC_DOMAIN_XML_FROM_NATIVE = 135, + /** + * @generate: both + */ + REMOTE_PROC_DOMAIN_XML_TO_NATIVE = 136, + /** + * @generate: both + * @priority: high + */ + REMOTE_PROC_NUM_OF_DEFINED_INTERFACES = 137, + /** + * @generate: both + * @priority: high + */ + REMOTE_PROC_LIST_DEFINED_INTERFACES = 138, + /** + * @generate: both + * @priority: high + */ + REMOTE_PROC_NUM_OF_SECRETS = 139, + /** + * @generate: both + * @priority: high + */ + REMOTE_PROC_LIST_SECRETS = 140, + + + + /** + * @generate: both + * @priority: high + */ + REMOTE_PROC_SECRET_LOOKUP_BY_UUID = 141, + /** + * @generate: both + * @priority: high + */ + REMOTE_PROC_SECRET_DEFINE_XML = 142, + /** + * @generate: both + * @priority: high + */ + REMOTE_PROC_SECRET_GET_XML_DESC = 143, + /** + * @generate: both + * @priority: high + */ + REMOTE_PROC_SECRET_SET_VALUE = 144, + /** + * @generate: none + * @priority: high + */ + REMOTE_PROC_SECRET_GET_VALUE = 145, + /** + * @generate: both + * @priority: high + */ + REMOTE_PROC_SECRET_UNDEFINE = 146, + /** + * @generate: both + * @priority: high + */ + REMOTE_PROC_SECRET_LOOKUP_BY_USAGE = 147, + /** + * @generate: both + * @writestream: 1 + */ + REMOTE_PROC_DOMAIN_MIGRATE_PREPARE_TUNNEL = 148, + /** + * @generate: server + * @priority: high + */ + REMOTE_PROC_IS_SECURE = 149, + /** + * @generate: both + * @priority: high + */ + REMOTE_PROC_DOMAIN_IS_ACTIVE = 150, + + + + /** + * @generate: both + * @priority: high + */ + REMOTE_PROC_DOMAIN_IS_PERSISTENT = 151, + /** + * @generate: both + * @priority: high + */ + REMOTE_PROC_NETWORK_IS_ACTIVE = 152, + /** + * @generate: both + * @priority: high + */ + REMOTE_PROC_NETWORK_IS_PERSISTENT = 153, + /** + * @generate: both + * @priority: high + */ + REMOTE_PROC_STORAGE_POOL_IS_ACTIVE = 154, + /** + * @generate: both + * @priority: high + */ + REMOTE_PROC_STORAGE_POOL_IS_PERSISTENT = 155, + /** + * @generate: both + * @priority: high + */ + REMOTE_PROC_INTERFACE_IS_ACTIVE = 156, + /** + * @generate: both + * @priority: high + */ + REMOTE_PROC_GET_LIB_VERSION = 157, + /** + * @generate: both + * @priority: high + */ + REMOTE_PROC_CPU_COMPARE = 158, + /** + * @generate: none + */ + REMOTE_PROC_DOMAIN_MEMORY_STATS = 159, + /** + * @generate: both + */ + REMOTE_PROC_DOMAIN_ATTACH_DEVICE_FLAGS = 160, + + + + /** + * @generate: both + */ + REMOTE_PROC_DOMAIN_DETACH_DEVICE_FLAGS = 161, + /** + * @generate: both + */ + REMOTE_PROC_CPU_BASELINE = 162, + /** + * @generate: both + */ + REMOTE_PROC_DOMAIN_GET_JOB_INFO = 163, + /** + * @generate: both + */ + REMOTE_PROC_DOMAIN_ABORT_JOB = 164, + /** + * @generate: both + */ + REMOTE_PROC_STORAGE_VOL_WIPE = 165, + /** + * @generate: both + */ + REMOTE_PROC_DOMAIN_MIGRATE_SET_MAX_DOWNTIME = 166, + /** + * @generate: none + * @priority: high + */ + REMOTE_PROC_DOMAIN_EVENTS_REGISTER_ANY = 167, + /** + * @generate: none + * @priority: high + */ + REMOTE_PROC_DOMAIN_EVENTS_DEREGISTER_ANY = 168, + /** + * @generate: both + */ + REMOTE_PROC_DOMAIN_EVENT_REBOOT = 169, + /** + * @generate: both + */ + REMOTE_PROC_DOMAIN_EVENT_RTC_CHANGE = 170, + + + + /** + * @generate: both + */ + REMOTE_PROC_DOMAIN_EVENT_WATCHDOG = 171, + /** + * @generate: both + */ + REMOTE_PROC_DOMAIN_EVENT_IO_ERROR = 172, + /** + * @generate: both + */ + REMOTE_PROC_DOMAIN_EVENT_GRAPHICS = 173, + /** + * @generate: both + */ + REMOTE_PROC_DOMAIN_UPDATE_DEVICE_FLAGS = 174, + /** + * @generate: both + * @priority: high + */ + REMOTE_PROC_NWFILTER_LOOKUP_BY_NAME = 175, + /** + * @generate: both + * @priority: high + */ + REMOTE_PROC_NWFILTER_LOOKUP_BY_UUID = 176, + /** + * @generate: both + * @priority: high + */ + REMOTE_PROC_NWFILTER_GET_XML_DESC = 177, + /** + * @generate: both + * @priority: high + */ + REMOTE_PROC_NUM_OF_NWFILTERS = 178, + /** + * @generate: both + * @priority: high + */ + REMOTE_PROC_LIST_NWFILTERS = 179, + /** + * @generate: both + * @priority: high + */ + REMOTE_PROC_NWFILTER_DEFINE_XML = 180, + + + + /** + * @generate: both + * @priority: high + */ + REMOTE_PROC_NWFILTER_UNDEFINE = 181, + /** + * @generate: both + */ + REMOTE_PROC_DOMAIN_MANAGED_SAVE = 182, + /** + * @generate: both + */ + REMOTE_PROC_DOMAIN_HAS_MANAGED_SAVE_IMAGE = 183, + /** + * @generate: both + */ + REMOTE_PROC_DOMAIN_MANAGED_SAVE_REMOVE = 184, + /** + * @generate: both + */ + REMOTE_PROC_DOMAIN_SNAPSHOT_CREATE_XML = 185, + /** + * @generate: both + * @priority: high + */ + REMOTE_PROC_DOMAIN_SNAPSHOT_GET_XML_DESC = 186, + /** + * @generate: both + * @priority: high + */ + REMOTE_PROC_DOMAIN_SNAPSHOT_NUM = 187, + /** + * @generate: both + * @priority: high + */ + REMOTE_PROC_DOMAIN_SNAPSHOT_LIST_NAMES = 188, + /** + * @generate: both + * @priority: high + */ + REMOTE_PROC_DOMAIN_SNAPSHOT_LOOKUP_BY_NAME = 189, + /** + * @generate: both + */ + REMOTE_PROC_DOMAIN_HAS_CURRENT_SNAPSHOT = 190, + + + + /** + * @generate: both + */ + REMOTE_PROC_DOMAIN_SNAPSHOT_CURRENT = 191, + /** + * @generate: both + */ + REMOTE_PROC_DOMAIN_REVERT_TO_SNAPSHOT = 192, + /** + * @generate: both + */ + REMOTE_PROC_DOMAIN_SNAPSHOT_DELETE = 193, + /** + * @generate: both + */ + REMOTE_PROC_DOMAIN_GET_BLOCK_INFO = 194, + /** + * @generate: both + */ + REMOTE_PROC_DOMAIN_EVENT_IO_ERROR_REASON = 195, + /** + * @generate: both + */ + REMOTE_PROC_DOMAIN_CREATE_WITH_FLAGS = 196, + /** + * @generate: both + */ + REMOTE_PROC_DOMAIN_SET_MEMORY_PARAMETERS = 197, + /** + * @generate: none + */ + REMOTE_PROC_DOMAIN_GET_MEMORY_PARAMETERS = 198, + /** + * @generate: both + */ + REMOTE_PROC_DOMAIN_SET_VCPUS_FLAGS = 199, + /** + * @generate: both + */ + REMOTE_PROC_DOMAIN_GET_VCPUS_FLAGS = 200, + + + + /** + * @generate: both + * @readstream: 2 + */ + REMOTE_PROC_DOMAIN_OPEN_CONSOLE = 201, + /** + * @generate: both + * @priority: high + */ + REMOTE_PROC_DOMAIN_IS_UPDATED = 202, + /** + * @generate: both + * @priority: high + */ + REMOTE_PROC_GET_SYSINFO = 203, + /** + * @generate: both + */ + REMOTE_PROC_DOMAIN_SET_MEMORY_FLAGS = 204, + /** + * @generate: both + */ + REMOTE_PROC_DOMAIN_SET_BLKIO_PARAMETERS = 205, + /** + * @generate: none + */ + REMOTE_PROC_DOMAIN_GET_BLKIO_PARAMETERS = 206, + /** + * @generate: both + */ + REMOTE_PROC_DOMAIN_MIGRATE_SET_MAX_SPEED = 207, + /** + * @generate: both + * @writestream: 1 + */ + REMOTE_PROC_STORAGE_VOL_UPLOAD = 208, + /** + * @generate: both + * @readstream: 1 + */ + REMOTE_PROC_STORAGE_VOL_DOWNLOAD = 209, + /** + * @generate: both + */ + REMOTE_PROC_DOMAIN_INJECT_NMI = 210, + + + + /** + * @generate: both + * @readstream: 1 + */ + REMOTE_PROC_DOMAIN_SCREENSHOT = 211, + /** + * @generate: none + * @priority: high + */ + REMOTE_PROC_DOMAIN_GET_STATE = 212, + /** + * @generate: none + */ + REMOTE_PROC_DOMAIN_MIGRATE_BEGIN3 = 213, + /** + * @generate: none + */ + REMOTE_PROC_DOMAIN_MIGRATE_PREPARE3 = 214, + /** + * @generate: server + * @writestream: 1 + */ + REMOTE_PROC_DOMAIN_MIGRATE_PREPARE_TUNNEL3 = 215, + /** + * @generate: none + */ + REMOTE_PROC_DOMAIN_MIGRATE_PERFORM3 = 216, + /** + * @generate: none + */ + REMOTE_PROC_DOMAIN_MIGRATE_FINISH3 = 217, + /** + * @generate: none + */ + REMOTE_PROC_DOMAIN_MIGRATE_CONFIRM3 = 218, + /** + * @generate: both + */ + REMOTE_PROC_DOMAIN_SET_SCHEDULER_PARAMETERS_FLAGS = 219, + /** + * @generate: both + */ + REMOTE_PROC_INTERFACE_CHANGE_BEGIN = 220, + + + + /** + * @generate: both + */ + REMOTE_PROC_INTERFACE_CHANGE_COMMIT = 221, + /** + * @generate: both + */ + REMOTE_PROC_INTERFACE_CHANGE_ROLLBACK = 222, + /** + * @generate: client + */ + REMOTE_PROC_DOMAIN_GET_SCHEDULER_PARAMETERS_FLAGS = 223, + /** + * @generate: none + */ + REMOTE_PROC_DOMAIN_EVENT_CONTROL_ERROR = 224, + /** + * @generate: both + */ + REMOTE_PROC_DOMAIN_PIN_VCPU_FLAGS = 225, + /** + * @generate: both + */ + REMOTE_PROC_DOMAIN_SEND_KEY = 226, + /** + * @generate: none + * @priority: high + */ + REMOTE_PROC_NODE_GET_CPU_STATS = 227, + /** + * @generate: none + * @priority: high + */ + REMOTE_PROC_NODE_GET_MEMORY_STATS = 228, + /** + * @generate: both + * @priority: high + */ + REMOTE_PROC_DOMAIN_GET_CONTROL_INFO = 229, + /** + * @generate: none + */ + REMOTE_PROC_DOMAIN_GET_VCPU_PIN_INFO = 230, + + + + /** + * @generate: both + * @priority: high + */ + REMOTE_PROC_DOMAIN_UNDEFINE_FLAGS = 231, + /** + * @generate: both + */ + REMOTE_PROC_DOMAIN_SAVE_FLAGS = 232, + /** + * @generate: both + */ + REMOTE_PROC_DOMAIN_RESTORE_FLAGS = 233, + /** + * @generate: both + * @priority: high + */ + REMOTE_PROC_DOMAIN_DESTROY_FLAGS = 234, + /** + * @generate: both + * @priority: high + */ + REMOTE_PROC_DOMAIN_SAVE_IMAGE_GET_XML_DESC = 235, + /** + * @generate: both + * @priority: high + */ + REMOTE_PROC_DOMAIN_SAVE_IMAGE_DEFINE_XML = 236, + /** + * @generate: both + */ + REMOTE_PROC_DOMAIN_BLOCK_JOB_ABORT = 237, + /** + * @generate: none + */ + REMOTE_PROC_DOMAIN_GET_BLOCK_JOB_INFO = 238, + /** + * @generate: both + */ + REMOTE_PROC_DOMAIN_BLOCK_JOB_SET_SPEED = 239, + /** + * @generate: both + */ + REMOTE_PROC_DOMAIN_BLOCK_PULL = 240, + + + + /** + * @generate: none + */ + REMOTE_PROC_DOMAIN_EVENT_BLOCK_JOB = 241, + /** + * @generate: both + */ + REMOTE_PROC_DOMAIN_MIGRATE_GET_MAX_SPEED = 242, + /** + * @generate: none + */ + REMOTE_PROC_DOMAIN_BLOCK_STATS_FLAGS = 243, + /** + * @generate: both + * @priority: high + */ + REMOTE_PROC_DOMAIN_SNAPSHOT_GET_PARENT = 244, + /** + * @generate: both + */ + REMOTE_PROC_DOMAIN_RESET = 245, + /** + * @generate: both + * @priority: high + */ + REMOTE_PROC_DOMAIN_SNAPSHOT_NUM_CHILDREN = 246, + /** + * @generate: both + * @priority: high + */ + REMOTE_PROC_DOMAIN_SNAPSHOT_LIST_CHILDREN_NAMES = 247, + /** + * @generate: none + */ + REMOTE_PROC_DOMAIN_EVENT_DISK_CHANGE = 248, + /** + * @generate: none + */ + REMOTE_PROC_DOMAIN_OPEN_GRAPHICS = 249, + /** + * @generate: both + */ + REMOTE_PROC_NODE_SUSPEND_FOR_DURATION = 250, + + + + /** + * @generate: both + */ + REMOTE_PROC_DOMAIN_BLOCK_RESIZE = 251, + /** + * @generate: both + */ + REMOTE_PROC_DOMAIN_SET_BLOCK_IO_TUNE = 252, + /** + * @generate: none + */ + REMOTE_PROC_DOMAIN_GET_BLOCK_IO_TUNE = 253, + /** + * @generate: both + */ + REMOTE_PROC_DOMAIN_SET_NUMA_PARAMETERS = 254, + /** + * @generate: none + */ + REMOTE_PROC_DOMAIN_GET_NUMA_PARAMETERS = 255, + /** + * @generate: both + */ + REMOTE_PROC_DOMAIN_SET_INTERFACE_PARAMETERS = 256, + /** + * @generate: none + */ + REMOTE_PROC_DOMAIN_GET_INTERFACE_PARAMETERS = 257, + /** + * @generate: both + */ + REMOTE_PROC_DOMAIN_SHUTDOWN_FLAGS = 258, + /** + * @generate: both + */ + REMOTE_PROC_STORAGE_VOL_WIPE_PATTERN = 259, + /** + * @generate: both + */ + REMOTE_PROC_STORAGE_VOL_RESIZE = 260, + + + + /** + * @generate: both + */ + REMOTE_PROC_DOMAIN_PM_SUSPEND_FOR_DURATION = 261, + /** + * @generate: none + */ + REMOTE_PROC_DOMAIN_GET_CPU_STATS = 262, + /** + * @generate: none + */ + REMOTE_PROC_DOMAIN_GET_DISK_ERRORS = 263, + /** + * @generate: both + */ + REMOTE_PROC_DOMAIN_SET_METADATA = 264, + /** + * @generate: both + */ + REMOTE_PROC_DOMAIN_GET_METADATA = 265, + /** + * @generate: both + */ + REMOTE_PROC_DOMAIN_BLOCK_REBASE = 266, + /** + * @generate: both + */ + REMOTE_PROC_DOMAIN_PM_WAKEUP = 267, + /** + * @generate: both + */ + REMOTE_PROC_DOMAIN_EVENT_TRAY_CHANGE = 268, + /** + * @generate: both + */ + REMOTE_PROC_DOMAIN_EVENT_PMWAKEUP = 269, + /** + * @generate: both + */ + REMOTE_PROC_DOMAIN_EVENT_PMSUSPEND = 270, + + + + /** + * @generate: both + */ + REMOTE_PROC_DOMAIN_SNAPSHOT_IS_CURRENT = 271, + /** + * @generate: both + */ + REMOTE_PROC_DOMAIN_SNAPSHOT_HAS_METADATA = 272, + /** + * @generate: none + * @priority: high + */ + REMOTE_PROC_CONNECT_LIST_ALL_DOMAINS = 273, + /** + * @generate: none + * @priority: high + */ + REMOTE_PROC_DOMAIN_LIST_ALL_SNAPSHOTS = 274, + /** + * @generate: none + * @priority: high + */ + REMOTE_PROC_DOMAIN_SNAPSHOT_LIST_ALL_CHILDREN = 275, + /** + * @generate: both + */ + REMOTE_PROC_DOMAIN_EVENT_BALLOON_CHANGE = 276, + /** + * @generate: both + */ + REMOTE_PROC_DOMAIN_GET_HOSTNAME = 277, + /** + * @generate: none + * @priority: high + */ + REMOTE_PROC_DOMAIN_GET_SECURITY_LABEL_LIST = 278, + /** + * @generate: none + */ + REMOTE_PROC_DOMAIN_PIN_EMULATOR = 279, + /** + * @generate: none + */ + REMOTE_PROC_DOMAIN_GET_EMULATOR_PIN_INFO = 280, + + + + /** + * @generate: none + * @priority: high + */ + REMOTE_PROC_CONNECT_LIST_ALL_STORAGE_POOLS = 281, + /** + * @generate: none + * @priority: high + */ + REMOTE_PROC_STORAGE_POOL_LIST_ALL_VOLUMES = 282, + /** + * @generate: none + * @priority: high + */ + REMOTE_PROC_CONNECT_LIST_ALL_NETWORKS = 283, + /** + * @generate: none + * @priority: high + */ + REMOTE_PROC_CONNECT_LIST_ALL_INTERFACES = 284, + /** + * @generate: none + * @priority: high + */ + REMOTE_PROC_CONNECT_LIST_ALL_NODE_DEVICES = 285, + /** + * @generate: none + * @priority: high + */ + REMOTE_PROC_CONNECT_LIST_ALL_NWFILTERS = 286, + /** + * @generate: none + * @priority: high + */ + REMOTE_PROC_CONNECT_LIST_ALL_SECRETS = 287, + /** + * @generate: both + */ + REMOTE_PROC_NODE_SET_MEMORY_PARAMETERS = 288, + /** + * @generate: none + */ + REMOTE_PROC_NODE_GET_MEMORY_PARAMETERS = 289, + /** + * @generate: both + */ + REMOTE_PROC_DOMAIN_BLOCK_COMMIT = 290, + + + + /** + * @generate: both + * @priority: high + */ + REMOTE_PROC_NETWORK_UPDATE = 291, + /** + * @generate: both + */ + REMOTE_PROC_DOMAIN_EVENT_PMSUSPEND_DISK = 292, + /** + * @generate: none + */ + REMOTE_PROC_NODE_GET_CPU_MAP = 293, + /** + * @generate: both + */ + REMOTE_PROC_DOMAIN_FSTRIM = 294, + /** + * @generate: both + */ + REMOTE_PROC_DOMAIN_SEND_PROCESS_SIGNAL = 295, + /** + * @generate: both + * @readstream: 2 + */ + REMOTE_PROC_DOMAIN_OPEN_CHANNEL = 296, + /** + * @generate: both + * @priority: high + */ + REMOTE_PROC_NODE_DEVICE_LOOKUP_SCSI_HOST_BY_WWN = 297, + /** + * @generate: none + */ + REMOTE_PROC_DOMAIN_GET_JOB_STATS = 298, + /** + * @generate: both + */ + REMOTE_PROC_DOMAIN_MIGRATE_GET_COMPRESSION_CACHE = 299, + /** + * @generate: both + */ + REMOTE_PROC_DOMAIN_MIGRATE_SET_COMPRESSION_CACHE = 300 + + /* * Notice how the entries are grouped in sets of 10 ? * Nice isn't it. Please keep it this way when adding more. - * - * Each function must have a three-word comment. The first word is - * whether gendispatch.pl handles daemon, the second whether - * it handles src/remote. Additional flags can be specified after a - * pipe. - * The last argument describes priority of API. There are two accepted - * values: low, high; Each API that might eventually access hypervisor's - * monitor (and thus block) MUST fall into low priority. However, there - * are some exceptions to this rule, e.g. domainDestroy. Other APIs MAY - * be marked as high priority. If in doubt, it's safe to choose low. - * Low is taken as default, and thus can be left out. - * - * The (readstream|writestream)@<offset> flag lets daemon and src/remote - * create a stream. The direction is defined from the src/remote point - * of view. A readstream transfers data from daemon to src/remote. The - * <offset> specifies at which offset the stream parameter is inserted - * in the function parameter list. */ + */ }; diff --git a/src/rpc/gendispatch.pl b/src/rpc/gendispatch.pl index ef387e0..65a0ac2 100755 --- a/src/rpc/gendispatch.pl +++ b/src/rpc/gendispatch.pl @@ -82,10 +82,11 @@ sub name_to_TypeName { # Read the input file (usually remote_protocol.x) and form an # opinion about the name, args and return type of each RPC. -my ($name, $ProcName, $id, $flags, %calls, @calls); +my ($name, $ProcName, $id, $flags, %calls, @calls, %opts); my $collect_args_members = 0; my $collect_ret_members = 0; +my $collect_opts = 0; my $last_name; open PROTOCOL, "<$protocol" or die "cannot open $protocol: $!"; @@ -103,6 +104,19 @@ while (<PROTOCOL>) { } elsif ($_ =~ m/^\s*(.*\S)\s*$/) { push(@{$calls{$name}->{ret_members}}, $1); } + } elsif ($collect_opts) { + if (m,^\s*\*\s*\@(\w+)\s*:\s*(\w+)\s*$,) { + $opts{$1} = $2; + } elsif (m,^\s*\*/\s*$,) { + $collect_opts = 0; + } elsif (m,^\s*\*\s*$,) { + # pass + } else { + die "cannot parse $_"; + } + } elsif (m,/\*\*,) { + %opts = (); + $collect_opts = 1; } elsif (/^struct (${structprefix}_(.*)_args)/ || /^struct (${structprefix}(.*)Args)/) { my $structname = $1; @@ -171,11 +185,10 @@ while (<PROTOCOL>) { $collect_args_members = 0; $collect_ret_members = 0; - } elsif (/^\s*(${procprefix}_PROC_(.*?))\s*=\s*(\d+)\s*,?(.*)$/) { + } elsif (/^\s*(${procprefix}_PROC_(.*?))\s*=\s*(\d+)\s*,?\s*$/) { my $constname = $1; $name = $2; $id = $3; - $flags = $4; $ProcName = name_to_ProcName ($name); $name = lc $name; $name =~ s/_//g; @@ -195,41 +208,45 @@ while (<PROTOCOL>) { $calls{$name}->{constname} = $constname; if ($opt_b or $opt_k) { - if (!($flags =~ m/^\s*\/\*\s*(\S+)\s+(\S+)\s*(\|.*)?\s+(priority:(\S+))?\s*\*\/\s*$/)) { - die "invalid generator flags '$flags' for $constname" + if (!exists $opts{generate}) { + die "'\@generate' annotation missing for $constname"; } - my $genmode = $opt_b ? $1 : $2; - my $genflags = $3; - my $priority = defined $5 ? $5 : "low"; + if ($opts{generate} !~ /^(both|server|client|none)$/) { + die "'\@generate' annotation value '$opts{generate}' invalid"; + } - if ($genmode eq "autogen") { + if ($opts{generate} eq "both") { + push(@autogen, $ProcName); + } elsif ($opt_b && ($opts{generate} eq "server")) { + push(@autogen, $ProcName); + } elsif (!$opt_b && ($opts{generate} eq "client")) { push(@autogen, $ProcName); - } elsif ($genmode eq "skipgen") { - # ignore it - } else { - die "invalid generator flags for ${procprefix}_PROC_${name}" } - if (defined $genflags and $genflags ne "") { - if ($genflags =~ m/^\|\s*(read|write)stream@(\d+)\s*$/) { - $calls{$name}->{streamflag} = $1; - $calls{$name}->{streamoffset} = int($2); - } else { - die "invalid generator flags for ${procprefix}_PROC_${name}" - } + if (exists $opts{readstream}) { + $calls{$name}->{streamflag} = "read"; + $calls{$name}->{streamoffset} = int($opts{readstream}); + } elsif (exists $opts{writestream}) { + $calls{$name}->{streamflag} = "write"; + $calls{$name}->{streamoffset} = int($opts{writestream}); } else { $calls{$name}->{streamflag} = "none"; } + # for now, we distinguish only two levels of prioroty: # low (0) and high (1) - if ($priority eq "high") { - $calls{$name}->{priority} = 1; - } elsif ($priority eq "low") { - $calls{$name}->{priority} = 0; + if (exists $opts{priority}) { + if ($opts{priority} eq "high") { + $calls{$name}->{priority} = 1; + } elsif ($opts{priority} eq "low") { + $calls{$name}->{priority} = 0; + } else { + die "\@priority annotation value '$opts{priority}' invalid for $constname" + } } else { - die "invalid priority ${priority} for ${procprefix}_PROC_${name}" + $calls{$name}->{priority} = 0; } } -- 1.8.1.4

2 3

[libvirt] doc: write separate module for hostdev passthrough and in-use tracking
by Chunyan Liu 18 Apr '13

18 Apr '13

Hi, List, As the mail I've sent a week before: https://www.redhat.com/archives/libvir-list/2013-March/msg00730.html I'm willing to push this work forward so that the passthrough APIs could be reused by qemu driver and libxl driver (which doesn't support pci passthrough yet and tries to add this function recently), or other drivers. But since this work affacts a lot, I'm not sure if I can control it in a correct way. I write a draft to describe what I'm considering how to do, as in the following and in attachment. Hope to get your review, comment and guidence to improve the work before start coding. Any feedback will be very appreciated! Thanks! Chunyan ------------------------ DRAFT: Write separate module for hostdev passthrough 1. Purposes: * Move hostdev passthrough APIs from qemu_hostdev.ch to separate module so that they could be reused by other hypervisors too * Maintain global in-use state of hostdevs 2. Module design (draft): * New module name: hostdev_driver * New module files: hostdev_driver.ch hostdev_conf.ch * New Definitions: ## [src/driver.h] typedef struct _virHostdevDriver virHostdevDriver; typedef virHostdevDriver *virHostdevDriverPtr; struct _virHosedevDriver { const char *name; virDrvOpen open; virDrvClose close; virDrvPrepareHostdevs prepareHostdevs; virDrvPreparePciHostdevs preparePciHostdevs; virDrvprepareUsbHostdevs prepareUsbHostdevs; virDrvReattachHostdevs reattachHostdevs; virDrvReattachPciHostdevs reattachPciHostdevs; virDrvReattachUsbHostdevs reattachUsbHostdevs; virDrvGetActivePciHostdevList getActivePciHostdevList; virDrvGetActiveUsbHostdevList getActiveUsbHostdevList; virDrvGetDomainActivePciHostdevList getDomainActivePciHostdevList; virDrvGetDomainActiveUsbHostdevList getDomainActiveUsbHostdevList; }; ## [src/hostdev/hostdev_conf.h] typedef struct _virHostdevDriverState virHostdevDriverState; typedef virHostdevDriverState *virHostdevDriverStatePtr; struct _virHostdevDriverState { virMutex lock; virPCIDeviceListPtr activePciHostdevs; virPCIDeviceListPtr inactivePciHostdevs; virUSBDeviceListPtr activeUsbHostdevs; }; ## [src/hostdev/hostdev_driver.c] static virHostdevDriver hostdevDriver = { .name = "hostdev", .open = hostdevDriverOpen, .close = hostdevDriverClose, .prepareHostdevs = virPrepareHostdevs, .preparePciHostdevs = virPreparePciHostdevs, .prepareUsbHostdevs = virPrepareUsbHostdevs .reattachHostdevs = virReattachHostdevs, .reattachPciHostdevs = virReattachPciHostdevs, .reattachUsbHostdevs = virReattachUsbHostdevs, .getActivePciHostdevList = virGetActivePciHostdevList, .getActiveUsbHostdevList = virGetActiveUsbHostdevList, .getDomainActivePciHostdevList = virGetDomainActivePciHostdevList, .getDomainActiveUsbHostdevList = virGetDomainActiveUsbHostdevList, }; static virStateDriver hostdevStateDriver = { .name = "hostdev", .initialize = hostdevDriverStartup, .cleanup = hostdevDriverCleanup, .reload = hostdevDriverReload, }; * Changed Definitions: struct _virPCIDevice { ...... --- const char *used_by; /* The domain which uses the device */ +++ virDomainObjPtr used_by; /* include domname and conn info */ ...... }; struct _virUSBDevice { ...... --- const char *used_by; /* name of the domain using this dev */ +++ virDomainObjPtr used_by; /* include domname and conn info */ }; * APIs: typedef int (*virDrvPrepareHostdevs)(virHostdevDriverPtr driver, virDomainObjPtr vm, unsigned int flags); /* - workflow: call PrepareHostdevPciDevices and PrepareHostdevUsbDevices to do specific work. - reference: int qemuPrepareHostDevices(virQEMUDriverPtr driver, virDomainDefPtr def, bool coldBoot); - new parameter: - flags: - could set "coldBoot" for usb usage - could set hypervisor tag since for qemu it will use pci-stub, for libxl, it will use pciback. */ typedef int (*virDrvPreparePciHostdevs)(virHostdevDriverPtr driver, virDomainObjPtr vm, virDomainHostdevDefPtr *hostdevs, int nhostdevs, unsigned int flags); /* - workflow: 1. check things (e.g. assignability to non-managed device) 2. detach managed device and reset work 3. set usedby to 'vm', update activePciHostdevs and inactivePciHostdevs - reference: int qemuPrepareHostdevPCIDevices(virQEMUDriverPtr driver, const char *name, const unsigned char *uuid, virDomainHostdevDefPtr *hostdevs, int nhostdevs); */ typedef int (*virDrvprepareUsbHostdevs)((virHostdevDriverPtr driver, virDomainObjPtr vm, virUSBDeviceListPtr list); /* - workflow: check usb device, set usedby to 'vm', update activeUsbHostdevs - reference: int qemuPrepareHostdevUSBDevices(virQEMUDriverPtr driver, const char *name, virUSBDeviceListPtr list); */ typedef void (*virDrvReattachHostdevs)(virHostdevDriverPtr driver, virDomainObjPtr vm); /* - workflow: call reattachPciHostDevices and reattachUsbHostDevices to do specific work. - reference: void qemuDomainReAttachHostDevices(virQEMUDriverPtr driver, virDomainDefPtr def) */ typedef void (*virDrvReattachPciHostdevs) (virHostdevDriverPtr driver, virDomainObjPtr vm, virDomainHostdevDefPtr *hostdevs, int nhostdevs); /* - workflow: unbind and bind to original driver, free usedby, activePciHostdevs and inactivePciHostdevs. - reference: void qemuDomainReAttachHostdevDevices(virQEMUDriverPtrdriver, const char *name, virDomainHostdevDefPtr *hostdevs, int nhostdevs) */ typedef void (*virDrvReattachUsbHostdevs) (virHostdevDriverPtr driver, virDomainObjPtr vm, virDomainHostdevDefPtr *hostdevs, int nhostdevs); /* - workflow: free usedby, activePciHostdevs and inactivePciHostdevs. - reference: static void qemuDomainReAttachHostUsbDevices(virQEMUDriverPtr driver, const char *name, virDomainHostdevDefPtr *hostdevs, int nhostdevs) */ typedef virPCIDeviceListPtr (*virDrvGetActivePciHostdevList) (virHostdevDriverPtr driver); /* - could be used to check if a pci hostdev is already in-use. */ typedef virUSBDeviceListPtr (*virDrvGetActiveUsbHostdevList) (virHostdevDriverPtr driver); /* - could be used to check if a usb hostdev is already in-use. */ typedef virPCIDeviceListPtr (*virDrvGetDomainActivePciHostdevList) (virHostdevDriverPtr driver); /* - could be used to reattach all in-use pci hostdevs by a domain (e.g. when a domain shutdown) */ typedef virPCIDeviceListPtr (*virDrvGetDomainActiveUsbHostdevList) (virHostdevDriverPtr driver); /* - could be used to reattach all in-use usb hostdevs by a domain */ 3. Use the module * Register hostdev driver - define 'virHostdevDriverTab' and 'virHostdevDriverTabCount' - add hostdev driver register work in libvirtd.c daemonInitialize() * add hostdev driver areas to _virConnect struct _virConnect { ....... virNWFilterDriverPtr nwfilterDriver; +++ virHostdevDriverPtr hostdevDriver; ....... void * nwfilterPrivateData; +++ void * hostdevPrivateData; ....... } * add conn->hostdevDriver parser in libvirt.c: do_open() Then hypervisor drivers can get use of hostdev driver APIs. * moments that could be affacted: - domain start/shutoff - attach/detach hostdev to domain --------------------------------------------

3 6

[libvirt] [[libvirt-designer v4] 1/2] Add transfer annotation to *init.argv
by Zeeshan Ali (Khattak) 18 Apr '13

18 Apr '13

From: "Zeeshan Ali (Khattak)" <zeeshanak(a)gnome.org> Add transfer annotation to argv parameter of library init functions. --- libvirt-designer/libvirt-designer-main.c | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/libvirt-designer/libvirt-designer-main.c b/libvirt-designer/libvirt-designer-main.c index 927eb21..ce69024 100644 --- a/libvirt-designer/libvirt-designer-main.c +++ b/libvirt-designer/libvirt-designer-main.c @@ -33,7 +33,9 @@ /** * gvir_designer_init: * @argc: (inout): pointer to application's argc - * @argv: (inout) (array length=argc) (allow-none): pointer to application's argv + * @argv: (inout) (array length=argc) (allow-none) (transfer none): Address of the + * <parameter>argv</parameter> parameter of main(), or %NULL. Any options + * understood by GTK+ are stripped before return. */ void gvir_designer_init(int *argc, char ***argv) @@ -57,7 +59,9 @@ static void gvir_log_handler(const gchar *log_domain G_GNUC_UNUSED, /** * gvir_designer_init_check: * @argc: (inout): pointer to application's argc - * @argv: (inout) (array length=argc) (allow-none): pointer to application's argv + * @argv: (inout) (array length=argc) (allow-none) (transfer none): Address of the + * <parameter>argv</parameter> parameter of main(), or %NULL. Any options + * understood by GTK+ are stripped before return. * @err: pointer to a #GError to which a message will be posted on error */ gboolean gvir_designer_init_check(int *argc, -- 1.8.1.4

2 3

[libvirt] [PATCH v4 1/2] Add NVRAM device
by Li Zhang 18 Apr '13

18 Apr '13

From: Li Zhang <zhlcindy(a)linux.vnet.ibm.com> For pSeries guest in QEMU, NVRAM is one kind of spapr-vio device. Users are allowed to specify spapr-vio devices'address. But NVRAM is not supported in libvirt. So this patch is to add NVRAM device to allow users to specify its address. In QEMU, NVRAM device's address is specified by "-global spapr-nvram.reg=xxxxx". In libvirt, XML file is defined as the following: <nvram> <address type='spapr-vio' reg='0x3000'/> </nvram> Signed-off-by: Li Zhang <zhlcindy(a)linux.vnet.ibm.com> --- v4 -> v3: * Sperate NVRAM definition from qemu command line parser. v3 -> v2: * Add NVRAM in domaincommon.rng and formatdomain.xml.in suggested by Daniel P.Berrange * Add NVRAM test cases suggested by Daniel P.Berrange * Remove nvram allocation when it is not specified suggested by Daniel P.Berrange * Add several error reports suggested by Daniel P.Berrange v2 -> v1: * Add NVRAM parameters parsing in qemuParseCommandLine docs/formatdomain.html.in | 35 ++++++++++++++++++++ docs/schemas/domaincommon.rng | 10 ++++++ src/conf/domain_conf.c | 76 +++++++++++++++++++++++++++++++++++++++++++ src/conf/domain_conf.h | 10 ++++++ 4 files changed, 131 insertions(+) diff --git a/docs/formatdomain.html.in b/docs/formatdomain.html.in index d400e35..f590304 100644 --- a/docs/formatdomain.html.in +++ b/docs/formatdomain.html.in @@ -4415,6 +4415,41 @@ qemu-kvm -net nic,model=? /dev/null </dl> + <h4><a name="elementsNVRAM">NVRAM device</a></h4> + + One NVRAM device is always added to pSeries guests on PPC64. + And on PPC64, NVRAM devices' address type are VIO which + allows users to change.<code>nvram</code> element in XML file + is provided to specify its address. + Currently, libvirt only considers configuration for pSeries guests. + + + Example: usage of NVRAM configuration + +<pre> + ... + <devices> + <nvram> + <address type='spapr-vio' reg='0x3000'/> + </nvram> + </devices> + ... +</pre> + <dl> + <dt><code>spapr-vio</code></dt> + <dd> + + VIO device address type, this is only for PPC64. + + </dd> + <dt><code>reg</code></dt> + <dd> + + Devices' address + + </dd> + </dl> + <h3><a name="seclabel">Security label</a></h3> diff --git a/docs/schemas/domaincommon.rng b/docs/schemas/domaincommon.rng index 2c31f76..2b58d95 100644 --- a/docs/schemas/domaincommon.rng +++ b/docs/schemas/domaincommon.rng @@ -2764,6 +2764,13 @@ </optional> </element> </define> + <define name="nvram"> + <element name="nvram"> + <optional> + <ref name="address"/> + </optional> + </element> + </define> <define name="memballoon"> <element name="memballoon"> <attribute name="model"> @@ -3208,6 +3215,9 @@ <optional> <ref name="memballoon"/> </optional> + <optional> + <ref name="nvram"/> + </optional> </interleave> </element> </define> diff --git a/src/conf/domain_conf.c b/src/conf/domain_conf.c index 36a46da..3a5e59a 100644 --- a/src/conf/domain_conf.c +++ b/src/conf/domain_conf.c @@ -193,6 +193,7 @@ VIR_ENUM_IMPL(virDomainDevice, VIR_DOMAIN_DEVICE_LAST, "smartcard", "chr", "memballoon", + "nvram", "rng") VIR_ENUM_IMPL(virDomainDeviceAddress, VIR_DOMAIN_DEVICE_ADDRESS_TYPE_LAST, @@ -1552,6 +1553,16 @@ void virDomainMemballoonDefFree(virDomainMemballoonDefPtr def) VIR_FREE(def); } +void virDomainNVRAMDefFree(virDomainNVRAMDefPtr def) +{ + if (!def) + return; + + virDomainDeviceInfoClear(&def->info); + + VIR_FREE(def); +} + void virDomainWatchdogDefFree(virDomainWatchdogDefPtr def) { if (!def) @@ -1718,6 +1729,7 @@ void virDomainDeviceDefFree(virDomainDeviceDefPtr def) case VIR_DOMAIN_DEVICE_SMARTCARD: case VIR_DOMAIN_DEVICE_CHR: case VIR_DOMAIN_DEVICE_MEMBALLOON: + case VIR_DOMAIN_DEVICE_NVRAM: case VIR_DOMAIN_DEVICE_LAST: break; } @@ -1910,6 +1922,7 @@ void virDomainDefFree(virDomainDefPtr def) virDomainWatchdogDefFree(def->watchdog); virDomainMemballoonDefFree(def->memballoon); + virDomainNVRAMDefFree(def->nvram); for (i = 0; i < def->nseclabels; i++) virSecurityLabelDefFree(def->seclabels[i]); @@ -2478,6 +2491,12 @@ virDomainDeviceInfoIterateInternal(virDomainDefPtr def, if (cb(def, &device, &def->rng->info, opaque) < 0) return -1; } + if (def->nvram) { + device.type = VIR_DOMAIN_DEVICE_NVRAM; + device.data.nvram = def->nvram; + if (cb(def, &device, &def->nvram->info, opaque) < 0) + return -1; + } device.type = VIR_DOMAIN_DEVICE_HUB; for (i = 0; i < def->nhubs ; i++) { device.data.hub = def->hubs[i]; @@ -2509,6 +2528,7 @@ virDomainDeviceInfoIterateInternal(virDomainDefPtr def, case VIR_DOMAIN_DEVICE_SMARTCARD: case VIR_DOMAIN_DEVICE_CHR: case VIR_DOMAIN_DEVICE_MEMBALLOON: + case VIR_DOMAIN_DEVICE_NVRAM: case VIR_DOMAIN_DEVICE_LAST: case VIR_DOMAIN_DEVICE_RNG: break; @@ -7994,6 +8014,23 @@ error: goto cleanup; } +static virDomainNVRAMDefPtr +virDomainNVRAMDefParseXML(const xmlNodePtr node, + unsigned int flags) +{ + virDomainNVRAMDefPtr def; + + if (VIR_ALLOC(def) < 0) { + virReportOOMError(); + return NULL; + } + + if ( virDomainDeviceInfoParseXML(node, NULL, &def->info, flags) < 0 ) + return NULL; + + return def; +} + static virSysinfoDefPtr virSysinfoParseXML(const xmlNodePtr node, xmlXPathContextPtr ctxt) @@ -11105,6 +11142,26 @@ virDomainDefParseXML(xmlDocPtr xml, VIR_FREE(nodes); } + def->nvram = NULL; + if ((n = virXPathNodeSet("./devices/nvram", ctxt, &nodes)) < 0) { + goto error; + } + + if (n > 1) { + virReportError(VIR_ERR_INTERNAL_ERROR, "%s", + _("only a single nvram device is supported")); + goto error; + } + + if (n > 0) { + virDomainNVRAMDefPtr nvram = + virDomainNVRAMDefParseXML(nodes[0], flags); + if (!nvram) + goto error; + def->nvram = nvram; + VIR_FREE(nodes); + } + /* analysis of the hub devices */ if ((n = virXPathNodeSet("./devices/hub", ctxt, &nodes)) < 0) { goto error; @@ -14159,6 +14216,21 @@ virDomainMemballoonDefFormat(virBufferPtr buf, } static int +virDomainNVRAMDefFormat(virBufferPtr buf, + virDomainNVRAMDefPtr def, + unsigned int flags) +{ + virBufferAsprintf(buf, " <nvram>\n"); + if (virDomainDeviceInfoIsSet(&def->info, flags)) + if (virDomainDeviceInfoFormat(buf, &def->info, flags) < 0) + return -1; + + virBufferAddLit(buf, " </nvram>\n"); + + return 0; +} + +static int virDomainSysinfoDefFormat(virBufferPtr buf, virSysinfoDefPtr def) { @@ -15469,6 +15541,9 @@ virDomainDefFormatInternal(virDomainDefPtr def, if (def->rng) virDomainRNGDefFormat(buf, def->rng, flags); + if (def->nvram) + virDomainNVRAMDefFormat(buf, def->nvram, flags); + virBufferAddLit(buf, " </devices>\n"); virBufferAdjustIndent(buf, 2); @@ -16749,6 +16824,7 @@ virDomainDeviceDefCopy(virDomainDeviceDefPtr src, case VIR_DOMAIN_DEVICE_SMARTCARD: case VIR_DOMAIN_DEVICE_CHR: case VIR_DOMAIN_DEVICE_MEMBALLOON: + case VIR_DOMAIN_DEVICE_NVRAM: case VIR_DOMAIN_DEVICE_LAST: virReportError(VIR_ERR_INTERNAL_ERROR, _("Copying definition of '%d' type " diff --git a/src/conf/domain_conf.h b/src/conf/domain_conf.h index 3ab0f15..d98bec1 100644 --- a/src/conf/domain_conf.h +++ b/src/conf/domain_conf.h @@ -109,6 +109,9 @@ typedef virDomainChrDef *virDomainChrDefPtr; typedef struct _virDomainMemballoonDef virDomainMemballoonDef; typedef virDomainMemballoonDef *virDomainMemballoonDefPtr; +typedef struct _virDomainNVRAMDef virDomainNVRAMDef; +typedef virDomainNVRAMDef *virDomainNVRAMDefPtr; + typedef struct _virDomainSnapshotObj virDomainSnapshotObj; typedef virDomainSnapshotObj *virDomainSnapshotObjPtr; @@ -137,6 +140,7 @@ typedef enum { VIR_DOMAIN_DEVICE_SMARTCARD, VIR_DOMAIN_DEVICE_CHR, VIR_DOMAIN_DEVICE_MEMBALLOON, + VIR_DOMAIN_DEVICE_NVRAM, VIR_DOMAIN_DEVICE_RNG, VIR_DOMAIN_DEVICE_LAST @@ -163,6 +167,7 @@ struct _virDomainDeviceDef { virDomainSmartcardDefPtr smartcard; virDomainChrDefPtr chr; virDomainMemballoonDefPtr memballoon; + virDomainNVRAMDefPtr nvram; virDomainRNGDefPtr rng; } data; }; @@ -1442,6 +1447,9 @@ struct _virDomainMemballoonDef { virDomainDeviceInfo info; }; +struct _virDomainNVRAMDef { + virDomainDeviceInfo info; +}; enum virDomainSmbiosMode { VIR_DOMAIN_SMBIOS_NONE = 0, @@ -1883,6 +1891,7 @@ struct _virDomainDef { /* Only 1 */ virDomainWatchdogDefPtr watchdog; virDomainMemballoonDefPtr memballoon; + virDomainNVRAMDefPtr nvram; virCPUDefPtr cpu; virSysinfoDefPtr sysinfo; virDomainRedirFilterDefPtr redirfilter; @@ -2041,6 +2050,7 @@ int virDomainChrSourceDefCopy(virDomainChrSourceDefPtr src, void virDomainSoundCodecDefFree(virDomainSoundCodecDefPtr def); void virDomainSoundDefFree(virDomainSoundDefPtr def); void virDomainMemballoonDefFree(virDomainMemballoonDefPtr def); +void virDomainNVRAMDefFree(virDomainNVRAMDefPtr def); void virDomainWatchdogDefFree(virDomainWatchdogDefPtr def); void virDomainVideoDefFree(virDomainVideoDefPtr def); virDomainHostdevDefPtr virDomainHostdevDefAlloc(void); -- 1.8.1.4

1 4

[libvirt] [PATCH v2.5 00/10] add support for scsi-generic for virtio-scsi
by Han Cheng 18 Apr '13

18 Apr '13

This patch series tried to add support for scsi-generic for virtio-scsi. Changes from v2: - Lots of fixs on comments Thank you guys who reviewed these. Thank you for your help. There is some work left. As time comes up and I don't have much time to do this, I'd like to ask Osier to help me finish this. Could you? TODO: change readonly to bool and change related managed and missing for consistency. Han Cheng (10): conf: Change help function docs/schemas: Move PortNumber and sourceinfoadapter to basictypes.rng conf: Introduce scsi hostdev qemu: New cap flag for scsi-generic utils: util functions for scsi hostdev qemu: Build qemu command line for scsi-generic qemu: Basic management functions for scsi hostdev qemu: cgroup and selinux for scsi hostdev qemu: hotplug support for scsi hostdev tests: tests for scsi hostdev docs/formatdomain.html.in | 38 docs/schemas/basictypes.rng | 42 + docs/schemas/domaincommon.rng | 34 docs/schemas/storagepool.rng | 42 - po/POTFILES.in | 1 src/Makefile.am | 1 src/conf/domain_audit.c | 10 src/conf/domain_conf.c | 189 ++++ src/conf/domain_conf.h | 14 src/libvirt_private.syms | 24 src/qemu/qemu_capabilities.c | 14 src/qemu/qemu_capabilities.h | 2 src/qemu/qemu_cgroup.c | 67 + src/qemu/qemu_cgroup.h | 3 src/qemu/qemu_command.c | 138 +++ src/qemu/qemu_command.h | 6 src/qemu/qemu_conf.h | 2 src/qemu/qemu_driver.c | 3 src/qemu/qemu_hostdev.c | 218 +++++ src/qemu/qemu_hostdev.h | 10 src/qemu/qemu_hotplug.c | 211 ++++- src/qemu/qemu_process.c | 3 src/security/security_selinux.c | 56 + src/util/virscsi.c | 392 ++++++++++ src/util/virscsi.h | 83 ++ tests/qemuhelpdata/qemu-1.0-device | 10 tests/qemuhelpdata/qemu-1.1.0-device | 10 tests/qemuhelpdata/qemu-1.2.0-device | 5 tests/qemuhelpdata/qemu-kvm-1.2.0-device | 5 tests/qemuhelptest.c | 19 tests/qemuxml2argvdata/qemuxml2argv-hostdev-scsi-boot.args | 10 tests/qemuxml2argvdata/qemuxml2argv-hostdev-scsi-boot.xml | 34 tests/qemuxml2argvdata/qemuxml2argv-hostdev-scsi-readonly.args | 10 tests/qemuxml2argvdata/qemuxml2argv-hostdev-scsi-readonly.xml | 35 tests/qemuxml2argvtest.c | 9 tests/qemuxml2xmltest.c | 3 36 files changed, 1624 insertions(+), 129 deletions(-)

3 16

[libvirt] [PATCH 0/3 v3] cleanup: About header including
by Osier Yang 18 Apr '13

18 Apr '13

These 3 patches are the left ones in v2. 6/10 ~ 9/10 in v2 are merged into one single patch. Osier Yang (3): syntax-check: Don't include public headers in internal source syntax-check: Only allows to include public headers in external tools docs: Update HACKING cfg.mk | 21 +++++++++++++++++++++ daemon/remote.c | 2 -- docs/hacking.html.in | 8 ++++---- include/libvirt/libvirt-lxc.h | 2 +- include/libvirt/libvirt-qemu.h | 2 +- python/libvirt-lxc-override.c | 4 ++-- python/libvirt-override.c | 4 ++-- python/libvirt-qemu-override.c | 4 ++-- python/typewrappers.h | 4 ++-- src/libvirt-qemu.c | 1 - tests/shunloadhelper.c | 5 ++--- tools/virsh.c | 4 ++-- 12 files changed, 39 insertions(+), 22 deletions(-) -- 1.8.1.4

2 12

[libvirt] Got invaliProperty error when retrieve properties from ESXi server
by Dennis Zou (yunzou) 18 Apr '13

18 Apr '13

Hi team, When I tried to retrieve the property of storage.perDatastoreUsage under specify virtual machine and got a invalidProperty error but other properties(eg. summary.storage, guest) under the same virtual machine are ok. Then I write a same program use vsphere web service sdk in java and catch the http package by wireshark. By comparing, I found that we lost soapAction field in http request header, it seems to like this: SOAPAction: "urn:vim25/5.0". I checked the vsphere development document and found some key description as following: When a client application connects to a Web service running on an vSphere server (ESX/ESXi or vCenter Server system), the server detects the version of the API that was used to develop the client and makes available only those operations supported by the client. Client applications convey information about the API version used in the SOAP messages that they send to a vSphere server. These SOAP messages include a versionID in the soapAction attribute. The details are handled transparently by the SOAP toolkit and the client proxy code. The server adjusts its behavior based on the client's version information, exposing the API version that the client supports to the client. If you are developing a client application that must support multiple server versions at the same time (ESX/ESXi 5.0 and ESX/ESXi 3.x, for example), you must obtain information about the API versions that are supported on the server and provide logic in your code to use or not use features, based upon the version information. Finally, I added the soapAction field into http request header and got a correct result. So I added some lines in function esxVI_Context_Connect() at src/esx/esx_vi.c The code change is bold line as follwing: int esxVI_Context_Connect(esxVI_Context *ctx, const char *url, const char *ipAddress, const char *username, const char *password, esxUtil_ParsedUri *parsedUri) { char * soapAction = NULL; int result = -1; if (ctx == NULL || url == NULL || ipAddress == NULL || username == NULL || password == NULL || ctx->url != NULL || ctx->service != NULL || ctx->curl != NULL) { ESX_VI_ERROR(VIR_ERR_INTERNAL_ERROR, "%s", _("Invalid argument")); return -1; } if (esxVI_CURL_Alloc(&ctx->curl) < 0 || esxVI_CURL_Connect(ctx->curl, parsedUri) < 0 || esxVI_String_DeepCopyValue(&ctx->url, url) < 0 || esxVI_String_DeepCopyValue(&ctx->ipAddress, ipAddress) < 0 || esxVI_String_DeepCopyValue(&ctx->username, username) < 0 || esxVI_String_DeepCopyValue(&ctx->password, password) < 0) { return -1; } if (VIR_ALLOC(ctx->sessionLock) < 0) { virReportOOMError(); return -1; } if (virMutexInit(ctx->sessionLock) < 0) { ESX_VI_ERROR(VIR_ERR_INTERNAL_ERROR, "%s", _("Could not initialize session mutex")); return -1; } if (esxVI_RetrieveServiceContent(ctx, &ctx->service) < 0) { return -1; } if (virAsprintf(&soapAction, "SOAPAction: \"urn:vim25/%s\"", ctx->service->about->apiVersion) < 0) { virReportOOMError(); goto cleanup; } ctx->curl->headers = curl_slist_append(ctx->curl->headers, soapAction); if (STREQ(ctx->service->about->apiType, "HostAgent") || STREQ(ctx->service->about->apiType, "VirtualCenter")) { if (STRPREFIX(ctx->service->about->apiVersion, "2.5")) { ctx->apiVersion = esxVI_APIVersion_25; } else if (STRPREFIX(ctx->service->about->apiVersion, "4.0")) { ctx->apiVersion = esxVI_APIVersion_40; } else if (STRPREFIX(ctx->service->about->apiVersion, "4.1")) { ctx->apiVersion = esxVI_APIVersion_41; } else if (STRPREFIX(ctx->service->about->apiVersion, "4.")) { ctx->apiVersion = esxVI_APIVersion_4x; VIR_WARN( "Found untested VI API major/minor version '%s'", ctx->service->about->apiVersion); } else if (STRPREFIX(ctx->service->about->apiVersion, "5.0")) { ctx->apiVersion = esxVI_APIVersion_50; } else if (STRPREFIX(ctx->service->about->apiVersion, "5.1")) { ctx->apiVersion = esxVI_APIVersion_51; }else if (STRPREFIX(ctx->service->about->apiVersion, "5.")) { ctx->apiVersion = esxVI_APIVersion_5x; VIR_WARN( "Found untested VI API major/minor version '%s'", ctx->service->about->apiVersion); } else { ESX_VI_ERROR(VIR_ERR_INTERNAL_ERROR, _("Expecting VI API major/minor version '2.5', '4.x' or " "'5.x' but found '%s'"), ctx->service->about->apiVersion); return -1; } if (STREQ(ctx->service->about->productLineId, "gsx")) { if (STRPREFIX(ctx->service->about->version, "2.0")) { ctx->productVersion = esxVI_ProductVersion_GSX20; } else { ESX_VI_ERROR(VIR_ERR_INTERNAL_ERROR, _("Expecting GSX major/minor version '2.0' but " "found '%s'"), ctx->service->about->version); return -1; } } else if (STREQ(ctx->service->about->productLineId, "esx") || STREQ(ctx->service->about->productLineId, "embeddedEsx")) { if (STRPREFIX(ctx->service->about->version, "3.5")) { ctx->productVersion = esxVI_ProductVersion_ESX35; } else if (STRPREFIX(ctx->service->about->version, "4.0")) { ctx->productVersion = esxVI_ProductVersion_ESX40; } else if (STRPREFIX(ctx->service->about->version, "4.1")) { ctx->productVersion = esxVI_ProductVersion_ESX41; } else if (STRPREFIX(ctx->service->about->version, "4.")) { ctx->productVersion = esxVI_ProductVersion_ESX4x; VIR_WARN( "Found untested ESX major/minor version '%s'", ctx->service->about->version); } else if (STRPREFIX(ctx->service->about->version, "5.0")) { ctx->productVersion = esxVI_ProductVersion_ESX50; } else if (STRPREFIX(ctx->service->about->version, "5.1")) { ctx->productVersion = esxVI_ProductVersion_ESX51; }else if (STRPREFIX(ctx->service->about->version, "5.")) { ctx->productVersion = esxVI_ProductVersion_ESX5x; VIR_WARN( "Found untested ESX major/minor version '%s'", ctx->service->about->version); } else { ESX_VI_ERROR(VIR_ERR_INTERNAL_ERROR, _("Expecting ESX major/minor version '3.5', " "'4.x' or '5.x' but found '%s'"), ctx->service->about->version); return -1; } } else if (STREQ(ctx->service->about->productLineId, "vpx")) { if (STRPREFIX(ctx->service->about->version, "2.5")) { ctx->productVersion = esxVI_ProductVersion_VPX25; } else if (STRPREFIX(ctx->service->about->version, "4.0")) { ctx->productVersion = esxVI_ProductVersion_VPX40; } else if (STRPREFIX(ctx->service->about->version, "4.1")) { ctx->productVersion = esxVI_ProductVersion_VPX41; } else if (STRPREFIX(ctx->service->about->version, "4.")) { ctx->productVersion = esxVI_ProductVersion_VPX4x; VIR_WARN( "Found untested VPX major/minor version '%s'", ctx->service->about->version); } else if (STRPREFIX(ctx->service->about->version, "5.0")) { ctx->productVersion = esxVI_ProductVersion_VPX50; } else if (STRPREFIX(ctx->service->about->version, "5.1")) { ctx->productVersion = esxVI_ProductVersion_VPX51; }else if (STRPREFIX(ctx->service->about->version, "5.")) { ctx->productVersion = esxVI_ProductVersion_VPX5x; VIR_WARN( "Found untested VPX major/minor version '%s'", ctx->service->about->version); } else { ESX_VI_ERROR(VIR_ERR_INTERNAL_ERROR, _("Expecting VPX major/minor version '2.5', '4.x' " "or '5.x' but found '%s'"), ctx->service->about->version); return -1; } } else { ESX_VI_ERROR(VIR_ERR_INTERNAL_ERROR, _("Expecting product 'gsx' or 'esx' or 'embeddedEsx' " "or 'vpx' but found '%s'"), ctx->service->about->productLineId); return -1; } } else { ESX_VI_ERROR(VIR_ERR_INTERNAL_ERROR, _("Expecting VI API type 'HostAgent' or 'VirtualCenter' " "but found '%s'"), ctx->service->about->apiType); return -1; } if (ctx->productVersion & esxVI_ProductVersion_ESX) { /* * FIXME: Actually this should be detected by really calling * QueryVirtualDiskUuid and checking if a NotImplemented fault is * returned. But currently we don't deserialized the details of a * possbile fault and therefore we don't know if the fault was a * NotImplemented fault or not. */ ctx->hasQueryVirtualDiskUuid = true; } if (ctx->productVersion & esxVI_ProductVersion_VPX) { ctx->hasSessionIsActive = true; } if (esxVI_Login(ctx, username, password, NULL, &ctx->session) < 0 || esxVI_BuildSelectSetCollection(ctx) < 0) { return -1; } result = 0; cleanup: VIR_FREE(soapAction); return result; } int esxVI_Context_LookupManagedObjects(esxVI_Context *ctx) { /* Lookup Datacenter */ if (esxVI_LookupDatacenter(ctx, NULL, ctx->service->rootFolder, NULL, &ctx->datacenter, esxVI_Occurrence_RequiredItem) < 0) { return -1; } ctx->datacenterPath = strdup(ctx->datacenter->name); if (ctx->datacenterPath == NULL) { virReportOOMError(); return -1; } /* Lookup (Cluster)ComputeResource */ if (esxVI_LookupComputeResource(ctx, NULL, ctx->datacenter->hostFolder, NULL, &ctx->computeResource, esxVI_Occurrence_RequiredItem) < 0) { return -1; } if (ctx->computeResource->resourcePool == NULL) { ESX_VI_ERROR(VIR_ERR_INTERNAL_ERROR, "%s", _("Could not retrieve resource pool")); return -1; } ctx->computeResourcePath = strdup(ctx->computeResource->name); if (ctx->computeResourcePath == NULL) { virReportOOMError(); return -1; } if (STRCASEEQ(ctx->computeResource->_reference->type, "ClusterComputeResource")) { ctx->computeResourceType = esxVI_ComputeResourceType_Cluster; } else { ctx->computeResourceType = esxVI_ComputeResourceType_Basic; } /* Lookup HostSystem */ if (esxVI_LookupHostSystem(ctx, NULL, ctx->computeResource->_reference, NULL, &ctx->hostSystem, esxVI_Occurrence_RequiredItem) < 0) { return -1; } ctx->hostSystemName = strdup(ctx->hostSystem->name); if (ctx->hostSystemName == NULL) { virReportOOMError(); return -1; } return 0; } int esxVI_Context_LookupHostManagedObjects(esxVI_Context *ctx, virHostNameList *hostNameListPtr) { int result = -1; esxVI_Datacenter *datacenters = NULL; esxVI_Datacenter *candidateDatacenter = NULL; esxVI_ComputeResource *computeResources = NULL; esxVI_ComputeResource *candidateComputeResource = NULL; esxVI_HostSystem *hostSystems = NULL; esxVI_HostSystem *candidateHostSystem = NULL; esxVI_HostContext *current = NULL; virHostNameEntry *hostNameEntry = NULL; char *hostName = NULL; /* create hash table for host contexts*/ ctx->hostCtxs = virHashCreate(500, esxVI_HostContext_HashDataFree); /* list Datacenter */ if (esxVI_ListDatacenter(ctx, NULL, ctx->service->rootFolder, NULL, &datacenters, esxVI_Occurrence_OptionalList) < 0) { goto cleanup; } for (candidateDatacenter = datacenters; NULL != candidateDatacenter; candidateDatacenter = candidateDatacenter->_next) { /* list (Cluster)ComputeResource */ if (esxVI_ListComputeResource(ctx, NULL, candidateDatacenter->hostFolder, NULL, &computeResources, esxVI_Occurrence_OptionalList) < 0) { goto cleanup; } for (candidateComputeResource = computeResources; NULL != candidateComputeResource; candidateComputeResource = candidateComputeResource->_next) { /* list HostSystem */ if (esxVI_ListHostSystem(ctx, NULL, candidateComputeResource->_reference, NULL, &hostSystems, esxVI_Occurrence_OptionalList) < 0) { goto cleanup; } for (candidateHostSystem = hostSystems; NULL != candidateHostSystem; ) { /* Allocate a HostContext entry */ if (esxVI_HostContext_Alloc(&current) < 0 || esxVI_Datacenter_Alloc(&current->datacenter) < 0 || esxVI_ComputeResource_Alloc( &current->computeResource) < 0) { goto cleanup; } /* Fill the HostContext entry */ esxVI_ManagedObjectReference_DeepCopy( &current->datacenter->_reference, candidateDatacenter->_reference); esxVI_ManagedObjectReference_DeepCopy( &current->datacenter->hostFolder, candidateDatacenter->hostFolder); esxVI_ManagedObjectReference_DeepCopy( &current->datacenter->vmFolder, candidateDatacenter->vmFolder); current->datacenter->name = strdup(candidateDatacenter->name); current->datacenterPath = strdup(candidateDatacenter->name); esxVI_ManagedObjectReference_DeepCopy( &current->computeResource->_reference, candidateComputeResource->_reference); esxVI_ManagedObjectReference_DeepCopy( &current->computeResource->host, candidateComputeResource->host); esxVI_ManagedObjectReference_DeepCopy( &current->computeResource->resourcePool, candidateComputeResource->resourcePool); current->computeResource->name = strdup( candidateComputeResource->name); current->computeResourcePath = strdup( candidateComputeResource->name); if (STRCASEEQ(candidateComputeResource->_reference->type, "ClusterComputeResource")) { current->computeResourceType = esxVI_ComputeResourceType_Cluster; } else { current->computeResourceType = esxVI_ComputeResourceType_Basic; } current->hostSystem = candidateHostSystem; candidateHostSystem = candidateHostSystem->_next; current->hostSystem->_next = NULL; current->hostSystemName = strdup(current->hostSystem->name); hostName = strdup(current->hostSystem->name); current->maxVcpus = -1; current->supportsVMotion = esxVI_Boolean_Undefined; current->supportsLongMode = esxVI_Boolean_Undefined; current->usedCpuTimeCounterId = -1; /* update hash table */ if (virHashAddEntry(ctx->hostCtxs, hostName, current) < 0) { goto cleanup; } current = NULL; if (NULL != hostNameListPtr) { /* fill the hostNameList */ if (VIR_ALLOC(hostNameEntry) < 0) { virReportOOMError(); goto cleanup; } hostNameEntry->hostName = strdup(hostName); hostNameEntry->next = *hostNameListPtr; *hostNameListPtr = hostNameEntry; hostNameEntry = NULL; } free(hostName); } hostSystems = NULL; } esxVI_ComputeResource_Free(&computeResources); } result = 0; cleanup: esxVI_Datacenter_Free(&datacenters); esxVI_ComputeResource_Free(&computeResources); return result; } Regards, Dennis

1 0

[libvirt] [PATCH] conf: fix comment about parsing graphics listen address
by Ján Tomko 18 Apr '13

18 Apr '13

--- Pushed under the trivial rule. src/conf/domain_conf.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/conf/domain_conf.c b/src/conf/domain_conf.c index 548368e..1643f30 100644 --- a/src/conf/domain_conf.c +++ b/src/conf/domain_conf.c @@ -7291,7 +7291,7 @@ virDomainGraphicsListenDefParseXML(virDomainGraphicsListenDefPtr def, if (network && network[0]) { if (def->type != VIR_DOMAIN_GRAPHICS_LISTEN_TYPE_NETWORK) { /* network='xxx' never makes sense with anything except - * type='address' */ + * type='network' */ virReportError(VIR_ERR_XML_ERROR, "%s", _("network attribute not allowed when listen type is not network")); goto error; -- 1.8.1.5

1 0

[libvirt] [PATCH] Fix typo in ocaml_libvirt_storage_vol_get_info
by David Scott 18 Apr '13

18 Apr '13

The info.capacity was being overwritten with the info.allocation. Signed-off-by: David Scott <dave.scott(a)eu.citrix.com> --- libvirt/libvirt_c_oneoffs.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libvirt/libvirt_c_oneoffs.c b/libvirt/libvirt_c_oneoffs.c index 80e1c03..01985f5 100644 --- a/libvirt/libvirt_c_oneoffs.c +++ b/libvirt/libvirt_c_oneoffs.c @@ -1418,7 +1418,7 @@ ocaml_libvirt_storage_vol_get_info (value volv) rv = caml_alloc (3, 0); Store_field (rv, 0, Val_int (info.type)); v = caml_copy_int64 (info.capacity); Store_field (rv, 1, v); - v = caml_copy_int64 (info.allocation); Store_field (rv, 1, v); + v = caml_copy_int64 (info.allocation); Store_field (rv, 2, v); CAMLreturn (rv); #else -- 1.8.1.2

2 1

[libvirt] [PATCH 0/7] qemu: PCI bridge support
by Ján Tomko 18 Apr '13

18 Apr '13

This series allows PCI bridges to be added and used. After patch 6/7 more buses are usable if the bridges and addresses are explicitly specified in the XML. After 7/7 bridges are auto-added and the new buses are used automatically. (This only works if there is enough space on bus 0 for the bridges, otherwise they need to be specified manually). (Documentation, tests and bridge hotplug is still missing) Contains slightly modified li guang's patches 1-3: https://www.redhat.com/archives/libvir-list/2013-February/msg00981.html and reworked address allocation for multiple buses https://www.redhat.com/archives/libvir-list/2013-February/msg00793.html Ján Tomko (5): qemu: QEMU_PCI constant consistency qemu: move PCI address check out of qemuPCIAddressAsString qemu: switch PCI address set from hash table to an array qemu: Add support for plugging devices into PCI bridges qemu: auto-add and use bridges liguang (2): add pci-bridge controller type qemu: build command line for pci-bridge device docs/schemas/domaincommon.rng | 1 + src/conf/domain_conf.c | 3 +- src/conf/domain_conf.h | 1 + src/qemu/qemu_capabilities.c | 2 + src/qemu/qemu_capabilities.h | 1 + src/qemu/qemu_command.c | 551 ++++++++++++++++++++++++++++++------------ src/qemu/qemu_command.h | 4 +- tests/qemuhelptest.c | 21 +- 8 files changed, 421 insertions(+), 163 deletions(-) -- 1.8.1.5

8 46

[libvirt] [PATCH] Partially revert "cleanup: Don't include libvirt/libvirt.h"
by Eric Blake 17 Apr '13

17 Apr '13

This reverts the changes to remote_protocol.x made in commit 2d25fd4f410f65b40ef1c3d65124775a944f4025. It turns out that if 'dwarves' is installed, 'make check' fails if any *_LAST enum values are visible from within the .o file compiled from a .x file; including <libvirt/libvirt.h> prior to "internal.h" is the documented way to avoid exposing *_LAST enum elements. --- Pushing under the build-breaker rule. src/remote/remote_protocol.x | 1 + 1 file changed, 1 insertion(+) diff --git a/src/remote/remote_protocol.x b/src/remote/remote_protocol.x index d384225..b957b8e 100644 --- a/src/remote/remote_protocol.x +++ b/src/remote/remote_protocol.x @@ -36,6 +36,7 @@ * 'REMOTE_'. This makes names quite long. */ +%#include <libvirt/libvirt.h> %#include "internal.h" %#include <arpa/inet.h> -- 1.8.1.4

2 1

[libvirt] [PATCH 00/10 v2] Cleanup: About header including
by Osier Yang 17 Apr '13

17 Apr '13

The public headers libvirt/libvirt.h, libvirt/virterror.h, libvirt/libvirt-qemu.h, and libvirt/libvirt-lxc.h shouldn't be included in internal source directly. Osier Yang (10): cleanup: Remove the duplicate header cleanup: Only include testutils.h once cleanup: Don't include libvirt/libvirt.h cleanup: Don't include libvirt/virterror.h syntax-check: Don't include duplicate header syntax-check: Don't include "libvirt.h" and "virterror.h" in "" form syntax-check: Include libvirt.h and virterror.h in <> form only in external tools syntax-check: Don't include libvirt-{qemu,lxc}.h internally syntax-check: Include libvirt-{qemu,lxc}.h in <> form only in external tools docs: Update hacking cfg.mk | 106 ++++++++++++++++++++++++++++++++++++++ daemon/libvirtd.c | 1 - daemon/remote.c | 2 - docs/hacking.html.in | 8 +-- include/libvirt/libvirt-lxc.h | 2 +- include/libvirt/libvirt-qemu.h | 2 +- python/libvirt-lxc-override.c | 4 +- python/libvirt-override.c | 4 +- python/libvirt-qemu-override.c | 4 +- python/typewrappers.h | 4 +- src/conf/node_device_conf.c | 1 - src/libvirt-qemu.c | 1 - src/network/bridge_driver.c | 1 - src/nodeinfo.h | 1 - src/openvz/openvz_conf.c | 1 - src/openvz/openvz_driver.c | 1 - src/parallels/parallels_driver.c | 1 - src/phyp/phyp_driver.c | 2 - src/qemu/qemu_capabilities.h | 1 - src/qemu/qemu_driver.c | 1 - src/remote/remote_driver.h | 2 - src/remote/remote_protocol.x | 1 - src/security/security_selinux.c | 1 - src/uml/uml_driver.c | 1 - src/util/virkeycode.h | 1 - src/util/virnetdevtap.c | 1 - src/xen/xen_hypervisor.c | 1 - tests/domainsnapshotxml2xmltest.c | 4 +- tests/esxutilstest.c | 4 +- tests/lxcxml2xmltest.c | 4 +- tests/openvzutilstest.c | 4 +- tests/qemuargv2xmltest.c | 4 +- tests/qemuhelptest.c | 4 +- tests/qemumonitortest.c | 4 +- tests/qemuxml2argvtest.c | 4 +- tests/qemuxml2xmltest.c | 4 +- tests/qemuxmlnstest.c | 4 +- tests/shunloadhelper.c | 2 - tests/shunloadtest.c | 4 +- tests/vmx2xmltest.c | 4 +- tests/xml2vmxtest.c | 4 +- tools/virsh.c | 4 +- 42 files changed, 148 insertions(+), 66 deletions(-) -- 1.8.1.4

3 24

[libvirt] [PATCH] qemu: Allow the disk wwn to have "0x" prefix
by Osier Yang 17 Apr '13

17 Apr '13

The recent qemu requires "0x" prefix for the disk wwn, this patch changes virValidateWWN to allow the prefix, and prepend "0x" if it's not specified. E.g. qemu-kvm: -device scsi-hd,bus=scsi0.0,channel=0,scsi-id=0,lun=0,\ drive=drive-scsi0-0-0-0,id=scsi0-0-0-0,wwn=6000c60016ea71ad: Property 'scsi-hd.wwn' doesn't take value '6000c60016ea71ad' --- docs/schemas/basictypes.rng | 2 +- src/qemu/qemu_command.c | 8 ++++++-- src/util/virutil.c | 12 +++++++++--- tests/qemuxml2argvdata/qemuxml2argv-disk-ide-wwn.args | 2 +- tests/qemuxml2argvdata/qemuxml2argv-disk-scsi-disk-wwn.args | 4 ++-- tests/qemuxml2argvdata/qemuxml2argv-disk-scsi-disk-wwn.xml | 2 +- 6 files changed, 20 insertions(+), 10 deletions(-) diff --git a/docs/schemas/basictypes.rng b/docs/schemas/basictypes.rng index adaedd8..34c2254 100644 --- a/docs/schemas/basictypes.rng +++ b/docs/schemas/basictypes.rng @@ -280,7 +280,7 @@ <define name='wwn'> <data type='string'> - <param name='pattern'>[0-9a-fA-F]{16}</param> + <param name='pattern'>(0x)?[0-9a-fA-F]{16}</param> </data> </define> diff --git a/src/qemu/qemu_command.c b/src/qemu/qemu_command.c index 63b9350..009d42d 100644 --- a/src/qemu/qemu_command.c +++ b/src/qemu/qemu_command.c @@ -3358,8 +3358,12 @@ qemuBuildDriveDevStr(virDomainDefPtr def, disk->blockio.physical_block_size); } - if (disk->wwn) - virBufferAsprintf(&opt, ",wwn=%s", disk->wwn); + if (disk->wwn) { + if (STRPREFIX(disk->wwn, "0x")) + virBufferAsprintf(&opt, ",wwn=%s", disk->wwn); + else + virBufferAsprintf(&opt, ",wwn=0x%s", disk->wwn); + } if (disk->vendor) virBufferAsprintf(&opt, ",vendor=%s", disk->vendor); diff --git a/src/util/virutil.c b/src/util/virutil.c index 0e4063b..6890362 100644 --- a/src/util/virutil.c +++ b/src/util/virutil.c @@ -3228,12 +3228,18 @@ bool virIsDevMapperDevice(const char *dev_name ATTRIBUTE_UNUSED) bool virValidateWWN(const char *wwn) { int i; + const char *p = wwn; - for (i = 0; wwn[i]; i++) - if (!c_isxdigit(wwn[i])) + if (STRPREFIX(wwn, "0x")) { + p += 2; + } + + for (i = 0; p[i]; i++) { + if (!c_isxdigit(p[i])) break; + } - if (i != 16 || wwn[i]) { + if (i != 16 || p[i]) { virReportError(VIR_ERR_INTERNAL_ERROR, "%s", _("Malformed wwn: %s")); return false; diff --git a/tests/qemuxml2argvdata/qemuxml2argv-disk-ide-wwn.args b/tests/qemuxml2argvdata/qemuxml2argv-disk-ide-wwn.args index 1633d29..3b9693c 100644 --- a/tests/qemuxml2argvdata/qemuxml2argv-disk-ide-wwn.args +++ b/tests/qemuxml2argvdata/qemuxml2argv-disk-ide-wwn.args @@ -2,5 +2,5 @@ LC_ALL=C PATH=/bin HOME=/home/test USER=test LOGNAME=test \ /usr/bin/qemu -S -M pc -m 214 -smp 1 -nographic -nodefaults \ -monitor unix:/tmp/test-monitor,server,nowait -no-acpi -boot c -usb \ -drive file=/dev/HostVG/QEMUGuest1,if=none,id=drive-ide0-0-1,serial=WD-WMAP9A966149 \ --device ide-hd,bus=ide.0,unit=1,drive=drive-ide0-0-1,id=ide0-0-1,wwn=5000c50015ea71ad \ +-device ide-hd,bus=ide.0,unit=1,drive=drive-ide0-0-1,id=ide0-0-1,wwn=0x5000c50015ea71ad \ -device virtio-balloon-pci,id=balloon0,bus=pci.0,addr=0x3 diff --git a/tests/qemuxml2argvdata/qemuxml2argv-disk-scsi-disk-wwn.args b/tests/qemuxml2argvdata/qemuxml2argv-disk-scsi-disk-wwn.args index 0393640..0dd2aa9 100644 --- a/tests/qemuxml2argvdata/qemuxml2argv-disk-scsi-disk-wwn.args +++ b/tests/qemuxml2argvdata/qemuxml2argv-disk-scsi-disk-wwn.args @@ -5,7 +5,7 @@ LC_ALL=C PATH=/bin HOME=/home/test USER=test LOGNAME=test \ -device lsi,id=scsi1,bus=pci.0,addr=0x4 \ -usb \ -drive file=/dev/HostVG/QEMUGuest1,if=none,id=drive-scsi0-0-1-0 \ --device scsi-cd,bus=scsi0.0,channel=0,scsi-id=1,lun=0,drive=drive-scsi0-0-1-0,id=scsi0-0-1-0,wwn=5000c50015ea71ac \ +-device scsi-cd,bus=scsi0.0,channel=0,scsi-id=1,lun=0,drive=drive-scsi0-0-1-0,id=scsi0-0-1-0,wwn=0x5000c50015ea71ac \ -drive file=/dev/HostVG/QEMUGuest2,if=none,id=drive-scsi0-0-0-0 \ --device scsi-hd,bus=scsi0.0,channel=0,scsi-id=0,lun=0,drive=drive-scsi0-0-0-0,id=scsi0-0-0-0,wwn=5000c50015ea71ad \ +-device scsi-hd,bus=scsi0.0,channel=0,scsi-id=0,lun=0,drive=drive-scsi0-0-0-0,id=scsi0-0-0-0,wwn=0x5000c50015ea71ad \ -device virtio-balloon-pci,id=balloon0,bus=pci.0,addr=0x5 diff --git a/tests/qemuxml2argvdata/qemuxml2argv-disk-scsi-disk-wwn.xml b/tests/qemuxml2argvdata/qemuxml2argv-disk-scsi-disk-wwn.xml index dc35548..caf957b 100644 --- a/tests/qemuxml2argvdata/qemuxml2argv-disk-scsi-disk-wwn.xml +++ b/tests/qemuxml2argvdata/qemuxml2argv-disk-scsi-disk-wwn.xml @@ -25,7 +25,7 @@ <source dev='/dev/HostVG/QEMUGuest2'/> <target dev='sdb' bus='scsi'/> <address type='drive' controller='0' bus='0' target='0' unit='0'/> - <wwn>5000c50015ea71ad</wwn> + <wwn>0x5000c50015ea71ad</wwn> </disk> <controller type='usb' index='0'/> <controller type='scsi' index='0' model='virtio-scsi'/> -- 1.8.1.4

2 3

[libvirt] [PATCH] docs: Add detailed notes snapshots, blockcommit, blockpull
by Kashyap Chamarthy 17 Apr '13

17 Apr '13

More elaborate notes on snapshots, blockpull, blockcommit. Much of this is derived from various dicussions with Eric Blake, Jeff Cody, Kevin Wolf (thanks a lot!) & several others on IRC and mailing lists and a lot of adhoc testing. I didn't wanted this to get lost. I also plan to add notes for 'blockcopy' once I complete testing with upstream libvirt/qemu git. NOTE: This document is formatted using reStructuredText. And can be trivially converted to HTML using: # rst2html snapshots-blockcommit-blockpull.rst > snapshots-blockcommit-blockpull.html ('rst2html' is part of python-docutils package.) I didn't send an html PATCH directly, as I thought, this'd be more readable. Any comments, criticisms more than welcome. --- docs/snapshots-blockcommit-blockpull.rst | 646 ++++++++++++++++++++++++++++++ 1 files changed, 646 insertions(+), 0 deletions(-) create mode 100644 docs/snapshots-blockcommit-blockpull.rst diff --git a/docs/snapshots-blockcommit-blockpull.rst b/docs/snapshots-blockcommit-blockpull.rst new file mode 100644 index 0000000000000000000000000000000000000000..99c30223a004ee5291e2914b788ac7fe04eee3c8 --- /dev/null +++ b/docs/snapshots-blockcommit-blockpull.rst @@ -0,0 +1,646 @@ +.. ---------------------------------------------------------------------- + Note: All these tests were performed with latest qemu-git,libvirt-git (as of + 20-Oct-2012 on a Fedora-18 alpha machine +.. ---------------------------------------------------------------------- + + +Introduction +============ + +A virtual machine snapshot is a view of a virtual machine(its OS & all its +applications) at a given point in time. So that, one can revert to a known sane +state, or take backups while the guest is running live. So, before we dive into +snapshots, let's have an understanding of backing files and overlays. + + + +QCOW2 backing files & overlays +------------------------------ + +In essence, QCOW2(Qemu Copy-On-Write) gives you an ability to create a base-image, +and create several 'disposable' copy-on-write overlay disk images on top of the +base image(also called backing file). Backing files and overlays are +extremely useful to rapidly instantiate thin-privisoned virtual machines(more on +it below). Especially quite useful in development & test environments, so that +one could quickly revert to a known state & discard the overlay. + +**Figure-1** + +:: + + .--------------. .-------------. .-------------. .-------------. + | | | | | | | | + | RootBase |<---| Overlay-1 |<---| Overlay-1A <--- | Overlay-1B | + | (raw/qcow2) | | (qcow2) | | (qcow2) | | (qcow2) | + '--------------' '-------------' '-------------' '-------------' + +The above figure illustrates - RootBase is the backing file for Overlay-1, which +in turn is backing file for Overlay-2, which in turn is backing file for +Overlay-3. + +**Figure-2** +:: + + .-----------. .-----------. .------------. .------------. .------------. + | | | | | | | | | | + | RootBase |<--- Overlay-1 |<--- Overlay-1A <--- Overlay-1B <--- Overlay-1C | + | | | | | | | | | (Active) | + '-----------' '-----------' '------------' '------------' '------------' + ^ ^ + | | + | | .-----------. .------------. + | | | | | | + | '-------| Overlay-2 |<---| Overlay-2A | + | | | | (Active) | + | '-----------' '------------' + | + | + | .-----------. .------------. + | | | | | + '------------| Overlay-3 |<---| Overlay-3A | + | | | (Active) | + '-----------' '------------' + +The above figure is just another representation which indicates, we can use a +'single' backing file, and create several overlays -- which can be used further, +to create overlays on top of them. + + +**NOTE**: Backing files are always opened **read-only**. In other words, once + an overlay is created, its backing file should not be modified(as the + overlay depends on a particular state of the backing file). Refer + below ('blockcommit' section) for relevant info on this. + + +**Example** : + +:: + + [FedoraBase.img] ----- <- [Fedora-guest-1.qcow2] <- [Fed-w-updates.qcow2] <- [Fedora-guest-with-updates-1A] + \ + \--- <- [Fedora-guest-2.qcow2] <- [Fed-w-updates.qcow2] <- [Fedora-guest-with-updates-2A] + +(Arrow to be read as Fed-w-updates.qcow2 has Fedora-guest-1.qcow2 as its backing file.) + +In the above example, say, *FedoraBase.img* has a freshly installed Fedora-17 OS on it, +and let's establish it as our backing file. Now, FedoraBase can be used as a +read-only 'template' to quickly instantiate two(or more) thinly provisioned +Fedora-17 guests(say Fedora-guest-1.qcow2, Fedora-guest-2.qcow2) by creating +QCOW2 overlay files pointing to our backing file. Also, the example & *Figure-2* +above illustrate that a single root-base image(FedoraBase.img) can be used +to create multiple overlays -- which can subsequently have their own overlays. + + + To create two thinly-provisioned Fedora clones(or overlays) using a single + backing file, we can invoke qemu-img as below: :: + + + # qemu-img create -b /export/vmimages/RootBase.img -f qcow2 \ + /export/vmimages/Fedora-guest-1.qcow2 + + # qemu-img create -b /export/vmimages/RootBase.img -f qcow2 \ + /export/vmimages/Fedora-guest-2.qcow2 + + Now, both the above images *Fedora-guest-1* & *Fedora-guest-2* are ready to + boot. Continuting with our example, say, now you want to instantiate a + Fedora-17 guest, but this time, with full Fedora updates. This can be + accomplished by creating another overlay(Fedora-guest-with-updates-1A) - but + this overly would point to 'Fed-w-updates.qcow2' as its backing file (which + has the full Fedora updates) :: + + # qemu-img create -b /export/vmimages/Fed-w-updates.qcow2 -f qcow2 \ + /export/vmimages/Fedora-guest-with-updates-1A.qcow2 + + + Information about a disk image, like virtual size, disk size, backing file(if it + exists) can be obtained by using 'qemu-img' as below: + :: + + # qemu-img info /export/vmimages/Fedora-guest-with-updates-1A.qcow2 + + NOTE: With latest qemu, an entire backing chain can be recursively + enumerated by doing: + :: + + # qemu-img info --backing-chain /export/vmimages/Fedora-guest-with-updates-1A.qcow2 + + + +Snapshot Terminology: +--------------------- + + - **Internal Snapshots** -- A single qcow2 image file holds both the saved state + & the delta since that saved point. This can be further classified as :- + + (1) **Internal disk snapshot**: The state of the virtual disk at a given + point in time. Both the snapshot & delta since the snapshot are + stored in the same qcow2 file. Can be taken when the guest is 'live' + or 'offline'. + + - Libvirt uses QEMU's 'qemu-img' command when the guest is 'offline'. + - Libvirt uses QEMU's 'savevm' command when the guest is 'live'. + + (2) **Internal system checkpoint**: RAM state, device state & the + disk-state of a running guest, are all stored in the same originial + qcow2 file. Can be taken when the guest is running 'live'. + + - Libvirt uses QEMU's 'savevm' command when the guest is 'live' + + + - **External Snapshots** -- Here, when a snapshot is taken, the saved state will + be stored in one file(from that point, it becomes a read-only backing + file) & a new file(overlay) will track the deltas from that saved state. + This can be further classified as :- + + (1) **External disk snapshot**: The snapshot of the disk is saved in one + file, and the delta since the snapshot is tracked in a new qcow2 + file. Can be taken when the guest is 'live' or 'offline'. + + - Libvirt uses QEMU's 'transaction' cmd under the hood, when the + guest is 'live'. + + - Libvirt uses QEMU's 'qemu-img' cmd under the hood when the + guest is 'offline'(this implementation is in progress, as of + writing this). + + (2) **External system checkpoint**: Here, the guest's disk-state will be + saved in one file, its RAM & device-state will be saved in another + new file (This implementation is in progress upstream libvirt, as of + writing this). + + + + - **VM State**: Saves the RAM & device state of a running guest(not 'disk-state') to + a file, so that it can be restored later. This simliar to doing hibernate + of the system. (NOTE: The disk-state should be unmodified at the time of + restoration.) + + - Libvirt uses QEMU's 'migrate' (to file) cmd under the hood. + + + +Creating snapshots +================== + - Whenever an 'external' snapshot is issued, a /new/ overlay image is + created to facilitate guest writes, and the previous image becomes a + snapshot. + + - **Create a disk-only internal snapshot** + + (1) If I have a guest named 'f17vm1', to create an offline or online + 'internal' snapshot called 'snap1' with description 'snap1-desc' :: + + # virsh snapshot-create-as f17vm1 snap1 snap1-desc + + (2) List the snapshot ; and query using *qemu-img* tool to view + the image info & its internal snapshot details :: + + # virsh snapshot-list f17vm1 + # qemu-img info /home/kashyap/vmimages/f17vm1.qcow2 + + + + - **Create a disk-only external snapshot** : + + (1) List the block device associated with the guest. :: + + # virsh domblklist f17-base + Target Source + --------------------------------------------- + vda /export/vmimages/f17-base.qcow2 + + # + + (2) Create external disk-only snapshot (while the guest is *running*). :: + + # virsh snapshot-create-as --domain f17-base snap1 snap1-desc \ + --disk-only --diskspec vda,snapshot=external,file=/export/vmimages/sn1-of-f17-base.qcow2 \ + --atomic + Domain snapshot snap1 created + # + + * Once the above command is issued, the original disk-image + of f17-base will become the backing_file & a new overlay + image is created to track the new changes. Here on, libvirt + will use this overlay for further write operations(while + using the original image as a read-only backing_file). + + (3) Now, list the block device associated(use cmd from step-1, above) + with the guest,again, to ensure it reflects the new overlay image as + the current block device in use. :: + + # virsh domblklist f17-base + Target Source + ---------------------------------------------------- + vda /export/vmimages/sn1-of-f17-base.qcow2 + + # + + + + +Reverting to snapshots +====================== +As of writing this, reverting to 'Internal Snapshots'(system checkpoint or +disk-only) is possible. + + To revert to a snapshot named 'snap1' of domain f17vm1 :: + + # virsh snapshot-revert --domain f17vm1 snap1 + +Reverting to 'external disk snapshots' using *snapshot-revert* is a little more +tricky, as it involves slightly complicated process of dealing with additional +snapshot files - whether to merge 'base' images into 'top' or to merge other way +round ('top' into 'base'). + +That said, there are a couple of ways to deal with external snapshot files by +merging them to reduce the external snapshot disk image chain by performing +either a **blockpull** or **blockcommit** (more on this below). + +Further improvements on this front is in work upstream libvirt as of writing +this. + + + +Merging snapshot files +====================== +External snapshots are incredibly useful. But, with plenty of external snapshot +files, there comes a problem of maintaining and tracking all these inidivdual +files. At a later point in time, we might want to 'merge' some of these snapshot +files (either backing_files into overlays or vice-versa) to reduce the length of +the image chain. To accomplish that, there are two mechanisms: + + + blockcommit: merges data from **top** into **base** (in other + words, merge overlays into backing files). + + + + blockpull: Populates a disk image with data from its backing file. Or + merges data from **base** into **top** (in other words, merge backing files + into overlays). + + +blockcommit +----------- + +Block Commit allows you to merge from a 'top' image(within a disk backing file +chain) into a lower-level 'base' image. To rephrase, it allows you to +merge overlays into backing files. Once the **blockcommit** operation is finished, +any portion that depends on the 'top' image, will now be pointing to the 'base'. + +This is useful in flattening(or collapsing or reducing) backing file chain +length after taking several external snapshots. + + +Let's understand with an illustration below: + +We have a base image called 'RootBase', which has a disk image chain with 4 +external snapshots. With 'Active' as the current active-layer, where 'live' guest +writes happen. There are a few possibilities of resulting image chains that we +can end up with, using 'blockcommit' : + + (1) Data from Snap-1, Snap-2 and Snap-3 can be merged into 'RootBase' + (resulting in RootBase becoming the backing_file of 'Active', and thus + invalidating Snap-1, Snap-2, & Snap-3). + + (2) Data from Snap-1 and Snap-2 can be merged into RootBase(resulting in + Rootbase becoming the backing_file of Snap-3, and thus invalidating + Snap-1 & Snap-2). + + (3) Data from Snap-1 can be merged into RootBase(resulting in RootBase + becoming the backing_file of Snap-2, and thus invalidating Snap-1). + + (4) Data from Snap-2 can be merged into Snap-1(resulting in Snap-1 becoming + the backing_file of Snap-3, and thus invalidating Snap-2). + + (5) Data from Snap-3 can be merged into Snap-2(resulting in Snap-2 becoming + the backing_file for 'Active', and thus invalidating Snap-3). + + (6) Data from Snap-2 and Snap-3 can be merged into Snap-1(resulting in + Snap-1 becoming the backing_file of 'Active', and thus invalidating + Snap-2 & Snap-3). + + NOTE: Eventually(not supported in qemu as of writing this), we can also + merge down the 'Active' layer(the top-most overlay) into its + backing_files. Once it is supported, the 'top' argument can become + optional, and default to active layer. + + +(The below figure illustrates case (6) from the above) + +**Figure-3** +:: + + .------------. .------------. .------------. .------------. .------------. + | | | | | | | | | | + | RootBase <--- Snap-1 <--- Snap-2 <--- Snap-3 <--- Snap-4 | + | | | | | | | | | (Active) | + '------------' '------------' '------------' '------------' '------------' + / | + / | + / commit data | + / | + / | + / | + v commit data | + .------------. .------------. <--------------------' .------------. + | | | | | | + | RootBase <--- Snap-1 |<---------------------------------| Snap-4 | + | | | | Backing File | (Active) | + '------------' '------------' '------------' + +For instance, if we have the below scenario: + + Actual: [base] <- sn1 <- sn2 <- sn3 <- sn4(this is active) + + Desired: [base] <- sn1 <- sn4 (thus invalidating sn2,sn3) + + Any of the below two methods is valid (as of 17-Oct-2012 qemu-git). With + method-a, operation will be faster & correct if we don't care about + sn2(because, it'll be invalidated). Note that, method-b is slower, but sn2 + will remain valid. (Also note that, the guest is 'live' in all these cases). + + **(method-a)**: + :: + + # virsh blockcommit --domain f17 vda --base /export/vmimages/sn1.qcow2 --top /export/vmimages/sn3.qcow2 --wait --verbose + + [OR] + + **(method-b)**: + :: + + # virsh blockcommit --domain f17 vda --base /export/vmimages/sn2.qcow2 --top /export/vmimages/sn3.qcow2 --wait --verbose + # virsh blockcommit --domain f17 vda --base /export/vmimages/sn1.qcow2 --top /export/vmimages/sn2.qcow2 --wait --verbose + + NOTE: If we had to do manually with *qemu-img* cmd, we can only do method-b at the moment. + + +**Figure-4** +:: + + .------------. .------------. .------------. .------------. .------------. + | | | | | | | | | | + | RootBase <--- Snap-1 <--- Snap-2 <--- Snap-3 <--- Snap-4 | + | | | | | | | | | (Active) | + '------------' '------------' '------------' '------------' '------------' + / | | + / | | + / | | + commit data / commit data | | + / | | + / | commit data | + v | | + .------------.<----------------------|-------------' .------------. + | |<----------------------' | | + | RootBase | | Snap-4 | + | |<-------------------------------------------------| (Active) | + '------------' Backing File '------------' + + +The above figure is another representation of reducing the disk image chain +using blockcommit. Data from Snap-1, Snap-2, Snap-3 are merged(/committed) +into RootBase, & now the current 'Active' image now pointing to 'RootBase' as its +backing file(instead of Snap-3, which was the case *before* blockcommit). Note +that, now intermediate images Snap-1, Snap-1, Snap-3 will be invalidated(as they were +dependent on a particular state of RootBase). + +blockpull +--------- +Block Pull(also called 'Block Stream' in QEMU's paralance) allows you to merge +into 'base' from a 'top' image(within a disk backing file chain). To rephrase it +allows merging backing files into an overlay(active). This works in the +opposite side of 'blockcommit' to flatten the snapshot chain. At the moment, +**blockpull** can pull only into the active layer(the top-most image). It's +worth noting here that, intermediate images are not invalidated once a blockpull +operation is complete (while blockcommit, invalidates them). + + +Consider the below illustration: + +**Figure-5** +:: + + .------------. .------------. .------------. .------------. .------------. + | | | | | | | | | | + | RootBase <--- Snap-1 <--- Snap-2 <--- Snap-3 <--- Snap-4 | + | | | | | | | | | (Active) | + '------------' '------------' '------------' '------------' '------------' + | | \ + | | \ + | | \ + | | \ stream data + | | stream data \ + | stream data | \ + | | v + .------------. | '---------------> .------------. + | | '---------------------------------> | | + | RootBase | | Snap-4 | + | | <---------------------------------------- | (Active) | + '------------' Backing File '------------' + + + +The above figure illustrates that, using block-copy we can pull data from +Snap-1, Snap-2 and Snap-3 into the 'Active' layer, resulting in 'RootBase' +becoming the backing file for the 'Active' image (instead of 'Snap-3', which was +the case before doing the blockpull operation). + +The command flow would be: + (1) Assuming a external disk-only snapshot was created as mentioned in + *Creating Snapshots* section: + + (2) A blockpull operation can be issued this way, to achieve the desired + state of *Figure-5*-- [RootBase] <- [Active]. :: + + # virsh blockpull --domain RootBase --path var/lib/libvirt/images/active.qcow2 --base /var/lib/libvirt/images/RootBase.qcow2 --wait --verbose + + + As a follow up, we can do the below to clean-up the snapshot *tracking* + metadata by libvirt (note: the below does not 'remove' the files, it + just cleans up the snapshot tracking metadata). :: + + # virsh snapshot-delete --domain RootBase Snap-3 --metadata + # virsh snapshot-delete --domain RootBase Snap-2 --metadata + # virsh snapshot-delete --domain RootBase Snap-1 --metadata + + + + +**Figure-6** +:: + + .------------. .------------. .------------. .------------. .------------. + | | | | | | | | | | + | RootBase <--- Snap-1 <--- Snap-2 <--- Snap-3 <--- Snap-4 | + | | | | | | | | | (Active) | + '------------' '------------' '------------' '------------' '------------' + | | | \ + | | | \ + | | | \ stream data + | | | stream data \ + | | | \ + | | stream data | \ + | stream data | '------------------> v + | | .--------------. + | '---------------------------------> | | + | | Snap-4 | + '----------------------------------------------------> | (Active) | + '--------------' + 'Standalone' + (w/o backing + file) + +The above figure illustrates, once blockpull operation is complete, by +pulling/streaming data from RootBase, Snap-1, Snap-2, Snap-3 into 'Active', all +the backing files can be discarded and 'Active' now will be a standalone image +without any backing files. + +Command flow would be: + (0) Assuming 4 external disk-only (live) snapshots were created as + mentioned in *Creating Snapshots* section, + + (1) Let's check the snapshot overlay images size *before* blockpull operation (note the image of 'Active'): + :: + + # ls -lash /var/lib/libvirt/images/RootBase.img + 608M -rw-r--r--. 1 qemu qemu 1.0G Oct 11 17:54 /var/lib/libvirt/images/RootBase.img + + # ls -lash /var/lib/libvirt/images/*Snap* + 840K -rw-------. 1 qemu qemu 896K Oct 11 17:56 /var/lib/libvirt/images/Snap-1.qcow2 + 392K -rw-------. 1 qemu qemu 448K Oct 11 17:56 /var/lib/libvirt/images/Snap-2.qcow2 + 456K -rw-------. 1 qemu qemu 512K Oct 11 17:56 /var/lib/libvirt/images/Snap-3.qcow2 + 2.9M -rw-------. 1 qemu qemu 3.0M Oct 11 18:10 /var/lib/libvirt/images/Active.qcow2 + + (2) Also, check the disk image information of 'Active'. It can noticed that + 'Active' has Snap-3 as its backing file. :: + + # qemu-img info /var/lib/libvirt/images/Active.qcow2 + image: /var/lib/libvirt/images/Active.qcow2 + file format: qcow2 + virtual size: 1.0G (1073741824 bytes) + disk size: 2.9M + cluster_size: 65536 + backing file: /var/lib/libvirt/images/Snap-3.qcow2 + + (3) Do the **blockpull** operation. :: + + # virsh blockpull --domain ptest2-base --path /var/lib/libvirt/images/Active.qcow2 --wait --verbose + Block Pull: [100 %] + Pull complete + + (4) Let's again check the snapshot overlay images size *after* + blockpull operation. It can be noticed, 'Active' is now considerably larger. :: + + # ls -lash /var/lib/libvirt/images/*Snap* + 840K -rw-------. 1 qemu qemu 896K Oct 11 17:56 /var/lib/libvirt/images/Snap-1.qcow2 + 392K -rw-------. 1 qemu qemu 448K Oct 11 17:56 /var/lib/libvirt/images/Snap-2.qcow2 + 456K -rw-------. 1 qemu qemu 512K Oct 11 17:56 /var/lib/libvirt/images/Snap-3.qcow2 + 1011M -rw-------. 1 qemu qemu 3.0M Oct 11 18:29 /var/lib/libvirt/images/Active.qcow2 + + + (5) Also, check the disk image information of 'Active'. It can now be + noticed that 'Active' is a standalone image without any backing file - + which is the desired state of *Figure-6*.:: + + # qemu-img info /var/lib/libvirt/images/Active.qcow2 + image: /var/lib/libvirt/images/Active.qcow2 + file format: qcow2 + virtual size: 1.0G (1073741824 bytes) + disk size: 1.0G + cluster_size: 65536 + + (6) We can now clean-up the snapshot tracking metadata by libvirt to + reflect the new reality :: + + # virsh snapshot-delete --domain RootBase Snap-3 --metadata + + (7) Optionally, one can check, the guest disk contents by invoking + *guestfish* tool(part of *libguestfs*) **READ-ONLY** (*--ro* option + below does it) as below :: + + # guestfish --ro -i -a /var/lib/libvirt/images/Active.qcow2 + + +Deleting snapshots (and 'offline commit') +========================================= + +Deleting (live/offline) *Internal Snapshots* (where the originial & all the named snapshots +are stored in a single QCOW2 file), is quite straight forward. :: + + # virsh snapshot-delete --domain f17vm --snapshotname snap6 + + [OR] + + # virsh snapshot-delete f17vm snap6 + +Deleting External snapshots (offline), Libvirt has not acquired the capability. +But, it can be done via *qemu-img* manipulation. + +Say, we have this image chain(the guest is *offline* here): **base <- sn1 <- sn2 <- sn3** +(arrow to be read as 'sn3 has sn2 as its backing file'). + + +And, we want to delete the second snapshot(sn2). It's possible to do it in two +ways: + + + - **Method (1)**: **base <- sn1 <- sn3** (by copying sn2 into sn1) + - **Method (2)**: **base <- sn1 <- sn3** (by copying sn2 into sn3) + +Method (1) +---------- +To end up with this image chain : **base <- sn1 <- sn3** (by copying *sn2* into *sn1*) + +**NOTE**: This is only possible *if* sn1 isn't used by more images as their backing +file, or they'd get corrupted!! + + (a) We're doing an *offline commit* (similar to what *blockcommit* can do + to an *online* guest). :: + + # qemu-img commit sn2.qcow2 + + - This will *commit* the changes from sn2 into its backing file(which is + sn1). + + (b) Now that we've comitted changes from sn2 into sn1, let's change the + backing file link in sn3 to point to sn1. :: + + # qemu-img rebase -u -b sn1.qcow2 sn3.qcow2 + + - **NOTE**: This is 'Unsafe mode' -- in this mode, only the backing file + name is changed w/o any checks on the file contents. The user must + take care of specifying the correct new backing file, or the + guest-visible. This mode is useful for renaming or moving the + backing file to somewhere else. It can be used without an + accessible old backing file, i.e. you can use it to fix an image + whose backing file has already been moved/renamed. + + + (c) Now, we can delete the sn2 disk image(as the changes are now committed + to sn1). :: + + # rm sn2.qcow2 + + +Method (2) +---------- +To end up with this image chain : **base <- sn1 <- sn3** (by copying *sn2* into *sn3*) + + (a) Copy contents of sn2(the old backing file) into sn3, and change the backing file link of sn3 to sn1:: + + # qemu-img rebase -b sn1.qcow2 sn3.qcow2 + + - Apart from changing backing file link of sn3 to sn1, the above cmd + will it also /copy/ the contents from sn2 into sn3). + + - In other words: This is 'Safe mode', which is the default -- + any clusters that differ between the new backing_file(in this + case, sn1) and the old backing file(in this case, sn2) of + filename(in this case, sn3) are merged into filename(sn3), before + actually changing the backing file. + + (b) Now, we can delete the sn2 disk image(as the changes are now committed to + sn1). :: + + # rm sn2.qcow2 + -- 1.7.7.6

4 6

[libvirt] [libvirt-php] Allow block devices in libvirt_domain_disk_add
by Martijn Otto 17 Apr '13

17 Apr '13

I have written a patch to the libvirt_domain_disk_add function which does the following things: 1. Allow adding block devices as disks with libvirt_domain_disk_add 2. Removed the xflags parameter (not relevant as no xml is returned anyways) 3. Added the cache option to control cache mode (either default, none, writeback or writethrough) 4. Added the io option to control the used io mode (either default, native or threads) Besides this, I have rewritten it so it now uses the virDomainAttachDevice function, which is meant for these things, instead of pasting the device XML by hand into the domain XML. I have uploaded this to my github clone at https://github.com/martijnotto/libvirt-php It would be nice if this could be included in a future release of lbvirt-php. If the code does not meet a coding standard for libvirt-php or if there is some other concern, please let me know and I will do what I can to improve it. With Regards, Martijn Otto Copernica B.V.

1 0

[libvirt] [PATCH v2] qemu: Implement CPUs check against machine type's cpu-max
by Michal Novotny 17 Apr '13

17 Apr '13

Implement check whether (maximum) vCPUs doesn't exceed machine type's cpu-max settings. Signed-off-by: Michal Novotny <minovotn(a)redhat.com> --- src/conf/capabilities.h | 1 + src/qemu/qemu_capabilities.c | 40 +++++++++++++++++++++++++++++++++++++++- src/qemu/qemu_capabilities.h | 3 ++- src/qemu/qemu_monitor.h | 1 + src/qemu/qemu_monitor_json.c | 7 +++++++ src/qemu/qemu_process.c | 27 +++++++++++++++++++++++++++ 6 files changed, 77 insertions(+), 2 deletions(-) diff --git a/src/conf/capabilities.h b/src/conf/capabilities.h index abcf6de..16bf3de 100644 --- a/src/conf/capabilities.h +++ b/src/conf/capabilities.h @@ -46,6 +46,7 @@ typedef virCapsGuestMachine *virCapsGuestMachinePtr; struct _virCapsGuestMachine { char *name; char *canonical; + int cpu_max; }; typedef struct _virCapsGuestDomainInfo virCapsGuestDomainInfo; diff --git a/src/qemu/qemu_capabilities.c b/src/qemu/qemu_capabilities.c index d10c8aa..a03a643 100644 --- a/src/qemu/qemu_capabilities.c +++ b/src/qemu/qemu_capabilities.c @@ -243,6 +243,7 @@ struct _virQEMUCaps { size_t nmachineTypes; char **machineTypes; char **machineAliases; + int *machineMaxCpus; }; struct _virQEMUCapsCache { @@ -322,6 +323,7 @@ virQEMUCapsSetDefaultMachine(virQEMUCapsPtr qemuCaps, { char *name = qemuCaps->machineTypes[defIdx]; char *alias = qemuCaps->machineAliases[defIdx]; + int cpu_max = qemuCaps->machineMaxCpus[defIdx]; memmove(qemuCaps->machineTypes + 1, qemuCaps->machineTypes, @@ -329,8 +331,12 @@ virQEMUCapsSetDefaultMachine(virQEMUCapsPtr qemuCaps, memmove(qemuCaps->machineAliases + 1, qemuCaps->machineAliases, sizeof(qemuCaps->machineAliases[0]) * defIdx); + memmove(qemuCaps->machineMaxCpus + 1, + qemuCaps->machineMaxCpus, + sizeof(qemuCaps->machineMaxCpus[0]) * defIdx); qemuCaps->machineTypes[0] = name; qemuCaps->machineAliases[0] = alias; + qemuCaps->machineMaxCpus[0] = cpu_max; } /* Format is: @@ -377,7 +383,8 @@ virQEMUCapsParseMachineTypesStr(const char *output, } if (VIR_REALLOC_N(qemuCaps->machineTypes, qemuCaps->nmachineTypes + 1) < 0 || - VIR_REALLOC_N(qemuCaps->machineAliases, qemuCaps->nmachineTypes + 1) < 0) { + VIR_REALLOC_N(qemuCaps->machineAliases, qemuCaps->nmachineTypes + 1) < 0 || + VIR_REALLOC_N(qemuCaps->machineMaxCpus, qemuCaps->nmachineTypes + 1) < 0) { VIR_FREE(name); VIR_FREE(canonical); goto no_memory; @@ -390,6 +397,8 @@ virQEMUCapsParseMachineTypesStr(const char *output, qemuCaps->machineTypes[qemuCaps->nmachineTypes-1] = name; qemuCaps->machineAliases[qemuCaps->nmachineTypes-1] = NULL; } + /* Value 0 means "unknown" as it's not exposed by QEMU binary */ + qemuCaps->machineMaxCpus[qemuCaps->nmachineTypes-1] = 0; } while ((p = next)); @@ -1718,6 +1727,8 @@ virQEMUCapsPtr virQEMUCapsNewCopy(virQEMUCapsPtr qemuCaps) goto no_memory; if (VIR_ALLOC_N(ret->machineAliases, qemuCaps->nmachineTypes) < 0) goto no_memory; + if (VIR_ALLOC_N(ret->machineMaxCpus, qemuCaps->nmachineTypes) < 0) + goto no_memory; ret->nmachineTypes = qemuCaps->nmachineTypes; for (i = 0 ; i < qemuCaps->nmachineTypes ; i++) { if (!(ret->machineTypes[i] = strdup(qemuCaps->machineTypes[i]))) @@ -1725,6 +1736,7 @@ virQEMUCapsPtr virQEMUCapsNewCopy(virQEMUCapsPtr qemuCaps) if (qemuCaps->machineAliases[i] && !(ret->machineAliases[i] = strdup(qemuCaps->machineAliases[i]))) goto no_memory; + ret->machineMaxCpus[i] = qemuCaps->machineMaxCpus[i]; } return ret; @@ -1885,9 +1897,11 @@ int virQEMUCapsGetMachineTypesCaps(virQEMUCapsPtr qemuCaps, goto no_memory; if (!(mach->canonical = strdup(qemuCaps->machineTypes[i]))) goto no_memory; + mach->cpu_max = qemuCaps->machineMaxCpus[i]; } else { if (!(mach->name = strdup(qemuCaps->machineTypes[i]))) goto no_memory; + mach->cpu_max = qemuCaps->machineMaxCpus[i]; } (*machines)[i] = mach; } @@ -1923,6 +1937,25 @@ const char *virQEMUCapsGetCanonicalMachine(virQEMUCapsPtr qemuCaps, } +int virQEMUCapsGetMachineMaxCpus(virQEMUCapsPtr qemuCaps, + const char *name) +{ + size_t i; + + if (!name) + return 0; + + for (i = 0 ; i < qemuCaps->nmachineTypes ; i++) { + if (!qemuCaps->machineMaxCpus[i]) + continue; + if (STREQ(qemuCaps->machineTypes[i], name)) + return qemuCaps->machineMaxCpus[i]; + } + + return 0; +} + + static int virQEMUCapsProbeQMPCommands(virQEMUCapsPtr qemuCaps, qemuMonitorPtr mon) @@ -2073,6 +2106,10 @@ virQEMUCapsProbeQMPMachineTypes(virQEMUCapsPtr qemuCaps, virReportOOMError(); goto cleanup; } + if (VIR_ALLOC_N(qemuCaps->machineMaxCpus, nmachines) < 0) { + virReportOOMError(); + goto cleanup; + } for (i = 0 ; i < nmachines ; i++) { if (machines[i]->alias) { @@ -2087,6 +2124,7 @@ virQEMUCapsProbeQMPMachineTypes(virQEMUCapsPtr qemuCaps, } if (machines[i]->isDefault) defIdx = i; + qemuCaps->machineMaxCpus[i] = machines[i]->cpu_max; } qemuCaps->nmachineTypes = nmachines; diff --git a/src/qemu/qemu_capabilities.h b/src/qemu/qemu_capabilities.h index 728add5..8205ff7 100644 --- a/src/qemu/qemu_capabilities.h +++ b/src/qemu/qemu_capabilities.h @@ -224,7 +224,8 @@ size_t virQEMUCapsGetMachineTypes(virQEMUCapsPtr qemuCaps, char ***names); const char *virQEMUCapsGetCanonicalMachine(virQEMUCapsPtr qemuCaps, const char *name); - +int virQEMUCapsGetMachineMaxCpus(virQEMUCapsPtr qemuCaps, + const char *name); int virQEMUCapsGetMachineTypesCaps(virQEMUCapsPtr qemuCaps, size_t *nmachines, virCapsGuestMachinePtr **machines); diff --git a/src/qemu/qemu_monitor.h b/src/qemu/qemu_monitor.h index f39f009..c097953 100644 --- a/src/qemu/qemu_monitor.h +++ b/src/qemu/qemu_monitor.h @@ -650,6 +650,7 @@ struct _qemuMonitorMachineInfo { char *name; bool isDefault; char *alias; + int cpu_max; }; int qemuMonitorGetMachines(qemuMonitorPtr mon, diff --git a/src/qemu/qemu_monitor_json.c b/src/qemu/qemu_monitor_json.c index 6fdd650..adbd865 100644 --- a/src/qemu/qemu_monitor_json.c +++ b/src/qemu/qemu_monitor_json.c @@ -4024,6 +4024,13 @@ int qemuMonitorJSONGetMachines(qemuMonitorPtr mon, goto cleanup; } } + + if (virJSONValueObjectHasKey(child, "cpu-max") && + virJSONValueObjectGetNumberInt(child, "cpu-max", &info->cpu_max) < 0) { + virReportError(VIR_ERR_INTERNAL_ERROR, "%s", + _("query-machines reply has malformed 'cpu-max' data")); + goto cleanup; + } } ret = n; diff --git a/src/qemu/qemu_process.c b/src/qemu/qemu_process.c index ce9f501..9eae8f6 100644 --- a/src/qemu/qemu_process.c +++ b/src/qemu/qemu_process.c @@ -3266,6 +3266,30 @@ qemuSetUnprivSGIO(virDomainDiskDefPtr disk) return ret; } +static bool +qemuValidateCpuMax(virDomainDefPtr def, virQEMUCapsPtr qemuCaps) +{ + int cpu_max; + + cpu_max = virQEMUCapsGetMachineMaxCpus(qemuCaps, def->os.machine); + if (!cpu_max) + return true; + + if (def->vcpus > cpu_max) { + virReportError(VIR_ERR_CONFIG_UNSUPPORTED, + "%s", _("CPUs greater than specified machine type limit")); + return false; + } + + if (def->maxvcpus > cpu_max) { + virReportError(VIR_ERR_CONFIG_UNSUPPORTED, + "%s", _("Maximum CPUs greater than specified machine type limit")); + return false; + } + + return true; +} + int qemuProcessStart(virConnectPtr conn, virQEMUDriverPtr driver, virDomainObjPtr vm, @@ -3481,6 +3505,9 @@ int qemuProcessStart(virConnectPtr conn, vm->def->emulator))) goto cleanup; + if (!qemuValidateCpuMax(vm->def, priv->qemuCaps)) + goto cleanup; + if (qemuAssignDeviceAliases(vm->def, priv->qemuCaps) < 0) goto cleanup; -- 1.7.11.7

2 1

[libvirt] (no subject)
by David Scott 17 Apr '13

17 Apr '13

Hi, I've made an improved set of ocaml bindings for the libvirt event mechanism which I'm now reasonably happy with. Improvements since my first submission: 1. it's possible to deregister callbacks with 'deregister_any' 2. the callback to ocaml now re-acquires the heap lock *before* creating and registering OCaml values with the GC: the previous patches got this wrong and bad things happened 3. it's possible to add and remove timer callbacks: this is used in the example to run Gc.compact() often, to check for problems 4. I removed the symbol detection for bindings present in libvirt 0.9.1 Let me know what you think! Thanks, Dave

1 3

[libvirt] [PATCH 0/2] Add cpu-max to capabilities and implement checks
by Michal Novotny 17 Apr '13

17 Apr '13

Add cpu-max to capabilities XML output for newer QEMU and implement checks whether the number of vCPUs/maximum vCPUS doesn't exceed machine limit. For older versions of QEMU cpu-max field is not present so the cpu-max is set to 0 to disable checking and act the same as without this patch. Signed-off-by: Michal Novotny <minovotn(a)redhat.com> Michal Novotny (2): cpu-max: Implement cpu-max attribute into capabilities XML output qemu: Implement validation for max-cpu in XML docs/schemas/capability.rng | 5 +++++ src/conf/capabilities.c | 4 ++++ src/conf/capabilities.h | 1 + src/qemu/qemu_capabilities.c | 40 +++++++++++++++++++++++++++++++++++++++- src/qemu/qemu_capabilities.h | 3 ++- src/qemu/qemu_driver.c | 30 ++++++++++++++++++++++++++++++ src/qemu/qemu_monitor.h | 1 + src/qemu/qemu_monitor_json.c | 7 +++++++ 8 files changed, 89 insertions(+), 2 deletions(-) -- 1.7.11.7

2 4

[libvirt] patch request
by Thierry Parmentelat 17 Apr '13

17 Apr '13

Just in case this is useful, I'd like to submit this change below that was not present at least in 1.0.4 http://git.onelab.eu/?p=libvirt.git;a=blob;f=virgetlasterror.patch;h=9109d5… I'd taken this hint here in the first place http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=700077 -- Thierry

2 3

[libvirt] Google Summer of Code 2013 ideas wiki open
by Stefan Hajnoczi 16 Apr '13

16 Apr '13

I have created the Google Summer of Code 2013 wiki page where you can add project ideas: http://wiki.qemu.org/Google_Summer_of_Code_2013 Please add project ideas you are willing to mentor. If you have an idea but cannot mentor this year, feel free to add it but please try to find a mentor for it. If you want to be a mentor, please see http://wiki.qemu.org/Google_Summer_of_Code_2013#Information_for_mentors. Background on Google Summer of Code: Google Summer of Code is a scheme where Google funds university students to work on open source projects for 12 weeks during the summer. It has been held annually since 2005. GSoC is an opportunity for students to experience open source development. Organizations sometimes find long-term contributors, and either way it's a fun experience. Open source organizations can apply to take part. Those chosen will be able to take on one or more students for the summer. Each student is paired with a mentor who helps them and evaluates their progress. Last year QEMU was not accepted but we were successful in years before that. I believe Google will announce GSoC again this year (there is no guarantee though) and I have created the wiki page so we can begin organizing project ideas that students can choose from. This way we'll be ready to put together a good organization application. Stefan

6 54

[libvirt] [PATCH 0/6] cleanup: About including header
by Osier Yang 16 Apr '13

16 Apr '13

4 14

[libvirt] [PATCH 0/5] VirtualBox version 4.2 support for libvirt vbox driver
by ryan woodsmall 16 Apr '13

16 Apr '13

This patch set adds VirtualBox 4.2 initial support for the libvirt vbox driver. I've tested enough to check capabilities, create a VM, destroy it, etc. Five patches total: - Patch 1 is the C API header file from Oracle, cleaned up for libvirt. - Patch 2 is the version specific source file for version dependency. - Patch 3 is the src/Makefile.am change to pick up the two new files. - Patch 4 is the vbox driver support for the new VirtualBox API/version. - Patch 5 is the vbox_tmpl.c template support for the new version. A few things have changed in the VirtualBox API - some small (capitalizations of things in function names like Ip to IP and Dhcp to DHCP) and some much larger (FindMedium is superceded by OpenMedium). The biggest change for the sake of this patch set is the signature of CreateMachine is quite a bit different. Using the Oracle source as a guide, to spin up a VM with a given UUID, it looks like a text flag has to be passed in a new argument to CreateMachine. This flag is built in the VirtualBox 4.2 specific ifdefs and is kind of ugly but works. Additionally, there is now (unused) VM groups support in CreateMachine and the previous 'osTypeId' arg is currently set to nsnull as in the Oracle code. The FindMedium to OpenMedium changes were more straightforward and are pretty clear. The rest of the vbox template changes are basically spelling/capitalization changes from the looks of things. This probably isn't perfect, but it works on git and patched against 0.10.2 for a few quick tests. Not currently on the list, so ping me directly if you need any other info on these, or if anything could use additional work. Thanks! -r ryan woodsmall (5): vbox C API header for VirtualBox v4.2 vbox version-specific C file for VirtualBox v4.2 Makefile.am additions for VirtualBox v4.2 vbox driver support for VirtualBox v4.2 vbox template support for VirtualBox v4.2 src/Makefile.am | 3 +- src/vbox/vbox_CAPI_v4_2.h | 8855 +++++++++++++++++++++++++++++++++++++++++++++ src/vbox/vbox_V4_2.c | 13 + src/vbox/vbox_driver.c | 8 + src/vbox/vbox_tmpl.c | 90 +- 5 files changed, 8958 insertions(+), 11 deletions(-) create mode 100644 src/vbox/vbox_CAPI_v4_2.h create mode 100644 src/vbox/vbox_V4_2.c

4 9

[libvirt] [PATCH] Check for QMP query-tpm-models
by Stefan Berger 16 Apr '13

16 Apr '13

Check for QMP query-tpm-models and set a capability flag. Do not use this QMP command if it is not supported. Signed-off-by: Stefan Berger <stefanb(a)linux.vnet.ibm.com> --- src/qemu/qemu_capabilities.c | 8 ++++++++ src/qemu/qemu_capabilities.h | 1 + 2 files changed, 9 insertions(+) Index: libvirt/src/qemu/qemu_capabilities.h =================================================================== --- libvirt.orig/src/qemu/qemu_capabilities.h +++ libvirt/src/qemu/qemu_capabilities.h @@ -179,6 +179,7 @@ enum virQEMUCapsFlags { QEMU_CAPS_MACHINE_USB_OPT = 137, /* -machine xxx,usb=on/off */ QEMU_CAPS_DEVICE_TPM_PASSTHROUGH = 138, /* -tpmdev passthrough */ QEMU_CAPS_DEVICE_TPM_TIS = 139, /* -device tpm_tis */ + QEMU_CAPS_TPM_MODELS = 140, /* query-tpm-models QMP command */ QEMU_CAPS_LAST, /* this must always be the last item */ }; Index: libvirt/src/qemu/qemu_capabilities.c =================================================================== --- libvirt.orig/src/qemu/qemu_capabilities.c +++ libvirt/src/qemu/qemu_capabilities.c @@ -220,6 +220,8 @@ VIR_ENUM_IMPL(virQEMUCaps, QEMU_CAPS_LAS "machine-usb-opt", "tpm-passthrough", "tpm-tis", + + "query-tpm-models", /* 140 */ ); struct _virQEMUCaps { @@ -1962,6 +1964,8 @@ virQEMUCapsProbeQMPCommands(virQEMUCapsP virQEMUCapsSet(qemuCaps, QEMU_CAPS_ADD_FD); else if (STREQ(name, "nbd-server-start")) virQEMUCapsSet(qemuCaps, QEMU_CAPS_NBD_SERVER); + else if (STREQ(name, "query-tpm-models")) + virQEMUCapsSet(qemuCaps, QEMU_CAPS_TPM_MODELS); VIR_FREE(name); } VIR_FREE(commands); @@ -2144,6 +2148,10 @@ virQEMUCapsProbeQMPTPM(virQEMUCapsPtr qe { int nentries, i; char **entries = NULL; + + if (!virQEMUCapsGet(qemuCaps, QEMU_CAPS_TPM_MODELS)) + return 0; + if ((nentries = qemuMonitorGetTPMModels(mon, &entries)) < 0) return -1;

3 4

[libvirt] [PATCH v2] Fix compilation error in util/vircgroup.c
by Stefan Berger 16 Apr '13

16 Apr '13

Fix the error util/vircgroup.c: In function 'virCgroupNewDomainPartition': util/vircgroup.c:1299:11: error: declaration of 'dirname' shadows a global declaration [-Werror=shadow] Signed-off-by: Stefan Berger <stefanb(a)linux.vnet.ibm.com> --- src/util/vircgroup.c | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) Index: libvirt-acl/src/util/vircgroup.c =================================================================== --- libvirt-acl.orig/src/util/vircgroup.c +++ libvirt-acl/src/util/vircgroup.c @@ -1296,13 +1296,13 @@ int virCgroupNewDomainPartition(virCgrou virCgroupPtr *group) { int rc; - char *dirname = NULL; + char *grpname = NULL; - if (virAsprintf(&dirname, "%s.%s.libvirt", + if (virAsprintf(&grpname, "%s.%s.libvirt", name, driver) < 0) return -ENOMEM; - rc = virCgroupNew(dirname, partition, -1, group); + rc = virCgroupNew(grpname, partition, -1, group); if (rc == 0) { /* @@ -1322,7 +1322,7 @@ int virCgroupNewDomainPartition(virCgrou } } - VIR_FREE(dirname); + VIR_FREE(grpname); return rc; } #else

2 2

[libvirt] [PATCH] Fix build breaker with ATTRIBUTE_NONNULL defs
by John Ferlan 16 Apr '13

16 Apr '13

Using "./autogen.sh --system lv_cv_static_analysis=yes" for my daily Coverity builds resulted in the following error when building: In file included from util/vircgrouppriv.h:32:0, from util/vircgroup.c:44: util/vircgroup.h:59:5: error: nonnull argument with out-of-range operand number (argument 1, operand 5) util/vircgroup.h:74:5: error: nonnull argument references non-pointer operand (argument 1, operand 4) make[3]: *** [libvirt_util_la-vircgroup.lo] Error 1 make[3]: Leaving directory `/home/jferlan/libvirt.cov.curr/src' make[2]: *** [all] Error 2 make[2]: Leaving directory `/home/jferlan/libvirt.cov.curr/src' make[1]: *** [all-recursive] Error 1 make[1]: Leaving directory `/home/jferlan/libvirt.cov.curr' make: *** [all] Error 2 --- src/util/vircgroup.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/util/vircgroup.h b/src/util/vircgroup.h index f324fa3..b030e4a 100644 --- a/src/util/vircgroup.h +++ b/src/util/vircgroup.h @@ -54,11 +54,11 @@ int virCgroupNewPartition(const char *path, int virCgroupNewDriver(const char *name, bool create, int controllers, virCgroupPtr *group) - ATTRIBUTE_NONNULL(1) ATTRIBUTE_NONNULL(5); + ATTRIBUTE_NONNULL(1) ATTRIBUTE_NONNULL(4); int virCgroupNewSelf(virCgroupPtr *group) ATTRIBUTE_NONNULL(1); int virCgroupNewDomainDriver(virCgroupPtr driver, @@ -69,11 +69,11 @@ int virCgroupNewDomainDriver(virCgroupPtr driver, int virCgroupNewDomainPartition(virCgroupPtr partition, const char *driver, const char *name, bool create, virCgroupPtr *group) - ATTRIBUTE_NONNULL(1) ATTRIBUTE_NONNULL(2) ATTRIBUTE_NONNULL(4); + ATTRIBUTE_NONNULL(1) ATTRIBUTE_NONNULL(2) ATTRIBUTE_NONNULL(5); int virCgroupNewVcpu(virCgroupPtr domain, int vcpuid, bool create, virCgroupPtr *group) -- 1.8.1.4

2 1

[libvirt] [PATCH] Resolve valgrind error
by John Ferlan 16 Apr '13

16 Apr '13

Commit id '1acfc171' resulted in the following valgrind failure: ==25317== 136 (24 direct, 112 indirect) bytes in 1 blocks are definitely lost in loss record 4 of 4 ==25317== at 0x4A06B6F: calloc (vg_replace_malloc.c:593) ==25317== by 0x4C6F851: virAlloc (viralloc.c:124) ==25317== by 0x4C71493: virBitmapNew (virbitmap.c:74) ==25317== by 0x4C71B79: virBitmapNewData (virbitmap.c:434) ==25317== by 0x402EF2: test8 (virbitmaptest.c:436) ==25317== by 0x40499F: virtTestRun (testutils.c:157) ==25317== by 0x402E8D: mymain (virbitmaptest.c:474) ==25317== by 0x404FDA: virtTestMain (testutils.c:719) ==25317== by 0x39D0821A04: (below main) (in /usr/lib64/libc-2.16.so) --- tests/virbitmaptest.c | 15 ++++++++------- 1 file changed, 8 insertions(+), 7 deletions(-) diff --git a/tests/virbitmaptest.c b/tests/virbitmaptest.c index 90be0e2..2464c33 100644 --- a/tests/virbitmaptest.c +++ b/tests/virbitmaptest.c @@ -432,24 +432,25 @@ static int test8(const void *v ATTRIBUTE_UNUSED) { virBitmapPtr bitmap = NULL; char data[108] = {0x00,}; + int ret = -1; bitmap = virBitmapNewData(data, sizeof(data)); if (!bitmap) - goto error; + goto cleanup; if (!virBitmapIsAllClear(bitmap)) - goto error; + goto cleanup; if (virBitmapSetBit(bitmap, 11) < 0) - goto error; + goto cleanup; if (virBitmapIsAllClear(bitmap)) - goto error; + goto cleanup; - return 0; -error: + ret = 0; +cleanup: virBitmapFree(bitmap); - return -1; + return ret; } static int -- 1.8.1.4

2 2

[libvirt] [PATCH] Fix compilation error in util/vircgroup.c
by Stefan Berger 16 Apr '13

16 Apr '13

Fix the error util/vircgroup.c: In function 'virCgroupNewDomainPartition': util/vircgroup.c:1299:11: error: declaration of 'dirname' shadows a global declaration [-Werror=shadow] Signed-off-by: Stefan Berger <stefanb(a)linux.vnet.ibm.com> --- src/util/vircgroup.c | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) Index: libvirt-acl/src/util/vircgroup.c =================================================================== --- libvirt-acl.orig/src/util/vircgroup.c +++ libvirt-acl/src/util/vircgroup.c @@ -1296,13 +1296,13 @@ int virCgroupNewDomainPartition(virCgrou virCgroupPtr *group) { int rc; - char *dirname = NULL; + char *directoryname = NULL; - if (virAsprintf(&dirname, "%s.%s.libvirt", + if (virAsprintf(&directoryname, "%s.%s.libvirt", name, driver) < 0) return -ENOMEM; - rc = virCgroupNew(dirname, partition, -1, group); + rc = virCgroupNew(directoryname, partition, -1, group); if (rc == 0) { /* @@ -1322,7 +1322,7 @@ int virCgroupNewDomainPartition(virCgrou } } - VIR_FREE(dirname); + VIR_FREE(directoryname); return rc; } #else

2 1

[libvirt] [PATCH 0/2] Suppress log message when using unsupported TPM QMP commands
by Stefan Berger 16 Apr '13

16 Apr '13

Properly suppress log messages when using unsupported TPM QMP commands. Stefan

2 4

[libvirt] [libvirt-glib] Fix year in NEWS
by Christophe Fergeau 16 Apr '13

16 Apr '13

We switched from 2012 to 2013!! --- I've pushed this under the trivial rule. Christophe NEWS | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/NEWS b/NEWS index b5f0c84..b1f4a80 100644 --- a/NEWS +++ b/NEWS @@ -1,7 +1,7 @@ libvirt-glib News ================= -0.1.6 - Mar 18, 2012 +0.1.6 - Mar 18, 2013 ==================== - Prefer g_strlcpy over strncpy @@ -14,7 +14,7 @@ - Replace FSF address with URL in copyright headers - Add API for changing Spice compression parameters. -0.1.5 - Jan 14, 2012 +0.1.5 - Jan 14, 2013 ==================== - Add support for bridge interface types in guest XML -- 1.8.1.4

1 0

[libvirt] [PATCH 0/7] Add support for guest-based CPU-hotplug and unplug
by Peter Krempa 16 Apr '13

16 Apr '13

This is a initial support for guest-agent based cpu hotplug that was commited in upstream qemu and should be released in qemu 1.5.0. This patchset relies on the existing qemuDomainSetVcpusFlags API that is not really ideal for this purpose but is usable. I'm planing on adding a specific-cpu hotplug API later that will allow more control. Peter Krempa (7): lib: Fix docs about return value of virDomainGetVcpusFlags() virsh-domain: Refactor cmdVcpucount and fix output on inactive domains qemu_agent: Add support for appending arrays to commands qemu_agent: Introduce helpers for agent based CPU hot(un)plug Introduce flag VIR_DOMAIN_VCPU agent, for agent based CPU unlpug(shutdown). qemu: Implement request of vCPU state using the guest agent qemu: Implement support for VIR_DOMAIN_VCPU in qemuDomainSetVcpusFlags include/libvirt/libvirt.h.in | 1 + src/libvirt.c | 9 +- src/qemu/qemu_agent.c | 141 +++++++++++++++++++++ src/qemu/qemu_agent.h | 12 ++ src/qemu/qemu_driver.c | 181 ++++++++++++++++++++++++--- tools/virsh-domain.c | 283 +++++++++++++++++++++---------------------- tools/virsh.pod | 13 +- 7 files changed, 474 insertions(+), 166 deletions(-) -- 1.8.1.5

2 15

[libvirt] [PATCH v4 00/11] Add support for guests with TPM passthrough device
by Stefan Berger 16 Apr '13

16 Apr '13

Hello! The following set of patches adds support to libvirt for adding a TPM passthrough device to a QEMU guest. Support for this was recently accepted into QEMU. This set of patches borrows a lot from the recently added support for rng's. Regards, Stefan --- v3->v4: - followed tree to ce65b435 - followed comments on v3 v2->v3: - followed tree to 03122d7b - some nits I found fixed v1->v2: - Followed Daniel Berrange's comments (except for the one on de-consolidating the JSON monitor code)

2 28

[libvirt] [PATCH 1/2] qemu: Support discard for disk
by Osier Yang 16 Apr '13

16 Apr '13

QEMU introduced "discard" option for drive since commit a9384aff53, <...> @var{discard} is one of "ignore" (or "off") or "unmap" (or "on") and controls whether @dfn{discard} (also known as @dfn{trim} or @dfn{unmap}) requests are ignored or passed to the filesystem. Some machine types may not support discard requests. </...> This patch exposes the support in libvirt. --- docs/formatdomain.html.in | 8 +++++ docs/schemas/domaincommon.rng | 11 +++++++ src/conf/domain_conf.c | 18 +++++++++++ src/conf/domain_conf.h | 10 ++++++ src/libvirt_private.syms | 1 + src/qemu/qemu_capabilities.c | 3 ++ src/qemu/qemu_capabilities.h | 1 + src/qemu/qemu_command.c | 11 +++++++ .../qemuxml2argv-disk-discard.args | 8 +++++ .../qemuxml2argvdata/qemuxml2argv-disk-discard.xml | 36 ++++++++++++++++++++++ tests/qemuxml2argvtest.c | 3 ++ tests/qemuxml2xmltest.c | 2 ++ 12 files changed, 112 insertions(+) create mode 100644 tests/qemuxml2argvdata/qemuxml2argv-disk-discard.args create mode 100644 tests/qemuxml2argvdata/qemuxml2argv-disk-discard.xml diff --git a/docs/formatdomain.html.in b/docs/formatdomain.html.in index 0cc56d9..f1cee14 100644 --- a/docs/formatdomain.html.in +++ b/docs/formatdomain.html.in @@ -1676,6 +1676,14 @@ network. By default copy-on-read is off. Since 0.9.10 (QEMU and KVM only) </li> + <li> + The optional <code>discard</code> attribute controls whether + to discard (also known as "trim" or "unmap") requests are + ignored or passed to the filesystem. The value can be either + "on" (allow the discard request to be passed) or "off" (ingore + the discard request). + Since 1.0.5 (QEMU and KVM only) + </li> </ul> </dd> <dt><code>boot</code></dt> diff --git a/docs/schemas/domaincommon.rng b/docs/schemas/domaincommon.rng index 468c49c..4b71e16 100644 --- a/docs/schemas/domaincommon.rng +++ b/docs/schemas/domaincommon.rng @@ -1304,6 +1304,9 @@ <optional> <ref name="copy_on_read"/> </optional> + <optional> + <ref name="discard"/> + </optional> <empty/> </element> </define> @@ -1399,6 +1402,14 @@ <value>off</value> </choice> </attribute> + </define> + <define name="discard"> + <attribute name='discard'> + <choice> + <value>on</value> + <value>off</value> + </choice> + </attribute> </define> <define name="controller"> <element name="controller"> diff --git a/src/conf/domain_conf.c b/src/conf/domain_conf.c index 548368e..4925a80 100644 --- a/src/conf/domain_conf.c +++ b/src/conf/domain_conf.c @@ -728,6 +728,10 @@ VIR_ENUM_IMPL(virDomainTPMModel, VIR_DOMAIN_TPM_MODEL_LAST, VIR_ENUM_IMPL(virDomainTPMBackend, VIR_DOMAIN_TPM_TYPE_LAST, "passthrough") +VIR_ENUM_IMPL(virDomainDiskDiscard, VIR_DOMAIN_DISK_DISCARD_LAST, + "default", + "on", + "off") #define VIR_DOMAIN_XML_WRITE_FLAGS VIR_DOMAIN_XML_SECURE #define VIR_DOMAIN_XML_READ_FLAGS VIR_DOMAIN_XML_INACTIVE @@ -4324,6 +4328,7 @@ virDomainDiskDefParseXML(virDomainXMLOptionPtr xmlopt, char *wwn = NULL; char *vendor = NULL; char *product = NULL; + char *discard = NULL; int expected_secret_usage = -1; int auth_secret_usage = -1; @@ -4551,6 +4556,7 @@ virDomainDiskDefParseXML(virDomainXMLOptionPtr xmlopt, ioeventfd = virXMLPropString(cur, "ioeventfd"); event_idx = virXMLPropString(cur, "event_idx"); copy_on_read = virXMLPropString(cur, "copy_on_read"); + discard = virXMLPropString(cur, "discard"); } else if (!mirror && xmlStrEqual(cur->name, BAD_CAST "mirror") && !(flags & VIR_DOMAIN_XML_INACTIVE)) { char *ready; @@ -4997,6 +5003,14 @@ virDomainDiskDefParseXML(virDomainXMLOptionPtr xmlopt, def->copy_on_read = cor; } + if (discard) { + if ((def->discard = virDomainDiskDiscardTypeFromString(discard)) <= 0) { + virReportError(VIR_ERR_CONFIG_UNSUPPORTED, + _("unknown disk discard mode '%s'"), discard); + goto error; + } + } + if (devaddr) { if (virDomainParseLegacyDeviceAddress(devaddr, &def->info.addr.pci) < 0) { @@ -5116,6 +5130,7 @@ cleanup: VIR_FREE(ioeventfd); VIR_FREE(event_idx); VIR_FREE(copy_on_read); + VIR_FREE(discard); VIR_FREE(devaddr); VIR_FREE(serial); virStorageEncryptionFree(encryption); @@ -13291,6 +13306,7 @@ virDomainDiskDefFormat(virBufferPtr buf, const char *event_idx = virDomainVirtioEventIdxTypeToString(def->event_idx); const char *copy_on_read = virDomainVirtioEventIdxTypeToString(def->copy_on_read); const char *sgio = virDomainDiskSGIOTypeToString(def->sgio); + const char *discard = virDomainDiskDiscardTypeToString(def->discard); char uuidstr[VIR_UUID_STRING_BUFLEN]; @@ -13367,6 +13383,8 @@ virDomainDiskDefFormat(virBufferPtr buf, virBufferAsprintf(buf, " event_idx='%s'", event_idx); if (def->copy_on_read) virBufferAsprintf(buf, " copy_on_read='%s'", copy_on_read); + if (def->discard) + virBufferAsprintf(buf, " discard='%s'", discard); virBufferAddLit(buf, "/>\n"); } diff --git a/src/conf/domain_conf.h b/src/conf/domain_conf.h index f1f01fa..fdf5d93 100644 --- a/src/conf/domain_conf.h +++ b/src/conf/domain_conf.h @@ -600,6 +600,14 @@ enum virDomainDiskSGIO { VIR_DOMAIN_DISK_SGIO_LAST }; +enum virDomainDiskDiscard { + VIR_DOMAIN_DISK_DISCARD_DEFAULT = 0, + VIR_DOMAIN_DISK_DISCARD_ON, + VIR_DOMAIN_DISK_DISCARD_OFF, + + VIR_DOMAIN_DISK_DISCARD_LAST +}; + typedef struct _virDomainBlockIoTuneInfo virDomainBlockIoTuneInfo; struct _virDomainBlockIoTuneInfo { unsigned long long total_bytes_sec; @@ -682,6 +690,7 @@ struct _virDomainDiskDef { bool rawio_specified; int rawio; /* no = 0, yes = 1 */ int sgio; /* enum virDomainDiskSGIO */ + int discard; /* enum virDomainDiskDiscard */ size_t nseclabels; virSecurityDeviceLabelDefPtr *seclabels; @@ -2416,6 +2425,7 @@ VIR_ENUM_DECL(virDomainDiskIo) VIR_ENUM_DECL(virDomainDiskSecretType) VIR_ENUM_DECL(virDomainDiskSGIO) VIR_ENUM_DECL(virDomainDiskTray) +VIR_ENUM_DECL(virDomainDiskDiscard) VIR_ENUM_DECL(virDomainIoEventFd) VIR_ENUM_DECL(virDomainVirtioEventIdx) VIR_ENUM_DECL(virDomainDiskCopyOnRead) diff --git a/src/libvirt_private.syms b/src/libvirt_private.syms index 30fdcd7..a24bf5c 100644 --- a/src/libvirt_private.syms +++ b/src/libvirt_private.syms @@ -140,6 +140,7 @@ virDomainDiskDefFree; virDomainDiskDefGenSecurityLabelDef; virDomainDiskDefGetSecurityLabelDef; virDomainDiskDeviceTypeToString; +virDomainDiskDiscardTypeToString; virDomainDiskErrorPolicyTypeFromString; virDomainDiskErrorPolicyTypeToString; virDomainDiskFindByBusAndDst; diff --git a/src/qemu/qemu_capabilities.c b/src/qemu/qemu_capabilities.c index b174672..278012c 100644 --- a/src/qemu/qemu_capabilities.c +++ b/src/qemu/qemu_capabilities.c @@ -222,6 +222,7 @@ VIR_ENUM_IMPL(virQEMUCaps, QEMU_CAPS_LAST, "tpm-tis", "query-tpm-models", /* 140 */ + "drive-discard", ); struct _virQEMUCaps { @@ -963,6 +964,8 @@ virQEMUCapsComputeCmdFlags(const char *help, virQEMUCapsSet(qemuCaps, QEMU_CAPS_DRIVE_COPY_ON_READ); if (strstr(help, "bps=")) virQEMUCapsSet(qemuCaps, QEMU_CAPS_DRIVE_IOTUNE); + if (strstr(help, "discard")) + virQEMUCapsSet(qemuCaps, QEMU_CAPS_DRIVE_DISCARD); } if ((p = strstr(help, "-vga")) && !strstr(help, "-std-vga")) { const char *nl = strstr(p, "\n"); diff --git a/src/qemu/qemu_capabilities.h b/src/qemu/qemu_capabilities.h index 7a0f191..17c5335 100644 --- a/src/qemu/qemu_capabilities.h +++ b/src/qemu/qemu_capabilities.h @@ -180,6 +180,7 @@ enum virQEMUCapsFlags { QEMU_CAPS_DEVICE_TPM_PASSTHROUGH = 138, /* -tpmdev passthrough */ QEMU_CAPS_DEVICE_TPM_TIS = 139, /* -device tpm_tis */ QEMU_CAPS_TPM_MODELS = 140, /* query-tpm-models QMP command */ + QEMU_CAPS_DRIVE_DISCARD = 141, /* -drive discard=off(ignore)|on(unmap) */ QEMU_CAPS_LAST, /* this must always be the last item */ }; diff --git a/src/qemu/qemu_command.c b/src/qemu/qemu_command.c index 63b9350..36f20eb 100644 --- a/src/qemu/qemu_command.c +++ b/src/qemu/qemu_command.c @@ -2987,6 +2987,17 @@ qemuBuildDriveStr(virConnectPtr conn ATTRIBUTE_UNUSED, } } + if (disk->discard) { + if (virQEMUCapsGet(qemuCaps, QEMU_CAPS_DRIVE_DISCARD)) { + virBufferAsprintf(&opt, ",discard=%s", + virDomainDiskDiscardTypeToString(disk->discard)); + } else { + virReportError(VIR_ERR_CONFIG_UNSUPPORTED, "%s", + _("discard is not supported by this QEMU binary")); + goto error; + } + } + if (virQEMUCapsGet(qemuCaps, QEMU_CAPS_MONITOR_JSON)) { const char *wpolicy = NULL, *rpolicy = NULL; diff --git a/tests/qemuxml2argvdata/qemuxml2argv-disk-discard.args b/tests/qemuxml2argvdata/qemuxml2argv-disk-discard.args new file mode 100644 index 0000000..02e2ddb --- /dev/null +++ b/tests/qemuxml2argvdata/qemuxml2argv-disk-discard.args @@ -0,0 +1,8 @@ +LC_ALL=C PATH=/bin HOME=/home/test USER=test LOGNAME=test \ +/usr/bin/qemu -S -M pc-0.13 -m 1024 -smp 1 -nographic -nodefaults \ +-monitor unix:/tmp/test-monitor,server,nowait -no-acpi -boot dc -usb \ +-drive file=/var/lib/libvirt/images/f14.img,if=none,id=drive-virtio-disk0,discard=on \ +-device virtio-blk-pci,bus=pci.0,addr=0x4,drive=drive-virtio-disk0,id=virtio-disk0 \ +-drive file=/var/lib/libvirt/Fedora-14-x86_64-Live-KDE.iso,if=none,media=cdrom,id=drive-ide0-1-0,discard=off \ +-device ide-drive,bus=ide.1,unit=0,drive=drive-ide0-1-0,id=ide0-1-0 \ +-device virtio-balloon-pci,id=balloon0,bus=pci.0,addr=0x3 diff --git a/tests/qemuxml2argvdata/qemuxml2argv-disk-discard.xml b/tests/qemuxml2argvdata/qemuxml2argv-disk-discard.xml new file mode 100644 index 0000000..fa38a2d --- /dev/null +++ b/tests/qemuxml2argvdata/qemuxml2argv-disk-discard.xml @@ -0,0 +1,36 @@ +<domain type='qemu'> + <name>test</name> + <uuid>92d7a226-cfae-425b-a6d3-00bbf9ec5c9e</uuid> + <memory unit='KiB'>1048576</memory> + <currentMemory unit='KiB'>1048576</currentMemory> + <vcpu placement='static'>1</vcpu> + <os> + <type arch='x86_64' machine='pc-0.13'>hvm</type> + <boot dev='cdrom'/> + <boot dev='hd'/> + <bootmenu enable='yes'/> + </os> + <clock offset='utc'/> + <on_poweroff>destroy</on_poweroff> + <on_reboot>restart</on_reboot> + <on_crash>restart</on_crash> + <devices> + <emulator>/usr/bin/qemu</emulator> + <disk type='file' device='disk'> + <driver name='qemu' type='qcow2' discard='on'/> + <source file='/var/lib/libvirt/images/f14.img'/> + <target dev='vda' bus='virtio'/> + <address type='pci' domain='0x0000' bus='0x00' slot='0x04' function='0x0'/> + </disk> + <disk type='file' device='cdrom'> + <driver name='qemu' type='raw' discard='off'/> + <source file='/var/lib/libvirt/Fedora-14-x86_64-Live-KDE.iso'/> + <target dev='hdc' bus='ide'/> + <readonly/> + <address type='drive' controller='0' bus='1' target='0' unit='0'/> + </disk> + <controller type='usb' index='0'/> + <controller type='ide' index='0'/> + <memballoon model='virtio'/> + </devices> +</domain> diff --git a/tests/qemuxml2argvtest.c b/tests/qemuxml2argvtest.c index be6c759..e09e070 100644 --- a/tests/qemuxml2argvtest.c +++ b/tests/qemuxml2argvtest.c @@ -575,6 +575,9 @@ mymain(void) QEMU_CAPS_DRIVE, QEMU_CAPS_DRIVE_COPY_ON_READ, QEMU_CAPS_VIRTIO_TX_ALG, QEMU_CAPS_DEVICE, QEMU_CAPS_VIRTIO_BLK_SCSI, QEMU_CAPS_VIRTIO_BLK_SG_IO); + DO_TEST("disk-discard", + QEMU_CAPS_DRIVE, QEMU_CAPS_DRIVE_DISCARD, + QEMU_CAPS_DEVICE); DO_TEST("disk-snapshot", QEMU_CAPS_DRIVE, QEMU_CAPS_DRIVE_CACHE_V2, QEMU_CAPS_DRIVE_FORMAT); DO_TEST("event_idx", diff --git a/tests/qemuxml2xmltest.c b/tests/qemuxml2xmltest.c index 3a1683f..7f32661 100644 --- a/tests/qemuxml2xmltest.c +++ b/tests/qemuxml2xmltest.c @@ -255,6 +255,8 @@ mymain(void) DO_TEST("disk-scsi-disk-vpd"); DO_TEST("disk-source-pool"); + DO_TEST("disk-discard"); + DO_TEST("virtio-rng-random"); DO_TEST("virtio-rng-egd"); -- 1.8.1.4

1 1

[libvirt] [Concern] Whether some clock element stability need to be check before migration?
by Xuesong Zhang 16 Apr '13

16 Apr '13

hi, This is xuesong from libvirt-QE, which is the feature owner of stable guest ABI in rhel7. I met one issue, but didn't know whether it is one bug or not, since in Guest ABI , some aspects belong to "a host-side configuration parameter". So, would you please kindly help to check it? Thanks. Description While check the guest clock ABI stability, some scenario fails. version kernel-3.8.0-0.43.el7.x86_64 qemu-kvm-1.3.0-6.el7.x86_64 libvirt-1.0.3-1.el7.x86_64 reproduced 100% Steps 1. prepare migration env on two hosts. 2. prepare one guest with the parallel. ...... <clock offset='localtime'> <timer name='rtc' tickpolicy='catchup' track='guest'> <catchup threshold='123' slew='120' limit='10000'/> </timer> <timer name='pit' tickpolicy='delay'/> </clock> .... 3. start and dumpxml this gust # virsh start mig Domain mig started # virsh dumpxml mig > mig.xml 4. Change "localtime" to "utc" in the generated xml "mig.xml". ...... <clock offset='utc'> <timer name='rtc' tickpolicy='catchup' track='guest'> <catchup threshold='123' slew='120' limit='10000'/> </timer> <timer name='pit' tickpolicy='delay'/> </clock> .... 5. migration with the modified xml is successfully.------------------it is issue, should prompt error message. # virsh migrate --live --xml mig.xml mig qemu+ssh://{$target host IP}/system 6. run step3 again, then change timer name "pit" to "platform". Note, maker sure only one change point in the xml. then migration with the modified xml. ----------------this result is as expect # virsh migrate mig --live --xml mig.xml qemu+ssh://10.66.83.38/system error: unsupported configuration: Target timer platform does not match source pit 7. run step3 again, then delete timer name "pit". Note, maker sure only one change point in the xml. then migration with the modified xml.-----------------the result is as expect #virsh migrate mig --live --xml mig.xml qemu+ssh://10.66.83.38/system error: unsupported configuration: Target domain timers do not match source 8. run step3 again, then change timer track from "guest" to "wall". Note, maker sure only one change point in the xml. then migration with the modified xml. The migration is succssful.------------------------------------------------------------------------------------------------------------------it is issue, should prompt error message. # virsh migrate --live --xml mig.xml mig qemu+ssh://{$target host IP}/system 9. run step3 again, then change timer tickpolicy from "catchup" to "delay". Note, maker sure only one change point in the xml. then migration with the modified xml. The migration is succssful.------------------------------------------------------------------------------------------------------------------it is issue, should prompt error message. # virsh migrate --live --xml mig.xml mig qemu+ssh://{$target host IP}/system Actual result The result in step 6 and 7 is as expected, the result of step 5, 8 and 9 is wrong. Expect result The clock element stability in step 5, 8 and 9 need to be check before migration. Addtional info: Below is all checks about clock in upstream 1.0.4 maint branch: In timer check Function virDomainTimerDefCheckABIStability only check 1.NAME :src->name != dst->name 2.PRESENT :src->present != dst->present) 3.FREQUENCY :src->name == VIR_DOMAIN_TIMER_NAME_TSC) { if (src->frequency != dst->frequency) } 4.MODE src->mode != dst->mode And in another function , check timer's numbers 5.NUMBERS : src->clock.ntimers != dst->clock.ntimers Best Regards, Zhang Xuesong IRC: xuzhang internal phone: 88393

1 0

[libvirt] [PATCH 0/2] Check domain status before virsh suspend or resume
by John Ferlan 16 Apr '13

16 Apr '13

https://bugzilla.redhat.com/show_bug.cgi?id=906644 Added checks to both virsh suspend and virsh resume for the domain to be in a the right state before trying the suspend/resume. Similar checks to examples/domsuspend/suspend.c. Although not stated in case, any thoughts on adding a --force for both? John Ferlan (2): virsh: Add check for domain state before attempting resume virsh: Add check for domain state before attempting suspend tools/virsh-domain.c | 40 +++++++++++++++++++++++++++++++--------- 1 file changed, 31 insertions(+), 9 deletions(-) -- 1.8.1.4

3 5

[libvirt] [PATCH] Add missing param to virCgroupForDriver stub
by Daniel P. Berrange 16 Apr '13

16 Apr '13

From: "Daniel P. Berrange" <berrange(a)redhat.com> The virCgroupForDriver method recently gained an 'int controllers' parameter, but the stub impl did not Signed-off-by: Daniel P. Berrange <berrange(a)redhat.com> Pushed as a Win32 build breaker fix --- src/util/vircgroup.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/src/util/vircgroup.c b/src/util/vircgroup.c index dc2b431..56d56f9 100644 --- a/src/util/vircgroup.c +++ b/src/util/vircgroup.c @@ -984,7 +984,8 @@ out: int virCgroupForDriver(const char *name ATTRIBUTE_UNUSED, virCgroupPtr *group ATTRIBUTE_UNUSED, bool privileged ATTRIBUTE_UNUSED, - bool create ATTRIBUTE_UNUSED) + bool create ATTRIBUTE_UNUSED, + int controllers ATTRIBUTE_UNUSED) { /* Claim no support */ return -ENXIO; -- 1.8.1.4

1 0

[libvirt] [PATCH] Conditionally compile storagevolxml2argvtest
by Daniel P. Berrange 16 Apr '13

16 Apr '13

From: "Daniel P. Berrange" <berrange(a)redhat.com> Only compile storagevolxml2argvtest if WITH_STORAGE is set, because it links to that driver Signed-off-by: Daniel P. Berrange <berrange(a)redhat.com> Pushed as a Win32 build breaker fix --- tests/Makefile.am | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/tests/Makefile.am b/tests/Makefile.am index 888968d..7689453 100644 --- a/tests/Makefile.am +++ b/tests/Makefile.am @@ -174,7 +174,9 @@ endif test_programs += nwfilterxml2xmltest +if WITH_STORAGE test_programs += storagevolxml2argvtest +endif test_programs += storagevolxml2xmltest storagepoolxml2xmltest @@ -472,11 +474,15 @@ nwfilterxml2xmltest_SOURCES = \ testutils.c testutils.h nwfilterxml2xmltest_LDADD = $(LDADDS) +if WITH_STORAGE storagevolxml2argvtest_SOURCES = \ storagevolxml2argvtest.c \ testutils.c testutils.h storagevolxml2argvtest_LDADD = \ ../src/libvirt_driver_storage_impl.la $(LDADDS) +else +EXTRA_DIST += storagevolxml2argvtest.c +endif storagevolxml2xmltest_SOURCES = \ storagevolxml2xmltest.c \ -- 1.8.1.4

1 0

[libvirt] [PATCH] Conditionalize use of symlink() function in test suite
by Daniel P. Berrange 16 Apr '13

16 Apr '13

From: "Daniel P. Berrange" <berrange(a)redhat.com> On Win32 symlink() is not available, so virstoragetest.c must be conditionalized to avoid compile failures. Signed-off-by: Daniel P. Berrange <berrange(a)redhat.com> Pushed as a win32 build breaker fix --- configure.ac | 2 +- tests/virstoragetest.c | 6 ++++++ 2 files changed, 7 insertions(+), 1 deletion(-) diff --git a/configure.ac b/configure.ac index 11b332f..789d1c9 100644 --- a/configure.ac +++ b/configure.ac @@ -194,7 +194,7 @@ dnl Availability of various common functions (non-fatal if missing), dnl and various less common threadsafe functions AC_CHECK_FUNCS_ONCE([cfmakeraw geteuid getgid getgrnam_r getmntent_r \ getpwuid_r getuid initgroups kill mmap newlocale posix_fallocate \ - posix_memalign regexec sched_getaffinity setns]) + posix_memalign regexec sched_getaffinity setns symlink]) dnl Availability of pthread functions (if missing, win32 threading is dnl assumed). Because of $LIB_PTHREAD, we cannot use AC_CHECK_FUNCS_ONCE. diff --git a/tests/virstoragetest.c b/tests/virstoragetest.c index d495e6a..0e6ba9e 100644 --- a/tests/virstoragetest.c +++ b/tests/virstoragetest.c @@ -161,12 +161,14 @@ testPrepImages(void) if (virCommandRun(cmd, NULL) < 0) goto skip; +#ifdef HAVE_SYMLINK /* Create some symlinks in a sub-directory. */ if (symlink("../qcow2", datadir "/sub/link1") < 0 || symlink("../wrap", datadir "/sub/link2") < 0) { fprintf(stderr, "unable to create symlink"); goto cleanup; } +#endif ret = 0; cleanup: @@ -364,6 +366,7 @@ mymain(void) canonraw, absraw, datadir, VIR_STORAGE_FILE_RAW, true, 1024, false, }; +#if HAVE_SYMLINK const testFileData link1_rel = { canonraw, "../raw", "sub/../sub/..", VIR_STORAGE_FILE_RAW, true, 1024, false, @@ -380,6 +383,7 @@ mymain(void) canonqcow2, "../sub/link1", datadir "/sub/../sub", VIR_STORAGE_FILE_QCOW2, true, 1024, false, }; +#endif /* The actual tests, in several groups. */ @@ -522,6 +526,7 @@ mymain(void) chain12a, EXP_PASS, chain12b, ALLOW_PROBE | EXP_PASS); +#ifdef HAVE_SYMLINK /* Rewrite qcow2 and wrap file to use backing names relative to a * symlink from a different directory */ virCommandFree(cmd); @@ -545,6 +550,7 @@ mymain(void) chain13a, ALLOW_PROBE | EXP_PASS, chain13c, EXP_PASS, chain13c, ALLOW_PROBE | EXP_PASS); +#endif /* Final cleanup */ testCleanupImages(); -- 1.8.1.4

1 0

[libvirt] [PATCH] Fix signature of dummy virNetlinkCommand stub
by Daniel P. Berrange 16 Apr '13

16 Apr '13

From: "Daniel P. Berrange" <berrange(a)redhat.com> The second param of virNetlinkCommand should be struct nlmsghdr, not unsigned char. Signed-off-by: Daniel P. Berrange <berrange(a)redhat.com> Pushed as a Win32 build breaker fix --- src/util/virnetlink.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/util/virnetlink.c b/src/util/virnetlink.c index af1985c..d340eda 100644 --- a/src/util/virnetlink.c +++ b/src/util/virnetlink.c @@ -752,7 +752,7 @@ virNetlinkShutdown(void) } int virNetlinkCommand(struct nl_msg *nl_msg ATTRIBUTE_UNUSED, - unsigned char **respbuf ATTRIBUTE_UNUSED, + struct nlmsghdr **resp ATTRIBUTE_UNUSED, unsigned int *respbuflen ATTRIBUTE_UNUSED, uint32_t src_pid ATTRIBUTE_UNUSED, uint32_t dst_pid ATTRIBUTE_UNUSED, -- 1.8.1.4

1 0