[Spice-devel] [PATCH] spice-vmc: add a reset handler used to unregister spice interface (and trigger server mouse)

2010-08-24 Thread Alon Levy
In the scenario where we already switched to agent mouse, and then reset, there was no indication to the device, and from there to spice, that the guest driver and agent have quit. This allows us to switch to server side mouse until the agent starts up again. In case the agent doesn't start, we

Re: [Spice-devel] [PATCH] vdservice: don't start when no qxl device present

2010-08-24 Thread Alon Levy
- "Gerd Hoffmann" wrote: > On 08/24/10 13:57, Alon Levy wrote: > > From 349e6a5bf8804b25c3af6e1f439122a7cc269cc2 Mon Sep 17 00:00:00 > 2001 > > From: Alon Levy > > Date: Tue, 24 Aug 2010 14:53:03 +0300 > > Subject: [PATCH] vdservice: don't start when no qxl device present > > Point being? >

Re: [Spice-devel] [PATCH] vdservice: don't start when no qxl device present

2010-08-24 Thread Gerd Hoffmann
On 08/24/10 13:57, Alon Levy wrote: From 349e6a5bf8804b25c3af6e1f439122a7cc269cc2 Mon Sep 17 00:00:00 2001 From: Alon Levy Date: Tue, 24 Aug 2010 14:53:03 +0300 Subject: [PATCH] vdservice: don't start when no qxl device present Point being? The display configuration bits are useless without q

[Spice-devel] [PATCH] vdservice: don't start when no qxl device present

2010-08-24 Thread Alon Levy
>From 349e6a5bf8804b25c3af6e1f439122a7cc269cc2 Mon Sep 17 00:00:00 2001 From: Alon Levy Date: Tue, 24 Aug 2010 14:53:03 +0300 Subject: [PATCH] vdservice: don't start when no qxl device present --- common/vdcommon.h |2 ++ vdagent/desktop_layout.cpp | 19 +-- vdagen

Re: [Spice-devel] [PATCH 7/7] Remove unused surfaces_used

2010-08-24 Thread Yonit Halperin
On 08/24/2010 11:41 AM, al...@redhat.com wrote: From: Alexander Larsson --- display/qxldd.h |1 - display/res.c | 12 +--- 2 files changed, 1 insertions(+), 12 deletions(-) Ack ___ Spice-devel mailing list Spice-devel@lists.freede

Re: [Spice-devel] [PATCH 6/7] New list based surface allocator

2010-08-24 Thread Yonit Halperin
On 08/24/2010 11:41 AM, al...@redhat.com wrote: From: Alexander Larsson We store a free list in the SurfaceInfos, using a field not otherwise used for free surfaces. Also, treat base_mem == NULL as "surface in use" for easy checking. base_mem != NULL --- display/qxldd.h |1 + display

Re: [Spice-devel] [PATCH 5/7] Add union to SurfaceInfo to put in next_free pointer

2010-08-24 Thread Yonit Halperin
On 08/24/2010 11:41 AM, al...@redhat.com wrote: From: Alexander Larsson --- display/driver.c |2 +- display/qxldd.h | 10 +++--- display/surface.c |2 +- display/surface.h |2 +- 4 files changed, 10 insertions(+), 6 deletions(-) Ack. Liked the recycling.

Re: [Spice-devel] [PATCH 4/7] Free surface id if surface construction failed

2010-08-24 Thread Yonit Halperin
On 08/24/2010 11:41 AM, al...@redhat.com wrote: From: Alexander Larsson --- display/driver.c |6 -- 1 files changed, 4 insertions(+), 2 deletions(-) Ack ___ Spice-devel mailing list Spice-devel@lists.freedesktop.org http://lists.freedeskto

Re: [Spice-devel] [PATCH 3/7] Release resources before allocating

2010-08-24 Thread Yonit Halperin
On 08/24/2010 11:41 AM, al...@redhat.com wrote: From: Alexander Larsson Before trying to allocate memory we free a large chunk of freeable resources. Leaving the resources unfreed before allocating increases the risks of unnecessarily fragmenting the heap, which is bad, especially for the vram w

Re: [Spice-devel] [PATCH 2/7] Grab malloc_sem in FreeMem

2010-08-24 Thread Yonit Halperin
On 08/24/2010 11:41 AM, al...@redhat.com wrote: From: Alexander Larsson This is needed as much as in AllocMem to protect the mspaces data from concurrent access. --- display/res.c |2 ++ 1 files changed, 2 insertions(+), 0 deletions(-) diff --git a/display/res.c b/display/res.c index 1f6

Re: [Spice-devel] [PATCH 1/7] Make malloc_sem global

2010-08-24 Thread Yonit Halperin
On 08/24/2010 11:41 AM, al...@redhat.com wrote: From: Alexander Larsson It protects shared data (mspaces info) so it needs to be shared. Ack. I guess cmd_sem and print_sem should be made global as well. ___ Spice-devel mailing list Spice-devel@lists.

[Spice-devel] [PATCH 7/7] Remove unused surfaces_used

2010-08-24 Thread alexl
From: Alexander Larsson --- display/qxldd.h |1 - display/res.c | 12 +--- 2 files changed, 1 insertions(+), 12 deletions(-) diff --git a/display/qxldd.h b/display/qxldd.h index 2cb2db4..82886a9 100644 --- a/display/qxldd.h +++ b/display/qxldd.h @@ -198,7 +198,6 @@ typedef struc

[Spice-devel] [PATCH 6/7] New list based surface allocator

2010-08-24 Thread alexl
From: Alexander Larsson We store a free list in the SurfaceInfos, using a field not otherwise used for free surfaces. Also, treat base_mem == NULL as "surface in use" for easy checking. --- display/qxldd.h |1 + display/res.c |4 display/surface.c |2 +- display/surface.h

[Spice-devel] [PATCH 5/7] Add union to SurfaceInfo to put in next_free pointer

2010-08-24 Thread alexl
From: Alexander Larsson --- display/driver.c |2 +- display/qxldd.h | 10 +++--- display/surface.c |2 +- display/surface.h |2 +- 4 files changed, 10 insertions(+), 6 deletions(-) diff --git a/display/driver.c b/display/driver.c index 1c8e1f6..f088df5 100644 --- a/display/

[Spice-devel] [PATCH 4/7] Free surface id if surface construction failed

2010-08-24 Thread alexl
From: Alexander Larsson --- display/driver.c |6 -- 1 files changed, 4 insertions(+), 2 deletions(-) diff --git a/display/driver.c b/display/driver.c index 46df8f2..1c8e1f6 100644 --- a/display/driver.c +++ b/display/driver.c @@ -1248,14 +1248,16 @@ HBITMAP APIENTRY DrvCreateDeviceBitma

[Spice-devel] [PATCH 3/7] Release resources before allocating

2010-08-24 Thread alexl
From: Alexander Larsson Before trying to allocate memory we free a large chunk of freeable resources. Leaving the resources unfreed before allocating increases the risks of unnecessarily fragmenting the heap, which is bad, especially for the vram which needs large free contiguous allocations for

[Spice-devel] [PATCH 2/7] Grab malloc_sem in FreeMem

2010-08-24 Thread alexl
From: Alexander Larsson This is needed as much as in AllocMem to protect the mspaces data from concurrent access. --- display/res.c |2 ++ 1 files changed, 2 insertions(+), 0 deletions(-) diff --git a/display/res.c b/display/res.c index 1f67dbf..6fe375f 100644 --- a/display/res.c +++ b/disp

[Spice-devel] [PATCH 1/7] Make malloc_sem global

2010-08-24 Thread alexl
From: Alexander Larsson It protects shared data (mspaces info) so it needs to be shared. --- display/driver.c | 22 ++ display/qxldd.h |2 +- display/res.c| 13 +++-- 3 files changed, 18 insertions(+), 19 deletions(-) diff --git a/display/driver.c b/disp

[Spice-devel] [PATCH 0/7] Updated driver fixes for 0.6.0

2010-08-24 Thread alexl
From: Alexander Larsson Makes malloc_sem global and protect FreeMem Releases a chunk of resources before allocating New free-list based surface allocator Alexander Larsson (7): Make malloc_sem global Grab malloc_sem in FreeMem Release resources before allocating Free surface id if surfac

[Spice-devel] SPICE Client memory consumption

2010-08-24 Thread xuzhinong999
Hi, spice developers, I am trying to port SPICE client to an embedded linux system now. Due to the resource limitation (DDR memory 128MB, Nand flahs 128MB) and CPU capability limitation (ARM11176, 800MHZ frequency), i want to know how many memories will SPICE client consume in a typical us