[Spice-devel] [PATCH] clipboard tester

2010-08-23 Thread Alon Levy
I've been using this to test the clipboard. --- tests/clipboard.py | 308 1 files changed, 308 insertions(+), 0 deletions(-) create mode 100755 tests/clipboard.py diff --git a/tests/clipboard.py b/tests/clipboard.py new file mode 100755 inde

[Spice-devel] [PATCH 2/2] client: Don't leak xshm segments

2010-08-23 Thread alexl
From: Alexander Larsson --- client/x11/platform.cpp |8 1 files changed, 8 insertions(+), 0 deletions(-) diff --git a/client/x11/platform.cpp b/client/x11/platform.cpp index 1b54bc3..f80f0dc 100644 --- a/client/x11/platform.cpp +++ b/client/x11/platform.cpp @@ -277,6 +277,14 @@ XIm

[Spice-devel] [PATCH 1/2] Don't try xshm any more if it fails for a permanent reason

2010-08-23 Thread alexl
From: Alexander Larsson This is copied from how Gtk+ detects Xshm failures. --- client/x11/platform.cpp | 16 1 files changed, 16 insertions(+), 0 deletions(-) diff --git a/client/x11/platform.cpp b/client/x11/platform.cpp index 3b9f4af..1b54bc3 100644 --- a/client/x11/platfo

[Spice-devel] [PATCH 0/2] Better xshm handling in client

2010-08-23 Thread alexl
From: Alexander Larsson With this we no longer continue to try using xsh if we get some form of error that is likely to be permanent. Also, we stop leaking xshm segments. Alexander Larsson (2): Don't try xshm any more if it fails for a permanent reason client: Don't leak xshm segments clie

[Spice-devel] Pushed removal of STREAM_TRACE and USE_EXCLUDE_RGN

2010-08-23 Thread Alexander Larsson
I pushed two trivial patches to remove the STREAM_TRACE and USE_EXCLUDE_RGN defines. We didn't really maintain the else cases anyway, and there is no need to use the old untested implementations. red_worker.c is compex as is without #ifdef spaghetti. -- =-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-

Re: [Spice-devel] [PATCH] client: disable clipboard for now (defined out)

2010-08-23 Thread Alexander Larsson
On Mon, 2010-08-23 at 10:34 -0400, Alon Levy wrote: > --- > client/application.cpp |2 ++ > 1 files changed, 2 insertions(+), 0 deletions(-) > > diff --git a/client/application.cpp b/client/application.cpp > index 490cd8c..8eac8f6 100644 > --- a/client/application.cpp > +++ b/client/applicati

[Spice-devel] [PATCH] client: disable clipboard for now (defined out)

2010-08-23 Thread Alon Levy
--- client/application.cpp |2 ++ 1 files changed, 2 insertions(+), 0 deletions(-) diff --git a/client/application.cpp b/client/application.cpp index 490cd8c..8eac8f6 100644 --- a/client/application.cpp +++ b/client/application.cpp @@ -1325,7 +1325,9 @@ void Application::on_app_activated()

Re: [Spice-devel] Fwd: [PATCH 0/8] vdagent update to virtio and clipboard, take 2

2010-08-23 Thread Alexander Larsson
I looked this over a bit, but I don't really know the vdagent codebase. The split up and the general layout of these patches look good to me, and is something we need. So, go ahead and commit this, then we'll at least have a working agent for 0.6. -- =-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-

Re: [Spice-devel] [PATCH] Limiting video streaming to the primary surface. freedesktop bug #28088.

2010-08-23 Thread Yonit Halperin
Fixed and pushed - Original Message - From: "Alexander Larsson" To: "Yonit Halperin" Cc: spice-devel@lists.freedesktop.org Sent: Monday, August 23, 2010 1:15:57 PM (GMT+0200) Auto-Detected Subject: Re: [Spice-devel] [PATCH] Limiting video streaming to the primary surface. freedesktop bu

Re: [Spice-devel] [PATCH] Limiting video streaming to the primary surface. freedesktop bug #28088.

2010-08-23 Thread Alexander Larsson
On Sun, 2010-08-22 at 09:17 +0300, Yonit Halperin wrote: > @@ -3262,10 +3271,12 @@ static inline int red_current_add(RedWorker *worker, > Ring *ring, Drawable *drawa > red_streams_update_clip(worker, drawable); > } else { > #ifdef STREAM_TRACE > -red_detach_streams_behind(w

Re: [Spice-devel] [PATCH 7/8] Store surfaces_used in a bit-array

2010-08-23 Thread Alexander Larsson
On Sun, 2010-08-22 at 15:56 +0300, Yonit Halperin wrote: > On 08/20/2010 09:54 PM, al...@redhat.com wrote: > > From: Alexander Larsson > > > > This is smaller than a byte array, and allows us to skip full > > blocks of 32 ids in one check. > > --- > > Hi, > why not use a linked list, for free surf

Re: [Spice-devel] [PATCH 5/8] Make malloc_sem global

2010-08-23 Thread Alexander Larsson
On Sun, 2010-08-22 at 14:58 +0300, Yonit Halperin wrote: > On 08/20/2010 09:54 PM, al...@redhat.com wrote: > > From: Alexander Larsson > > > > It protects shared data (mspace info) so it needs to be shared. > > --- > > Don't see a problem with making it global, but not sure it is needed. > Can al

Re: [Spice-devel] [PATCH 4/8] Always release free resources when allocating

2010-08-23 Thread Alexander Larsson
On Sun, 2010-08-22 at 14:19 +0300, Yonit Halperin wrote: > On 08/20/2010 09:54 PM, al...@redhat.com wrote: > > From: Alexander Larsson > > > > Leaving around potentially freeable resources when you're about > > to allocate just increases the risk for fragmentation, which is bad, > > especially for

Re: [Spice-devel] [PATCH 0/8] Win32 driver fixes for 0.6.0

2010-08-23 Thread Alexander Larsson
On Sun, 2010-08-22 at 16:51 +0300, Yonit Halperin wrote: > On 08/20/2010 09:54 PM, al...@redhat.com wrote: > > With surfaces_info global we avoid the problem with late resource > > freeing accessing the wrong surface_info[id]. However, isn't there > > also a possible problem where the pdev is not o

[Spice-devel] What do i need?

2010-08-23 Thread Leuenberger Tim
Hello Spice Team For a project from school, I need to test the Spice Software. Can you please help me with answering the following questions? What Software do I need to manage my virtual Images? What Software do I need to bring the images from Server to Client? What operatingsystem do I need on t

Re: [Spice-devel] [PATCH 2/2] server: remove the no longer used glz drawables list that was maintained for each surface.

2010-08-23 Thread Alexander Larsson
On Mon, 2010-08-23 at 09:36 +0300, Yonit Halperin wrote: > From: Yonit Halperin > > --- > server/red_worker.c |8 > 1 files changed, 0 insertions(+), 8 deletions(-) Ack -- =-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-= Alexander Larsson

Re: [Spice-devel] [PATCH 1/2] server: remove unnecessary dependency between surfaces and Glz drawables

2010-08-23 Thread Alexander Larsson
On Mon, 2010-08-23 at 09:36 +0300, Yonit Halperin wrote: > From: Yonit Halperin > > Fixes freedesktop bug #28568 > --- Ack -- =-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-= Alexander LarssonRed Hat, Inc al...@redha