Re: [libvirt] [Qemu-devel] [PATCH v7 0/4] Add Mediated device support

2016-09-02 Thread Laine Stump
On 09/02/2016 05:44 PM, Paolo Bonzini wrote: On 02/09/2016 22:19, John Ferlan wrote: We don't have such a pool for GPU's (yet) - although I suppose they could just become a class of storage pools. The issue being nodedev device objects are not saved between reboots. They are generated on the

Re: [libvirt] [Qemu-devel] [PATCH v7 0/4] Add Mediated device support

2016-09-02 Thread Paolo Bonzini
On 02/09/2016 20:33, Kirti Wankhede wrote: > We could even do: >> > >> > echo $UUID1:$GROUPA > create >> > >> > where $GROUPA is the group ID of a previously created mdev device into >> > which $UUID1 is to be created and added to the same group. > >From the point of view of libvirt, I think

Re: [libvirt] [Qemu-devel] [PATCH v7 0/4] Add Mediated device support

2016-09-02 Thread Paolo Bonzini
On 02/09/2016 22:19, John Ferlan wrote: > We don't have such a pool for GPU's (yet) - although I suppose they > could just become a class of storage pools. > > The issue being nodedev device objects are not saved between reboots. > They are generated on the fly. Hence the "create-nodedev' API -

Re: [libvirt] [Qemu-devel] [PATCH v7 0/4] Add Mediated device support

2016-09-02 Thread John Ferlan
On 09/02/2016 02:33 PM, Kirti Wankhede wrote: > > On 9/2/2016 10:55 PM, Paolo Bonzini wrote: >> >> >> On 02/09/2016 19:15, Kirti Wankhede wrote: >>> On 9/2/2016 3:35 PM, Paolo Bonzini wrote: my-vgpu pci__86_00_0

Re: [libvirt] [Qemu-devel] [PATCH v7 0/4] Add Mediated device support

2016-09-02 Thread John Ferlan
On 09/02/2016 06:05 AM, Paolo Bonzini wrote: > > > On 02/09/2016 07:21, Kirti Wankhede wrote: >> On 9/2/2016 10:18 AM, Michal Privoznik wrote: >>> Okay, maybe I'm misunderstanding something. I just thought that users >>> will consult libvirt's nodedev driver (e.g. virsh nodedev-list && virsh

Re: [libvirt] [Qemu-devel] [PATCH v7 0/4] Add Mediated device support

2016-09-02 Thread Alex Williamson
On Fri, 2 Sep 2016 13:55:19 -0400 Laine Stump wrote: > On 09/01/2016 12:59 PM, Alex Williamson wrote: > > On Thu, 1 Sep 2016 18:47:06 +0200 > > Michal Privoznik wrote: > > > >> On 31.08.2016 08:12, Tian, Kevin wrote: > From: Alex Williamson

Re: [libvirt] [PATCH v3 1/4] libxl: support serial list

2016-09-02 Thread Jim Fehlig
On 08/31/2016 08:36 PM, Jim Fehlig wrote: > On 08/17/2016 08:20 PM, Bob Liu wrote: >> Add support for multi serial devices, after this patch virsh can be used to >> connect different serial devices of running domains. E.g. >> vish # console --devname serial >> >> Note: >> This depends on a

Re: [libvirt] [PATCH V2] virpci: support driver_override sysfs interface

2016-09-02 Thread Jim Fehlig
On 08/31/2016 01:42 PM, Jim Fehlig wrote: > On 08/30/2016 11:59 PM, Laine Stump wrote: >> On 08/01/2016 11:36 PM, Jim Fehlig wrote: >>> libvirt uses the new_id PCI sysfs interface to bind a PCI stub driver >>> to a PCI device. The new_id interface is known to be buggy and racey, >>> hence a more

Re: [libvirt] [Qemu-devel] [PATCH v7 0/4] Add Mediated device support

2016-09-02 Thread Kirti Wankhede
On 9/2/2016 10:55 PM, Paolo Bonzini wrote: > > > On 02/09/2016 19:15, Kirti Wankhede wrote: >> On 9/2/2016 3:35 PM, Paolo Bonzini wrote: >>> >>> my-vgpu >>> pci__86_00_0 >>> >>> >>>0695d332-7831-493f-9e71-1c85c8911a08 >>> >>> >>> >>> After

[libvirt] firewalld transaction model

2016-09-02 Thread Thomas Woerner
Hello, the transaction model that has been introduced with firewalld-0.4.2 makes it possible to group rules together and to apply them at once and quick. For this the restore commands of iptables, ip6tables and ebtables are used as long as they are available. At the moment the transaction model

Re: [libvirt] [Qemu-devel] [PATCH v7 0/4] Add Mediated device support

2016-09-02 Thread Laine Stump
On 09/01/2016 12:59 PM, Alex Williamson wrote: On Thu, 1 Sep 2016 18:47:06 +0200 Michal Privoznik wrote: On 31.08.2016 08:12, Tian, Kevin wrote: From: Alex Williamson [mailto:alex.william...@redhat.com] Sent: Wednesday, August 31, 2016 12:17 AM Hi folks, At KVM Forum

Re: [libvirt] [Qemu-devel] [PATCH v7 0/4] Add Mediated device support

2016-09-02 Thread Paolo Bonzini
On 02/09/2016 19:15, Kirti Wankhede wrote: > On 9/2/2016 3:35 PM, Paolo Bonzini wrote: >> >> my-vgpu >> pci__86_00_0 >> >> >>0695d332-7831-493f-9e71-1c85c8911a08 >> >> >> >> After creating the vGPU, if required by the host driver, all the other

Re: [libvirt] [Qemu-devel] [PATCH v7 0/4] Add Mediated device support

2016-09-02 Thread Kirti Wankhede
On 9/2/2016 3:35 PM, Paolo Bonzini wrote: > > > On 02/09/2016 07:21, Kirti Wankhede wrote: >> On 9/2/2016 10:18 AM, Michal Privoznik wrote: >>> Okay, maybe I'm misunderstanding something. I just thought that users >>> will consult libvirt's nodedev driver (e.g. virsh nodedev-list && virsh >>>

[libvirt] [libvirt-php] libvirt_connect not reading out credential info on 0.5.2

2016-09-02 Thread Fernando Casas Schössow
I'm running libvirt-php 0.5.2 on CentOS 7 with libvirt 2.1.0. Using virsh I'm able to connect to hyper-v hosts correctly but using libvirt-php it fails during authentication since it seems that the credentials are not being passed along. This is the php code I'm using: $credentials =

Re: [libvirt] [PATCH V2 2/2] libxl: allow vendor/product addressing for USB hostdevs

2016-09-02 Thread Cedric Bosdonnat
On Fri, 2016-09-02 at 15:24 +0200, Martin Kletzander wrote: > On Wed, Aug 10, 2016 at 06:39:12PM -0600, Jim Fehlig wrote: > > > > From: Cédric Bosdonnat > > > > libxl only has API to address the host USB devices by bus/device. > > Find the bus/device if the user only

Re: [libvirt] wiki accounts for GSOC submissions

2016-09-02 Thread Martin Kletzander
On Mon, Aug 22, 2016 at 01:30:13PM -0400, Daniel P. Berrange wrote: On Fri, Aug 19, 2016 at 05:27:25PM +0200, Ján Tomko wrote: On Fri, Aug 19, 2016 at 05:13:57PM +0200, Martin Kletzander wrote: > On Thu, Aug 11, 2016 at 10:29:31AM -0400, Cole Robinson wrote: > > On 08/11/2016 10:26 AM, Daniel

Re: [libvirt] [gconfig v2 6/6] tests: Add test for GVirConfigDomainGraphicsListenAddress

2016-09-02 Thread Christophe Fergeau
On Wed, Aug 17, 2016 at 06:58:51PM +0300, Visarion Alexandru wrote: > From: Visarion Alexandru > > --- > tests/test-gconfig.c | 36 > ++ > .../xml/gconfig-domain-device-graphics-listen.xml | 7 + > 2 files

Re: [libvirt] [gconfig v2 5/6] config: Add spice listen getter/setter

2016-09-02 Thread Christophe Fergeau
On Wed, Aug 17, 2016 at 06:58:50PM +0300, Visarion Alexandru wrote: > From: Visarion Alexandru > > Learn to get/set the listen devices that spice graphics is using. > > When setting the listen devices, first remove the 'listen' > attribute to avoid the inconsistencies

Re: [libvirt] [gconfig v2 4/6] config: Add vnc listen getter/setter

2016-09-02 Thread Christophe Fergeau
On Wed, Aug 17, 2016 at 06:58:49PM +0300, Visarion Alexandru wrote: > From: Visarion Alexandru > > Learn to get/set the listen devices that vnc is using. > > When setting the listen devices, first remove the 'listen' attribute > to avoid inconsistencies checks between

Re: [libvirt] [gconfig v2 3/6] Introduce libvirt-gconfig-domain-graphics-listen-address

2016-09-02 Thread Christophe Fergeau
On Wed, Aug 17, 2016 at 06:58:48PM +0300, Visarion Alexandru wrote: > From: Visarion Alexandru > > This is needed to be able to change the address a graphics > device is listening for. > --- > libvirt-gconfig/Makefile.am| 2 + >

Re: [libvirt] [gconfig v2 2/6] Introduce libvirt-domain-graphics-listen

2016-09-02 Thread Christophe Fergeau
Acked-by: Christophe Fergeau On Wed, Aug 17, 2016 at 06:58:47PM +0300, Visarion Alexandru wrote: > From: Visarion Alexandru > > Abstract class which represents a listen child node > of the graphics device. > --- > libvirt-gconfig/Makefile.am

Re: [libvirt] [gconfig v2 1/6] libvirt-gconfig: Include GIO dependency

2016-09-02 Thread Christophe Fergeau
Hey, On Wed, Aug 17, 2016 at 06:58:46PM +0300, Visarion Alexandru wrote: > From: Visarion Alexandru > > We will need a GInetAddress for setting a graphics device's > listen address, so let's include GIO2 in libvirt-gconfig. > --- > configure.ac| 2 ++

[libvirt] Release of libvirt-2.2.0

2016-09-02 Thread Daniel Veillard
Since the issue I had with rc2 was apparently just on my setup, I just pushed the release. It is available as usual as signed tarball and rpms from : ftp://libvirt.org/libvirt/ I also pushed 2.2.0 for the python bindings which you can find at: ftp://libvirt.org/libvirt/python/ This is

Re: [libvirt] [PATCH] libxl: add memory attach support

2016-09-02 Thread Martin Kletzander
On Wed, Aug 31, 2016 at 12:12:55PM +0100, Joao Martins wrote: On 08/31/2016 10:18 AM, Martin Kletzander wrote: On Wed, Aug 31, 2016 at 03:56:02PM +0800, Bob Liu wrote: On 08/30/2016 11:20 PM, Joao Martins wrote: Hey! On 08/30/2016 11:00 AM, Bob Liu wrote: Support for

Re: [libvirt] Querying block device job status and semantics of virDomainBlockJobInfo()

2016-09-02 Thread Kashyap Chamarthy
On Thu, Sep 01, 2016 at 02:11:34PM -0500, Eric Blake wrote: > On 09/01/2016 08:57 AM, Kashyap Chamarthy wrote: > > So, I'm trying to understand how libvirt reports the "cur" and "end" > > values. I've read the virDomainBlockJobInfo() struct, it wasn't crystal > > clear. It states: > > > > /* >

Re: [libvirt] [PATCH V2 2/2] libxl: allow vendor/product addressing for USB hostdevs

2016-09-02 Thread Martin Kletzander
On Wed, Aug 10, 2016 at 06:39:12PM -0600, Jim Fehlig wrote: From: Cédric Bosdonnat libxl only has API to address the host USB devices by bus/device. Find the bus/device if the user only provided the vendor/product of the USB device. Signed-off-by: Jim Fehlig

Re: [libvirt] [PATCH V2 1/2] Add virHostdevFindUSBDevice to private symbols

2016-09-02 Thread Martin Kletzander
On Wed, Aug 10, 2016 at 06:39:11PM -0600, Jim Fehlig wrote: From: Cédric Bosdonnat Finding an USB device from the vendor/device values will be needed by libxl driver to convert from vendor/device to bus/dev addresses. Signed-off-by: Jim Fehlig ---

Re: [libvirt] [PATCH] conf: Fix initialization value of 'multi' in PCI address

2016-09-02 Thread Martin Kletzander
On Mon, Aug 15, 2016 at 06:22:25AM +0200, Xian Han Yu wrote: The 'multi' element in PCI address struct used as 'virTristateSwitch', and its default value is 'VIR_TRISTATE_SWITCH_ABSENT'. Current PCI process use 'false' to initialization 'multi', which is ambiguously for assignment or comparison.

Re: [libvirt] [PATCH] Make --postcopy flag mandatory with --postcopy-after-precopy

2016-09-02 Thread Jiri Denemark
On Fri, Sep 02, 2016 at 07:52:13 -0400, Kothapally Madhu Pavan wrote: > --postcopy-after-precopy is just an aditional flag for > postcopy migration. > > Signed-off-by: Kothapally Madhu Pavan > --- > tools/virsh-domain.c |6 +++--- > tools/virsh.pod |4 ++--

Re: [libvirt] [PATCH] Make --postcopy flag mandatory with --postcopy-after-precopy

2016-09-02 Thread Jiri Denemark
On Fri, Sep 02, 2016 at 07:52:13 -0400, Kothapally Madhu Pavan wrote: > --postcopy-after-precopy is just an aditional flag for > postcopy migration. > > Signed-off-by: Kothapally Madhu Pavan > --- > tools/virsh-domain.c |6 +++--- > tools/virsh.pod |4 ++--

Re: [libvirt] [Libvirt-announce] libvirt-2.2.0 Release Candidate 2 is out

2016-09-02 Thread Martin Kletzander
On Wed, Aug 31, 2016 at 12:21:00PM -0400, Laine Stump wrote: On 08/30/2016 03:04 PM, Daniel Veillard wrote: It's tagged in git, with signed tarball and rpms at the usual place: ftp://libvirt.org/libvirt/ but it looks broken to me, if I use virt manager and try to start a guest it

Re: [libvirt] [PATCH V2 0/2] libxl USB vendor/product support

2016-09-02 Thread Cedric Bosdonnat
I'ld ACK this v2 of my patch series, but not sure I'm the right guy to do it. Anyone else can review it? -- Cedric On Wed, 2016-08-10 at 18:39 -0600, Jim Fehlig wrote: > V2 of Cedric's patches to allow specifying USB hostdevs by vendor/product > in the libxl driver > >

[libvirt] [PATCH] Make --postcopy flag mandatory with --postcopy-after-precopy

2016-09-02 Thread Kothapally Madhu Pavan
--postcopy-after-precopy is just an aditional flag for postcopy migration. Signed-off-by: Kothapally Madhu Pavan --- tools/virsh-domain.c |6 +++--- tools/virsh.pod |4 ++-- 2 files changed, 5 insertions(+), 5 deletions(-) diff --git a/tools/virsh-domain.c

[libvirt] [PATCH] qemuDomainUpdateDeviceConfig: Allow full disk update

2016-09-02 Thread Michal Privoznik
https://bugzilla.redhat.com/show_bug.cgi?id=1368417 So far, when it comes to 'virsh update-device --config' of disks we are limiting ourselves for just the disk source update and just for CDROMs and floppies. This makes no sense. Especially if you look around and see that we already allow full

Re: [libvirt] [libvirt-php][PATCH] m4: Check for php modules more wisely

2016-09-02 Thread Michal Privoznik
On 02.09.2016 12:27, Christophe Fergeau wrote: > Hey, > > On Thu, Sep 01, 2016 at 01:30:33PM +0200, Michal Privoznik wrote: >> I've got two version of PHP installed on my system, however one >> of them has imagick the other one doesn't. During configure I've >> noticed that wrong assumption has

[libvirt] [libvirt-glib 2/2] object: Fix gvir_connection_create_storage_pool() leak

2016-09-02 Thread Christophe Fergeau
We need to free the string returned by gvir_config_object_to_xml() after using it. --- libvirt-gobject/libvirt-gobject-connection.c | 6 -- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/libvirt-gobject/libvirt-gobject-connection.c

[libvirt] [libvirt-glib 1/2] config: Fix gvir_config_xml_node_to_string() leak

2016-09-02 Thread Christophe Fergeau
If xmlNodeDump() fails, we would be leaking the xmlBuffer we created. This commit ensures we don't return early before this buffer is freed. --- libvirt-gconfig/libvirt-gconfig-helpers.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libvirt-gconfig/libvirt-gconfig-helpers.c

Re: [libvirt] [libvirt-php][PATCH] m4: Check for php modules more wisely

2016-09-02 Thread Christophe Fergeau
Hey, On Thu, Sep 01, 2016 at 01:30:33PM +0200, Michal Privoznik wrote: > I've got two version of PHP installed on my system, however one > of them has imagick the other one doesn't. During configure I've > noticed that wrong assumption has been made. Configure script > wrongly assumed the plugin

Re: [libvirt] [Qemu-devel] [PATCH v7 0/4] Add Mediated device support

2016-09-02 Thread Paolo Bonzini
On 02/09/2016 07:21, Kirti Wankhede wrote: > On 9/2/2016 10:18 AM, Michal Privoznik wrote: > > Okay, maybe I'm misunderstanding something. I just thought that users > > will consult libvirt's nodedev driver (e.g. virsh nodedev-list && virsh > > nodedev-dumpxml $id) to fetch vGPU capabilities and

Re: [libvirt] [PATCH] qemu: fix state change lock held byremoteDispatchDomainBlockJobAbort forever

2016-09-02 Thread Xiubo Li
... + 2016-08-26 07:51:05.685+: 8916: warning : qemuDomainObjBeginJobInternal:1571 : Cannot start job (query, none) for domain instance-1b3a; current job is (modify, none) owned by (8914 remoteDispatchDomainBlockJobAbort, 0 ) for (30s, 0s)

Re: [libvirt] [PATCH v5 0/8] perf: add more perf events support

2016-09-02 Thread Ren, Qiaowei
> -Original Message- > From: John Ferlan [mailto:jfer...@redhat.com] > Sent: Tuesday, August 30, 2016 7:40 PM > To: Ren, Qiaowei ; libvir-list@redhat.com > Subject: Re: [PATCH v5 0/8] perf: add more perf events support > > > > On 08/29/2016 10:00 PM, Ren, Qiaowei

[libvirt] DUID-LLT (DHCPv6 with time-based UUID generation) - how does it work in libvirt?

2016-09-02 Thread Yaniv Kaul
Reading the examples, I'm not sure how to get DUID-LLT working (which is unfortunate, as it seems to be the default of dhclient). I got DUID-LL working, but DUID-LLT has a timestamp - which is generated, AFAIK, by the client - how can the XML contain the correct DUID-LLT entry? The examples in the

Re: [libvirt] [PATCH v7 0/4] Add Mediated device support

2016-09-02 Thread Kirti Wankhede
On 9/2/2016 1:31 AM, Alex Williamson wrote: > On Thu, 1 Sep 2016 23:52:02 +0530 > Kirti Wankhede wrote: > >> Alex, >> Thanks for summarizing the discussion. >> >> On 8/31/2016 9:18 PM, Alex Williamson wrote: >>> On Wed, 31 Aug 2016 15:04:13 +0800 >>> Jike Song