- Devel - libvirt List Archives

[libvirt] [python PATCH] libvirt-override: fix some build warnings
by Pavel Hrdina 02 Sep '14

02 Sep '14

Remove unused label 'cleanup' in 'libvirt_virConnectGetAllDomainStats' function and remove unused variable 'conn' in function 'libvirt_virDomainListGetStats'. Signed-off-by: Pavel Hrdina <phrdina(a)redhat.com> --- Pushed as trivial libvirt-override.c | 3 --- 1 file changed, 3 deletions(-) diff --git a/libvirt-override.c b/libvirt-override.c index dfbe9d3..17ea042 100644 --- a/libvirt-override.c +++ b/libvirt-override.c @@ -8039,7 +8039,6 @@ libvirt_virConnectGetAllDomainStats(PyObject *self ATTRIBUTE_UNUSED, if (!(py_retval = convertDomainStatsRecord(records, nrecords))) py_retval = VIR_PY_NONE; - cleanup: virDomainStatsRecordListFree(records); return py_retval; @@ -8053,7 +8052,6 @@ libvirt_virDomainListGetStats(PyObject *self ATTRIBUTE_UNUSED, PyObject *pyobj_conn; PyObject *py_retval; PyObject *py_domlist; - virConnectPtr conn; virDomainStatsRecordPtr *records = NULL; virDomainPtr *doms = NULL; int nrecords; @@ -8065,7 +8063,6 @@ libvirt_virDomainListGetStats(PyObject *self ATTRIBUTE_UNUSED, if (!PyArg_ParseTuple(args, (char *)"OOii:virDomainListGetStats", &pyobj_conn, &py_domlist, &stats, &flags)) return NULL; - conn = (virConnectPtr) PyvirConnect_Get(pyobj_conn); if (PyList_Check(py_domlist)) { ndoms = PyList_Size(py_domlist); -- 1.8.5.5

1 0

[libvirt] [libvirt-python][PATCH] Post-release version bump for new dev cycle
by Michal Privoznik 02 Sep '14

02 Sep '14

Signed-off-by: Michal Privoznik <mprivozn(a)redhat.com> --- Pushed under trivial rule. setup.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/setup.py b/setup.py index f5c6542..167f568 100755 --- a/setup.py +++ b/setup.py @@ -309,7 +309,7 @@ class my_clean(clean): _c_modules, _py_modules = get_module_lists() setup(name = 'libvirt-python', - version = '1.2.8', + version = '1.2.9', url = 'http://www.libvirt.org', maintainer = 'Libvirt Maintainers', maintainer_email = 'libvir-list(a)redhat.com', -- 1.8.5.5

1 0

[libvirt] [PATCH] Post-release version bump for new dev cycle
by Michal Privoznik 02 Sep '14

02 Sep '14

Signed-off-by: Michal Privoznik <mprivozn(a)redhat.com> --- Pushed under trivial rule. configure.ac | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/configure.ac b/configure.ac index f93c6c2..b4fb99a 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.8], [libvir-list(a)redhat.com] [], [http://libvirt.org] +AC_INIT([libvirt], [1.2.9], [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]) -- 1.8.5.5

1 0

[libvirt] Release of libvirt-1.2.8
by Daniel Veillard 02 Sep '14

02 Sep '14

It's out ! Tagged and signed in git as well as tarball and rpms are available from the usual place: ftp://libvirt.org/libvirt/ I also made a release for the python bindings available from ftp://libvirt.org/libvirt/python/ This is a rather large release (surprizingly for August !) with a lot of refactoring, new APIs and a large set of bug fixes (seems there is a new and more stringent Coverity release :-) . No lack of improvement patches either so a rather balanced release: Features: - blockcopy: virDomainBlockCopy with XML destination, typed params (Eric Blake) - lib: Introduce API for retrieving bulk domain stats (Peter Krempa) - Introduce virDomainOpenGraphicsFD API (Ján Tomko) - storage: ZFS support (Roman Bogorodskiy) Documentation: - fix bootmenu timeout description (Martin Kletzander) - conf: add support for bootmenu timeout (Martin Kletzander) - maint: fix comment typo (Eric Blake) - formatdomain: Reformat vCPU description (John Ferlan) - virsh: Fix help info for freepages (Li Yang) - virsh: man: Add LXC format info for domxml-from/to-native (Li Yang) - virsh: Fix comment for net-undefine (Li Yang) - man: virsh: add missing auto-converge option for 'migrate' (Pradipta Kr. Banerjee) - nwfilter: add missing dscp attribute (Jianwei Hu) - virsh: man: Crosslink "desc" and "metadata" sections (Peter Krempa) - man: virsh: Add 'vcpu_period' and 'vcpu_quota' support info for LXC (Li Yang) - man: virsh: Add man page for "virsh metadata" (Peter Krempa) - fix missing forward slash (Jianwei Hu) - use correct hints per bus type in <disk> examples (Eric Blake) - use unique dev names in <disk> examples (Eric Blake) - virsh: clean up attach-interface paragraph in man page (Laine Stump) - audit: Fix some comments (Wang Rui) Portability: - spec: drop anything older than Fedora 13 (Eric Blake) - fix mingw build (Pavel Hrdina) - build: fix mingw build with virCommandReorderFDs (Martin Kletzander) - build: force configure failed when perl is missing (Jincheng Miao) - Include param.h in case of HAVE_BSD_CPU_AFFINITY (Guido Günther) Bug Fixes: - blockcopy: allow larger buf-size (Eric Blake) - selinux: properly label tap FDs with imagelabel (Martin Kletzander) - Fix connection to already running session libvirtd (Christophe Fergeau) - storage: zfs: fix double listing of new volumes (Roman Bogorodskiy) - qemu_driver: Resolve Coverity FORWARD_NULL (John Ferlan) - virnetserverservice: Resolve Coverity ARRAY_VS_SINGLETON (John Ferlan) - libxl_migration: Resolve Coverity NULL_RETURNS (John Ferlan) - qemu_command: Resolve Coverity DEADCODE (John Ferlan) - qemu_driver: Resolve Coverity DEADCODE (John Ferlan) - domain_conf: Resolve Coverity DEADCODE (John Ferlan) - qemu_monitor: Resolve Coverity NESTING_INDENT_MISMATCH (John Ferlan) - storage_conf: Resolve Coverity RESOURCE_LEAK (John Ferlan) - qemu_driver: Resolve Coverity RESOURCE_LEAK (John Ferlan) - phyp_driver: Resolve Coverity RESOURCE_LEAK (John Ferlan) - libxl_migration: Resolve Coverity RESOURCE_LEAK (John Ferlan) - bridge_driver: Resolve Coverity RESOURCE_LEAK (John Ferlan) - virsh-network: Resolve Coverity RESOURCE_LEAK (John Ferlan) - network_conf: Resolve Coverity RESOURCE_LEAK (John Ferlan) - qemu_capabilities: Resolve Coverity RESOURCE_LEAK (John Ferlan) - libxl_domain: Resolve Coverity RESOURCE_LEAK (John Ferlan) - qemu_agent: Resolve Coverity RESOURCE_LEAK (John Ferlan) - qemu_command: Resolve Coverity RESOURCE_LEAK (John Ferlan) - cpu_x86: Resolve Coverity RESOURCE_LEAK (John Ferlan) - domain_conf: Resolve Coverity RESOURCE_LEAK (John Ferlan) - daemon: Resolve Coverity NEGATIVE_RETURNS (John Ferlan) - qemu_capabilities: Resolve Coverity RESOURCE_LEAK (Wang Rui) - tests: Resolve Coverity RESOURCE_LEAK (Wang Rui) - util: Resolve Coverity RESOURCE_LEAK (Wang Rui) - virsh: fix keepalive error msg (Erik Skultety) - qemu_capabilities: Resolve Coverity NULL_RETURNS (John Ferlan) - qemu_driver: Resolve Coverity CONSTANT_EXPRESSION_RESULT (John Ferlan) - domain_conf: Resolve Coverity REVERSE_INULL (John Ferlan) - qemu_command: Resolve Coverity REVERSE_INULL (John Ferlan) - domain_conf: Resolve Coverity REVERSE_INULL (John Ferlan) - storage_driver: Resolve Coverity REVERSE_INULL (John Ferlan) - xen_xm: Resolve Coverity USE_AFTER_FREE (John Ferlan) - xen_common: Resolve Coverity USE_AFTER_FREE (John Ferlan) - parallels: Resolve Coverity USE_AFTER_FREE (John Ferlan) - conf: fix leak with def->mem.hugepages (Martin Kletzander) - qemu: call endjob in RevertToSnapshot (Jincheng Miao) - virsh: Initialize vshData in cmdMigrate (Ján Tomko) - libxl: fix memory corruption introduced by commit b55cc5f4e (Jim Fehlig) - iotune: setting an invalid value now reports error (Erik Skultety) - xenconfig: Resolve Coverity RESOURCE_LEAK (John Ferlan) - virnetsocket: Resolve Coverity RESOURCE_LEAK (John Ferlan) - commandtest: Resolve Coverity RESOURCE_LEAK (John Ferlan) - numatune: setting --mode does not work well (Erik Skultety) - qemu: min_guarantee: Parameter 'min_guarantee' not supported (Erik Skultety) - Parallels: fix error with video card RAM dimension (Alexander Burluka) - util: compare floor attribute in virNetDevBandwidthEqual (Martin Kletzander) - conf: net: Correctly switch how to format address fields (Peter Krempa) - Perform disk config validity checking for attach-device config (John Ferlan) - nodeCapsInitNUMA: Avoid @cpumap leak (Michal Privoznik) - qemu: Issue rtc-reset-reinjection command after guest-set-time (Michal Privoznik) - qemu: forbid negative blkio values (Martin Kletzander) - lxc: forbid negative blkio values (Martin Kletzander) - qemu: Fix build error introduced in 653137eb957a278b556c6226424aad5395a (Peter Krempa) - qemu: blkiotune: Avoid accessing non-existing disk configuration (Peter Krempa) - qemu: Label all TAP FDs (Michal Privoznik) - cmdMigrate: move vshConnect before vshWatchJob (Chunyan Liu) - qemu: cpu: unplug: Remove vcpu pinning on cold cpu unplug (Peter Krempa) - qemu: Redundant listen address entry in quest xml (Erik Skultety) - daemon: Fix driver registration ordering (Michal Privoznik) - network: fix crash when starting a network with no <pf> element (Laine Stump) - qemu_process: fix memleak found by coverity (Pavel Hrdina) - Maximum vlanid should be 4095 in interface.rng (Jianwei Hu) - LXC: Fix virLXCControllerSetupDevPTS() wrt user namespaces (Richard Weinberger) - conf: fix parsing 'cmd_per_lun' and 'max_sectors' (Mo yuxiang) - bhyve: fix error message in bhyveStateInitialize (Dmitry Guryanov) - qemu: migration: Check domain live state after exitting the monitor (Peter Krempa) - qemu: managedsave: Check that VM is alive after entering async job (Peter Krempa) - numatune: Fix parsing of empty nodeset (0,^0) (Erik Skultety) - conf: Format interface's driver more frequently (Michal Privoznik) - network: populate interface pool immediately when network is started (Laine Stump) - network: make networkCreateInterfacePool more robust (Laine Stump) - Make 'uri' command a bit more prominent. (Guido Günther) - Don't fail qemu driver intialization if we can't determine hugepage size (Guido Günther) - blockjob: fix use-after-free in blockcopy (Eric Blake) - blockjob: avoid memory leak during block pivot (Eric Blake) - snapshot: conf: Enforce absolute paths on disk and memory images (Peter Krempa) - Node Devices: Fix nodedev-list for fc_host & vports. (Prerna Saxena) - blockjob: correctly report active commit for job info (Eric Blake) - Fix vlanid attribute name in nwfilter d (Jianwei Hu) - Don't overwrite errors from virNetDevBandwidthSet (Ján Tomko) - virsh: check if domiftune parameters fit into UINT (Ján Tomko) - storage: Refresh storage pool after upload (John Ferlan) Improvements: - qemu: Allow use of iothreads for disk definitions (John Ferlan) - domain_conf: Add support for iothreads in disk definition (John Ferlan) - qemu: Add support for iothreads (John Ferlan) - domain_conf: Introduce iothreads XML (John Ferlan) - qemu: Implement bulk stats API and one of the stats groups to return (Peter Krempa) - virsh: Implement command to excercise the bulk stats APIs (Peter Krempa) - remote: Implement bulk domain stats APIs in the remote driver (Peter Krempa) - lib: Add few flags for the bulk stats APIs (Peter Krempa) - conf: Add helper to free domain list (Peter Krempa) - vbox: Register per partes (Michal Privoznik) - virDriverLoadModule: Honor libvirt func name tranlsation (Michal Privoznik) - virdrivermoduletest: Test all the modules (Michal Privoznik) - domain_conf: fix internal flag verification (Eric Blake) - API: Tweak virDomainOpenGraphics to return fd directly (Eric Blake) - virsh: domain: Split out code to lookup domain from string (Peter Krempa) - Wire up virDomainOpenGraphicsFD in QEMU driver (Ján Tomko) - Add RPC implementation for virDomainOpenGraphicsFd (Ján Tomko) - blkdeviotune: check for overflow when parsing XML (Erik Skultety) - storage: remove unused 'canonPath' in virStorageFileGetMetadata (Chen Fan) - Add new 'kvm' domain feature and ability to hide KVM signature (Alex Williamson) - maint: drop spurious semicolons (Eric Blake) - daemon: Fix option -v missing info priority log (Zhou Yimin) - qemu: add support for splash-timeout (Martin Kletzander) - qemu: add capability probing for splash-timeout (Martin Kletzander) - storage: zfs: implement download and upload (Roman Bogorodskiy) - fdstream: introduce virFDStreamOpenBlockDevice (Roman Bogorodskiy) - fdstream: report error if virSetNonBlock fails (Roman Bogorodskiy) - qemu: check for active domain after agent interaction (Eric Blake) - Parallels: Change config report errors code. (Alexander Burluka) - Parallels: add virNodeGetCPUMap(). (Alexander Burluka) - daemon: use socket activation with systemd (Martin Kletzander) - rpc: pass listen FD to the daemon being started (Martin Kletzander) - util: add virCommandPassListenFDs() function (Martin Kletzander) - tests: support dynamic prefixes in commandtest (Martin Kletzander) - cfg.mk: allow integers to be assigned a value computed with i|j|k (Martin Kletzander) - daemon: support passing FDs from the calling process (Martin Kletzander) - rpc: set listen backlog on FDs as well as on other sockets (Martin Kletzander) - remote: create virNetServerServiceNewFDOrUNIX() wrapper (Martin Kletzander) - util: abstract parsing of passed FDs into virGetListenFDs() (Martin Kletzander) - virsh: Don't print extra '-'s in error message for -k and -K options (Peter Krempa) - hvsupport: Adapt to vbox driver rewrite (Michal Privoznik) - qemu: hotplug: Sanitize shared device removal on media change (Peter Krempa) - qemu: conf: Split out code to retrieve hostdev key and reuse it (Peter Krempa) - qemu: conf: Split up qemuRemoveSharedDevice into per-device-type functions (Peter Krempa) - qemu: conf: Split up qemuAddSharedDevice into per-device-type functions (Peter Krempa) - qemu: conf: rename qemuCheckSharedDevice to qemuCheckSharedDisk (Peter Krempa) - qemu: shared: Split out shared device list remove code (Peter Krempa) - qemu: shared: Split out insertion code to the shared device list (Peter Krempa) - qemu: hotplug: Format proper source string for cdrom media change (Peter Krempa) - qemu: hotplug: Change arguments for qemuDomainChangeEjectableMedia (Peter Krempa) - qemu: hotplug: Add helper to initialize/teardown new disks for VMs (Peter Krempa) - qemu: hotplug: Untangle cleanup paths in qemuDomainChangeEjectableMedia (Peter Krempa) - conf: Pass virStorageSource into virDomainDiskSourceIsBlockType (Peter Krempa) - qemu: Explicitly state that hotplugging cdroms and floppies doesn't work (Peter Krempa) - conf: fix comment (Giuseppe Scrivano) - conf, virDomainFSDefPtr: rename "path" argument to "target" (Giuseppe Scrivano) - bhyve: add volumes support (Roman Bogorodskiy) - storage: make disk source pool translation generic (Roman Bogorodskiy) - driver: Move virDrvNetworkGetDHCPLeases to the appropriate section (Peter Krempa) - qemu: allow device block I/O tuning in session mode (Martin Kletzander) - src/xenconfig: move common parsing/formatting to xen_common (Jim Fehlig) - src/xenconfig: wrap common formatting code (Kiarie Kahurani) - src/xenconfig: wrap common parsing code (Kiarie Kahurani) - xen: rename xenxs to xenconfig (Jim Fehlig) - qemu: process: Pin on per-vcpu basis instead of per-vcpupin element (Peter Krempa) - conf: Refactor virDomainVcpuPinDefParseXML (Peter Krempa) - conf: cpupin: Remove useless checking of vcpupin element count (Peter Krempa) - qemu: process: Remove unnecessary argument and rename function (Peter Krempa) - vbox: Introducing vboxCommonDriver (Taowei) - vbox: Add registerDomainEvent (Taowei) - vbox: Rewrite vboxNode functions (Taowei) - vbox: Rewrite vboxConnectListAllDomains (Taowei) - vbox: Rewrite vboxDomainScreenshot (Taowei) - vbox: Rewrite vboxDomainSnapshotDelete (Taowei) - vbox: Rewrite vboxDomainRevertToSnapshot (Taowei) - vbox: Rewrite vboxDomainSnapshotHasMetadata (Taowei) - vbox: Rewrite vboxDomainSnapshotIsCurrent (Taowei) - vbox: Rewrite vboxDomainSnapshotCurrent (Taowei) - vbox: Rewrite vboxDomainSnapshotGetParent (Taowei) - vbox: Rewrite vboxDomainHasCurrentSnapshot (Taowei) - vbox: Rewrite vboxSnapshotLookupByName (Taowei) - vbox: Rewrite vboxDomainSnapshotListNames (Taowei) - vbox: Rewrite vboxDomainSnapshotNum (Taowei) - vbox: Rewrite vboxDomainSnapshotGetXMLDesc (Taowei) - vbox: Rewrite vboxDomainSnapshotCreateXML (Taowei) - vbox: Add API for vboxDomainSnapshotCreateXML (Taowei) - vbox: Rewrite vboxDomainDetachDeviceFlags (Taowei) - vbox: Rewrite vboxDomainDetachDevice (Taowei) - vbox: Rewrite vboxDomainUpdateDeviceFlags (Taowei) - vbox: Rewrite vboxDomainAttachDeviceFlags (Taowei) - vbox: Rewrite vboxDomainAttachDevice (Taowei) - vbox: Rewrite vboxDomainUndefine (Taowei) - vbox: Rewrite vboxConnectNumOfDefinedDomains (Taowei) - vbox: Rewrite vboxConnectListDefinedDomains (Taowei) - vbox: Rewrite vboxDomainGetXMLDesc (Taowei) - vbox: Add API for vboxDomainGetXMLDesc (Taowei) - vbox: Rewrite vboxDomainGetMaxVcpus (Taowei) - vbox: Rewrite vboxDomainGetVcpusFlags (Taowei) - vbox: Rewrite vboxDomainSetVcpus (Taowei) - vbox: Rewrite vboxDomainSetVcpusFlags (Taowei) - vbox: Rewrite vboxDomainGetState (Taowei) - vbox: Rewrite vboxDomainGetInfo (Taowei) - vbox: Rewrite vboxDomainSetMemory (Taowei) - vbox: Rewrite vboxDomainGetOSType (Taowei) - vbox: Rewrite vboxDomainDestroy (Taowei) - vbox: Rewrite vboxDomainDestroyFlags (Taowei) - vbox: Rewrite vboxDomainReboot (Taowei) - vbox: Rewrite vboxDomainShutdown (Taowei) - vbox: Rewrite vboxDomainShutdownFlags (Taowei) - vbox: Rewrite vboxDomainResume (Taowei) - vbox: Rewrite vboxDomainSuspend (Taowei) - vbox: Rewrite vboxDomainIsUpdated (Taowei) - vbox: Rewrite vboxDomainIsPersistent (Taowei) - vbox: Rewrite vboxDomainIsActive (Taowei) - vbox: Rewrite vboxDomainLookupByName (Taowei) - vbox: Rewrite vboxDomainCreateXML (Taowei) - vbox: Rewrite vboxDomainCreate (Taowei) - vbox: Rewrite vboxDomainCreateWithFlags (Taowei) - vbox: Rewrite vboxDomainDefineXML (Taowei) - vbox: Rewrite vboxDomainUndefineFlags (Taowei) - vbox: Rewrite vboxDomainLookupByUUID (Taowei) - vbox: Rewrite vboxDomainLookupById (Taowei) - vbox: Rewrite vboxConnectNumOfDomains (Taowei) - vbox: Rewrite vboxConnectListDomains (Taowei) - vbox: Rewrite vboxConnectGetCapabilities (Taowei) - vbox: Rewrite vboxConnectGetMaxVcpus (Taowei) - vbox: Rewrite vboxConnectIsAlive (Taowei) - vbox: Rewrite vboxConnectIsEncrypted (Taowei) - vbox: Rewrite vboxConnectIsSecure (Taowei) - vbox: Rewrite vboxConnectGetHostname (Taowei) - vbox: Rewrite vboxConnectGetVersion (Taowei) - vbox: Rewrite vboxDomainSave (Taowei) - vbox: Rewrite vboxConnectClose (Taowei) - vbox: Begin to rewrite, vboxConnectOpen (Taowei) - src/xenxs: Refactor code formating vif device config (Jim Fehlig) - src/xenxs: Refactor code formating peripheral device config (Kiarie Kahurani) - src/xenxs: Refactor code formating Vfb config (Kiarie Kahurani) - src/xenxs: Refactor code formating OS config (Kiarie Kahurani) - src/xenxs: Refactor code formating CPU config (Kiarie Kahurani) - src/xenxs: Refactor code formating xm disk config (Kiarie Kahurani) - src/xenxs: Refactor code formating Char devices config (Kiarie Kahurani) - src/xenxs: Refactor code formating event actions config (Kiarie Kahurani) - src/xenxs: Refactor code formating virtual time config (Kiarie Kahurani) - src/xenxs: Refactor code formating memory config (Kiarie Kahurani) - src/xenxs: Refactor code formating general VM config (Kiarie Kahurani) - qemu: Tidy up job handling during live migration (Sam Bobroff) - LXC: resolve issues in lxcDomainSetMaxMemory (Chen Hanxiao) - daemon: Limit default log level to journald to VIR_LOG_INFO (Peter Krempa) - libvirtd: conf: Mention support for logging into journald (Peter Krempa) - maint: improve syntax check for space around = (Eric Blake) - qemu_conf: Undefine the correct symbol (Michal Privoznik) - qemu: process: Fix header format of qemuProcessSetVcpuAffinities (Peter Krempa) - qemu: use guest-fsfreeze-freeze-list command if mountpoints to freeze specified (Tomoki Sekiyama) - qemu: Actually clear bandwidth settings (Jianwei Hu) - src/xenxs: Refactor code parsing OS config (Kiarie Kahurani) - src/xenxs: Refactor code parsing general config (Kiarie Kahurani) - src/xenxs: Refactor code parsing emulated hardware config (Kiarie Kahurani) - src/xenxs: Refactor code parsing Vif config (Kiarie Kahurani) - src/xenxs: Refactor code parsing Char devices config (Kiarie Kahurani) - src/xenxs: Refactor code parsing Vfb config (Kiarie Kahurani) - src/xenxs: Refactor code parsing xm disk config (Kiarie Kahurani) - src/xenxs: Refactor code parsing CPU features (Kiarie Kahurani) - src/xenxs: Refactor code parsing PCI config (Kiarie Kahurani) - src/xenxs: Refactor code parsing event actions (Kiarie Kahurani) - src/xenxs: Refactor code parsing virtual time config (Kiarie Kahurani) - src/xenxs: Refactor code parsing memory config (Kiarie Kahurani) - conf: Add USB sound card support and implement it for qemu (Peter Krempa) - hostdev: Add iSCSI hostdev XML (John Ferlan) - domain_conf: Common routine to handle network storage host xml def (John Ferlan) - qemu: Make virFileFindHugeTLBFS fault tolerant (Michal Privoznik) - qemu: reword caps-related error (Martin Kletzander) - domtop: Turn parse_argv into void (Michal Privoznik) - qemu: fix comment in qemu.conf (Martin Kletzander) Cleanups: - virsh: drop unused variable (Eric Blake) - Fix spacing around commas (Michal Privoznik) - examples: test: Kill unsupported maxMemory element (Peter Krempa) - cleanup spaces between parentheses and braces (Martin Kletzander) - qemu_command: fix block indentation (Giuseppe Scrivano) - qemu: Remove extraneous space in function prototypes (John Ferlan) - daemon: Fix indentation in libvirtd.c (Wang Rui) - domtop: Remove unused variable (Michal Privoznik) thanks everybody for your contribution to this new release, be it with reports, ideas, patches, reviews or help on the localization ! 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] [python PATCH v2 0/5] Implement new APIs
by Pavel Hrdina 02 Sep '14

02 Sep '14

new in v2: - moved function to appropriate place in libvirt-override.c - fixed generator to resolve enum reference - fixed memory leak in virDomainListGetStats and sanyti test - implemented API for virDomainBlockCopy Pavel Hrdina (3): generator: resolve one level of enum reference API: Implement bindings for virDomainListGetStats Implement API bindings for virDomainBlockCopy Peter Krempa (2): API: Skip 'virDomainStatsRecordListFree' API: Implement bindings for virConnectGetAllDomainStats generator.py | 20 ++++- libvirt-override-api.xml | 10 +++ libvirt-override-virConnect.py | 100 +++++++++++++++++++++++ libvirt-override.c | 180 +++++++++++++++++++++++++++++++++++++++++ sanitytest.py | 6 ++ 5 files changed, 315 insertions(+), 1 deletion(-) -- 1.8.5.5

2 13

[libvirt] [python PATCH 0/5] Implement bindings for bulk stats API
by Peter Krempa 01 Sep '14

01 Sep '14

The last patch is to ease review to be able to build the series. Peter Krempa (5): generator: enum: Don't sort enums by names API: Skip 'virDomainStatsRecordListFree' API: Implement bindings for virConnectGetAllDomainStats API: Implement bindings for virDomainListGetStats DO NOT APPLY: Fix build with missing virDomainBlockCopy API generator.py | 6 +- libvirt-override-virConnect.py | 100 ++++++++++++++++++++++++++++ libvirt-override.c | 144 +++++++++++++++++++++++++++++++++++++++++ 3 files changed, 248 insertions(+), 2 deletions(-) -- 2.0.2

3 11

[libvirt] [PATCH for 1.2.8] selinux: properly label tap FDs with imagelabel
by Martin Kletzander 01 Sep '14

01 Sep '14

The cleanup in commit cf976d9d used secdef->label to label the tap FDs, but that is not possible since it's process-only label (svirt_t) and not a object label (e.g. svirt_image_t). Starting a domain failed with EPERM, but simply using secdef->label instead fixes it. Signed-off-by: Martin Kletzander <mkletzan(a)redhat.com> --- src/security/security_selinux.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/security/security_selinux.c b/src/security/security_selinux.c index 5d18493..e8c13db 100644 --- a/src/security/security_selinux.c +++ b/src/security/security_selinux.c @@ -2340,7 +2340,7 @@ virSecuritySELinuxSetTapFDLabel(virSecurityManagerPtr mgr ATTRIBUTE_UNUSED, if (!secdef || !secdef->label) return 0; - return virSecuritySELinuxFSetFilecon(fd, secdef->label); + return virSecuritySELinuxFSetFilecon(fd, secdef->imagelabel); } static char * -- 2.1.0

2 1

[libvirt] [PATCH v4 0/3] OVMF exposure
by Michal Privoznik 01 Sep '14

01 Sep '14

diff to v3: -Pure rebae & resend diff to v2: -Adapted to Laszlo's review on v2 Michal Privoznik (3): conf: Extend <loader/> and introduce <nvram/> qemu: Implement extended loader and nvram qemu: Automatically create NVRAM store docs/formatdomain.html.in | 19 ++- docs/schemas/domaincommon.rng | 21 ++++ libvirt.spec.in | 2 + src/Makefile.am | 1 + src/conf/domain_conf.c | 87 +++++++++++++- src/conf/domain_conf.h | 22 +++- src/libvirt_private.syms | 3 + src/qemu/libvirtd_qemu.aug | 3 + src/qemu/qemu.conf | 14 +++ src/qemu/qemu_command.c | 97 ++++++++++++++- src/qemu/qemu_conf.c | 93 +++++++++++++++ src/qemu/qemu_conf.h | 5 + src/qemu/qemu_process.c | 132 +++++++++++++++++++++ src/qemu/test_libvirtd_qemu.aug.in | 3 + src/security/security_dac.c | 8 ++ src/security/security_selinux.c | 8 ++ src/security/virt-aa-helper.c | 4 +- src/vbox/vbox_common.c | 7 +- src/xenapi/xenapi_driver.c | 3 +- src/xenconfig/xen_common.c | 7 +- src/xenconfig/xen_sxpr.c | 16 +-- .../qemuxml2argvdata/qemuxml2argv-bios-nvram.args | 10 ++ tests/qemuxml2argvdata/qemuxml2argv-bios-nvram.xml | 40 +++++++ tests/qemuxml2argvtest.c | 2 + .../qemuxml2xmlout-pci-bridge-many-disks.xml | 2 +- tests/qemuxml2xmltest.c | 2 + tests/sexpr2xmldata/sexpr2xml-fv-autoport.xml | 2 +- tests/sexpr2xmldata/sexpr2xml-fv-empty-kernel.xml | 2 +- tests/sexpr2xmldata/sexpr2xml-fv-force-hpet.xml | 2 +- tests/sexpr2xmldata/sexpr2xml-fv-force-nohpet.xml | 2 +- tests/sexpr2xmldata/sexpr2xml-fv-kernel.xml | 2 +- tests/sexpr2xmldata/sexpr2xml-fv-legacy-vfb.xml | 2 +- tests/sexpr2xmldata/sexpr2xml-fv-localtime.xml | 2 +- tests/sexpr2xmldata/sexpr2xml-fv-net-ioemu.xml | 2 +- tests/sexpr2xmldata/sexpr2xml-fv-net-netfront.xml | 2 +- tests/sexpr2xmldata/sexpr2xml-fv-parallel-tcp.xml | 2 +- .../sexpr2xml-fv-serial-dev-2-ports.xml | 2 +- .../sexpr2xml-fv-serial-dev-2nd-port.xml | 2 +- tests/sexpr2xmldata/sexpr2xml-fv-serial-file.xml | 2 +- tests/sexpr2xmldata/sexpr2xml-fv-serial-null.xml | 2 +- tests/sexpr2xmldata/sexpr2xml-fv-serial-pipe.xml | 2 +- tests/sexpr2xmldata/sexpr2xml-fv-serial-pty.xml | 2 +- tests/sexpr2xmldata/sexpr2xml-fv-serial-stdio.xml | 2 +- .../sexpr2xml-fv-serial-tcp-telnet.xml | 2 +- tests/sexpr2xmldata/sexpr2xml-fv-serial-tcp.xml | 2 +- tests/sexpr2xmldata/sexpr2xml-fv-serial-udp.xml | 2 +- tests/sexpr2xmldata/sexpr2xml-fv-serial-unix.xml | 2 +- tests/sexpr2xmldata/sexpr2xml-fv-sound-all.xml | 2 +- tests/sexpr2xmldata/sexpr2xml-fv-sound.xml | 2 +- tests/sexpr2xmldata/sexpr2xml-fv-usbmouse.xml | 2 +- tests/sexpr2xmldata/sexpr2xml-fv-usbtablet.xml | 2 +- tests/sexpr2xmldata/sexpr2xml-fv-utc.xml | 2 +- tests/sexpr2xmldata/sexpr2xml-fv-v2.xml | 2 +- tests/sexpr2xmldata/sexpr2xml-fv.xml | 2 +- tests/sexpr2xmldata/sexpr2xml-no-source-cdrom.xml | 2 +- tests/xmconfigdata/test-escape-paths.xml | 2 +- tests/xmconfigdata/test-fullvirt-force-hpet.xml | 2 +- tests/xmconfigdata/test-fullvirt-force-nohpet.xml | 2 +- tests/xmconfigdata/test-fullvirt-localtime.xml | 2 +- tests/xmconfigdata/test-fullvirt-net-ioemu.xml | 2 +- tests/xmconfigdata/test-fullvirt-net-netfront.xml | 2 +- tests/xmconfigdata/test-fullvirt-new-cdrom.xml | 2 +- tests/xmconfigdata/test-fullvirt-old-cdrom.xml | 2 +- tests/xmconfigdata/test-fullvirt-parallel-tcp.xml | 2 +- .../test-fullvirt-serial-dev-2-ports.xml | 2 +- .../test-fullvirt-serial-dev-2nd-port.xml | 2 +- tests/xmconfigdata/test-fullvirt-serial-file.xml | 2 +- tests/xmconfigdata/test-fullvirt-serial-null.xml | 2 +- tests/xmconfigdata/test-fullvirt-serial-pipe.xml | 2 +- tests/xmconfigdata/test-fullvirt-serial-pty.xml | 2 +- tests/xmconfigdata/test-fullvirt-serial-stdio.xml | 2 +- .../test-fullvirt-serial-tcp-telnet.xml | 2 +- tests/xmconfigdata/test-fullvirt-serial-tcp.xml | 2 +- tests/xmconfigdata/test-fullvirt-serial-udp.xml | 2 +- tests/xmconfigdata/test-fullvirt-serial-unix.xml | 2 +- tests/xmconfigdata/test-fullvirt-sound.xml | 2 +- tests/xmconfigdata/test-fullvirt-usbmouse.xml | 2 +- tests/xmconfigdata/test-fullvirt-usbtablet.xml | 2 +- tests/xmconfigdata/test-fullvirt-utc.xml | 2 +- tests/xmconfigdata/test-no-source-cdrom.xml | 2 +- tests/xmconfigdata/test-pci-devs.xml | 2 +- 81 files changed, 639 insertions(+), 82 deletions(-) create mode 100644 tests/qemuxml2argvdata/qemuxml2argv-bios-nvram.args create mode 100644 tests/qemuxml2argvdata/qemuxml2argv-bios-nvram.xml -- 1.8.5.5

4 21

[libvirt] (no subject)
by Raghavendra K T 01 Sep '14

01 Sep '14

Bcc: Raghavendra K T <raghavendra.kt(a)linux.vnet.ibm.com> Subject: Re: [libvirt] [RFC] exclusive vcpu-cpu pinning Reply-To: Raghavendra K T <raghavendra.kt(a)linux.vnet.ibm.com> In-Reply-To: <53DA24CF.8030600(a)redhat.com> * J?n Tomko <jtomko(a)redhat.com> [2014-07-31 13:13:19]: > Hello developers! > > Currently, our default cgroup layout is: > -top level cgroup > \-machine (machine.slice with systemd) > `-vm1.libvirt-qemu (machine-qemu\x2dvm1.scope with systemd) > `-emulator > `-vcpu0 > \-vcpu1 > \-vm2.libvirt-qemu > `-emulator > `-vcpu0 > `-vcpu1 > > To free some CPUs for exclusive use, either all processes from the top level > cgroup should be moved to another one (which does not seem like a great idea) > or isolcpus= should be specified on the kernel command line. > > The cpuset.cpu_exclusive option can be set on a cgroup if > * all the groups up to the top level group have it set > * the cpuset of the current group is a subset of the parent group > and no siblings use any cpus from the current cpuset > > This would mean that to keep the existing nested structure, all vcpus and the > emulator thread would need to have an exclusive CPU, e.g: > <vcpu placement='static' cpuset='4-6'>2</vcpu> > <cputune exclusive='yes'> > <vcpupin vcpu='0' cpuset='5'/> > <vcpupin vcpu='1' cpuset='6'/> > <emulatorpin cpuset='4'/> > </cputune> > > (The only two issues I found: > 1) libvirt would have to mess with systemd's 'machine-scope' behind it's back > (setting cpu_exclusive) > 2) creating machines without explicit cpu pinning fails, as libvirt tries to > write all the cpus to the cpuset, even those the other machine uses > exclusively) > > I've also thought about just keeping track of the 'exclusived' CPUs in > libvirt. This would not work across drivers. And it could possibly be needed > to solve issue 2). > > Do you think any of these options would be useful? > > Bug: https://bugzilla.redhat.com/show_bug.cgi?id=996758 > > Jan > Hi Jan, I am not familiar with libvirt internals, but eager to solve the problem (I also tried to solve the same problem, I had POC kernel solution which was rightly rejected because we could solve with userspace). Could we have a dedicated cpuset for vms (which asks for dedicated cpuset may be via xml tag? <description>dedicated</description>) [ This is very similar to what you have proposed ] suppose we have 2 vms of 8 vcpus (vm1 dedicated, vm2 non-dedicated) on a 16 pcpu machine, the modified cpuset cgroup hierarchy looks like this (for cpuset only): | root (cpuset.cpus = 0-15) | \_ machine (tasks = system tasks) (cpuset.cpus = 0-7, exclusive=1) \_ vm2.libvirt-qemu (cpuset.cpus = 0-7, exclusive=1) | \_ vm2.libvirt-qemu (cpuset.cpus = 8-15, exclusive=1) But as you have mentioned above libvirt will have to 1. modify the cpuset hierarchy behind systemd 2. move all the system tasks to machine (only for cpuset) 3. assign all the non dedicated cpuset to /machine hierarchy 4. assign dedicated/exclusive cpus to vms automatically. ofcourse we cannot have 100% of cpus to be dedicated and we will have to ensure that we do have some cpus left for system tasks/non dedicated vms etc. I see we could achieve above requirement with a userspace daemon, But I think libvirt way of solving would be ideal. Do you think the above solution is too intrusive? Please let us know your thoughts.

1 0

[libvirt] [PATCH 00/11] bulk stats: QEMU implementation
by Francesco Romani 01 Sep '14

01 Sep '14

This patchset enhances the QEMU support for the new bulk stats API. What is added is the equivalent of these APIs: virDomainBlockInfo virDomainGetInfo - for balloon stats virDomainGetCPUStats virDomainBlockStatsFlags virDomainInterfaceStats virDomainGetVcpusFlags virDomainGetVcpus This subset of API is the one oVirt relies on. The patchset is organized as follows: - the first 4 patches do refactoring to extract internal helper functions to be used by the old API and by the new bulk one. For block stats on helper is actually added instead of extracted. - since some groups require access to the QEMU monitor, one patch extend the internal interface to easily accomodate that - finally, the last six patches implement the support for the bulk API. Francesco Romani (11): qemu: extract helper to get the current balloon qemu: extract helper to gather vcpu data qemu: add helper to get the block stats qemu: extract helper to get block info qemu: bulk stats: pass connection to workers qemu: bulk stats: implement CPU stats group qemu: bulk stats: implement balloon group qemu: bulk stats: implement VCPU group qemu: bulk stats: implement interface group qemu: bulk stats: implement block group qemu: bulk stats: implement blockinfo group include/libvirt/libvirt.h.in | 6 + src/qemu/qemu_driver.c | 558 ++++++++++++++++++++++++++++++++++++++----- 2 files changed, 502 insertions(+), 62 deletions(-) -- 1.9.3

2 13

[libvirt] [PATCH] Fix connection to already running session libvirtd
by Christophe Fergeau 01 Sep '14

01 Sep '14

Since 1b807f92, connecting with virsh to an already running session libvirtd fails with: $ virsh list --all error: failed to connect to the hypervisor error: no valid connection error: Failed to connect socket to '/run/user/1000/libvirt/libvirt-sock': Transport endpoint is already connected This is caused by a logic error in virNetSocketNewConnectUnix: even if the connection to the daemon socket succeeded, we still try to spawn the daemon and then connect to it. This commit changes the logic to not try to spawn libvirtd if we successfully connected to its socket. With whitespace changes removed, this patch becomes just this: diff --git a/src/rpc/virnetsocket.c b/src/rpc/virnetsocket.c index f913365..79540b3 100644 --- a/src/rpc/virnetsocket.c +++ b/src/rpc/virnetsocket.c @@ -574,7 +574,8 @@ int virNetSocketNewConnectUNIX(const char *path, remoteAddr.data.un.sun_path[0] = '\0'; retry: - if (connect(fd, &remoteAddr.data.sa, remoteAddr.len) < 0 && !spawnDaemon) { + if (connect(fd, &remoteAddr.data.sa, remoteAddr.len) < 0) { + if (!spawnDaemon) { virReportSystemError(errno, _("Failed to connect socket to '%s'"), path); goto error; @@ -634,6 +635,7 @@ int virNetSocketNewConnectUNIX(const char *path, if (virNetSocketForkDaemon(binary, passfd) < 0) goto error; } + } localAddr.len = sizeof(localAddr.data); if (getsockname(fd, &localAddr.data.sa, &localAddr.len) < 0) { --- src/rpc/virnetsocket.c | 102 +++++++++++++++++++++++++------------------------ 1 file changed, 52 insertions(+), 50 deletions(-) diff --git a/src/rpc/virnetsocket.c b/src/rpc/virnetsocket.c index 79258ef..8fc5d80 100644 --- a/src/rpc/virnetsocket.c +++ b/src/rpc/virnetsocket.c @@ -573,65 +573,67 @@ int virNetSocketNewConnectUNIX(const char *path, remoteAddr.data.un.sun_path[0] = '\0'; retry: - if (connect(fd, &remoteAddr.data.sa, remoteAddr.len) < 0 && !spawnDaemon) { - virReportSystemError(errno, _("Failed to connect socket to '%s'"), - path); - goto error; - } else if (spawnDaemon) { - int status = 0; - pid_t pid = 0; - - if ((passfd = socket(PF_UNIX, SOCK_STREAM, 0)) < 0) { - virReportSystemError(errno, "%s", _("Failed to create socket")); + if (connect(fd, &remoteAddr.data.sa, remoteAddr.len) < 0) { + if (!spawnDaemon) { + virReportSystemError(errno, _("Failed to connect socket to '%s'"), + path); goto error; - } + } else if (spawnDaemon) { + int status = 0; + pid_t pid = 0; - /* - * We have to fork() here, because umask() is set - * per-process, chmod() is racy and fchmod() has undefined - * behaviour on sockets according to POSIX, so it doesn't - * work outside Linux. - */ - if ((pid = virFork()) < 0) - goto error; + if ((passfd = socket(PF_UNIX, SOCK_STREAM, 0)) < 0) { + virReportSystemError(errno, "%s", _("Failed to create socket")); + goto error; + } - if (pid == 0) { - umask(0077); - if (bind(passfd, &remoteAddr.data.sa, remoteAddr.len) < 0) - _exit(EXIT_FAILURE); + /* + * We have to fork() here, because umask() is set + * per-process, chmod() is racy and fchmod() has undefined + * behaviour on sockets according to POSIX, so it doesn't + * work outside Linux. + */ + if ((pid = virFork()) < 0) + goto error; - _exit(EXIT_SUCCESS); - } + if (pid == 0) { + umask(0077); + if (bind(passfd, &remoteAddr.data.sa, remoteAddr.len) < 0) + _exit(EXIT_FAILURE); - if (virProcessWait(pid, &status, false) < 0) - goto error; + _exit(EXIT_SUCCESS); + } - if (status != EXIT_SUCCESS) { - /* - * OK, so the subprocces failed to bind() the socket. This may mean - * that another daemon was starting at the same time and succeeded - * with its bind(). So we'll try connecting again, but this time - * without spawning the daemon. - */ - spawnDaemon = false; - goto retry; - } + if (virProcessWait(pid, &status, false) < 0) + goto error; - if (listen(passfd, 0) < 0) { - virReportSystemError(errno, "%s", - _("Failed to listen on socket that's about " - "to be passed to the daemon")); - goto error; - } + if (status != EXIT_SUCCESS) { + /* + * OK, so the subprocces failed to bind() the socket. This may mean + * that another daemon was starting at the same time and succeeded + * with its bind(). So we'll try connecting again, but this time + * without spawning the daemon. + */ + spawnDaemon = false; + goto retry; + } - if (connect(fd, &remoteAddr.data.sa, remoteAddr.len) < 0) { - virReportSystemError(errno, _("Failed to connect socket to '%s'"), - path); - goto error; - } + if (listen(passfd, 0) < 0) { + virReportSystemError(errno, "%s", + _("Failed to listen on socket that's about " + "to be passed to the daemon")); + goto error; + } - if (virNetSocketForkDaemon(binary, passfd) < 0) - goto error; + if (connect(fd, &remoteAddr.data.sa, remoteAddr.len) < 0) { + virReportSystemError(errno, _("Failed to connect socket to '%s'"), + path); + goto error; + } + + if (virNetSocketForkDaemon(binary, passfd) < 0) + goto error; + } } localAddr.len = sizeof(localAddr.data); -- 1.9.3

3 4

[libvirt] [PATCH v2] Fix connection to already running session libvirtd
by Christophe Fergeau 01 Sep '14

01 Sep '14

Since 1b807f92, connecting with virsh to an already running session libvirtd fails with: $ virsh list --all error: failed to connect to the hypervisor error: no valid connection error: Failed to connect socket to '/run/user/1000/libvirt/libvirt-sock': Transport endpoint is already connected This is caused by a logic error in virNetSocketNewConnectUnix: even if the connection to the daemon socket succeeded, we still try to spawn the daemon and then connect to it. This commit changes the logic to not try to spawn libvirtd if we successfully connected to its socket. Most of this commit is whitespace changes, use of -w is used to look at it. --- Changes since v1: - Removed now redundant test in the else branch src/rpc/virnetsocket.c | 102 +++++++++++++++++++++++++------------------------ 1 file changed, 52 insertions(+), 50 deletions(-) diff --git a/src/rpc/virnetsocket.c b/src/rpc/virnetsocket.c index 79258ef..9780e17 100644 --- a/src/rpc/virnetsocket.c +++ b/src/rpc/virnetsocket.c @@ -573,65 +573,67 @@ int virNetSocketNewConnectUNIX(const char *path, remoteAddr.data.un.sun_path[0] = '\0'; retry: - if (connect(fd, &remoteAddr.data.sa, remoteAddr.len) < 0 && !spawnDaemon) { - virReportSystemError(errno, _("Failed to connect socket to '%s'"), - path); - goto error; - } else if (spawnDaemon) { - int status = 0; - pid_t pid = 0; - - if ((passfd = socket(PF_UNIX, SOCK_STREAM, 0)) < 0) { - virReportSystemError(errno, "%s", _("Failed to create socket")); + if (connect(fd, &remoteAddr.data.sa, remoteAddr.len) < 0) { + if (!spawnDaemon) { + virReportSystemError(errno, _("Failed to connect socket to '%s'"), + path); goto error; - } + } else { + int status = 0; + pid_t pid = 0; - /* - * We have to fork() here, because umask() is set - * per-process, chmod() is racy and fchmod() has undefined - * behaviour on sockets according to POSIX, so it doesn't - * work outside Linux. - */ - if ((pid = virFork()) < 0) - goto error; + if ((passfd = socket(PF_UNIX, SOCK_STREAM, 0)) < 0) { + virReportSystemError(errno, "%s", _("Failed to create socket")); + goto error; + } - if (pid == 0) { - umask(0077); - if (bind(passfd, &remoteAddr.data.sa, remoteAddr.len) < 0) - _exit(EXIT_FAILURE); + /* + * We have to fork() here, because umask() is set + * per-process, chmod() is racy and fchmod() has undefined + * behaviour on sockets according to POSIX, so it doesn't + * work outside Linux. + */ + if ((pid = virFork()) < 0) + goto error; - _exit(EXIT_SUCCESS); - } + if (pid == 0) { + umask(0077); + if (bind(passfd, &remoteAddr.data.sa, remoteAddr.len) < 0) + _exit(EXIT_FAILURE); - if (virProcessWait(pid, &status, false) < 0) - goto error; + _exit(EXIT_SUCCESS); + } - if (status != EXIT_SUCCESS) { - /* - * OK, so the subprocces failed to bind() the socket. This may mean - * that another daemon was starting at the same time and succeeded - * with its bind(). So we'll try connecting again, but this time - * without spawning the daemon. - */ - spawnDaemon = false; - goto retry; - } + if (virProcessWait(pid, &status, false) < 0) + goto error; - if (listen(passfd, 0) < 0) { - virReportSystemError(errno, "%s", - _("Failed to listen on socket that's about " - "to be passed to the daemon")); - goto error; - } + if (status != EXIT_SUCCESS) { + /* + * OK, so the subprocces failed to bind() the socket. This may mean + * that another daemon was starting at the same time and succeeded + * with its bind(). So we'll try connecting again, but this time + * without spawning the daemon. + */ + spawnDaemon = false; + goto retry; + } - if (connect(fd, &remoteAddr.data.sa, remoteAddr.len) < 0) { - virReportSystemError(errno, _("Failed to connect socket to '%s'"), - path); - goto error; - } + if (listen(passfd, 0) < 0) { + virReportSystemError(errno, "%s", + _("Failed to listen on socket that's about " + "to be passed to the daemon")); + goto error; + } - if (virNetSocketForkDaemon(binary, passfd) < 0) - goto error; + if (connect(fd, &remoteAddr.data.sa, remoteAddr.len) < 0) { + virReportSystemError(errno, _("Failed to connect socket to '%s'"), + path); + goto error; + } + + if (virNetSocketForkDaemon(binary, passfd) < 0) + goto error; + } } localAddr.len = sizeof(localAddr.data); -- 1.9.3

2 2

[libvirt] [PATCH] conf: Check migration_host is valid or not during libvirt restarts
by Chen Fan 01 Sep '14

01 Sep '14

if user specified an invalid strings as migration hostname, like setting: migration_host = "XXXXXXX", libvirt should check it and return error during lbivirt restart. Signed-off-by: Chen Fan <chen.fan.fnst(a)cn.fujitsu.com> --- src/qemu/qemu_conf.c | 40 ++++++++++++++++++++++++++++++++++++++++ 1 file changed, 40 insertions(+) diff --git a/src/qemu/qemu_conf.c b/src/qemu/qemu_conf.c index e2ec54f..450ac5b 100644 --- a/src/qemu/qemu_conf.c +++ b/src/qemu/qemu_conf.c @@ -33,6 +33,7 @@ #include <fcntl.h> #include <sys/wait.h> #include <arpa/inet.h> +#include <netdb.h> #include "virerror.h" #include "qemu_conf.h" @@ -650,6 +651,45 @@ int virQEMUDriverConfigLoadFile(virQEMUDriverConfigPtr cfg, GET_VALUE_LONG("seccomp_sandbox", cfg->seccompSandbox); GET_VALUE_STR("migration_host", cfg->migrateHost); + if (cfg->migrateHost) { + struct addrinfo hints; + struct addrinfo *res; + + memset(&hints, 0, sizeof(hints)); + hints.ai_flags = AI_ADDRCONFIG; + hints.ai_family = AF_UNSPEC; + + if (getaddrinfo(cfg->migrateHost, NULL, &hints, &res) != 0) { + virReportError(VIR_ERR_CONF_SYNTAX, + _("migration_host: '%s' is not a valid hostname"), + cfg->migrateHost); + goto cleanup; + } + + if (res == NULL) { + virReportError(VIR_ERR_CONF_SYNTAX, + _("No IP address for host '%s' found"), + cfg->migrateHost); + goto cleanup; + } + + freeaddrinfo(res); + + if (STRPREFIX(cfg->migrateHost, "localhost")) { + virReportError(VIR_ERR_CONF_SYNTAX, "%s", + _("setting migration_host to 'localhost' is not allowed")); + goto cleanup; + } + + if (STREQ(cfg->migrateHost, "127.0.0.1") || + STREQ(cfg->migrateHost, "::1")) { + virReportError(VIR_ERR_CONF_SYNTAX, "%s", + _("setting migration_host to '127.0.0.1' or '::1' " + "is not allowed")); + goto cleanup; + } + } + GET_VALUE_STR("migration_address", cfg->migrationAddress); GET_VALUE_BOOL("log_timestamp", cfg->logTimestamp); -- 1.9.3

3 2

[libvirt] [PATCH] qemu: Add network bandwidth settings for ethernet type interfaces
by Anirban Chakraborty 01 Sep '14

01 Sep '14

Please apply. thanks, Anirban Chakraborty Signed-off-by: Anirban Chakraborty <abchak(a)juniper.net> --- src/qemu/qemu_command.c | 5 +++++ src/qemu/qemu_hotplug.c | 3 +++ 2 files changed, 9 insertions(+), 1 deletion(-) diff --git a/src/qemu/qemu_command.c b/src/qemu/qemu_command.c index 2184caa..258c6a7 100644 --- a/src/qemu/qemu_command.c +++ b/src/qemu/qemu_command.c @@ -7251,6 +7251,11 @@ qemuBuildInterfaceCommandLine(virCommandPtr cmd, if (tapfd[0] < 0) goto cleanup; } + /* Configure network bandwidth for ethernet type network interfaces */ + if (actualType == VIR_DOMAIN_NET_TYPE_ETHERNET) + if (virNetDevBandwidthSet(net->ifname, + virDomainNetGetActualBandwidth(net), false) < 0) + goto cleanup; if ((actualType == VIR_DOMAIN_NET_TYPE_NETWORK || actualType == VIR_DOMAIN_NET_TYPE_BRIDGE || diff --git a/src/qemu/qemu_hotplug.c b/src/qemu/qemu_hotplug.c index a364c52..aeb53c5 100644 --- a/src/qemu/qemu_hotplug.c +++ b/src/qemu/qemu_hotplug.c @@ -940,6 +940,9 @@ int qemuDomainAttachNetDevice(virConnectPtr conn, if (qemuOpenVhostNet(vm->def, net, priv->qemuCaps, vhostfd, &vhostfdSize) < 0) goto cleanup; } else if (actualType == VIR_DOMAIN_NET_TYPE_ETHERNET) { + if (virNetDevBandwidthSet(net->ifname, + virDomainNetGetActualBandwidth(net), false) < 0) + goto cleanup; vhostfdSize = 1; if (VIR_ALLOC(vhostfd) < 0) goto cleanup; -- 1.8.2.3

1 0

[libvirt] Cosmetic bug on libvirt.org
by Christophe Fergeau 29 Aug '14

29 Aug '14

Hey, I just noticed that clicking on "FAQ" in the left sidebar on libvirt.org main page highlights the "Wiki" cell instead of highlighting "FAQ". I have no idea how the website works, so I'm just reporting it here. Cheers, Christophe

2 1

[libvirt] [PATCH 1.2.8] storage: zfs: fix double listing of new volumes
by Roman Bogorodskiy 29 Aug '14

29 Aug '14

Currently, after calling commands to create a new volumes, virStorageBackendZFSCreateVol calls virStorageBackendZFSFindVols that calls virStorageBackendZFSParseVol. virStorageBackendZFSParseVol checks if a volume already exists by trying to get it using virStorageVolDefFindByName. For a just created volume it returns NULL, so volume is reported as new and appended to pool->volumes. This causes a volume to be listed twice as storageVolCreateXML appends this new volume to the list as well. Fix that by passing a new volume definition to virStorageBackendZFSParseVol so it could determine if it needs to add this volume to the list. --- src/storage/storage_backend_zfs.c | 45 ++++++++++++++++++++++----------------- 1 file changed, 26 insertions(+), 19 deletions(-) diff --git a/src/storage/storage_backend_zfs.c b/src/storage/storage_backend_zfs.c index 2d74055..d8201ac 100644 --- a/src/storage/storage_backend_zfs.c +++ b/src/storage/storage_backend_zfs.c @@ -58,7 +58,8 @@ virStorageBackendZFSCheckPool(virConnectPtr conn ATTRIBUTE_UNUSED, static int virStorageBackendZFSParseVol(virStoragePoolObjPtr pool, - const char *volume) + virStorageVolDefPtr vol, + const char *volume_string) { int ret = -1; char **tokens; @@ -66,9 +67,9 @@ virStorageBackendZFSParseVol(virStoragePoolObjPtr pool, char **name_tokens = NULL; char *vol_name; bool is_new_vol = false; - virStorageVolDefPtr vol = NULL; + virStorageVolDefPtr volume = NULL; - if (!(tokens = virStringSplitCount(volume, "\t", 0, &count))) + if (!(tokens = virStringSplitCount(volume_string, "\t", 0, &count))) return -1; if (count != 2) @@ -79,36 +80,41 @@ virStorageBackendZFSParseVol(virStoragePoolObjPtr pool, vol_name = name_tokens[1]; - vol = virStorageVolDefFindByName(pool, vol_name); + if (vol == NULL) + volume = virStorageVolDefFindByName(pool, vol_name); + else + volume = vol; - if (vol == NULL) { - if (VIR_ALLOC(vol) < 0) + if (volume == NULL) { + if (VIR_ALLOC(volume) < 0) goto cleanup; is_new_vol = true; - vol->type = VIR_STORAGE_VOL_BLOCK; + volume->type = VIR_STORAGE_VOL_BLOCK; - if (VIR_STRDUP(vol->name, vol_name) < 0) + if (VIR_STRDUP(volume->name, vol_name) < 0) goto cleanup; } - if (!vol->key && VIR_STRDUP(vol->key, tokens[0]) < 0) + if (!volume->key && VIR_STRDUP(volume->key, tokens[0]) < 0) goto cleanup; - if (vol->target.path == NULL) { - if (virAsprintf(&vol->target.path, "%s/%s", - pool->def->target.path, vol->name) < 0) + if (volume->target.path == NULL) { + if (virAsprintf(&volume->target.path, "%s/%s", + pool->def->target.path, volume->name) < 0) goto cleanup; } - if (virStrToLong_ull(tokens[1], NULL, 10, &vol->target.capacity) < 0) { + if (virStrToLong_ull(tokens[1], NULL, 10, &volume->target.capacity) < 0) { virReportError(VIR_ERR_INTERNAL_ERROR, "%s", _("malformed volsize reported")); goto cleanup; } if (is_new_vol && - VIR_APPEND_ELEMENT(pool->volumes.objs, pool->volumes.count, vol) < 0) + VIR_APPEND_ELEMENT(pool->volumes.objs, + pool->volumes.count, + volume) < 0) goto cleanup; ret = 0; @@ -116,12 +122,13 @@ virStorageBackendZFSParseVol(virStoragePoolObjPtr pool, virStringFreeList(tokens); virStringFreeList(name_tokens); if (is_new_vol && (ret == -1)) - virStorageVolDefFree(vol); + virStorageVolDefFree(volume); return ret; } static int -virStorageBackendZFSFindVols(virStoragePoolObjPtr pool) +virStorageBackendZFSFindVols(virStoragePoolObjPtr pool, + virStorageVolDefPtr vol) { virCommandPtr cmd = NULL; char *volumes_list = NULL; @@ -157,7 +164,7 @@ virStorageBackendZFSFindVols(virStoragePoolObjPtr pool) if (STREQ(lines[i], "")) continue; - if (virStorageBackendZFSParseVol(pool, lines[i]) < 0) + if (virStorageBackendZFSParseVol(pool, vol, lines[i]) < 0) continue; } @@ -233,7 +240,7 @@ virStorageBackendZFSRefreshPool(virConnectPtr conn ATTRIBUTE_UNUSED, } /* Obtain a list of volumes */ - if (virStorageBackendZFSFindVols(pool) < 0) + if (virStorageBackendZFSFindVols(pool, NULL) < 0) goto cleanup; cleanup: @@ -285,7 +292,7 @@ virStorageBackendZFSCreateVol(virConnectPtr conn ATTRIBUTE_UNUSED, if (virCommandRun(cmd, NULL) < 0) goto cleanup; - if (virStorageBackendZFSFindVols(pool) < 0) + if (virStorageBackendZFSFindVols(pool, vol) < 0) goto cleanup; ret = 0; -- 2.0.2

2 2

Re: [libvirt] [Qemu-devel] IO accounting overhaul
by Stefan Hajnoczi 29 Aug '14

29 Aug '14

On Thu, Aug 28, 2014 at 04:38:09PM +0200, Benoît Canet wrote: > I collected some items of a cloud provider wishlist regarding I/O accouting. > > In a cloud I/O accouting can have 3 purpose: billing, helping the customers > and doing metrology to help the cloud provider seeks hidden costs. > > I'll cover the two former topic in this mail because they are the most important > business wize. > > 1) prefered place to collect billing IO accounting data: > -------------------------------------------------------- > For billing purpose the collected data must be as close as possible to what the > customer would see by using iostats in his vm. > > The first conclusion we can draw is that the choice of collecting IO accouting > data used for billing in the block devices models is right. I agree. When statistics are collected at lower layers it becomes are for the end user to understand numbers that include hidden costs for image formats, network protocols, etc. > 2) what to do with occurences of rare events: > --------------------------------------------- > > Another point is that QEMU developpers agree that they don't know which policy > to apply to some I/O accounting events. > Must QEMU discard invalid I/O write IO or account them as done ? > Must QEMU count a failed read I/O as done ? > > When discusting this with a cloud provider the following appears: these decisions > are really specific to each cloud provider and QEMU should not implement them. > The right thing to do is to add accouting counters to collect these events. > > Moreover these rare events are precious troubleshooting data so it's an additional > reason not to toss them. Sounds good, network interface statistics also include error counters. > 3) list of block I/O accouting metrics wished for billing and helping the customers > ----------------------------------------------------------------------------------- > > Basic I/O accouting data will end up making the customers bills. > Extra I/O accouting informations would be a precious help for the cloud provider > to implement a monitoring panel like Amazon Cloudwatch. One thing to be aware of is that counters inside QEMU cannot be trusted. If a malicious guest can overwrite memory in QEMU then the counters can be manipulated. For most purposes this should be okay. Just be aware that evil guests could manipulate their counters if a security hole is found in QEMU. > Here is the list of counters and statitics I would like to help implement in QEMU. > > This is the most important part of the mail and the one I would like the community > review the most. > > Once this list is settled I would proceed to implement the required infrastructure > in QEMU before using it in the device models. > > /* volume of data transfered by the IOs */ > read_bytes > write_bytes > > /* operation count */ > read_ios > write_ios > flush_ios > > /* how many invalid IOs the guest submit */ > invalid_read_ios > invalid_write_ios > invalid_flush_ios > > /* how many io error happened */ > read_ios_error > write_ios_error > flush_ios_error > > /* account the time passed doing IOs */ > total_read_time > total_write_time > total_flush_time > > /* since when the volume is iddle */ > qvolume_iddleness_time ? > > /* the following would compute latecies for slices of 1 seconds then toss the > * result and start a new slice. A weighted sumation of the instant latencies > * could help to implement this. > */ > 1s_read_average_latency > 1s_write_average_latency > 1s_flush_average_latency > > /* the former three numbers could be used to further compute a 1 minute slice value */ > 1m_read_average_latency > 1m_write_average_latency > 1m_flush_average_latency > > /* the former three numbers could be used to further compute a 1 hours slice value */ > 1h_read_average_latency > 1h_write_average_latency > 1h_flush_average_latency > > /* 1 second average number of requests in flight */ > 1s_read_queue_depth > 1s_write_queue_depth > > /* 1 minute average number of requests in flight */ > 1m_read_queue_depth > 1m_write_queue_depth > > /* 1 hours average number of requests in flight */ > 1h_read_queue_depth > 1h_write_queue_depth I think libvirt captures similar data. At least virt-manager displays graphs with similar data (maybe for CPU, memory, or network instead of disk). > 4) Making this happen > ------------------------- > > Outscale want to make these IO stat happen and gave me the go to do whatever > grunt is required to do so. > That said we could collaborate on some part of the work. Seems like a nice improvement to the query-blockstats available today. CCing libvirt for management stack ideas. Stefan

2 1

[libvirt] [PATCH 0/2] Couple more Coverity issues
by John Ferlan 29 Aug '14

29 Aug '14

After perusing the pile of 70 or so warnings - these two stuck out as ones that were low hanging fruit and not false positives. Many of the remaining "issues" are false positives or perhaps even bugs in Coverity, but I understand why they're being flagged. Freeing memory from counted lists where the incoming count must be zero based on code path - for some reason Coverity flags them because the incoming list memory is NULL and the for loop deref would be bad. The issue is Coverity doesn't seem to dig deep enough to determine that the count and list pointer are linked, sigh (yes, a lot of those). John Ferlan (2): virnetserverservice: Resolve Coverity ARRAY_VS_SINGLETON qemu_driver: Resolve Coverity FORWARD_NULL src/qemu/qemu_driver.c | 3 +-- src/rpc/virnetserverservice.c | 2 +- 2 files changed, 2 insertions(+), 3 deletions(-) -- 1.9.3

2 7

[libvirt] [PATCH v2 0/4] Introduce support for virtio-blk-pci iothreads
by John Ferlan 28 Aug '14

28 Aug '14

v1: http://www.redhat.com/archives/libvir-list/2014-August/msg01155.html Changes since v1 Patches 1-3 - purely from code review Patch 4 - rework the checking of the to be added disk that has the iothread property set to be done during qemuBuildDriveDevStr() after the config check. This way the same checks are done for both start and hotplug. Only set the "inuse" bit after qemuBuildDriveDevStr() returns successfully for both start and hotplug. This also enforces only setting for this path Since the only way a disk with the property can be added is if the current emulator supports the feature, the calls to set/clear the bit if iothread is set should be safe from not needing to also ensure iothreadmap exists. John Ferlan (4): domain_conf: Introduce iothreads XML qemu: Add support for iothreads domain_conf: Add support for iothreads in disk definition qemu: Allow use of iothreads for disk definitions docs/formatdomain.html.in | 34 ++++++++++++ docs/schemas/domaincommon.rng | 14 +++++ src/conf/domain_conf.c | 47 +++++++++++++++- src/conf/domain_conf.h | 4 ++ src/qemu/qemu_capabilities.c | 2 + src/qemu/qemu_capabilities.h | 1 + src/qemu/qemu_command.c | 64 ++++++++++++++++++++++ src/qemu/qemu_hotplug.c | 6 ++ .../qemuxml2argv-iothreads-disk.args | 17 ++++++ .../qemuxml2argv-iothreads-disk.xml | 40 ++++++++++++++ tests/qemuxml2argvdata/qemuxml2argv-iothreads.args | 8 +++ tests/qemuxml2argvdata/qemuxml2argv-iothreads.xml | 29 ++++++++++ tests/qemuxml2argvtest.c | 4 ++ tests/qemuxml2xmltest.c | 2 + 14 files changed, 271 insertions(+), 1 deletion(-) create mode 100644 tests/qemuxml2argvdata/qemuxml2argv-iothreads-disk.args create mode 100644 tests/qemuxml2argvdata/qemuxml2argv-iothreads-disk.xml create mode 100644 tests/qemuxml2argvdata/qemuxml2argv-iothreads.args create mode 100644 tests/qemuxml2argvdata/qemuxml2argv-iothreads.xml -- 1.9.3

3 16

[libvirt] [PATCH v2 0/2] network: Bring netdevs online later
by Matthew Rosato 28 Aug '14

28 Aug '14

The following patchset introduces code to defer setting netdevs online (and therefore registering MACs) until right before beginning guest CPU execution. The first patch introduces some infrastructure changes in preparation of the actual function added in the 2nd patch. Associated BZ: https://bugzilla.redhat.com/show_bug.cgi?id=1081461 Changes for v2: * Ping for comments, esp. on patch #2. * Moved @flags operand of virNetDevMacVLanCreateWithVPortProfile to the end of the operand list. * Minor changes based on comments by Martin Kletzander. * Added detail to patch #2 commit message. * Martin suggested that I replace various ? operations with if/else statements, but I ended up leaving them alone as they were being used to conditionally assign values to constant fields. * I left the contents of qemu_interface as-is, rather than collapsing them into non-qemu-specific functions, in order to keep Makefile linkage consistent & happy (needs to be part of QEMU_DRIVER_SOURCES). Instead, incorporated copyright suggestions from previous comments. Martin, if you feel strongly about not having these new functions in a qemu-specific part, feel free to comment. Changes since RFC: * Add a separate patch to introduce a flags field for macvlan/macvtap creation. * Use macvlan/tap IFUP flags to skip virNetDevSetOnline (for qemu only). * Add hotplug support. * For macvlan, save the current virNetDevVPortProfileOp in virDomainNetDef during qemuPhysIfaceConnect. As Laine mentioned, could use this field in a future patch to eliminate passing virNetDevVPortProfileOp everywhere. * Add qemu_interface.c and qemu_interface.h to encapsulate new functions. Matthew Rosato (2): util: Introduce flags field for macvtap creation network: Bring netdevs online later src/Makefile.am | 3 +- src/conf/domain_conf.h | 2 ++ src/lxc/lxc_process.c | 6 ++-- src/qemu/qemu_command.c | 12 +++++-- src/qemu/qemu_hotplug.c | 7 ++++ src/qemu/qemu_interface.c | 78 +++++++++++++++++++++++++++++++++++++++++++ src/qemu/qemu_interface.h | 32 ++++++++++++++++++ src/qemu/qemu_process.c | 4 +++ src/util/virnetdevmacvlan.c | 36 ++++++++++++-------- src/util/virnetdevmacvlan.h | 16 ++++++--- 10 files changed, 172 insertions(+), 24 deletions(-) create mode 100644 src/qemu/qemu_interface.c create mode 100644 src/qemu/qemu_interface.h -- 1.7.9.5

2 6

[libvirt] [PATCH 00/19] More Coverity patches
by John Ferlan 28 Aug '14

28 Aug '14

I almost didn't want to do this due to the sheer volume, but figured at the very least the bulk of these are resource leaks found by the much pickier new coverity scanner. After this there are "only" 70 issues found... John Ferlan (19): libxl_migration: Resolve Coverity NULL_RETURNS daemon: Resolve Coverity NEGATIVE_RETURNS domain_conf: Resolve Coverity RESOURCE_LEAK cpu_x86: Resolve Coverity RESOURCE_LEAK qemu_command: Resolve Coverity RESOURCE_LEAK qemu_agent: Resolve Coverity RESOURCE_LEAK libxl_domain: Resolve Coverity RESOURCE_LEAK qemu_capabilities: Resolve Coverity RESOURCE_LEAK network_conf: Resolve Coverity RESOURCE_LEAK virsh-network: Resolve Coverity RESOURCE_LEAK bridge_driver: Resolve Coverity RESOURCE_LEAK libxl_migration: Resolve Coverity RESOURCE_LEAK phyp_driver: Resolve Coverity RESOURCE_LEAK qemu_driver: Resolve Coverity RESOURCE_LEAK storage_conf: Resolve Coverity RESOURCE_LEAK qemu_monitor: Resolve Coverity NESTING_INDENT_MISMATCH domain_conf: Resolve Coverity DEADCODE qemu_driver: Resolve Coverity DEADCODE qemu_command: Resolve Coverity DEADCODE daemon/remote.c | 24 ++++++++++++------------ src/conf/domain_conf.c | 28 ++++++++++++++++++++++++---- src/conf/network_conf.c | 2 ++ src/conf/storage_conf.c | 2 ++ src/cpu/cpu_x86.c | 15 ++++++++++----- src/libxl/libxl_domain.c | 4 +++- src/libxl/libxl_migration.c | 11 +++++++++-- src/network/bridge_driver.c | 1 + src/phyp/phyp_driver.c | 1 + src/qemu/qemu_agent.c | 6 ++++-- src/qemu/qemu_capabilities.c | 2 +- src/qemu/qemu_command.c | 9 +++++---- src/qemu/qemu_driver.c | 8 ++++++++ src/qemu/qemu_monitor.c | 3 ++- tools/virsh-network.c | 2 +- 15 files changed, 85 insertions(+), 33 deletions(-) -- 1.9.3

4 31

[libvirt] Automatically affinitize hostdev interrupts to vm vCpus (qemu/kvm)
by Mooney, Sean K 28 Aug '14

28 Aug '14

Hi I would like to ask for comments and propose a possible new libvirt feature. Problem statement: At present when you boot a vm via libvirt it is possible to pin vm vCPUs to host CPUs to improve performance in the guest under certain conditions. If hostdev interrupts are not pined to a specific core/cpuset suboptimal processing of irqs may occur reducing the performance of both guest and host. I would like to propose extending libvirt to automatically pin interrupts for hostdev devices if they are present in the guest. By affinitizing interrupts, cache line sharing between the specified interrupt and guest can be achieved. If CPU affinity for the guest, as set by the cpuset parameter, is intelligently chosen to place the guest on the same numa node as the hostdev, cross socket traffic can be mitigated. As a result, latency which would be introduce if the interrupt processing was scheduled to a non-local numa node CPU can be reduced via interrupt pinning. Proposed change: * util/virpci and util/virhostdev will be extended to retrieve IRQ and msi_interupt information from sysfs. * util/virinterupt will be created * util/virinterupt will implement managing interrupt affinity via /proc/irq/x/smp_affinity * qemuProcessInitCpuAffinity will be extended to conditionally affinitize hostdev interrupts to vm vCpus when hostdevs are present in the vm definition. Alternative implementation: In addition to the above changes the hostdev element could be extended to include a cpuset attribute: * if the cpuset is auto: the interrupts would be pinned to the same cpuset as the vms vCPU * if a cpuset is specified: the interrupts would be affinitized as per the set cpuset * if the cpuset is native: the interrupts will not be pinned and the current behaviour will not be changed. * If the cpuset attribute is not present either the behaviour of auto or native could be used as a default. o Using auto as the default would allow transparent use of the feature. o Using native as the default would allow no changes to any existing deployments unless the feature is requested. Any feedback is welcomed. Regards Sean. -------------------------------------------------------------- Intel Shannon Limited Registered in Ireland Registered Office: Collinstown Industrial Park, Leixlip, County Kildare Registered Number: 308263 Business address: Dromore House, East Park, Shannon, Co. Clare This e-mail and any attachments may contain confidential material for the sole use of the intended recipient(s). Any review or distribution by others is strictly prohibited. If you are not the intended recipient, please contact the sender and delete all copies.

2 2

[libvirt] Proposition for the implementation of new features for Hyper-V driver
by Adrien Kantcheff 28 Aug '14

28 Aug '14

Dear libvirt developers, During my final student training in the French company Bull, with a previous student (Simon RASTELLO), we developed new features for Hyper-V driver. For a project called OpenCloudware, our work was to bring new functionalities using libvirt API in order to do basic actions on virtual machines hosted on Hyper-V. You may be interested in pushing our developments in the official release. The libvirt driver already provides functions to enumerate and get WMI classes by sending WQL requests to WMI provider and also to call WMI class methods. For that last kind of communication, a method requires input parameters. There are two kinds of argument: basic (integer, string...) and complex (objects, end point references, embedded instances). Actually, just the first argument passing mode is available with libvirt driver. But the second one is very useful because there is many WMI methods with complex types parameters, and for the moment it constraints developers to call WMI methods only with basic types parameters. So in order to expand WMI methods calls, we have implemented the second argument passing mode. Thanks to this new argument passing mode, we are available to set new functionalities. Our contributions include: * hyperv_driver.c o hypervDomainDefineXML o hypervDomainCreateXML o hypervDomainUndefine o hypervDomainUndefineFlags o hypervDomainShutdown o hypervDomainShutdownFlags o hypervDomainGetVcpus o hypervDomainGetVcpusFlags o hypervConnectGetMaxVcpus o hypervDomainGetMaxVcpus o hypervDomainSetVcpus o hypervDomainSetVcpusFlags o hypervDomainSetMemory o hypervDomainSetMemoryFlags o hypervDomainSetMaxMemory o hypervNodeGetFreeMemory o hypervDomainAttachDevice o hypervDomainAttachDeviceFlags o hypervDomainGetSchedulerParameters o hypervDomainGetSchedulerParametersFlags o hypervDomainGetSchedulerType o hypervConnectGetCapabilities o hypervConnectGetVersion o hypervDomainSetAutostart o hypervDomainGetAutostart * hyperv_network.c o hypervConnectNumOfNetworks o hypervConnectListNetworks o hypervConnectNumOfDefinedNetworks o hypervConnectListDefinedNetworks o hypervNetworkLookupByName o hypervNetworkGetXMLDesc * hyperv_storage_driver.c o hypervConnectNumOfStoragePools (stub) o hypervConnectListStoragePools (stub) o hypervConnectNumOfDefinedStoragePools (stub) o hypervStoragePoolLookupByName (stub) o hypervStorageVolLookupByPath (stub) * hyperv_private.h o Add mutex to protect against concurrent calls o Add virDomainXMLOptionPtr to parse Domain XML * hyperv_wmi.h o Structures for complex arguments: objects, EPR (end point references) and embedded instances * hyperv_wmi.c o Methods to invoke WMI methods with complex arguments * hyperv_wmi_generator.input o CIM_DataFile o Win32_ComputerSystemProduct o Msvm_VirtualSystemManagementService o Msvm_VirtualSystemGlobalSettingData o Msvm_ResourceAllocationSettingData o Msvm_AllocationCapabilities o Msvm_VirtualSwitch o Msvm_SwitchPort o Msvm_SyntheticEthernetPortSettingData o Msvm_VirtualSwitchManagementService o Win32_OperatingSystem o Win32_PerfFormattedData_HvStats_HyperVHypervisorVirtualProcessor o Win32_PerfRawData_HvStats_HyperVHypervisorVirtualProcessor * hyperv_wmi_generator.py o Add CIM_DataFile class header to be generated o Add tab classes and types to be generated o Add a function to print header types * openwsman.h o Add ws_xml_create_doc signature o Add xml_parser_get_root signature Attach files contain sources and an pdf of our contributions. I'm ending my training this week but I'm available to answer any questions you may have. You can use my personal email: adrien.kantcheff(a)gmail.com<mailto:adrien.kantcheff@gmail.com> I also put in copy of this email my tutors Yves VINTER and Christian BOURGEOIS. Feel free to contact them as well. Best regards, Adrien KANTCHEFF Bull

2 1

[libvirt] run qemu-agent-command via binding (ruby/python/php)
by Vasiliy Tolstov 28 Aug '14

28 Aug '14

Hi! Does it possible(featured, planned) to run some qemu-agent-command via libvirt binding (i'm interesting on ruby and php).? I'm understand that i can connect via socket and run it, but it very usable to get this ability inside binding. -- Vasiliy Tolstov, e-mail: v.tolstov(a)selfip.ru jabber: vase(a)selfip.ru

3 4

[libvirt] [PATCHv4] qemu: Implement bulk stats API and one of the stats groups to return
by Peter Krempa 28 Aug '14

28 Aug '14

Implement the API function for virDomainListGetStats and virConnectGetAllDomainStats in a modular way and implement the VIR_DOMAIN_STATS_STATE group of statistics. Although it may look like the function looks universal I'd rather not expose it to other drivers as the coming stats groups are likely to do qemu specific stuff to obtain the stats. --- Notes: Version 4: - fixed handling and error checking of @stats - domain filtering flags are now rejected when passing in a domain list src/qemu/qemu_driver.c | 198 +++++++++++++++++++++++++++++++++++++++++++++++++ 1 file changed, 198 insertions(+) diff --git a/src/qemu/qemu_driver.c b/src/qemu/qemu_driver.c index 73959da..45a080b 100644 --- a/src/qemu/qemu_driver.c +++ b/src/qemu/qemu_driver.c @@ -17190,6 +17190,203 @@ qemuConnectGetDomainCapabilities(virConnectPtr conn, } +static int +qemuDomainGetStatsState(virDomainObjPtr dom, + virDomainStatsRecordPtr record, + int *maxparams, + unsigned int privflags ATTRIBUTE_UNUSED) +{ + if (virTypedParamsAddInt(&record->params, + &record->nparams, + maxparams, + "state.state", + dom->state.state) < 0) + return -1; + + if (virTypedParamsAddInt(&record->params, + &record->nparams, + maxparams, + "state.reason", + dom->state.reason) < 0) + return -1; + + return 0; +} + + +typedef int +(*qemuDomainGetStatsFunc)(virDomainObjPtr dom, + virDomainStatsRecordPtr record, + int *maxparams, + unsigned int flags); + +struct qemuDomainGetStatsWorker { + qemuDomainGetStatsFunc func; + unsigned int stats; +}; + +static struct qemuDomainGetStatsWorker qemuDomainGetStatsWorkers[] = { + { qemuDomainGetStatsState, VIR_DOMAIN_STATS_STATE}, + { NULL, 0 } +}; + + +static int +qemuDomainGetStatsCheckSupport(unsigned int *stats, + bool enforce) +{ + unsigned int supportedstats = 0; + size_t i; + + for (i = 0; qemuDomainGetStatsWorkers[i].func; i++) + supportedstats |= qemuDomainGetStatsWorkers[i].stats; + + if (*stats == 0) { + *stats = supportedstats; + return 0; + } + + if (enforce && + *stats & ~supportedstats) { + virReportError(VIR_ERR_ARGUMENT_UNSUPPORTED, + _("Stats types bits 0x%x are not supported by this daemon"), + *stats & ~supportedstats); + return -1; + } + + *stats &= supportedstats; + return 0; +} + + +static int +qemuDomainGetStats(virConnectPtr conn, + virDomainObjPtr dom, + unsigned int stats, + virDomainStatsRecordPtr *record, + unsigned int flags) +{ + int maxparams = 0; + virDomainStatsRecordPtr tmp; + size_t i; + int ret = -1; + + if (VIR_ALLOC(tmp) < 0) + goto cleanup; + + for (i = 0; qemuDomainGetStatsWorkers[i].func; i++) { + if (stats & qemuDomainGetStatsWorkers[i].stats) { + if (qemuDomainGetStatsWorkers[i].func(dom, tmp, &maxparams, + flags) < 0) + goto cleanup; + } + } + + if (!(tmp->dom = virGetDomain(conn, dom->def->name, dom->def->uuid))) + goto cleanup; + + *record = tmp; + tmp = NULL; + ret = 0; + + cleanup: + if (tmp) { + virTypedParamsFree(tmp->params, tmp->nparams); + VIR_FREE(tmp); + } + + return ret; +} + + + +static int +qemuConnectGetAllDomainStats(virConnectPtr conn, + virDomainPtr *doms, + unsigned int ndoms, + unsigned int stats, + virDomainStatsRecordPtr **retStats, + unsigned int flags) +{ + virQEMUDriverPtr driver = conn->privateData; + virDomainPtr *domlist = NULL; + virDomainObjPtr dom = NULL; + virDomainStatsRecordPtr *tmpstats = NULL; + bool enforce = !!(flags & VIR_CONNECT_GET_ALL_DOMAINS_STATS_ENFORCE_STATS); + int ntempdoms; + int nstats = 0; + size_t i; + int ret = -1; + + if (ndoms) + virCheckFlags(VIR_CONNECT_GET_ALL_DOMAINS_STATS_ENFORCE_STATS, -1); + else + virCheckFlags(VIR_CONNECT_LIST_DOMAINS_FILTERS_ACTIVE | + VIR_CONNECT_LIST_DOMAINS_FILTERS_PERSISTENT | + VIR_CONNECT_LIST_DOMAINS_FILTERS_STATE | + VIR_CONNECT_GET_ALL_DOMAINS_STATS_ENFORCE_STATS, -1); + + if (virConnectGetAllDomainStatsEnsureACL(conn) < 0) + return -1; + + if (qemuDomainGetStatsCheckSupport(&stats, enforce) < 0) + return -1; + + if (!ndoms) { + unsigned int lflags = flags & (VIR_CONNECT_LIST_DOMAINS_FILTERS_ACTIVE | + VIR_CONNECT_LIST_DOMAINS_FILTERS_PERSISTENT | + VIR_CONNECT_LIST_DOMAINS_FILTERS_STATE); + + if ((ntempdoms = virDomainObjListExport(driver->domains, + conn, + &domlist, + virConnectGetAllDomainStatsCheckACL, + lflags)) < 0) + goto cleanup; + + ndoms = ntempdoms; + doms = domlist; + } + + if (VIR_ALLOC_N(tmpstats, ndoms + 1) < 0) + goto cleanup; + + for (i = 0; i < ndoms; i++) { + virDomainStatsRecordPtr tmp = NULL; + + if (!(dom = qemuDomObjFromDomain(doms[i]))) + continue; + + if (!domlist && + !virConnectGetAllDomainStatsCheckACL(conn, dom->def)) + continue; + + if (qemuDomainGetStats(conn, dom, stats, &tmp, flags) < 0) + goto cleanup; + + if (tmp) + tmpstats[nstats++] = tmp; + + virObjectUnlock(dom); + dom = NULL; + } + + *retStats = tmpstats; + tmpstats = NULL; + + ret = nstats; + + cleanup: + if (dom) + virObjectUnlock(dom); + + virDomainStatsRecordListFree(tmpstats); + virDomainListFree(domlist); + + return ret; +} + + static virDriver qemuDriver = { .no = VIR_DRV_QEMU, .name = QEMU_DRIVER_NAME, @@ -17387,6 +17584,7 @@ static virDriver qemuDriver = { .domainSetTime = qemuDomainSetTime, /* 1.2.5 */ .nodeGetFreePages = qemuNodeGetFreePages, /* 1.2.6 */ .connectGetDomainCapabilities = qemuConnectGetDomainCapabilities, /* 1.2.7 */ + .connectGetAllDomainStats = qemuConnectGetAllDomainStats, /* 1.2.8 */ }; -- 2.0.2

2 2