Re: [Spice-devel] Getting involved with implementation of remote spice for 3D accelerated VMs

2017-01-09 Thread Victor Toso
Hi, On Mon, Jan 09, 2017 at 04:27:35AM -0500, Frediano Ziglio wrote: > The patches for Virgl are working and require only tweaks. We wanted > to have some working PoC for all next 3D software, that include GPU > passtrough to make sure that the Virgl doesn't complicate the overall > design. We

Re: [Spice-devel] [spice-gtk 1/3] ssl: Stop creating our own X509_LOOKUP_METHOD

2017-01-09 Thread Pavel Grunt
On Thu, 2016-12-22 at 17:03 +0100, Christophe Fergeau wrote: > From: Sebastian Andrzej Siewior > > OpenSSL 1.1.0 does not seem to provide API to do that anymore. > > There is no need to create a custom lookup to begin with. This > method > here has no callbacks

Re: [Spice-devel] [spice-gtk 2/3] ssl: Rework our custom BIO type

2017-01-09 Thread Pavel Grunt
On Thu, 2016-12-22 at 17:03 +0100, Christophe Fergeau wrote: > From: Sebastian Andrzej Siewior > > This commit changes to an actual new BIO method rather than reusing > an > existing BIO method, and overriding only the fields that we need. > The approach before this

Re: [Spice-devel] Getting involved with implementation of remote spice for 3D accelerated VMs

2017-01-09 Thread Frediano Ziglio
> > Hi, > > On Mon, Jan 09, 2017 at 04:27:35AM -0500, Frediano Ziglio wrote: > > The patches for Virgl are working and require only tweaks. We wanted > > to have some working PoC for all next 3D software, that include GPU > > passtrough to make sure that the Virgl doesn't complicate the overall

Re: [Spice-devel] [spice-gtk] streaming: Allows protocol to have more codecs

2017-01-09 Thread Pavel Grunt
Ack Pavel On Mon, 2017-01-09 at 09:51 +, Frediano Ziglio wrote: > The source file contains a list of fixed codecs for streaming. > This list is tested to include all codecs from spice-protocol. > If spice-protocol is extended to add more codecs this file starts > failing to compile due to

Re: [Spice-devel] [PATCH v2 2/2] Quiet uninitialized variable warning.

2017-01-09 Thread Pavel Grunt
Hi Michal, On Fri, 2017-01-06 at 14:25 +0100, Michal Suchanek wrote: > Signed-off-by: Michal Suchanek > --- >  src/vdagentd/vdagentd.c | 1 + >  1 file changed, 1 insertion(+) > > diff --git a/src/vdagentd/vdagentd.c b/src/vdagentd/vdagentd.c > index 991514e..60a866e 100644 >

Re: [Spice-devel] Getting involved with implementation of remote spice for 3D accelerated VMs

2017-01-09 Thread Frediano Ziglio
The patches for Virgl are working and require only tweaks. We wanted to have some working PoC for all next 3D software, that include GPU passtrough to make sure that the Virgl doesn't complicate the overall design. We plan to have these definitive PoCs in a couple of months. About video

[Spice-devel] [spice-gtk] streaming: Allows protocol to have more codecs

2017-01-09 Thread Frediano Ziglio
The source file contains a list of fixed codecs for streaming. This list is tested to include all codecs from spice-protocol. If spice-protocol is extended to add more codecs this file starts failing to compile due to missing codecs. This patch allow to compile supporting less codecs than the ones

Re: [Spice-devel] [PATCH v2 1/2] Do endian swapping.

2017-01-09 Thread Pavel Grunt
Hi Michal, I have some style comments On Fri, 2017-01-06 at 14:25 +0100, Michal Suchanek wrote: > This allows running big endian and little endian guest side by side > using > cut & paste between them. > > There is a general design idea that swapping should come as close to >

Re: [Spice-devel] [ovirt-users] Video performance

2017-01-09 Thread Victor Toso
Hi, > Hi, > > I have installed ovirt engine 4.0.5 on centos 7.3. I installed > ovirt-node on a separate phyical system and added it to ovirt-engine. > I installed windows 7 in a VM on the node. I installed all the virtio > drivers in the guest. The protocol used by me to access VM is Spice >

[Spice-devel] [PATCH v3 2/2] Quiet uninitialized variable warning.

2017-01-09 Thread Michal Suchanek
Signed-off-by: Michal Suchanek --- src/vdagentd/vdagentd.c | 1 + 1 file changed, 1 insertion(+) diff --git a/src/vdagentd/vdagentd.c b/src/vdagentd/vdagentd.c index 4cac473..f759b4c 100644 --- a/src/vdagentd/vdagentd.c +++ b/src/vdagentd/vdagentd.c @@ -334,6 +334,7 @@ static

[Spice-devel] [PATCH v3 1/2] Do endian swapping.

2017-01-09 Thread Michal Suchanek
This allows running big endian and little endian guest side by side using cut & paste between them. There is a general design idea that swapping should come as close to virtio_read/virtio_write as possible. In particular, the protocol between vdagent and vdagentd is guest-specific and in native

Re: [Spice-devel] [PATCH v10 01/11] sound: Convert SndChannelClient to GObject

2017-01-09 Thread Christophe Fergeau
On Fri, Jan 06, 2017 at 04:30:33AM -0500, Frediano Ziglio wrote: > I think the main problem reviewing this patch is that people keeps > thinking that this big patch is the results of many small changes. > As explained is not so the patch is big and it's better to consider > the overall code before

Re: [Spice-devel] [PATCH v2 2/2] Quiet uninitialized variable warning.

2017-01-09 Thread Michal Suchánek
On Mon, 09 Jan 2017 10:51:34 +0100 Pavel Grunt wrote: Hello, > Hi Michal, > > On Fri, 2017-01-06 at 14:25 +0100, Michal Suchanek wrote: > > Signed-off-by: Michal Suchanek > > --- > >  src/vdagentd/vdagentd.c | 1 + > >  1 file changed, 1 insertion(+) > >

Re: [Spice-devel] [spice-gtk 3/3] ssl: Use accessors rather than direct struct access

2017-01-09 Thread Pavel Grunt
On Thu, 2016-12-22 at 17:03 +0100, Christophe Fergeau wrote: > From: Sebastian Andrzej Siewior > > In OpenSSL 1.1.0, the struct fields are private so we can no longer > directly access them. > > The accessors are not available in previous OpenSSL releases, so we > need

Re: [Spice-devel] Getting involved with implementation of remote spice for 3D accelerated VMs

2017-01-09 Thread Victor Toso
Hi, On Mon, Jan 09, 2017 at 11:46:56AM -0500, Frediano Ziglio wrote: > > > > Hi, > > > > On Mon, Jan 09, 2017 at 04:27:35AM -0500, Frediano Ziglio wrote: > > > The patches for Virgl are working and require only tweaks. We wanted > > > to have some working PoC for all next 3D software, that

Re: [Spice-devel] Getting involved with implementation of remote spice for 3D accelerated VMs

2017-01-09 Thread Victor Toso
Hi, On Mon, Jan 09, 2017 at 05:20:10PM +0100, Behrooz Shabani wrote: > Thanks for the info Victor. > > I will try to set it up and because I am new to the code base and have > no idea where to start, I will ping you on IRC for help if you don't > mind (assuming you're `toso`). That's me, yes :)

Re: [Spice-devel] Getting involved with implementation of remote spice for 3D accelerated VMs

2017-01-09 Thread Behrooz Shabani
Thanks for the info Victor. I will try to set it up and because I am new to the code base and have no idea where to start, I will ping you on IRC for help if you don't mind (assuming you're `toso`). On Mon, Jan 9, 2017 at 5:02 PM, Victor Toso wrote: > Hi, > > On Mon, Jan