- Devel - libvirt List Archives

[libvirt] [PATCH 0/2] Use more of qemuDomObjFromDomain
by Michal Privoznik 25 Dec '12

25 Dec '12

I know we are in the freeze so I will push this after upstream is thaw. Michal Privoznik (2): qemu: Relax locking in DomainHasManagedSaveImage and DomainMonitorCommand qemu: Convert some APIs to use qemuDomObjFromDomain src/qemu/qemu_driver.c | 478 +++++------------------------------------------ 1 files changed, 51 insertions(+), 427 deletions(-) -- 1.7.8.6

2 7

[libvirt] libvirt will wait 20 minutes or hang when the network interface down
by Benjamin Wang (gendwang) 24 Dec '12

24 Dec '12

Hi, I find that when the network interface is down. In most scenarios, the libvirt will wait 20 minutes and report the exception. In seldom scenarios, the polling thread will hang even if the network is recovered. The following is the formal description from libcurl website: http://curl.haxx.se/docs/faq.html (Section "4.19 Why doesn't cURL return an error when the network cable is unplugged?") The following is the similar case about thread hand: http://curl.haxx.se/mail/lib-2010-07/0108.html For "wait 20 minutes", although this is the TCP normal mechanism, but if a server manages tons of thousands of devices by libvirt. When 1000 devices are down, This could cause thread leak for a long period. For thread hang, this could cause thread leak forever. I tried to add the following codes in esx_vi.c. It seems that these code can avoid the above issues. Would you give your comments? if (curl->headers == NULL) { virReportError(VIR_ERR_INTERNAL_ERROR, "%s", _("Could not build CURL header list")); return -1; } + curl_easy_setopt(curl->handle, CURLOPT_LOW_SPEED_LIMIT, 10); + curl_easy_setopt(curl->handle, CURLOPT_LOW_SPEED_TIME, 120); curl_easy_setopt(curl->handle, CURLOPT_USERAGENT, "libvirt-esx"); curl_easy_setopt(curl->handle, CURLOPT_HEADER, 0); curl_easy_setopt(curl->handle, CURLOPT_FOLLOWLOCATION, 0); curl_easy_setopt(curl->handle, CURLOPT_SSL_VERIFYPEER, B.R. Benjamin Wang

2 2

[libvirt] [PATCH 0/8] Fix PowerPC CPU driver
by Jiri Denemark 22 Dec '12

22 Dec '12

Current PowerPC driver is too complicated and mostly wrong as it was apparently created from x86 driver and not properly addapted for PowerPC architecture. More details about the issues are described in individual patches. Jiri Denemark (8): cpu: Introduce cpuModelIsAllowed internal API cpu: Make comparing PowerPC CPUs easier to read cpu: Fix PowerPCNodeData cpu: Fix loading PowerPC vendor from cpu_map.xml cpu: Reimplement PowerPCBaseline cpu: Reimplement PowerPCDecode cpu: Remove hardcoded list of PowerPC models cpu: Rename PowerPCUpdate and PowerPCDataFree functions src/cpu/cpu.c | 17 ++ src/cpu/cpu.h | 5 + src/cpu/cpu_generic.c | 19 +-- src/cpu/cpu_map.xml | 12 +- src/cpu/cpu_powerpc.c | 462 +++++++++++++++++--------------------------------- src/cpu/cpu_x86.c | 11 +- 6 files changed, 193 insertions(+), 333 deletions(-) -- 1.8.0.2

3 17

[libvirt] Release of libvirt-1.0.1
by Daniel Veillard 22 Dec '12

22 Dec '12

So as scheduled, and since we got only a few fixes on top of rc2 I made the release this morning, it is tagged in the git tree and tarball and rpms are available as usual at: ftp://libvirt.org/libvirt/ This is a balanced release with both new features, a lot of improvements and a reasonable amount of bug fixes too, overall a bit more than 350 commit, accumulated since 1.0.0. Features: - Introduce virtlockd daemon (Daniel P. Berrange) - parallels: add disk and network device support (Dmitry Guryanov) - Add virDomainSendProcessSignal API (Daniel P. Berrange) - Introduce virDomainFSTrim() public API (Michal Privoznik) - add fuse support for libvirt lxc (Gao feng) - Add Gluster protocol as supported network disk backend (Harsh Prateek Bora) - various snapshot improvements (Peter Krempa, Eric Blake) Documentation: - fix some typos in examples (Eric Blake) - Fix location of libvirt.conf and auth.conf (Michał Łomnicki) - Document offline migration (Jiri Denemark) - libvirt.c: Fix wording and grammar in virDomainFSTrim (Michal Privoznik) - Fix a few spaces (Ján Tomko) - boot order for host and redirected USB devices (Ján Tomko) - AbortJob: Fix documentation (Michal Privoznik) - Fix "virsh create" example (Guido Günther) Portability: - build: minor build fixes for BSD (Eric Blake) - Socket identity support for FreeBSD. (Roman Bogorodskiy) - Require SANLK_INQ_WAIT for inq_lockspace support (Guido Günther) - locking: Fix VPATH build and distribute generated files (Jiri Denemark) - build: Fix VPATH build (Jiri Denemark) - Change virCgroupGetAppRoot stub on non-Linux to avoid unused param warning (Daniel P. Berrange) - Qemu FreeBSD: fix compilation (Roman Bogorodskiy) - Drop mntent.h include. (Roman Bogorodskiy) - virsh: Fix usage of header termios.h (Peter Krempa) - pci: Fix building of 32bit PCI command array (Peter Krempa) - spec: require dbus-daemon when using libvirtd in Fedora (Eric Blake) - systemd: require dbus service (Eric Blake) - rpc: fix build failure with older dbus (Eric Blake) - build: fix build --without-network (Ján Tomko) - build: trivial fix error: implicit declaration of function 'malloc' (Natanael Copa) - run bootstrap if .gnulib is not present (Hu Tao) - add ppc64 and s390x to arches where qemu-kvm exists (Dan Horák) - nodeinfo: port nodecpumap to RHEL5 (Eric Blake) - spec: don't enable cgconfig under systemd (Eric Blake) Bug Fixes: - network: fix (non)update of dnsmasq config during virDomainUpdateDeviceFlags (Laine Stump) - sanlock: Re-add lockspace unconditionally (Michal Privoznik) - install: fix virtlockd installation (Eric Blake) - qemu: don't fail update netdev on bridge detach failure (Laine Stump) - selinux: fix NULL dereference in GetSecurityMountOptions (Ján Tomko) - network: prevent dnsmasq from listening on localhost (Laine Stump) - Fix error reporting when fetching SCSI/LVM keys (Daniel P. Berrange) - Fix probing of QED file format (Daniel P. Berrange) - Fix memory leak in QEMU QMP capabilities initialization (Daniel P. Berrange) - qemu: snapshot: Remove memory image if external checkpoint fails (Peter Krempa) - qemu: snapshot: Don't leak XML definition if restarting of CPUs fails (Peter Krempa) - qemu: fix uninitialized variable warning in doPeer2PeerMigrate (Ján Tomko) - conf: Remove duplicate declaration of virNetworkDNSDefPtr (Jiri Denemark) - qemu: Restart CPUs with valid async job type when doing external snapshots (Peter Krempa) - qemu: eliminate bogus error log when changing netdev's bridge (Laine Stump) - qemu: capabilities: fix machine name/canonical swappage (Cole Robinson) - Inhibit daemon shutdown during driver initialization (Guido Günther) - util: Don't fail virGetGroupIDByName when group not found (Christophe Fergeau) - util: Don't fail virGetUserIDByName when user not found (Christophe Fergeau) - storage: Error out earlier if the volume target path already exists (Osier Yang) - remote: Avoid the thread race condition (Daniel P. Berrange) - qemu: Fix memory (and FD) leak on PCI device detach (Jiri Denemark) - qemu: Don't free PCI device if adding it to activePciHostdevs fails (Jiri Denemark) - qemu: Fix error code when attaching existing device (Jiri Denemark) - Fix memory leak introduced by commit 501bfad1 (Jim Fehlig) - Shut down session libvirtd cleanly on host shutdown/user logout (Alexander Larsson) - Inhibit desktop shutdown while any virtual machines are running (Daniel P. Berrange) - conf: prevent crash with no uuid in cephx auth secret (Ján Tomko) - virsh: don't crash if shutdown --mode not provided (Eric Blake) - qemu: Fix up the default machine type for QMP probing (Viktor Mihajlovski) - Fix uninitialized variables (Guido Günther) - qemu: don't attempt undefined QMP commands (Eric Blake) - storage: fix scsi detach regression with cgroup ACLs (Eric Blake) - nwfilter: report an error on OOM (Ján Tomko) - virsh: check the return value of virStoragePoolGetAutostart (Ján Tomko) - conf: fix uninitialized variable in virDomainListSnapshots (Ján Tomko) - rpc: don't destroy xdr before creating it in virNetMessageEncodeHeader (Ján Tomko) - virsh: use correct sizeof when allocating cpumap (Ján Tomko) - util: fix virBitmap allocation in virProcessInfoGetAffinity (Ján Tomko) - conf: fix NULL check in virNetDevBandwidthParse (Ján Tomko) - Don't fail hard when we can't connect to the monitor (Guido Günther) - node_memory: Fix bug of node_memory_tune (Osier Yang) - node_memory: Do not fail if there is parameter unsupported (Osier Yang) - network: fix crash when portgroup has no name (Laine Stump) - maint: Fix use of invalid reboot flags (Michal Privoznik) - util: check for NULL parameter in virFileWrapperFdCatchError (Ján Tomko) - conf: snapshot: check return value of virDomainSnapshotObjListNum (Ján Tomko) - conf: check the return value of virXPathNodeSet (Ján Tomko) - cgroup: fix impossible overrun in virCgroupAddTaskController (Ján Tomko) - nwfilter: fix NULL pointer check in virNWFilterSnoopReqNew (Ján Tomko) - qemu: Fix error messages when dispatching guest agent commands (Peter Krempa) - qemu: Fix QMP Capabability Probing Failure (Viktor Mihajlovski) - qemu: Wait for monitor socket even without pid (Viktor Mihajlovski) - bitmap: fix typo to use UL type of integer constant in virBitmapIsAllSet (Guannan Ren) - qemu: fix a crash when save file can't be opened (Ján Tomko) - Fix leak of virNetworkPtr in LXC startup failure path (Daniel P. Berrange) - Fix error reporting in virNetDevVethDelete (Daniel P. Berrange) - Ensure transient def is removed if LXC start fails (Daniel P. Berrange) - Ensure failure to create macvtap device aborts LXC start (Daniel P. Berrange) - Avoid crash when LXC start fails with no interface target (Daniel P. Berrange) - Treat missing driver cgroup as fatal in LXC driver (Daniel P. Berrange) - Ensure LXC container exits if cgroups setup fails (Daniel P. Berrange) - Fix error handling in virSecurityManagerGetMountOptions (Daniel P. Berrange) - storage: fix device detach regression with cgroup ACLs (Eric Blake) - lxc: Don't crash if no security driver is specified in libvirt_lxc (Peter Krempa) - lxc: Avoid segfault of libvirt_lxc helper on early cleanup paths (Peter Krempa) - storage: fix logical volume cloning (Ján Tomko) - Fix exiting of libvirt_lxc program on container quit (Daniel P. Berrange) - Skip deleted timers when calculting next timeout (Daniel P. Berrange) - Fix virDiskNameToIndex to actually ignore partition numbers (Daniel P. Berrange) - qemu: Stop recursive detection of image chains when an image is missing (Peter Krempa) - storage: Fix bug of fs pool destroying (Osier Yang) - qemu: fix RBD attach regression (Scott Sullivan) - qemu: allow larger discrepency between memory & currentMemory in domain xml (Laine Stump) - sanlock: Retry after EINPROGRESS (Michal Privoznik) - virsh: fix uninitialized variable in cmdSnapshotEdit (Ján Tomko) - util: fix index when building lock owners array (Laine Stump) - Fix uninitialized variable in virLXCControllerSetupDevPTS (Daniel P. Berrange) - tests: Remove temporary directories in qemumonitorjsontest (Peter Krempa) - tests: Fix qemumonitorjsontest deadlock when the machine is under load (Peter Krempa) - qemu: Fix domain ID numbering race condition (Peter Krempa) - storage: fix broken backing chain (Philipp Hahn) - qemu: Don't corrupt pointer in qemuDomainSaveMemory() (Peter Krempa) - esx: Yet another connection fix for 5.1 (Martin Kletzander) - iohelper: Don't report errors on special FDs (Michal Privoznik) - qemu: Fix possible race when pausing guest (Peter Krempa) - conf: net: Fix deadlock if assignment of network def fails (Peter Krempa) - conf: net: Fix helper for applying new network definition (Peter Krempa) Improvements: - spec: Include lockd files in libvirt-daemon package (Jiri Denemark) - build: Install both qemu-lockd.conf and qemu-sanlock.conf (Jiri Denemark) - build: use fewer cat processes (Eric Blake) - build: Distribute more files (Jiri Denemark) - virsh: use vshReconnect for non-default connections too (Ján Tomko) - virsh: don't lie about reconnection in vshReconnect (Ján Tomko) - Add support for locking based on SCSI volume ID (Daniel P. Berrange) - Add support for locking based on LVM volume uuid (Daniel P. Berrange) - Add ability to maintain disk leases indirectly (Daniel P. Berrange) - Add a virtlockd client as a lock driver impl (Daniel P. Berrange) - Add support for re-exec() of virtlockd upon SIGUSR1 (Daniel P. Berrange) - Enable systemd socket activation with virtlockd (Daniel P. Berrange) - Implement dispatch functions for lock protocol in virtlockd (Daniel P. Berrange) - Define a wire protocol for talking to the virtlockd daemon (Daniel P. Berrange) - Introduce basic infrastructure for virtlockd daemon (Daniel P. Berrange) - Refactor creation of lock manager plugins (Daniel P. Berrange) - Support network boot for HVM guests in libxl (Jim Fehlig) - Add lots of debugging to storage file probing code (Daniel P. Berrange) - Log warning if storage magic matches, but version does not (Daniel P. Berrange) - network: match xml warning message (Eric Blake) - Fix make check with different object directory (Viktor Mihajlovski) - qemu: Improve error reporting from qemuDomainManagedSaveRemove (Peter Krempa) - qemu: Small code cleanups in the managedsave functions (Peter Krempa) - qemu: Refactor managed save functions to use domain lookup helpers (Peter Krempa) - qemu: Add a new domain lookup helper and improve the docs (Peter Krempa) - util: Fix warning message in previous patch (Peter Krempa) - util: rework error reporting in virGet(User|Group)IDByName (Peter Krempa) - Refactor virDomainHostdevFind method (Daniel P. Berrange) - Slightly refactor hostdev parsing / formating (Daniel P. Berrange) - Remove bogus const return values in storage file APIs (Daniel P. Berrange) - Add missing export of virStorageFileGetLVMKey & virStorageFileGetSCSIKey (Daniel P. Berrange) - Fix sorting of libvirt_private.syms and add syntax check rule (Daniel P. Berrange) - uml: Report error if inotify fails on driver startup (Cole Robinson) - tools: Only install guests init script if --with-init=script=redhat (Cole Robinson) - add vnc unix sockets to apparmor policy (Serge Hallyn) - add security hook for permitting hugetlbfs access (Serge Hallyn) - maint: mention when HACKING is rebuilt during make (Eric Blake) - qemu: reuse qemuMigrationIsAllowed when doing save and managedsave (Peter Krempa) - qemu: snapshot: Report better error message if migration isn't allowed (Peter Krempa) - qemu: Make qemuMigrationIsAllowed more reusable (Peter Krempa) - qemu: assume seccomp sandbox is supported since qemu 1.2 (Ján Tomko) - domain: Keep assigned class_id in domstatus XML (Michal Privoznik) - network: Create real network status files (Michal Privoznik) - bandwidth: Create network bandwidth (un)plug functions (Michal Privoznik) - bandwidth: Create rate update function (Michal Privoznik) - bandwidth: Create (un)plug functions (Michal Privoznik) - bandwidth: Create hierarchical shaping classes (Michal Privoznik) - bandwidth: add new 'floor' attribute (Michal Privoznik) - bandwidth: Attach sfq to leaf node (Michal Privoznik) - parallels: handle network adapters of type 'routed' (Dmitry Guryanov) - parallels: add routed pseudo network (Dmitry Guryanov) - parallels: parse virtual network properties (Dmitry Guryanov) - parallels: add network driver (Dmitry Guryanov) - parallels: move parallelsParseError to parallels_utils.h (Dmitry Guryanov) - parallels: add support of network interfaces to parallelsDomainDefineXML (Dmitry Guryanov) - parallels: parse information about network interfaces (Dmitry Guryanov) - examples: Fix balloon event callback (Jiri Denemark) - network: put dnsmasq parameters in conf-file instead of command line (Gene Czarcinski) - network: add support for DHCPv6 (Gene Czarcinski) - conf: split <forward> parser/clear into separate functions (Laine Stump) - conf: put data for network <forward> element into its own struct (Laine Stump) - conf: make virNetworkIpDefClear consistent with other functions (Laine Stump) - conf: rename some labels and functions in network_conf (Laine Stump) - network: backend functions for updating network dns host/srv/txt (Laine Stump) - conf: clear and parse functions for dns host/srv/txt records (Laine Stump) - conf: rename network dns host/srv/txt arrays (Laine Stump) - conf: use VIR_(INSERT|DELETE)_ELEMENT in virNetworkUpdate backend (Laine Stump) - util: add VIR_(APPEND|INSERT|DELETE)_ELEMENT (Laine Stump) - parallels: add support of removing disks (Dmitry Guryanov) - parallels: apply config after VM creation (Dmitry Guryanov) - parallels: add support of disks creation (Dmitry Guryanov) - parallels: add function parallelsGetDiskBusName (Dmitry Guryanov) - parallels: split parallelsStorageVolumeDelete function (Dmitry Guryanov) - parallels: fill volumes capacity parameter (Dmitry Guryanov) - parallels: add info about volumes (Dmitry Guryanov) - parallels: fix leaks in parallelsFindVolumes (Dmitry Guryanov) - parallels: create storage pools by VM list (Dmitry Guryanov) - parallels: remove unused code from storage driver (Dmitry Guryanov) - parallels: split parallelsStorageOpen function (Dmitry Guryanov) - parallels: handle disk devices in parallelsDomainDefineXML (Dmitry Guryanov) - parallels: add info about hard disk devices (Dmitry Guryanov) - Add support for offline migration (liguang) - S390: Assign default model "virtio" for network interfaces (Viktor Mihajlovski) - virsh: allow metadata preallocation when creating volumes (Ján Tomko) - storage: allow metadata preallocation when creating qcow2 images (Ján Tomko) - qemu: Allow the user to specify vendor and product for disk (Osier Yang) - Convert libxl driver to Xen 4.2 (Jim Fehlig) - network: prevent a few invalid configuration combinations (Laine Stump) - network: allow guest to guest IPv6 without gateway definition (Gene Czarcinski) - util: Do not keep PCI device config file open (Jiri Denemark) - util: Slightly refactor PCI list functions (Jiri Denemark) - qemu: Simplify the code (Osier Yang) - qemu: improve error for failed JSON commands (Eric Blake) - qemu: nicer error message if live disk snapshot unsupported (Eric Blake) - implement managedsave in libvirt xen legacy driver (Bamvor Jian Zhang) - Bind connection close callback APIs to python binding (Daniel P. Berrange) - Replace polling for active VMs with signalling by drivers (Daniel P. Berrange) - Only let VM drivers block libvirtd timed shutdown (Daniel P. Berrange) - Make QEMU perform managed save of all VMs on stop of libvirtd (Daniel P. Berrange) - build: fix incremental autogen.sh when no AUTHORS is present (Eric Blake) - Add iSCSI backend storage driver for ESX (Ata E Husain Bohra) - qemu: support live update of an interface's filter (Laine Stump) - nwfilter: utility function virNWFilterVarValueEqual (Stefan Berger) - conf: fix virDomainNetGetActualDirect*() and BridgeName() (Laine Stump) - virsh: Report errors if arguments of the schedinfo command are incorrect (Peter Krempa) - logging: more API needing to log flags (Eric Blake) - s/flags=%u/flags=%x/ in earlier commit (Daniel P. Berrange) - Allow duration=0 for virsh nodesuspend (Daniel P. Berrange) - Allow comma separated list of shutdown/reboot modes with virsh (Daniel P. Berrange) - Introduce APIs for splitting/joining strings (Daniel P. Berrange) - Add support for shutdown / reboot APIs in LXC driver (Daniel P. Berrange) - Introduce two new methods for triggering controlled shutdown/reboot (Daniel P. Berrange) - Move reboot/shutdown flags combination check into QEMU driver (Daniel P. Berrange) - Add APIs for talking to init via /dev/initctl (Daniel P. Berrange) - Ensure virDomainShutdownFlags logs flags parameter (Daniel P. Berrange) - Quote client identity in SASL whitelist log message (Daniel P. Berrange) - qemu: Refactor error reporting in qemu driver configuration parser (Peter Krempa) - qemu: Refactor config parameter retrieval (Peter Krempa) - network: use dnsmasq --bind-dynamic when available (Laine Stump) - util: new virSocketAddrIsPrivate function (Laine Stump) - util: capabilities detection for dnsmasq (Laine Stump) - virsh: do timing even for unusable connections (Ján Tomko) - Turn some dual-state int parameters into booleans (Daniel P. Berrange) - Introduce a 'stop' method to virDriverState (Daniel P. Berrange) - Implement virDomainSendProcessSignal for LXC driver (Daniel P. Berrange) - Add a 'send-process-signal' command to virsh (Daniel P. Berrange) - Specify remote protocol for virDomainSendProcessSignal (Daniel P. Berrange) - Rename misc QEMU structs/enums to use normal naming style (Daniel P. Berrange) - Replace 'struct qemud_driver *' with virQEMUDriverPtr (Daniel P. Berrange) - virsh: fix error messages in iface-bridge (Ján Tomko) - libssh2_session: support DSS keys as well (Ján Tomko) - virsh: Rewrite cmdDomDisplay (Martin Kletzander) - virsh: Expose new virDomainFSTrim API (Michal Privoznik) - qemu: Implement virDomainFSTrim (Michal Privoznik) - remote: Implement virDomainFSTrim (Michal Privoznik) - qemu: Add QEMU version computation to QMP probing (Viktor Mihajlovski) - mount fuse's meminfo file to container's /proc/meminfo (Gao feng) - make /proc/meminfo isolate with host through fuse (Gao feng) - add interface virCgroupGetAppRoot (Gao feng) - add interface virCgroupGetMemSwapUsage (Gao feng) - Add a configure option for fuse support in LXC driver (Gao feng) - Alphabetically sort libvirt_daemon.syms (Daniel P. Berrange) - Change bridge driver to use named initializers with virDriverState (Daniel P. Berrange) - virdbus: Add virDBusGetSessionBus helper (Alexander Larsson) - Specify name of target interface with macvlan error (Daniel P. Berrange) - Store initpid in the domain status XML for LXC (Daniel P. Berrange) - Pass virSecurityManagerPtr object further down into LXC setup code (Daniel P. Berrange) - tests: Add tests for gluster protocol based network disks support (Harsh Prateek Bora) - qemu: Add support for gluster protocol based network storage backend. (Harsh Prateek Bora) - Refactor ESX storage driver to implement facade pattern (Ata E Husain Bohra) - Add private data pointer to virStoragePool and virStorageVol (Ata E Husain Bohra) - cpu: Add Intel Haswell cpu model (Peter Krempa) - util: Use virReportSystemError for system error in pci.c (Osier Yang) - virsh: Report error when taking a snapshot with empty --memspec argument (Peter Krempa) - Warn if requesting update to non-existent timer/handle watch (Daniel P. Berrange) - qemuhelpdata: Revert my 'fix' (Michal Privoznik) - conf: Report sensible error for invalid disk name (Martin Kletzander) - Log an audit message with the LXC init pid (Daniel P. Berrange) - Use virNetServerRun instead of custom main loop (Daniel P. Berrange) - storage: Improve virStorageBackendFileSystemStop (Osier Yang) - tests: update qemuhelptest data (Ján Tomko) - tests: add boot order for host and redirected USB to qemu argv test (Ján Tomko) - conf: add support for booting from redirected USB devices (Ján Tomko) - qemu: graphics support for simultaneous one of each sdl, vnc, spice (Alon Levy) - qemu: refactor graphics code to not hardcode a single display (Alon Levy) - snapshot: make cloning of domain definition easier (Eric Blake) - snapshot: expose location through virsh snapshot-info (Eric Blake) - snapshot: implement new filter sets (Eric Blake) - snapshot: add virsh back-compat support for new filters (Eric Blake) - snapshot: add two more filter sets to API (Eric Blake) - nodeinfo: support kernels that lack socket information (Eric Blake) - snapshot: require user to supply external memory file name (Eric Blake) - virsh: save: report an error if XML file can't be read (Ján Tomko) - virsh: Use virNodeGetCPUMap if possible (Viktor Mihajlovski) - python: Use virNodeGetCPUMap where possible (Viktor Mihajlovski) - qemu, lxc: Change host CPU number detection logic. (Viktor Mihajlovski) - nodeinfo: enable nodeGetCPUCount for older kernels (Viktor Mihajlovski) - Use helper functions to format the journal iov array (Miloslav Trmač) - Add metadata to virLogOutputFunc (Miloslav Trmač) - Add a metadata parameter to virLog{, V}Message (Miloslav Trmač) - qemu: add bootindex for usb-host and usb-redir devices (Ján Tomko) - build: rerun bootstrap if AUTHORS is missing (Eric Blake) - qemu: Don't force port=0 for SPICE (Michal Privoznik) - snapshot: qemu: Fix detection of external snapshots when deleting (Peter Krempa) - nodeinfotest: Add test data from a AMD bulldozer machine. (Peter Krempa) - nodeinfotest: Add test data for 2 processor host with broken NUMA (Peter Krempa) - nodeinfo: Add check and workaround to guarantee valid cpu topologies (Peter Krempa) - qemu: Allow migration to be cancelled at prepare phase (Michal Privoznik) - capabilities: defaultConsoleTargetType can depend on architecture (Viktor Mihajlovski) - snapshot: qemu: Add support for external inactive snapshots (Peter Krempa) - virsh: add snapshot-create-as memspec support (Eric Blake) - virsh: make ,, escape parsing common (Eric Blake) - qemu: Emit event if 'cont' fails (Michal Privoznik) - Introduce new VIR_DOMAIN_EVENT_SUSPENDED_API_ERROR event (Michal Privoznik) - virsh: document which term is older (Eric Blake) - tests: Add test for controller order (Michal Privoznik) - qemu: Add controllers in specified order (Michal Privoznik) - qemu: Wrap controllers code into dummy loop (Michal Privoznik) - spec: replace scriptlets with new systemd macros (Václav Pavlín) - gitignore: ignore more files (liguang) - snapshot: qemu: Remove restrictions preventing external checkpoints (Peter Krempa) - snapshot: qemu: Add support for external checkpoints (Peter Krempa) - snapshot: qemu: Rename qemuDomainSnapshotCreateActive (Peter Krempa) - snapshot: qemu: Add async job type for snapshots (Peter Krempa) - snapshot: Add flag to enable creating checkpoints in live state (Peter Krempa) - qemu: Split out domain memory saving code to allow reuse (Peter Krempa) - qemu: Clean up snapshot retrieval to use the new helper (Peter Krempa) - cpu: Add AMD Opteron G5 cpu model (Peter Krempa) - cpu: Add newly added cpu flags (Peter Krempa) - snapshot: merge pre-snapshot checks (Eric Blake) - snapshot: populate new XML info for qemu snapshots (Eric Blake) - snapshot: improve disk align checking (Eric Blake) - snapshot: new XML for external system checkpoint (Eric Blake) - snapshot: simplify OOM checking during parse (Eric Blake) - Document bracket whitespace rules & add syntax-check rule (Daniel P. Berrange) - net: Mark network persistent when assigning persistent definition (Peter Krempa) - net: Add support for changing persistent networks to transient (Peter Krempa) - net: Re-use checks when creating transient networks (Peter Krempa) - net: Remove dnsmasq and radvd files also when destroying transient nets (Peter Krempa) - net: Move creation of dnsmasq hosts file to function starting dnsmasq (Peter Krempa) - net: Change argument type of virNetworkObjIsDuplicate() (Peter Krempa) - build: Do not ignore logging.c in sc_prohibit_mkstemp (Jiri Denemark) - qemu: use default machine type if missing it in qemu command line (Guannan Ren) Cleanups - storage: Remove the redundant white lines (Osier Yang) - Fix the indention (Osier Yang) - Do not export symbol virStateActive anymore (Osier Yang) - maint: Misc whitespace cleanups (Peter Krempa) - Fix the coding style (Osier Yang) - Fix indentions (Osier Yang) - qemu: Remove full stop from error messages (Jiri Denemark) - build: more fix to avoid C99 for loop (Hu Tao) - nwfilter: drop dead code (Eric Blake) - qemu: Drop word "either" from comments for agent monitor functions (Peter Krempa) - s/qemud/qemu/ in QEMU driver sources (Daniel P. Berrange) - Remove bogus newline at end of debug log message (Daniel P. Berrange) - build: avoid C99 for loop (Eric Blake) - util: Fix the indention (Osier Yang) - qemu/qemu_command.c: fix indent of label (Alon Levy) - qemu: Beautify code indent in migration codes (liguang) - qemu: Fix function header formating of 2 functions (Peter Krempa) - Remove spurious whitespace between function name & open brackets (Daniel P. Berrange) - Remove a chunk which should not have been pushed as part of 1.0.0 (Daniel Veillard) - logging.c: Properly indent and ignore one syntax-check rule (Michal Privoznik) Thanks everybody for your contributions, be it reports, code, reviews, ideas or documentation ! Daniel -- Daniel Veillard | Open Source and Standards, Red Hat veillard(a)redhat.com | libxml Gnome XML XSLT toolkit http://xmlsoft.org/ http://veillard.com/ | virtualization library http://libvirt.org/

4 4

[libvirt] [PATCH] S390: Re-enable capability probing for virtio devices.
by Viktor Mihajlovski 22 Dec '12

22 Dec '12

Since we switched to QMP probing, the object types are spelled out explicitly, i.e. virtio-net-pci. This has effectively disabled the capability detection of s390 virtio devices. The trivial fix is to add the s390 virtio types explicitly to qemuCapsObjectProps. Signed-off-by: Viktor Mihajlovski <mihajlov(a)linux.vnet.ibm.com> --- src/qemu/qemu_capabilities.c | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/src/qemu/qemu_capabilities.c b/src/qemu/qemu_capabilities.c index a6ac169..a705a53 100644 --- a/src/qemu/qemu_capabilities.c +++ b/src/qemu/qemu_capabilities.c @@ -1400,6 +1400,10 @@ static struct qemuCapsObjectTypeProps qemuCapsObjectProps[] = { ARRAY_CARDINALITY(qemuCapsObjectPropsVirtioBlk) }, { "virtio-net-pci", qemuCapsObjectPropsVirtioNet, ARRAY_CARDINALITY(qemuCapsObjectPropsVirtioNet) }, + { "virtio-blk-s390", qemuCapsObjectPropsVirtioBlk, + ARRAY_CARDINALITY(qemuCapsObjectPropsVirtioBlk) }, + { "virtio-net-s390", qemuCapsObjectPropsVirtioNet, + ARRAY_CARDINALITY(qemuCapsObjectPropsVirtioNet) }, { "pci-assign", qemuCapsObjectPropsPciAssign, ARRAY_CARDINALITY(qemuCapsObjectPropsPciAssign) }, { "kvm-pci-assign", qemuCapsObjectPropsPciAssign, -- 1.7.9.5

3 3

[libvirt] [PATCH] util: fix botched check for new netlink request filters
by Laine Stump 22 Dec '12

22 Dec '12

This is an adjustment to the fix for https://bugzilla.redhat.com/show_bug.cgi?id=889319 to account for two bonehead mistakes I made. commit ac2797cf2af2fd0e64c58a48409a8175d24d6f86 attempted to fix a problem with netlink in newer kernels requiring an extra attribute with a filter flag set in order to receive an IFLA_VFINFO_LIST from netlink. Unfortunately, the #ifdef that protected against compiling it in on systems without the new flag went a bit too far, assuring that the new code would *never* be compiled. The first problem was that, while some IFLA_* enum values are also not, so checking to see if it's #defined is not a valid method of determining whether or not to add the attribute. Fortunately, the flag that is being set (RTEXT_FILTER_VF) *is* #defined, and it is never present if IFLA_EXT_MASK isn't, so it's sufficient to just check for that flag. And to top it off, due to the code not actually compiling when I thought it did, I didn't realize that I'd been given the wrong arglist to nla_put() - you can't just send a const value to nla_put, you have to send it a pointer to memory containing what you want to add to the message, along with the length of that memory. This time I've actually sent the patch over to the other machine that's expriencing the problem, applied it to the branch being used (0.10.2) and verified that it works properly, i.e. it does fix the problem it's supposed to fix. :-/ --- src/util/virnetdev.c | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/src/util/virnetdev.c b/src/util/virnetdev.c index e5a0d6d..898e77a 100644 --- a/src/util/virnetdev.c +++ b/src/util/virnetdev.c @@ -1275,13 +1275,17 @@ virNetDevLinkDump(const char *ifname, int ifindex, goto buffer_too_small; } -# if defined(IFLA_EXT_MASK) && defined(RTEXT_FILTER_VF) +# if defined(RTEXT_FILTER_VF) /* if this filter exists in the kernel's netlink implementation, * we need to set it, otherwise the response message will not * contain the IFLA_VFINFO_LIST that we're looking for. */ - if (nla_put(nl_msg, IFLA_EXT_MASK, RTEXT_FILTER_VF) < 0) + { + uint32_t ifla_ext_mask = RTEXT_FILTER_VF; + + if (nla_put(nl_msg, IFLA_EXT_MASK, sizeof(ifla_ext_mask), &ifla_ext_mask) < 0) goto buffer_too_small; + } # endif if (virNetlinkCommand(nl_msg, recvbuf, &recvbuflen, -- 1.7.11.7

2 2

[libvirt] qemumonitorjsontest still hangs
by Eric Blake 22 Dec '12

22 Dec '12

I ran 'make check' under a heavily loaded system, and it hung in qemumonitorjsontest as follows: (gdb) t a a bt Thread 2 (Thread 0x2b0f8000e700 (LWP 423)): #0 0x00000032f96e998d in poll () from /lib64/libc.so.6 #1 0x00002b0f7f3a1951 in virEventPollRunOnce () at util/event_poll.c:615 #2 0x00002b0f7f39fb26 in virEventRunDefaultImpl () at util/event.c:247 #3 0x00000000004c25fa in qemuMonitorTestWorker (opaque=0x1ad8bc0) at qemumonitortestutils.c:308 #4 0x00002b0f7f3bbc39 in virThreadHelper (data=0x1ada580) at util/threads-pthread.c:161 #5 0x00000032f9a07d15 in start_thread () from /lib64/libpthread.so.0 #6 0x00000032f96f246d in clone () from /lib64/libc.so.6 Thread 1 (Thread 0x2b0f7fe0bbc0 (LWP 402)): #0 0x00000032f9a08e60 in pthread_join () from /lib64/libpthread.so.0 #1 0x00002b0f7f3bbeae in virThreadJoin (thread=0x1ad8be8) at util/threads-pthread.c:236 #2 0x00000000004c27b0 in qemuMonitorTestFree (test=0x1ad8bc0) at qemumonitortestutils.c:362 #3 0x00000000004144ba in testQemuMonitorJSONGetMachines (data=0x1ac7650) at qemumonitorjsontest.c:298 #4 0x0000000000414ef2 in virtTestRun (title=0x4c3434 "GetMachines", nloops=1, body=0x413f3e <testQemuMonitorJSONGetMachines>, data=0x1ac7650) at testutils.c:160 ---Type <return> to continue, or q <return> to quit--- #5 0x00000000004149f4 in mymain () at qemumonitorjsontest.c:446 #6 0x0000000000416092 in virtTestMain (argc=1, argv=0x7fffc02dcac8, func=0x41493d <mymain>) at testutils.c:719 #7 0x0000000000414aab in main (argc=1, argv=0x7fffc02dcac8) at qemumonitorjsontest.c:455 This was with latest libvirt.git, so I'm worried we still have a nasty race somewhere in there. -- Eric Blake eblake redhat com +1-919-301-3266 Libvirt virtualization library http://libvirt.org

1 0

[libvirt] [PATCH] blockjob: fix memleak that prevented block pivot
by Eric Blake 22 Dec '12

22 Dec '12

https://bugzilla.redhat.com/show_bug.cgi?id=888426 The code for doing a block-copy was supposed to track the destination file in drive->mirror, but was set up to do all mallocs prior to starting the copy so that OOM wouldn't leave things partially started. However, the wrong variable was being written; later in the code we silently did 'disk->mirror = mirror' which was still NULL, and thus leaking memory and leaving libvirt to think that the mirror job was never started, which prevented a pivot operation after a copy. * src/qemu/qemu_driver.c (qemuDomainBlockCopy): Initialize correct variable. --- src/qemu/qemu_driver.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/src/qemu/qemu_driver.c b/src/qemu/qemu_driver.c index 7d8dd21..184f852 100644 --- a/src/qemu/qemu_driver.c +++ b/src/qemu/qemu_driver.c @@ -13429,7 +13429,7 @@ qemuDomainBlockCopy(virDomainPtr dom, const char *path, } if (!format && disk->mirrorFormat > 0) format = virStorageFileFormatTypeToString(disk->mirrorFormat); - if (!(disk->mirror = strdup(dest))) { + if (!(mirror = strdup(dest))) { virReportOOMError(); goto endjob; } -- 1.7.1

2 2

[libvirt] vhost-net on ethernet interface type
by Raja Sivaramakrishnan 22 Dec '12

22 Dec '12

Hello, I would like to enable vhost-net on an interface of ethernet type. However, this doesn't work, probably due to the following code in libvirt. if (actualType == VIR_DOMAIN_NET_TYPE_NETWORK || actualType == VIR_DOMAIN_NET_TYPE_BRIDGE || actualType == VIR_DOMAIN_NET_TYPE_DIRECT) { /* Attempt to use vhost-net mode for these types of network device */ Can VIR_DOMAIN_NET_TYPE_ETHERNET also be included in the check above? Or is there some reason to disable vhost-net for this type? Thanks, Raja

2 2

[libvirt] [PATCH 0/8] Dump domain's IP addresses
by Michal Privoznik 22 Dec '12

22 Dec '12

It's been a while since I tried get this in. I've reworked the patches, the exposed API, and start new round of reviews. Moreover, during work I've come to point, where extending qemu guest agent seemed wise: http://lists.nongnu.org/archive/html/qemu-devel/2012-12/msg03264.html Therefore I am introducing 'flags' field, which can contain some boolean values we don't have yet, e.g. IFF_UP, IFF_PROMISC, etc. Re: 'dstaddr' field in struct _virDomainInterfaceIPAddress; It's basically a join of: union { char *dstaddr; /* for IFF_POINTOPOINT interface */ char *broadaddr; /* for IFF_BROADCAST interface */ } Since an interface cannot has both flags set (see man 3 getifaddrs) I've joined both into one 'char *dstaddr'. I know it is not mnemonic as the union, so I left it for discussion. I can change it if you want to. Michal Privoznik (8): Introduce virDomainInterfacesAddresses API Introduce virDomainInterfaceFree API qemu_agent: Implement 'guest-network-get-interfaces' command handling qemu: Implement qemuDomainInterfacesAddresses virsh: Expose virDomainInterfacesAddresses remote: Implement virDomainInterfacesAddresses python: Expose virDomainInterfacesAddresses python: create example for dumping domain IP addresses daemon/remote.c | 138 +++++++++++++++++++++++++++++++++ examples/python/Makefile.am | 2 +- examples/python/README | 1 + examples/python/domipaddrs.py | 50 ++++++++++++ include/libvirt/libvirt.h.in | 52 +++++++++++++ python/generator.py | 3 + python/libvirt-override-api.xml | 7 ++ python/libvirt-override.c | 120 +++++++++++++++++++++++++++++ src/driver.h | 6 ++ src/libvirt.c | 102 +++++++++++++++++++++++++ src/libvirt_public.syms | 7 ++ src/qemu/qemu_agent.c | 164 ++++++++++++++++++++++++++++++++++++++++ src/qemu/qemu_agent.h | 2 + src/qemu/qemu_driver.c | 68 +++++++++++++++++ src/remote/remote_driver.c | 86 +++++++++++++++++++++ src/remote/remote_protocol.x | 27 ++++++- src/remote_protocol-structs | 27 +++++++ tools/virsh-domain.c | 102 +++++++++++++++++++++++++ 18 files changed, 962 insertions(+), 2 deletions(-) create mode 100755 examples/python/domipaddrs.py -- 1.8.0.2

2 12

[libvirt] [PATCH 0/6] BUG fix for PowerPC.
by Li Zhang 21 Dec '12

21 Dec '12

From: Li Zhang <zhlcindy(a)linux.vnet.ibm.com> This patch series is to fix bugs on PowerPC we found when testing libvirt. Li Zhang (6): Set std VGA model as default model for ppc64. Use -machine option for PPC64 Remove QEMU_CAPS_NO_ACPI capability for non-x86 platform Support all backend serial devices for pSeries guest Set the default machine as pseries for PPC64 Use help string "-device driver," when getting additional caps. src/conf/capabilities.c | 6 ++++ src/conf/domain_conf.c | 2 ++ src/qemu/qemu_capabilities.c | 12 ++++++-- src/qemu/qemu_command.c | 31 +++++++++++--------- .../qemuxml2argv-machine-core-off.args | 2 +- .../qemuxml2argv-machine-core-on.args | 2 +- .../qemuxml2argv-pseries-basic.args | 2 +- .../qemuxml2argv-pseries-vio-user-assigned.args | 2 +- .../qemuxml2argvdata/qemuxml2argv-pseries-vio.args | 2 +- 9 files changed, 39 insertions(+), 22 deletions(-) -- 1.7.10.1

3 19

[libvirt] [PATCH v2 0/28] Rename files in src/util to have a 'vir' prefix
by Daniel P. Berrange 21 Dec '12

21 Dec '12

Currently the files in src/util are 50/50 split between those having a 'vir' prefix and those which don't. Finish the job by remaining all remaining files to have a 'vir' prefix too, guaranteeing that they can't clash with system headers[1]. The only case which was not a plain rename, was merging processinfo.{c,h} into the virprocess.{c,h} since the functionality logically belongs together. At the end src/Makefile.am is updated to sort util files alphabetically. Left as further work - make sure all functions in these files also have a 'vir' prefix. eg dnsmasq, ebtables, iptables, usb and pci files are violators in this regard. Also left, much of virutil.c should move into virfile.c Changes in v2: - Rebased to latest GIT - Turned on rename detection to make review easier Daniel [1] Not a theoretical problem - we already hit a clsah on 'usb.h' before forcing us to use 'hostusb.h' and 'memory.h' clashes with a system header but we never noticed so far. In-Reply-To:

3 32

[libvirt] [test-API][PATCH] managedsave: change the checking method about bypass cache
by hongming 21 Dec '12

21 Dec '12

Check whether the O_DIRECT flag is in use on the managed save file during the duration of the managed save, since that is the real effect of the current implementation of the --bypass-cache flag. --- repos/managedsave/managedsave.py | 66 +++++++++++++++++++------------------ 1 files changed, 34 insertions(+), 32 deletions(-) diff --git a/repos/managedsave/managedsave.py b/repos/managedsave/managedsave.py index 4a92151..b898876 100644 --- a/repos/managedsave/managedsave.py +++ b/repos/managedsave/managedsave.py @@ -2,6 +2,8 @@ import os import math +import thread +import time import libvirt from libvirt import libvirtError @@ -39,39 +41,42 @@ def check_savefile_create(*args): logger.info("managed save file exists") return True -def compare_cachedfile(cachebefore, cacheafter): - """Compare cached value before managed save and its value after - managed save """ - - diff = cacheafter - cachebefore - logger.info("diff is %s " % diff) - percent = math.fabs(diff)/cachebefore - logger.info("diff percent is %s " % percent) - if percent < 0.05: - return True - else: - return False - -def get_cachevalue(): - """Get the file system cached value """ - - cmds = "head -n4 /proc/meminfo|grep Cached|awk '{print $2}'" - (status, output) = utils.exec_cmd(cmds, shell=True) - if status != 0: - logger.error("Fail to run cmd line to get cache") - return 1 +def get_fileflags(): + """Get the file flags of managed save file""" + cmds = "cat /proc/$(lsof -w /var/lib/libvirt/qemu/save/"+guestname+".save"\ + "|awk '/libvirt_i/{print $2}')/fdinfo/1|grep flags|awk '{print $NF}'" + global fileflags + while True: + (status, output) = utils.exec_cmd(cmds, shell=True) + if status == 0: + for i in range(len(output)): + logger.info("The flags of saved file %s" % output[i]) + fileflags = output[i][-5] + else: + logger.error("Fail to get the flags of saved file") + return 1 + + thread.exit_thread() + +def check_fileflag(fileflags): + """Check the file flags of managed save file if include O_DIRECT""" + if int(fileflags) == 4: + logger.info("file flags include O_DIRECT") + return True else: - logger.debug(output[0]) - cachevalue= int(output[0]) - return cachevalue + logger.error("file flags doesn't include O_DIRECT") + return False def managedsave(params): """Managed save a running domain""" global logger logger = params['logger'] + global guestname guestname = params['guestname'] flags = params ['flags'] + global fileflags + fileflags = '' #Save given flags to sharedmod.data sharedmod.data['flagsave'] = flags @@ -122,17 +127,14 @@ def managedsave(params): #If given flags include bypass-cache,check if bypass file system cache if flagn % 2 == 1: - logger.info("Given flags include --bypass-cache") - os.system('echo 3 > /proc/sys/vm/drop_caches') - cache_before = get_cachevalue() - logger.info("Cached value before managedsave is %s" % cache_before) + logger.info("Given flags include --bypass-cache") + thread.start_new_thread(get_fileflags,()) + # Guarantee get_fileflags shell has run before managed save + time.sleep(5) domobj.managedSave(flagn) - cache_after = get_cachevalue() - logger.info("Cached value after managedsave is %s" % cache_after) - - if compare_cachedfile(cache_before, cache_after): + if check_fileflag(fileflags): logger.info("Bypass file system cache successfully") else: logger.error("Bypass file system cache failed") -- 1.7.7.6

1 0

[libvirt] [PATCH 0/2] account for change in netlink kernel ABI
by Laine Stump 21 Dec '12

21 Dec '12

The first of these two patches changes libvirt to account for a change in netlink's kernel ABI that happened upstream several months ago. The second patch adds in error logs to the functions that failed after the kernel change. Laine Stump (2): util: fix functions that retrieve SRIOV VF info util: add missing error log messages when failing to get netlink VFINFO src/util/virnetdev.c | 87 +++++++++++++++++++++++++++++----------------------- 1 file changed, 49 insertions(+), 38 deletions(-) -- 1.7.11.7

2 4

[libvirt] [PATCH 1/1] Add sysinfo data and enable virSysinfoRead test case for POWER
by Li Zhang 21 Dec '12

21 Dec '12

From: Li Zhang <zhlcindy(a)linux.vnet.ibm.com> This patch is to enable virSysinfoRead test case for POWER, and provide sysinfo data on POWER. Signed-off-by: Li Zhang <zhlcindy(a)linux.vnet.ibm.com> Reviewed-by: Viktor Mihajlovski <mihajlov(a)linux.vnet.ibm.com> --- tests/sysinfodata/ppccpuinfo.data | 85 ++++++++++++++++++++++++++++++++++ tests/sysinfodata/ppcsysinfo.expect | 87 +++++++++++++++++++++++++++++++++++ tests/sysinfotest.c | 25 ---------- 3 files changed, 172 insertions(+), 25 deletions(-) create mode 100644 tests/sysinfodata/ppccpuinfo.data create mode 100644 tests/sysinfodata/ppcsysinfo.expect diff --git a/tests/sysinfodata/ppccpuinfo.data b/tests/sysinfodata/ppccpuinfo.data new file mode 100644 index 0000000..bf4433c --- /dev/null +++ b/tests/sysinfodata/ppccpuinfo.data @@ -0,0 +1,85 @@ +processor : 0 +cpu : POWER7 (raw), altivec supported +clock : 8.388608MHz +revision : 2.3 (pvr 003f 0203) + +processor : 4 +cpu : POWER7 (raw), altivec supported +clock : 8.388608MHz +revision : 2.3 (pvr 003f 0203) + +processor : 8 +cpu : POWER7 (raw), altivec supported +clock : 8.388608MHz +revision : 2.3 (pvr 003f 0203) + +processor : 12 +cpu : POWER7 (raw), altivec supported +clock : 8.388608MHz +revision : 2.3 (pvr 003f 0203) + +processor : 16 +cpu : POWER7 (raw), altivec supported +clock : 8.388608MHz +revision : 2.3 (pvr 003f 0203) + +processor : 20 +cpu : POWER7 (raw), altivec supported +clock : 8.388608MHz +revision : 2.3 (pvr 003f 0203) + +processor : 24 +cpu : POWER7 (raw), altivec supported +clock : 8.388608MHz +revision : 2.3 (pvr 003f 0203) + +processor : 28 +cpu : POWER7 (raw), altivec supported +clock : 8.388608MHz +revision : 2.3 (pvr 003f 0203) + +processor : 32 +cpu : POWER7 (raw), altivec supported +clock : 8.388608MHz +revision : 2.3 (pvr 003f 0203) + +processor : 36 +cpu : POWER7 (raw), altivec supported +clock : 8.388608MHz +revision : 2.3 (pvr 003f 0203) + +processor : 40 +cpu : POWER7 (raw), altivec supported +clock : 8.388608MHz +revision : 2.3 (pvr 003f 0203) + +processor : 44 +cpu : POWER7 (raw), altivec supported +clock : 8.388608MHz +revision : 2.3 (pvr 003f 0203) + +processor : 48 +cpu : POWER7 (raw), altivec supported +clock : 8.388608MHz +revision : 2.3 (pvr 003f 0203) + +processor : 52 +cpu : POWER7 (raw), altivec supported +clock : 8.388608MHz +revision : 2.3 (pvr 003f 0203) + +processor : 56 +cpu : POWER7 (raw), altivec supported +clock : 8.388608MHz +revision : 2.3 (pvr 003f 0203) + +processor : 60 +cpu : POWER7 (raw), altivec supported +clock : 8.388608MHz +revision : 2.3 (pvr 003f 0203) + +timebase : 512000000 +platform : PowerNV +model : 8246-L2C +machine : PowerNV 8246-L2C +firmware : OPAL v2 diff --git a/tests/sysinfodata/ppcsysinfo.expect b/tests/sysinfodata/ppcsysinfo.expect new file mode 100644 index 0000000..32407e0 --- /dev/null +++ b/tests/sysinfodata/ppcsysinfo.expect @@ -0,0 +1,87 @@ +<sysinfo type='smbios'> + <system> + <entry name='version'>PowerNV 8246-L2C</entry> + <entry name='serial'>8246-L2C</entry> + <entry name='family'>PowerNV</entry> + </system> + <processor> + <entry name='socket_destination'>0</entry> + <entry name='type'>POWER7 (raw), altivec supported</entry> + <entry name='version'>2.3 (pvr 003f 0203)</entry> + </processor> + <processor> + <entry name='socket_destination'>4</entry> + <entry name='type'>POWER7 (raw), altivec supported</entry> + <entry name='version'>2.3 (pvr 003f 0203)</entry> + </processor> + <processor> + <entry name='socket_destination'>8</entry> + <entry name='type'>POWER7 (raw), altivec supported</entry> + <entry name='version'>2.3 (pvr 003f 0203)</entry> + </processor> + <processor> + <entry name='socket_destination'>12</entry> + <entry name='type'>POWER7 (raw), altivec supported</entry> + <entry name='version'>2.3 (pvr 003f 0203)</entry> + </processor> + <processor> + <entry name='socket_destination'>16</entry> + <entry name='type'>POWER7 (raw), altivec supported</entry> + <entry name='version'>2.3 (pvr 003f 0203)</entry> + </processor> + <processor> + <entry name='socket_destination'>20</entry> + <entry name='type'>POWER7 (raw), altivec supported</entry> + <entry name='version'>2.3 (pvr 003f 0203)</entry> + </processor> + <processor> + <entry name='socket_destination'>24</entry> + <entry name='type'>POWER7 (raw), altivec supported</entry> + <entry name='version'>2.3 (pvr 003f 0203)</entry> + </processor> + <processor> + <entry name='socket_destination'>28</entry> + <entry name='type'>POWER7 (raw), altivec supported</entry> + <entry name='version'>2.3 (pvr 003f 0203)</entry> + </processor> + <processor> + <entry name='socket_destination'>32</entry> + <entry name='type'>POWER7 (raw), altivec supported</entry> + <entry name='version'>2.3 (pvr 003f 0203)</entry> + </processor> + <processor> + <entry name='socket_destination'>36</entry> + <entry name='type'>POWER7 (raw), altivec supported</entry> + <entry name='version'>2.3 (pvr 003f 0203)</entry> + </processor> + <processor> + <entry name='socket_destination'>40</entry> + <entry name='type'>POWER7 (raw), altivec supported</entry> + <entry name='version'>2.3 (pvr 003f 0203)</entry> + </processor> + <processor> + <entry name='socket_destination'>44</entry> + <entry name='type'>POWER7 (raw), altivec supported</entry> + <entry name='version'>2.3 (pvr 003f 0203)</entry> + </processor> + <processor> + <entry name='socket_destination'>48</entry> + <entry name='type'>POWER7 (raw), altivec supported</entry> + <entry name='version'>2.3 (pvr 003f 0203)</entry> + </processor> + <processor> + <entry name='socket_destination'>52</entry> + <entry name='type'>POWER7 (raw), altivec supported</entry> + <entry name='version'>2.3 (pvr 003f 0203)</entry> + </processor> + <processor> + <entry name='socket_destination'>56</entry> + <entry name='type'>POWER7 (raw), altivec supported</entry> + <entry name='version'>2.3 (pvr 003f 0203)</entry> + </processor> + <processor> + <entry name='socket_destination'>60</entry> + <entry name='type'>POWER7 (raw), altivec supported</entry> + <entry name='version'>2.3 (pvr 003f 0203)</entry> + </processor> +</sysinfo> diff --git a/tests/sysinfotest.c b/tests/sysinfotest.c index 79ee128..353bc76 100644 --- a/tests/sysinfotest.c +++ b/tests/sysinfotest.c @@ -50,27 +50,6 @@ struct testSysinfoData { char *expected; /* (required) file containing output of virSysinfoFormat */ }; -# if defined(__powerpc__) || defined(__powerpc64__) -/* TODO ppc: remove the if defined() branch - to enable the real test run after providing test data, see below. - */ -static int -testSysinfo(const void *data ATTRIBUTE_UNUSED) -{ - return EXIT_AM_SKIP; -} - -static int -sysinfotest_run(const char *test ATTRIBUTE_UNUSED, - const char *decoder ATTRIBUTE_UNUSED, - const char *sysinfo ATTRIBUTE_UNUSED, - const char *cpuinfo ATTRIBUTE_UNUSED, - const char *expected ATTRIBUTE_UNUSED) -{ - return testSysinfo(NULL); -} -# else - static int testSysinfo(const void *data) { @@ -143,7 +122,6 @@ error: VIR_FREE(testdata.expected); return ret; } -# endif /* defined(__powerpc__) ... */ # if defined(__s390__) || defined(__s390x__) static int @@ -158,9 +136,6 @@ test_s390(void) VIRT_TEST_MAIN(test_s390) # elif defined(__powerpc__) || defined(__powerpc64__) -/* TODO for PPC owner: provide test data - and enable the real sysinfotest_run above -*/ static int test_ppc(void) { -- 1.7.10.1

3 2

[libvirt] [PATCH] security: fix #endif comment in security_stack.h
by Ján Tomko 21 Dec '12

21 Dec '12

--- Pushed under the trivial rule. --- src/security/security_stack.h | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/src/security/security_stack.h b/src/security/security_stack.h index 4ba55f7..b38f9a9 100644 --- a/src/security/security_stack.h +++ b/src/security/security_stack.h @@ -35,4 +35,4 @@ virSecurityStackGetPrimary(virSecurityManagerPtr mgr); virSecurityManagerPtr* virSecurityStackGetNested(virSecurityManagerPtr mgr); -#endif /* __VIR_SECURITY_DAC */ +#endif /* __VIR_SECURITY_STACK */ -- 1.7.8.6

1 0

[libvirt] [PATCH 0/5] Miscellaneous cleanups
by Martin Kletzander 21 Dec '12

21 Dec '12

My cleanup branch got up to 5 commits so in order to get rid of it, I'm sending it. Some of them could be most probably pushed as trivial, but since some are older (even though the commit date is new), I'm sending it in the pack together. Martin Kletzander (5): fix typo in the word affinities conf: minor indentation cleanups conf: Don't format cputune element when not needed conf: eliminate redundant use of VIR_ALLOC docs: aesthetical cleanups docs/hacking.html.in | 12 ++++---- src/conf/domain_conf.c | 80 ++++++++++++++++++++++-------------------------- src/libxl/libxl_driver.c | 4 +-- src/qemu/qemu_process.c | 8 ++--- 4 files changed, 48 insertions(+), 56 deletions(-) -- 1.8.0.2

2 14

[libvirt] [PATCH] conf: cpu: Fix to escape cpu model, vendor_id and vendor
by Ken ICHIKAWA 20 Dec '12

20 Dec '12

This patch fixes a problem that encoded xml special charas of cpu model, vendor_id and vendor are saved as decoded xml special charas. If I define model name including encoded xml special charas in domain XML like below: <domain> <cpu mode='custom' match='exact'> <model fallback='allow'>><</model> </cpu> </domain> domain xml definition file is saved as <domain> <cpu mode='custom' match='exact'> <model fallback='allow'>><</model> </cpu> </domain> After that, if libvirtd is restarted, dumpxml command, edit command and etc. are disabled with below error until I stop libvirtd and fix the xml file manually. 2012-12-20 04:56:54.371+0000: 11893: error : catchXMLError:691 : /etc/libvirt/qemu/f17.xml:25: StartTag: invalid element name <model fallback='allow'>><</model> ------------------------------^ Also vendor_id and vendor have same problem. Signed-off-by: Ken ICHIKAWA <ichikawa.ken(a)jp.fujitsu.com> --- src/conf/cpu_conf.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/src/conf/cpu_conf.c b/src/conf/cpu_conf.c index 3973c83..e9f9431 100644 --- a/src/conf/cpu_conf.c +++ b/src/conf/cpu_conf.c @@ -609,17 +609,17 @@ virCPUDefFormatBuf(virBufferPtr buf, } virBufferAsprintf(buf, " fallback='%s'", fallback); if (def->vendor_id) - virBufferAsprintf(buf, " vendor_id='%s'", def->vendor_id); + virBufferEscapeString(buf, " vendor_id='%s'", def->vendor_id); } if (formatModel && def->model) { - virBufferAsprintf(buf, ">%s</model>\n", def->model); + virBufferEscapeString(buf, ">%s</model>\n", def->model); } else { virBufferAddLit(buf, "/>\n"); } } if (formatModel && def->vendor) - virBufferAsprintf(buf, "<vendor>%s</vendor>\n", def->vendor); + virBufferEscapeString(buf, "<vendor>%s</vendor>\n", def->vendor); if (def->sockets && def->cores && def->threads) { virBufferAddLit(buf, "<topology"); -- 1.7.11.7

2 1

[libvirt] [PATCH v2.1 00/11] Rework storage migration
by Michal Privoznik 20 Dec '12

20 Dec '12

This patch set re-implements migration with storage for enough new qemu. Currently, you can migrate a domain to a host without need for shared storage. This is done by setting 'blk' or 'inc' attribute (representing VIR_MIGRATE_NON_SHARED_DISK and VIR_MIGRATE_NON_SHARED_INC flags respectively) of 'migrate' monitor command. However, the qemu implementation is buggy and applications are advised to switch to new impementation which, moreover, offers some nice features, like migrating only explicitly specified disks. The new functionality is controlled via 'nbd-server-*' and 'drive-mirror' commands. The flow is meant to look like this: 1) User invokes libvirt's migrate functionality. 2) libvirt checks that no block jobs are active on the source. 3) libvirt starts the destination QEMU and sets up the NBD server using the nbd-server-start and nbd-server-add commands. 4) libvirt starts drive-mirror with a destination pointing to the remote NBD server, for example nbd:host:port:exportname=diskname (where diskname is the -drive id specified on the destination). 5) once all mirroring jobs reach steady state, libvirt invokes the migrate command. 6) once migration completed, libvirt invokes the nbd-server-stop command on the destination QEMU. If we just skip the 2nd step and there is an active block-job, qemu will fail in step 4. No big deal. Since we try to NOT break migration and keep things compatible, this feature is enabled iff both sides support it. Since there's obvious need for some data transfer between src and dst, I've put it into qemuCookieMigration: 1) src -> dest: (QEMU_MIGRATION_PHASE_BEGIN3 -> QEMU_MIGRATION_PHASE_PREPARE) <nbd> <disk size='17179869184'/> </nbd> Hey destination, I know how to use this cool new feature. Moreover, these are the disks I'll send you. Each one of them is X bytes big. It's one of the prerequisite - the file (disk->src) on dst exists and has at least the same size as on dst. 2) dst -> src: (QEMU_MIGRATION_PHASE_PREPARE -> QEMU_MIGRATION_PHASE_PERFORM3) <nbd port='X'/> Okay, I (destination) support this feature as well. I've created all files as you (src) told me to and you can start rolling data. I am listening on port X. 3) src -> dst: (QEMU_MIGRATION_PHASE_PERFORM3 -> QEMU_MIGRATION_PHASE_FINISH3) <nbd port='-1'/> Migration completed, destination, you may shut the NBD server down. If either src or dst doesn't support NBD, it is not used and whole process fall backs to old implementation. diff to v1: -Eric's and Daniel's suggestions worked in. To point out the bigger ones: don't do NBD style when TUNNELLED requested, added 'b:writable' to 'nbd-server-add' -drop '/qemu-migration/nbd/disk/@src' attribute from migration cookie. As pointed out by Jirka, disk->src can be changed during migration (e.g. by migration hook or by passed xml). So I've tried (as suggested on the list) passing disk alias. However, since qemu hasn't been started on destination yet, the aliases hasn't been generated yet. So we have to rely on ordering completely. diff to v2: -rebase to reflect changes made by offline migration patch -send initial nbd cookie only if needed The patches 1,3 and 5 has been ACKed already. Michal Privoznik (11): qemu: Introduce NBD_SERVER capability Introduce NBD migration cookie qemu: Introduce nbd-server-start command qemu: Introduce nbd-server-add command qemu: Introduce nbd-server-stop command qemu_migration: Introduce qemuMigrationStartNBDServer qemu_migration: Move port allocation to a separate func qemu_migration: Implement qemuMigrationStartNBDServer() qemu_migration: Implement qemuMigrationDriveMirror qemu_migration: Check size prerequisites qemu_migration: Stop NBD server at Finish phase src/qemu/qemu_capabilities.c | 3 + src/qemu/qemu_capabilities.h | 1 + src/qemu/qemu_migration.c | 624 ++++++++++++++++++++++++++++++++++++++++--- src/qemu/qemu_monitor.c | 63 +++++ src/qemu/qemu_monitor.h | 7 + src/qemu/qemu_monitor_json.c | 95 +++++++ src/qemu/qemu_monitor_json.h | 7 + 7 files changed, 768 insertions(+), 32 deletions(-) -- 1.8.0.2

2 14

[libvirt] [PATCH] maint: formatting cleanups in buf.c
by Eric Blake 20 Dec '12

20 Dec '12

* src/util/buf.c: Use consistent formatting. --- Pushing under the trivial rule. src/util/buf.c | 18 +++++++++--------- 1 file changed, 9 insertions(+), 9 deletions(-) diff --git a/src/util/buf.c b/src/util/buf.c index e5a6ab7..e5406da 100644 --- a/src/util/buf.c +++ b/src/util/buf.c @@ -106,8 +106,8 @@ virBufferGetIndent(const virBufferPtr buf, bool dynamic) /** * virBufferGrow: - * @buf: the buffer - * @len: the minimum free size to allocate on top of existing used space + * @buf: the buffer + * @len: the minimum free size to allocate on top of existing used space * * Grow the available space of a buffer to at least @len bytes. * @@ -282,8 +282,8 @@ virBufferUse(const virBufferPtr buf) /** * virBufferAsprintf: * @buf: the buffer to append to - * @format: the format - * @...: the variable list of arguments + * @format: the format + * @...: the variable list of arguments * * Do a formatted print to an XML buffer. Auto indentation may be applied. */ @@ -299,8 +299,8 @@ virBufferAsprintf(virBufferPtr buf, const char *format, ...) /** * virBufferVasprintf: * @buf: the buffer to append to - * @format: the format - * @argptr: the variable list of arguments + * @format: the format + * @argptr: the variable list of arguments * * Do a formatted print to an XML buffer. Auto indentation may be applied. */ @@ -458,7 +458,7 @@ virBufferEscapeSexpr(virBufferPtr buf, virBufferEscape(buf, '\\', "\\'", format, str); } -/* Work around spurious strchr() diagnostics given by -Wlogical-op * +/* Work around spurious strchr() diagnostics given by -Wlogical-op * for gcc < 4.6. Doing it via a local pragma keeps the damage * smaller than disabling it on the package level. Unfortunately, the * affected GCCs don't allow diagnostic push/pop which would have @@ -522,7 +522,7 @@ virBufferEscape(virBufferPtr buf, char escape, const char *toescape, /** * virBufferURIEncodeString: * @buf: the buffer to append to - * @str: the string argument which will be URI-encoded + * @str: the string argument which will be URI-encoded * * Append the string to the buffer. The string will be URI-encoded * during the append (ie any non alpha-numeric characters are replaced @@ -630,7 +630,7 @@ virBufferEscapeShell(virBufferPtr buf, const char *str) /** * virBufferStrcat: * @buf: the buffer to append to - * @...: the variable list of strings, the last argument must be NULL + * @...: the variable list of strings, the last argument must be NULL * * Concatenate strings to an XML buffer. Auto indentation may be applied * after each string argument. -- 1.8.0.2

1 0

[libvirt] [PATCH] build: make broken -Wlogical-op test be gcc-only
by Eric Blake 20 Dec '12

20 Dec '12

Commit 8b8fcdea introduced a check for broken gcc -Wlogical-op, but did not guard the check against non-gcc compilers, which might lead to spurious failures when another compiler encounters an unknown pragma. Additionally, all of our compiler warning logic should belong in a single file, and use cache variables to allow overriding the decision at configure time if necessary. * configure.ac (BROKEN_GCC_WLOGICALOP): Move... * m4/virt-compile-warnings.m4 (LIBVIRT_COMPILE_WARNINGS): ...here, and update to modern autoconf idioms. --- Fixes the issues mentioned here: https://www.redhat.com/archives/libvir-list/2012-December/msg01197.html https://www.redhat.com/archives/libvir-list/2012-December/msg01211.html configure.ac | 23 ----------------------- m4/virt-compile-warnings.m4 | 22 ++++++++++++++++++++++ 2 files changed, 22 insertions(+), 23 deletions(-) diff --git a/configure.ac b/configure.ac index 20caa92..3c97e4f 100644 --- a/configure.ac +++ b/configure.ac @@ -255,29 +255,6 @@ AC_CHECK_TYPE([struct ifreq], #include <net/if.h> ]]) -dnl Check whether strchr(s, char variable) causes a bogus compile warning -dnl which is the case with a certain range of GCC versions -AC_MSG_CHECKING([whether GCC -Wlogical-op is broken]) - -save_CFLAGS="$CFLAGS" -CFLAGS="-O2 -Wlogical-op -Werror" - -AC_TRY_COMPILE([#include <string.h>], - [const char *haystack; - char needle; - return strchr(haystack, needle) == haystack;], - [gcc_false_strchr_warning=no], - [gcc_false_strchr_warning=yes]) - -CFLAGS="$save_CFLAGS" - -if test "x$gcc_false_strchr_warning" = xyes; then - AC_DEFINE_UNQUOTED([BROKEN_GCC_WLOGICALOP], 1, - [GCC -Wlogical-op is reporting false positive on strchr]) -fi - -AC_MSG_RESULT([$gcc_false_strchr_warning]) - dnl Our only use of libtasn1.h is in the testsuite, and can be skipped dnl if the header is not present. Assume -ltasn1 is present if the dnl header could be found. diff --git a/m4/virt-compile-warnings.m4 b/m4/virt-compile-warnings.m4 index 4a0f289..a08fcac 100644 --- a/m4/virt-compile-warnings.m4 +++ b/m4/virt-compile-warnings.m4 @@ -95,6 +95,22 @@ AC_DEFUN([LIBVIRT_COMPILE_WARNINGS],[ dontwarn="$dontwarn -Wmissing-declarations" fi + dnl Check whether strchr(s, char variable) causes a bogus compile + dnl warning, which is the case with GCC < 4.6 on some glibc + AC_CACHE_CHECK([whether GCC -Wlogical-op gives bogus warnings], + [lv_cv_gcc_wlogical_op_broken], [ + save_CFLAGS="$CFLAGS" + CFLAGS="-O2 -Wlogical-op -Werror" + AC_COMPILE_IFELSE([AC_LANG_PROGRAM([[ + #include <string.h> + ]], [[ + const char *haystack; + char needle; + return strchr(haystack, needle) == haystack;]])], + [lv_cv_gcc_wlogical_op_broken=no], + [lv_cv_gcc_wlogical_op_broken=yes]) + CFLAGS="$save_CFLAGS"]) + # We might fundamentally need some of these disabled forever, but # ideally we'd turn many of them on dontwarn="$dontwarn -Wfloat-equal" @@ -196,4 +212,10 @@ AC_DEFUN([LIBVIRT_COMPILE_WARNINGS],[ WARN_PYTHON_CFLAGS=$WARN_CFLAGS AC_SUBST(WARN_PYTHON_CFLAGS) WARN_CFLAGS=$save_WARN_CFLAGS + + if test "$gl_cv_warn_c__Wlogical_op" = yes && + test "$lv_cv_gcc_wlogical_op_broken" = yes; then + AC_DEFINE_UNQUOTED([BROKEN_GCC_WLOGICALOP], 1, + [Define to 1 if gcc -Wlogical-op reports false positives on strchr]) + fi ]) -- 1.8.0.2

2 2

[libvirt] [PATCH] build: use strchr now that we can work around broken gcc
by Eric Blake 20 Dec '12

20 Dec '12

Revert the complex workaround of commit 39d91e9, now that we have a nicer framework for shutting up broken gcc. * src/util/buf.c (virBufferEscape): Simplify. --- src/util/buf.c | 15 ++++++++++----- 1 file changed, 10 insertions(+), 5 deletions(-) diff --git a/src/util/buf.c b/src/util/buf.c index 030dc97..e5a6ab7 100644 --- a/src/util/buf.c +++ b/src/util/buf.c @@ -458,2 +458,2 @@ virBufferEscapeSexpr(virBufferPtr buf, virBufferEscape(buf, '\\', "\\'", format, str); } +/* Work around spurious strchr() diagnostics given by -Wlogical-op + * for gcc < 4.6. Doing it via a local pragma keeps the damage + * smaller than disabling it on the package level. Unfortunately, the + * affected GCCs don't allow diagnostic push/pop which would have + * further reduced the impact. */ +#if BROKEN_GCC_WLOGICALOP +# pragma GCC diagnostic ignored "-Wlogical-op" +#endif + /** * virBufferEscape: * @buf: the buffer to append to @@ -499,11 +508,7 @@ virBufferEscape(virBufferPtr buf, char escape, const char *toescape, cur = str; out = escaped; while (*cur != 0) { - /* strchr work-around for gcc 4.3 & 4.4 bug with -Wlogical-op - * http://gcc.gnu.org/bugzilla/show_bug.cgi?id=36513 - */ - char needle[2] = { *cur, 0 }; - if (strstr(toescape, needle)) + if (strchr(toescape, *cur)) *out++ = escape; *out++ = *cur; cur++; -- 1.8.0.2

2 1

[libvirt] [PATCH] build: Check for broken GCC -Wlogical-op was too optimistic
by Viktor Mihajlovski 20 Dec '12

20 Dec '12

The check for a broken GCC -Wlogical-op implemententation was written under the wrong assumption that unknown warning options would be gracefully ignored. Thanks to Eric Blake and Daniel Berrange for pointing that out. Signed-off-by: Viktor Mihajlovski <mihajlov(a)linux.vnet.ibm.com> --- configure.ac | 8 +++++++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/configure.ac b/configure.ac index 20caa92..828ae01 100644 --- a/configure.ac +++ b/configure.ac @@ -260,7 +260,13 @@ dnl which is the case with a certain range of GCC versions AC_MSG_CHECKING([whether GCC -Wlogical-op is broken]) save_CFLAGS="$CFLAGS" -CFLAGS="-O2 -Wlogical-op -Werror" +for w_opt in $WARN_CFLAGS; do + case $w_opt in + "-Wlogical-op" | "-Werror" ) + CFLAGS="$CFLAGS $w_opt" + ;; + esac +done AC_TRY_COMPILE([#include <string.h>], [const char *haystack; -- 1.7.9.5

2 1

[libvirt] virtulazation error
by Gangadahr Jena 20 Dec '12

20 Dec '12

hi every one when i am start windows os using virtual macine manager on redhat 6 os it showing this type of error-Error starting domain: unsupported configuration: Only PCI device addresses with function=0 are supported with this QEMU binary

2 1

[libvirt] [PATCH 0/2] FreeBSD: nodeinfo support and CPU affinity stubs
by Roman Bogorodskiy 19 Dec '12

19 Dec '12

With this set of changes it's now possible to connect to libvirtd using virsh and start/shutdown domains with the obvious limitation that networking is not supported at this point. CPU affinity most likely could be implemented using cpu_setaffinity(2) and cpu_getaffinity(2) on FreeBSD, but networking support looks more important now. Roman Bogorodskiy (2): FreeBSD: add nodeinfo support. FreeBSD: stub out CPU affinity functions. src/nodeinfo.c | 75 +++++++++++++++++++++++++++++++++++++++++++++++++- src/util/processinfo.c | 22 +++++++++++++++ 2 files changed, 96 insertions(+), 1 deletion(-) -- 1.8.0

3 16