[libvirt] [PATCH] Revert "maint: Update to latest gnulib"
by Erik Skultety
Revert @f8172388c which broke the build on Centos 6, because of old
autoconf (< 2.63b) not properly quoting arguments for shell.
---
So, the last working commit is cbc700208, but I didn't want to revert to that
one directly, since I'd like to poke gnulib about the issue first and find a
proper solution.
Anyhow, pushed under the build breaker rule.
.gnulib | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/.gnulib b/.gnulib
index 47c634c05..ce4ee4cbb 160000
--- a/.gnulib
+++ b/.gnulib
@@ -1 +1 @@
-Subproject commit 47c634c05a946651e2834dcf0cd7cd0dc7a21d3c
+Subproject commit ce4ee4cbb596a9d7de2786cf8c48cf62a4edede7
--
2.13.3
7 years, 3 months
[libvirt] libvirt 3.5.0 locks
by Vasiliy Tolstov
Hi. I'm testing libvirt 3.5.0 and have many issues like this:
Jul 14 09:47:25 cn25 libvirtd[18820]: 2017-07-14 06:47:25.598+0000:
18841: error : qemuDomainObjBeginJobInternal:3872 : Timed out during
operation: cannot acquire state change lock (held by
remoteDispatchConnectGetAllDomainStats)
Jul 14 09:47:25 cn25 libvirtd[18820]: 2017-07-14 06:47:25.598+0000:
18841: warning : qemuDomainObjBeginJobInternal:3860 : Cannot start job
(query, none) for domain 147565; current job is (query, none) owned by
(18828 remoteDispatchConnectGetAllDomainStats, 0 <null>) for (53670s,
0s)
Jul 14 09:46:55 cn25 libvirtd[18820]: 2017-07-14 06:46:55.576+0000:
18836: error : virNetDevTapInterfaceStats:751 : internal error:
/proc/net/dev: Interface not found
Jul 14 09:46:55 cn25 libvirtd[18820]: 2017-07-14 06:46:55.542+0000:
18832: warning : qemuGetProcessInfo:1428 : cannot parse process status
data
Jul 14 09:46:31 cn25 libvirtd[18820]: 2017-07-14 06:46:31.050+0000:
18838: warning : qemuGetProcessInfo:1428 : cannot parse process status
data
Domain in status in shutdown
as i see tap device already removed from the system, so
virNetDevTapInterfaceStats says right info.
My question - why lock now released and why only libvirt restart can
helps in this case
--
Vasiliy Tolstov,
e-mail: v.tolstov(a)selfip.ru
7 years, 3 months
[libvirt] [PATCH v2] virt-host-validate: Fix warning for IOMMU detection on PPC
by Nitesh Konkar
Fix the warning generated on PPC by virt-host-validate
for IOMMU
Signed-off-by: Nitesh Konkar <nitkon12(a)linux.vnet.ibm.com>
---
tools/virt-host-validate-common.c | 10 +++++++---
1 file changed, 7 insertions(+), 3 deletions(-)
diff --git a/tools/virt-host-validate-common.c b/tools/virt-host-validate-common.c
index 6faed04..51fa8c0 100644
--- a/tools/virt-host-validate-common.c
+++ b/tools/virt-host-validate-common.c
@@ -35,6 +35,7 @@
#include "virfile.h"
#include "virt-host-validate-common.h"
#include "virstring.h"
+#include "virarch.h"
#define VIR_FROM_THIS VIR_FROM_NONE
@@ -442,8 +443,7 @@ int virHostValidateIOMMU(const char *hvname,
virBitmapPtr flags;
struct stat sb;
const char *bootarg = NULL;
- bool isAMD = false, isIntel = false;
-
+ bool isAMD = false, isIntel = false, isPPC = false;
flags = virHostValidateGetCPUFlags();
if (flags && virBitmapIsBitSet(flags, VIR_HOST_VALIDATE_CPU_FLAG_VMX))
@@ -453,9 +453,10 @@ int virHostValidateIOMMU(const char *hvname,
virBitmapFree(flags);
- virHostMsgCheck(hvname, "%s", _("for device assignment IOMMU support"));
+ isPPC = ARCH_IS_PPC64(virArchFromHost());
if (isIntel) {
+ virHostMsgCheck(hvname, "%s", _("for device assignment IOMMU support"));
if (access("/sys/firmware/acpi/tables/DMAR", F_OK) == 0) {
virHostMsgPass();
bootarg = "intel_iommu=on";
@@ -467,6 +468,7 @@ int virHostValidateIOMMU(const char *hvname,
return -1;
}
} else if (isAMD) {
+ virHostMsgCheck(hvname, "%s", _("for device assignment IOMMU support"));
if (access("/sys/firmware/acpi/tables/IVRS", F_OK) == 0) {
virHostMsgPass();
bootarg = "iommu=pt iommu=1";
@@ -477,6 +479,8 @@ int virHostValidateIOMMU(const char *hvname,
"hardware platform");
return -1;
}
+ } else if (isPPC) {
+ /* Empty Block */
} else {
virHostMsgFail(level,
"Unknown if this platform has IOMMU support");
--
2.7.4
7 years, 3 months
[libvirt] [PATCH] libxl: Avoid a variable named 'stat'
by George Dunlap
Using a variable named 'stat' clashes with the system function
'stat()' causing compiler warnings on some platforms:
libxl/libxl_driver.c: In function 'libxlDomainBlockStatsVBD':
libxl/libxl_driver.c:5387: error: declaration of 'stat' shadows a global declaration [-Wshadow]
/usr/include/sys/stat.h:455: error: shadowed declaration is here [-Wshadow]
Signed-off-by: George Dunlap <george.dunlap(a)citrix.com>
---
CC: Daniel P. Berrange <berrange(a)redhat.com>
CC: Jim Fehlig <jfehlig(a)suse.com>
CC: xen-devel(a)xenproject.org
CC: libvir-list(a)redhat.com
---
src/libxl/libxl_driver.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/src/libxl/libxl_driver.c b/src/libxl/libxl_driver.c
index c261e2155..4c7775358 100644
--- a/src/libxl/libxl_driver.c
+++ b/src/libxl/libxl_driver.c
@@ -5374,7 +5374,7 @@ libxlDomainBlockStatsVBD(virDomainObjPtr vm,
int devno = libxlDiskPathToID(dev);
int size;
char *path, *name, *val;
- unsigned long long stat;
+ unsigned long long status;
path = name = val = NULL;
if (devno < 0) {
@@ -5401,12 +5401,12 @@ libxlDomainBlockStatsVBD(virDomainObjPtr vm,
# define LIBXL_SET_VBDSTAT(FIELD, VAR, MUL) \
if ((virAsprintf(&name, "%s/"FIELD, path) < 0) || \
(virFileReadAll(name, 256, &val) < 0) || \
- (sscanf(val, "%llu", &stat) != 1)) { \
+ (sscanf(val, "%llu", &status) != 1)) { \
virReportError(VIR_ERR_OPERATION_FAILED, \
_("cannot read %s"), name); \
goto cleanup; \
} \
- VAR += (stat * MUL); \
+ VAR += (status * MUL); \
VIR_FREE(name); \
VIR_FREE(val);
--
2.14.1
7 years, 3 months
[libvirt] [PATCH] qemu: Report error on failure to set isolation group
by Andrea Bolognani
This is more user-friendly because the error will be
displayed directly instead of being buried in the log.
Signed-off-by: Andrea Bolognani <abologna(a)redhat.com>
---
src/qemu/qemu_domain_address.c | 17 ++++++++++-------
1 file changed, 10 insertions(+), 7 deletions(-)
diff --git a/src/qemu/qemu_domain_address.c b/src/qemu/qemu_domain_address.c
index 8066ed123..16bf0cdf9 100644
--- a/src/qemu/qemu_domain_address.c
+++ b/src/qemu/qemu_domain_address.c
@@ -1012,10 +1012,11 @@ qemuDomainFillDeviceIsolationGroup(virDomainDefPtr def,
tmp = virPCIDeviceAddressGetIOMMUGroupNum(hostAddr);
if (tmp < 0) {
- VIR_WARN("Can't look up isolation group for host device "
- "%04x:%02x:%02x.%x",
- hostAddr->domain, hostAddr->bus,
- hostAddr->slot, hostAddr->function);
+ virReportError(VIR_ERR_INTERNAL_ERROR,
+ _("Can't look up isolation group for host device "
+ "%04x:%02x:%02x.%x"),
+ hostAddr->domain, hostAddr->bus,
+ hostAddr->slot, hostAddr->function);
goto cleanup;
}
@@ -1056,9 +1057,11 @@ qemuDomainFillDeviceIsolationGroup(virDomainDefPtr def,
tmp = qemuDomainFindUnusedIsolationGroup(def);
if (tmp == 0) {
- VIR_WARN("Can't obtain usable isolation group for interface "
- "configured to use hostdev-backed network '%s'",
- iface->data.network.name);
+ virReportError(VIR_ERR_INTERNAL_ERROR,
+ _("Can't obtain usable isolation group for "
+ "interface configured to use hostdev-backed "
+ "network '%s'"),
+ iface->data.network.name);
goto cleanup;
}
--
2.13.5
7 years, 3 months
[libvirt] Question about "f16c", "rdrand" and "vme" in cpu_map.xml
by Jay Zhou
Hi all,
"f16c", "rdrand" and "vme" are all standard features, the IvyBridge CPU model
has these three features in the file of src/cpu/cpu_map.xml, but the higher
model of Haswell-noTSX, Haswell, Broadwell-noTSX and Broadwell do not, which
makes it incompatible.
BTW, I noticed that "f16c", "rdrand" and "vme" are a part of features of
Haswell-noTSX, Haswell, Broadwell-noTSX and Broadwell in QEMU.
The commit 732d3d147adecb867f62c67ab3743c0aefd87e50(which introduces the
Haswell-noTSX CPU model in libvirt) said that:
"Inheritance among CPU model is cool but it makes reviewing CPU model
definitions and comparing them to CPU models from QEMU rather hard and
unpleasant."
I do not get the point "it makes reviewing CPU model definitions and comparing
them to CPU models from QEMU rather hard and unpleasant" clearly. Could anyone
explain it more explicitly?
Looking forward to your replay.
Thanks,
Jay
7 years, 3 months
[libvirt] [PATCH] maint: Update to latest gnulib
by Erik Skultety
This pulls in, among other new things, vc-list-files fix to make
syntax-check work with git worktrees.
Signed-off-by: Erik Skultety <eskultet(a)redhat.com>
---
.gnulib | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/.gnulib b/.gnulib
index ce4ee4cbb..47c634c05 160000
--- a/.gnulib
+++ b/.gnulib
@@ -1 +1 @@
-Subproject commit ce4ee4cbb596a9d7de2786cf8c48cf62a4edede7
+Subproject commit 47c634c05a946651e2834dcf0cd7cd0dc7a21d3c
--
2.13.3
7 years, 3 months
[libvirt] [PATCH v5 00/15] virObject adjustments for common object
by John Ferlan
v4: https://www.redhat.com/archives/libvir-list/2017-August/msg00537.html
Changes since v4...
I've removed the virObjectLookupKeys completely relying on LookupHash
completely. The adjusted LookupHash object uses named "objsUUID" and
"objsName" as well as API's to specifically Search or ForEach by
Name or UUID. The New, Add, and Remove API's were adjusted in order
to handle creating things properly.
Even if that adjustment isn't considered what's desired, at least look
at patches 1, 10, and 14 separately.
John Ferlan (15):
util: Use VIR_ERROR instead of VIR_WARN
util: Introduce virObjectLookupHash
util: Introduce virObjectLookupHash{Add|Remove}
util: Introduce virObjectLookupHashFind[Locked]
util: Introduce virObjectLookupHashForEach{UUID|Name}
util: Introduce virObjectLookupHashSearch{UUID|Name}[Locked]
nodedev: Use virObjectLookupHash
secret: Use virObjectLookupHash
util: Introduce virObjectLookupHashClone
Revert "interface: Consume @def in virInterfaceObjNew"
interface: Use virObjectLookupHash
test: Clean up test driver Interface interactions
util: Introduce virObjectLookupHashPrune
network: Fix virNetworkObjBridgeInUse return type
network: Use virObjectLookupHash
src/conf/virinterfaceobj.c | 289 ++++++++++---------
src/conf/virnetworkobj.c | 280 ++++++------------
src/conf/virnetworkobj.h | 2 +-
src/conf/virnodedeviceobj.c | 273 +++++-------------
src/conf/virsecretobj.c | 251 +++++-----------
src/libvirt_private.syms | 14 +
src/test/test_driver.c | 55 +---
src/util/virobject.c | 675 +++++++++++++++++++++++++++++++++++++++++++-
src/util/virobject.h | 123 ++++++++
9 files changed, 1189 insertions(+), 773 deletions(-)
--
2.9.5
7 years, 3 months
[libvirt] [PATCH] vz: build fix
by Nikolay Shirokovskiy
6e6faf6d changed vzDomObjAlloc signature in source but not in header file.
---
src/vz/vz_utils.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/vz/vz_utils.h b/src/vz/vz_utils.h
index ea2e2da..d213183 100644
--- a/src/vz/vz_utils.h
+++ b/src/vz/vz_utils.h
@@ -116,7 +116,7 @@ struct vzDomObj {
typedef struct vzDomObj *vzDomObjPtr;
-void* vzDomObjAlloc(void);
+void* vzDomObjAlloc(void *opaque);
void vzDomObjFree(void *p);
virDomainObjPtr vzDomObjFromDomain(virDomainPtr domain);
--
1.8.3.1
7 years, 3 months
[libvirt] [PATCH 0/2] conf: check address type for USB hostdevs
by Ján Tomko
*** BRB HERE ***
Ján Tomko (2):
conf: move hostdev address validation to virDomainHostdevDefValidate
conf: check address type for USB hostdevs
src/conf/domain_conf.c | 83 ++++++++++++++++++++++++++++++++++----------------
1 file changed, 56 insertions(+), 27 deletions(-)
--
2.13.0
7 years, 3 months
