forked from pool/libvirt
a870a15461
- Introduce virtlockd daemon
- parallels: add disk and network device support
- Add virDomainSendProcessSignal API
- Introduce virDomainFSTrim() public API
- add fuse support for libvirt lxc
- Add Gluster protocol as supported network disk backend
- various snapshot improvements
- Add upstream patches to fix bugs in 1.0.1
66ff2ddc-virtlockd-systemd-file-perms.patch,
462a6962-script-fixes1.patch, cb854b8f-script-fixes2.patch,
5ec4b22b-script-fixes3.patch, a1fd56cb-script-fixes4.patch,
68e7bc45-libxl-link-fix.patch
- Rework SUSE patches for the various init scripts
Dropped use-init-script-redhat.patch and added
libvirtd-init-script.patch, libvirt-guests-init-script.patch,
and virtlockd-init-script.patch
- Drop upstream patches: 371ddc98-xen-sysctl-9.patch,
416eca18-xenstore-header-fix.patch,
f644361b-virCommand-env.patch, 2b32735a-virCommand-env.patch,
9785f2b6-fix-xen-sysctl9.patch
OBS-URL: https://build.opensuse.org/package/show/Virtualization/libvirt?expand=0&rev=238
111 lines
4.1 KiB
Diff
111 lines
4.1 KiB
Diff
commit 7906a668fa8d5c21cc729db8a13b08e3dd1d241f
|
|
Author: Jim Fehlig <jfehlig@novell.com>
|
|
Date: Wed Jan 27 16:11:41 2010 -0700
|
|
|
|
Do not search xenstore for disk device IDs
|
|
|
|
Disk devices can be referenced by name in Xen, e.g. when modifying
|
|
their configuration or remvoving them. As such, don't search
|
|
xenstore for a device ID corresponding to the disk device. Instead,
|
|
search the disks contained in the domain definition and use the
|
|
disk's target name if found.
|
|
|
|
This approach allows removing a disk when domain is inactive. We
|
|
obviously can't search xenstore when the domain is inactive.
|
|
|
|
Index: libvirt-1.0.1/src/xen/xend_internal.c
|
|
===================================================================
|
|
--- libvirt-1.0.1.orig/src/xen/xend_internal.c
|
|
+++ libvirt-1.0.1/src/xen/xend_internal.c
|
|
@@ -61,6 +61,7 @@
|
|
|
|
static int
|
|
virDomainXMLDevID(virDomainPtr domain,
|
|
+ virDomainDefPtr domDef,
|
|
virDomainDeviceDefPtr dev,
|
|
char *class,
|
|
char *ref,
|
|
@@ -2764,7 +2765,7 @@ xenDaemonAttachDeviceFlags(virDomainPtr
|
|
|
|
sexpr = virBufferContentAndReset(&buf);
|
|
|
|
- if (virDomainXMLDevID(domain, dev, class, ref, sizeof(ref))) {
|
|
+ if (virDomainXMLDevID(domain, def, dev, class, ref, sizeof(ref))) {
|
|
/* device doesn't exist, define it */
|
|
ret = xend_op(domain->conn, domain->name, "op", "device_create",
|
|
"config", sexpr, NULL);
|
|
@@ -2885,7 +2886,7 @@ xenDaemonUpdateDeviceFlags(virDomainPtr
|
|
|
|
sexpr = virBufferContentAndReset(&buf);
|
|
|
|
- if (virDomainXMLDevID(domain, dev, class, ref, sizeof(ref))) {
|
|
+ if (virDomainXMLDevID(domain, def, dev, class, ref, sizeof(ref))) {
|
|
virReportError(VIR_ERR_OPERATION_INVALID, "%s",
|
|
_("requested device does not exist"));
|
|
goto cleanup;
|
|
@@ -2980,7 +2981,7 @@ xenDaemonDetachDeviceFlags(virDomainPtr
|
|
def, xml, VIR_DOMAIN_XML_INACTIVE)))
|
|
goto cleanup;
|
|
|
|
- if (virDomainXMLDevID(domain, dev, class, ref, sizeof(ref)))
|
|
+ if (virDomainXMLDevID(domain, def, dev, class, ref, sizeof(ref)))
|
|
goto cleanup;
|
|
|
|
if (dev->type == VIR_DOMAIN_DEVICE_HOSTDEV) {
|
|
@@ -3926,6 +3927,7 @@ struct xenUnifiedDriver xenDaemonDriver
|
|
*/
|
|
static int
|
|
virDomainXMLDevID(virDomainPtr domain,
|
|
+ virDomainDefPtr domDef,
|
|
virDomainDeviceDefPtr dev,
|
|
char *class,
|
|
char *ref,
|
|
@@ -3934,8 +3936,12 @@ virDomainXMLDevID(virDomainPtr domain,
|
|
xenUnifiedPrivatePtr priv = domain->conn->privateData;
|
|
char *xref;
|
|
char *tmp;
|
|
+ unsigned int i;
|
|
+ virDomainDiskDefPtr disk;
|
|
|
|
if (dev->type == VIR_DOMAIN_DEVICE_DISK) {
|
|
+ if (dev->data.disk->dst == NULL)
|
|
+ return -1;
|
|
if (dev->data.disk->driverName &&
|
|
STREQ(dev->data.disk->driverName, "tap"))
|
|
strcpy(class, "tap");
|
|
@@ -3945,19 +3951,21 @@ virDomainXMLDevID(virDomainPtr domain,
|
|
else
|
|
strcpy(class, "vbd");
|
|
|
|
- if (dev->data.disk->dst == NULL)
|
|
- return -1;
|
|
- xenUnifiedLock(priv);
|
|
- xref = xenStoreDomainGetDiskID(domain->conn, domain->id,
|
|
- dev->data.disk->dst);
|
|
- xenUnifiedUnlock(priv);
|
|
- if (xref == NULL)
|
|
- return -1;
|
|
-
|
|
- tmp = virStrcpy(ref, xref, ref_len);
|
|
- VIR_FREE(xref);
|
|
- if (tmp == NULL)
|
|
- return -1;
|
|
+ /* For disks, the device name can be used directly.
|
|
+ * If disk device exists in domain definintion,
|
|
+ * copy it to ref and return success.
|
|
+ */
|
|
+ for (i = 0; i < domDef->ndisks; i++) {
|
|
+ disk = domDef->disks[i];
|
|
+ if (STREQ(dev->data.disk->dst, disk->dst)) {
|
|
+ tmp = virStrcpy(ref, disk->dst, ref_len);
|
|
+ if (tmp == NULL)
|
|
+ return -1;
|
|
+ else
|
|
+ return 0;
|
|
+ }
|
|
+ }
|
|
+ return -1;
|
|
} else if (dev->type == VIR_DOMAIN_DEVICE_NET) {
|
|
char mac[30];
|
|
virDomainNetDefPtr def = dev->data.net;
|