- Devel - libvirt List Archives

[libvirt] [PATCH] Post-release version bump to 1.2.17
by Daniel P. Berrange 01 Jun '15

01 Jun '15

--- configure.ac | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) Pushed under trivial rule. NB we might decide to switch to 1.3.0 before next release, so this is just placeholder for now. diff --git a/configure.ac b/configure.ac index 6122fa5..0da3ae8 100644 --- a/configure.ac +++ b/configure.ac @@ -16,7 +16,7 @@ dnl You should have received a copy of the GNU Lesser General Public dnl License along with this library. If not, see dnl <http://www.gnu.org/licenses/>. -AC_INIT([libvirt], [1.2.16], [libvir-list(a)redhat.com] [], [http://libvirt.org] +AC_INIT([libvirt], [1.2.17], [libvir-list(a)redhat.com] [], [http://libvirt.org] AC_CONFIG_SRCDIR([src/libvirt.c]) AC_CONFIG_AUX_DIR([build-aux]) AC_CONFIG_HEADERS([config.h]) -- 2.4.1

1 0

[libvirt] [PATCH python] Include tests in MANIFEST
by Daniel P. Berrange 01 Jun '15

01 Jun '15

The unit tests were missing from the tar.gz archives due to not being listed in the MANIFEST.in file Signed-off-by: Daniel P. Berrange <berrange(a)redhat.com> --- MANIFEST.in | 2 ++ 1 file changed, 2 insertions(+) diff --git a/MANIFEST.in b/MANIFEST.in index 230baea..b5ba783 100644 --- a/MANIFEST.in +++ b/MANIFEST.in @@ -34,6 +34,8 @@ include MANIFEST include README include sanitytest.py include setup.py +include tests/test_conn.py +include tests/test_domain.py include tox.ini include typewrappers.c include typewrappers.h -- 2.4.1

1 0

[libvirt] [PATCHv2] qemu: fix unsuitable error report when get memory stats
by Wang Yufei 01 Jun '15

01 Jun '15

From: Zhang Bo <oscar.zhangbo(a)huawei.com> when we run the command 'virsh dommemstat xxx', althrough memballoon's model is set 'none' in vm's XML, it still reports an error in libvirtd.log. error : qemuMonitorFindBalloonObjectPath:1042 : internal error: Cannot determine balloon device path Apparently, if we don't set memballoon, we don't need to set balloon device path. Signed-off-by: Wang Yufei <james.wangyufei(a)huawei.com> Signed-off-by: Zhang Bo <oscar.zhangbo(a)huawei.com> --- src/qemu/qemu_monitor.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/src/qemu/qemu_monitor.c b/src/qemu/qemu_monitor.c index f959b74..c72702d 100644 --- a/src/qemu/qemu_monitor.c +++ b/src/qemu/qemu_monitor.c @@ -1711,7 +1711,9 @@ qemuMonitorGetMemoryStats(qemuMonitorPtr mon, QEMU_CHECK_MONITOR(mon); if (mon->json) { - ignore_value(qemuMonitorFindBalloonObjectPath(mon, "/")); + if (mon->vm->def->memballoon && + mon->vm->def->memballoon->model != VIR_DOMAIN_MEMBALLOON_MODEL_NONE) + ignore_value(qemuMonitorFindBalloonObjectPath(mon, "/")); mon->ballooninit = true; return qemuMonitorJSONGetMemoryStats(mon, mon->balloonpath, stats, nr_stats); -- 1.7.12.4

2 1

[libvirt] [libvirt-glib] storage-pool: API to get/set autostart flag
by Zeeshan Ali (Khattak) 01 Jun '15

01 Jun '15

Add binding for virStoragePoolGetAutostart & virStoragePoolSetAutostart. --- libvirt-gobject/libvirt-gobject-storage-pool.c | 52 ++++++++++++++++++++++++++ libvirt-gobject/libvirt-gobject-storage-pool.h | 5 +++ libvirt-gobject/libvirt-gobject.sym | 6 +++ 3 files changed, 63 insertions(+) diff --git a/libvirt-gobject/libvirt-gobject-storage-pool.c b/libvirt-gobject/libvirt-gobject-storage-pool.c index f3eac0d..e520c6a 100644 --- a/libvirt-gobject/libvirt-gobject-storage-pool.c +++ b/libvirt-gobject/libvirt-gobject-storage-pool.c @@ -1048,6 +1048,58 @@ gboolean gvir_storage_pool_delete (GVirStoragePool *pool, return TRUE; } +/** + * gvir_storage_pool_get_autostart: + * @pool: the storage pool + * @err: return location for any #GError + * + * Return value: #True if autostart is enabled, #False otherwise. + */ +gboolean gvir_storage_pool_get_autostart(GVirStoragePool *pool, + GError **err) +{ + gboolean ret = FALSE; + + g_return_val_if_fail(GVIR_IS_STORAGE_POOL(pool), ret); + g_return_val_if_fail(err == NULL || *err == NULL, ret); + + if (virStoragePoolGetAutostart(pool->priv->handle, &ret)) { + gvir_set_error_literal(err, GVIR_STORAGE_POOL_ERROR, + 0, + "Failed to get autostart flag from storage pool"); + return FALSE; + } + + return ret; +} + +/** + * gvir_storage_pool_set_autostart: + * @pool: the storage pool + * @autostart: New value for autostart flag + * @err: return location for any #GError + * + * Sets whether or not storage pool @pool is started automatically on boot. + * + * Return value: #TRUE on success, #FALSE otherwise. + */ +gboolean gvir_storage_pool_set_autostart(GVirStoragePool *pool, + gboolean autostart, + GError **err) +{ + g_return_val_if_fail(GVIR_IS_STORAGE_POOL(pool), FALSE); + g_return_val_if_fail(err == NULL || *err == NULL, FALSE); + + if (virStoragePoolSetAutostart(pool->priv->handle, autostart)) { + gvir_set_error_literal(err, GVIR_STORAGE_POOL_ERROR, + 0, + "Failed to set autostart flag on storage pool"); + return FALSE; + } + + return TRUE; +} + static void gvir_storage_pool_delete_helper(GSimpleAsyncResult *res, GObject *object, diff --git a/libvirt-gobject/libvirt-gobject-storage-pool.h b/libvirt-gobject/libvirt-gobject-storage-pool.h index f8529f0..f7f879c 100644 --- a/libvirt-gobject/libvirt-gobject-storage-pool.h +++ b/libvirt-gobject/libvirt-gobject-storage-pool.h @@ -166,6 +166,11 @@ void gvir_storage_pool_delete_async (GVirStoragePool *pool, gboolean gvir_storage_pool_delete_finish(GVirStoragePool *pool, GAsyncResult *result, GError **err); +gboolean gvir_storage_pool_get_autostart(GVirStoragePool *pool, + GError **err); +gboolean gvir_storage_pool_set_autostart(GVirStoragePool *pool, + gboolean autostart, + GError **err); G_END_DECLS diff --git a/libvirt-gobject/libvirt-gobject.sym b/libvirt-gobject/libvirt-gobject.sym index 927cad9..dcda675 100644 --- a/libvirt-gobject/libvirt-gobject.sym +++ b/libvirt-gobject/libvirt-gobject.sym @@ -265,4 +265,10 @@ LIBVIRT_GOBJECT_0.2.0 { gvir_domain_open_graphics_fd; } LIBVIRT_GOBJECT_0.1.9; +LIBVIRT_GOBJECT_0.2.1 { + global: + gvir_storage_pool_get_autostart; + gvir_storage_pool_set_autostart; +} LIBVIRT_GOBJECT_0.2.0; + # .... define new API here using predicted next version number .... -- 2.1.0

2 4

[libvirt] Release of libvirt-1.2.16
by Daniel Veillard 01 Jun '15

01 Jun '15

It's out ! Tagged in git, signed tarball and rpms are in place: ftp://libvirt.org/libvirt/ I also made a libvirt-python-1.2.16 release fund at: ftp://libvirt.org/libvirt/python/ This is a releatively small release, less than 150 commits, maybe the effect of the travels to OpenStack summit this month, also bigger pieces lining up for 1.3.0 planned for next month. There is still a few features, portability patches and tuning for ARM 64, and a fair amount of bug fixes, this should be a relatively safe release (famous last words :-) Features: - Introduce pci-serial (Michal Privoznik) - Introduce virDomainSetUserPassword API (Ján Tomko) - libvirt: Introduce protected key mgmt ops (Tony Krowiak) - add domain vmport feature (Marc-André Lureau) Documentation: - update github project name (Daniel P. Berrange) - formatstorage: Update <permissions> docs (Cole Robinson) - Document new RO repo mirrors (Michal Privoznik) - Document that virNodeGetInfo can return mhz == 0. (Richard W.M. Jones) - fix a small xml error in docs (Luyao Huang) - tests: Fix grammar in comments. (Andrea Bolognani) - add domain vmport feature (Marc-André Lureau) Portability: - qemu: Fix compilation error when enum variable size differs from 'int' (Peter Krempa) - bhyve: fix build with gcc48 (Roman Bogorodskiy) - Add missing XDR_FLAGS (Pavel Fedin) - node_device: fix libvirt build if WITH_HAL is defined (Maxim Nestratov) - util: Avoid shadow of 'ulong' in virMemoryMaxValue (Peter Krempa) - Fix build --without-network (Martin Kletzander) - maint: extend PIE support check (Roman Bogorodskiy) - bhyve: fix bhyvexml2argvtest build with gcc (Roman Bogorodskiy) Bug Fixes: - Allocate priv->vioserialaddrs unconditionally (Ján Tomko) - Properly free the xmlDocPtr when loading pool state (Ján Tomko) - storage: Fix problem with disk backend pool allocation calculation (John Ferlan) - storage: Don't adjust pool alloc/avail values for disk backend (John Ferlan) - virsh: Fix to list online cpus using virsh capabilities (Kothapally Madhu Pavan) - Fix the event name in vshEventTrayChangePrint (Ján Tomko) - Fix shrinking volumes with the delta flag (Ján Tomko) - interface: don't error out if a bond has no interfaces (Lubomir Rintel) - qemu: Limit rtc-reset-reinjection requirement to x86 only. (Andrea Bolognani) - storage_fs: Create directory with UID if needed (Martin Kletzander) - zfs: fix storagepoolxml2xml test (Roman Bogorodskiy) - qemu: Force capabilities cache refresh if libvirtd date is different (John Ferlan) - qemu: Resolve Coverity RESOURCE_LEAK (John Ferlan) - conf: Resolve Coverity NEGATIVE_RETURNS (John Ferlan) - storage: fs: Only force directory permissions if required (Cole Robinson) - parallels: fix possible crash in case of errors in prlsdkLoadDomain (Maxim Nestratov) - virsh: reject negative values for scaled integer (Pavel Hrdina) - network: Resolve Coverity FORWARD_NULL (John Ferlan) - conf: Resolve Coverity FORWARD_NULL (John Ferlan) - storage: Resolve Coverity FORWARD_NULL (John Ferlan) - netdev: fail when setting up an SRIOV VF if PF is offline (Laine Stump) - qemuDomainDetachChrDevice: Fix chardev hot-unplug (Michal Privoznik) - qemuDomainAttachChrDevice: Fix chardev hotplug (Michal Privoznik) - qemu: Log error if domain uses security driver which is not loaded (Erik Skultety) - conf: Restore the XML parser context in virDomainMemoryDefParseXML (Luyao Huang) - conf: Catch memory size overflow earlier (Peter Krempa) - virfile: virDirCreate: Fix ALLOW_EXIST conditional (Cole Robinson) - qemu: Fix numatune nodeset reporting (Martin Kletzander) - xenconfig: fix spice mousemode and copypaste (Jim Fehlig) - xenconfig: fix spicepasswd handling (Jim Fehlig) - node_device: update sriov/iommu info before dumpxml of a device (Laine Stump) - virsh: Fix dommemstat --period option type. (Andrea Bolognani) - qemu: fix exceptions in qemuAssignDeviceControllerAlias (Laine Stump) - qemu: Don't give up on first error in qemuMigrationCancelDriverMirror (Jiri Denemark) - reject out of range memory in SetMemory APIs (Ján Tomko) - qemu: Clear autofil fill flag when pinning iothread (John Ferlan) - virsh: Report an error when cpulist parsing fails (Luyao Huang) - Assign virtio-serial addresses after coldplugging a device (Ján Tomko) - qemu: vnc: error out for invalid port number (Pavel Hrdina) - conf: Remove source host name check for iSCSI (John Ferlan) - qemu: fix double free when RNG cold-plug fails (Luyao Huang) - libvirt-guests: Initialize SYNC_TIME (Michal Privoznik) - conf: Report error for unknown shmem ioeventfd value (Luyao Huang) - qemu: Fix balloon size handling with memory hot(un)plug (Peter Krempa) - conf: Fix up balloon size after removing a memory device from def (Peter Krempa) - caps: Fix regression defaulting to host arch (Cole Robinson) - processSerialChangedEvent: Close agent monitor early (Michal Privoznik) - qemuDomainShutdownFlags: check for domain activeness prior to guest presence (Michal Privoznik) - lxc: don't up the veth interfaces unless explicitly asked to (Lubomir Rintel) - qemu: Resolve Coverity FORWARD_NULL (John Ferlan) - qemu: Resolve Coverity FORWARD_NULL (John Ferlan) - qemu: Resolve Coverity FORWARD_NULL (John Ferlan) - xen: Resolve Coverity FORWARD_NULL (John Ferlan) - qemu: Resolve Coverity FORWARD_NULL (John Ferlan) - qemu: Resolve Coverity IDENTICAL_BRANCHES (John Ferlan) - vbox: Resolve Coverity RESOURCE_LEAK (John Ferlan) - libxl: Resolve Coverity RESOURCE_LEAK (John Ferlan) - conf: Add the cpu duplicate use check for vm numa settings (Luyao Huang) - qemu: Properly rename persistent def after migration (Jiri Denemark) - storage: fs: Don't try to chown directory unless user requested (Cole Robinson) - storage: fs: Don't attempt directory creation if it already exists (Cole Robinson) - storage: fs: Don't overwrite virDirCreate error (Cole Robinson) - virsh: Don't check migrate parameters (Jiri Denemark) - qemu: Forbid unsupported parameters for tunnelled migration (Jiri Denemark) - qemu: fix the vm deadlock when deleting a nonexistent iothread (Luyao Huang) - tools: fix the wrong check when use virsh setvcpus --maximum (Luyao Huang) Improvements: - libxl: support QXL video device (Jim Fehlig) - libxl: support SPICE graphics for HVM domains (Jim Fehlig) - libxl: change reservedVNCPorts to reservedGraphicsPorts (Jim Fehlig) - libxl: populate build_info vfb in separate function (Jim Fehlig) - debug: assure NULLSTR() around all %s args in debug at top of public APIs (Laine Stump) - node_device: more informative error log when device isn't found (Laine Stump) - Simplify allocation check in storageVolResize (Ján Tomko) - parallels: suppress console output from parallels SDK (Maxim Nestratov) - util: make it more robust to calculate timeout value (Zhang Bo) - util: improve the sysinfo element XML format (Luyao Huang) - conf: Avoid formatting empty redirfilter element (Luyao Huang) - maint: update to latest gnulib (Eric Blake) - qemu: Add libvirt version check to refresh capabilities algorithm (John Ferlan) - conf: storage: Don't emit empty <permissions> block (Cole Robinson) - storage: conf: Don't set any default <mode> in the XML (Cole Robinson) - parallels: move up updating parameter in prlsdkLoadDomain (Maxim Nestratov) - zfs: fix storagepoolxml2xml test (Roman Bogorodskiy) - util: better error message after failure to initialize firewall backend (Laine Stump) - interface: allow multiple IPv4 addresses + dhcp on a single interface (Laine Stump) - storage: conf: Don't output owner/group -1 (Cole Robinson) - sysinfo: Fix reports on ARM (Michal Privoznik) - qemu: Implement pci-serial (Michal Privoznik) - qemu: wire up virDomainSetUserPassword (Ján Tomko) - virsh: add set-user-password command (Ján Tomko) - threadpool: Switch to detached threads (Jiri Denemark) - Use virDomainDiskByName where appropriate (Jiri Denemark) - Add wrappers for virDomainDiskIndexBy* (Jiri Denemark) - virDomainNumatuneGetMode: Report if numatune was defined (Michal Privoznik) - Taint domains using cdrom-passthrough (John Ferlan) - virfile: virDirCreate: Drop redundant FORCE_PERMS flag (Cole Robinson) - xenconfig: format spice listenAddr when formating ports (Jim Fehlig) - xenconfig: use local variable for graphics def (Jim Fehlig) - node_device: replace duplicated code in hal and udev backends (Laine Stump) - node_device: new functions to get sriov/iommu info from sysfs (Laine Stump) - node device: prepare node_device_linux_sysfs.c to add more functions (Laine Stump) - nodedev: change if-else if in update_caps to switch (Laine Stump) - conf: make virNodeDevCapData an official type (Laine Stump) - virsh: Improve handling of send-process-signal --pid. (Andrea Bolognani) - virsh: Improve error message on integer value parsing failure. (Andrea Bolognani) - libvirt: tests: test protected key mgmt ops support (Tony Krowiak) - libvirt: qemu: enable/disable protected key management ops (Tony Krowiak) - libxl: provide impl for nodeGetSecurityModel (Jim Fehlig) - qemu: log error when domain has an unsupported IDE controller (Laine Stump) - qemu: clean up qemuBuildCommandline loop that builds controller args (Laine Stump) - qemu: remove test for allowing ide controller in s390, rename usb tests (Laine Stump) - qemu: use controller alias when constructing device/controller args (Laine Stump) - conf: utility to return alias of a controller based on type/index (Laine Stump) - qemu: Keep track of what disks are being migrated (Jiri Denemark) - Move QEMU-only fields from virDomainDiskDef into privateData (Jiri Denemark) - Rename virDomainHasBlockjob as qemuDomainHasBlockjob (Jiri Denemark) - Add privateData to virDomainDiskDef (Jiri Denemark) - Some alignment fixes in lxc_controller and jsontest (Martin Kletzander) - gendispatch: Don't generate long lines (Martin Kletzander) - virnetserver: Remove unnecessary double space (Martin Kletzander) - rpc: Don't mix max_clients and max_workers in PostExecRestart (Martin Kletzander) - conf: Expose iothreadids when delete non sequential iothreadids (John Ferlan) - parallels: remove connection wide wait timeout (Nikolay Shirokovskiy) - qemuMigrationPrepareAny: Drop useless variable @now (zhang bo) - maint: Ignore Vim swap files (Andrea Bolognani) - virSysinfoParseProcessor: Drop useless check for NULL (Michal Privoznik) - libxl: support VNC passwd (Jim Fehlig) - conf: Adjust duplicate source host port check (John Ferlan) - daemon: Suppress logging of VIR_ERR_NO_DOMAIN_METADATA (Peter Krempa) - conf_capabilities: fix wrong indentation (Pavel Hrdina) - XML: escape strings where we should do it (Pavel Hrdina) - qemu: eliminate duplicated code in qemuBuildDriveDevStr() (Laine Stump) - qemu: use qemuDomainMachineIsI440FX() in appropriate place (Laine Stump) - Ignore bridge template names with multiple printf conversions (Ján Tomko) - conf: Always truncate balloon size to maximum memory size (Peter Krempa) - qemu: Convert qemuConnectGetAllDomainStats to use new helpers (Peter Krempa) - conf: Add helper to convert list of virDomains to a list of virDomainObjs (Peter Krempa) - conf: Refactor domain list collection critical section (Peter Krempa) - conf: Rename virDomainObjListFilter type to virDomainObjListACLFilter (Peter Krempa) - conf: Extract code to filter domain list into a separate function (Peter Krempa) - util: Make the virDomainListFree helper more universal (Peter Krempa) - tests: Remove redundant aarch64 tests (Cole Robinson) - qemu: multiqueue for ccw devices (Boris Fiuczynski) - libxl: support soundhw for hvm domains (Jim Fehlig) - libxl: add logrotate config file (Jim Fehlig) - qemu: Implement GIC (Michal Privoznik) - Introduce GIC feature (Michal Privoznik) - polkit: Allow password-less access for 'libvirt' group (Cole Robinson) - storage: fs: Fill in permissions on pool refresh (Cole Robinson) - tests: add machine vmport qemu test (Marc-André Lureau) - qemu: add machine vmport argument (Marc-André Lureau) - qemu: add virQEMUCapsSupportsVmport (Marc-André Lureau) - qemu: move qemuDomainMachineIs{I440FX,Q35} (Marc-André Lureau) - qemu: add QEMU_CAPS_MACHINE_VMPORT_OPT (Marc-André Lureau) - domain/conf: add VIR_DOMAIN_FEATURE_VMPORT (Marc-André Lureau) - virsh: introduce new macros to help check flag requirements (Pavel Hrdina) - qemu: use new macros for setvcpus to check flags and cleanup the code (Pavel Hrdina) - use new macro helpers to check flag requirements (Pavel Hrdina) - use new macro helpers to check exclusive flags (Pavel Hrdina) - internal: introduce macro helpers to check flag requirements (Pavel Hrdina) - internal: introduce macro helpers to reject exclusive flags (Pavel Hrdina) So thanks everybody for your help pulling out this last release of the 1.2 cycle, be it with patches, ideas, reports, reviews ... Enjoy ! 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] check if console PTY is null before attempting to open
by Shivaprasad G Bhat 31 May '15

31 May '15

Console devices have their pty devices assigned when the qemu is actually started. The libvirt spends considerable amount of time to start the qemu if the guest has multiple passthrough devices. If time is spent during the hostdev preparation, someone attempts to open the console, the libvirt crashes in virChrdevLockFilePath().The patch attempts to fix the crash by adding a check before attempting to open. Signed-off-by: Shivaprasad G Bhat <sbhat(a)linux.vnet.ibm.com> --- src/qemu/qemu_driver.c | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/src/qemu/qemu_driver.c b/src/qemu/qemu_driver.c index 1233d8f..fec928f 100644 --- a/src/qemu/qemu_driver.c +++ b/src/qemu/qemu_driver.c @@ -16278,6 +16278,12 @@ qemuDomainOpenConsole(virDomainPtr dom, goto cleanup; } + if (!(chr->source.data.file.path)) { + virReportError(VIR_ERR_OPERATION_FAILED, "%s", + _("PTY device is not yet assigned")); + goto cleanup; + } + /* handle mutually exclusive access to console devices */ ret = virChrdevOpen(priv->devs, &chr->source,

3 2

[libvirt] [PATCH 0/3] Better error reporting of bad IP address ranges
by Laine Stump 29 May '15

29 May '15

This resolves: https://bugzilla.redhat.com/show_bug.cgi?id=985653 Laine Stump (3): network: validate DHCP ranges are completely within defined network network: cleanup range loop in networkDnsmasqConfContents util: report all address range errors in virSocketAddrGetRange() src/conf/network_conf.c | 18 +-- src/network/bridge_driver.c | 24 +-- src/util/virsocketaddr.c | 180 ++++++++++++++++++--- src/util/virsocketaddr.h | 6 +- tests/networkxml2xmlupdatein/dhcp-range-10.xml | 1 + tests/networkxml2xmlupdatein/dhcp-range.xml | 2 +- .../dhcp6host-routed-network-another-range.xml | 2 +- .../dhcp6host-routed-network-range.xml | 2 +- tests/networkxml2xmlupdatetest.c | 5 + tests/sockettest.c | 55 ++++--- 10 files changed, 228 insertions(+), 67 deletions(-) create mode 100644 tests/networkxml2xmlupdatein/dhcp-range-10.xml -- 2.1.0

3 6

[libvirt] [PATCH V2 libvirt-python] Fix duplicate entries in AUTHORS
by Jim Fehlig 29 May '15

29 May '15

The generated AUTHORS file contains many duplicates. If an author has N commits, there will be N entries for the author in AUTHORS. Check if an author already exists in the list before appending. While at it, add a .mailmap (derived from libivrt's .mailmap) to futher tidy the generated AUTHORS list. Signed-off-by: Jim Fehlig <jfehlig(a)suse.com> --- V2: Add .mailmap. I started with libvirt's .mailmap and removed entries where there was no corresponding author in the libvirt-python repo. .mailmap | 16 ++++++++++++++++ setup.py | 4 +++- 2 files changed, 19 insertions(+), 1 deletion(-) diff --git a/.mailmap b/.mailmap new file mode 100644 index 0000000..d8ac934 --- /dev/null +++ b/.mailmap @@ -0,0 +1,16 @@ +# 'git shortlog --help' and look for mailmap for the format of each line + +# Email consolidation: +# <Preferred address in AUTHORS> <other alias used by same author> + +<jdenemar(a)redhat.com> <Jiri.Denemark(a)gmail.com> +<jfehlig(a)suse.com> <jfehlig(a)novell.com> +<stefanb(a)us.ibm.com> <stefanb(a)linux.vnet.ibm.com> + +# Name consolidation: +# Preferred author spelling <preferred email> +Alex Jia <ajia(a)redhat.com> +Ján Tomko <jtomko(a)redhat.com> +MATSUDA Daiki <matsudadik(a)intellilink.co.jp> +Serge E. Hallyn <serge.hallyn(a)canonical.com> +Philipp Hahn <hahn(a)univention.de> diff --git a/setup.py b/setup.py index 2571742..7a1964b 100755 --- a/setup.py +++ b/setup.py @@ -176,7 +176,9 @@ class my_sdist(sdist): f = os.popen("git log --pretty=format:'%aN <%aE>'") authors = [] for line in f: - authors.append(" " + line.strip()) + line = " " + line.strip() + if line not in authors: + authors.append(line) authors.sort(key=str.lower) -- 2.3.7

2 2

[libvirt] [PATCH] Allocate priv->vioserialaddrs unconditionally
by Ján Tomko 29 May '15

29 May '15

When attempting to hotplug a virtio-serial console to a domain that had no virtio-serial controllers (not even those that are added by libvirt when some devices need them) at daemon startup, report a user-friendly error: error: Failed to attach device from console.xml error: internal error: no virtio-serial controllers are available instead of crashing the daemon: Process terminating with default action of signal 11 (SIGSEGV): dumping core Access not within mapped region at address 0x8 at 0x531028F: virDomainVirtioSerialAddrNext (domain_addr.c:916) by 0x531028F: virDomainVirtioSerialAddrAssign (domain_addr.c:1029) by 0x1CBF68: qemuDomainAttachChrDevice (qemu_hotplug.c:1565) by 0x1BCD5E: qemuDomainAttachDeviceLive (qemu_driver.c:7997) by 0x1BCD5E: qemuDomainAttachDeviceFlags (qemu_driver.c:8743) Introduced in v1.2.14-30-g5903378. --- src/qemu/qemu_command.c | 3 --- 1 file changed, 3 deletions(-) diff --git a/src/qemu/qemu_command.c b/src/qemu/qemu_command.c index d8ce511..11c6823 100644 --- a/src/qemu/qemu_command.c +++ b/src/qemu/qemu_command.c @@ -1447,9 +1447,6 @@ qemuDomainAssignVirtioSerialAddresses(virDomainDefPtr def, virDomainVirtioSerialAddrSetPtr addrs = NULL; qemuDomainObjPrivatePtr priv = NULL; - if (virDomainControllerFindByType(def, VIR_DOMAIN_CONTROLLER_TYPE_VIRTIO_SERIAL) == -1) - return 0; - if (!(addrs = virDomainVirtioSerialAddrSetCreate())) goto cleanup; -- 2.3.6

2 1

[libvirt] [PATCH] Properly free the xmlDocPtr when loading pool state
by Ján Tomko 29 May '15

29 May '15

Use xmlFreeDoc instead of plain xmlFree. 4 bytes in 1 blocks are definitely lost in loss record 9 of 1,084 at 0x4C29F80: malloc (in /usr/lib64/valgrind/vgpreload_memcheck-amd64-linux.so) by 0x70730D6: xmlStrndup (in /usr/lib64/libxml2.so.2.9.2) by 0x701E3DC: xmlNewDoc (in /usr/lib64/libxml2.so.2.9.2) by 0x70C39F8: xmlSAX2StartDocument (in /usr/lib64/libxml2.so.2.9.2) by 0x7017245: xmlParseDocument (in /usr/lib64/libxml2.so.2.9.2) by 0x7017606: xmlDoRead (in /usr/lib64/libxml2.so.2.9.2) by 0x5309DAD: virXMLParseHelper (virxml.c:742) by 0x5367584: virStoragePoolLoadState (storage_conf.c:1863) --- src/conf/storage_conf.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/conf/storage_conf.c b/src/conf/storage_conf.c index 7857a5e..df536d4 100644 --- a/src/conf/storage_conf.c +++ b/src/conf/storage_conf.c @@ -1894,7 +1894,7 @@ virStoragePoolLoadState(virStoragePoolObjListPtr pools, cleanup: VIR_FREE(stateFile); - xmlFree(xml); + xmlFreeDoc(xml); xmlXPathFreeContext(ctxt); return pool; -- 2.3.6

2 1