Re: [Spice-devel] spice hardware acceleration

2013-07-05 Thread Dave Airlie
On Fri, Jul 5, 2013 at 12:25 PM, bigclouds bigclo...@163.com wrote: hi i am not farmilar with this eara,i have several question ,trouble you all to answer me. it involves both server and client 1.if hardware accelaration needs special hardware? 2.if i have such hardware, how to use it with

[Spice-devel] [PATCH 2/3] qxl: fix broken cursor hotspot on KMS driver

2013-07-05 Thread Dave Airlie
From: Dave Airlie airl...@redhat.com bugzilla: http://bugzilla.redhat.com/974662 Signed-off-by: Dave Airlie airl...@redhat.com --- configure.ac | 3 ++- src/qxl_drmmode.c | 14 +- 2 files changed, 15 insertions(+), 2 deletions(-) diff --git a/configure.ac b/configure.ac index

[Spice-devel] [PATCH 3/3] qxl: disable composite/a8 surfaces under KMS for now

2013-07-05 Thread Dave Airlie
From: Dave Airlie airl...@redhat.com this just disables composite and a8 surfaces under KMS there are still some issues when enabled. Signed-off-by: Dave Airlie airl...@redhat.com --- src/qxl_uxa.c | 8 1 file changed, 8 insertions(+) diff --git a/src/qxl_uxa.c b/src/qxl_uxa.c index

[Spice-devel] [PATCH 1/3] qxl: disable image cache for KMS

2013-07-05 Thread Dave Airlie
From: Dave Airlie airl...@redhat.com Currently this code can't work with KMS, need to work out how the image cache could be effectively used with KMS enabled. Signed-off-by: Dave Airlie airl...@redhat.com --- src/qxl_image.c | 12 +++- 1 file changed, 7 insertions(+), 5 deletions(-)

[Spice-devel] [PATCH] Use RING_FOREACH_SAFE in red_channel.c functions which are missing it

2013-07-05 Thread David Gibson
Currently, both red_channel_pipes_add_type() and red_channel_pipes_add_empty_msg() use plaing RING_FOREACH() which is not safe versus removals from the ring within the loop body. Although it's rare, such a removal can occur in both cases. In the case of red_channel_pipes_add_type() we have:

Re: [Spice-devel] [PATCH] Use RING_FOREACH_SAFE in red_channel.c functions which are missing it

2013-07-05 Thread Christophe Fergeau
On Fri, Jul 05, 2013 at 05:11:46PM +1000, David Gibson wrote: Currently, both red_channel_pipes_add_type() and red_channel_pipes_add_empty_msg() use plaing RING_FOREACH() which is not safe versus removals from the ring within the loop body. Although it's rare, such a removal can occur in

[Spice-devel] [PATCH spice-gtk] usb-device-manager: Add support for libusb hotplug API

2013-07-05 Thread Hans de Goede
For now this makes the usb-code even more of a #ifdef party (albeit only slightly so). But it does give us (theoretical, untested) usb support on Mac OS X, and once the libusb Windows backend also gets hotplug support, we can bump the required libusb version to a new enough one, and drop both the

Re: [Spice-devel] [PATCH spice-gtk] usb-device-manager: Add support for libusb hotplug API

2013-07-05 Thread Marc-André Lureau
ack - Mensaje original - For now this makes the usb-code even more of a #ifdef party (albeit only slightly so). But it does give us (theoretical, untested) usb support on Mac OS X, and once the libusb Windows backend also gets hotplug support, we can bump the required libusb version

Re: [Spice-devel] [PATCH] Use RING_FOREACH_SAFE in red_channel.c functions which are missing it

2013-07-05 Thread Yonit Halperin
Ack. Thanks! We have recently associated this problem with some opened bugs we have. I believe Uri is working on a patch for a similar fix to the red_pipes.* routines in red_worker. On 07/05/2013 03:11 AM, David Gibson wrote: Currently, both red_channel_pipes_add_type() and

Re: [Spice-devel] Questions regarding Bug 62033 - Means to detect local-only

2013-07-05 Thread Fedor Lyakhov
Hello everyone, I continue working on this bug62033 means to detect local only issue, and want to confirm whether I'm going in the right direction so the result can be committed into Spice. My plan is: 1. Announce capability of DisplayConfig by vdagentd, then receive VDAgentDisplayConfig message

Re: [Spice-devel] [PATCH] Use RING_FOREACH_SAFE in red_channel.c functions which are missing it

2013-07-05 Thread Hans de Goede
Hi, On 07/05/2013 09:11 AM, David Gibson wrote: Currently, both red_channel_pipes_add_type() and red_channel_pipes_add_empty_msg() use plaing RING_FOREACH() which is not safe versus removals from the ring within the loop body. Thanks for the patch, applied and pushed. Regards, Hans

Re: [Spice-devel] Questions regarding Bug 62033 - Means to detect local-only

2013-07-05 Thread Marc-André Lureau
hi On Fri, Jul 5, 2013 at 2:40 PM, Fedor Lyakhov fedor.lyak...@gmail.com wrote: Hello everyone, I continue working on this bug62033 means to detect local only issue, and want to confirm whether I'm going in the right direction so the result can be committed into Spice. My plan is: 1.

Re: [Spice-devel] Questions regarding Bug 62033 - Means to detect local-only

2013-07-05 Thread Hans de Goede
Hi, On 07/05/2013 03:08 PM, Marc-André Lureau wrote: hi On Fri, Jul 5, 2013 at 2:40 PM, Fedor Lyakhov fedor.lyak...@gmail.com wrote: Hello everyone, I continue working on this bug62033 means to detect local only issue, and want to confirm whether I'm going in the right direction so the

[Spice-devel] [PATCH qxl-win] display: apply the fix in fc314927bc48835e to Alpha Bitmaps

2013-07-05 Thread Yonit Halperin
rhbz#968050 In contrast to Microsoft Msdn documentation, the iUniq of a SURFOBJ doesn't always change when the surface changes. However, it seems that the iUniq of the associated color_trans (XLATEOBJ) changes, while its flXlate=XO_TRIVIAL. Since we tried to retrieve the alpha bitmap key only by

Re: [Spice-devel] Questions regarding Bug 62033 - Means to detect local-only

2013-07-05 Thread Fedor Lyakhov
Thanks for your answers! Looks like the issue is more complicated than I originally thought. But I'm still excited to continue working on it. Ok, I'm going to follow Hans' view now. One thing I don't understand: why would we want to apply these 'effects' settings via vdagentd (system level)? For

Re: [Spice-devel] Questions regarding Bug 62033 - Means to detect local-only

2013-07-05 Thread Hans de Goede
Hi, On 07/05/2013 06:07 PM, Fedor Lyakhov wrote: Thanks for your answers! Looks like the issue is more complicated than I originally thought. But I'm still excited to continue working on it. Ok, I'm going to follow Hans' view now. Good. One thing I don't understand: why would we want to

Re: [Spice-devel] [PATCH] Use RING_FOREACH_SAFE in red_channel.c functions which are missing it

2013-07-05 Thread David Gibson
On Fri, Jul 05, 2013 at 08:35:21AM -0400, Yonit Halperin wrote: Ack. Thanks! We have recently associated this problem with some opened bugs we have. I believe Uri is working on a patch for a similar fix to the red_pipes.* routines in red_worker.