[libvirt] [PATCH v1 0/7] Enable memory-backend-file.discard-data
by Michal Privoznik
Qemu introduced this in 11ae6ed8affdd13. A small refactor is needed
however.
Michal Privoznik (7):
qemu: Rename qemuMonitorGetObjectProps to qemuMonitorGetDeviceProps
qemu_capabilities: s/ObjectProps/DeviceProps/g
qemu_capabilities: Separate out device props fetching
qemu_monitor: Introduce qemuMonitorGetObjectProps
qemu_capabilities: Introduce QEMU_CAPS_OBJECT_MEMORY_FILE_DISCARD
tests: Update qemucapabilitiesdata
qemu: Enable memory-backend-file.discard-data whenever possible
src/qemu/qemu_capabilities.c | 269 +++++----
src/qemu/qemu_capabilities.h | 1 +
src/qemu/qemu_command.c | 6 +
src/qemu/qemu_monitor.c | 21 +-
src/qemu/qemu_monitor.h | 5 +-
src/qemu/qemu_monitor_json.c | 62 ++-
src/qemu/qemu_monitor_json.h | 6 +-
.../qemucapabilitiesdata/caps_2.1.1.x86_64.replies | 20 +-
tests/qemucapabilitiesdata/caps_2.1.1.x86_64.xml | 2 +-
.../caps_2.10.0.aarch64.replies | 30 +-
tests/qemucapabilitiesdata/caps_2.10.0.aarch64.xml | 2 +-
.../qemucapabilitiesdata/caps_2.10.0.ppc64.replies | 28 +-
tests/qemucapabilitiesdata/caps_2.10.0.ppc64.xml | 2 +-
.../qemucapabilitiesdata/caps_2.10.0.s390x.replies | 32 +-
tests/qemucapabilitiesdata/caps_2.10.0.s390x.xml | 2 +-
.../caps_2.10.0.x86_64.replies | 540 +++++++++---------
tests/qemucapabilitiesdata/caps_2.10.0.x86_64.xml | 2 +-
.../qemucapabilitiesdata/caps_2.11.0.s390x.replies | 26 +-
tests/qemucapabilitiesdata/caps_2.11.0.s390x.xml | 2 +-
.../caps_2.12.0.aarch64.replies | 24 +-
tests/qemucapabilitiesdata/caps_2.12.0.aarch64.xml | 2 +-
.../qemucapabilitiesdata/caps_2.12.0.ppc64.replies | 22 +-
tests/qemucapabilitiesdata/caps_2.12.0.ppc64.xml | 2 +-
.../qemucapabilitiesdata/caps_2.12.0.s390x.replies | 24 +-
.../caps_2.12.0.x86_64.replies | 612 +++++++++++----------
tests/qemucapabilitiesdata/caps_2.12.0.x86_64.xml | 3 +-
.../qemucapabilitiesdata/caps_2.4.0.x86_64.replies | 20 +-
tests/qemucapabilitiesdata/caps_2.4.0.x86_64.xml | 2 +-
.../qemucapabilitiesdata/caps_2.5.0.x86_64.replies | 22 +-
tests/qemucapabilitiesdata/caps_2.5.0.x86_64.xml | 2 +-
.../caps_2.6.0.aarch64.replies | 30 +-
tests/qemucapabilitiesdata/caps_2.6.0.aarch64.xml | 2 +-
.../qemucapabilitiesdata/caps_2.6.0.ppc64.replies | 28 +-
tests/qemucapabilitiesdata/caps_2.6.0.ppc64.xml | 2 +-
.../qemucapabilitiesdata/caps_2.6.0.x86_64.replies | 22 +-
tests/qemucapabilitiesdata/caps_2.6.0.x86_64.xml | 2 +-
.../qemucapabilitiesdata/caps_2.7.0.s390x.replies | 28 +-
tests/qemucapabilitiesdata/caps_2.7.0.s390x.xml | 2 +-
.../qemucapabilitiesdata/caps_2.7.0.x86_64.replies | 22 +-
tests/qemucapabilitiesdata/caps_2.7.0.x86_64.xml | 2 +-
.../qemucapabilitiesdata/caps_2.8.0.s390x.replies | 32 +-
tests/qemucapabilitiesdata/caps_2.8.0.s390x.xml | 2 +-
.../qemucapabilitiesdata/caps_2.8.0.x86_64.replies | 22 +-
tests/qemucapabilitiesdata/caps_2.8.0.x86_64.xml | 2 +-
.../qemucapabilitiesdata/caps_2.9.0.ppc64.replies | 28 +-
tests/qemucapabilitiesdata/caps_2.9.0.ppc64.xml | 2 +-
.../qemucapabilitiesdata/caps_2.9.0.s390x.replies | 32 +-
tests/qemucapabilitiesdata/caps_2.9.0.s390x.xml | 2 +-
.../qemucapabilitiesdata/caps_2.9.0.x86_64.replies | 536 +++++++++---------
tests/qemucapabilitiesdata/caps_2.9.0.x86_64.xml | 2 +-
50 files changed, 1458 insertions(+), 1135 deletions(-)
--
2.16.1
7 years, 2 months
[libvirt] [PATCH] docs: index.html.in: Fix a typo in "virtualization platforms" link
by Erik Skultety
This was pointing to an .html.in file rather than to the intended html
document.
Signed-off-by: Erik Skultety <eskultet(a)redhat.com>
---
Pushed under the trivial rule.
docs/index.html.in | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/docs/index.html.in b/docs/index.html.in
index 4783c39e3..1f9f44839 100644
--- a/docs/index.html.in
+++ b/docs/index.html.in
@@ -28,7 +28,7 @@
The libvirt project:
</p>
<ul>
- <li>is a toolkit to manage <a href="platforms.html.in">virtualization platforms</a></li>
+ <li>is a toolkit to manage <a href="platforms.html">virtualization platforms</a></li>
<li>is accessible from C, Python, Perl, Java and more</li>
<li>is licensed under open source licenses</li>
<li>supports <a href="drvqemu.html">KVM</a>,
--
2.14.3
7 years, 2 months
[libvirt] [PATCH 0/9] virClassNew rework
by Michal Privoznik
This is inspired by commit of f574e2e5214fb9. Also, the more the code is
unified the easier it is to maintain.
Michal Privoznik (9):
domain_event: s/MetadataCange/MetadataChange/g
util: Make structs follow our naming convention
datatypes: Rename @parent to @parentName in virNodeDevice
src: Unify virObject member name
src: Unify dispose function names
Introduce virNetSASLContextDispose
virobject: Introduce VIR_CLASS_NEW() macro
virobject: Check if @parent is the first member in class
cfg.mk: Introduce syntax-check rule to prefer VIR_CLASS_NEW
cfg.mk | 8 ++
src/access/viraccessmanager.c | 6 +-
src/bhyve/bhyve_conf.c | 6 +-
src/conf/capabilities.c | 10 +--
src/conf/domain_capabilities.c | 12 +--
src/conf/domain_conf.c | 22 ++----
src/conf/domain_event.c | 130 +++++++++++---------------------
src/conf/network_event.c | 12 +--
src/conf/node_device_event.c | 18 ++---
src/conf/object_event.c | 12 +--
src/conf/secret_event.c | 18 ++---
src/conf/storage_event.c | 18 ++---
src/conf/virdomainobjlist.c | 6 +-
src/conf/virinterfaceobj.c | 12 +--
src/conf/virnetworkobj.c | 12 +--
src/conf/virnodedeviceobj.c | 14 ++--
src/conf/virsecretobj.c | 12 +--
src/conf/virstorageobj.c | 24 ++----
src/datatypes.c | 8 +-
src/datatypes.h | 30 ++++----
src/interface/interface_backend_netcf.c | 6 +-
src/libvirt-admin.c | 8 +-
src/libvirt-domain-snapshot.c | 2 +-
src/libvirt-domain.c | 2 +-
src/libvirt-host.c | 2 +-
src/libvirt-interface.c | 2 +-
src/libvirt-network.c | 2 +-
src/libvirt-nodedev.c | 8 +-
src/libvirt-nwfilter.c | 2 +-
src/libvirt-secret.c | 2 +-
src/libvirt-storage.c | 4 +-
src/libvirt-stream.c | 2 +-
src/libxl/libxl_conf.c | 6 +-
src/libxl/libxl_domain.c | 6 +-
src/libxl/libxl_migration.c | 6 +-
src/logging/log_handler.c | 6 +-
src/lxc/lxc_conf.c | 6 +-
src/lxc/lxc_monitor.c | 6 +-
src/node_device/node_device_driver.c | 4 +-
src/node_device/node_device_udev.c | 6 +-
src/qemu/qemu_agent.c | 6 +-
src/qemu/qemu_capabilities.c | 8 +-
src/qemu/qemu_conf.c | 6 +-
src/qemu/qemu_domain.c | 36 +++------
src/qemu/qemu_monitor.c | 6 +-
src/rpc/virkeepalive.c | 6 +-
src/rpc/virnetclient.c | 6 +-
src/rpc/virnetclientprogram.c | 8 +-
src/rpc/virnetclientstream.c | 6 +-
src/rpc/virnetdaemon.c | 6 +-
src/rpc/virnetlibsshsession.c | 6 +-
src/rpc/virnetsaslcontext.c | 18 +++--
src/rpc/virnetserver.c | 6 +-
src/rpc/virnetserverclient.c | 6 +-
src/rpc/virnetserverprogram.c | 8 +-
src/rpc/virnetserverservice.c | 8 +-
src/rpc/virnetsocket.c | 6 +-
src/rpc/virnetsshsession.c | 6 +-
src/rpc/virnettlscontext.c | 12 +--
src/security/security_manager.c | 6 +-
src/test/test_driver.c | 6 +-
src/util/virclosecallbacks.c | 6 +-
src/util/virdnsmasq.c | 9 +--
src/util/virfdstream.c | 10 +--
src/util/virfilecache.c | 8 +-
src/util/virhash.c | 6 +-
src/util/virhostdev.c | 6 +-
src/util/viridentity.c | 6 +-
src/util/virmacmap.c | 8 +-
src/util/virmacmap.h | 2 +-
src/util/virmdev.c | 6 +-
src/util/virobject.c | 43 ++++++-----
src/util/virobject.h | 7 ++
src/util/virpci.c | 6 +-
src/util/virportallocator.c | 6 +-
src/util/virresctrl.c | 12 +--
src/util/virscsi.c | 6 +-
src/util/virscsivhost.c | 6 +-
src/util/virusb.c | 6 +-
src/vbox/vbox_common.c | 6 +-
src/vz/vz_driver.c | 6 +-
tests/virfilecachetest.c | 8 +-
82 files changed, 322 insertions(+), 517 deletions(-)
--
2.16.1
7 years, 2 months
[libvirt] [PATCH] virsh: Clear vsh last error during virshCommandOptVolBy
by John Ferlan
https://bugzilla.redhat.com/show_bug.cgi?id=1529256
If one of the virStorageVolLookupBy{Key|Name|Path} succeeds and
we have a @vol, then clear the last libvirt error; otherwise, a
subsequent "other" failure may cause vshReportError to erroneously
report the wrong error as well as a reported 'vshError' error that
caused the failure.
Signed-off-by: John Ferlan <jferlan(a)redhat.com>
---
tools/virsh-volume.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/tools/virsh-volume.c b/tools/virsh-volume.c
index bacbec0d27..9d6ebd2325 100644
--- a/tools/virsh-volume.c
+++ b/tools/virsh-volume.c
@@ -129,6 +129,8 @@ virshCommandOptVolBy(vshControl *ctl, const vshCmd *cmd,
else
vshError(ctl, _("failed to get vol '%s', specifying --%s "
"might help"), n, pooloptname);
+ } else {
+ vshResetLibvirtError();
}
/* If the pool was specified, then make sure that the returned
--
2.13.6
7 years, 2 months
[libvirt] [PATCH 0/2] Trivial fixes
by Radostin Stoyanov
Radostin Stoyanov (2):
lxc: s/subtreee/subtree/
lxc: Clean up /.oldroot
src/lxc/lxc_container.c | 5 ++++-
1 file changed, 4 insertions(+), 1 deletion(-)
--
2.14.3
7 years, 2 months
[libvirt] [PATCH] qemu_domain: Don't leak @paths in qemuDomainNamespaceSetupDisk
by Michal Privoznik
Introduced in d3db304d2e4. Instead of returning immediately we
need to jump onto cleanup label where @paths is freed.
Signed-off-by: Michal Privoznik <mprivozn(a)redhat.com>
---
Pushed under trivial rule.
src/qemu/qemu_domain.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/qemu/qemu_domain.c b/src/qemu/qemu_domain.c
index 7000de6a91..6e18886d07 100644
--- a/src/qemu/qemu_domain.c
+++ b/src/qemu/qemu_domain.c
@@ -11199,7 +11199,7 @@ qemuDomainNamespaceSetupDisk(virDomainObjPtr vm,
}
if (qemuDomainNamespaceMknodPaths(vm, paths, npaths) < 0)
- return -1;
+ goto cleanup;
ret = 0;
cleanup:
--
2.16.1
7 years, 2 months
[libvirt] [PATCH 0/3] qemu: use FD passing for monitor (and other unix sockets)
by Daniel P. Berrangé
This series makes use of the chardev fd passing arriving in QEMU 2.12
to get rid of the startup race wrt opening the QEMU monitor.
The same thing benefits the agent too but I've not tackled that yet.
Daniel P. Berrangé (3):
qemu: probe for -chardev 'fd' parameter for FD passing
qemu: support passing pre-opened UNIX socket listen FD
qemu: don't retry connect() if doing FD passing
src/qemu/qemu_capabilities.c | 4 ++-
src/qemu/qemu_capabilities.h | 1 +
src/qemu/qemu_command.c | 58 ++++++++++++++++++++++++++++++++++++++++++--
src/qemu/qemu_monitor.c | 54 ++++++++++++++++++++++++-----------------
src/qemu/qemu_monitor.h | 1 +
src/qemu/qemu_process.c | 27 +++++++++++++++------
tests/qemumonitortestutils.c | 1 +
7 files changed, 114 insertions(+), 32 deletions(-)
--
2.14.3
7 years, 2 months
[libvirt] [PATCH 0/2] A couple of openvz fixups
by John Ferlan
Patch 1 came from "the next part" of the code when I determined
that the Unref was unnecessary
Patch 2 is a result of Jim Fehlig's review that openvzDomainGetHostname
doesn't really need the error/cleanup logic.
John Ferlan (2):
openvz: Remove unnecessary Unref in openvzLoadDomains
openvz: Clean up openvzDomainGetHostname
src/openvz/openvz_conf.c | 1 -
src/openvz/openvz_driver.c | 8 ++------
2 files changed, 2 insertions(+), 7 deletions(-)
--
2.13.6
7 years, 2 months
[libvirt] [dbus PATCH] maint: rename test directory to tests
by Pavel Hrdina
Our ci.centos.org expects the project uses 'tests' name.
Signed-off-by: Pavel Hrdina <phrdina(a)redhat.com>
---
.travis.yml | 2 +-
Makefile.am | 2 +-
configure.ac | 2 +-
{test => tests}/Makefile.am | 0
{test => tests}/conftest.py | 0
{test => tests}/libvirttest.py | 0
{test => tests}/test_connect.py | 0
{test => tests}/test_domain.py | 0
{test => tests}/test_network.py | 0
{test => tests}/travis-run | 0
10 files changed, 3 insertions(+), 3 deletions(-)
rename {test => tests}/Makefile.am (100%)
rename {test => tests}/conftest.py (100%)
rename {test => tests}/libvirttest.py (100%)
rename {test => tests}/test_connect.py (100%)
rename {test => tests}/test_domain.py (100%)
rename {test => tests}/test_network.py (100%)
rename {test => tests}/travis-run (100%)
diff --git a/.travis.yml b/.travis.yml
index 71c2e9c..d60eaf2 100644
--- a/.travis.yml
+++ b/.travis.yml
@@ -1,7 +1,7 @@
dist: trusty
language: c
sudo: true
-script: test/travis-run
+script: tests/travis-run
env:
- ARCH=amd64
- ARCH=i386
diff --git a/Makefile.am b/Makefile.am
index a890ff1..16f57b2 100644
--- a/Makefile.am
+++ b/Makefile.am
@@ -1,5 +1,5 @@
-SUBDIRS = data src test
+SUBDIRS = data src tests
ACLOCAL_AMFLAGS = -I m4
diff --git a/configure.ac b/configure.ac
index ea2c330..6f166f2 100644
--- a/configure.ac
+++ b/configure.ac
@@ -75,5 +75,5 @@ AC_CONFIG_FILES([run],
AC_OUTPUT(Makefile
data/Makefile
src/Makefile
- test/Makefile
+ tests/Makefile
libvirt-dbus.spec)
diff --git a/test/Makefile.am b/tests/Makefile.am
similarity index 100%
rename from test/Makefile.am
rename to tests/Makefile.am
diff --git a/test/conftest.py b/tests/conftest.py
similarity index 100%
rename from test/conftest.py
rename to tests/conftest.py
diff --git a/test/libvirttest.py b/tests/libvirttest.py
similarity index 100%
rename from test/libvirttest.py
rename to tests/libvirttest.py
diff --git a/test/test_connect.py b/tests/test_connect.py
similarity index 100%
rename from test/test_connect.py
rename to tests/test_connect.py
diff --git a/test/test_domain.py b/tests/test_domain.py
similarity index 100%
rename from test/test_domain.py
rename to tests/test_domain.py
diff --git a/test/test_network.py b/tests/test_network.py
similarity index 100%
rename from test/test_network.py
rename to tests/test_network.py
diff --git a/test/travis-run b/tests/travis-run
similarity index 100%
rename from test/travis-run
rename to tests/travis-run
--
2.14.3
7 years, 2 months
[libvirt] [jenkins-ci PATCH] guests: Only attempt to fix intltool-update if it exists
by Andrea Bolognani
If we're running against a freshly-provisioned FreeBSD guest,
intltool won't have been installed yet and attempts to fix it
will fail. Make sure the file exists.
Signed-off-by: Andrea Bolognani <abologna(a)redhat.com>
---
guests/tasks/kludges.yml | 8 ++++++++
1 file changed, 8 insertions(+)
diff --git a/guests/tasks/kludges.yml b/guests/tasks/kludges.yml
index 4dd06a4..0ad61a7 100644
--- a/guests/tasks/kludges.yml
+++ b/guests/tasks/kludges.yml
@@ -52,6 +52,13 @@
# wait for the port to be fixed, we can patch things up ourselves.
#
# See https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=227444
+- name: Look for intltool-update
+ stat:
+ path: /usr/local/bin/intltool-update
+ register: intltoolupdate
+ when:
+ - os_name == 'FreeBSD'
+
- name: Fix intltool-update
replace:
path: /usr/local/bin/intltool-update
@@ -59,3 +66,4 @@
replace: '\1 !~ /\\$\\{?$2}?/;'
when:
- os_name == 'FreeBSD'
+ - intltoolupdate.stat.exists
--
2.14.3
7 years, 2 months
