- Devel - libvirt List Archives

[libvirt] [PATCH] disk: Provide a default storage source format type.
by John Ferlan 02 Mar '15

02 Mar '15

https://bugzilla.redhat.com/show_bug.cgi?id=1181062 According to the formatstorage.html description for <source> element and "format" attribute: "All drivers are required to have a default value for this, so it is optional." As it turns out the disk backend did not choose a default value, so I added a default of "msdos" if the source type is "unknown" as well as updating the storage.html backend disk volume driver documentation to indicate the default format is dos. Signed-off-by: John Ferlan <jferlan(a)redhat.com> --- docs/storage.html.in | 1 + src/storage/storage_backend_disk.c | 13 ++++++++++--- 2 files changed, 11 insertions(+), 3 deletions(-) diff --git a/docs/storage.html.in b/docs/storage.html.in index f920dae..b9b503d 100644 --- a/docs/storage.html.in +++ b/docs/storage.html.in @@ -346,6 +346,7 @@ on the size and placement of volumes. The 'free extents' information will detail the regions which are available for creating new volumes. A volume cannot span across 2 different free extents. + It will default to using <code>msdos</code> as the pool source format. </p> <h3>Example pool input</h3> diff --git a/src/storage/storage_backend_disk.c b/src/storage/storage_backend_disk.c index 39082cc..9a79325 100644 --- a/src/storage/storage_backend_disk.c +++ b/src/storage/storage_backend_disk.c @@ -468,13 +468,20 @@ virStorageBackendDiskBuildPool(virConnectPtr conn ATTRIBUTE_UNUSED, } if (ok_to_mklabel) { - /* eg parted /dev/sda mklabel msdos */ + /* eg parted /dev/sda mklabel --script msdos */ + int format = pool->def->source.format; + const char *fmt; + if (format == VIR_STORAGE_POOL_DISK_UNKNOWN || + format == VIR_STORAGE_POOL_DISK_DOS) + fmt = "msdos"; + else + fmt = virStoragePoolFormatDiskTypeToString(format); + cmd = virCommandNewArgList(PARTED, pool->def->source.devices[0].path, "mklabel", "--script", - ((pool->def->source.format == VIR_STORAGE_POOL_DISK_DOS) ? "msdos" : - virStoragePoolFormatDiskTypeToString(pool->def->source.format)), + fmt, NULL); ret = virCommandRun(cmd, NULL); } -- 2.1.0

2 1

[libvirt] [PATCH] docs: add a note that attr 'managed' is only used by PCI devices
by Erik Skultety 02 Mar '15

02 Mar '15

Our documentation isn't 100% clear about hostdev 'managed' attribute usage, because it only makes sense to use it with PCI devices, yet we format this attribute to all hostdev devices. By adding a note into the docs, we can possibly avoid confusion from customer's side and also avoid a solution using ternary logic. Resolves: https://bugzilla.redhat.com/show_bug.cgi?id=1155887 --- docs/formatdomain.html.in | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/docs/formatdomain.html.in b/docs/formatdomain.html.in index fb0a0d1..6276a61 100644 --- a/docs/formatdomain.html.in +++ b/docs/formatdomain.html.in @@ -3110,6 +3110,12 @@ then this setting not required. </dd> </dl> + <p> + Note: The <code>managed</code> attribute is only used with PCI devices + and is ignored by all the other device types, thus setting + <code>managed</code> explicitly with other than PCI device has the same + effect as omitting it. + </p> </dd> <dt><code>source</code></dt> <dd>The source element describes the device as seen from the host using -- 1.9.3

2 2

[libvirt] [PATCH] storage: sheepdog: Avoid skipping variable initialization
by Peter Krempa 02 Mar '15

02 Mar '15

Commit 155ca616eb231181f6978efc9e3a1eb0eb60af8a added a error message that skips initialization of the 'cmd' variable. Fortunately it was not released. --- Pushed under the trivial && build-breaker rule. src/storage/storage_backend_sheepdog.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/src/storage/storage_backend_sheepdog.c b/src/storage/storage_backend_sheepdog.c index f389d9b..af15c3b 100644 --- a/src/storage/storage_backend_sheepdog.c +++ b/src/storage/storage_backend_sheepdog.c @@ -263,6 +263,7 @@ virStorageBackendSheepdogBuildVol(virConnectPtr conn, unsigned int flags) { int ret = -1; + virCommandPtr cmd = NULL; virCheckFlags(0, -1); @@ -272,7 +273,7 @@ virStorageBackendSheepdogBuildVol(virConnectPtr conn, goto cleanup; } - virCommandPtr cmd = virCommandNewArgList(COLLIE, "vdi", "create", vol->name, NULL); + cmd = virCommandNewArgList(COLLIE, "vdi", "create", vol->name, NULL); virCommandAddArgFormat(cmd, "%llu", vol->target.capacity); virStorageBackendSheepdogAddHostArg(cmd, pool); if (virCommandRun(cmd, NULL) < 0) -- 2.2.2

1 0

[libvirt] [libvirt-test-API][PATCH 0/2] Add freepages test cases
by Jincheng Miao 02 Mar '15

02 Mar '15

Add freepages test cases. Jincheng Miao (2): Add freepage test Add freepage test case to test_connection.conf cases/test_connection.conf | 6 +++ repos/virconn/free_pages.py | 97 +++++++++++++++++++++++++++++++++++++++++++ 2 files changed, 103 insertions(+), 0 deletions(-) create mode 100644 repos/virconn/free_pages.py

2 3

[libvirt] Release of libvirt-1.2.13
by Daniel Veillard 02 Mar '15

02 Mar '15

As planned I just made the release for libvirt-1.2.13, it's tagged in git, and signed tarballs and rpms are available at the usual place: ftp://libvirt.org/libvirt/ I also pushed a 1.2.13 for libvirt-python but it's really for completeness as it's virtuallyequivalent to 1.2.12. February is short, that release packs 'only' 220 commits, mostly bug fixes and a fair set of improvment, with the NUMA handling area getting a fairly large update: Features: - a lot of NUMA updates and improvments (Peter Krempa) Documentation: - Add mist.io as libvirt-based application (Michal Privoznik) - Fix typos in messages (Yuri Chornoivan) - Clarify behavior or virDomainDetachDevice (Ján Tomko) - virsh: Add missing parenthesis into man page (Martin Kletzander) - man: moved virsh command cpu-models (Daniel Hansel) - clarify nat range behavior (Ján Tomko) - add page about virtlockd setup (Daniel P. Berrange) - split out sanlock setup docs (Daniel P. Berrange) - Fix version reference in vcpu/iothread scheduling (Martin Kletzander) - S390: Documentation for ccw address type (Stefan Zimmermann) - conf: fix a minor typo (Erik Skultety) - virsh: man: Document behavior of some blkdeviotune's flags when querying (Peter Krempa) Portability: - virprocess: fix MinGW build and RHEL-5 build (Pavel Hrdina) - virutil: fix MinGW build (Pavel Hrdina) - Make tests independant of system page size (Daniel P. Berrange) Bug Fixes: - Really fix XML formatting flags in SaveImageUpdateDef (Ján Tomko) - Ignore listen attribute of <graphics> for type network listens (Ján Tomko) - util: storage: Fix error type in virStorageSourceParseBackingURI (Peter Krempa) - util: storagefile: Don't crash on gluster URIs without path (Peter Krempa) - conf: error out on invalid host id (Luyao Huang) - qemu: fix ifindex array reported to systemd (Laine Stump) - util: check for null ifname inside virNetDevBandwidthSet() (Laine Stump) - network: only clear bandwidth if it has been set (Laine Stump) - Fix error messages in virStorageFileGetMetadataFromFD (Ján Tomko) - Assign default SCSI controller model before checking attribute validity (Ján Tomko) - qemu: Use correct flags for ABI stability check in SaveImageUpdateDef (Michal Privoznik) - qemu: fix memory leak while starting a guest (Pavel Hrdina) - network_conf: Forbid commas in DNS TXT (Michal Privoznik) - qemuMigrationDriveMirror: Listen to events (Michal Privoznik) - qemu: Exit job on error path of qemuDomainSetVcpusFlags() (Peter Krempa) - daemon: Fix segfault by reloading daemon right after start (Pavel Hrdina) - qemu_cgroup: initialize mem_mask to NULL (Pavel Hrdina) - PowerPC : Forbid NULL CPU model with 'host-model' mode. (Prerna Saxena) - qemu: fix vm deadlock when try to use numatune in session mode (Luyao Huang) - virsh: fix IP address in vncdisplay for listen type='network' (Luyao Huang) - libxl: Resolve Coverity CHECKED_RETURN (John Ferlan) - security: Resolve Coverity RESOURCE_LEAK (John Ferlan) - lxc: Fix container cleanup for LXCProcessStart (Luyao Huang) - lxc: Move console checks in LXCProcessStart (Luyao Huang) - security: Refactor virSecurityManagerGenLabel (Erik Skultety) - security: introduce virSecurityManagerCheckAllLabel function (Erik Skultety) - Fix a syntax error in the description text of libvirtd.conf (Zhang Bo) - Allow shrinking of file based volumes (Daniel P. Berrange) - qemu: do upfront check for vcpupids being null when querying pinning (Daniel P. Berrange) - qemu: fix setting of VM CPU affinity with TCG (Daniel P. Berrange) - libxl: disable VNC and SDL until explicitly enabled (Marek Marczykowski-Górecki) - libxl: pass ipaddr to libxl toolstack (Marek Marczykowski-Górecki) - domain: avoid potential memory leak in virDomainGraphicsListenSet*() (Laine Stump) - domain: backfill listen address to parent <graphics> listen attribute (Laine Stump) - virsh: fix IP address in domdisplay for listen type='network' (Luyao Huang) - bhyvexml2argvmock: change int to size_t for tapfdSize (Pavel Hrdina) - qemu: fix crash when migrateuri has no scheme (Luyao Huang) - conf: Fix libvirtd crash and memory leak caused by virDomainVcpuPinDel() (Zhang Bo) - conf: forbid seclabel duplicates for domain devices (Erik Skultety) - Fix qemu job handling in SetSchedulerParameters (Ján Tomko) - conf: disallow invalid values for video attributes (Martin Kletzander) - Clear dynamic seclabels on LXCProcessStart failure (Cédric Bosdonnat) - network: don't allow multiple portgroups with the same name in a network (Laine Stump) - Error out when custom tap device path makes no sense (Ján Tomko) - Only parse custom vhost path for virtio interfaces (Ján Tomko) - Trivially support DomainHasManagedSaveImage (Ján Tomko) - libxl: update dom->id in libxlDomainCreateWithFlags (Marek Marczykowski-Górecki) - qemu: Properly report error on uuid mismatch in the migration cookie (Luyao Huang) - lxc: fix double close handshakefds[1] (Luyao Huang) - util: storage: Fix parsing of nbd:// URI without path (Peter Krempa) - conf: Properly report error of unsupported input bus type (Luyao Huang) - conf: Properly report error when an unsupported chr device name is passed (Luyao Huang) - virnetdev: fix some issues found by coverity and mingw builds (Pavel Hrdina) - storage: Need to clear pool prior to refreshPool during Autostart (John Ferlan) - qemu: command: Don't combine old and modern NUMA node creation (Peter Krempa) - xend: Don't crash in virDomainXMLDevID (Michal Privoznik) - conf: Don't mangle vcpu placement randomly (Michal Privoznik) - qemu: Don't unconditionally delete file in qemuOpenFileAs (John Ferlan) - virfile: Need to check for ENOTCONN from recvfd failure (John Ferlan) - Remove flag checking in MacVLanCreate helper stub (Ján Tomko) - storage: Check the partition name against provided name (John Ferlan) - storage: Fix check for partition type for disk backing volumes (John Ferlan) - qemu: fix cannot set graphic passwd via qemuDomainSaveImageDefineXML (Luyao Huang) - hotplug: only add a chardev to vmdef after monitor call (Ján Tomko) - systemd: don't report an error if the guest is already terminated (Daniel P. Berrange) Improvements: - conf: error out on missing dhcp host attributes (Ján Tomko) - schema: Allow interleaving the /domain/os/type element (Peter Krempa) - schema: Add virtio-mmio address type into RNG (Martin Kletzander) - cpu: Format <cpu/> properly (Michal Privoznik) - virCPUDefFormatBufFull: Use our general error handling pattern (Michal Privoznik) - util: Introduce virBufferAddBuffer (Michal Privoznik) - Rework s390 architecture checking (Stefan Zimmermann) - Prevent default creation of usb controller on s390 and s390x (Stefan Zimmermann) - po: Add config file for zanata (Daniel P. Berrange) - libvirt-guests: Allow time sync on guests resume (Michal Privoznik) - qemu: Fix AAVMF/OVMF #define names (Cole Robinson) - libxl: support backend domain setting for disk and net devices (Marek Marczykowski) - conf: support backend domain name in disk and network devices (Marek Marczykowski) - network: allow <pf> together with <interface>/<address> in network status (Laine Stump) - qemu: caps: Add capability bit for the "pc-dimm" device (Peter Krempa) - conf: Hoist validation of memory size into the post parse callback (Peter Krempa) - conf: numa: Check ABI stability of NUMA configuration (Peter Krempa) - conf: ABI: Hugepage backing definition is not guest ABI (Peter Krempa) - conf: Move all NUMA configuration to virDomainNuma (Peter Krempa) - conf: numa: Add setter/getter for NUMA node memory size (Peter Krempa) - conf: numa: Add accessor to NUMA node's memory access mode (Peter Krempa) - conf: numa: Add accessor for the NUMA node cpu mask (Peter Krempa) - conf: numa: Add helper to get guest NUMA node count and refactor users (Peter Krempa) - qemu: command: Unify retrieval of NUMA cell count in qemuBuildNumaArgStr (Peter Krempa) - conf: numa: Don't pass double pointer to virDomainNumatuneParseXML (Peter Krempa) - numa: conf: Tweak parameters of virDomainNumatuneSet (Peter Krempa) - conf: numa: Always allocate the NUMA config (Peter Krempa) - conf: Allocate domain definition with the new helper (Peter Krempa) - conf: Separate helper for creating domain objects (Peter Krempa) - conf: numa: Format <numatune> XML only if necessary (Peter Krempa) - conf: numa: Refactor logic in virDomainNumatuneParseXML (Peter Krempa) - conf: numa: Reformat virDomainNumatuneParseXML (Peter Krempa) - conf: numa: Improve error message in case a numa node doesn't have cpus (Peter Krempa) - conf: numa: Recalculate rather than remember total NUMA cpu count (Peter Krempa) - conf: Move enum virMemAccess to the NUMA code and rename it (Peter Krempa) - conf: numa: Rename virDomainNumatune to virDomainNuma (Peter Krempa) - conf: Move NUMA cell formatter to numa_conf (Peter Krempa) - conf: numa: Don't duplicate NUMA cell cpumask (Peter Krempa) - conf: Refactor virDomainNumaDefCPUParseXML (Peter Krempa) - conf: Move NUMA cell parsing code from cpu conf to numa conf (Peter Krempa) - conf: Move numatune_conf to numa_conf (Peter Krempa) - virsh: fix vcpupin info (Pavel Hrdina) - virQEMUCapsCacheLookupCopy: Filter qemuCaps based on machineType (Michal Privoznik) - virQEMUCapsCacheLookupCopy: Pass machine type (Michal Privoznik) - virsh-edit: Make force editing usable (Martin Kletzander) - parallels: Set the first HDD from XML as bootable (Mikhail Feoktistov) - parallels: Use IS_CT() macro instead of STREQ("exe") (Mikhail Feoktistov) - parallels: code aligment (Mikhail Feoktistov) - Search for schemas and cpu_map.xml in source tree (Jiri Denemark) - qemuProcessHandleBlockJob: Take status into account (Michal Privoznik) - qemuProcessHandleBlockJob: Set disk->mirrorState more often (Michal Privoznik) - PowerPC : Make 'qemu-system-ppc64' the default emulator on ppc64[le]. (Prerna Saxena) - qemuBuildMemoryBackendStr: Report backend requirement more appropriately (Michal Privoznik) - qemuxml2argvtest: Fake response from numad (Michal Privoznik) - numatune_conf: Expose virDomainNumatuneNodeSpecified (Michal Privoznik) - lxc: Modify/add some debug messages (John Ferlan) - Wire up mrg_rxbuf option for qemu (Ján Tomko) - Add mrg_rxbuf option to virtio interfaces (Ján Tomko) - virsh attach-interface: Allow macvtap hotplug (Michal Privoznik) - virsh attach-interface: Use virDomainNetType{From,To}String() (Michal Privoznik) - virsh attach-interface: Use enum instead of arbitrary integers (Michal Privoznik) - libvirt_private.syms: Expose virDomainNetTypeFromString (Michal Privoznik) - qemu: Add support for setting vCPU and I/O thread scheduler setting (Martin Kletzander) - docs, schema, conf: Add support for setting scheduler parameters of guest threads (Martin Kletzander) - util: Add virProcessSetScheduler() function for scheduler settings (Martin Kletzander) - domain: include portgroup in interface status xml (Laine Stump) - qemu: qemuOpenFileAs - set flag VIR_FILE_OPEN_FORCE_MODE (John Ferlan) - virfile: Adjust error path for virFileOpenForked (John Ferlan) - schema: allow multiple seclabel for devices in domaincommon.rng (Erik Skultety) - qemu: Implement random number generator hotunplug (Luyao Huang) - qemu: Implement random number generator hotplug (Luyao Huang) - qemu: Implement random number generator cold (un)plug (Peter Krempa) - conf: Add helpers to insert/remove/find RNG devices in domain def (Luyao Huang) - audit: export virDomainAuditRNG (Luyao Huang) - qemu: command: Refactor creation of RNG device commandline (Peter Krempa) - qemu: command: Break some very long lines in qemuBuildRNGDevStr() (Peter Krempa) - qemu: command: Shuffle around formatting of alias for RNG device backend (Peter Krempa) - qemu: command: Make RNG backend device IDs unique (Luyao Huang) - qemu: refactor qemuBuildRNGDeviceArgs to allow reuse in RNG hotplug (Luyao Huang) - qemu: Add helper to assign RNG device aliases (Luyao Huang) - conf: Introduce helper to find duplicate device address (Peter Krempa) - S390: ccw support for virsh attach-disk address parameter (Stefan Zimmermann) - maint: improve usage of autogen's --no-git (Stefan Zimmermann) - qemu: include libvirt & QEMU versions in QEMU log files (Daniel P. Berrange) - nodedev: check/add for scsi_host caps for NumOfCaps and ListCaps (John Ferlan) - nodedev: check/add for scsi_host caps for ListDevices (Shivaprasad G Bhat) - cpu: add Freescale ppc64 CPU models (Olivia Yin) - tests: virstoragetest: Switch backing chain test to use automatic numbering (Peter Krempa) - virsh-volume: add support for --reflink (Chen Hanxiao) - util: make virNetDev(Get|Set)IFFlags() static (Laine Stump) - qemu: command: Refactor NUMA backend object formatting to use JSON objs (Peter Krempa) - qemu: command: Switch to bytes when formatting size for memory backends (Peter Krempa) - qemu: command: Unify values for boolean values when formating memory backends (Peter Krempa) - qemu: command: Shuffle around formating of alias for memory backend objs (Peter Krempa) - qemu: Extract code to setup memory backing objects (Peter Krempa) - qemu: command: Add helper to format -object strings from JSON representation (Peter Krempa) - util: json: add helper to iterate JSON object key=value pairs (Peter Krempa) - util: json: Add functions to convert JSON arrays from/to virBitmaps (Peter Krempa) - util: bitmap: Add option to allocate bitmap without reporting error (Peter Krempa) - util: json: make value object creator universal by supporting adding (Peter Krempa) - test: utils: Add helpers for automatic numbering of test cases (Peter Krempa) - conf: numatune: Extract code for requesting memory nodeset from formatting (Peter Krempa) - qemu: change macvtap device options in response to NIC_RX_FILTER_CHANGED (Tony Krowiak) - util: Functions for getting/setting device options (Tony Krowiak) - qemu: Adjust EndAsyncJob for qemuDomainSaveInternal error path (John Ferlan) - schemas: Allow all generic elements and attributes for all interfaces (Michal Privoznik) - qemu: Add AAVMF to the list of known UEFIs (Michal Privoznik) - qemu: Allow UEFI paths to be specified at compile time (Michal Privoznik) - Fix syntax-check (Ján Tomko) - util: bitmap: Tolerate NULL bitmaps in virBitmapEqual (Peter Krempa) - storage: When delete extended partition, need to refresh pool (John Ferlan) - storage: Adjust how to refresh extended partition disk data (John Ferlan) - storage: Attempt error recovery in virStorageBackendDiskCreateVol (John Ferlan) - storage: Move virStorageBackendDiskDeleteVol (John Ferlan) - libvirt.spec: remove vbox storage and network .so files (Pavel Hrdina) - Split qemuDomainChrInsert into two parts (Ján Tomko) - lxc: report veth device indexes to systemd (Daniel P. Berrange) - lxc: more logging during startup paths (Daniel P. Berrange) - lxc: delay setup of cgroup until we have the init pid (Daniel P. Berrange) - lxc: only write XML once for lxc controller (Daniel P. Berrange) - lxc: re-arrange startup synchronization sequence with controller (Daniel P. Berrange) - lxc: don't build pidfile string multiple times (Daniel P. Berrange) - qemu: report TAP device indexes to systemd (Daniel P. Berrange) - Fix shadowed variable warning (Ján Tomko) - storage: add a flag to clone files on btrfs (Chen Hanxiao) - storage: introduce btrfsCloneFile() for COW copy (Chen Hanxiao) - Removing probing of secondary drivers (Daniel P. Berrange) - don't disable state driver when libvirtd is not built (Daniel P. Berrange) - Remove all secondary driver private data fields (Daniel P. Berrange) - Remove use of interfacePrivateData from udev driver (Daniel P. Berrange) - Remove use of nodeDevicePrivateData from nodeDev driver (Daniel P. Berrange) - Remove use of storagePrivateData/networkPrivateData from vbox (Daniel P. Berrange) - Remove use of nwfilterPrivateData from nwfilter driver (Daniel P. Berrange) - Remove use of secretPrivateData from secret driver (Daniel P. Berrange) - qemu: Fix job handling in qemuDomainSetMetadata (Peter Krempa) - qemu: Fix job type in qemuDomainGetBlockIoTune (Peter Krempa) - qemu: Fix job handling in qemuDomainSetSchedulerParametersFlags (Peter Krempa) - qemu: Fix job handling in qemuDomainSetMemoryParameters (Peter Krempa) - qemu: Fix job handling in qemuDomainSetAutostart (Peter Krempa) - qemu: Fix job handling in qemuDomainPinEmulator (Peter Krempa) - qemu: Fix job handling in qemuDomainPinVcpuFlags (Peter Krempa) So thanks everyone for the help with this release, be it with reports, patches, reviews or documentation ! Daniel -- Daniel Veillard | Open Source and Standards, Red Hat veillard(a)redhat.com | libxml Gnome XML XSLT toolkit http://xmlsoft.org/ http://veillard.com/ | virtualization library http://libvirt.org/

1 0

[libvirt] [PATCH] docs: Add mist.io as libvirt-based application
by Michal Privoznik 01 Mar '15

01 Mar '15

As reported on the libvirt-users list [1], there's new web application called mist.io which uses libvirt as one of its backends. Lets add it into our list of libivrt based applications. 1: https://www.redhat.com/archives/libvirt-users/2015-February/msg00096.html Signed-off-by: Michal Privoznik <mprivozn(a)redhat.com> --- docs/apps.html.in | 8 ++++++++ 1 file changed, 8 insertions(+) diff --git a/docs/apps.html.in b/docs/apps.html.in index 4403ce6..79ee73a 100644 --- a/docs/apps.html.in +++ b/docs/apps.html.in @@ -404,6 +404,14 @@ functions, such as live migration that allows for load balancing between cluster nodes, monitoring CPU, memory. </dd> + <dt><a href="http://mist.io/">mist.io</a></dt> + <dd> + Mist.io is an open source project and a service that can assist you in + managing your virtual machines on a unified way, providing a simple + interface for all of your infrastructure (multiple public cloud + providers, OpenStack based public/private clouds, Docker servers, bare + metal servers and now KVM hypervisors). + </dd> </dl> <h2><a name="mobile">Mobile applications</a></h2> -- 2.0.5

2 2

[libvirt] new warning from ar on rawhide systems
by Eric Blake 28 Feb '15

28 Feb '15

When compiling libvirt (and probably other packages) on Fedora rawhide systems, I'm seeing a new warning message on every link line. # rpm -q libtool gcc binutils libtool-2.4.6-1.fc23.x86_64 gcc-5.0.0-0.16.fc23.x86_64 binutils-2.25-6.fc23.x86_64 For an example of the warning during a 'make V=1': > libtool: link: rm -fr .libs/libvirt_driver_qemu_impl.a .libs/libvirt_driver_qemu_impl.la > libtool: link: ar cru .libs/libvirt_driver_qemu_impl.a qemu/.libs/libvirt_driver_qemu_impl_la-qemu_agent.o qemu/.libs/libvirt_driver_qemu_impl_la-qemu_capabilities.o qemu/.libs/libvirt_driver_qemu_impl_la-qemu_command.o qemu/.libs/libvirt_driver_qemu_impl_la-qemu_domain.o qemu/.libs/libvirt_driver_qemu_impl_la-qemu_cgroup.o qemu/.libs/libvirt_driver_qemu_impl_la-qemu_hostdev.o qemu/.libs/libvirt_driver_qemu_impl_la-qemu_hotplug.o qemu/.libs/libvirt_driver_qemu_impl_la-qemu_conf.o qemu/.libs/libvirt_driver_qemu_impl_la-qemu_process.o qemu/.libs/libvirt_driver_qemu_impl_la-qemu_migration.o qemu/.libs/libvirt_driver_qemu_impl_la-qemu_monitor.o qemu/.libs/libvirt_driver_qemu_impl_la-qemu_monitor_text.o qemu/.libs/libvirt_driver_qemu_impl_la-qemu_monitor_json.o qemu/.libs/libvirt_driver_qemu_impl_la-qemu_driver.o qemu/.libs/libvirt_driver_qemu_impl_la-qemu_interface.o > ar: `u' modifier ignored since `D' is the default (see `U') > libtool: link: ranlib .libs/libvirt_driver_qemu_impl.a Reading 'man ar', it looks like a relatively new binutils invention, one that Fedora chose to flip on by default, but also one that might be worth using even when ar is built with 'U' rather than 'D' as the default: > D Operate in deterministic mode. When adding files and the archive > index use zero for UIDs, GIDs, timestamps, and use consistent file > modes for all files. When this option is used, if ar is used with > identical options and identical input files, multiple runs will > create identical output files regardless of the input files' > owners, groups, file modes, or modification times. > > If binutils was configured with --enable-deterministic-archives, > then this mode is on by default. It can be disabled with the U > modifier, below. Is it worth teaching libtool to change ARFLAGS to be 'crD' instead of 'cru' when it is detected that ar is new enough to support deterministic libraries, in part to shut up the warning message being printed on every single libtool link line? (Note that I would explicitly include 'D', because even though Fedora chose to make 'D' the default, other distros may choose to make 'U' the default). Or conversely, do we want ARFLAGS to be 'cruU', to force non-deterministic mode, since any speedups made possible by timestamp comparison ('u') are only possible in non-deterministic mode? Does the use of 'u' buy us much measurable time when repeatedly and incrementally linking large libraries, where the new 'D' mode is forced to link everything instead of just the new inputs? And of course there's the question of how easy is it to detect that ar is new enough to understand the 'D'/'U' dichotomy? Is this something I should redirect to automake instead of libtool? -- Eric Blake eblake redhat com +1-919-301-3266 Libvirt virtualization library http://libvirt.org

2 1

[libvirt] [PATCH 0/3] Only talk to qemu guest agent when the domain is running
by Ján Tomko 28 Feb '15

28 Feb '15

Previously, only virDomainObjIsActive() was used to check domain liveness. This includes states like 'VIR_DOMAIN_PMSUSPENDED', where QEMU is running, but the guest is not. Explicitly check the state against VIR_DOMAIN_RUNNING and refuse to communicate with the agent if it does not match. https://bugzilla.redhat.com/show_bug.cgi?id=872424 Ján Tomko (3): Check for qemu guest agent availability after getting the job Pass virDomainObjPtr to qemuDomainAgentAvailable Check if domain is running in qemuDomainAgentIsAvailable src/qemu/qemu_domain.c | 11 ++++++++++- src/qemu/qemu_domain.h | 2 +- src/qemu/qemu_driver.c | 49 +++++++++++++++++++++++++------------------------ 3 files changed, 36 insertions(+), 26 deletions(-) -- 2.0.5

2 4

Re: [libvirt] [Xen-devel] [xen-unstable test] 35257: regressions - FAIL
by Jim Fehlig 27 Feb '15

27 Feb '15

Ian Campbell wrote: > On Thu, 2015-02-26 at 20:14 +0000, xen.org wrote: > >> flight 35257 xen-unstable real [real] >> http://www.chiark.greenend.org.uk/~xensrcts/logs/35257/ >> >> Regressions :-( >> >> Tests which did not succeed and are blocking, >> including tests which could not be run: >> test-armhf-armhf-libvirt 12 guest-start.2 fail REGR. vs. 34629 >> > > logs: > http://www.chiark.greenend.org.uk/~xensrcts/logs/35257/test-armhf-armhf-lib… > > http://www.chiark.greenend.org.uk/~xensrcts/logs/35257/test-armhf-armhf-lib… > 2015-02-23 20:21:48 Z executing ssh ... root(a)10.80.229.106 virsh domxml-from-native xen-xl /etc/xen/debian.guest.osstest.cfg > /etc/xen/debian.guest.osstest.cfg.xml > error: failed to connect to the hypervisor > error: no valid connection > error: Failed to connect socket to '/var/run/libvirt/libvirt-sock': Connection refused > > http://www.chiark.greenend.org.uk/~xensrcts/logs/35257/test-armhf-armhf-lib… > appears to show no libvirtd process. > > http://www.chiark.greenend.org.uk/~xensrcts/logs/35257/test-armhf-armhf-lib… > says: > 2015-02-23 20:13:15.556+0000: 2133: info : libvirt version: 1.2.13 > 2015-02-23 20:13:15.556+0000: 2133: error : dnsmasqCapsRefreshInternal:726 : Cannot check dnsmasq binary dnsmasq: No such file or directory > > 2015-02-23 20:13:15.845+0000: 2133: error : > virFirewallValidateBackend:193 : direct firewall backend requested, > but /sbin/ebtables is not available: No such file or directory Odd, since ebtables was found when building checking for ebtables... /sbin/ebtables But AFAICT, that wont prevent libvirtd from starting. > > I think these are just spurious. > > 2015-02-23 20:13:15.845+0000: 2133: error : virFirewallApply:936 : out of memory > > > 2015-02-23 20:13:16.092+0000: 2133: error : virExec:491 : Cannot find 'pm-is-supported' in path: No such file or directory > 2015-02-23 20:13:16.092+0000: 2133: warning : virQEMUCapsInit:999 : Failed to get host power management capabilities > > As are these two. > > 2015-02-23 20:13:16.400+0000: 2133: error : virFirewallApply:936 : out of memory > > Has these OOM messages resulted in libvirtd exiting? No, I don't think so. The related code is int virFirewallApply(virFirewallPtr firewall) { size_t i, j; int ret = -1; virMutexLock(&ruleLock); if (!firewall || firewall->err == ENOMEM) { virReportOOMError(); goto cleanup; ... } I suspect 'firewall' is null, so OOM error is reported and the function returns -1. But I also don't see this preventing libvirtd from starting. I've cc'd the libvirt list for verification that these errors won't prevent libvirtd from starting. > I don't see any > evidence of a crash elsewhere in the logs (i.e. no "process segfaulted" > in dmesg, no OOM killing going on etc). > > We don't seem to collect dom0 freemem info, but that most likely > wouldn't help given the libvirtd process has exited. > > > Any ideas where to look next? Can you access the test environment and try starting libvirtd in the foreground? Or enable debug log level in /etc/libvirt/libvirtd.conf? Regards, Jim

2 1

[libvirt] [PATCH] Really fix XML formatting flags in SaveImageUpdateDef
by Ján Tomko 27 Feb '15

27 Feb '15

Commit cf2d4c6 used a logical or instead of bitwise or, effectively passing 1, that is VIR_DOMAIN_XML_INACTIVE. Yep, that's right. This was caught by a warning when building with clang. https://bugzilla.redhat.com/show_bug.cgi?id=1183869 --- src/qemu/qemu_driver.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/qemu/qemu_driver.c b/src/qemu/qemu_driver.c index e9aa0b5..e282464 100644 --- a/src/qemu/qemu_driver.c +++ b/src/qemu/qemu_driver.c @@ -5734,7 +5734,7 @@ qemuDomainSaveImageUpdateDef(virQEMUDriverPtr driver, if (!(newdef_migr = qemuDomainDefCopy(driver, newdef, - QEMU_DOMAIN_FORMAT_LIVE_FLAGS || + QEMU_DOMAIN_FORMAT_LIVE_FLAGS | VIR_DOMAIN_XML_MIGRATABLE))) goto cleanup; -- 2.0.5

3 2