Re: [Spice-devel] Can we directly redir usb hub?

2013-06-25 Thread fpemud
On 06/25/2013 02:00 AM, Hans de Goede wrote: > Hi, > > On 06/24/2013 11:40 AM, 邵鄭叡 wrote: >> Hi all, >> maybe I should post this message to [usbredir and qemu devel list] , >> but I just didnt find it :) >> >> By default, spice-gtk filters out 'usb hubs' from redir list, and >> only show 'non-hub n

Re: [Spice-devel] [PATCH] red_worker: fix for stuck display_channel over WAN (jpeg_enabled=true)

2013-06-25 Thread Yonit Halperin
On 06/25/2013 02:50 PM, Marc-André Lureau wrote: Hi - Mensaje original - The image descriptor flags shouldn't be copied as is from the flags that were set by the driver. Specifically, the CACHE_ME flag shouldn't be copied, since it is possible that (a) the image won't be cached (b) the

Re: [Spice-devel] [PATCH] red_worker: fix for stuck display_channel over WAN (jpeg_enabled=true)

2013-06-25 Thread Marc-André Lureau
Hi - Mensaje original - > The image descriptor flags shouldn't be copied as is from the flags that > were set by the driver. Specifically, the CACHE_ME flag shouldn't be copied, > since it is possible that (a) the image won't be cached (b) the image > is already cached, but in its lossy ve

[Spice-devel] [PATCH] red_worker: fix for stuck display_channel over WAN (jpeg_enabled=true)

2013-06-25 Thread Yonit Halperin
The image descriptor flags shouldn't be copied as is from the flags that were set by the driver. Specifically, the CACHE_ME flag shouldn't be copied, since it is possible that (a) the image won't be cached (b) the image is already cached, but in its lossy version, and we may want to set the bit for

Re: [Spice-devel] [PATCH qxl-win v2] display: fix deadlock when dbg_level >= 15

2013-06-25 Thread Uri Lublin
On 06/25/2013 12:01 AM, Yonit Halperin wrote: DebugPrintV first locks print_sem, and then locks io_sem. async_io, locks io_sem. In ordr to avoid a deadlock, DebugPrintV MUSTN'T be called when io_sem is locked. I also moved the locking of io_sem, so that reading pdev->use_async will also be protec

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

2013-06-25 Thread Fedor Lyakhov
Hi, 1. Yes, exactly, I meant to use this option. 2. And yes, that's my point - why vdagent should tweak settings for all desktops (it is impossible to support all desktops) - but we need to provide some means for desktop developers to take in account the DE is viewed via Spice (local or remote cl

Re: [Spice-devel] [PATCH spice-gtk] channel-main: Convert text line-endings if necessary (rhbz#752350)

2013-06-25 Thread Marc-André Lureau
Hi On Mon, Jun 24, 2013 at 2:31 PM, Hans de Goede wrote: > Signed-off-by: Hans de Goede > --- > gtk/channel-main.c | 73 > +++--- > 1 file changed, 69 insertions(+), 4 deletions(-) > > diff --git a/gtk/channel-main.c b/gtk/channel-main.c > index

Re: [Spice-devel] why getScanCode() will not work for keyboard input

2013-06-25 Thread Daniel P. Berrange
On Tue, Jun 25, 2013 at 01:26:46PM +0200, Marc-André Lureau wrote: > Hi > > On Mon, Jun 24, 2013 at 8:08 AM, i iordanov wrote: > > Hi Marc-Andre, > > > > (And anybody else who wants to weigh in on this). After some research, > > reading and experimentation, here is what I've discovered. > > > > g

Re: [Spice-devel] why getScanCode() will not work for keyboard input

2013-06-25 Thread Marc-André Lureau
Hi On Mon, Jun 24, 2013 at 8:08 AM, i iordanov wrote: > Hi Marc-Andre, > > (And anybody else who wants to weigh in on this). After some research, > reading and experimentation, here is what I've discovered. > > getScanCode() works only for physical devices (e.g. USB keyboards, bluetooth > keyboar

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

2013-06-25 Thread Marc-André Lureau
Hi - Mensaje original - > OK, so first implementation will work via --spice-disable-effects option. As > far as I understand, this user-provided option flags should already be > available at the agent, need to handle appropriate message as in Windows > vdagent, correct? There is already: