Re: [libvirt] [PATCH v2 3/3] test: Need to relock afer virDomainObjListRemove

2018-04-03 Thread Marc Hartmayer
On Mon, Apr 02, 2018 at 02:27 PM +0200, John Ferlan wrote: > For all @dom's fetched from a testDomObjFromDomain because > virDomainObjListRemove will return an unlocked domain object > we should relock it prior to the cleanup label which will use > virDomainObjEndAPI which would Unlock and Unref t

Re: [libvirt] [PATCH v2 2/3] test: Use virDomainObjListFindByIDRef

2018-04-03 Thread Marc Hartmayer
On Mon, Apr 02, 2018 at 02:27 PM +0200, John Ferlan wrote: > Rather than using virDomainObjListFindByID, let's be more consistent > and return a reffed and locked object. Since we're using the Ref API, > use virDomainObjEndAPI on @dom and not just virObjectUnlock. > > Signed-off-by: John Ferlan >

Re: [libvirt] [PATCH v2 1/3] test: Use virDomainObjListFindByUUIDRef

2018-04-03 Thread Marc Hartmayer
On Mon, Apr 02, 2018 at 02:27 PM +0200, John Ferlan wrote: > Rather than using virDomainObjListFindByUUID, let's be more consistent > and return a reffed and locked object. Since we're using the Ref API, > use virDomainObjEndAPI on @dom and not just virObjectUnlock. > > Signed-off-by: John Ferlan

Re: [libvirt] [PATCH 03/11] conf: Remove dubious code from virDomainPCIAddressSetGrow()

2018-04-03 Thread Laine Stump
On 04/03/2018 07:12 PM, John Ferlan wrote: > > On 03/28/2018 10:06 AM, Andrea Bolognani wrote: >> I haven't been able to come up with a single scenario in which >> the code in question would be executed; even if there was one, >> it would be due to the user specifying a *partial* PCI topology >> in

Re: [libvirt] [PATCH 11/11] news: Update for pcie-to-pci-bridge support

2018-04-03 Thread John Ferlan
On 03/28/2018 10:06 AM, Andrea Bolognani wrote: > Signed-off-by: Andrea Bolognani > --- > docs/news.xml | 9 + > 1 file changed, 9 insertions(+) > Just make sure there isn't any wierd placement by merges or loss of patch 10 before this series finally gets pushed... Reviewed-by: John

Re: [libvirt] [PATCH 10/11] news: Add 4.3.0 release

2018-04-03 Thread John Ferlan
On 03/28/2018 10:06 AM, Andrea Bolognani wrote: > Signed-off-by: Andrea Bolognani > --- > docs/news.xml | 8 > 1 file changed, 8 insertions(+) > Unnecessary now... John -- libvir-list mailing list libvir-list@redhat.com https://www.redhat.com/mailman/listinfo/libvir-list

Re: [libvirt] [PATCH 09/11] tests: Use pcie-to-pci-bridge for aarch64-traditional-pci

2018-04-03 Thread John Ferlan
On 03/28/2018 10:06 AM, Andrea Bolognani wrote: > Now that support for the pcie-to-pci-bridge controller has > been implemented, adding the QEMU_CAPS_DEVICE_PCIE_PCI_BRIDGE > capability to the existing test is enough to cause the guest > to use pcie-to-pci-bridge instead of dmi-to-pci-bridge. >

Re: [libvirt] [PATCH 05/11] qemu: Add QEMU_CAPS_DEVICE_PCIE_PCI_BRIDGE

2018-04-03 Thread John Ferlan
On 03/28/2018 10:06 AM, Andrea Bolognani wrote: > This capability will be set when the pcie-pci-bridge device > is available in the QEMU binary. > > Signed-off-by: Andrea Bolognani > --- > src/qemu/qemu_capabilities.c | 2 ++ > src/qemu/qemu_capabilities.h

Re: [libvirt] [PATCH 06/11] qemu: Implement pcie-to-pci-bridge controller

2018-04-03 Thread John Ferlan
On 03/28/2018 10:06 AM, Andrea Bolognani wrote: > The new controller will not yet be used automatically by > libvirt, but at this point it's already possible to configure > a guest to use it. > > Signed-off-by: Andrea Bolognani > --- > docs/formatdomain.html.in | 3 ++- > docs/schemas/do

Re: [libvirt] [PATCH 08/11] conf: Prefer pcie-to-pci-bridge to dmi-to-pci-bridge

2018-04-03 Thread John Ferlan
On 03/28/2018 10:06 AM, Andrea Bolognani wrote: > Both pcie-to-pci-bridge and dmi-to-pci-bridge can be used to > create a traditional PCI topology in a pure PCIe guest such as > those using the x86_64/q35 or aarch64/virt machine type; > however, the former should be preferred, as it doesn't need

Re: [libvirt] [PATCH 07/11] conf: Add virDomainPCIAddressSet.isPCIeToPCIBridgeSupported

2018-04-03 Thread John Ferlan
On 03/28/2018 10:06 AM, Andrea Bolognani wrote: > Just like the existing areMultipleRootsSupported, this will > allow us to change the results of the driver-agnostic PCI > address allocation logic based on whether the QEMU binary > supports certain features. > > Signed-off-by: Andrea Bolognani

Re: [libvirt] [PATCH 04/11] conf: Rename virDomainPCIAddressSet.areMultipleRootsSupported

2018-04-03 Thread John Ferlan
On 03/28/2018 10:06 AM, Andrea Bolognani wrote: > We're going to add a similarly-named attribute later, and we'd > like to be consistent. > > Signed-off-by: Andrea Bolognani > --- > src/conf/domain_addr.c | 2 +- > src/conf/domain_addr.h | 2 +- > src/qemu/qemu_domain_address.c

Re: [libvirt] [PATCH 03/11] conf: Remove dubious code from virDomainPCIAddressSetGrow()

2018-04-03 Thread John Ferlan
On 03/28/2018 10:06 AM, Andrea Bolognani wrote: > I haven't been able to come up with a single scenario in which > the code in question would be executed; even if there was one, > it would be due to the user specifying a *partial* PCI topology > in the guest XML, which is of course entirely unsup

Re: [libvirt] [PATCH 02/11] tests: Add aarch64-traditional-pci test

2018-04-03 Thread John Ferlan
On 03/28/2018 10:06 AM, Andrea Bolognani wrote: > This test shows what happens when you add a traditional PCI > device such as pci-serial to a pure PCIe machine type such > as aarch64/virt. > > Signed-off-by: Andrea Bolognani > --- > .../qemuxml2argvdata/aarch64-traditional-pci.args | 26

Re: [libvirt] [PATCH 01/11] docs: Tweak PCI controller model documentation

2018-04-03 Thread John Ferlan
On 03/28/2018 10:06 AM, Andrea Bolognani wrote: > Instead of first listing the models on their own, and then > listing them again grouped by the libvirt release they were > introduced in, have a single list. > > Signed-off-by: Andrea Bolognani > --- > docs/formatdomain.html.in | 20 ---

Re: [libvirt] [libvirt PATCH] qemu: Add support for specifying SPICE TLS ciphers

2018-04-03 Thread Jiri Denemark
On Tue, Apr 03, 2018 at 17:23:50 +0200, Ján Tomko wrote: > From: Christophe Fergeau > > This commit adds a 'spice_tls_ciphers' parameter in > qemu.conf which allows to configure which TLS ciphers > SPICE will be using for its TLS connections. > > https://bugzilla.redhat.com/show_bug.cgi?id=15620

Re: [libvirt] [PATCH] docs: add page describing goals for host platform version support

2018-04-03 Thread Jiri Denemark
On Tue, Apr 03, 2018 at 15:56:11 +0100, Daniel P. Berrangé wrote: > Describe how we decide which host platforms to support for libvirt, > which in turn makes it easier to decide when a platform / software > version can be dropped. > > Signed-off-by: Daniel P. Berrangé > --- > docs/index.html.in

Re: [libvirt] [PATCH] docs: add page describing goals for host platform version support

2018-04-03 Thread Daniel P . Berrangé
On Tue, Apr 03, 2018 at 06:35:06PM +0200, Pino Toscano wrote: > On Tuesday, 3 April 2018 18:28:46 CEST John Ferlan wrote: > > Could we add some sort of table where we could keep track of "known" and > > supported OS versions, what libvirt version was used, what QEMU version > > was used? > > Do yo

Re: [libvirt] [PATCH] docs: add page describing goals for host platform version support

2018-04-03 Thread Daniel P . Berrangé
On Tue, Apr 03, 2018 at 12:28:46PM -0400, John Ferlan wrote: > > > On 04/03/2018 10:56 AM, Daniel P. Berrangé wrote: > > Describe how we decide which host platforms to support for libvirt, > > which in turn makes it easier to decide when a platform / software > > version can be dropped. > > > >

Re: [libvirt] [PATCH] docs: add page describing goals for host platform version support

2018-04-03 Thread Pino Toscano
On Tuesday, 3 April 2018 18:28:46 CEST John Ferlan wrote: > Could we add some sort of table where we could keep track of "known" and > supported OS versions, what libvirt version was used, what QEMU version > was used? Do you mean like what repology does? https://repology.org/metapackage/qemu/ver

Re: [libvirt] [PATCH] docs: add page describing goals for host platform version support

2018-04-03 Thread John Ferlan
On 04/03/2018 10:56 AM, Daniel P. Berrangé wrote: > Describe how we decide which host platforms to support for libvirt, > which in turn makes it easier to decide when a platform / software > version can be dropped. > > Signed-off-by: Daniel P. Berrangé > --- > docs/index.html.in | 2 +- >

Re: [libvirt] [PATCH] docs: add page describing goals for host platform version support

2018-04-03 Thread Andrea Bolognani
On Tue, 2018-04-03 at 15:56 +0100, Daniel P. Berrangé wrote: > Describe how we decide which host platforms to support for libvirt, > which in turn makes it easier to decide when a platform / software > version can be dropped. > > Signed-off-by: Daniel P. Berrangé > --- > docs/index.html.in |

Re: [libvirt] [PATCH 00/44] Require QEMU 1.3.0 or newer

2018-04-03 Thread John Ferlan
[...] >> I understand QEMU is just but one variable in the complex equation. >> Listing QEMU was just an example for this series based on the 8 years >> since 0.12 which is the minimum we support now vs. 5 years since 1.3 >> which is where this series was headed. Using the 2-3 years you responded

Re: [libvirt] [PATCH 2/2] qemu: deny privilege elevation and spawn in seccomp

2018-04-03 Thread Daniel P . Berrangé
On Sat, Mar 31, 2018 at 10:15:03PM +0200, Ján Tomko wrote: > If QEMU uses a seccomp blacklist (since 2.11), -sandbox on > no longer tries to whitelist all the calls, but uses sets > of blacklists: > default (always blacklisted with -sandbox on) > obsolete (defaults to deny) Perhaps list that anywa

Re: [libvirt] [PATCH 00/44] Require QEMU 1.3.0 or newer

2018-04-03 Thread Daniel P . Berrangé
On Tue, Apr 03, 2018 at 05:29:52PM +0200, Andrea Bolognani wrote: > On Tue, 2018-04-03 at 15:59 +0100, Daniel P. Berrangé wrote: > > On Tue, Apr 03, 2018 at 04:43:41PM +0200, Andrea Bolognani wrote: > > > Most of the compelling features introduced by any libvirt release > > > require the correspond

Re: [libvirt] [PATCH 00/44] Require QEMU 1.3.0 or newer

2018-04-03 Thread Andrea Bolognani
On Tue, 2018-04-03 at 15:59 +0100, Daniel P. Berrangé wrote: > On Tue, Apr 03, 2018 at 04:43:41PM +0200, Andrea Bolognani wrote: > > Most of the compelling features introduced by any libvirt release > > require the corresponding QEMU feature to be available. And, as > > I've argued elsewhere in the

[libvirt] [libvirt PATCH] qemu: Add support for specifying SPICE TLS ciphers

2018-04-03 Thread Ján Tomko
From: Christophe Fergeau This commit adds a 'spice_tls_ciphers' parameter in qemu.conf which allows to configure which TLS ciphers SPICE will be using for its TLS connections. https://bugzilla.redhat.com/show_bug.cgi?id=1562032 Signed-off-by: Christophe Fergeau Signed-off-by: Ján Tomko --- Th

Re: [libvirt] [PATCH] docs: add page describing goals for host platform version support

2018-04-03 Thread Pavel Hrdina
On Tue, Apr 03, 2018 at 03:56:11PM +0100, Daniel P. Berrangé wrote: > Describe how we decide which host platforms to support for libvirt, > which in turn makes it easier to decide when a platform / software > version can be dropped. > > Signed-off-by: Daniel P. Berrangé > --- > docs/index.html.i

Re: [libvirt] [PATCH 00/44] Require QEMU 1.3.0 or newer

2018-04-03 Thread Daniel P . Berrangé
On Tue, Apr 03, 2018 at 10:57:44AM -0400, John Ferlan wrote: > > > On 04/03/2018 10:04 AM, Daniel P. Berrangé wrote: > > On Tue, Apr 03, 2018 at 09:29:01AM -0400, John Ferlan wrote: > >> Looking at git history and considering adjustments to the VERSION file > >> in the QEMU tree, I get: > > > >

Re: [libvirt] [PATCH 00/44] Require QEMU 1.3.0 or newer

2018-04-03 Thread Daniel P . Berrangé
On Tue, Apr 03, 2018 at 04:43:41PM +0200, Andrea Bolognani wrote: > On Tue, 2018-04-03 at 15:00 +0100, Daniel P. Berrangé wrote: > > On Tue, Apr 03, 2018 at 01:23:15PM +0200, Andrea Bolognani wrote: > > > Bumping our minimum QEMU version wouldn't affect that kind of > > > scenario, as long as we ke

Re: [libvirt] [PATCH 00/44] Require QEMU 1.3.0 or newer

2018-04-03 Thread John Ferlan
On 04/03/2018 10:04 AM, Daniel P. Berrangé wrote: > On Tue, Apr 03, 2018 at 09:29:01AM -0400, John Ferlan wrote: >> >> >> On 04/03/2018 07:23 AM, Andrea Bolognani wrote: >>> On Tue, 2018-04-03 at 12:57 +0200, Peter Krempa wrote: On Tue, Apr 03, 2018 at 11:45:19 +0100, Daniel Berrange wrote:

[libvirt] [PATCH] docs: add page describing goals for host platform version support

2018-04-03 Thread Daniel P . Berrangé
Describe how we decide which host platforms to support for libvirt, which in turn makes it easier to decide when a platform / software version can be dropped. Signed-off-by: Daniel P. Berrangé --- docs/index.html.in | 2 +- docs/platforms.html.in | 74 +++

Re: [libvirt] [PATCH v5 09/10] virsh: implement new command for launch security

2018-04-03 Thread Erik Skultety
> > +}, > > +{.name = NULL} > > +}; > > + > > +static const vshCmdOptDef opts_launch_security[] = { > > +VIRSH_COMMON_OPT_DOMAIN_FULL(0), > > +{.name = "get", > > + .type = VSH_OT_STRING, > > + .help = N_("Show the launch-security info") > > +}, > > +VIRSH_COMMON_OPT

Re: [libvirt] [PATCH 00/44] Require QEMU 1.3.0 or newer

2018-04-03 Thread Andrea Bolognani
On Tue, 2018-04-03 at 15:00 +0100, Daniel P. Berrangé wrote: > On Tue, Apr 03, 2018 at 01:23:15PM +0200, Andrea Bolognani wrote: > > Bumping our minimum QEMU version wouldn't affect that kind of > > scenario, as long as we keep making sure libvirt itself builds on > > RHEL 6 - which we already do a

Re: [libvirt] [PATCH v5 09/10] virsh: implement new command for launch security

2018-04-03 Thread Erik Skultety
On Mon, Apr 02, 2018 at 09:18:55AM -0500, Brijesh Singh wrote: > Add new 'launch-security' command, the command can be used to get or set > the launch security information when booting encrypted VMs. > > Signed-off-by: Brijesh Singh > --- > tools/virsh-domain.c | 84 > +++

Re: [libvirt] [PATCH 00/44] Require QEMU 1.3.0 or newer

2018-04-03 Thread Andrea Bolognani
On Tue, 2018-04-03 at 09:29 -0400, John Ferlan wrote: > Maybe we need to formulate some sort of "annual processing" to declare > minimal version support and declare certain non-maintained drivers to be > dead. I believe it becomes harder and harder to support a policy of must > keep support for som

Re: [libvirt] [PATCH 00/44] Require QEMU 1.3.0 or newer

2018-04-03 Thread Pavel Hrdina
On Tue, Apr 03, 2018 at 02:49:34PM +0100, Daniel P. Berrangé wrote: > On Tue, Apr 03, 2018 at 12:57:45PM +0200, Peter Krempa wrote: > > On Tue, Apr 03, 2018 at 11:45:19 +0100, Daniel Berrange wrote: > > > On Fri, Mar 30, 2018 at 03:15:16PM +0200, Ján Tomko wrote: > > > > It's been a while since we

Re: [libvirt] [PATCH 0/7] Remove the legacy xen driver

2018-04-03 Thread Daniel P . Berrangé
On Wed, Mar 28, 2018 at 08:58:25PM -0600, Jim Fehlig wrote: > Long overdue first cut at remove the old xen driver. The first 3 patches > move existing tests to WITH_LIBXL since we'll want to continue supporting > conversion of the various xen config formats. The remain patches remove > the cruft. >

[libvirt] [PATCH v1 0/2] Fix qcow2 fully allocated filesystem objects.

2018-04-03 Thread Wim Ten Have
From: Wim ten Have When tools like virt-install request to create a fully allocated filesystem object storage by setting the parameter sparse=no, libvirt doesn't allow that to happen for qcow2 formatted files. Regardless of its XML instuction request libvirt always targets its filesystem object

[libvirt] [PATCH v1 1/2] storage: extend preallocation flags support for qemu-img

2018-04-03 Thread Wim Ten Have
From: Wim ten Have This patch adds support to qcow2 formatted filesystem object storage by instructing qemu-img to build them with preallocation=falloc whenever the XML described storage matches its . For all other cases the filesystem stored objects are built with preallocation=metadata. Sign

[libvirt] [PATCH] tests: utils: Fix out-of-bounds access to memory in virTestCompareToFile

2018-04-03 Thread Peter Krempa
Similarly to 3506f1ecfde, the contents of 'cmpcontent' may be an empty string so the following code would access memory out of the array. Signed-off-by: Peter Krempa --- tests/testutils.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/tests/testutils.c b/tests/testutils.c

[libvirt] [PATCH v1 2/2] tests: add qemu-img test for preallocation=falloc

2018-04-03 Thread Wim Ten Have
From: Wim ten Have Signed-off-by: Wim ten Have --- tests/storagevolxml2argvdata/qcow2-nocapacity-convert-prealloc.argv | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tests/storagevolxml2argvdata/qcow2-nocapacity-convert-prealloc.argv b/tests/storagevolxml2argvdata/qcow2-n

Re: [libvirt] [PATCH 00/44] Require QEMU 1.3.0 or newer

2018-04-03 Thread Daniel P . Berrangé
On Tue, Apr 03, 2018 at 09:29:01AM -0400, John Ferlan wrote: > > > On 04/03/2018 07:23 AM, Andrea Bolognani wrote: > > On Tue, 2018-04-03 at 12:57 +0200, Peter Krempa wrote: > >> On Tue, Apr 03, 2018 at 11:45:19 +0100, Daniel Berrange wrote: > >>> On Fri, Mar 30, 2018 at 03:15:16PM +0200, Ján Tom

Re: [libvirt] [PATCH 00/44] Require QEMU 1.3.0 or newer

2018-04-03 Thread Daniel P . Berrangé
On Tue, Apr 03, 2018 at 01:23:15PM +0200, Andrea Bolognani wrote: > On Tue, 2018-04-03 at 12:57 +0200, Peter Krempa wrote: > > On Tue, Apr 03, 2018 at 11:45:19 +0100, Daniel Berrange wrote: > > > On Fri, Mar 30, 2018 at 03:15:16PM +0200, Ján Tomko wrote: > > > > It's been a while since we last bump

Re: [libvirt] [PATCH v5 03/10] conf: introduce launch-security element in domain

2018-04-03 Thread Erik Skultety
... > > > > typedef enum { > > +VIR_DOMAIN_LAUNCH_SECURITY_NONE, > > +VIR_DOMAIN_LAUNCH_SECURITY_SEV, > > + > > +VIR_DOMAIN_LAUNCH_SECURITY_LAST, > > +} virDomainLaunchSecurity; > > + > > +typedef struct _virDomainSevDef virDomainSevDef; > > +typedef virDomainSevDef *virDomainSevDefPt

Re: [libvirt] [PATCH 00/44] Require QEMU 1.3.0 or newer

2018-04-03 Thread Daniel P . Berrangé
On Tue, Apr 03, 2018 at 12:57:45PM +0200, Peter Krempa wrote: > On Tue, Apr 03, 2018 at 11:45:19 +0100, Daniel Berrange wrote: > > On Fri, Mar 30, 2018 at 03:15:16PM +0200, Ján Tomko wrote: > > > It's been a while since we last bumped the minimum QEMU version. > > > Let's get rid of -help parsing a

Re: [libvirt] [PATCH 02/20] remote: Don't hard code the feature VIR_DRV_FEATURE_REMOTE_CLOSE_CALLBACK as available

2018-04-03 Thread Marc Hartmayer
On Tue, Apr 03, 2018 at 02:05 PM +0200, "Daniel P. Berrangé" wrote: > On Tue, Apr 03, 2018 at 01:46:13PM +0200, Marc Hartmayer wrote: >> On Wed, Mar 21, 2018 at 07:57 AM +0100, Marc Hartmayer >> wrote: >> > On Mon, Mar 19, 2018 at 04:06 PM +0100, "Daniel P. Berrangé" >> > wrote: >> >> On Thu,

Re: [libvirt] [PATCH 00/44] Require QEMU 1.3.0 or newer

2018-04-03 Thread John Ferlan
On 04/03/2018 07:23 AM, Andrea Bolognani wrote: > On Tue, 2018-04-03 at 12:57 +0200, Peter Krempa wrote: >> On Tue, Apr 03, 2018 at 11:45:19 +0100, Daniel Berrange wrote: >>> On Fri, Mar 30, 2018 at 03:15:16PM +0200, Ján Tomko wrote: It's been a while since we last bumped the minimum QEMU ve

[libvirt] [PATCH python] Post-release version bump to 4.3.0

2018-04-03 Thread Daniel P . Berrangé
Signed-off-by: Daniel P. Berrangé --- Pushed as trivial change. setup.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/setup.py b/setup.py index c40908b..eff9d54 100755 --- a/setup.py +++ b/setup.py @@ -334,7 +334,7 @@ class my_clean(clean): _c_modules, _py_modules = get_

Re: [libvirt] [PATCH] rpm: add deps on rpcgen/libtirpc-devel

2018-04-03 Thread Andrea Bolognani
On Tue, 2018-04-03 at 13:42 +0100, Daniel P. Berrangé wrote: > Since RPC support moved out of glibc we need to have explicit deps on > the new packages providing this functionality > > Signed-off-by: Daniel P. Berrangé > --- > libvirt.spec.in | 5 + > 1 file changed, 5 insertions(+) > > dif

[libvirt] [PATCH] qemu: Format gic-version=2 on the command line

2018-04-03 Thread Andrea Bolognani
Up until now we have only formatted non-default GIC versions on the command line, in order to maintain compatibility with older QEMU versions that didn't implement the gic-version option to begin with; however, doing so is entirely unnecessary for newer QEMU versions, where the option is available.

[libvirt] [PATCH] rpm: add deps on rpcgen/libtirpc-devel

2018-04-03 Thread Daniel P . Berrangé
Since RPC support moved out of glibc we need to have explicit deps on the new packages providing this functionality Signed-off-by: Daniel P. Berrangé --- libvirt.spec.in | 5 + 1 file changed, 5 insertions(+) diff --git a/libvirt.spec.in b/libvirt.spec.in index b55a947ec9..97143c68ae 100644

Re: [libvirt] [PATCH 02/20] remote: Don't hard code the feature VIR_DRV_FEATURE_REMOTE_CLOSE_CALLBACK as available

2018-04-03 Thread Daniel P . Berrangé
On Tue, Apr 03, 2018 at 01:46:13PM +0200, Marc Hartmayer wrote: > On Wed, Mar 21, 2018 at 07:57 AM +0100, Marc Hartmayer > wrote: > > On Mon, Mar 19, 2018 at 04:06 PM +0100, "Daniel P. Berrangé" > > wrote: > >> On Thu, Mar 08, 2018 at 01:20:25PM +0100, Marc Hartmayer wrote: > >>> Don't assume t

[libvirt] [RFC v2] external (pull) backup API

2018-04-03 Thread Nikolay Shirokovskiy
Hi, all. This is another RFC on pull backup API. This API provides means to read domain di

Re: [libvirt] Race condition between qemuDomainCreate and qemuDomainDestroy

2018-04-03 Thread Marc Hartmayer
On Tue, Mar 20, 2018 at 11:25 AM +0100, Marc Hartmayer wrote: > Hi, > > there is a race condition between 'qemuDomainCreate' and > 'qemuDomainDestroy' causing a NULL pointer segmentation fault when > accessing priv->monConfig. The race condition can be easily reproduced > using gdb. > > (gdb) set

Re: [libvirt] [PATCH 02/20] remote: Don't hard code the feature VIR_DRV_FEATURE_REMOTE_CLOSE_CALLBACK as available

2018-04-03 Thread Marc Hartmayer
On Wed, Mar 21, 2018 at 07:57 AM +0100, Marc Hartmayer wrote: > On Mon, Mar 19, 2018 at 04:06 PM +0100, "Daniel P. Berrangé" > wrote: >> On Thu, Mar 08, 2018 at 01:20:25PM +0100, Marc Hartmayer wrote: >>> Don't assume that the feature VIR_DRV_FEATURE_REMOTE_CLOSE_CALLBACK is >>> available for e

Re: [libvirt] [PATCH 1/2] libxl: drop support for Xen < 4.6

2018-04-03 Thread Martin Kletzander
On Wed, Mar 28, 2018 at 03:13:35PM +0100, Daniel P. Berrangé wrote: On Wed, Mar 28, 2018 at 08:10:34AM -0600, Jim Fehlig wrote: On 03/28/2018 07:28 AM, Daniel P. Berrangé wrote: > On Wed, Mar 28, 2018 at 10:03:51AM +0100, Daniel P. Berrangé wrote: > > On Tue, Mar 27, 2018 at 04:31:29PM -0600, Ji

Re: [libvirt] [PATCH] util: json: Remove yajl bits from virJSONValueToStr

2018-04-03 Thread Peter Krempa
On Tue, Apr 03, 2018 at 12:23:13 +0100, Daniel Berrange wrote: > On Sat, Mar 31, 2018 at 11:01:15AM +0200, Peter Krempa wrote: > > Rather than depending on yajl bits for creating the JSON structure > > replace it by few virBuffer bits. This will make the JSON formatter > > libary agnostic. > > I d

Re: [libvirt] [PATCH 00/44] Require QEMU 1.3.0 or newer

2018-04-03 Thread Andrea Bolognani
On Tue, 2018-04-03 at 12:57 +0200, Peter Krempa wrote: > On Tue, Apr 03, 2018 at 11:45:19 +0100, Daniel Berrange wrote: > > On Fri, Mar 30, 2018 at 03:15:16PM +0200, Ján Tomko wrote: > > > It's been a while since we last bumped the minimum QEMU version. > > > Let's get rid of -help parsing and brin

Re: [libvirt] [PATCH] util: json: Remove yajl bits from virJSONValueToStr

2018-04-03 Thread Daniel P . Berrangé
On Sat, Mar 31, 2018 at 11:01:15AM +0200, Peter Krempa wrote: > Rather than depending on yajl bits for creating the JSON structure > replace it by few virBuffer bits. This will make the JSON formatter > libary agnostic. I don't think this is a good idea as it means we have to reinvent the wheel to

Re: [libvirt] [PATCH 0/9] json: Fix leak/double-free, clean up code and privatize virJSONValue

2018-04-03 Thread Daniel P . Berrangé
On Tue, Apr 03, 2018 at 01:07:29PM +0200, Peter Krempa wrote: > On Tue, Apr 03, 2018 at 13:01:48 +0200, Peter Krempa wrote: > > On Tue, Apr 03, 2018 at 11:56:33 +0100, Daniel Berrange wrote: > > > On Fri, Mar 30, 2018 at 12:59:07PM +0200, Peter Krempa wrote: > > > > Coverity was not wrong about the

Re: [libvirt] [PATCH 0/9] json: Fix leak/double-free, clean up code and privatize virJSONValue

2018-04-03 Thread Peter Krempa
On Tue, Apr 03, 2018 at 13:01:48 +0200, Peter Krempa wrote: > On Tue, Apr 03, 2018 at 11:56:33 +0100, Daniel Berrange wrote: > > On Fri, Mar 30, 2018 at 12:59:07PM +0200, Peter Krempa wrote: > > > Coverity was not wrong about the usage of 'a'/'A' modifiers for > > > virJSONValueObjectAddVArgs as no

Re: [libvirt] [PATCH 0/9] json: Fix leak/double-free, clean up code and privatize virJSONValue

2018-04-03 Thread Peter Krempa
On Tue, Apr 03, 2018 at 11:56:33 +0100, Daniel Berrange wrote: > On Fri, Mar 30, 2018 at 12:59:07PM +0200, Peter Krempa wrote: > > Coverity was not wrong about the usage of 'a'/'A' modifiers for > > virJSONValueObjectAddVArgs as noted in [1]. Fix the possible > > leak/double-free, and add test to m

Re: [libvirt] [PATCH 00/44] Require QEMU 1.3.0 or newer

2018-04-03 Thread Peter Krempa
On Tue, Apr 03, 2018 at 11:45:19 +0100, Daniel Berrange wrote: > On Fri, Mar 30, 2018 at 03:15:16PM +0200, Ján Tomko wrote: > > It's been a while since we last bumped the minimum QEMU version. > > Let's get rid of -help parsing and bring our test suite closer > > to real world usage by implying lot

Re: [libvirt] [PATCH 0/9] json: Fix leak/double-free, clean up code and privatize virJSONValue

2018-04-03 Thread Daniel P . Berrangé
On Fri, Mar 30, 2018 at 12:59:07PM +0200, Peter Krempa wrote: > Coverity was not wrong about the usage of 'a'/'A' modifiers for > virJSONValueObjectAddVArgs as noted in [1]. Fix the possible > leak/double-free, and add test to make sure it works as expected. Do you have any idea how to trigger the

Re: [libvirt] [PATCH 8/9] virjson: add support for Jansson

2018-04-03 Thread Daniel P . Berrangé
On Tue, Apr 03, 2018 at 12:45:28PM +0200, Peter Krempa wrote: > On Tue, Apr 03, 2018 at 11:39:20 +0100, Daniel Berrange wrote: > > On Sat, Mar 31, 2018 at 11:13:13AM +0200, Peter Krempa wrote: > > > On Thu, Mar 29, 2018 at 01:09:57 +0200, Ján Tomko wrote: > > > > Check for the presence of Jansson l

Re: [libvirt] [PATCH 8/9] virjson: add support for Jansson

2018-04-03 Thread Peter Krempa
On Tue, Apr 03, 2018 at 11:39:20 +0100, Daniel Berrange wrote: > On Sat, Mar 31, 2018 at 11:13:13AM +0200, Peter Krempa wrote: > > On Thu, Mar 29, 2018 at 01:09:57 +0200, Ján Tomko wrote: > > > Check for the presence of Jansson library and prefer it to yajl > > > if possible. > > > > > > The minim

Re: [libvirt] [PATCH 00/44] Require QEMU 1.3.0 or newer

2018-04-03 Thread Daniel P . Berrangé
On Fri, Mar 30, 2018 at 03:15:16PM +0200, Ján Tomko wrote: > It's been a while since we last bumped the minimum QEMU version. > Let's get rid of -help parsing and bring our test suite closer > to real world usage by implying lots of capabilities. NACK, this is effectively dropping support for RHEL

Re: [libvirt] [PATCH 8/9] virjson: add support for Jansson

2018-04-03 Thread Daniel P . Berrangé
On Sat, Mar 31, 2018 at 11:13:13AM +0200, Peter Krempa wrote: > On Thu, Mar 29, 2018 at 01:09:57 +0200, Ján Tomko wrote: > > Check for the presence of Jansson library and prefer it to yajl > > if possible. > > > > The minimum required version is 2.7. > > > > Internally, virJSONValue still stores

Re: [libvirt] [PATCH 7/9] build: link setuid_rpc_client against JSON_LIBS

2018-04-03 Thread Daniel P . Berrangé
On Sat, Mar 31, 2018 at 03:16:55PM +0200, Ján Tomko wrote: > On Sat, Mar 31, 2018 at 11:05:38AM +0200, Peter Krempa wrote: > > On Thu, Mar 29, 2018 at 01:09:56 +0200, Ján Tomko wrote: > > > I have no idea how it builds without and switching to Jansson > > > exposes this. > > > > > > Signed-off-by:

[libvirt] [RESEND PATCH 40/44] Deprecate QEMU_CAPS_DUMP_GUEST_CORE

2018-04-03 Thread Ján Tomko
Implied by QEMU >= 1.2.0. Signed-off-by: Ján Tomko --- src/qemu/qemu_capabilities.c | 1 - src/qemu/qemu_capabilities.h | 2 +- src/qemu/qemu_command.c | 19 +-- tests/qemucapabilitiesdata/caps_1.3.

[libvirt] [RESEND PATCH 11/44] Deprecate QEMU_CAPS_NO_USER_CONFIG

2018-04-03 Thread Ján Tomko
Implied by QEMU >= 1.2.0. Delete this one first, because QEMU_CAPS_NODEFCONFIG is only used when QEMU_CAPS_NO_USER_CONFIG is unsupported. Signed-off-by: Ján Tomko --- src/qemu/qemu_capabilities.c | 1 - src/qemu/qemu_capabilities.h

[libvirt] [RESEND PATCH 15/44] Deprecate QEMU_CAPS_RTC

2018-04-03 Thread Ján Tomko
Implied by QEMU >= 1.2.0. Signed-off-by: Ján Tomko --- src/qemu/qemu_capabilities.c | 1 - src/qemu/qemu_capabilities.h | 2 +- src/qemu/qemu_command.c| 41 -- tests/qemucapabilitiesdata/caps_1.3.1.x86_

[libvirt] [RESEND PATCH 28/44] Deprecate QEMU_CAPS_NO_SHUTDOWN

2018-04-03 Thread Ján Tomko
Implied by QEMU >= 1.2.0. Signed-off-by: Ján Tomko --- src/qemu/qemu_capabilities.c | 1 - src/qemu/qemu_capabilities.h | 2 +- src/qemu/qemu_command.c | 4 +--- src/qemu/qemu_driver.c

[libvirt] [RESEND PATCH 39/44] Deprecate QEMU_CAPS_MACHINE_OPT

2018-04-03 Thread Ján Tomko
Implied by QEMU >= 1.2.0. Signed-off-by: Ján Tomko --- src/qemu/qemu_capabilities.c | 1 - src/qemu/qemu_capabilities.h | 2 +- src/qemu/qemu_command.c| 317 + tests/qemucapabilitiesdata/caps_1.3.1.x8

[libvirt] [RESEND PATCH 43/44] Deprecate QEMU_CAPS_DISPLAY

2018-04-03 Thread Ján Tomko
Implied by QEMU >= 1.2.0. Signed-off-by: Ján Tomko --- src/qemu/qemu_capabilities.c | 12 src/qemu/qemu_capabilities.h | 2 +- src/qemu/qemu_command.c | 8 ++-- tests/qemucapab

[libvirt] [RESEND PATCH 44/44] Deprecate QEMU_CAPS_MACHINE_USB_OPT

2018-04-03 Thread Ján Tomko
Implied by QEMU >= 1.3.0. Signed-off-by: Ján Tomko --- src/qemu/qemu_capabilities.c | 4 src/qemu/qemu_capabilities.h | 2 +- src/qemu/qemu_command.c | 3 +-- test

[libvirt] [RESEND PATCH 10/44] Deprecate QEMU_CAPS_MONITOR_JSON

2018-04-03 Thread Ján Tomko
We require QEMU >= 0.15.0, assume every QEMU supports it. Sadly that does not let us trivially drop qemuMonitor's priv->monJSON bool, because of qemuDomainQemuAttach. Signed-off-by: Ján Tomko --- src/qemu/qemu_capabilities.c | 1 - src/qemu/qemu_capabilities.h

Re: [libvirt] [PATCH] lxc_container: Set source file description

2018-04-03 Thread Michal Privoznik
On 04/02/2018 09:55 AM, Radostin Stoyanov wrote: > Signed-off-by: Radostin Stoyanov > --- > src/lxc/lxc_container.c | 2 +- > src/lxc/lxc_container.h | 2 +- > 2 files changed, 2 insertions(+), 2 deletions(-) ACKed and pushed. Michal -- libvir-list mailing list libvir-list@redhat.com https://w

[libvirt] [PATCH 1/2] vmware: Fix initialization of VMware Fusion

2018-04-03 Thread Rainer Müller
The vmware driver wants to execute vmware-vmx from the same directory in which vmrun was found. However, on VMware Fusion 10 vmrun at /Applications/VMware Fusion.app/Contents/Public/vmrun is a symlink pointing to ../Library/vmrun. vmware-vmx cannot be found, as it is not in PATH, but only in this L

[libvirt] [PATCH 0/2] Fix vmware driver for VMware Fusion 10

2018-04-03 Thread Rainer Müller
These changes were required to get VMware Fusion working at all. I tested with VMware Fusion 10.1.1 on macOS 10.12 Sierra. I am not sure whether calling virCapabilitiesInitCaches() makes sense at all on macOS. This function looks highly specific to Linux as it relies on /sys/devices/system, so it

[libvirt] [PATCH 2/2] vmware: Failures in cache info init are non-fatal

2018-04-03 Thread Rainer Müller
This is also not fatal on other drivers. Signed-off-by: Rainer Müller --- src/vmware/vmware_conf.c | 7 ++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/src/vmware/vmware_conf.c b/src/vmware/vmware_conf.c index 891d9a47f..b9f18e6ac 100644 --- a/src/vmware/vmware_conf.c +++ b/

[libvirt] [PATCH v3 0/3] Improve handling of multipath devices

2018-04-03 Thread Michal Privoznik
The saga continues. Turns out, this is bug in both libvirt AND kernel. Firstly, kernel uses different CGroup for checking open() than for ioctl() [1]. However, libvirt needs to allow all targets in the CGroup too. Not only it's logical, kernel devel says it never worked. QEMU was able to open the

[libvirt] [PATCH v3 3/3] news: Document device mapper fix

2018-04-03 Thread Michal Privoznik
Signed-off-by: Michal Privoznik --- docs/news.xml | 10 ++ 1 file changed, 10 insertions(+) diff --git a/docs/news.xml b/docs/news.xml index 87f52e83ef..065b60b6eb 100644 --- a/docs/news.xml +++ b/docs/news.xml @@ -49,6 +49,16 @@ + + + Improve h

[libvirt] [PATCH v3 1/3] util: Introduce virDevMapperGetTargets

2018-04-03 Thread Michal Privoznik
This helper fetches dependencies for given device mapper target. At the same time, we need to provide a dummy log function because by default libdevmapper prints out error messages to stderr which we need to suppress. Signed-off-by: Michal Privoznik --- src/libvirt_private.syms | 4 + src/uti

[libvirt] [PATCH v3 2/3] qemu_cgroup: Handle device mapper targets properly

2018-04-03 Thread Michal Privoznik
https://bugzilla.redhat.com/show_bug.cgi?id=1557769 Problem with device mapper targets is that there can be several other devices 'hidden' behind them. For instance, /dev/dm-1 can consist of /dev/sda, /dev/sdb and /dev/sdc. Therefore, when setting up devices CGroup and namespaces we have to take t

Re: [libvirt] [PATCH 00/44] Require QEMU 1.3.0 or newer

2018-04-03 Thread Ján Tomko
On Fri, Mar 30, 2018 at 03:15:16PM +0200, Ján Tomko wrote: It's been a while since we last bumped the minimum QEMU version. Let's get rid of -help parsing and bring our test suite closer to real world usage by implying lots of capabilities. Ján Tomko (44): The following patches were > 300 kB:

[libvirt] [PATCH 3/4] util: virlog: Introduce wildcard to log filters

2018-04-03 Thread Erik Skultety
Since the introduction of log tuning capabilities to virt-admin by @06b91785, this has been a much needed missing improvement on the way to deprecate the global 'log_level'. Signed-off-by: Erik Skultety --- src/remote/libvirtd.conf | 4 +++- src/util/virlog.c| 44 +++

[libvirt] [PATCH 0/4] Introduce wildcard to log filters

2018-04-03 Thread Erik Skultety
First, 2 tiny fixes along the way, then the actual change, which lets you do the following for example during runtime: virt-admin daemon-log-filters "1:* 4:remote 4:json" which previously, given the default config setting log_level=3, translated into virt-admin daemon-log-filters \ "1:acce

[libvirt] [PATCH 4/4] news: Update the news file with the log filter wildcard improvement

2018-04-03 Thread Erik Skultety
Signed-off-by: Erik Skultety --- docs/news.xml | 12 1 file changed, 12 insertions(+) diff --git a/docs/news.xml b/docs/news.xml index 87f52e83e..6dab4844d 100644 --- a/docs/news.xml +++ b/docs/news.xml @@ -47,6 +47,18 @@ supported. In fact, kernel has been supporting thi

Re: [libvirt] [dbus PATCH v4 0/5] New APIs and Domain Lifecycle signal change

2018-04-03 Thread Pavel Hrdina
On Tue, Apr 03, 2018 at 10:29:55AM +0200, Katerina Koukiou wrote: > * Implemented Suspend and Resume APIs. > * Reused virsh functions for taking the Event Strings names from ENUM. > * Merged all Domain Lifecycle events signals into one signal. > > Changes from v3: > * s/arg2/arg1/ since Event Type

[libvirt] [PATCH 1/4] virlog: Fix a typo in virLogParseFilter's error msg

2018-04-03 Thread Erik Skultety
This was some copy-paste leftover. Signed-off-by: Erik Skultety --- src/util/virlog.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/util/virlog.c b/src/util/virlog.c index dd927f0ba..5810643e1 100644 --- a/src/util/virlog.c +++ b/src/util/virlog.c @@ -1699,7 +1699,7 @@

[libvirt] [PATCH 2/4] libvirtd.conf: Document that we do a 'first' match on log filters

2018-04-03 Thread Erik Skultety
When applying the log filters, one has to define the more specific filters before the generic ones, because the first filter that matches will be applied. However, we've been missing this information in the config, so it always has been a trial-error scenario figuring out that e.g. '4:util 1:util.p

Re: [libvirt] [dbus PATCH v4 5/5] Merge all Domain lifecycle signals into one signal called Domain.

2018-04-03 Thread Pavel Hrdina
On Tue, Apr 03, 2018 at 10:30:00AM +0200, Katerina Koukiou wrote: > Instead of having multiple signals regarding to domain events, > like DomainStarted, DomainUndefined etc, we will have only one > called DomainEvents, and the specific event type will be specified in > the signals arguments. > > T

Re: [libvirt] [dbus PATCH v4 4/5] events: Introduce virtDBusEventsDomainEventToString function

2018-04-03 Thread Pavel Hrdina
On Tue, Apr 03, 2018 at 10:29:59AM +0200, Katerina Koukiou wrote: > This function replaces the manual translation of Event ENUMS. > > Signed-off-by: Katerina Koukiou > --- > src/events.c | 55 +-- > 1 file changed, 21 insertions(+), 34 deletion

[libvirt] [dbus PATCH v4 1/5] Implement Suspend method for Domain interface.

2018-04-03 Thread Katerina Koukiou
Signed-off-by: Katerina Koukiou --- data/org.libvirt.Domain.xml | 4 src/domain.c| 21 + test/test_domain.py | 15 +++ 3 files changed, 40 insertions(+) diff --git a/data/org.libvirt.Domain.xml b/data/org.libvirt.Domain.xml index 3921

[libvirt] [dbus PATCH v4 5/5] Merge all Domain lifecycle signals into one signal called Domain.

2018-04-03 Thread Katerina Koukiou
Instead of having multiple signals regarding to domain events, like DomainStarted, DomainUndefined etc, we will have only one called DomainEvents, and the specific event type will be specified in the signals arguments. The domain name argument in not needed in the signal since we can fetch it from

[libvirt] [dbus PATCH v4 3/5] utils: Introduce functions and macros for translating ENUMs to strings

2018-04-03 Thread Katerina Koukiou
The functions were copied from src/util/virutil.* files from libvirt project Signed-off-by: Katerina Koukiou --- src/util.c | 27 +++ src/util.h | 28 2 files changed, 55 insertions(+) diff --git a/src/util.c b/src/util.c index d6c27f3..be651

[libvirt] [dbus PATCH v4 4/5] events: Introduce virtDBusEventsDomainEventToString function

2018-04-03 Thread Katerina Koukiou
This function replaces the manual translation of Event ENUMS. Signed-off-by: Katerina Koukiou --- src/events.c | 55 +-- 1 file changed, 21 insertions(+), 34 deletions(-) diff --git a/src/events.c b/src/events.c index dada55f..1877115 100644 -

[libvirt] [dbus PATCH v4 2/5] Implement Resume method for Domain interface.

2018-04-03 Thread Katerina Koukiou
Signed-off-by: Katerina Koukiou --- data/org.libvirt.Domain.xml | 4 src/domain.c| 21 + test/test_domain.py | 17 + 3 files changed, 42 insertions(+) diff --git a/data/org.libvirt.Domain.xml b/data/org.libvirt.Domain.xml index f3

  1   2   >