Re: [Spice-devel] Help with solving a thread safety issue

2014-09-08 Thread Jeremy White
On 09/08/2014 03:33 PM, nicolas prochazka wrote: sorry, nothing , a lot of video frames seems to be drop. The issue that my patch would solve, if you had encountered this problem, would be random crashes and hangs. There was some sense that you lost sound after a few minutes. Is that proble

Re: [Spice-devel] Help with solving a thread safety issue

2014-09-08 Thread nicolas prochazka
sorry, nothing , a lot of video frames seems to be drop. 2014-09-08 20:39 GMT+02:00 Jeremy White : > Hi Nicolas, > > On 09/08/2014 12:56 PM, nicolas prochazka wrote: > >> Hello, >> I've done differents tests, with the differents combinaisons between >> qemu X and spice-server, >> I can say that

Re: [Spice-devel] Help with solving a thread safety issue

2014-09-08 Thread Jeremy White
Hi Nicolas, On 09/08/2014 12:56 PM, nicolas prochazka wrote: Hello, I've done differents tests, with the differents combinaisons between qemu X and spice-server, I can say that is since video adaptative streaming, the video streaming is bad compare to the old version (spice 0.12.3 / qemu 1.7

Re: [Spice-devel] Help with solving a thread safety issue

2014-09-08 Thread nicolas prochazka
Hello, I've done differents tests, with the differents combinaisons between qemu X and spice-server, I can say that is since video adaptative streaming, the video streaming is bad compare to the old version (spice 0.12.3 / qemu 1.7 ). When i set SPICE_DISABLE_ADAPTIVE_STREAMING=Y , my tests sho

Re: [Spice-devel] Help with solving a thread safety issue

2014-09-08 Thread Jeremy White
On Sat, Sep 06, 2014 at 08:59:47AM -0500, Jeremy White wrote: I think there actually is the same issue when running within QEMU: sound is emitted from a QEMU thread: Is there a bug or someone that is working on that issue? I'd be curious to see if my patch resolves the problem for them. I've

Re: [Spice-devel] [PATCH 2/9] build-sys: check for spicy-screenshot

2014-09-08 Thread Christophe Fergeau
On Mon, Nov 18, 2013 at 05:33:38AM -0500, Marc-André Lureau wrote: > It's an old patch, I should have modified those as well: Something similar to the initial patch has been pushed as 686750e4 > > server/tests/test_multiple.py:parser.add_argument('--client', default='spicy') > tests/migrate.py:

Re: [Spice-devel] [PATCH 3/9] clean-up: remove unused function

2014-09-08 Thread Christophe Fergeau
ACK. On Mon, Nov 18, 2013 at 11:28:27AM +0100, Marc-André Lureau wrote: > From: Marc-André Lureau > > --- > server/reds.c | 5 - > server/reds.h | 1 - > 2 files changed, 6 deletions(-) > > diff --git a/server/reds.c b/server/reds.c > index 146dfc7..6197547 100644 > --- a/server/reds.c > +

Re: [Spice-devel] [PATCH 1/9] reds: lookup corresponding channel id

2014-09-08 Thread Christophe Fergeau
ACK, but please mention this bug in the commit log. Christophe On Thu, Aug 28, 2014 at 04:22:43PM +0200, Marc-André Lureau wrote: > ping > > this patch solves the assert from bug > https://bugzilla.redhat.com/show_bug.cgi?id=1058625 > > > On Mon, Nov 18, 2013 at 11:28 AM, Marc-André Lureau < >

Re: [Spice-devel] Help with solving a thread safety issue

2014-09-08 Thread Christophe Fergeau
On Mon, Sep 08, 2014 at 11:26:58AM +0200, Christophe Fergeau wrote: > Hey, > > On Sat, Sep 06, 2014 at 08:59:47AM -0500, Jeremy White wrote: > > >I think there actually is the same issue when running within QEMU: sound > > >is emitted from a QEMU thread: > > > > Is there a bug or someone that is

Re: [Spice-devel] [vdagent-linux 1/2] randr: remove monitors.xml on auto-configuration

2014-09-08 Thread Marc-André Lureau
ping On Wed, Aug 27, 2014 at 7:22 PM, Marc-André Lureau < marcandre.lur...@redhat.com> wrote: > From: Marc-Andre Lureau > > GNOME will restore monitors.xml configuration whenever the timestamp > "config > change". The "change" timestamp is the last user applied > configuration, whereas the "con

Re: [Spice-devel] [vdagent-linux v2] randr: handle XRRScreenChangeNotifyEvent

2014-09-08 Thread Marc-André Lureau
ping On Wed, Aug 27, 2014 at 8:51 PM, Marc-André Lureau < marcandre.lur...@redhat.com> wrote: > From: Marc-Andre Lureau > > Make sure vdagent follows the current monitor configuration. This > solves a number of issue where the pointer input is off, because the > uinput tablet isn't reconfigure

Re: [Spice-devel] Help with solving a thread safety issue

2014-09-08 Thread Christophe Fergeau
Hey, On Sat, Sep 06, 2014 at 08:59:47AM -0500, Jeremy White wrote: > >I think there actually is the same issue when running within QEMU: sound > >is emitted from a QEMU thread: > > Is there a bug or someone that is working on that issue? I'd be curious to > see if my patch resolves the problem f

Re: [Spice-devel] [vdagent-linux] data: remove rsyslog config files

2014-09-08 Thread Christophe Fergeau
Hey, Yup, makes sense to me, though I can't parse the commit log. A short "It's possible to filter spice-vdagent/spice-vdagentd log messages out of /var/log/messages with journalctl SYSLOG_IDENTIFIER=spice-vdagent SYSLOG_IDENTIFIER=spice-vdagentd" would do. ACK, and hopefully we will get bug repor