Re: [Spice-devel] video passthrough ideas

2011-05-12 Thread Zeeshan Ali (Khattak)
On Thu, May 12, 2011 at 6:56 PM, Marc-André Lureau wrote: > My concern for VAAPI though (and other acceleration API) is that it is not a > sink, which handle completely decoding and displaying. I understand it can > take care of some expensive operations, but the result probably has to be > ret

Re: [Spice-devel] video passthrough ideas

2011-05-12 Thread Zeeshan Ali (Khattak)
On Thu, May 12, 2011 at 9:57 PM, Marc-André Lureau wrote: > Hi > > - Original Message - >> b. GStreamer currently do no have working VAAPI plugins and AFAIK, >> it won't have any for at least a few months more. > > What do you mean? It seems they exist according to the wiki: > http://www.

Re: [Spice-devel] video passthrough ideas

2011-05-12 Thread Marc-André Lureau
Hi - Original Message - > b. GStreamer currently do no have working VAAPI plugins and AFAIK, > it won't have any for at least a few months more. What do you mean? It seems they exist according to the wiki: http://www.freedesktop.org/wiki/Software/vaapi (http://www.splitted-desktop.com/~

Re: [Spice-devel] video passthrough ideas

2011-05-12 Thread Marc-André Lureau
Hi Zeeshan, (quick reply) - Original Message - > Hi everyone, > 1. VAAPI (Video Acceleration API): The idea here is to implement a > custom VAAPI driver for the guest that redirects all decoding/playback > calls to the client through a spice channel. On the client side, we > create a chan

[Spice-devel] video passthrough ideas

2011-05-12 Thread Zeeshan Ali (Khattak)
Hi everyone, As some of you already know, I've been looking into the needed "video passthrough" feature: i-e sending the encoded stream to client and decode it on client-side. Although I'm still investigating on how to best handle this, I thought I already share my thoughts/findings so far so if

[Spice-devel] [PATCH] common: add WIN64 ifdef for spice_bit_find_msb

2011-05-12 Thread Arnon Gilboa
inline __asm is not supported in x64 --- common/bitops.h |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/common/bitops.h b/common/bitops.h index 4823662..449409b 100644 --- a/common/bitops.h +++ b/common/bitops.h @@ -27,7 +27,7 @@ extern "C" { #endif -#ifdef WIN32 +

Re: [Spice-devel] [PATCH 5/6] client/windows: undef SIZE_MAX in stdint.h

2011-05-12 Thread Christophe Fergeau
On Thu, May 12, 2011 at 12:37:30PM +0300, Arnon Gilboa wrote: > eliminating redefinition warning > --- > client/windows/stdint.h |2 +- > 1 files changed, 1 insertions(+), 1 deletions(-) > > diff --git a/client/windows/stdint.h b/client/windows/stdint.h > index 45b133e..768ca03 100644 > --- a

Re: [Spice-devel] [PATCH 4/6] common: fix ssl_verify windows build errors

2011-05-12 Thread Christophe Fergeau
On Thu, May 12, 2011 at 12:37:29PM +0300, Arnon Gilboa wrote: > --- > common/ssl_verify.c |5 +++-- > common/ssl_verify.h |5 + > 2 files changed, 8 insertions(+), 2 deletions(-) ACK pgpHkruUV2CRG.pgp Description: PGP signature ___ Spice-d

Re: [Spice-devel] [PATCH 2/6] client/windows: inc version to 0, 9, 0, 0

2011-05-12 Thread Christophe Fergeau
On Thu, May 12, 2011 at 02:30:41PM +0300, Arnon Gilboa wrote: > I guess you are right and that's because of my fat fingers. > However, looking at the following funny values, it would be better > changed simply to DEFAULT. > 1255 ANSI Hebrew; Hebrew (Windows) > 1252 ANSI Latin 1; Western European (W

Re: [Spice-devel] [PATCH 6/6] common: use INLINE instead of inline

2011-05-12 Thread Christophe Fergeau
On Thu, May 12, 2011 at 02:56:45PM +0300, Arnon Gilboa wrote: > Christophe Fergeau wrote: > >On Thu, May 12, 2011 at 12:37:31PM +0300, Arnon Gilboa wrote: > >>needed for spice/common files used by the client, server & qxl driver. > >>in windows _inline works for both c/c++, while inline is c++ only

Re: [Spice-devel] [PATCH 1/6] client/windows: init PACKAGE_VERSION

2011-05-12 Thread Christophe Fergeau
On Thu, May 12, 2011 at 12:37:26PM +0300, Arnon Gilboa wrote: > in windows, we set PACKAGE_VERSION to the binary version > since we don't have config.h as generated by linux configure ACK Christophe > --- > client/common.h |2 ++ > client/windows/main.cpp |6 +++--- > 2 files cha

Re: [Spice-devel] [PATCH 3/6] client/windows: add common\ssl_verify.c/h to project

2011-05-12 Thread Christophe Fergeau
On Thu, May 12, 2011 at 12:37:28PM +0300, Arnon Gilboa wrote: > disable WarnAsError, due to c/c++ warnings Are they new warnings that weren't there before? Maybe they could be fixed? Christophe pgpk1xjVcgp6j.pgp Description: PGP signature ___ Spice-de

Re: [Spice-devel] about Spice

2011-05-12 Thread Emre Erenoglu
2011/5/12 纪树平 : > Hello, I am a student in China. > > Recently I am interested in Spice and I tried to use it. But I failed. > > I do not know where is the problem. > > Here I want to ask a question: for the Client, the Host and the Guest OS, > which one should be 64bits? Hi, The host should be 6

[Spice-devel] about Spice

2011-05-12 Thread 纪树平
Hello, I am a student in China. Recently I am interested in Spice and I tried to use it. But I failed. I do not know where is the problem. Here I want to ask a question: for the Client, the Host and the Guest OS, which one should be 64bits? Thanks _

Re: [Spice-devel] [PATCH 6/6] common: use INLINE instead of inline

2011-05-12 Thread Arnon Gilboa
Christophe Fergeau wrote: On Thu, May 12, 2011 at 12:37:31PM +0300, Arnon Gilboa wrote: needed for spice/common files used by the client, server & qxl driver. in windows _inline works for both c/c++, while inline is c++ only. compiling the client with mixed c/c++ code required this define.

Re: [Spice-devel] [PATCH 2/6] client/windows: inc version to 0, 9, 0, 0

2011-05-12 Thread Arnon Gilboa
Christophe Fergeau wrote: On Thu, May 12, 2011 at 12:37:27PM +0300, Arnon Gilboa wrote: @@ -111,7 +111,7 @@ RED_ICON_RES_ID ICON"spice.ico" #if !defined(AFX_RESOURCE_DLL) || defined(AFX_TARG_NEUSD) #ifdef _WIN32 LANGUAGE LANG_NEUTRAL, SUBLANG_SYS_DEFAULT -#pragm

Re: [Spice-devel] [PATCH 6/6] common: use INLINE instead of inline

2011-05-12 Thread Christophe Fergeau
On Thu, May 12, 2011 at 12:37:31PM +0300, Arnon Gilboa wrote: > needed for spice/common files used by the client, server & qxl driver. > in windows _inline works for both c/c++, while inline is c++ only. > compiling the client with mixed c/c++ code required this define. I'd tend to change inline t

Re: [Spice-devel] [PATCH spice-protocol] macros: define INLINE

2011-05-12 Thread Christophe Fergeau
On Thu, May 12, 2011 at 11:41:34AM +0200, Christophe Fergeau wrote: > Hi, > > On Thu, May 12, 2011 at 11:49:38AM +0300, Arnon Gilboa wrote: > > needed for spice/common files used by the client, server & qxl driver. > > in windows _inline works for both c/c++, while inline is c++ only. > > compilin

Re: [Spice-devel] [PATCH spice-protocol] macros: define INLINE

2011-05-12 Thread Arnon Gilboa
Christophe Fergeau wrote: Hi, On Thu, May 12, 2011 at 11:49:38AM +0300, Arnon Gilboa wrote: needed for spice/common files used by the client, server & qxl driver. in windows _inline works for both c/c++, while inline is c++ only. compiling the client with mixed c/c++ code required this defin

Re: [Spice-devel] [PATCH 2/6] client/windows: inc version to 0, 9, 0, 0

2011-05-12 Thread Christophe Fergeau
On Thu, May 12, 2011 at 12:37:27PM +0300, Arnon Gilboa wrote: > @@ -111,7 +111,7 @@ RED_ICON_RES_ID ICON > "spice.ico" > #if !defined(AFX_RESOURCE_DLL) || defined(AFX_TARG_NEUSD) > #ifdef _WIN32 > LANGUAGE LANG_NEUTRAL, SUBLANG_SYS_DEFAULT > -#pragma code_page(1255) >

Re: [Spice-devel] [PATCH spice-protocol] macros: define INLINE

2011-05-12 Thread Christophe Fergeau
Hi, On Thu, May 12, 2011 at 11:49:38AM +0300, Arnon Gilboa wrote: > needed for spice/common files used by the client, server & qxl driver. > in windows _inline works for both c/c++, while inline is c++ only. > compiling the client with mixed c/c++ code required this define. If this change is done

Re: [Spice-devel] [PATCH] client: fix return code when missing host

2011-05-12 Thread Arnon Gilboa
ack Alon Levy wrote: --- client/application.cpp |1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/client/application.cpp b/client/application.cpp index e308ad1..606f7e2 100644 --- a/client/application.cpp +++ b/client/application.cpp @@ -2437,6 +2437,7 @@ bool Application

[Spice-devel] [PATCH] client: fix return code when missing host

2011-05-12 Thread Alon Levy
--- client/application.cpp |1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/client/application.cpp b/client/application.cpp index e308ad1..606f7e2 100644 --- a/client/application.cpp +++ b/client/application.cpp @@ -2437,6 +2437,7 @@ bool Application::process_cmd_line(int a

[Spice-devel] [PATCH 6/6] common: use INLINE instead of inline

2011-05-12 Thread Arnon Gilboa
needed for spice/common files used by the client, server & qxl driver. in windows _inline works for both c/c++, while inline is c++ only. compiling the client with mixed c/c++ code required this define. --- common/bitops.h |6 -- common/canvas_utils.c |2 +- common/lines.c

[Spice-devel] [PATCH 5/6] client/windows: undef SIZE_MAX in stdint.h

2011-05-12 Thread Arnon Gilboa
eliminating redefinition warning --- client/windows/stdint.h |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/client/windows/stdint.h b/client/windows/stdint.h index 45b133e..768ca03 100644 --- a/client/windows/stdint.h +++ b/client/windows/stdint.h @@ -293,7 +293,7 @@ typ

[Spice-devel] [PATCH 4/6] common: fix ssl_verify windows build errors

2011-05-12 Thread Arnon Gilboa
--- common/ssl_verify.c |5 +++-- common/ssl_verify.h |5 + 2 files changed, 8 insertions(+), 2 deletions(-) diff --git a/common/ssl_verify.c b/common/ssl_verify.c index 5e3722e..603d0f3 100644 --- a/common/ssl_verify.c +++ b/common/ssl_verify.c @@ -31,7 +31,7 @@ #include #ifndef

[Spice-devel] [PATCH 3/6] client/windows: add common\ssl_verify.c/h to project

2011-05-12 Thread Arnon Gilboa
disable WarnAsError, due to c/c++ warnings --- client/windows/redc.vcproj | 12 ++-- 1 files changed, 10 insertions(+), 2 deletions(-) diff --git a/client/windows/redc.vcproj b/client/windows/redc.vcproj index 87d018c..17032f7 100644 --- a/client/windows/redc.vcproj +++ b/client/windows

[Spice-devel] [PATCH 2/6] client/windows: inc version to 0,9,0,0

2011-05-12 Thread Arnon Gilboa
--- client/windows/redc.rc | 10 +- 1 files changed, 5 insertions(+), 5 deletions(-) diff --git a/client/windows/redc.rc b/client/windows/redc.rc index 6c717df..e6780cc 100644 --- a/client/windows/redc.rc +++ b/client/windows/redc.rc @@ -60,8 +60,8 @@ ALT_IMAGE_RES_IDBITMAP

[Spice-devel] [PATCH 1/6] client/windows: init PACKAGE_VERSION

2011-05-12 Thread Arnon Gilboa
in windows, we set PACKAGE_VERSION to the binary version since we don't have config.h as generated by linux configure --- client/common.h |2 ++ client/windows/main.cpp |6 +++--- 2 files changed, 5 insertions(+), 3 deletions(-) diff --git a/client/common.h b/client/common.h index

[Spice-devel] [PATCH 0/6] client: fix broken windows build

2011-05-12 Thread Arnon Gilboa
Arnon Gilboa (6): client/windows: init PACKAGE_VERSION client/windows: inc version to 0,9,0,0 client/windows: add common\ssl_verify.c/h to project common: fix ssl_verify windows build errors client/windows: undef SIZE_MAX in stdint.h common: use INLINE instead of inline client/common.

[Spice-devel] [PATCH spice-protocol] macros: define INLINE

2011-05-12 Thread Arnon Gilboa
needed for spice/common files used by the client, server & qxl driver. in windows _inline works for both c/c++, while inline is c++ only. compiling the client with mixed c/c++ code required this define. --- spice/macros.h |6 ++ 1 files changed, 6 insertions(+), 0 deletions(-) diff --git

Re: [Spice-devel] Clipboard sharing...

2011-05-12 Thread Arnon Gilboa
Andrew Cathrow wrote: - Original Message - From: "Naga Mohan Pothula" To: spice-devel@lists.freedesktop.org Sent: Wednesday, May 11, 2011 6:35:13 PM Subject: [Spice-devel] Clipboard sharing... Hi, I downloaded Spice v0.8.1/v0.8.0 and tested for Clipboard sharing on Win7 x64, RHEL

Re: [Spice-devel] Windows 7 64bit QXL driver not digitally signed

2011-05-12 Thread Arnon Gilboa
As I know, we currently don't sign the upstream windows driver. However, this tool will do what you need. http://www.ngohq.com/home.php?page=dseo John Daily wrote: Hi All, I've successfully built the 64bit driver for my Windows 7 64 bit VM. However, Windows will not use the driver after install

[Spice-devel] Windows 7 64bit QXL driver not digitally signed

2011-05-12 Thread John Daily
Hi All, I've successfully built the 64bit driver for my Windows 7 64 bit VM. However, Windows will not use the driver after installation because it is cannot verify if it is digitally signed. I've followed the directions from the HowTo .