[libvirt] [RFC v2 00/20] qmp: Report bus information on 'query-machines'

2016-11-25 Thread Eduardo Habkost
Changes v1 -> v2: * v1 series subject was: "qmp: Report supported device types on 'query-machines'" * Now we return additional bus information: bus ID, bus type, and the list of accepted device types on each bus * Now hotplug can be covered because accepted-device-types can be set by

[libvirt] [RFC v2 07/20] qmp: Add 'always-available-buses' field to 'query-machines'

2016-11-25 Thread Eduardo Habkost
The new field will return a list MachineBusInfo structs, containing information about the buses that are always created by the machine (even if -nodefaults is used). Note that some machine options may enable or disable some bus types and affect the set of available buses. Introspection of those

Re: [libvirt] QEMU soundcards vulnerable to jack retasking?

2016-11-25 Thread bancfc
On 2016-11-24 18:18, Daniel P. Berrange wrote: This is better sent to the QEMU mailing list, rather than libvirt, since the former is where the QEMU audio experts are... Regards, Daniel OK done. Good call, thanks Dan. -- libvir-list mailing list libvir-list@redhat.com

Re: [libvirt] Libvirt domain event usage and consistency

2016-11-25 Thread Michal Privoznik
On 25.11.2016 18:33, Roman Mohr wrote: > On Fri, Nov 25, 2016 at 6:04 PM, Michal Privoznik > wrote: > >> On 25.11.2016 17:54, Roman Mohr wrote: >>> On Fri, Nov 25, 2016 at 4:34 PM, Michal Privoznik >>> wrote: >>> On 25.11.2016 14:38, Roman Mohr

Re: [libvirt] [PATCH v2 13/31] qemu: Probe CPU models for KVM and TCG

2016-11-25 Thread Jiri Denemark
On Thu, Nov 24, 2016 at 19:10:18 +0100, Pavel Hrdina wrote: > On Mon, Nov 21, 2016 at 12:21:09AM +0100, Jiri Denemark wrote: > > CPU models (and especially some additional details which we will start > > probing for later) differ depending on the accelerator. Thus we need to > > call

Re: [libvirt] [PATCH v3] qemu: Make QMP probing process reusable

2016-11-25 Thread Jiri Denemark
On Fri, Nov 25, 2016 at 19:56:23 +0100, Jiri Denemark wrote: > The code that runs a new QEMU process to be used for probing > capabilities is separated into four reusable functions so that any code > that wants to probe a QEMU process may just follow a few simple steps: > > cmd =

[libvirt] [PATCH v3] qemu: Make QMP probing process reusable

2016-11-25 Thread Jiri Denemark
The code that runs a new QEMU process to be used for probing capabilities is separated into four reusable functions so that any code that wants to probe a QEMU process may just follow a few simple steps: cmd = virQEMUCapsInitQMPCommandNew(...); mon = virQEMUCapsInitQMPCommandRun(cmd,

Re: [libvirt] [PATCH v2 01/31] qemu: Make QMP probing process reusable

2016-11-25 Thread Jiri Denemark
On Thu, Nov 24, 2016 at 14:54:48 +0100, Pavel Hrdina wrote: > On Mon, Nov 21, 2016 at 12:20:57AM +0100, Jiri Denemark wrote: > > The code that runs a new QEMU process to be used for probing > > capabilities is separated into four reusable functions so that any code > > that wants to probe a QEMU

Re: [libvirt] [PATCH] qemu: capabilities: Don't partially reprope caps on process reconnect

2016-11-25 Thread Jiri Denemark
On Fri, Nov 25, 2016 at 17:14:48 +0100, Peter Krempa wrote: > Thanks to the complex capability caching code virQEMUCapsProbeQMP was > never called when we were starting a new qemu VM. On the other hand, > when we are reconnecting to the qemu process we reload the capability > list from the status

Re: [libvirt] Libvirt domain event usage and consistency

2016-11-25 Thread Roman Mohr
On Fri, Nov 25, 2016 at 6:04 PM, Michal Privoznik wrote: > On 25.11.2016 17:54, Roman Mohr wrote: > > On Fri, Nov 25, 2016 at 4:34 PM, Michal Privoznik > > wrote: > > > >> On 25.11.2016 14:38, Roman Mohr wrote: > >>> Hi, > >>> > >>> I recently started

Re: [libvirt] [PATCH] Pass GPG_TTY env var to the ssh binary

2016-11-25 Thread Guilhem Moulin
And I didn't test this carefully, my apologies :-( Whether gpg-agent can prompt the password depends on the pinentry program in use, but for pinentry-curses this also requires to pass TERM. Patch modified accordingly. From: Guilhem Moulin Subject: [PATCH] Pass GPG_TTY

Re: [libvirt] Libvirt domain event usage and consistency

2016-11-25 Thread Michal Privoznik
On 25.11.2016 17:54, Roman Mohr wrote: > On Fri, Nov 25, 2016 at 4:34 PM, Michal Privoznik > wrote: > >> On 25.11.2016 14:38, Roman Mohr wrote: >>> Hi, >>> >>> I recently started to use the libvirt domain events. With them I increase >>> the responsiveness of my VM state

Re: [libvirt] Libvirt domain event usage and consistency

2016-11-25 Thread Roman Mohr
On Fri, Nov 25, 2016 at 4:34 PM, Michal Privoznik wrote: > On 25.11.2016 14:38, Roman Mohr wrote: > > Hi, > > > > I recently started to use the libvirt domain events. With them I increase > > the responsiveness of my VM state wachers. > > In general it works pretty well. I

[libvirt] [PATCH] qemu: capabilities: Don't partially reprope caps on process reconnect

2016-11-25 Thread Peter Krempa
Thanks to the complex capability caching code virQEMUCapsProbeQMP was never called when we were starting a new qemu VM. On the other hand, when we are reconnecting to the qemu process we reload the capability list from the status XML file. This means that the flag preventing the function being

Re: [libvirt] [PATCH V2] qemu: Redefine the "unlimited" memory limits one more time

2016-11-25 Thread Daniel P. Berrange
On Fri, Nov 25, 2016 at 05:12:41PM +0100, Martin Kletzander wrote: > On Fri, Nov 25, 2016 at 03:39:17PM +, Daniel P. Berrange wrote: > > On Fri, Nov 25, 2016 at 04:35:14PM +0100, Martin Kletzander wrote: > > > On Fri, Nov 25, 2016 at 03:19:54PM +0100, Viktor Mihajlovski wrote: > > > > On

Re: [libvirt] [PATCH V2] qemu: Redefine the "unlimited" memory limits one more time

2016-11-25 Thread Martin Kletzander
On Fri, Nov 25, 2016 at 03:39:17PM +, Daniel P. Berrange wrote: On Fri, Nov 25, 2016 at 04:35:14PM +0100, Martin Kletzander wrote: On Fri, Nov 25, 2016 at 03:19:54PM +0100, Viktor Mihajlovski wrote: > On 18.11.2016 17:44, Viktor Mihajlovski wrote: > > With kernel 3.18 (since commit

Re: [libvirt] [PATCH] Revert "vz: fixed race in vzDomainAttach/DettachDevice"

2016-11-25 Thread Maxim Nestratov
22-Nov-16 15:05, Nikolay Shirokovskiy пишет: ACK Pushed now. Thanx. Maxim -- libvir-list mailing list libvir-list@redhat.com https://www.redhat.com/mailman/listinfo/libvir-list

Re: [libvirt] [REPOST PATCH v2 5/9] qemu: Adjust various bool BlockIoTune set_ values into mask

2016-11-25 Thread Erik Skultety
On Mon, Nov 21, 2016 at 06:35:50PM -0500, John Ferlan wrote: > Rather than have multiple bool values, create a single enum with bits > representing what can be set. > > Signed-off-by: John Ferlan > --- > src/qemu/qemu_driver.c | 113 >

Re: [libvirt] [PATCH V2] qemu: Redefine the "unlimited" memory limits one more time

2016-11-25 Thread Daniel P. Berrange
On Fri, Nov 25, 2016 at 04:35:14PM +0100, Martin Kletzander wrote: > On Fri, Nov 25, 2016 at 03:19:54PM +0100, Viktor Mihajlovski wrote: > > On 18.11.2016 17:44, Viktor Mihajlovski wrote: > > > With kernel 3.18 (since commit 3e32cb2e0a12b6915056ff04601cf1bb9b44f967) > > > the > > > "unlimited"

Re: [libvirt] [PATCH V2] qemu: Redefine the "unlimited" memory limits one more time

2016-11-25 Thread Martin Kletzander
On Fri, Nov 25, 2016 at 03:19:54PM +0100, Viktor Mihajlovski wrote: On 18.11.2016 17:44, Viktor Mihajlovski wrote: With kernel 3.18 (since commit 3e32cb2e0a12b6915056ff04601cf1bb9b44f967) the "unlimited" value for cgroup memory limits has changed once again as its byte value is now computed

Re: [libvirt] Libvirt domain event usage and consistency

2016-11-25 Thread Michal Privoznik
On 25.11.2016 14:38, Roman Mohr wrote: > Hi, > > I recently started to use the libvirt domain events. With them I increase > the responsiveness of my VM state wachers. > In general it works pretty well. I just listen to the events and do a > periodic resync to cope with missed events. > > While

Re: [libvirt] [REPOST PATCH v2 3/9] qemu: Adjust maxparams logic for qemuDomainGetBlockIoTune

2016-11-25 Thread Erik Skultety
On Mon, Nov 21, 2016 at 06:35:48PM -0500, John Ferlan wrote: > Rather than using negative logic and setting the maxparams to a lesser > value based on which capabilities exist, alter the logic to modify the > maxparams based on a base value plus the found capabilities. Reduces the > chance that

Re: [libvirt] [PATCH] Pass GPG_TTY env var to the ssh binary

2016-11-25 Thread Martin Kletzander
On Mon, Nov 14, 2016 at 11:13:22AM +0100, Guilhem Moulin wrote: Hi Daniel, On Mon, 14 Nov 2016 at 10:02:55 +, Daniel P. Berrange wrote: On Sat, Nov 12, 2016 at 02:19:37PM +0100, Guido Günther wrote: This came in via the Debian BTS: http://bugs.debian.org/43863 This seems to be the

Re: [libvirt] [PATCH 3/3] Add support for preallocated fd memory - doc

2016-11-25 Thread Martin Kletzander
On Tue, Nov 01, 2016 at 12:11:16PM +, Jaroslav Safka wrote: Add html documentation for memoryBacking element. We usually merge documentation with the conf changes. --- docs/formatdomain.html.in | 10 ++ 1 file changed, 10 insertions(+) diff --git a/docs/formatdomain.html.in

Re: [libvirt] [PATCH 2/3] Add support for preallocated fd memory

2016-11-25 Thread Martin Kletzander
On Tue, Nov 01, 2016 at 12:11:15PM +, Jaroslav Safka wrote: This second change introduces support for preallocated shared file descriptor based memory backing. It allows vhost-user to be used without hugepages. Also token memAccess in numa cell is used (if not present, default value from

Re: [libvirt] [PATCH] libvirt: qemu: Fix domain termination caused by query-hotpluggable-cpus not enabled

2016-11-25 Thread Peter Krempa
On Fri, Nov 25, 2016 at 14:25:33 +0100, Boris Fiuczynski wrote: > On 11/25/2016 10:07 AM, Peter Krempa wrote: > > On Fri, Nov 25, 2016 at 10:03:38 +0100, Peter Krempa wrote: > > > On Fri, Nov 25, 2016 at 09:19:18 +0100, Boris Fiuczynski wrote: > > > > [...] > > Peter, > looking at your commit

Re: [libvirt] [Qemu-devel] [PATCH v5 05/12] target-i386: Make plus_features/minus_features QOM-based

2016-11-25 Thread Markus Armbruster
Eduardo Habkost writes: > Instead of using custom feature name lookup code for > plus_features/minus_features, save the property names used in > "[+-]feature" and use object_property_set_bool() to set them. > > We don't need a feat2prop() call because we now have alias >

Re: [libvirt] [PATCH 1/3] Add support for preallocated fd memory

2016-11-25 Thread Martin Kletzander
On Tue, Nov 01, 2016 at 12:11:14PM +, Jaroslav Safka wrote: This first change introduces xml parsing support for preallocated shared file descriptor based memory backing. It allows vhost-user to be used without hugepages. New xml elements: --- docs/schemas/domaincommon.rng

Re: [libvirt] [PATCH] Pass GPG_TTY env var to the ssh binary

2016-11-25 Thread Guido Günther
Hi Daniel, On Mon, Nov 14, 2016 at 10:02:55AM +, Daniel P. Berrange wrote: > On Sat, Nov 12, 2016 at 02:19:37PM +0100, Guido Günther wrote: > > This came in via the Debian BTS: > > > > http://bugs.debian.org/43863 > > This seems to be the wrong bug number. I've updated the commit message

Re: [libvirt] [Qemu-ppc] [RFC PATCH qemu] spapr_pci: Create PCI-express root bus by default

2016-11-25 Thread Andrea Bolognani
On Wed, 2016-11-23 at 16:02 +1100, David Gibson wrote: > > > The change from OHCI to XHCI only affected the *default* USB > > > controller, which libvirt tries its best not to use anyway: > > > instead, it will prefer to use '-M ...,usb=off' along with > > > '-device ...' and set both the

Re: [libvirt] [PATCH] Qemu : Do not distinguish a 'hangup' from an 'eof'

2016-11-25 Thread Martin Kletzander
On Fri, Oct 28, 2016 at 12:33:28PM -0700, Prerna Saxena wrote: An errno=ECONNRESET received on a monitor socket reflects that the guest may have closed the socket. How would they close it? Does that happen when the process is dying? Today, we just mark it as a 'hangup' and do not trigger

Re: [libvirt] [PATCH V2] qemu: Redefine the "unlimited" memory limits one more time

2016-11-25 Thread Viktor Mihajlovski
On 18.11.2016 17:44, Viktor Mihajlovski wrote: > With kernel 3.18 (since commit 3e32cb2e0a12b6915056ff04601cf1bb9b44f967) the > "unlimited" value for cgroup memory limits has changed once again as its byte > value is now computed from a page counter. > The new "unlimited" value reported by the

[libvirt] [PATCH V3 1/2] util: Allow to query the presence of host CPU bitmaps

2016-11-25 Thread Viktor Mihajlovski
The functions to retrieve online and present host CPU information are only supported on Linux for the time being. This leads to runtime errors if these function are used on other platforms. To avoid that, code in higher levels using the functions must replicate the conditional compilation in

[libvirt] [PATCH V3 0/2] Allow hot plugged host CPUs

2016-11-25 Thread Viktor Mihajlovski
This is a rework of the original patch allowing to use all host CPUs if the cpuset controller is not configured by libvirt. The major enhancement is that we won't try to retrieve the online host CPU map on platforms where this is not supported and just fall back to the old behavior. V3: - Added

[libvirt] [PATCH V3 2/2] qemu: Allow use of hot plugged host CPUs if no affinity set

2016-11-25 Thread Viktor Mihajlovski
If the cpuset cgroup controller is disabled in /etc/libvirt/qemu.conf QEMU virtual machines can in principle use all host CPUs, even if they are hot plugged, if they have no explicit CPU affinity defined. However, there's libvirt code supposed to handle the situation where the libvirt daemon

Re: [libvirt] [Qemu-ppc] [RFC PATCH qemu] spapr_pci: Create PCI-express root bus by default

2016-11-25 Thread Andrea Bolognani
On Wed, 2016-11-23 at 16:00 +1100, David Gibson wrote: > > Existing libvirt versions assume that pseries guests have > > a legacy PCI root bus, and will base their PCI address > > allocation / PCI topology decisions on that fact: they > > will, for example, use legacy PCI bridges. >  > Um.. yeah..

[libvirt] Libvirt domain event usage and consistency

2016-11-25 Thread Roman Mohr
Hi, I recently started to use the libvirt domain events. With them I increase the responsiveness of my VM state wachers. In general it works pretty well. I just listen to the events and do a periodic resync to cope with missed events. While watching the events I ran into a few interesting

Re: [libvirt] [PATCH] libvirt: qemu: Fix domain termination caused by query-hotpluggable-cpus not enabled

2016-11-25 Thread Boris Fiuczynski
On 11/25/2016 10:07 AM, Peter Krempa wrote: On Fri, Nov 25, 2016 at 10:03:38 +0100, Peter Krempa wrote: On Fri, Nov 25, 2016 at 09:19:18 +0100, Boris Fiuczynski wrote: [...] src/qemu/qemu_process.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git

Re: [libvirt] [PATCH 1/1] reset vcpu pin info from config for zero mask

2016-11-25 Thread Martin Kletzander
On Wed, Oct 19, 2016 at 02:39:46PM +0300, Konstantin Neumoin wrote: The option for removing vcpu pinning information from config was added in: '7ea9778 vcpupin: add vcpupin resetting feature to qemu driver' By the way, this hash is ambiguous, 7ea9778c is better, but I rather use 7ea9778c8a3d

[libvirt] [PATCH v2 04/10] admin: Introduce virAdmConnectGetLoggingOutputs

2016-11-25 Thread Erik Skultety
Enable libvirt users to query logging output settings. Signed-off-by: Erik Skultety --- daemon/admin.c | 37 + include/libvirt/libvirt-admin.h | 4 src/admin/admin_protocol.x | 16 +++-

[libvirt] [PATCH v2 00/10] admin: Introduce runtime logging APIs

2016-11-25 Thread Erik Skultety
v1 here https://www.redhat.com/archives/libvir-list/2016-November/msg9.html since v1: - incorporated notes raised during review - allowed passing of NULL via the public APIs * behaves the same way as an empty string, but lead to even more code reduction in 'daemonSetupLogging' for

[libvirt] [PATCH v2 07/10] admin: Introduce virAdmConnectSetLoggingFilters

2016-11-25 Thread Erik Skultety
Enable libvirt users to modify logging filters of a daemon from outside. Signed-off-by: Erik Skultety --- daemon/admin.c | 10 ++ include/libvirt/libvirt-admin.h | 4 src/admin/admin_protocol.x | 12 +++-

[libvirt] [PATCH v2 02/10] admin: Allow passing NULL to virLogSetFilters

2016-11-25 Thread Erik Skultety
Along with an empty string, it should also be possible for users to pass NULL to the public APIs which in turn would trigger a routine (future work) responsible for defining an appropriate default logging output given the current circumstances. Signed-off-by: Erik Skultety

[libvirt] [PATCH v2 10/10] admin: Update the news file to include the new logging features

2016-11-25 Thread Erik Skultety
Signed-off-by: Erik Skultety --- docs/news.html.in | 4 1 file changed, 4 insertions(+) diff --git a/docs/news.html.in b/docs/news.html.in index 59bf20d..f2a65f5 100644 --- a/docs/news.html.in +++ b/docs/news.html.in @@ -31,6 +31,10 @@ Add the capability to

[libvirt] [PATCH v2 09/10] admin: Add an example demonstrating how to use the logging APIs

2016-11-25 Thread Erik Skultety
Provide a simple C example demonstrating the use of both query APIs as well as setter APIs. Signed-off-by: Erik Skultety --- .gitignore | 1 + examples/Makefile.am | 3 +- examples/admin/logging.c | 102 +++

[libvirt] [PATCH v2 01/10] virlog: Introduce virLog{Get, Set}DefaultOutput

2016-11-25 Thread Erik Skultety
These helpers will manage the log destination defaults (fetch/set). The reason for this is to stay consistent with the current daemon's behaviour with respect to /etc/libvirt/.conf file, since both assignment of an empty string or not setting the log output variable at all trigger the daemon's

[libvirt] [PATCH v2 06/10] admin: Introduce virAdmConnectSetLoggingOutputs

2016-11-25 Thread Erik Skultety
Enable libvirt users to modify daemon's logging output settings from outside. If either an empty string or NULL is passed, a default logging output will be used the same way as it would be in case writing an empty string to the libvirtd.conf Signed-off-by: Erik Skultety ---

[libvirt] [PATCH v2 05/10] admin: Introduce virAdmConnectGetLoggingFilters

2016-11-25 Thread Erik Skultety
Enable libvirt users to query logging filter settings. Signed-off-by: Erik Skultety --- daemon/admin.c | 47 + include/libvirt/libvirt-admin.h | 4 src/admin/admin_protocol.x | 16 +-

[libvirt] [PATCH v2 08/10] virt-admin: Wire-up the logging APIs

2016-11-25 Thread Erik Skultety
Finally, now that all APIs have been introduced, wire them up to virt-admin and introduce daemon-log-outputs and daemon-log-filters commands. Signed-off-by: Erik Skultety --- tools/virt-admin.c | 120 +++

[libvirt] [PATCH v2 03/10] daemon: Hook up the virLog{Get, Set}DefaultOutput to the daemon's init routine

2016-11-25 Thread Erik Skultety
Now that virLog{Get,Set}DefaultOutput routines are introduced we can wire them up to the daemon's logging initialization code. Also, change the order of operations a bit so that we still strictly honor our precedence of settings: cmdline > env > config now that outputs and filters are not appended

Re: [libvirt] [PATCH 2/2] docs: news: Mention changes in memory slot number allocation

2016-11-25 Thread Andrea Bolognani
On Fri, 2016-11-25 at 13:44 +0100, Peter Krempa wrote: > > You'll also need to close the element properly, or > > building the documentation will fail. >  > Right. The format of the file, where we wrap the text back to the column > where the tag starts rather than wrapping it to the column where

Re: [libvirt] [PATCH v4 0/9] Implementation of QEMU vhost-scsi

2016-11-25 Thread Andrea Bolognani
On Fri, 2016-11-25 at 07:30 -0500, John Ferlan wrote: > > > This patch series provides a libvirt implementation of the vhost-scsi > > > interface in QEMU. > > > > Can you please provide a release notes entry briefly > > describing this change? You can look at existing entries > > in

Re: [libvirt] [PATCH 2/2] docs: news: Mention changes in memory slot number allocation

2016-11-25 Thread Peter Krempa
On Fri, Nov 25, 2016 at 13:36:01 +0100, Andrea Bolognani wrote: > On Fri, 2016-11-25 at 12:14 +0100, Peter Krempa wrote: > > --- > >  docs/news.html.in | 4 > >  1 file changed, 4 insertions(+) > >  > > diff --git a/docs/news.html.in b/docs/news.html.in > > index 0a4b767..2ba25f5 100644 > >

Re: [libvirt] [PATCH 2/2] docs: news: Mention changes in memory slot number allocation

2016-11-25 Thread Andrea Bolognani
On Fri, 2016-11-25 at 12:14 +0100, Peter Krempa wrote: > --- >  docs/news.html.in | 4 >  1 file changed, 4 insertions(+) >  > diff --git a/docs/news.html.in b/docs/news.html.in > index 0a4b767..2ba25f5 100644 > --- a/docs/news.html.in > +++ b/docs/news.html.in > @@ -34,6 +34,10 @@ >   

Re: [libvirt] [PATCH 1/2] docs: NEWS: Mention 'gluster_debug_level' qemu.conf option in the news

2016-11-25 Thread Andrea Bolognani
On Fri, 2016-11-25 at 12:14 +0100, Peter Krempa wrote: > --- > docs/news.html.in | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git a/docs/news.html.in b/docs/news.html.in > index 54eb8ad..0a4b767 100644 > --- a/docs/news.html.in > +++ b/docs/news.html.in > @@ -31,6 +31,9 @@ >

Re: [libvirt] [PATCH v4 0/9] Implementation of QEMU vhost-scsi

2016-11-25 Thread John Ferlan
On 11/25/2016 05:50 AM, Andrea Bolognani wrote: > On Mon, 2016-11-21 at 22:58 -0500, Eric Farman wrote: >> This patch series provides a libvirt implementation of the vhost-scsi >> interface in QEMU. > > Can you please provide a release notes entry briefly > describing this change? You can look

Re: [libvirt] virsh domjobinfo during storage migration

2016-11-25 Thread Ruben Kerkhof
On Fri, Nov 25, 2016 at 11:53 AM, Jiri Denemark wrote: >> How do you want to handle this, would you like me to open an issue in >> bugzilla? > > Yes please. Unless you want to provide the patches :-) I'd love to, but it is a bit outside of my area of expertise. I opened

[libvirt] [PATCH 0/2] NEWS updates for my recent changes

2016-11-25 Thread Peter Krempa
Peter Krempa (2): docs: NEWS: Mention 'gluster_debug_level' qemu.conf option in the news docs: news: Mention changes in memory slot number allocation docs/news.html.in | 7 +++ 1 file changed, 7 insertions(+) -- 2.10.2 -- libvir-list mailing list libvir-list@redhat.com

[libvirt] [PATCH 2/2] docs: news: Mention changes in memory slot number allocation

2016-11-25 Thread Peter Krempa
--- docs/news.html.in | 4 1 file changed, 4 insertions(+) diff --git a/docs/news.html.in b/docs/news.html.in index 0a4b767..2ba25f5 100644 --- a/docs/news.html.in +++ b/docs/news.html.in @@ -34,6 +34,10 @@ qemu: Users can now enable debug logging for native gluster

[libvirt] [PATCH 1/2] docs: NEWS: Mention 'gluster_debug_level' qemu.conf option in the news

2016-11-25 Thread Peter Krempa
--- docs/news.html.in | 3 +++ 1 file changed, 3 insertions(+) diff --git a/docs/news.html.in b/docs/news.html.in index 54eb8ad..0a4b767 100644 --- a/docs/news.html.in +++ b/docs/news.html.in @@ -31,6 +31,9 @@ Add the capability to pass through a scsi_host HBA and the

Re: [libvirt] [PATCH] virstring: Unify string list function names

2016-11-25 Thread Andrea Bolognani
On Fri, 2016-11-25 at 09:28 +0100, Michal Privoznik wrote: > We have couple of functions that operate over NULL terminated > lits of strings. However, our naming sucks: >  > virStringJoin > virStringFreeList > virStringFreeListCount > virStringArrayHasString > virStringGetFirstWithPrefix >  > We

Re: [libvirt] virsh domjobinfo during storage migration

2016-11-25 Thread Jiri Denemark
On Fri, Nov 25, 2016 at 11:38:36 +0100, Ruben Kerkhof wrote: > Hi Jiri, > > On Fri, Nov 25, 2016 at 9:26 AM, Jiri Denemark wrote: > > I guess it should be possible to also check the progress of all running > > block jobs so that we can report statistics about ongoing storage

[libvirt] [PATCH 3/3] docs: Update news

2016-11-25 Thread Jiri Denemark
Signed-off-by: Jiri Denemark --- docs/news.html.in | 2 ++ 1 file changed, 2 insertions(+) diff --git a/docs/news.html.in b/docs/news.html.in index 59bf20da3..3a0fea65e 100644 --- a/docs/news.html.in +++ b/docs/news.html.in @@ -35,6 +35,8 @@ Improvements

[libvirt] [PATCH 2/3] Consolidate documentation of virDomainMigrate{, ToURI}{, 2, 3}

2016-11-25 Thread Jiri Denemark
Only the latest APIs are fully documented and the documentation of the older variants (which are just limited versions of the new APIs anyway) points to the newest APIs. Signed-off-by: Jiri Denemark --- src/libvirt-domain.c | 348

[libvirt] [PATCH 0/3] Better documentation of migration APIs and flags

2016-11-25 Thread Jiri Denemark
We have 6 public APIs for migration: virDomainMigrate, virDomainMigrate2, virDomainMigrate3, virDomainMigrateToURI, virDomainMigrateToURI2, and virDomainMigrateToURI3. Each of them was documented separately, but the documentation was not consistent. Some APIs documented individual flags, some

[libvirt] [PATCH 1/3] Enhance documentation of virDomainMigrateFlags

2016-11-25 Thread Jiri Denemark
The enhanced documentation of VIR_MIGRATE_RDMA_PIN_ALL fixes https://bugzilla.redhat.com/show_bug.cgi?id=1373783 Signed-off-by: Jiri Denemark --- include/libvirt/libvirt-domain.h | 153 +-- 1 file changed, 132 insertions(+), 21

Re: [libvirt] [PATCH v4 0/9] Implementation of QEMU vhost-scsi

2016-11-25 Thread Andrea Bolognani
On Mon, 2016-11-21 at 22:58 -0500, Eric Farman wrote: > This patch series provides a libvirt implementation of the vhost-scsi > interface in QEMU. Can you please provide a release notes entry briefly describing this change? You can look at existing entries in docs/news.html.in to see how it

Re: [libvirt] virsh domjobinfo during storage migration

2016-11-25 Thread Ruben Kerkhof
Hi Jiri, On Fri, Nov 25, 2016 at 9:26 AM, Jiri Denemark wrote: > On Thu, Nov 24, 2016 at 22:54:10 +0100, Ruben Kerkhof wrote: >> Hi all, >> >> Running virsh domjobinfo on my CentOS 6 systems during a migration >> with --copy-storage-all, the output used to look like this: >>

Re: [libvirt] [PATCH v2 0/4] qemu: assign vfio devices to PCIe addresses when appropriate

2016-11-25 Thread Andrea Bolognani
On Mon, 2016-11-21 at 00:01 -0500, Laine Stump wrote: > Patches 1 and 4 were originally added to the end of the "more PCIe > less legacy PCI" patchset in its final incarnation, but all the other > patches were ACKed and pushed, while this needed a bit more work, > resulting in this "faux V2" -

[libvirt] [PATCH] NEWS: Add some missing entries

2016-11-25 Thread Andrea Bolognani
Catch up with changes that have been pushed but didn't include updates to the NEWS file themselves. --- Pushed under the "this NEWS file thing will need some time to catch on" rule. docs/news.html.in | 7 +++ 1 file changed, 7 insertions(+) diff --git a/docs/news.html.in b/docs/news.html.in

Re: [libvirt] [PATCH] virhostcpu: fix calculation of total CPU sockets

2016-11-25 Thread Daniel P. Berrange
On Fri, Nov 25, 2016 at 04:20:18PM +0800, Zhang Zhuoyu wrote: > CPU sockets calculation is inconsistent with physical sockets when > Host machine has more than one node. It only calculate the maximum > socket number of all CPU nodes instead of summing up. > > For example: > > Architecture:

[libvirt] [PATCH v5 1/2] conf: List only online cpus for virsh vcpupin

2016-11-25 Thread Nitesh Konkar
Currently when the vcpu placement is static and cpuset is not specified, CPU Affinity under virsh vcpupin shows 0..CPUMAX. This patchset will result in display of only online CPU's under CPU Affinity on linux. Signed-off-by: Nitesh Konkar --- src/conf/domain_conf.c

[libvirt] [PATCH v5 0/2] List only online cpus for vcpupin/emulatorpin when vcpu placement static

2016-11-25 Thread Nitesh Konkar
Currently when the vcpu placement is static and cpuset is not specified, CPU Affinity shows 0.. CPUMAX. This patchset will result in display of only online CPU's under CPU Affinity on linux. Fixes the following Bug: virsh dumpxml Fedora Fedora aecf3e5e-6f9a-42a3-9d6a-223a75569a66 3145728

[libvirt] [PATCH v5 2/2] conf: List only online cpus for virsh emulatorpin

2016-11-25 Thread Nitesh Konkar
Currently when the vcpu placement is static and cpuset is not specified, CPU Affinity under virsh emulatorpin shows 0..CPUMAX. This patchset will result in display of only online CPU's under CPU Affinity on linux. Signed-off-by: Nitesh Konkar ---

Re: [libvirt] [PATCH] libvirt: qemu: Fix domain termination caused by query-hotpluggable-cpus not enabled

2016-11-25 Thread Peter Krempa
On Fri, Nov 25, 2016 at 10:03:38 +0100, Peter Krempa wrote: > On Fri, Nov 25, 2016 at 09:19:18 +0100, Boris Fiuczynski wrote: [...] > > src/qemu/qemu_process.c | 3 +-- > > 1 file changed, 1 insertion(+), 2 deletions(-) > > > > diff --git a/src/qemu/qemu_process.c b/src/qemu/qemu_process.c > >

Re: [libvirt] [PATCH] libvirt: qemu: Fix domain termination caused by query-hotpluggable-cpus not enabled

2016-11-25 Thread Peter Krempa
On Fri, Nov 25, 2016 at 09:19:18 +0100, Boris Fiuczynski wrote: > (Re-)Starting libvirt on a system with running qemu domains which earlier > had been successfully started by libvirt results in the error > > internal error: unable to execute QEMU command 'query-hotpluggable-cpus': > The feature

[libvirt] [PATCH] cpu: Add support for pku and ospke Intel features for Memory Protection Keys

2016-11-25 Thread Lin Ma
qemu commit: f74eefe0 https://lwn.net/Articles/667156/ Signed-off-by: Lin Ma --- src/cpu/cpu_map.xml | 7 +++ 1 file changed, 7 insertions(+) diff --git a/src/cpu/cpu_map.xml b/src/cpu/cpu_map.xml index 6da8321..dca5720 100644 --- a/src/cpu/cpu_map.xml +++

Re: [libvirt] [PATCH 1/2] qemu: fix internal error: NUMA isn't available on this host

2016-11-25 Thread Martin Kletzander
On Thu, Nov 24, 2016 at 01:28:00PM +0100, Boris Fiuczynski wrote: If libvirt is compiled without NUMACTL support starting libvirtd reports a libvirt internal error "NUMA isn't available on this host" without checking if NUMA support is compiled into the libvirt binaries. This patch adds the

Re: [libvirt] [PATCH 1/2] qemu: Create hugepage path on per domain basis

2016-11-25 Thread Martin Kletzander
On Tue, Nov 22, 2016 at 01:45:42PM +0100, Michal Privoznik wrote: If you've ever tried running a huge page backed guest under different user than root, you probably failed. Problem is even Surely you mean different than the default user from qemu.conf. though we have corresponding APIs in

[libvirt] [PATCH] virstring: Unify string list function names

2016-11-25 Thread Michal Privoznik
We have couple of functions that operate over NULL terminated lits of strings. However, our naming sucks: virStringJoin virStringFreeList virStringFreeListCount virStringArrayHasString virStringGetFirstWithPrefix We can do better: virStringListJoin virStringListFree virStringListFreeCount

Re: [libvirt] virsh domjobinfo during storage migration

2016-11-25 Thread Jiri Denemark
On Thu, Nov 24, 2016 at 22:54:10 +0100, Ruben Kerkhof wrote: > Hi all, > > Running virsh domjobinfo on my CentOS 6 systems during a migration > with --copy-storage-all, the output used to look like this: > > Job type: Unbounded > Time elapsed: 1830632 ms > Data processed:

[libvirt] [PATCH] libvirt: qemu: Fix domain termination caused by query-hotpluggable-cpus not enabled

2016-11-25 Thread Boris Fiuczynski
(Re-)Starting libvirt on a system with running qemu domains which earlier had been successfully started by libvirt results in the error internal error: unable to execute QEMU command 'query-hotpluggable-cpus': The feature 'query-hotpluggable-cpus' is not enabled if the qemu binary does not

Re: [libvirt] [PATCH 2/2] security: Implement virSecurityManagerSetHugepages

2016-11-25 Thread Martin Kletzander
On Tue, Nov 22, 2016 at 01:45:43PM +0100, Michal Privoznik wrote: Since its introduction in 2012 this internal API did nothing. Now it's finally the right time to put it into a good use. It's implementation is fairly simple and exactly the same as virSecurityDomainSetPathLabel. Then why do

Re: [libvirt] [Qemu-devel] [RFC 00/15] qmp: Report supported device types on 'query-machines'

2016-11-25 Thread Markus Armbruster
Marcel Apfelbaum writes: > On 11/24/2016 05:41 PM, Markus Armbruster wrote: >> Marcel Apfelbaum writes: >> >>> On 11/24/2016 03:34 PM, Markus Armbruster wrote: Eduardo Habkost writes: > On Wed, Nov 23, 2016 at 06:43:16PM