[libvirt] [RFC PATCH 00/19] Configurable policy for handling deprecated interfaces
by Markus Armbruster
This series is RFC because it's incomplete, and the warning idea in
PATCH 18 is half-baked. I'm soliciting feed back from the management
application crowd: is this going into a useful direction?
The series adresses only deprecated commands and events. Good enough
to demonstrate the ideas, I think. A complete solution should
additionally cover arguments and return values. Feels feasible to me.
New option -compat deprecated-input=<in-policy>,deprecated-output=out-policy
configures the policy. Available input policies:
* accept: Accept deprecated commands with a warning (default)
* reject: Reject deprecated commands
* crash: Crash on deprecated command
Available output policies:
* accept: Emit deprecated events (default)
* hide: Suppress deprecated events
See also last item of
Subject: Minutes of KVM Forum BoF on deprecating stuff
Date: Fri, 26 Oct 2018 16:03:51 +0200
Message-ID: <87mur0ls8o.fsf(a)dusky.pond.sub.org>
https://lists.nongnu.org/archive/html/qemu-devel/2018-10/msg05828.html
Markus Armbruster (19):
tests/test-qmp-cmds: Factor out qmp_dispatch() test helpers
tests/test-qmp-cmds: Check responses more thoroughly
tests/test-qmp-cmds: Simplify test data setup
tests/test-qmp-event: Simplify test data setup
tests/test-qmp-event: Use qobject_is_equal()
tests/test-qmp-event: Check event is actually emitted
qapi: Add feature flags to remaining definitions
qapi: Consistently put @features parameter right after @ifcond
qapi: Inline do_qmp_dispatch() into qmp_dispatch()
qapi: Simplify how qmp_dispatch() deals with QCO_NO_SUCCESS_RESP
qapi: Simplify how qmp_dispatch() gets the request ID
qapi: Replace qmp_dispatch()'s TODO comment by an explanation
qapi: New special feature flag "deprecated"
qemu-options: New -compat to set policy for "funny" interfaces
qapi: Mark deprecated QMP commands with feature 'deprecated'
qapi: Implement -compat deprecated-input=reject for commands
qapi: Implement -compat deprecated-input=crash for commands
qapi: Include a warning in the response to a deprecated command
qapi: Implement -compat deprecated-output=hide for events
docs/devel/qapi-code-gen.txt | 21 +-
tests/qapi-schema/doc-good.texi | 32 ++-
qapi/common.json | 48 ++++
qapi/introspect.json | 28 ++-
qapi/machine.json | 24 +-
qapi/migration.json | 36 ++-
qapi/misc.json | 25 +-
include/qapi/compat-policy.h | 20 ++
include/qapi/qmp/dispatch.h | 1 +
qapi/qmp-dispatch.c | 140 +++++++-----
tests/test-qmp-cmds.c | 216 +++++++++++-------
tests/test-qmp-event.c | 181 ++++-----------
vl.c | 17 ++
qemu-options.hx | 24 ++
scripts/qapi/commands.py | 16 +-
scripts/qapi/doc.py | 16 +-
scripts/qapi/events.py | 16 +-
scripts/qapi/expr.py | 11 +-
scripts/qapi/introspect.py | 41 ++--
scripts/qapi/schema.py | 138 ++++++-----
scripts/qapi/types.py | 8 +-
scripts/qapi/visit.py | 8 +-
tests/Makefile.include | 1 +
tests/qapi-schema/alternate-base.err | 2 +-
tests/qapi-schema/doc-good.json | 18 +-
tests/qapi-schema/doc-good.out | 20 +-
.../qapi-schema/features-deprecated-type.err | 2 +
.../qapi-schema/features-deprecated-type.json | 3 +
.../qapi-schema/features-deprecated-type.out | 0
tests/qapi-schema/qapi-schema-test.json | 31 ++-
tests/qapi-schema/qapi-schema-test.out | 29 ++-
tests/qapi-schema/test-qapi.py | 19 +-
32 files changed, 731 insertions(+), 461 deletions(-)
create mode 100644 include/qapi/compat-policy.h
create mode 100644 tests/qapi-schema/features-deprecated-type.err
create mode 100644 tests/qapi-schema/features-deprecated-type.json
create mode 100644 tests/qapi-schema/features-deprecated-type.out
--
2.21.0
5 years, 2 months
[libvirt] [PATCH] internal: Use g_strcmp0 in STR(N)EQ_NULLABLE
by Peter Krempa
Signed-off-by: Peter Krempa <pkrempa(a)redhat.com>
---
src/internal.h | 6 ++----
1 file changed, 2 insertions(+), 4 deletions(-)
diff --git a/src/internal.h b/src/internal.h
index 5b0a2335f5..0ff9f496ac 100644
--- a/src/internal.h
+++ b/src/internal.h
@@ -86,10 +86,8 @@
#define STRCASEPREFIX(a, b) (c_strncasecmp(a, b, strlen(b)) == 0)
#define STRSKIP(a, b) (STRPREFIX(a, b) ? (a) + strlen(b) : NULL)
-#define STREQ_NULLABLE(a, b) \
- ((a) ? (b) && STREQ((a), (b)) : !(b))
-#define STRNEQ_NULLABLE(a, b) \
- ((a) ? !(b) || STRNEQ((a), (b)) : !!(b))
+#define STREQ_NULLABLE(a, b) (g_strcmp0(a, b) == 0)
+#define STRNEQ_NULLABLE(a, b) (g_strcmp0(a, b) != 0)
#define NUL_TERMINATE(buf) do { (buf)[sizeof(buf)-1] = '\0'; } while (0)
--
2.21.0
5 years, 2 months
[libvirt] [PATCH] tests: domaincaps: Fix build when WITH_QEMU is disabled
by Peter Krempa
doTestQemuInternal and doTestQemu are used only when WITH_QEMU is
enabled.
Signed-off-by: Peter Krempa <pkrempa(a)redhat.com>
---
tests/domaincapstest.c | 5 +++++
1 file changed, 5 insertions(+)
Pushed as a fix for broken build.
diff --git a/tests/domaincapstest.c b/tests/domaincapstest.c
index 0197c61c8a..f77accdb76 100644
--- a/tests/domaincapstest.c
+++ b/tests/domaincapstest.c
@@ -266,6 +266,9 @@ test_virDomainCapsFormat(const void *opaque)
return ret;
}
+
+#if WITH_QEMU
+
static int
doTestQemuInternal(const char *version,
const char *machine,
@@ -349,6 +352,8 @@ doTestQemu(const char *inputDir G_GNUC_UNUSED,
return 0;
}
+#endif
+
static int
mymain(void)
{
--
2.21.0
5 years, 2 months
[libvirt] [PATCH 00/75] Drop virAsprintf() in favor of g_strdup_printf()
by Michal Privoznik
You can find these also on my github:
https://github.com/zippy2/libvirt/tree/virasprintf_void
Good luck.
Michal Prívozník (75):
src: Don't rely on virAsprintf() returning string length
qemu_command: Change logic in qemuVirCommandGet{FDSet,GetDevSet}
access: Drop virAsprintf() and virAsprintfQuiet() retval checking
bhyve: Drop virAsprintf() and virAsprintfQuiet() retval checking
conf: Drop virAsprintf() and virAsprintfQuiet() retval checking
cpu: Drop virAsprintf() and virAsprintfQuiet() retval checking
esx: Drop virAsprintf() and virAsprintfQuiet() retval checking
hyperv: Drop virAsprintf() and virAsprintfQuiet() retval checking
interface: Drop virAsprintf() and virAsprintfQuiet() retval checking
libxl: Drop virAsprintf() and virAsprintfQuiet() retval checking
locking: Drop virAsprintf() and virAsprintfQuiet() retval checking
logging: Drop virAsprintf() and virAsprintfQuiet() retval checking
lxc: Drop virAsprintf() and virAsprintfQuiet() retval checking
network: Drop virAsprintf() and virAsprintfQuiet() retval checking
node_device: Drop virAsprintf() and virAsprintfQuiet() retval checking
nwfilter: Drop virAsprintf() and virAsprintfQuiet() retval checking
openvz: Drop virAsprintf() and virAsprintfQuiet() retval checking
phyp: Drop virAsprintf() and virAsprintfQuiet() retval checking
qemu: Drop virAsprintf() and virAsprintfQuiet() retval checking
remote: Drop virAsprintf() and virAsprintfQuiet() retval checking
rpc: Drop virAsprintf() and virAsprintfQuiet() retval checking
security: Drop virAsprintf() and virAsprintfQuiet() retval checking
storage: Drop virAsprintf() and virAsprintfQuiet() retval checking
test: Drop virAsprintf() and virAsprintfQuiet() retval checking
util: Drop virAsprintf() and virAsprintfQuiet() retval checking
secret: Drop virAsprintf() and virAsprintfQuiet() retval checking
vbox: Drop virAsprintf() and virAsprintfQuiet() retval checking
vmware: Drop virAsprintf() and virAsprintfQuiet() retval checking
vmx: Drop virAsprintf() and virAsprintfQuiet() retval checking
vz: Drop virAsprintf() and virAsprintfQuiet() retval checking
tests: Drop virAsprintf() and virAsprintfQuiet() retval checking
tools: Drop virAsprintf() and virAsprintfQuiet() retval checking
Drop virAsprintf() and virAsprintfQuiet() retval checking
virstring: Make virAsprintf() return no value
Rename virAsprintfQuiet() to virAsprintf()
virstring: Drop virAsprintfQuiet()
access: Use g_strdup_printf() instead of virAsprintf()
bhyve: Use g_strdup_printf() instead of virAsprintf()
conf: Use g_strdup_printf() instead of virAsprintf()
cpu: Use g_strdup_printf() instead of virAsprintf()
esx: Use g_strdup_printf() instead of virAsprintf()
hyperv: Use g_strdup_printf() instead of virAsprintf()
interface: Use g_strdup_printf() instead of virAsprintf()
libxl: Use g_strdup_printf() instead of virAsprintf()
locking: Use g_strdup_printf() instead of virAsprintf()
logging: Use g_strdup_printf() instead of virAsprintf()
lxc: Use g_strdup_printf() instead of virAsprintf()
network: Use g_strdup_printf() instead of virAsprintf()
node_device: Use g_strdup_printf() instead of virAsprintf()
nwfilter: Use g_strdup_printf() instead of virAsprintf()
openvz: Use g_strdup_printf() instead of virAsprintf()
phyp: Use g_strdup_printf() instead of virAsprintf()
qemu: Use g_strdup_printf() instead of virAsprintf()
remote: Use g_strdup_printf() instead of virAsprintf()
rpc: Use g_strdup_printf() instead of virAsprintf()
secret: Use g_strdup_printf() instead of virAsprintf()
security: Use g_strdup_printf() instead of virAsprintf()
storage: Use g_strdup_printf() instead of virAsprintf()
test: Use g_strdup_printf() instead of virAsprintf()
util: Use g_strdup_printf() instead of virAsprintf()
vbox: Use g_strdup_printf() instead of virAsprintf()
vmware: Use g_strdup_printf() instead of virAsprintf()
vmx: Use g_strdup_printf() instead of virAsprintf()
vz: Use g_strdup_printf() instead of virAsprintf()
tests: Use g_strdup_printf() instead of virAsprintf()
tools: Use g_strdup_printf() instead of virAsprintf()
Use g_strdup_printf instead of virAsprintf everywhere
virpcimock: Make @fakerootdir static
Drop virAsprintf()
Rename virVasprintfQuiet() to virVasprintf()
virstring: Drop virVasprintfQuiet()
Drop virVasprintf() retval checking
virstring: Make virVasprintf() return no value
Use g_strdup_vprintf() instead of virVasprintf() everywhere
Drop virVasprintf()
build-aux/syntax-check.mk | 10 +-
docs/hacking.html.in | 13 +-
m4/virt-compile-warnings.m4 | 2 -
src/access/viraccessdriverpolkit.c | 6 +-
src/bhyve/bhyve_capabilities.c | 6 +-
src/bhyve/bhyve_process.c | 16 +-
src/conf/capabilities.c | 3 +-
src/conf/domain_addr.c | 8 +-
src/conf/domain_audit.c | 60 +++-----
src/conf/domain_conf.c | 48 +++---
src/conf/moment_conf.c | 5 +-
src/conf/network_conf.c | 2 +-
src/conf/node_device_util.c | 3 +-
src/conf/snapshot_conf.c | 5 +-
src/conf/storage_conf.c | 6 +-
src/conf/virchrdev.c | 7 +-
src/conf/virnetworkobj.c | 8 +-
src/conf/virnetworkportdef.c | 2 +-
src/conf/virnwfilterbindingobj.c | 2 +-
src/conf/virstorageobj.c | 3 +-
src/cpu/cpu_ppc64.c | 30 ++--
src/cpu/cpu_x86.c | 27 ++--
src/driver.c | 6 +-
src/esx/esx_driver.c | 54 +++----
src/esx/esx_storage_backend_vmfs.c | 67 +++-----
src/esx/esx_stream.c | 11 +-
src/esx/esx_vi.c | 31 ++--
src/hyperv/hyperv_driver.c | 11 +-
src/hyperv/hyperv_wmi.c | 23 +--
src/interface/interface_backend_netcf.c | 7 +-
src/interface/interface_backend_udev.c | 14 +-
src/libvirt-admin.c | 12 +-
src/libvirt_private.syms | 2 -
src/libxl/libxl_conf.c | 34 ++---
src/libxl/libxl_domain.c | 34 ++---
src/libxl/libxl_domain.h | 2 +-
src/libxl/libxl_driver.c | 19 +--
src/libxl/libxl_logger.c | 14 +-
src/libxl/libxl_migration.c | 9 +-
src/libxl/xen_common.c | 11 +-
src/libxl/xen_xl.c | 18 +--
src/libxl/xen_xm.c | 6 +-
src/locking/lock_daemon.c | 17 +--
src/locking/lock_daemon_config.c | 5 +-
src/locking/lock_driver_lockd.c | 9 +-
src/locking/lock_driver_sanlock.c | 12 +-
src/locking/lock_manager.c | 4 +-
src/logging/log_daemon.c | 26 +---
src/logging/log_daemon_config.c | 5 +-
src/logging/log_manager.c | 5 +-
src/lxc/lxc_container.c | 68 ++++-----
src/lxc/lxc_controller.c | 79 ++++------
src/lxc/lxc_driver.c | 15 +-
src/lxc/lxc_fuse.c | 10 +-
src/lxc/lxc_monitor.c | 4 +-
src/lxc/lxc_native.c | 11 +-
src/lxc/lxc_process.c | 35 ++---
src/network/bridge_driver.c | 69 ++++-----
src/network/leaseshelper.c | 6 +-
src/node_device/node_device_driver.c | 3 +-
src/node_device/node_device_hal.c | 7 +-
src/node_device/node_device_udev.c | 11 +-
src/nwfilter/nwfilter_dhcpsnoop.c | 16 +-
src/openvz/openvz_conf.c | 18 +--
src/openvz/openvz_driver.c | 7 +-
src/phyp/phyp_driver.c | 19 +--
src/qemu/qemu_agent.c | 9 +-
src/qemu/qemu_alias.c | 114 +++++++-------
src/qemu/qemu_block.c | 45 +++---
src/qemu/qemu_blockjob.c | 12 +-
src/qemu/qemu_capabilities.c | 14 +-
src/qemu/qemu_checkpoint.c | 11 +-
src/qemu/qemu_command.c | 113 ++++++--------
src/qemu/qemu_conf.c | 141 ++++++-----------
src/qemu/qemu_conf.h | 4 +-
src/qemu/qemu_domain.c | 168 ++++++++-------------
src/qemu/qemu_domain.h | 6 +-
src/qemu/qemu_driver.c | 6 +-
src/qemu/qemu_firmware.c | 3 +-
src/qemu/qemu_hotplug.c | 80 ++++------
src/qemu/qemu_interface.c | 5 +-
src/qemu/qemu_interop_config.c | 9 +-
src/qemu/qemu_migration.c | 33 ++--
src/qemu/qemu_migration_cookie.c | 3 +-
src/qemu/qemu_monitor.c | 33 ++--
src/qemu/qemu_monitor_json.c | 26 +---
src/qemu/qemu_monitor_text.c | 15 +-
src/qemu/qemu_process.c | 74 +++------
src/qemu/qemu_slirp.c | 18 +--
src/qemu/qemu_tpm.c | 19 +--
src/qemu/qemu_vhost_user_gpu.c | 3 +-
src/remote/remote_daemon.c | 37 ++---
src/remote/remote_daemon_config.c | 5 +-
src/remote/remote_daemon_dispatch.c | 20 +--
src/remote/remote_driver.c | 20 +--
src/rpc/virnetclient.c | 23 +--
src/rpc/virnetlibsshsession.c | 17 +--
src/rpc/virnetsocket.c | 3 +-
src/rpc/virnetsshsession.c | 19 +--
src/rpc/virnettlscontext.c | 39 ++---
src/secret/secret_driver.c | 14 +-
src/security/security_apparmor.c | 38 ++---
src/security/security_dac.c | 30 ++--
src/security/security_selinux.c | 59 +++-----
src/security/security_util.c | 14 +-
src/security/virt-aa-helper.c | 73 ++-------
src/storage/storage_backend_disk.c | 7 +-
src/storage/storage_backend_gluster.c | 16 +-
src/storage/storage_backend_iscsi.c | 13 +-
src/storage/storage_backend_iscsi_direct.c | 25 ++-
src/storage/storage_backend_logical.c | 16 +-
src/storage/storage_backend_mpath.c | 9 +-
src/storage/storage_backend_rbd.c | 16 +-
src/storage/storage_backend_scsi.c | 11 +-
src/storage/storage_backend_sheepdog.c | 7 +-
src/storage/storage_backend_vstorage.c | 6 +-
src/storage/storage_backend_zfs.c | 13 +-
src/storage/storage_driver.c | 17 +--
src/storage/storage_file_gluster.c | 10 +-
src/storage/storage_util.c | 87 ++++-------
src/test/test_driver.c | 35 ++---
src/util/viraudit.c | 6 +-
src/util/virauth.c | 17 +--
src/util/virauthconfig.c | 9 +-
src/util/vircgroup.c | 39 ++---
src/util/vircgroupv1.c | 102 +++++--------
src/util/vircgroupv2.c | 97 ++++--------
src/util/vircommand.c | 12 +-
src/util/virconf.c | 10 +-
src/util/virdevmapper.c | 7 +-
src/util/virdnsmasq.c | 39 ++---
src/util/virebtables.c | 5 +-
src/util/virerror.c | 9 +-
src/util/virfile.c | 77 +++-------
src/util/virfirewall.c | 8 +-
src/util/virhostcpu.c | 11 +-
src/util/virhostdev.c | 3 +-
src/util/virhostmem.c | 21 +--
src/util/viriptables.c | 20 +--
src/util/viriscsi.c | 6 +-
src/util/virjson.c | 12 +-
src/util/virkmod.c | 3 +-
src/util/virlockspace.c | 2 +-
src/util/virlog.c | 115 ++++++--------
src/util/virlog.h | 2 +-
src/util/virmacmap.c | 2 +-
src/util/virmdev.c | 11 +-
src/util/virnetdev.c | 40 ++---
src/util/virnetdevbandwidth.c | 61 +++-----
src/util/virnetdevbridge.c | 16 +-
src/util/virnetdevip.c | 5 +-
src/util/virnetdevmacvlan.c | 6 +-
src/util/virnetdevopenvswitch.c | 18 +--
src/util/virnetdevtap.c | 6 +-
src/util/virnetdevveth.c | 9 +-
src/util/virnuma.c | 29 ++--
src/util/virpci.c | 85 ++++-------
src/util/virpidfile.c | 9 +-
src/util/virprocess.c | 24 ++-
src/util/virqemu.c | 6 +-
src/util/virrandom.c | 5 +-
src/util/virresctrl.c | 21 +--
src/util/virrotatingfile.c | 12 +-
src/util/virscsi.c | 31 ++--
src/util/virscsihost.c | 18 +--
src/util/virscsivhost.c | 4 +-
src/util/virsocketaddr.c | 19 +--
src/util/virstoragefile.c | 14 +-
src/util/virstring.c | 50 +-----
src/util/virstring.h | 50 +-----
src/util/virsystemd.c | 3 +-
src/util/virtpm.c | 44 +++---
src/util/virtypedparam.c | 12 +-
src/util/viruri.c | 3 +-
src/util/virusb.c | 19 +--
src/util/virutil.c | 18 +--
src/util/virvhba.c | 52 +++----
src/vbox/vbox_MSCOMGlue.c | 4 +-
src/vbox/vbox_XPCOMCGlue.c | 3 +-
src/vbox/vbox_common.c | 62 ++++----
src/vbox/vbox_network.c | 13 +-
src/vbox/vbox_snapshot_conf.c | 18 +--
src/vbox/vbox_storage.c | 5 +-
src/vbox/vbox_tmpl.c | 32 ++--
src/vmware/vmware_conf.c | 31 ++--
src/vmware/vmware_conf.h | 4 +-
src/vmx/vmx.c | 13 +-
src/vz/vz_driver.c | 3 +-
src/vz/vz_sdk.c | 46 ++----
tests/bhyveargv2xmltest.c | 10 +-
tests/bhyvexml2argvtest.c | 18 +--
tests/bhyvexml2xmltest.c | 10 +-
tests/commandtest.c | 27 ++--
tests/cputest.c | 95 +++++-------
tests/domaincapstest.c | 26 ++--
tests/domainconftest.c | 5 +-
tests/eventtest.c | 4 +-
tests/fchosttest.c | 7 +-
tests/fdstreamtest.c | 6 +-
tests/genericxml2xmltest.c | 10 +-
tests/interfacexml2xmltest.c | 5 +-
tests/libxlxml2domconfigtest.c | 8 +-
tests/lxcconf2xmltest.c | 20 +--
tests/lxcxml2xmltest.c | 10 +-
tests/networkxml2conftest.c | 9 +-
tests/networkxml2firewalltest.c | 16 +-
tests/networkxml2xmltest.c | 9 +-
tests/networkxml2xmlupdatetest.c | 15 +-
tests/nodedevxml2xmltest.c | 5 +-
tests/nssmock.c | 9 +-
tests/nwfilterxml2firewalltest.c | 15 +-
tests/nwfilterxml2xmltest.c | 9 +-
tests/openvzutilstest.c | 6 +-
tests/qemuagenttest.c | 4 +-
tests/qemublocktest.c | 34 ++---
tests/qemucapabilitiestest.c | 20 +--
tests/qemucaps2xmltest.c | 15 +-
tests/qemudomaincheckpointxml2xmltest.c | 5 +-
tests/qemudomainsnapshotxml2xmltest.c | 5 +-
tests/qemufirmwaretest.c | 4 +-
tests/qemuhotplugtest.c | 29 ++--
tests/qemumemlocktest.c | 5 +-
tests/qemumigparamstest.c | 23 ++-
tests/qemumonitorjsontest.c | 56 +++----
tests/qemumonitortestutils.c | 42 ++----
tests/qemusecuritymock.c | 5 +-
tests/qemusecuritytest.c | 3 +-
tests/qemuvhostusertest.c | 4 +-
tests/qemuxml2argvtest.c | 35 ++---
tests/qemuxml2xmltest.c | 41 ++---
tests/scsihosttest.c | 26 +---
tests/secretxml2xmltest.c | 14 +-
tests/securityselinuxlabeltest.c | 23 +--
tests/storagebackendsheepdogtest.c | 11 +-
tests/storagepoolcapstest.c | 6 +-
tests/storagepoolxml2argvtest.c | 11 +-
tests/storagepoolxml2xmltest.c | 9 +-
tests/storagevolxml2argvtest.c | 35 ++---
tests/storagevolxml2xmltest.c | 13 +-
tests/sysinfotest.c | 18 +--
tests/testutils.c | 17 +--
tests/testutils.h | 5 +-
tests/testutilsqemu.c | 11 +-
tests/vboxsnapshotxmltest.c | 5 +-
tests/virbuftest.c | 7 +-
tests/vircaps2xmltest.c | 15 +-
tests/vircgroupmock.c | 11 +-
tests/vircgrouptest.c | 5 +-
tests/virconftest.c | 7 +-
tests/virfilemock.c | 2 +-
tests/virfiletest.c | 3 +-
tests/virfilewrapper.c | 4 +-
tests/virhostcputest.c | 38 ++---
tests/virjsontest.c | 27 ++--
tests/virmacmaptest.c | 12 +-
tests/virnetdaemontest.c | 10 +-
tests/virnetdevmock.c | 8 +-
tests/virnetdevopenvswitchtest.c | 5 +-
tests/virnetsockettest.c | 6 +-
tests/virnetworkportxml2xmltest.c | 6 +-
tests/virnumamock.c | 3 +-
tests/virnwfilterbindingxml2xmltest.c | 7 +-
tests/virpcimock.c | 103 +++++--------
tests/virpcitest.c | 16 +-
tests/virrandommock.c | 5 +-
tests/virresctrltest.c | 15 +-
tests/virschematest.c | 19 +--
tests/virscsitest.c | 11 +-
tests/virshtest.c | 5 +-
tests/virstoragetest.c | 63 ++++----
tests/virtestmock.c | 8 +-
tests/virusbmock.c | 9 +-
tests/virusbtest.c | 6 +-
tests/vmwarevertest.c | 4 +-
tests/vmx2xmltest.c | 18 +--
tests/xlconfigtest.c | 8 +-
tests/xmconfigtest.c | 8 +-
tests/xml2vmxtest.c | 16 +-
tools/virsh-completer-domain.c | 7 +-
tools/virsh-completer-host.c | 13 +-
tools/virsh-completer.c | 6 +-
tools/virsh-domain-monitor.c | 21 ++-
tools/virsh-domain.c | 34 ++---
tools/virsh-host.c | 3 +-
tools/virsh-interface.c | 11 +-
tools/virsh-network.c | 3 +-
tools/virsh-pool.c | 15 +-
tools/virsh-volume.c | 9 +-
tools/virt-admin.c | 6 +-
tools/virt-host-validate-common.c | 10 +-
tools/vsh.c | 53 ++-----
291 files changed, 2096 insertions(+), 3822 deletions(-)
--
2.21.0
5 years, 2 months
[libvirt] [PATCH 00/12] tests/domaincaps: Use testQemuCapsIterate()
by Andrea Bolognani
I finally got annoyed enough by the fact that this test program,
unlike qemucapabilitiestest and qemucaps2xmltest, is not
self-maintaining.
The series, as posted to the list, is heavily snipped: grab the full
version with
$ git fetch https://gitlab.com/abologna/libvirt.git domaincapstest
Andrea Bolognani (12):
tests/qemucapabilities: Separate inputDir and outputDir
tests: testQemuCapsIterate: Don't ignore malformed file names
tests: testQemuCapsIterate: Validate suffix
tests: testQemuCapsIterate: Pass inputDir to callback
tests: testQemuCapsIterate: Pass suffix to callback
tests: testQemuCapsIterate: Pass prefix and version to callback
tests: Rename domaincapsschemadata/ -> domaincapsdata/
tests/domaincaps: Don't mess with test name
tests/domaincaps: Move most of DO_TEST_QEMU() into a function
tests/domaincaps: Don't require redundant information
tests/domaincaps: Make test matrix programmatic
tests/domaincaps: Use testQemuCapsIterate()
tests/Makefile.am | 2 +-
.../bhyve_basic.x86_64.xml | 0
.../bhyve_fbuf.x86_64.xml | 0
.../bhyve_uefi.x86_64.xml | 0
.../empty.xml | 0
.../libxl-xenfv.xml | 0
.../libxl-xenpv.xml | 0
.../qemu_1.5.3-q35.x86_64.xml} | 4 +-
.../qemu_1.5.3-tcg.x86_64.xml} | 6 +-
.../qemu_1.5.3.x86_64.xml} | 2 +-
.../qemu_1.6.0-q35.x86_64.xml} | 4 +-
.../qemu_1.6.0-tcg.x86_64.xml} | 6 +-
.../qemu_1.6.0.x86_64.xml} | 2 +-
.../qemu_1.7.0-q35.x86_64.xml} | 4 +-
.../qemu_1.7.0-tcg.x86_64.xml} | 4 +-
.../qemu_1.7.0.x86_64.xml | 0
.../qemu_2.1.1-q35.x86_64.xml} | 7 +-
.../qemu_2.1.1-tcg.x86_64.xml} | 9 +-
.../qemu_2.1.1.x86_64.xml} | 5 +-
.../qemu_2.10.0-q35.x86_64.xml} | 21 +-
.../qemu_2.10.0-tcg.x86_64.xml} | 3 +-
.../qemu_2.10.0-virt.aarch64.xml} | 3 +-
.../qemu_2.10.0.aarch64.xml} | 1 +
.../qemu_2.10.0.ppc64.xml} | 2 +-
.../qemu_2.10.0.s390x.xml} | 110 ++++-----
.../qemu_2.10.0.x86_64.xml} | 21 +-
.../qemu_2.11.0-q35.x86_64.xml} | 7 +-
.../qemu_2.11.0-tcg.x86_64.xml} | 34 ++-
.../qemu_2.11.0.s390x.xml} | 11 +-
.../qemu_2.11.0.x86_64.xml} | 7 +-
.../qemu_2.12.0-q35.x86_64.xml} | 6 +-
.../qemu_2.12.0-tcg.x86_64.xml} | 50 ++--
.../qemu_2.12.0-virt.aarch64.xml | 0
.../qemu_2.12.0.aarch64.xml} | 5 +-
.../qemu_2.12.0.ppc64.xml | 0
.../qemu_2.12.0.s390x.xml | 0
.../qemu_2.12.0.x86_64.xml | 0
.../qemu_2.4.0-q35.x86_64.xml} | 4 +-
.../qemu_2.4.0-tcg.x86_64.xml} | 6 +-
.../qemu_2.4.0.x86_64.xml} | 2 +-
.../qemu_2.5.0-q35.x86_64.xml} | 4 +-
.../qemu_2.5.0-tcg.x86_64.xml} | 6 +-
.../qemu_2.5.0.x86_64.xml} | 2 +-
.../qemu_2.6.0-q35.x86_64.xml} | 4 +-
.../qemu_2.6.0-tcg.x86_64.xml} | 4 +-
.../qemu_2.6.0-virt.aarch64.xml | 0
.../qemu_2.6.0.aarch64.xml | 0
.../qemu_2.6.0.ppc64.xml | 0
.../qemu_2.6.0.x86_64.xml | 0
.../qemu_2.7.0-q35.x86_64.xml} | 5 +-
.../qemu_2.7.0-tcg.x86_64.xml} | 7 +-
.../qemu_2.7.0.s390x.xml | 0
.../qemu_2.7.0.x86_64.xml} | 3 +-
.../qemu_2.8.0-q35.x86_64.xml} | 6 +-
.../qemu_2.8.0-tcg.x86_64.xml | 0
.../qemu_2.8.0.s390x.xml | 0
.../qemu_2.8.0.x86_64.xml | 0
.../qemu_2.9.0-q35.x86_64.xml | 0
.../qemu_2.9.0-tcg.x86_64.xml | 0
.../qemu_2.9.0.ppc64.xml} | 2 +-
.../qemu_2.9.0.s390x.xml} | 7 +-
.../qemu_2.9.0.x86_64.xml | 0
.../qemu_3.0.0-q35.x86_64.xml} | 11 +-
.../qemu_3.0.0-tcg.x86_64.xml} | 52 +++--
.../qemu_3.0.0.ppc64.xml} | 3 +-
.../qemu_3.0.0.s390x.xml | 0
.../qemu_3.0.0.x86_64.xml} | 7 +-
.../qemu_3.1.0-q35.x86_64.xml} | 6 +-
.../qemu_3.1.0-tcg.x86_64.xml} | 55 +++--
.../qemu_3.1.0.ppc64.xml} | 3 +-
.../qemu_3.1.0.x86_64.xml | 0
.../qemu_4.0.0-q35.x86_64.xml} | 6 +-
.../qemu_4.0.0-tcg.x86_64.xml} | 55 +++--
.../qemu_4.0.0-virt.aarch64.xml} | 16 +-
.../qemu_4.0.0.aarch64.xml} | 2 +-
.../qemu_4.0.0.ppc64.xml} | 2 +-
.../qemu_4.0.0.s390x.xml | 0
.../qemu_4.0.0.x86_64.xml | 0
.../qemu_4.1.0-q35.x86_64.xml} | 6 +-
.../qemu_4.1.0-tcg.x86_64.xml} | 59 +++--
.../qemu_4.1.0.x86_64.xml | 0
.../qemu_4.2.0-q35.x86_64.xml} | 6 +-
.../qemu_4.2.0-tcg.x86_64.xml} | 59 +++--
.../qemu_4.2.0-virt.aarch64.xml} | 14 +-
.../qemu_4.2.0.aarch64.xml | 0
.../qemu_4.2.0.ppc64.xml | 0
.../qemu_4.2.0.x86_64.xml | 0
tests/domaincapstest.c | 221 ++++++++----------
.../caps_1.5.3.x86_64.xml | 2 +-
.../caps_1.6.0.x86_64.xml | 2 +-
.../caps_1.7.0.x86_64.xml | 2 +-
.../caps_2.1.1.x86_64.xml | 2 +-
.../caps_2.10.0.aarch64.xml | 2 +-
.../caps_2.10.0.ppc64.xml | 2 +-
.../caps_2.10.0.s390x.xml | 2 +-
.../caps_2.10.0.x86_64.xml | 2 +-
.../caps_2.11.0.s390x.xml | 2 +-
.../caps_2.11.0.x86_64.xml | 2 +-
.../caps_2.12.0.aarch64.xml | 2 +-
.../caps_2.12.0.ppc64.xml | 2 +-
.../caps_2.12.0.s390x.xml | 2 +-
.../caps_2.12.0.x86_64.xml | 2 +-
.../caps_2.4.0.x86_64.xml | 2 +-
.../caps_2.5.0.x86_64.xml | 2 +-
.../caps_2.6.0.aarch64.xml | 2 +-
.../qemucapabilitiesdata/caps_2.6.0.ppc64.xml | 2 +-
.../caps_2.6.0.x86_64.xml | 2 +-
.../qemucapabilitiesdata/caps_2.7.0.s390x.xml | 2 +-
.../caps_2.7.0.x86_64.xml | 2 +-
.../qemucapabilitiesdata/caps_2.8.0.s390x.xml | 2 +-
.../caps_2.8.0.x86_64.xml | 2 +-
.../qemucapabilitiesdata/caps_2.9.0.ppc64.xml | 2 +-
.../qemucapabilitiesdata/caps_2.9.0.s390x.xml | 2 +-
.../caps_2.9.0.x86_64.xml | 2 +-
.../qemucapabilitiesdata/caps_3.0.0.ppc64.xml | 2 +-
.../qemucapabilitiesdata/caps_3.0.0.s390x.xml | 2 +-
.../caps_3.0.0.x86_64.xml | 2 +-
.../qemucapabilitiesdata/caps_3.1.0.ppc64.xml | 2 +-
.../caps_3.1.0.x86_64.xml | 2 +-
.../caps_4.0.0.aarch64.xml | 2 +-
.../qemucapabilitiesdata/caps_4.0.0.ppc64.xml | 2 +-
.../qemucapabilitiesdata/caps_4.0.0.s390x.xml | 2 +-
.../caps_4.0.0.x86_64.xml | 2 +-
.../caps_4.1.0.x86_64.xml | 2 +-
.../caps_4.2.0.aarch64.xml | 2 +-
.../qemucapabilitiesdata/caps_4.2.0.ppc64.xml | 2 +-
.../caps_4.2.0.x86_64.xml | 2 +-
tests/qemucapabilitiestest.c | 40 ++--
tests/qemucaps2xmltest.c | 22 +-
tests/testutilsqemu.c | 41 +++-
tests/testutilsqemu.h | 5 +-
tests/virschematest.c | 2 +-
132 files changed, 645 insertions(+), 524 deletions(-)
rename tests/{domaincapsschemadata => domaincapsdata}/bhyve_basic.x86_64.xml (100%)
rename tests/{domaincapsschemadata => domaincapsdata}/bhyve_fbuf.x86_64.xml (100%)
rename tests/{domaincapsschemadata => domaincapsdata}/bhyve_uefi.x86_64.xml (100%)
rename tests/{domaincapsschemadata => domaincapsdata}/empty.xml (100%)
rename tests/{domaincapsschemadata => domaincapsdata}/libxl-xenfv.xml (100%)
rename tests/{domaincapsschemadata => domaincapsdata}/libxl-xenpv.xml (100%)
copy tests/{domaincapsschemadata/qemu_1.7.0.x86_64.xml => domaincapsdata/qemu_1.5.3-q35.x86_64.xml} (98%)
copy tests/{domaincapsschemadata/qemu_1.7.0.x86_64.xml => domaincapsdata/qemu_1.5.3-tcg.x86_64.xml} (97%)
copy tests/{domaincapsschemadata/qemu_1.7.0.x86_64.xml => domaincapsdata/qemu_1.5.3.x86_64.xml} (99%)
copy tests/{domaincapsschemadata/qemu_1.7.0.x86_64.xml => domaincapsdata/qemu_1.6.0-q35.x86_64.xml} (98%)
copy tests/{domaincapsschemadata/qemu_1.7.0.x86_64.xml => domaincapsdata/qemu_1.6.0-tcg.x86_64.xml} (97%)
copy tests/{domaincapsschemadata/qemu_1.7.0.x86_64.xml => domaincapsdata/qemu_1.6.0.x86_64.xml} (99%)
copy tests/{domaincapsschemadata/qemu_1.7.0.x86_64.xml => domaincapsdata/qemu_1.7.0-q35.x86_64.xml} (98%)
copy tests/{domaincapsschemadata/qemu_1.7.0.x86_64.xml => domaincapsdata/qemu_1.7.0-tcg.x86_64.xml} (98%)
copy tests/{domaincapsschemadata => domaincapsdata}/qemu_1.7.0.x86_64.xml (100%)
copy tests/{domaincapsschemadata/qemu_1.7.0.x86_64.xml => domaincapsdata/qemu_2.1.1-q35.x86_64.xml} (96%)
copy tests/{domaincapsschemadata/qemu_1.7.0.x86_64.xml => domaincapsdata/qemu_2.1.1-tcg.x86_64.xml} (95%)
rename tests/{domaincapsschemadata/qemu_1.7.0.x86_64.xml => domaincapsdata/qemu_2.1.1.x86_64.xml} (97%)
copy tests/{domaincapsschemadata/qemu_2.9.0-q35.x86_64.xml => domaincapsdata/qemu_2.10.0-q35.x86_64.xml} (87%)
copy tests/{domaincapsschemadata/qemu_2.9.0-tcg.x86_64.xml => domaincapsdata/qemu_2.10.0-tcg.x86_64.xml} (98%)
copy tests/{domaincapsschemadata/qemu_2.6.0-virt.aarch64.xml => domaincapsdata/qemu_2.10.0-virt.aarch64.xml} (98%)
copy tests/{domaincapsschemadata/qemu_2.6.0.aarch64.xml => domaincapsdata/qemu_2.10.0.aarch64.xml} (98%)
copy tests/{domaincapsschemadata/qemu_2.12.0.ppc64.xml => domaincapsdata/qemu_2.10.0.ppc64.xml} (98%)
copy tests/{domaincapsschemadata/qemu_2.12.0.s390x.xml => domaincapsdata/qemu_2.10.0.s390x.xml} (96%)
copy tests/{domaincapsschemadata/qemu_2.9.0.x86_64.xml => domaincapsdata/qemu_2.10.0.x86_64.xml} (87%)
copy tests/{domaincapsschemadata/qemu_2.9.0-q35.x86_64.xml => domaincapsdata/qemu_2.11.0-q35.x86_64.xml} (95%)
copy tests/{domaincapsschemadata/qemu_2.9.0-tcg.x86_64.xml => domaincapsdata/qemu_2.11.0-tcg.x86_64.xml} (84%)
copy tests/{domaincapsschemadata/qemu_2.12.0.s390x.xml => domaincapsdata/qemu_2.11.0.s390x.xml} (97%)
copy tests/{domaincapsschemadata/qemu_2.9.0.x86_64.xml => domaincapsdata/qemu_2.11.0.x86_64.xml} (95%)
copy tests/{domaincapsschemadata/qemu_2.12.0.x86_64.xml => domaincapsdata/qemu_2.12.0-q35.x86_64.xml} (98%)
copy tests/{domaincapsschemadata/qemu_2.9.0-tcg.x86_64.xml => domaincapsdata/qemu_2.12.0-tcg.x86_64.xml} (78%)
rename tests/{domaincapsschemadata => domaincapsdata}/qemu_2.12.0-virt.aarch64.xml (100%)
copy tests/{domaincapsschemadata/qemu_2.6.0.aarch64.xml => domaincapsdata/qemu_2.12.0.aarch64.xml} (95%)
copy tests/{domaincapsschemadata => domaincapsdata}/qemu_2.12.0.ppc64.xml (100%)
rename tests/{domaincapsschemadata => domaincapsdata}/qemu_2.12.0.s390x.xml (100%)
rename tests/{domaincapsschemadata => domaincapsdata}/qemu_2.12.0.x86_64.xml (100%)
copy tests/{domaincapsschemadata/qemu_2.6.0.x86_64.xml => domaincapsdata/qemu_2.4.0-q35.x86_64.xml} (98%)
copy tests/{domaincapsschemadata/qemu_2.6.0.x86_64.xml => domaincapsdata/qemu_2.4.0-tcg.x86_64.xml} (97%)
copy tests/{domaincapsschemadata/qemu_2.6.0.x86_64.xml => domaincapsdata/qemu_2.4.0.x86_64.xml} (99%)
copy tests/{domaincapsschemadata/qemu_2.6.0.x86_64.xml => domaincapsdata/qemu_2.5.0-q35.x86_64.xml} (98%)
copy tests/{domaincapsschemadata/qemu_2.6.0.x86_64.xml => domaincapsdata/qemu_2.5.0-tcg.x86_64.xml} (97%)
copy tests/{domaincapsschemadata/qemu_2.6.0.x86_64.xml => domaincapsdata/qemu_2.5.0.x86_64.xml} (99%)
copy tests/{domaincapsschemadata/qemu_2.6.0.x86_64.xml => domaincapsdata/qemu_2.6.0-q35.x86_64.xml} (98%)
copy tests/{domaincapsschemadata/qemu_2.6.0.x86_64.xml => domaincapsdata/qemu_2.6.0-tcg.x86_64.xml} (98%)
rename tests/{domaincapsschemadata => domaincapsdata}/qemu_2.6.0-virt.aarch64.xml (100%)
rename tests/{domaincapsschemadata => domaincapsdata}/qemu_2.6.0.aarch64.xml (100%)
rename tests/{domaincapsschemadata => domaincapsdata}/qemu_2.6.0.ppc64.xml (100%)
copy tests/{domaincapsschemadata => domaincapsdata}/qemu_2.6.0.x86_64.xml (100%)
copy tests/{domaincapsschemadata/qemu_2.6.0.x86_64.xml => domaincapsdata/qemu_2.7.0-q35.x86_64.xml} (97%)
copy tests/{domaincapsschemadata/qemu_2.6.0.x86_64.xml => domaincapsdata/qemu_2.7.0-tcg.x86_64.xml} (96%)
rename tests/{domaincapsschemadata => domaincapsdata}/qemu_2.7.0.s390x.xml (100%)
rename tests/{domaincapsschemadata/qemu_2.6.0.x86_64.xml => domaincapsdata/qemu_2.7.0.x86_64.xml} (97%)
copy tests/{domaincapsschemadata/qemu_2.8.0.x86_64.xml => domaincapsdata/qemu_2.8.0-q35.x86_64.xml} (98%)
rename tests/{domaincapsschemadata => domaincapsdata}/qemu_2.8.0-tcg.x86_64.xml (100%)
copy tests/{domaincapsschemadata => domaincapsdata}/qemu_2.8.0.s390x.xml (100%)
rename tests/{domaincapsschemadata => domaincapsdata}/qemu_2.8.0.x86_64.xml (100%)
rename tests/{domaincapsschemadata => domaincapsdata}/qemu_2.9.0-q35.x86_64.xml (100%)
copy tests/{domaincapsschemadata => domaincapsdata}/qemu_2.9.0-tcg.x86_64.xml (100%)
copy tests/{domaincapsschemadata/qemu_2.12.0.ppc64.xml => domaincapsdata/qemu_2.9.0.ppc64.xml} (98%)
rename tests/{domaincapsschemadata/qemu_2.8.0.s390x.xml => domaincapsdata/qemu_2.9.0.s390x.xml} (97%)
rename tests/{domaincapsschemadata => domaincapsdata}/qemu_2.9.0.x86_64.xml (100%)
copy tests/{domaincapsschemadata/qemu_4.0.0.x86_64.xml => domaincapsdata/qemu_3.0.0-q35.x86_64.xml} (95%)
copy tests/{domaincapsschemadata/qemu_2.9.0-tcg.x86_64.xml => domaincapsdata/qemu_3.0.0-tcg.x86_64.xml} (78%)
rename tests/{domaincapsschemadata/qemu_2.12.0.ppc64.xml => domaincapsdata/qemu_3.0.0.ppc64.xml} (97%)
rename tests/{domaincapsschemadata => domaincapsdata}/qemu_3.0.0.s390x.xml (100%)
copy tests/{domaincapsschemadata/qemu_4.0.0.x86_64.xml => domaincapsdata/qemu_3.0.0.x86_64.xml} (95%)
copy tests/{domaincapsschemadata/qemu_3.1.0.x86_64.xml => domaincapsdata/qemu_3.1.0-q35.x86_64.xml} (98%)
copy tests/{domaincapsschemadata/qemu_2.9.0-tcg.x86_64.xml => domaincapsdata/qemu_3.1.0-tcg.x86_64.xml} (77%)
copy tests/{domaincapsschemadata/qemu_4.2.0.ppc64.xml => domaincapsdata/qemu_3.1.0.ppc64.xml} (97%)
rename tests/{domaincapsschemadata => domaincapsdata}/qemu_3.1.0.x86_64.xml (100%)
copy tests/{domaincapsschemadata/qemu_4.0.0.x86_64.xml => domaincapsdata/qemu_4.0.0-q35.x86_64.xml} (98%)
copy tests/{domaincapsschemadata/qemu_2.9.0-tcg.x86_64.xml => domaincapsdata/qemu_4.0.0-tcg.x86_64.xml} (77%)
copy tests/{domaincapsschemadata/qemu_4.2.0.aarch64.xml => domaincapsdata/qemu_4.0.0-virt.aarch64.xml} (95%)
copy tests/{domaincapsschemadata/qemu_4.2.0.aarch64.xml => domaincapsdata/qemu_4.0.0.aarch64.xml} (100%)
copy tests/{domaincapsschemadata/qemu_4.2.0.ppc64.xml => domaincapsdata/qemu_4.0.0.ppc64.xml} (98%)
rename tests/{domaincapsschemadata => domaincapsdata}/qemu_4.0.0.s390x.xml (100%)
rename tests/{domaincapsschemadata => domaincapsdata}/qemu_4.0.0.x86_64.xml (100%)
copy tests/{domaincapsschemadata/qemu_4.1.0.x86_64.xml => domaincapsdata/qemu_4.1.0-q35.x86_64.xml} (98%)
copy tests/{domaincapsschemadata/qemu_2.9.0-tcg.x86_64.xml => domaincapsdata/qemu_4.1.0-tcg.x86_64.xml} (75%)
copy tests/{domaincapsschemadata => domaincapsdata}/qemu_4.1.0.x86_64.xml (100%)
rename tests/{domaincapsschemadata/qemu_4.1.0.x86_64.xml => domaincapsdata/qemu_4.2.0-q35.x86_64.xml} (98%)
rename tests/{domaincapsschemadata/qemu_2.9.0-tcg.x86_64.xml => domaincapsdata/qemu_4.2.0-tcg.x86_64.xml} (75%)
copy tests/{domaincapsschemadata/qemu_4.2.0.aarch64.xml => domaincapsdata/qemu_4.2.0-virt.aarch64.xml} (95%)
rename tests/{domaincapsschemadata => domaincapsdata}/qemu_4.2.0.aarch64.xml (100%)
rename tests/{domaincapsschemadata => domaincapsdata}/qemu_4.2.0.ppc64.xml (100%)
rename tests/{domaincapsschemadata => domaincapsdata}/qemu_4.2.0.x86_64.xml (100%)
--
2.21.0
5 years, 2 months
[libvirt] [PATCH 00/19] util: buffer: Use glib's GString internally
by Peter Krempa
Converting to glib's helpers allows us to greatly simplify the error
handling from virBuffer.
Peter Krempa (19):
util: buffer: Simplify convoluted condition
util: buffer: Use 'cleanup' as label name in virBufferAddBuffer
util: buffer: Simplify escape buffer allocations
virsh: Reimplement _vshCalloc using g_malloc0_n
util: alloc: drop xalloc_oversized macro
util: buffer: Don't treat missing truncation in virBufferTrim as usage
error
util: buffer: Simplify handling of indent overflows
tests: virbuffer: Drop 'infinite loop' tests
tests: virbuffer: Remove unused test data struct
util: buffer: Split getting of effective indent out of
virBufferGetIndent
util: sysinfo: Use virXMLFormatElement and infrastructure in
virSysinfoFormat
util: virbuffer: Remove @dynamic from virBufferGetIndent
util: buffer: Properly URLencode strings
util: buffer: Encode URIs with upper case hex characters
util: buffer: Reimplement virBuffer internals using glib's GString
util: buffer: Remove error handling internals
util: buffer: Remove virBufferError
util: buffer: Remove virBufferCheckError
util: xml: Make virXMLFormatElement void
src/bhyve/bhyve_command.c | 3 -
src/bhyve/bhyve_driver.c | 5 -
src/conf/capabilities.c | 16 -
src/conf/checkpoint_conf.c | 3 -
src/conf/cpu_conf.c | 7 -
src/conf/domain_addr.c | 2 -
src/conf/domain_capabilities.c | 1 -
src/conf/domain_conf.c | 154 ++++-----
src/conf/interface_conf.c | 3 -
src/conf/network_conf.c | 5 +-
src/conf/node_device_conf.c | 3 -
src/conf/nwfilter_conf.c | 9 -
src/conf/secret_conf.c | 3 -
src/conf/snapshot_conf.c | 3 -
src/conf/storage_conf.c | 12 -
src/conf/virnetworkobj.c | 3 -
src/conf/virnetworkportdef.c | 3 -
src/conf/virnwfilterbindingdef.c | 3 -
src/conf/virnwfilterbindingobj.c | 3 -
src/conf/virsavecookie.c | 3 -
src/cpu/cpu_map.c | 2 -
src/cpu/cpu_x86.c | 10 -
src/esx/esx_driver.c | 12 -
src/esx/esx_util.c | 6 -
src/esx/esx_vi.c | 15 -
src/esx/esx_vi_methods.c | 3 -
src/hyperv/hyperv_driver.c | 3 -
src/hyperv/hyperv_wmi.c | 10 -
src/libvirt_private.syms | 3 +-
src/libxl/libxl_conf.c | 5 -
src/libxl/libxl_driver.c | 2 -
src/libxl/libxl_migration.c | 3 -
src/libxl/xen_common.c | 13 -
src/libxl/xen_xl.c | 9 -
src/libxl/xen_xm.c | 3 -
src/locking/lock_driver_sanlock.c | 3 -
src/lxc/lxc_container.c | 3 -
src/lxc/lxc_controller.c | 3 -
src/lxc/lxc_driver.c | 2 -
src/lxc/lxc_fuse.c | 4 -
src/network/bridge_driver.c | 6 -
src/node_device/node_device_udev.c | 3 -
src/nwfilter/nwfilter_ebiptables_driver.c | 8 -
src/nwfilter/nwfilter_gentech_driver.c | 6 -
src/nwfilter/nwfilter_learnipaddr.c | 5 -
src/phyp/phyp_driver.c | 4 -
src/qemu/qemu_capabilities.c | 3 +-
src/qemu/qemu_command.c | 130 --------
src/qemu/qemu_domain.c | 28 +-
src/qemu/qemu_driver.c | 2 -
src/qemu/qemu_migration_cookie.c | 3 -
src/qemu/qemu_monitor_json.c | 3 -
src/rpc/virnetlibsshsession.c | 6 -
src/rpc/virnetsocket.c | 8 -
src/rpc/virnetsshsession.c | 6 -
src/security/virt-aa-helper.c | 11 -
src/storage/storage_backend_rbd.c | 6 -
src/storage/storage_util.c | 13 -
src/util/viralloc.h | 19 --
src/util/virbitmap.c | 7 -
src/util/virbuffer.c | 361 +++++++---------------
src/util/virbuffer.h | 30 +-
src/util/vircommand.c | 11 +-
src/util/virconf.c | 6 -
src/util/virdnsmasq.c | 6 -
src/util/virfilecache.c | 3 -
src/util/virlog.c | 8 -
src/util/virnetdevopenvswitch.c | 3 -
src/util/virpidfile.c | 7 -
src/util/virqemu.c | 3 -
src/util/virresctrl.c | 4 +-
src/util/virstoragefile.c | 3 -
src/util/virstring.c | 5 -
src/util/virsysinfo.c | 24 +-
src/util/virsystemd.c | 6 -
src/util/viruri.c | 3 -
src/util/virxml.c | 19 +-
src/util/virxml.h | 5 +-
src/vmx/vmx.c | 3 -
src/vz/vz_driver.c | 5 -
tests/cputest.c | 4 -
tests/networkxml2firewalltest.c | 3 -
tests/nwfilterebiptablestest.c | 21 --
tests/nwfilterxml2firewalltest.c | 3 -
tests/qemublocktest.c | 6 -
tests/qemumonitorjsontest.c | 6 -
tests/qemumonitortestutils.c | 2 -
tests/testutils.c | 2 -
tests/testutilsqemuschema.c | 2 +-
tests/vboxsnapshotxmltest.c | 5 -
tests/virbuftest.c | 139 ++-------
tests/vircgrouptest.c | 2 -
tests/virfirewalltest.c | 30 --
tests/virkmodtest.c | 6 +-
tests/virnetdevbandwidthtest.c | 5 -
tests/virsystemdtest.c | 3 -
tests/viruritest.c | 2 +-
tools/virsh-checkpoint.c | 5 -
tools/virsh-domain-monitor.c | 6 -
tools/virsh-domain.c | 30 --
tools/virsh-pool.c | 9 -
tools/virsh-snapshot.c | 5 -
tools/virsh-volume.c | 4 -
tools/vsh.c | 33 +-
tools/vsh.h | 5 +-
105 files changed, 224 insertions(+), 1264 deletions(-)
--
2.21.0
5 years, 2 months
[libvirt] [libvirt-dbus][PATCH v3 0/2] Implement snapshots
by Simon Kobyda
Changes since v2:
Fix underscore in snapshot path
Use virDomainSnapshotGetDomain instead of own excessive calculations
Get rid of excessive code in tests
Appeareance polishes (fix indendation, one-line functions, typos...)
Simon Kobyda (2):
Introduce Domain Snapshot Interface
Implement snapshots APIs
data/org.libvirt.Domain.xml | 26 +++
data/org.libvirt.DomainSnapshot.xml | 41 +++++
src/connect.c | 6 +
src/connect.h | 1 +
src/domain.c | 154 ++++++++++++++++
src/domainsnapshot.c | 269 ++++++++++++++++++++++++++++
src/domainsnapshot.h | 9 +
src/meson.build | 1 +
src/util.c | 53 ++++++
src/util.h | 16 ++
tests/libvirttest.py | 12 ++
tests/meson.build | 1 +
tests/test_domain.py | 8 +
tests/test_snapshot.py | 43 +++++
tests/xmldata.py | 6 +
15 files changed, 646 insertions(+)
create mode 100644 data/org.libvirt.DomainSnapshot.xml
create mode 100644 src/domainsnapshot.c
create mode 100644 src/domainsnapshot.h
create mode 100755 tests/test_snapshot.py
--
2.21.0
5 years, 2 months
[libvirt] Test CPU signatures
by Leo Hou
Hi, Mr Jiri Denemark,
Because I can't subscribe the development of maili<https://www.redhat.com/mailman/listinfo/libvirt-users>n<https://www.redhat.com/mailman/listinfo/libvirt-users>g list<https://www.redhat.com/mailman/listinfo/libvirt-users> now, so I paste the log, I want to know
if the new CPU model be added ,who will patch the file Test CPU signatures *.sig.
cputest: Test CPU signatures
author Jiri Denemark <jdenemar(a)redhat.com>
Mon, 25 Feb 2019 09:05:34 +0000 (10:05 +0100)
committer Jiri Denemark <jdenemar(a)redhat.com>
Tue, 5 Mar 2019 13:47:49 +0000 (14:47 +0100)
commit dd08d27fc205c86961b6d957fbb92da5aead5d1b
tree 7e64a511bfaf0ec494631658ed1ccf605bd6a931 tree | snapshot
parent 61be05a00fd383f11070761fac5ae28272b784dd commit | diff
cputest: Test CPU signatures
The signature computation code is not too complicated and it will likely
never change so testing it is not very important. We do it mostly for a
nice side effect of easily accessible signature numbers for all CPU
data files.
Signed-off-by: Jiri Denemark <jdenemar(a)redhat.com>
Reviewed-by: Ján Tomko <jtomko(a)redhat.com>
________________________________
Leo-Hou(a)hotmail.com
5 years, 2 months
[libvirt] [PATCH v2 00/20] cleanup current build system
by Pavel Hrdina
As preparation to switch to Meson there are some things that needs be
cleaned up to make the conversion easier.
The important thing in Meson is that there is a strict separation
between source and build directory and the distributed tarball by
default contains only files tracked by git with a possibility to
write a script which would add some other sources into the tarball.
Regardless of the adoption of Meson these patches improve our current
build system to fully support VPATH builds.
Changes in v2:
- some patches from v1 are pushed now
- added a patch to mandate build dir != src dir
- added a patch to cleanup .gitignore
- added patches to fix sc_po_check
- improved some patches from v1
Daniel P. Berrangé (1):
build: mandate use of a build dir != src dir
Pavel Hrdina (19):
.gitignore: cleanup old and obsolete ignores
syntax-check.mk: fix sc_po_check rule
syntax-check.mk: cleanup sc_po_check dependencies
syntax-check.mk: cleanup generated_files list for sc_po_check
po: generate files into build directory
po: rewrite the way how we generate files
po: detect zanata binary during configure time
remote: unify rpc server dispatch generated files
src: generate source files into build directory
src: access: generate source files into build directory
src: admin: generate source files into build directory
src: esx: generate source files into build directory
src: hyperv: generate source files into build directory
src: locking: generate source files into build directory
src: logging: generate source files into build directory
src: lxc: generate source files into build directory
src: remote: generate source files into build directory
src: stop distributing generated source files
tools: stop distributing generated source files
.gitignore | 276 ++-------------------
build-aux/syntax-check.mk | 47 ++--
configure.ac | 6 +
libvirt.spec.in | 10 +-
m4/virt-external-programs.m4 | 2 +
po/Makefile.am | 47 ++--
po/POTFILES | 320 -------------------------
po/POTFILES.in | 357 ++++++++++++++++++++++++++++
src/Makefile.am | 13 +-
src/access/Makefile.inc.am | 17 +-
src/admin/Makefile.inc.am | 24 +-
src/bhyve/Makefile.inc.am | 1 +
src/esx/Makefile.inc.am | 9 +-
src/esx/esx_vi_generator.py | 11 +-
src/hyperv/Makefile.inc.am | 9 +-
src/hyperv/hyperv_wmi_generator.py | 11 +-
src/interface/Makefile.inc.am | 2 +
src/libxl/Makefile.inc.am | 2 +
src/locking/Makefile.inc.am | 16 +-
src/logging/Makefile.inc.am | 18 +-
src/lxc/Makefile.inc.am | 36 ++-
src/network/Makefile.inc.am | 2 +
src/node_device/Makefile.inc.am | 2 +
src/nwfilter/Makefile.inc.am | 2 +
src/qemu/Makefile.inc.am | 2 +
src/remote/Makefile.inc.am | 45 ++--
src/remote/remote_daemon_dispatch.c | 4 +-
src/rpc/Makefile.inc.am | 8 +-
src/secret/Makefile.inc.am | 2 +
src/storage/Makefile.inc.am | 2 +
src/util/Makefile.inc.am | 6 +-
src/vbox/Makefile.inc.am | 1 +
src/vz/Makefile.inc.am | 1 +
tests/Makefile.am | 4 +
tools/Makefile.am | 1 -
35 files changed, 606 insertions(+), 710 deletions(-)
delete mode 100644 po/POTFILES
create mode 100644 po/POTFILES.in
--
2.21.0
5 years, 2 months
[libvirt] [PATCH] build: src: fix libtool dependency issue
by Pavel Hrdina
Libtool gets a wrong order of arguments of libraries to install and it
fails when installing libvirt-admin.so that libvirt.so is not yet
installed. Caused by commit <3097282d8668693eb4b7c3fb1b4fe5b474996b9c>.
Signed-off-by: Pavel Hrdina <phrdina(a)redhat.com>
---
src/Makefile.am | 6 ++++--
1 file changed, 4 insertions(+), 2 deletions(-)
diff --git a/src/Makefile.am b/src/Makefile.am
index 34dfeeb9f9..ebc24610e2 100644
--- a/src/Makefile.am
+++ b/src/Makefile.am
@@ -100,6 +100,10 @@ COMMON_UNIT_VARS = \
-e 's|[@]sysconfdir[@]|$(sysconfdir)|g' \
$(NULL)
+# Set these variables before we include any of the subdirs as libvirt-admin.la
+# has to be installed after libvirt.la.
+lib_LTLIBRARIES += libvirt.la libvirt-qemu.la libvirt-lxc.la
+
include util/Makefile.inc.am
include conf/Makefile.inc.am
include cpu/Makefile.inc.am
@@ -146,8 +150,6 @@ SECDRIVER_LIBS += $(APPARMOR_LIBS)
endif WITH_SECDRIVER_APPARMOR
-lib_LTLIBRARIES += libvirt.la libvirt-qemu.la libvirt-lxc.la
-
moddir = $(libdir)/libvirt/connection-driver
confdir = $(sysconfdir)/libvirt
--
2.21.0
5 years, 2 months