- Devel - libvirt List Archives

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

19 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> + <p> + 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. + </p> + <p> + Example: usage of NVRAM configuration + </p> +<pre> + ... + <devices> + <nvram> + <address type='spapr-vio' reg='0x3000'/> + </nvram> + </devices> + ... +</pre> + <dl> + <dt><code>spapr-vio</code></dt> + <dd> + <p> + VIO device address type, this is only for PPC64. + </p> + </dd> + <dt><code>reg</code></dt> + <dd> + <p> + Devices' address + </p> + </dd> + </dl> + <h3><a name="seclabel">Security label</a></h3> <p> 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

4 7

[libvirt] QEMU/KVM support for non zero PCI domain
by Charles Peri 19 Apr '13

19 Apr '13

Hi, I am trying to run Virt-Manager, using KVM on a SGIUV100 system with RHEL 6.1. I am trying to enable PCI pass-thorugh for the Mellanox, but the PCI domain for the device is 0001. 0001:01:00.0 Ethernet controller: Mellanox Technologies MT26448 [ConnectX EN 10GigE, PCIe 2.0 5GT/s] (rev b0) 0001:02:00.0 InfiniBand: Mellanox Technologies MT26428 [ConnectX VPI PCIe 2.0 5GT/s - IB QDR / 10GigE] (rev b0) When I try to add the device, I get an error stating that "Can't assign device inside non- zero PCI segment, as this KVM module doesn't support it". Is there a version of KVM module that supports non zero PCI segment? Sorry, if this is not the right email list to ask this question, I checked on the KVM IRC, but haven't got any answer. Thanks, Charles

2 5

[libvirt] [PATCHv3] Support for static routes on a virtual bridge
by Gene Czarcinski 19 Apr '13

19 Apr '13

This patch adds support for adding a static route for a network. The "gateway" sub-element specifies the gateway's IP address. Both IPv4 and IPv6 static routes are supported although it is expected that this functionality will have more use with IPv6. This updates add the <route> element to define a static route. The gateway= subelement species the gateway address which is to receive the packets forwarded to the network specified on the <route> definition. Tests are done to validate that the input definitions are correct. For example, for a static route ip definition, the address must be a network address and not a host address. Additional checks are added to ensure that the specified gateway has a network defined on this bridge. Whan a static route is added to a bridge, there is a slight possibility that the gateway address will be incorrect. If this is handled as an error, that bridge becomes unusable and can only be recovered by rebooting. If the error is ignored, then that network can be destroyed and the network definition file edited to correct the problem. Unfortunately, the error message only appears in syslog. However, with the checks performed when the network definition file is parsed, it is unlikely that this condition will ever occur. The command used is of the following form: ip route add <address>/<prefix> via <gateway> dev <virbr-bridge> \ proto static metric 1 . Signed-off-by: Gene Czarcinski <gene(a)czarc.net> --- docs/formatnetwork.html.in | 90 ++++++ docs/schemas/network.rng | 22 ++ src/conf/network_conf.c | 329 ++++++++++++++++++++- src/conf/network_conf.h | 8 + src/libvirt_private.syms | 1 + src/network/bridge_driver.c | 40 +++ src/util/virnetdev.c | 46 +++ src/util/virnetdev.h | 5 + .../networkxml2xmlin/dhcp6host-routed-network.xml | 4 + .../networkxml2xmlout/dhcp6host-routed-network.xml | 4 + 10 files changed, 548 insertions(+), 1 deletion(-) diff --git a/docs/formatnetwork.html.in b/docs/formatnetwork.html.in index 4dd0415..9f9304a 100644 --- a/docs/formatnetwork.html.in +++ b/docs/formatnetwork.html.in @@ -529,6 +529,63 @@ starting. </p> + <h5><a name="elementsStaticroute">Static Routes</a></h5> + <p> + It has been possible to define static routes on a virtual bridge + device <span class="since">Since 1.0.5</span>. Static route + definitions are used to provide routing information to the + virtualization host for networks which are not defined as a + network on one of the virtual bridges on the virtualization host. + </p> + + <p> + As shown in this example <a href="formatnetwork.html#examplesNoGateway">here</a>, + it is possible to define a virtual bridge interface with no + IPv4 or IPv6 networks. Such interfaces are useful in supporting + networks which have no visibility or direct connectivity with the + virtualization host. However, such interfaces can be used to support + virtual networks which only have guest connectivity. A guest + with connectivity to the internal-only network and another network + with with regular connectivity can act as a gateway between the + networks. A static route is used provide the routing information + so that IP packets (and especially response packets) can be sent from + the virtualization host to guests on the hidden network. While + static routes can be useful with IPv4 networks, they are essential + in supporting IPv6 on seconday/hidden networks. + </p> + + <p> + Here is a fragment of a definition which shows the static + route specification as well as the IPv4 and IPv6 definitions + for network addresses which are referred to in the + <code>gateway</code> gateway address specifications. + </p> + + <pre> + ... + <ip address="192.168.122.1" netmask="255.255.255.0"> + <dhcp> + <range start="192.168.122.128" end="192.168.122.254" /> + </dhcp> + </ip> + <route address="192.168.222.0" prefix="24" gateway="192.168.122.2"> + </route> + <ip family="ipv6" address="2001:db8:ca2:2::1" prefix="64" /> + <route family="ipv6" address="2001:db8:ca2:3::" prefix="64" gateway="2001:db8:ca2:2::2"> + </route> + ... + </pre> + + <p> + Notice that the <code>route</code> elements look very similar to + the <code>ip</code> address specification. Although parts are the same, + the meaning and effects of the two definitions are very different. The + <code>ip</code> address specification results in an address being defined + on the virtual bridge whereas the <code>route</code> definition is used + to create the static route for the specified network via the gateway address + on the virtual bridge device. + </p> + <h3><a name="elementsAddress">Addressing</a></h3> <p> @@ -540,6 +597,13 @@ a forward mode of 'route' or 'nat'. </p> + <p> + Another optional element can add a static route definition to + the bridge device. The static route is specified by the + <code>route</code> element with <code>gateway</code> used to + specify the gateway. <span class="since">Since 1.0.5</span> + </p> + <pre> ... <mac address='00:16:3E:5D:C7:9E'/> @@ -560,6 +624,7 @@ </dhcp> </ip> <ip family="ipv6" address="2001:db8:ca2:2::1" prefix="64" /> + <route family="ipv6" address="2001:db9:ca1:1::" prefix="64" gateway="2001:db8:ca2:2::2" /> </network></pre> <dl> @@ -809,6 +874,31 @@ </ip> </network></pre> + <p> + Below is yet another IPv6 variation. This variation has only IPv6 + defined with DHCPv6 on the primary IPv6 network. A static link + if defined for a second IPv6 network which will not be visable on + the bridge interface but rill have a static route defined for this + network via the specified gateway. Note that the gateway address + must be valid for the networks supported on the bridge interface. + <span class="since">Since 1.0.5</span> + </p> + + <pre> + <network> + <name>net7</name> + <bridge name="virbr7" /> + <forward mode="route"/> + <ip family="ipv6" address="2001:db8:ca2:7::1" prefix="64" > + <dhcp> + <range start="2001:db8:ca2:7::100" end="2001:db8:ca2::1ff" /> + <host id="0:4:7e:7d:f0:7d:a8:bc:c5:d2:13:32:11:ed:16:ea:84:63" name="lucas" ip="2001:db8:ca2:2:3::4" /> + </dhcp> + </ip> + <route family="ipv6" address="2001:db8:ca2:8::" prefix="64" gateway="2001:db8:ca2:7::4" > + </route> + </network></pre> + <h3><a name="examplesPrivate">Isolated network config</a></h3> <p> diff --git a/docs/schemas/network.rng b/docs/schemas/network.rng index 6c3fae2..5bcba32 100644 --- a/docs/schemas/network.rng +++ b/docs/schemas/network.rng @@ -305,6 +305,28 @@ </optional> </element> </zeroOrMore> +  + <zeroOrMore> +  + <element name="route"> + <optional> + <attribute name="family"><ref name="addr-family"/></attribute> + </optional> + <optional> + <attribute name="address"><ref name="ipAddr"/></attribute> + </optional> + <optional> + <choice> + <attribute name="netmask"><ref name="ipv4Addr"/></attribute> + <attribute name="prefix"><ref name="ipPrefix"/></attribute> + </choice> + </optional> + <optional> + <attribute name="gateway"><ref name="ipAddr"/></attribute> + </optional> + </element> + </zeroOrMore> </interleave> </element> </define> diff --git a/src/conf/network_conf.c b/src/conf/network_conf.c index c5535e6..15e0ed0 100644 --- a/src/conf/network_conf.c +++ b/src/conf/network_conf.c @@ -1280,6 +1280,208 @@ cleanup: } static int +virNetworkRouteDefParseXML(const char *networkName, + xmlNodePtr node, + xmlXPathContextPtr ctxt, + virNetworkIpDefPtr def) +{ + /* + * virNetworkRouteDef (reall a virNetworkIpDef) object is already + * allocated as part of an array. + * On failure clear it out, but don't free it. + */ + + xmlNodePtr save; + char *address = NULL, *netmask = NULL; + char *gateway = NULL; + unsigned long prefix; + int result = -1; + virSocketAddr testAddr; + + save = ctxt->node; + ctxt->node = node; + + /* grab raw data from XML */ + def->family = virXPathString("string(./@family)", ctxt); + address = virXPathString("string(./@address)", ctxt); + if (virXPathULong("string(./@prefix)", ctxt, &prefix) < 0) + def->prefix = 0; + else + def->prefix = prefix; + + netmask = virXPathString("string(./@netmask)", ctxt); + + gateway = virXPathString("string(./@gateway)", ctxt); + + if (address) { + if (virSocketAddrParse(&def->address, address, AF_UNSPEC) < 0) { + virReportError(VIR_ERR_XML_ERROR, + _("Bad network address '%s' in route definition of network '%s'"), + address, networkName); + goto cleanup; + } + + } + + if (gateway) { + if (virSocketAddrParse(&def->gateway, gateway, AF_UNSPEC) < 0) { + virReportError(VIR_ERR_XML_ERROR, + _("Bad gateway address '%s' in route definition of network '%s'"), + gateway, networkName); + goto cleanup; + } + + } + + /* validate family vs. address and gateway address too*/ + if (def->family == NULL) { + if (!(VIR_SOCKET_ADDR_IS_FAMILY(&def->address, AF_INET) || + VIR_SOCKET_ADDR_IS_FAMILY(&def->address, AF_UNSPEC))) { + virReportError(VIR_ERR_CONFIG_UNSUPPORTED, + _("no family specified for non-IPv4 address '%s' in network '%s'"), + address, networkName); + goto cleanup; + } + if (gateway && + (!(VIR_SOCKET_ADDR_IS_FAMILY(&def->gateway, AF_INET) || + VIR_SOCKET_ADDR_IS_FAMILY(&def->gateway, AF_UNSPEC)))) { + virReportError(VIR_ERR_CONFIG_UNSUPPORTED, + _("no family specified for non-IPv4 gateway address '%s' in network '%s'"), + address, networkName); + goto cleanup; + } + if (def->prefix > 0 && def->prefix > 32) { + virReportError(VIR_ERR_CONFIG_UNSUPPORTED, + _("invalid prefix=%u specified for IPv4 address '%s' in network '%s'"), + def->prefix, address, networkName); + goto cleanup; + } + } else if (STREQ(def->family, "ipv4")) { + if (!VIR_SOCKET_ADDR_IS_FAMILY(&def->address, AF_INET)) { + virReportError(VIR_ERR_CONFIG_UNSUPPORTED, + _("family 'ipv4' specified for non-IPv4 address '%s' in network '%s'"), + address, networkName); + goto cleanup; + } + if (gateway && (!VIR_SOCKET_ADDR_IS_FAMILY(&def->gateway, AF_INET))) { + virReportError(VIR_ERR_CONFIG_UNSUPPORTED, + _("family 'ipv4' specified for non-IPv4 gateway address '%s' in network '%s'"), + address, networkName); + goto cleanup; + } + if (def->prefix > 0 && def->prefix > 32) { + virReportError(VIR_ERR_CONFIG_UNSUPPORTED, + _("invalid prefix=%u specified for IPv4 address '%s' in network '%s'"), + def->prefix, address, networkName); + goto cleanup; + } + } else if (STREQ(def->family, "ipv6")) { + if (!VIR_SOCKET_ADDR_IS_FAMILY(&def->address, AF_INET6)) { + virReportError(VIR_ERR_CONFIG_UNSUPPORTED, + _("family 'ipv6' specified for non-IPv6 address '%s' in network '%s'"), + address, networkName); + goto cleanup; + } + if (def->prefix == 0 || def->prefix > 64) { + virReportError(VIR_ERR_CONFIG_UNSUPPORTED, + _("invalid prefix=%u specified for IPv6 address '%s' in network '%s'"), + def->prefix, address, networkName); + goto cleanup; + } + if (netmask) { + virReportError(VIR_ERR_CONFIG_UNSUPPORTED, + _("specifying netmask invalid for IPv6 address '%s' in network '%s'"), + address, networkName); + goto cleanup; + } + if (gateway && (!VIR_SOCKET_ADDR_IS_FAMILY(&def->gateway, AF_INET6))) { + virReportError(VIR_ERR_CONFIG_UNSUPPORTED, + _("family 'ipv6' specified for non-IPv6 gateway address '%s' in network '%s'"), + gateway, networkName); + goto cleanup; + } + } else { + virReportError(VIR_ERR_XML_ERROR, + _("Unrecognized family '%s' in definition of network '%s'"), + def->family, networkName); + goto cleanup; + } + + /* parse/validate netmask */ + if (netmask) { + if (address == NULL) { + /* netmask is meaningless without an address */ + virReportError(VIR_ERR_CONFIG_UNSUPPORTED, + _("netmask specified without address in network '%s'"), + networkName); + goto cleanup; + } + + if (!VIR_SOCKET_ADDR_IS_FAMILY(&def->address, AF_INET)) { + virReportError(VIR_ERR_CONFIG_UNSUPPORTED, + _("netmask not supported for address '%s' in network '%s' (IPv4 only)"), + address, networkName); + goto cleanup; + } + + if (def->prefix > 0) { + /* can't have both netmask and prefix at the same time */ + virReportError(VIR_ERR_CONFIG_UNSUPPORTED, + _("network '%s' cannot have both prefix='%u' and a netmask"), + networkName, def->prefix); + goto cleanup; + } + + if (virSocketAddrParse(&def->netmask, netmask, AF_UNSPEC) < 0) + goto cleanup; + + if (!VIR_SOCKET_ADDR_IS_FAMILY(&def->netmask, AF_INET)) { + virReportError(VIR_ERR_CONFIG_UNSUPPORTED, + _("network '%s' has invalid netmask '%s' for address '%s' (both must be IPv4)"), + networkName, netmask, address); + goto cleanup; + } + } + + /* if static route gateway specified, make sure the address is a network address */ + if (def->prefix > 0) { + if (virSocketAddrMaskByPrefix(&def->address, def->prefix, &testAddr) < 0) { + virReportError(VIR_ERR_CONFIG_UNSUPPORTED, + _("error converting address '%s' with prefix=%u to network-address in network '%s'"), + address, def->prefix, networkName); + goto cleanup; + } + } + if (netmask) { + if (virSocketAddrMask(&def->address, &def->netmask, &testAddr) < 0) { + virReportError(VIR_ERR_CONFIG_UNSUPPORTED, + _("error converting address '%s' with netmask '%s' to network-address in network '%s'"), + address, netmask, networkName); + goto cleanup; + } + } + if (!virSocketAddrEqual(&def->address, &testAddr)) { + virReportError(VIR_ERR_CONFIG_UNSUPPORTED, + _("address '%s' in network '%s' is not a network-address"), + address, networkName); + goto cleanup; + } + + result = 0; + +cleanup: + if (result < 0) { + virNetworkIpDefClear(def); + } + VIR_FREE(address); + VIR_FREE(netmask); + VIR_FREE(gateway); + + ctxt->node = save; + return result; +} + +static int virNetworkPortGroupParseXML(virPortGroupDefPtr def, xmlNodePtr node, xmlXPathContextPtr ctxt) @@ -1676,8 +1878,9 @@ virNetworkDefParseXML(xmlXPathContextPtr ctxt) char *tmp; char *stp = NULL; xmlNodePtr *ipNodes = NULL; + xmlNodePtr *routeNodes = NULL; xmlNodePtr *portGroupNodes = NULL; - int nIps, nPortGroups; + int nIps, nPortGroups, nRoutes; xmlNodePtr dnsNode = NULL; xmlNodePtr virtPortNode = NULL; xmlNodePtr forwardNode = NULL; @@ -1831,6 +2034,82 @@ virNetworkDefParseXML(xmlXPathContextPtr ctxt) } VIR_FREE(ipNodes); + nRoutes = virXPathNodeSet("./route", ctxt, &routeNodes); + if (nRoutes < 0) + goto error; + + if (nRoutes > 0) { + int ii; + + /* allocate array to hold all the route definitions */ + if (VIR_ALLOC_N(def->routes, nRoutes) < 0) { + virReportOOMError(); + goto error; + } + /* parse each definition */ + for (ii = 0; ii < nRoutes; ii++) { + int ret = virNetworkRouteDefParseXML(def->name, routeNodes[ii], + ctxt, &def->routes[ii]); + if (ret < 0) + goto error; + def->nroutes++; + } + + /* now validate the correctness of any static route gateways specified + * + * note: the parameters within each definition are varified/assumed valid; + * the question being asked and answered here is if the specified gateway + * address has a network definition on this bridge + */ + nRoutes = def->nroutes; + nIps = def->nips; + for (ii = 0; ii < nRoutes; ii++) { + int jj; + virSocketAddr testAddr, testGw; + bool addrMatch; + virNetworkIpDefPtr gwdef = &def->routes[ii]; + if (VIR_SOCKET_ADDR_VALID(&gwdef->gateway)) { + addrMatch = false; + for (jj = 0; jj < nIps; jj++) { + virNetworkIpDefPtr def2 = &def->ips[jj]; + if ((VIR_SOCKET_ADDR_IS_FAMILY(&gwdef->gateway, AF_INET6)) && + (!VIR_SOCKET_ADDR_IS_FAMILY(&def2->address, AF_INET6))) + continue; + if ((VIR_SOCKET_ADDR_IS_FAMILY(&def2->address, AF_INET6)) && + (!VIR_SOCKET_ADDR_IS_FAMILY(&gwdef->gateway, AF_INET6))) + continue; + if ((VIR_SOCKET_ADDR_IS_FAMILY(&gwdef->gateway, AF_INET) || + VIR_SOCKET_ADDR_IS_FAMILY(&gwdef->gateway, AF_UNSPEC)) && + VIR_SOCKET_ADDR_IS_FAMILY(&def2->address, AF_INET6)) + continue; + if (def2->prefix > 0) { + virSocketAddrMaskByPrefix(&def2->address, def2->prefix, &testAddr); + virSocketAddrMaskByPrefix(&gwdef->gateway, def2->prefix, &testGw); + } + if (VIR_SOCKET_ADDR_VALID(&def2->netmask)) { + virSocketAddrMask(&def2->address, &def2->netmask, &testAddr); + virSocketAddrMask(&gwdef->gateway, &def2->netmask, &testGw); + } + if (VIR_SOCKET_ADDR_VALID(&testAddr) && + VIR_SOCKET_ADDR_VALID(&testGw) && + virSocketAddrEqual(&testAddr, &testGw)) { + addrMatch = true; + break; + } + } + if (!addrMatch) { + char *gw = virSocketAddrFormat(&gwdef->gateway); + virReportError(VIR_ERR_CONFIG_UNSUPPORTED, + _("invalid static route gateway specified: '%s'"), + gw); + VIR_FREE(gw); + goto error; + } + } + } + } + VIR_FREE(routeNodes); + forwardNode = virXPathNode("./forward", ctxt); if (forwardNode && virNetworkForwardDefParseXML(def->name, forwardNode, ctxt, &def->forward) < 0) { @@ -2203,6 +2482,49 @@ error: } static int +virNetworkRouteDefFormat(virBufferPtr buf, + const virNetworkIpDefPtr def) +{ + int result = -1; + + virBufferAddLit(buf, "<route"); + + if (def->family) { + virBufferAsprintf(buf, " family='%s'", def->family); + } + if (VIR_SOCKET_ADDR_VALID(&def->address)) { + char *addr = virSocketAddrFormat(&def->address); + if (!addr) + goto error; + virBufferAsprintf(buf, " address='%s'", addr); + VIR_FREE(addr); + } + if (VIR_SOCKET_ADDR_VALID(&def->netmask)) { + char *addr = virSocketAddrFormat(&def->netmask); + if (!addr) + goto error; + virBufferAsprintf(buf, " netmask='%s'", addr); + VIR_FREE(addr); + } + if (def->prefix > 0) { + virBufferAsprintf(buf," prefix='%u'", def->prefix); + } + if (VIR_SOCKET_ADDR_VALID(&def->gateway)) { + char *addr = virSocketAddrFormat(&def->gateway); + if (!addr) + goto error; + virBufferAsprintf(buf, " gateway='%s'", addr); + VIR_FREE(addr); + } + virBufferAddLit(buf, ">\n"); + virBufferAddLit(buf, "</route>\n"); + + result = 0; +error: + return result; +} + +static int virPortGroupDefFormat(virBufferPtr buf, const virPortGroupDefPtr def) { @@ -2400,6 +2722,11 @@ virNetworkDefFormatInternal(virBufferPtr buf, goto error; } + for (ii = 0; ii < def->nroutes; ii++) { + if (virNetworkRouteDefFormat(buf, &def->routes[ii]) < 0) + goto error; + } + if (virNetDevVPortProfileFormat(def->virtPortProfile, buf) < 0) goto error; diff --git a/src/conf/network_conf.h b/src/conf/network_conf.h index 6ce9a00..39aec7e 100644 --- a/src/conf/network_conf.h +++ b/src/conf/network_conf.h @@ -109,6 +109,8 @@ struct _virNetworkDNSDef { virNetworkDNSSrvDefPtr srvs; }; +/* NOTE: The folowing structure is used for both IP address + * definitions and for static route definitions */ typedef struct _virNetworkIpDef virNetworkIpDef; typedef virNetworkIpDef *virNetworkIpDefPtr; struct _virNetworkIpDef { @@ -124,6 +126,9 @@ struct _virNetworkIpDef { unsigned int prefix; /* ipv6 - only prefix allowed */ virSocketAddr netmask; /* ipv4 - either netmask or prefix specified */ + virSocketAddr gateway; /* gateway IP address for ip-route */ + + /* the following has no meaning if this is a static route definition */ size_t nranges; /* Zero or more dhcp ranges */ virSocketAddrRangePtr ranges; @@ -209,6 +214,9 @@ struct _virNetworkDef { size_t nips; virNetworkIpDefPtr ips; /* ptr to array of IP addresses on this network */ + size_t nroutes; + virNetworkIpDefPtr routes; /* ptr to array of static routes on this interface */ + virNetworkDNSDef dns; /* dns related configuration */ virNetDevVPortProfilePtr virtPortProfile; diff --git a/src/libvirt_private.syms b/src/libvirt_private.syms index 96eea0a..c51ba36 100644 --- a/src/libvirt_private.syms +++ b/src/libvirt_private.syms @@ -1466,6 +1466,7 @@ virNetDevReplaceMacAddress; virNetDevReplaceNetConfig; virNetDevRestoreMacAddress; virNetDevRestoreNetConfig; +virNetDevSetGateway; virNetDevSetIPv4Address; virNetDevSetMAC; virNetDevSetMTU; diff --git a/src/network/bridge_driver.c b/src/network/bridge_driver.c index e8b314a..2d4102e 100644 --- a/src/network/bridge_driver.c +++ b/src/network/bridge_driver.c @@ -2360,6 +2360,7 @@ out: return ret; } +/* add an IP address to a bridge */ static int networkAddAddrToBridge(virNetworkObjPtr network, virNetworkIpDefPtr ipdef) @@ -2380,6 +2381,28 @@ networkAddAddrToBridge(virNetworkObjPtr network, return 0; } +/* add an IP (static) route to a bridge */ +static int +networkAddRouteToBridge(virNetworkObjPtr network, + virNetworkIpDefPtr ipdef) +{ + int prefix = virNetworkIpDefPrefix(ipdef); + + if (prefix < 0) { + virReportError(VIR_ERR_INTERNAL_ERROR, + _("bridge '%s' has an invalid netmask or IP address"), + network->def->bridge); + return -1; + } + + if (virNetDevSetGateway(network->def->bridge, + &ipdef->address, + prefix, + &ipdef->gateway) < 0) + return -1; + return 0; +} + static int networkStartNetworkVirtual(struct network_driver *driver, virNetworkObjPtr network) @@ -2446,6 +2469,7 @@ networkStartNetworkVirtual(struct network_driver *driver, if (networkAddIptablesRules(driver, network) < 0) goto err1; + /* first, do all of the real addresses */ for (ii = 0; (ipdef = virNetworkDefGetIpByIndex(network->def, AF_UNSPEC, ii)); ii++) { @@ -2464,6 +2488,22 @@ networkStartNetworkVirtual(struct network_driver *driver, if (virNetDevSetOnline(network->def->bridge, 1) < 0) goto err2; + /* now, do all of the static routes */ + if (network->def->nroutes > 0) { + for (ii = 0; ii < network->def->nroutes; ii++) { + ipdef = &network->def->routes[ii]; + /* Add the IP route to the bridge */ + /* ignore errors, error msg will be generated */ + /* but libvirt will not know and net-destroy will work. */ + if (VIR_SOCKET_ADDR_VALID(&ipdef->gateway)) { + if (networkAddRouteToBridge(network, ipdef) < 0) { + /* an error occurred adding the static route */ + continue; /* for now, do nothing */ + } + } + } + } + /* If forward.type != NONE, turn on global IP forwarding */ if (network->def->forward.type != VIR_NETWORK_FORWARD_NONE && networkEnableIpForwarding(v4present, v6present) < 0) { diff --git a/src/util/virnetdev.c b/src/util/virnetdev.c index 00e0f94..6ac9d9d 100644 --- a/src/util/virnetdev.c +++ b/src/util/virnetdev.c @@ -769,6 +769,52 @@ cleanup: } /** + * virNetDevSetGateway: + * @ifname: the interface name + * @addr: the IP network address (IPv4 or IPv6) + * @prefix: number of 1 bits in the netmask + * @gateway: via address for route (same as @addr) + * + * Add a route for a network IP address to an interface. This function + * *does not* remove any previously added IP static routes. + * + * Returns 0 in case of success or -1 in case of error. + */ + +int virNetDevSetGateway(const char *ifname, + virSocketAddr *addr, + unsigned int prefix, + virSocketAddr *gateway) +{ + virCommandPtr cmd = NULL; + char *addrstr = NULL, *gatewaystr = NULL; + int ret = -1; + + if (!(addrstr = virSocketAddrFormat(addr))) + goto cleanup; + if (!(gatewaystr = virSocketAddrFormat(gateway))) + goto cleanup; + cmd = virCommandNew(IP_PATH); + virCommandAddArgList(cmd, "route", "add", NULL); + virCommandAddArgFormat(cmd, "%s/%u", addrstr, prefix); + virCommandAddArgList(cmd, "via", NULL); + virCommandAddArgFormat(cmd, "%s", gatewaystr); + virCommandAddArgList(cmd, "dev", ifname, NULL); + virCommandAddArgList(cmd, "proto", "static", "metric", NULL); + virCommandAddArgFormat(cmd, "%u", 1); + + if (virCommandRun(cmd, NULL) < 0) + goto cleanup; + + ret = 0; +cleanup: + VIR_FREE(addrstr); + VIR_FREE(gatewaystr); + virCommandFree(cmd); + return ret; +} + +/** * virNetDevClearIPv4Address: * @ifname: the interface name * @addr: the IP address (IPv4 or IPv6) diff --git a/src/util/virnetdev.h b/src/util/virnetdev.h index 06d0650..8b94ea8 100644 --- a/src/util/virnetdev.h +++ b/src/util/virnetdev.h @@ -42,6 +42,11 @@ int virNetDevSetIPv4Address(const char *ifname, virSocketAddr *addr, unsigned int prefix) ATTRIBUTE_NONNULL(1) ATTRIBUTE_NONNULL(2) ATTRIBUTE_RETURN_CHECK; +int virNetDevSetGateway(const char *ifname, + virSocketAddr *addr, + unsigned int prefix, + virSocketAddr *gateway) + ATTRIBUTE_NONNULL(1) ATTRIBUTE_NONNULL(2) ATTRIBUTE_RETURN_CHECK; int virNetDevClearIPv4Address(const char *ifname, virSocketAddr *addr, unsigned int prefix) diff --git a/tests/networkxml2xmlin/dhcp6host-routed-network.xml b/tests/networkxml2xmlin/dhcp6host-routed-network.xml index 2693d87..29e9561 100644 --- a/tests/networkxml2xmlin/dhcp6host-routed-network.xml +++ b/tests/networkxml2xmlin/dhcp6host-routed-network.xml @@ -19,4 +19,8 @@ <host id='0:1:0:1:18:aa:62:fe:0:16:3e:44:55:66' name='badbob' ip='2001:db8:ac10:fd01::1:24' /> </dhcp> </ip> + <route address="192.168.222.0" netmask="255.255.255.0" gateway="192.168.122.10"> + </route> + <route family="ipv6" address="2001:db8:ac10:fc00::" prefix="64" gateway="2001:db8:ac10:fd01::1:24"> + </route> </network> diff --git a/tests/networkxml2xmlout/dhcp6host-routed-network.xml b/tests/networkxml2xmlout/dhcp6host-routed-network.xml index 7305043..39af583 100644 --- a/tests/networkxml2xmlout/dhcp6host-routed-network.xml +++ b/tests/networkxml2xmlout/dhcp6host-routed-network.xml @@ -21,4 +21,8 @@ <host id='0:1:0:1:18:aa:62:fe:0:16:3e:44:55:66' name='badbob' ip='2001:db8:ac10:fd01::1:24' /> </dhcp> </ip> + <route address='192.168.222.0' netmask='255.255.255.0' gateway='192.168.122.10'> + </route> + <route family='ipv6' address='2001:db8:ac10:fc00::' prefix='64' gateway='2001:db8:ac10:fd01::1:24'> + </route> </network> -- 1.8.1.4

2 3

[libvirt] need help: what should be supported?
by Gene Czarcinski 19 Apr '13

19 Apr '13

I need some help/guidance as to what networking characteristics should be supported by libvirt. Mostly, I am asking what should be valid for a prefix. For IPv4: The prefix can theoretically range from 0 to 32. Typically, the values of 8, 16 or 24 is used. For an IP specification, should I be able to specify prefix=32 (netmask 255.255.255.255)? This is not a network specification but one for a specific guest-host ... but, what will the virtualization host have for the address on this interface? Proposed: prefix (or equivalent netmask) must be 8 < prefix < 29 For IPv6: Similar to IPv4 but the range is (theoretically) 0 to 128. However, DHCPv6 and RA will **ONLY** work for prefix=64 ... nothing greater and nothing less. For an network on an interface which is visible to the virtualization host, it must have an address for the host; therefore, prefix <127. I am not sure what an IP definition with prefix=0 means for a libvirt virtual network. Proposed: Prefix=64 for DHCPv6 and/or RA support. For other network addresses (such as static route), 7 < prefix <= 64. Most IPv6 software does not appear to handle subnets where prefix > 64 except for prefix=128 for a specific host address. That is not to say that prefix <64 prefix < 128 could not have a meaning and be useful. But, if supported, how should it be specified so that user problems are minimized ... or do we just support it and what happens is a user problem? Comments? Suggestions?? Gene

2 2

[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'> + <b><vlan trunk='yes' native_mode='tagged' native_tag='123'></b> + <b><tag id='42'/></b> + <b></vlan></b> + ... + </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. </p> - + <p> + For network connections using openvswitch it is possible to + configure the 'native-tagged' and 'native-untagged' vlan modes + <span class="since">(Since 1.0.3).</span> 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. + </p> <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'> + <b><vlan trunk='yes' native_mode='tagged' native_tag='123'></b> + <b><tag id='42'/></b> + <b></vlan></b> + ... + </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> + <p> + 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. + </p> + <p> + Example: usage of NVRAM configuration + </p> +<pre> + ... + <devices> + <nvram> + <address type='spapr-vio' reg='0x3000'/> + </nvram> + </devices> + ... +</pre> + <dl> + <dt><code>spapr-vio</code></dt> + <dd> + <p> + VIO device address type, this is only for PPC64. + </p> + </dd> + <dt><code>reg</code></dt> + <dd> + <p> + Devices' address + </p> + </dd> + </dl> + <h3><a name="seclabel">Security label</a></h3> <p> 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