Re: [libvirt] [PATCHv5 00/13] qemu: allow disabling certain virtio revisions

2016-09-20 Thread Cornelia Huck
On Mon, 19 Sep 2016 22:07:13 +0300 "Michael S. Tsirkin" wrote: > On Wed, Sep 07, 2016 at 09:38:04PM +0200, Sascha Silbe wrote: > > Dear Laine, > > > > Laine Stump writes: > > > > > On 09/07/2016 02:35 PM, Sascha Silbe wrote: > > >> "Daniel P. Berrange" writes: > > >> [...] > > >>>

Re: [libvirt] [PATCHv5 00/13] qemu: allow disabling certain virtio revisions

2016-09-19 Thread Michael S. Tsirkin
On Wed, Sep 07, 2016 at 09:38:04PM +0200, Sascha Silbe wrote: > Dear Laine, > > Laine Stump writes: > > > On 09/07/2016 02:35 PM, Sascha Silbe wrote: > >> "Daniel P. Berrange" writes: > >> [...] > >>>== QEMU virtio > >>> == QEMU virtio + disable-legacy > >> What would this d

Re: [libvirt] [PATCHv5 00/13] qemu: allow disabling certain virtio revisions

2016-09-19 Thread Boris Fiuczynski
On 09/13/2016 05:16 PM, Laine Stump wrote: On 09/13/2016 10:43 AM, Cornelia Huck wrote: What would this do for devices using the virtio-ccw transport? From libvirt's point of view, the option "disable-legacy=on" would be added to the device's commandline argument. Which would break s390x gu

Re: [libvirt] [PATCHv5 00/13] qemu: allow disabling certain virtio revisions

2016-09-13 Thread Laine Stump
On 09/13/2016 10:43 AM, Cornelia Huck wrote: [I've browsed through the thread a bit, but as I'm not a libvirt developer I may be missing some basic things] On Wed, 7 Sep 2016 16:34:17 -0400 Laine Stump wrote: On 09/07/2016 03:38 PM, Sascha Silbe wrote: Dear Laine, Laine Stump writes: On

Re: [libvirt] [PATCHv5 00/13] qemu: allow disabling certain virtio revisions

2016-09-13 Thread Cornelia Huck
[I've browsed through the thread a bit, but as I'm not a libvirt developer I may be missing some basic things] On Wed, 7 Sep 2016 16:34:17 -0400 Laine Stump wrote: > On 09/07/2016 03:38 PM, Sascha Silbe wrote: > > Dear Laine, > > > > Laine Stump writes: > > > >> On 09/07/2016 02:35 PM, Sascha S

Re: [libvirt] [PATCHv5 00/13] qemu: allow disabling certain virtio revisions

2016-09-07 Thread Laine Stump
On 09/07/2016 03:38 PM, Sascha Silbe wrote: Dear Laine, Laine Stump writes: On 09/07/2016 02:35 PM, Sascha Silbe wrote: "Daniel P. Berrange" writes: [...] == QEMU virtio == QEMU virtio + disable-legacy What would this do for devices using the virtio-ccw transport?

Re: [libvirt] [PATCHv5 00/13] qemu: allow disabling certain virtio revisions

2016-09-07 Thread Sascha Silbe
Dear Laine, Laine Stump writes: > On 09/07/2016 02:35 PM, Sascha Silbe wrote: >> "Daniel P. Berrange" writes: >> [...] >>>== QEMU virtio >>> == QEMU virtio + disable-legacy >> What would this do for devices using the virtio-ccw transport? > > From libvirt's point of view, t

Re: [libvirt] [PATCHv5 00/13] qemu: allow disabling certain virtio revisions

2016-09-07 Thread Laine Stump
On 09/07/2016 02:35 PM, Sascha Silbe wrote: Dear Daniel, "Daniel P. Berrange" writes: [...] == QEMU virtio == QEMU virtio + disable-legacy What would this do for devices using the virtio-ccw transport? From libvirt's point of view, the option "disable-legacy=on" would be

Re: [libvirt] [PATCHv5 00/13] qemu: allow disabling certain virtio revisions

2016-09-07 Thread Laine Stump
On 09/07/2016 12:40 PM, Daniel P. Berrange wrote: On Wed, Sep 07, 2016 at 05:52:02PM +0200, Peter Krempa wrote: On Wed, Sep 07, 2016 at 16:13:57 +0100, Daniel Berrange wrote: On Wed, Sep 07, 2016 at 05:10:01PM +0200, Peter Krempa wrote: On Wed, Aug 24, 2016 at 00:20:42 +0200, Ján Tomko wrote:

Re: [libvirt] [PATCHv5 00/13] qemu: allow disabling certain virtio revisions

2016-09-07 Thread Sascha Silbe
Dear Daniel, "Daniel P. Berrange" writes: [...] > == QEMU virtio >== QEMU virtio + disable-legacy What would this do for devices using the virtio-ccw transport? Sascha -- Softwareentwicklung Sascha Silbe, Niederhofenstraße 5/1, 71229 Leonberg https://se-silbe.de/ USt-IdNr. D

Re: [libvirt] [PATCHv5 00/13] qemu: allow disabling certain virtio revisions

2016-09-07 Thread Daniel P. Berrange
On Wed, Sep 07, 2016 at 05:52:02PM +0200, Peter Krempa wrote: > On Wed, Sep 07, 2016 at 16:13:57 +0100, Daniel Berrange wrote: > > On Wed, Sep 07, 2016 at 05:10:01PM +0200, Peter Krempa wrote: > > > On Wed, Aug 24, 2016 at 00:20:42 +0200, Ján Tomko wrote: > > > > https://bugzilla.redhat.com/show_bu

Re: [libvirt] [PATCHv5 00/13] qemu: allow disabling certain virtio revisions

2016-09-07 Thread Peter Krempa
On Wed, Sep 07, 2016 at 16:13:57 +0100, Daniel Berrange wrote: > On Wed, Sep 07, 2016 at 05:10:01PM +0200, Peter Krempa wrote: > > On Wed, Aug 24, 2016 at 00:20:42 +0200, Ján Tomko wrote: > > > https://bugzilla.redhat.com/show_bug.cgi?id=1227354 [...] > > Obviously making the user responsible for

Re: [libvirt] [PATCHv5 00/13] qemu: allow disabling certain virtio revisions

2016-09-07 Thread Daniel P. Berrange
On Wed, Sep 07, 2016 at 05:10:01PM +0200, Peter Krempa wrote: > On Wed, Aug 24, 2016 at 00:20:42 +0200, Ján Tomko wrote: > > https://bugzilla.redhat.com/show_bug.cgi?id=1227354 > > v1: https://www.redhat.com/archives/libvir-list/2016-July/msg01235.html > > v2: https://www.redhat.com/archives/libvir

Re: [libvirt] [PATCHv5 00/13] qemu: allow disabling certain virtio revisions

2016-09-07 Thread Peter Krempa
On Wed, Aug 24, 2016 at 00:20:42 +0200, Ján Tomko wrote: > https://bugzilla.redhat.com/show_bug.cgi?id=1227354 > v1: https://www.redhat.com/archives/libvir-list/2016-July/msg01235.html > v2: https://www.redhat.com/archives/libvir-list/2016-August/msg00412.html > * probe for the qemu capability >

[libvirt] [PATCHv5 00/13] qemu: allow disabling certain virtio revisions

2016-08-23 Thread Ján Tomko
https://bugzilla.redhat.com/show_bug.cgi?id=1227354 v1: https://www.redhat.com/archives/libvir-list/2016-July/msg01235.html v2: https://www.redhat.com/archives/libvir-list/2016-August/msg00412.html * probe for the qemu capability * add the attribute to virtio1-only devices such as virtio-gpu