>
> From: Marc-André Lureau
>
> ---
> NEWS | 2 +-
> server/Makefile.am | 62 +++
> server/{reds_sw_canvas.c => canvas.c} | 10 +-
> server/{reds_sw_canvas.h => canvas.h} | 9 +-
>
>
> From: Marc-André Lureau
>
> ---
> server/display-channel.c | 2 ++
> server/red_worker.c | 10 ++
> server/red_worker.h | 1 +
> 3 files changed, 13 insertions(+)
>
> diff --git a/server/display-channel.c b/server/display-channel.c
> index d168190..6b9affc 100644
> ---
>
> From: Yonit Halperin
>
> (1) receive timeout as a parameter.
> (2) add a return value and pass the handling
> of failures to the calling routine.
> ---
> server/red_channel.c | 4 +++-
> 1 file changed, 3 insertions(+), 1 deletion(-)
>
> diff --git a/server/red_channel.c b/server/red_c
>
> From: Marc-André Lureau
>
> ---
> server/red_worker.c | 16
> 1 file changed, 8 insertions(+), 8 deletions(-)
>
> diff --git a/server/red_worker.c b/server/red_worker.c
> index df1a2f1..da2889c 100644
> --- a/server/red_worker.c
> +++ b/server/red_worker.c
> @@ -87,7 +87,7
Hi
- Original Message -
> Hi,
>
> On Tue, Dec 01, 2015 at 07:19:58AM -0500, Marc-André Lureau wrote:
> > - Original Message -
> > > Hi,
> > >
> > > On Tue, Dec 01, 2015 at 06:41:47AM -0500, Marc-André Lureau wrote:
> > > > - Original Message -
> > > > > Hi
> > > > >
> > >
Hi,
On Tue, Dec 01, 2015 at 07:19:58AM -0500, Marc-André Lureau wrote:
> - Original Message -
> > Hi,
> >
> > On Tue, Dec 01, 2015 at 06:41:47AM -0500, Marc-André Lureau wrote:
> > > - Original Message -
> > > > Hi
> > > >
> > > > - Original Message -
> > > > > usbredir on
>
> From: Marc-André Lureau
>
> ---
> server/dispatcher.c | 25 +
> server/dispatcher.h | 2 ++
> server/red_worker.c | 40
> 3 files changed, 39 insertions(+), 28 deletions(-)
>
> diff --git a/server/dispatcher.c b/server/dispa
On Po, 2015-11-30 at 12:56 +0100, Christophe Fergeau wrote:
> spice-common has been duplicating glib logging methods for a long while.
> Now that spice-common is depending on glib, it's more consistent to use
> glib logging too. However, the code base is still using spice logging
> functions.
> Thi
>
> From: Marc-André Lureau
>
> ---
> server/red_worker.c | 5 -
> 1 file changed, 4 insertions(+), 1 deletion(-)
>
> diff --git a/server/red_worker.c b/server/red_worker.c
> index 2e3f522..9145875 100644
> --- a/server/red_worker.c
> +++ b/server/red_worker.c
> @@ -87,6 +87,7 @@ struct Re
- Original Message -
> Hi,
>
> On Tue, Dec 01, 2015 at 06:41:47AM -0500, Marc-André Lureau wrote:
> > - Original Message -
> > > Hi
> > >
> > > - Original Message -
> > > > usbredir on debug level is highly verbose. That does not help much when
> > > > we are trying to de
On Mon, Nov 30, 2015 at 02:22:54PM -0500, Frediano Ziglio wrote:
> >
> > It's causing issues with selinux. The same is achieved through
> > abrt/cores/...
> >
> > Acked-by: Jonathon Jongsma
> [...]
> Patch looks good but what about the interaction with Qemu/RHEV-M/whatever?
>
> I would wait for
On Mon, Nov 30, 2015 at 02:31:43PM -0500, Frediano Ziglio wrote:
> > +g_setenv("G_MESSAGES_DEBUG", "all", FALSE);
>
> Do not set environment in a library.
> This affects any usage of g_ logging macros in the process (and
> possibly sub processes).
Yes, but I don't think there's a
From: Marc-André Lureau
---
server/red_worker.c | 5 -
1 file changed, 4 insertions(+), 1 deletion(-)
diff --git a/server/red_worker.c b/server/red_worker.c
index 2e3f522..9145875 100644
--- a/server/red_worker.c
+++ b/server/red_worker.c
@@ -87,6 +87,7 @@ struct RedWorker {
int driver
From: Marc-André Lureau
---
server/display-channel.c | 2 ++
server/red_worker.c | 10 ++
server/red_worker.h | 1 +
3 files changed, 13 insertions(+)
diff --git a/server/display-channel.c b/server/display-channel.c
index d168190..6b9affc 100644
--- a/server/display-channel.
From: Marc-André Lureau
Clean up, more extensible.
Avoid server hanging when no client are connected.
---
server/Makefile.am | 2 -
server/red_worker.c| 395 -
server/red_worker.h| 1 +
server/spice_timer_queue.c | 273
From: Alon Levy
---
server/red_worker.c | 8
server/reds.c | 2 +-
2 files changed, 5 insertions(+), 5 deletions(-)
diff --git a/server/red_worker.c b/server/red_worker.c
index 1859d2a..0cd2c72 100644
--- a/server/red_worker.c
+++ b/server/red_worker.c
@@ -174,7 +174,7 @@ static
From: Alon Levy
---
server/tests/test_display_base.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/server/tests/test_display_base.c b/server/tests/test_display_base.c
index c2c5705..575c704 100644
--- a/server/tests/test_display_base.c
+++ b/server/tests/test_display_base.c
From: Marc-André Lureau
---
server/dispatcher.c | 25 +
server/dispatcher.h | 2 ++
server/red_worker.c | 40
3 files changed, 39 insertions(+), 28 deletions(-)
diff --git a/server/dispatcher.c b/server/dispatcher.c
index d6c03ca
From: Marc-André Lureau
This fixes the following scary racy corruption after glib loop switch:
==28173==
==28173== Debugger has detached. Valgrind regains control. We
continue.
==28173== Invalid read of size 8
==28173==at 0x4C7871E: reds_stream_read (reds.c:4521)
==28173==by 0x4C2F9D7:
From: Marc-André Lureau
---
NEWS | 2 +-
server/Makefile.am | 62 +++
server/{reds_sw_canvas.c => canvas.c} | 10 +-
server/{reds_sw_canvas.h => canvas.h} | 9 +-
server/{char_device
From: Yonit Halperin
(1) receive timeout as a parameter.
(2) add a return value and pass the handling
of failures to the calling routine.
---
server/red_channel.c | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/server/red_channel.c b/server/red_channel.c
index 948d354.
From: Marc-André Lureau
---
server/red_worker.c | 16
1 file changed, 8 insertions(+), 8 deletions(-)
diff --git a/server/red_worker.c b/server/red_worker.c
index df1a2f1..da2889c 100644
--- a/server/red_worker.c
+++ b/server/red_worker.c
@@ -87,7 +87,7 @@ struct RedWorker {
Signed-off-by: Marc-André Lureau
Signed-off-by: Frediano Ziglio
---
server/cursor-channel.c | 2 +-
server/red_worker.c | 4 ++--
server/red_worker.h | 2 +-
3 files changed, 4 insertions(+), 4 deletions(-)
diff --git a/server/cursor-channel.c b/server/cursor-channel.c
index 9d72299..84
Signed-off-by: Marc-André Lureau
Signed-off-by: Frediano Ziglio
---
server/display-channel.c | 185 +++
server/display-channel.h | 15
server/red_worker.c | 172 ---
3 files changed, 185 insertions(+),
Allocation on stack is sufficient and code is more similar to
QXL_CMD_MESSAGE and QXL_CMD_UPDATE cases.
Signed-off-by: Frediano Ziglio
---
server/red_worker.c | 14 ++
1 file changed, 6 insertions(+), 8 deletions(-)
diff --git a/server/red_worker.c b/server/red_worker.c
index ee26b6
Make possible to safely call display_channel_drawable_unref straight
forward to get_drawable call.
Signed-off-by: Frediano Ziglio
---
server/display-channel.c | 20 +---
1 file changed, 9 insertions(+), 11 deletions(-)
diff --git a/server/display-channel.c b/server/display-chann
- remove some red_ prefix;
- move red_drawable->self_bitmap check outside handle_self_bitmap;
- move update check outside red_get_area (renamed surface_read_bits);
- rename depend_on_surface_id argument to surface_id;
- rename success variable to add_to_pipe.
Signed-off-by: Marc-André Lureau
Sign
Signed-off-by: Marc-André Lureau
Signed-off-by: Frediano Ziglio
---
server/dcc-send.c | 8 ++--
1 file changed, 2 insertions(+), 6 deletions(-)
diff --git a/server/dcc-send.c b/server/dcc-send.c
index 2340252..30303e7 100644
--- a/server/dcc-send.c
+++ b/server/dcc-send.c
@@ -2012,14 +2012,
Changes:
- rebased on current master;
- removed merged patches;
- split and change "display: move more logic in add_drawable()";
- split "misc cleanups";
- added some patches.
Some patches on discussion were not posted.
I added all patches that was missing from first part of refactory.
Looks lon
From: Yonit Halperin
---
server/dcc.h | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/server/dcc.h b/server/dcc.h
index 14981ca..7b8a3b8 100644
--- a/server/dcc.h
+++ b/server/dcc.h
@@ -204,7 +204,8 @@ void dcc_release_item
(D
Signed-off-by: Marc-André Lureau
Signed-off-by: Frediano Ziglio
---
server/display-channel.c | 60 +
server/display-channel.h | 5
server/red_worker.c | 64 +++-
3 files changed, 68 insertions(+),
Signed-off-by: Frediano Ziglio
---
server/display-channel.c | 25 +
server/display-channel.h | 7 ++-
server/red_worker.c | 21 ++---
3 files changed, 25 insertions(+), 28 deletions(-)
diff --git a/server/display-channel.c b/server/display-channe
Hi,
On Tue, Dec 01, 2015 at 06:41:47AM -0500, Marc-André Lureau wrote:
> - Original Message -
> > Hi
> >
> > - Original Message -
> > > usbredir on debug level is highly verbose. That does not help much when
> > > we are trying to debug the spice-gtk side of usbredir.
> > >
> > > T
On 11/27/2015 11:54 AM, Frediano Ziglio wrote:
What about:
display: move checks inside display_channel_wait_for_migrate_data
Instead of relaying on the caller to use red_channel_waits_for_migrate_data
to check if there are client waiting for migration data check inside the
function. This mak
- Original Message -
> Hi
>
> - Original Message -
> > usbredir on debug level is highly verbose. That does not help much when
> > we are trying to debug the spice-gtk side of usbredir.
> >
> > This patch introduces SPICE_DEBUG_USBREDIR env var to set the log level
> > for usbre
Hi
- Original Message -
> usbredir on debug level is highly verbose. That does not help much when
> we are trying to debug the spice-gtk side of usbredir.
>
> This patch introduces SPICE_DEBUG_USBREDIR env var to set the log level
> for usbredir side. If this is not set, usbredir will beh
usbredir on debug level is highly verbose. That does not help much when
we are trying to debug the spice-gtk side of usbredir.
This patch introduces SPICE_DEBUG_USBREDIR env var to set the log level
for usbredir side. If this is not set, usbredir will behave the same way
as it was by checking SPIC
37 matches
Mail list logo
