- Devel - libvirt List Archives

[libvirt] [dbus PATCH] run.in: fix typo in abs_top_builddir variable
by Pavel Hrdina 22 Mar '18

22 Mar '18

Signed-off-by: Pavel Hrdina <phrdina(a)redhat.com> --- Pushed under trivial rule. Thanks Katerina! run.in | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/run.in b/run.in index 1e48dc8..e98edaf 100644 --- a/run.in +++ b/run.in @@ -15,7 +15,7 @@ # # Find this script -b=@abs_tob_builddir@ +b=@abs_top_builddir@ export VIRT_DBUS_INTERFACES_DIR="$b/data" -- 2.14.3

1 0

[libvirt] [dbus PATCH v3 0/5] rewrite to GDBus
by Pavel Hrdina 22 Mar '18

22 Mar '18

Changes in v3: - introduced run script to help running libvirt-dbus without installing it - improved README and HACKING Changes in v2: - removed patch that set the required libvirt version to libvirt-dbus version Pavel Hrdina (5): introduce support for GDBus implementation introduce run script to execute libvirt-dbus daemon switch from sd-bus to GDBus implementation main: introduce thread pool to process D-Bus messages docs: rewrite HACKING and README into markdown format and improve it .gitignore | 1 + HACKING | 199 ------------ HACKING.md | 191 +++++++++++ Makefile.am | 2 + README | 80 ----- README.md | 72 +++++ configure.ac | 17 +- data/Makefile.am | 7 + data/org.libvirt.Connect.xml | 56 ++++ data/org.libvirt.Domain.xml | 51 +++ libvirt-dbus.spec.in | 10 +- run.in | 22 ++ src/Makefile.am | 20 +- src/connect.c | 324 ++++++++----------- src/connect.h | 37 +-- src/domain.c | 748 ++++++++++++++++++++----------------------- src/domain.h | 6 +- src/events.c | 177 +++++----- src/gdbus.c | 453 ++++++++++++++++++++++++++ src/gdbus.h | 112 +++++++ src/main.c | 315 ++++++++---------- src/util.c | 167 ++++------ src/util.h | 53 ++- test/Makefile.am | 3 +- test/travis-run | 2 +- 25 files changed, 1796 insertions(+), 1329 deletions(-) delete mode 100644 HACKING create mode 100644 HACKING.md delete mode 100644 README create mode 100644 README.md create mode 100644 data/org.libvirt.Connect.xml create mode 100644 data/org.libvirt.Domain.xml create mode 100644 run.in create mode 100644 src/gdbus.c create mode 100644 src/gdbus.h -- 2.14.3

2 14

[libvirt] [jenkins-ci PATCH] guests: Drop Ubuntu 12.04 support
by Andrea Bolognani 22 Mar '18

22 Mar '18

As of commit d4ea32e85f9e, libvirt no longer performs Ubuntu 12.04 builds on Travis, and reproducing locally issues that would hit those CI jobs was the only reason introduced support in the first place. Signed-off-by: Andrea Bolognani <abologna(a)redhat.com> --- guests/host_vars/libvirt-ubuntu-12/install.yml | 3 --- guests/host_vars/libvirt-ubuntu-12/main.yml | 3 --- guests/vars/mappings.yml | 11 ----------- 3 files changed, 17 deletions(-) delete mode 100644 guests/host_vars/libvirt-ubuntu-12/install.yml delete mode 100644 guests/host_vars/libvirt-ubuntu-12/main.yml diff --git a/guests/host_vars/libvirt-ubuntu-12/install.yml b/guests/host_vars/libvirt-ubuntu-12/install.yml deleted file mode 100644 index 997304f..0000000 --- a/guests/host_vars/libvirt-ubuntu-12/install.yml +++ /dev/null @@ -1,3 +0,0 @@ ---- -install_url: http://archive.ubuntu.com/ubuntu/dists/precise/main/installer-amd64/ -install_config: preseed.cfg diff --git a/guests/host_vars/libvirt-ubuntu-12/main.yml b/guests/host_vars/libvirt-ubuntu-12/main.yml deleted file mode 100644 index 2d21f14..0000000 --- a/guests/host_vars/libvirt-ubuntu-12/main.yml +++ /dev/null @@ -1,3 +0,0 @@ ---- -projects: - - libvirt diff --git a/guests/vars/mappings.yml b/guests/vars/mappings.yml index deae69d..d94d648 100644 --- a/guests/vars/mappings.yml +++ b/guests/vars/mappings.yml @@ -137,14 +137,12 @@ mappings: deb: libgnutls28-dev pkg: gnutls rpm: gnutls-devel - Ubuntu12: libgnutls-dev Ubuntu14: libgnutls-dev go: default: golang FreeBSD: go CentOS6: - Ubuntu12: Ubuntu14: gobject-introspection: @@ -171,7 +169,6 @@ mappings: hal: FreeBSD: hal - Ubuntu12: libhal-dev intltool: default: intltool @@ -411,7 +408,6 @@ mappings: default: numad FreeBSD: Debian8: - Ubuntu12: Ubuntu14: openwsman: @@ -455,7 +451,6 @@ mappings: pkg: p5-Digest-MD5 rpm: perl-Digest-MD5 CentOS6: - Ubuntu12: Ubuntu14: perl-ExtUtils-MakeMaker: @@ -510,7 +505,6 @@ mappings: deb: libtest-lwp-useragent-perl pkg: p5-Test-LWP-UserAgent Fedora: perl-Test-LWP-UserAgent - Ubuntu12: perl-Test-Pod: deb: libtest-pod-perl @@ -526,7 +520,6 @@ mappings: deb: libtime-hr-perl pkg: p5-Time-HiRes rpm: perl-Time-HiRes - Ubuntu12: perl Ubuntu14: perl perl-XML-Twig: @@ -552,7 +545,6 @@ mappings: pkg-config: default: pkgconf rpm: pkgconfig - Ubuntu12: pkg-config polkit: default: polkit @@ -614,7 +606,6 @@ mappings: default: python3-requests FreeBSD: py36-requests CentOS: - Ubuntu12: qemu-img: default: qemu-utils @@ -642,7 +633,6 @@ mappings: sanlock: deb: libsanlock-dev rpm: sanlock-devel - Ubuntu12: screen: default: screen @@ -692,7 +682,6 @@ mappings: deb: wireshark-dev Fedora: wireshark-devel Debian8: - Ubuntu12: Ubuntu14: xen: -- 2.14.3

2 1

[libvirt] [jenkins-ci PATCH 0/3] guests: Various cleanups
by Andrea Bolognani 22 Mar '18

22 Mar '18

Applies on top of https://www.redhat.com/archives/libvir-list/2018-March/msg01093.html Andrea Bolognani (3): guests: Configure Jenkins agent based on secret availability guests: Don't expose 'jenkins' pseudo-package guests: Don't expose 'base' pseudo-package guests/host_vars/libvirt-centos-6/main.yml | 2 -- guests/host_vars/libvirt-centos-7/main.yml | 2 -- guests/host_vars/libvirt-debian-8/main.yml | 2 -- guests/host_vars/libvirt-debian-9/main.yml | 2 -- guests/host_vars/libvirt-debian-sid/main.yml | 1 - guests/host_vars/libvirt-fedora-26/main.yml | 2 -- guests/host_vars/libvirt-fedora-27/main.yml | 2 -- guests/host_vars/libvirt-fedora-rawhide/main.yml | 2 -- guests/host_vars/libvirt-freebsd-10/main.yml | 2 -- guests/host_vars/libvirt-freebsd-11/main.yml | 2 -- guests/host_vars/libvirt-freebsd-current/main.yml | 1 - guests/host_vars/libvirt-ubuntu-12/main.yml | 1 - guests/host_vars/libvirt-ubuntu-14/main.yml | 1 - guests/host_vars/libvirt-ubuntu-16/main.yml | 1 - guests/site.yml | 15 ++++++++++++--- guests/tasks/jenkins.yml | 7 +++++++ 16 files changed, 19 insertions(+), 26 deletions(-) -- 2.14.3

3 28

[libvirt] [PATCH v3 00/12] Basic implementation of persistent reservations
by Michal Privoznik 22 Mar '18

22 Mar '18

v3 of: https://www.redhat.com/archives/libvir-list/2018-February/msg01021.html Diff to v2: - John's review worked in The event on pr-helper process dying is still not implemented as qemu has no implementation yet. Michal Privoznik (12): virstoragefile: Introduce virStoragePRDef qemuDomainDiskChangeSupported: Deny changing reservations qemu: Introduce pr-manager-helper capability qemu: Generate alias and socket path for pr-helper qemu: Store pr runtime data in status XML qemu: Generate cmd line at startup qemu: Introduce pr_helper to qemu.conf qemu_domain: Track pr-helper PID in status XML qemu: Start PR daemon on domain startup qemu_hotplug: Hotplug of reservations qemu_hotplug: Hotunplug of reservations qemu: Detect pr-manager-helper capability docs/formatdomain.html.in | 25 ++- docs/schemas/domaincommon.rng | 34 +-- docs/schemas/storagecommon.rng | 50 +++++ m4/virt-driver-qemu.m4 | 5 + src/conf/domain_conf.c | 36 ++++ src/libvirt_private.syms | 6 + src/qemu/libvirtd_qemu.aug | 1 + src/qemu/qemu.conf | 4 + src/qemu/qemu_capabilities.c | 2 + src/qemu/qemu_capabilities.h | 1 + src/qemu/qemu_command.c | 48 +++++ src/qemu/qemu_conf.c | 7 +- src/qemu/qemu_conf.h | 1 + src/qemu/qemu_domain.c | 238 ++++++++++++++++++--- src/qemu/qemu_domain.h | 15 ++ src/qemu/qemu_hotplug.c | 94 ++++++++ src/qemu/qemu_process.c | 193 +++++++++++++++++ src/qemu/qemu_process.h | 7 + src/qemu/test_libvirtd_qemu.aug.in | 1 + src/util/virstoragefile.c | 181 ++++++++++++++++ src/util/virstoragefile.h | 19 ++ tests/qemucapabilitiesdata/caps_2.11.0.s390x.xml | 1 + .../disk-virtio-scsi-reservations-not-managed.args | 29 +++ .../disk-virtio-scsi-reservations-not-managed.xml | 41 ++++ .../disk-virtio-scsi-reservations.args | 29 +++ .../disk-virtio-scsi-reservations.xml | 39 ++++ tests/qemuxml2argvtest.c | 8 + .../disk-virtio-scsi-reservations-not-managed.xml | 1 + .../disk-virtio-scsi-reservations.xml | 1 + tests/qemuxml2xmltest.c | 4 + 30 files changed, 1064 insertions(+), 57 deletions(-) create mode 100644 tests/qemuxml2argvdata/disk-virtio-scsi-reservations-not-managed.args create mode 100644 tests/qemuxml2argvdata/disk-virtio-scsi-reservations-not-managed.xml create mode 100644 tests/qemuxml2argvdata/disk-virtio-scsi-reservations.args create mode 100644 tests/qemuxml2argvdata/disk-virtio-scsi-reservations.xml create mode 120000 tests/qemuxml2xmloutdata/disk-virtio-scsi-reservations-not-managed.xml create mode 120000 tests/qemuxml2xmloutdata/disk-virtio-scsi-reservations.xml -- 2.16.1

3 20

[libvirt] [PATCH 0/2] virCommand: Two trivial fixes
by Michal Privoznik 22 Mar '18

22 Mar '18

Although not pushed, so review appreciated. Michal Privoznik (2): virCommandPassFD: Give name to flags virCommandFDIsSet: Update documentation src/util/vircommand.c | 13 ++++++------- src/util/vircommand.h | 4 ++-- 2 files changed, 8 insertions(+), 9 deletions(-) -- 2.16.1

3 5

[libvirt] [RFC PATCH 0/2] Add vmgenid XML, schema, docs
by John Ferlan 22 Mar '18

22 Mar '18

Could be a v1, but figured I'd get feedback first - I have a "working example" for QEMU (supported since 2.9.0), but it's not quite ready to be fully posted. Currently only needed for windows guests, but there is some work upstream (somewhere) to make a Linux implementation. Still have to work out all the various "combinations" of when to add the vmgenid device and I think there's one combination that's not quite possible and may require some sort of set vmgenid guid value qmp command (that's being debated on qemu upstream now). To read up on the QEMU implementation, see from the top of a QEMU source tree: docs/specs/vmgenid.txt The (ahem) Microsoft details on the device is found at: http://go.microsoft.com/fwlink/?LinkId=260709 John Ferlan (2): conf: Introduce VIR_DOMAIN_DEVICE_VMGENID conf: Add VM Generation ID device docs/formatdomain.html.in | 54 ++++++++++++++ docs/schemas/domaincommon.rng | 21 ++++++ src/conf/domain_conf.c | 120 +++++++++++++++++++++++++++++- src/conf/domain_conf.h | 12 +++ src/qemu/qemu_domain.c | 1 + src/qemu/qemu_domain_address.c | 1 + src/qemu/qemu_driver.c | 6 ++ src/qemu/qemu_hotplug.c | 1 + tests/qemuxml2argvdata/vmgenid-auto.xml | 32 ++++++++ tests/qemuxml2argvdata/vmgenid.xml | 32 ++++++++ tests/qemuxml2xmloutdata/vmgenid-auto.xml | 1 + tests/qemuxml2xmloutdata/vmgenid.xml | 1 + tests/qemuxml2xmltest.c | 3 + 13 files changed, 284 insertions(+), 1 deletion(-) create mode 100644 tests/qemuxml2argvdata/vmgenid-auto.xml create mode 100644 tests/qemuxml2argvdata/vmgenid.xml create mode 120000 tests/qemuxml2xmloutdata/vmgenid-auto.xml create mode 120000 tests/qemuxml2xmloutdata/vmgenid.xml -- 2.13.6

3 5

[libvirt] [snmp PATCH 1/2] Replace U64 with struct counter64 to fix build
by Michael Corcoran 22 Mar '18

22 Mar '18

Upstream net-snmp removed the U64 typedef in 2016 as it clashed with a name in Perl, thus all usages of U64 must be replaced by struct counter64 to compile against recent versions of net-snmp. The replacements were made by hand, while referencing diff with new source generated by the mib2c tool. net-snmp have a mib2c-update tool which supposedly assists with updating existing (modified) source code but I was unable to get it to work. Signed-off-by: Michael Corcoran <tracer(a)outlook.co.nz> --- src/defaults/node-libvirtGuestCpuTime.m2d | 2 +- src/libvirtGuestTable-README-libvirtGuestTable.txt | 2 +- src/libvirtGuestTable.h | 4 ++-- src/libvirtGuestTable_data_get.c | 4 ++-- src/libvirtGuestTable_data_get.h | 2 +- src/libvirtGuestTable_data_set.h | 4 ++-- src/libvirtGuestTable_interface.c | 8 ++++---- 7 files changed, 13 insertions(+), 13 deletions(-) diff --git a/src/defaults/node-libvirtGuestCpuTime.m2d b/src/defaults/node-libvirtGuestCpuTime.m2d index 795f697..fc93506 100644 --- a/src/defaults/node-libvirtGuestCpuTime.m2d +++ b/src/defaults/node-libvirtGuestCpuTime.m2d @@ -11,7 +11,7 @@ ## If you do, it is likely to break lots of generated code that ## you will have to fix. ## -## @eval $m2c_decl = U64@ +## @eval $m2c_decl = "struct counter64"@ ## ######################################################################## ## Generate/use mapping functions? Useful if the MIB defines diff --git a/src/libvirtGuestTable-README-libvirtGuestTable.txt b/src/libvirtGuestTable-README-libvirtGuestTable.txt index 5a74dae..74e2274 100644 --- a/src/libvirtGuestTable-README-libvirtGuestTable.txt +++ b/src/libvirtGuestTable-README-libvirtGuestTable.txt @@ -722,7 +722,7 @@ The CPU time used by the virtual guest, in nanoseconds. * * * Its syntax is COUNTER64 (based on perltype COUNTER64) - * The net-snmp type is ASN_COUNTER64. The C type decl is U64 (U64) + * The net-snmp type is ASN_COUNTER64. The C type decl is U64 (struct counter64) */ /*--------------------------------------------------------------------- * LIBVIRT-MIB::libvirtGuestEntry.libvirtGuestRowStatus diff --git a/src/libvirtGuestTable.h b/src/libvirtGuestTable.h index ca27abd..6762238 100644 --- a/src/libvirtGuestTable.h +++ b/src/libvirtGuestTable.h @@ -119,9 +119,9 @@ typedef struct libvirtGuestTable_data_s { u_long libvirtGuestMemoryLimit; /* - * libvirtGuestCpuTime(7)/COUNTER64/ASN_COUNTER64/U64(U64)//l/A/w/e/r/d/h + * libvirtGuestCpuTime(7)/COUNTER64/ASN_COUNTER64/U64(struct counter64)//l/A/w/e/r/d/h */ - U64 libvirtGuestCpuTime; + struct counter64 libvirtGuestCpuTime; /* * libvirtGuestRowStatus(9)/RowStatus/ASN_INTEGER/long(u_long)//l/A/W/E/r/d/h diff --git a/src/libvirtGuestTable_data_get.c b/src/libvirtGuestTable_data_get.c index b252bdd..08ddb4a 100644 --- a/src/libvirtGuestTable_data_get.c +++ b/src/libvirtGuestTable_data_get.c @@ -442,7 +442,7 @@ The CPU time used by the virtual guest, in nanoseconds. * * * Its syntax is COUNTER64 (based on perltype COUNTER64) - * The net-snmp type is ASN_COUNTER64. The C type decl is U64 (U64) + * The net-snmp type is ASN_COUNTER64. The C type decl is U64 (struct counter64) */ /** * Extract the current value of the libvirtGuestCpuTime data. @@ -459,7 +459,7 @@ The CPU time used by the virtual guest, in nanoseconds. * @retval MFD_ERROR : Any other error */ int -libvirtGuestCpuTime_get( libvirtGuestTable_rowreq_ctx *rowreq_ctx, U64 * libvirtGuestCpuTime_val_ptr ) +libvirtGuestCpuTime_get( libvirtGuestTable_rowreq_ctx *rowreq_ctx, struct counter64 * libvirtGuestCpuTime_val_ptr ) { /** we should have a non-NULL pointer */ netsnmp_assert( NULL != libvirtGuestCpuTime_val_ptr ); diff --git a/src/libvirtGuestTable_data_get.h b/src/libvirtGuestTable_data_get.h index 450d112..6a0e7ff 100644 --- a/src/libvirtGuestTable_data_get.h +++ b/src/libvirtGuestTable_data_get.h @@ -55,7 +55,7 @@ extern "C" { int libvirtGuestCpuCount_get( libvirtGuestTable_rowreq_ctx *rowreq_ctx, u_long * libvirtGuestCpuCount_val_ptr ); int libvirtGuestMemoryCurrent_get( libvirtGuestTable_rowreq_ctx *rowreq_ctx, u_long * libvirtGuestMemoryCurrent_val_ptr ); int libvirtGuestMemoryLimit_get( libvirtGuestTable_rowreq_ctx *rowreq_ctx, u_long * libvirtGuestMemoryLimit_val_ptr ); - int libvirtGuestCpuTime_get( libvirtGuestTable_rowreq_ctx *rowreq_ctx, U64 * libvirtGuestCpuTime_val_ptr ); + int libvirtGuestCpuTime_get( libvirtGuestTable_rowreq_ctx *rowreq_ctx, struct counter64 * libvirtGuestCpuTime_val_ptr ); int libvirtGuestRowStatus_get( libvirtGuestTable_rowreq_ctx *rowreq_ctx, u_long * libvirtGuestRowStatus_val_ptr ); diff --git a/src/libvirtGuestTable_data_set.h b/src/libvirtGuestTable_data_set.h index cce0fe7..873f9f5 100644 --- a/src/libvirtGuestTable_data_set.h +++ b/src/libvirtGuestTable_data_set.h @@ -80,9 +80,9 @@ int libvirtGuestMemoryLimit_undo_setup( libvirtGuestTable_rowreq_ctx *rowreq_ctx int libvirtGuestMemoryLimit_set( libvirtGuestTable_rowreq_ctx *rowreq_ctx, u_long libvirtGuestMemoryLimit_val ); int libvirtGuestMemoryLimit_undo( libvirtGuestTable_rowreq_ctx *rowreq_ctx ); -int libvirtGuestCpuTime_check_value( libvirtGuestTable_rowreq_ctx *rowreq_ctx, U64 libvirtGuestCpuTime_val); +int libvirtGuestCpuTime_check_value( libvirtGuestTable_rowreq_ctx *rowreq_ctx, struct counter64 libvirtGuestCpuTime_val); int libvirtGuestCpuTime_undo_setup( libvirtGuestTable_rowreq_ctx *rowreq_ctx ); -int libvirtGuestCpuTime_set( libvirtGuestTable_rowreq_ctx *rowreq_ctx, U64 libvirtGuestCpuTime_val ); +int libvirtGuestCpuTime_set( libvirtGuestTable_rowreq_ctx *rowreq_ctx, struct counter64 libvirtGuestCpuTime_val ); int libvirtGuestCpuTime_undo( libvirtGuestTable_rowreq_ctx *rowreq_ctx ); int libvirtGuestRowStatus_check_value( libvirtGuestTable_rowreq_ctx *rowreq_ctx, u_long libvirtGuestRowStatus_val); diff --git a/src/libvirtGuestTable_interface.c b/src/libvirtGuestTable_interface.c index ce29c3b..ee48291 100644 --- a/src/libvirtGuestTable_interface.c +++ b/src/libvirtGuestTable_interface.c @@ -842,11 +842,11 @@ rc = libvirtGuestMemoryCurrent_get(rowreq_ctx, (u_long *)var->val.string ); rc = libvirtGuestMemoryLimit_get(rowreq_ctx, (u_long *)var->val.string ); break; - /* libvirtGuestCpuTime(7)/COUNTER64/ASN_COUNTER64/U64(U64)//l/A/w/e/r/d/h */ + /* libvirtGuestCpuTime(7)/COUNTER64/ASN_COUNTER64/U64(struct counter64)//l/A/w/e/r/d/h */ case COLUMN_LIBVIRTGUESTCPUTIME: - var->val_len = sizeof(U64); + var->val_len = sizeof(struct counter64); var->type = ASN_COUNTER64; -rc = libvirtGuestCpuTime_get(rowreq_ctx, (U64 *)var->val.string ); +rc = libvirtGuestCpuTime_get(rowreq_ctx, (struct counter64 *)var->val.string ); break; /* libvirtGuestRowStatus(9)/RowStatus/ASN_INTEGER/long(u_long)//l/A/W/E/r/d/h */ @@ -1022,7 +1022,7 @@ _libvirtGuestTable_check_column( libvirtGuestTable_rowreq_ctx *rowreq_ctx, rc = SNMP_ERR_NOTWRITABLE; break; - /* libvirtGuestCpuTime(7)/COUNTER64/ASN_COUNTER64/U64(U64)//l/A/w/e/r/d/h */ + /* libvirtGuestCpuTime(7)/COUNTER64/ASN_COUNTER64/U64(struct counter64)//l/A/w/e/r/d/h */ case COLUMN_LIBVIRTGUESTCPUTIME: rc = SNMP_ERR_NOTWRITABLE; break;

2 2

[libvirt] snapshot: support rbd disk
by Zhangzijian 22 Mar '18

22 Mar '18

Hello, Qemu support create snapshot for rbd disk. But Libvirt have not yet supported this feature now, is there any roadmap? Thanks.

1 0

[libvirt] [PATCH] news: Xen: announce removal of hard-code scheduler weight
by Jim Fehlig 21 Mar '18

21 Mar '18

Also describe a possible side-affect due to changes in the default (unspecified) value from 1000 to 256. Signed-off-by: Jim Fehlig <jfehlig(a)suse.com> --- docs/news.xml | 16 ++++++++++++++++ 1 file changed, 16 insertions(+) diff --git a/docs/news.xml b/docs/news.xml index 75a054308..93ab59cf4 100644 --- a/docs/news.xml +++ b/docs/news.xml @@ -66,6 +66,22 @@ If someone customizing IP address inside VM, it will be helpful. </description> </change> + <change> + <summary> + Xen: Remove hard-coded scheduler weight + </summary> + <description> + The libxl driver was accidentally hard-coding the per-domain + scheduler weight to 1000, silently ignoring any user-provided + <code><shares></code> in <code><cputune></code>. The + driver now honors <code><shares></code>, and defers setting + a default value to Xen. Note that the Xen default is 256, so any + domains started after this improvement will have one fourth the + shares of previously started domains. If all domains must have + equal CPU shares, administrators must manually set the weight of + previously started domains to 256, or restart them. + </description> + </change> </section> <section title="Bug fixes"> </section> -- 2.16.2

2 1