[Spice-devel] BSOD when booting Win7 Pro with spice

2011-01-27 Thread Julian Sikorski
Hi, I wanted to try spice on Fedora 14. I have a Win7 Pro 32bit image which was created with virt manager. I later added virtio network and drive to it, installing drivers in the guest as well. The problem is that if I try to use this machine as per F14 wiki, i.e.: qemu /var/lib/libvirt/images/win

Re: [Spice-devel] paravirtual mouse/tablet, v5

2011-01-27 Thread Alon Levy
On Thu, Jan 27, 2011 at 04:32:40PM +0100, Gerd Hoffmann wrote: > Hi, > > >>typedef struct qemu_pvmouse_ack { > >> uint32_t features; /* qemu_pvtable_features */ > > > >Why does this comment say "qemu_pvtable_features" and the one above > >says "qemu_pvmouse_features"? > > Not intentional, w

Re: [Spice-devel] paravirtual mouse/tablet, v5

2011-01-27 Thread Gerd Hoffmann
Hi, typedef struct qemu_pvmouse_ack { uint32_t features; /* qemu_pvtable_features */ Why does this comment say "qemu_pvtable_features" and the one above says "qemu_pvmouse_features"? Not intentional, will fix. Leftover because it is misspelled (t in table*t* missing), so the search+

Re: [Spice-devel] [Qemu-devel] paravirtual mouse/tablet, v5

2011-01-27 Thread Anthony Liguori
On 01/27/2011 08:53 AM, Gerd Hoffmann wrote: On 01/27/11 15:27, Anthony Liguori wrote: On 01/27/2011 07:11 AM, Gerd Hoffmann wrote: Hi, Next revision the pvmouse protocol. It is quite different now, I've decided to move to a model with one message per updated value, simliar to the linux input

Re: [Spice-devel] paravirtual mouse/tablet, v5

2011-01-27 Thread Alon Levy
On Thu, Jan 27, 2011 at 02:11:35PM +0100, Gerd Hoffmann wrote: > Hi, > > Next revision the pvmouse protocol. It is quite different now, I've > decided to move to a model with one message per updated value, > simliar to the linux input layer. There isn't a "mouse move" > message any more. A mo

Re: [Spice-devel] [Qemu-devel] paravirtual mouse/tablet, v5

2011-01-27 Thread Gerd Hoffmann
On 01/27/11 15:27, Anthony Liguori wrote: On 01/27/2011 07:11 AM, Gerd Hoffmann wrote: Hi, Next revision the pvmouse protocol. It is quite different now, I've decided to move to a model with one message per updated value, simliar to the linux input layer. There isn't a "mouse move" message any

Re: [Spice-devel] [Qemu-devel] paravirtual mouse/tablet, v5

2011-01-27 Thread Anthony Liguori
On 01/27/2011 07:11 AM, Gerd Hoffmann wrote: Hi, Next revision the pvmouse protocol. It is quite different now, I've decided to move to a model with one message per updated value, simliar to the linux input layer. There isn't a "mouse move" message any more. A mouse move event will be th

[Spice-devel] paravirtual mouse/tablet, v5

2011-01-27 Thread Gerd Hoffmann
Hi, Next revision the pvmouse protocol. It is quite different now, I've decided to move to a model with one message per updated value, simliar to the linux input layer. There isn't a "mouse move" message any more. A mouse move event will be three messages now: one to update X, one to upd

Re: [Spice-devel] [PATCH 3/3] client/windows: don't allocate console unless required

2011-01-27 Thread Arnon Gilboa
ack Alon Levy wrote: --- client/windows/platform.cpp | 36 +++- 1 files changed, 27 insertions(+), 9 deletions(-) diff --git a/client/windows/platform.cpp b/client/windows/platform.cpp index c41c39a..f57413d 100644 --- a/client/windows/platform.cpp +++ b/clien

Re: [Spice-devel] [PATCH 2/3] client: fix broken vs2008 build

2011-01-27 Thread Arnon Gilboa
ack Alon Levy wrote: --- client/application.cpp |2 -- client/common.h | 11 ++- client/process_loop.cpp |2 +- client/red_client.cpp |4 4 files changed, 15 insertions(+), 4 deletions(-) diff --git a/client/application.cpp b/client/application.cpp index 9

Re: [Spice-devel] [PATCH 1/3] client: --help should not need platform initialization

2011-01-27 Thread Arnon Gilboa
Ack, although see comments below Alon Levy wrote: separate initialization into before command line parsing and after, call later only if command line parsing succeeds (in particular, it "fails" if --help is given). --- client/application.cpp | 90 +++---

[Spice-devel] [PATCH 3/3] client/windows: don't allocate console unless required

2011-01-27 Thread Alon Levy
--- client/windows/platform.cpp | 36 +++- 1 files changed, 27 insertions(+), 9 deletions(-) diff --git a/client/windows/platform.cpp b/client/windows/platform.cpp index c41c39a..f57413d 100644 --- a/client/windows/platform.cpp +++ b/client/windows/platform.cpp @

[Spice-devel] [PATCH 2/3] client: fix broken vs2008 build

2011-01-27 Thread Alon Levy
--- client/application.cpp |2 -- client/common.h | 11 ++- client/process_loop.cpp |2 +- client/red_client.cpp |4 4 files changed, 15 insertions(+), 4 deletions(-) diff --git a/client/application.cpp b/client/application.cpp index 95633f8..f9ed391 100644 -

[Spice-devel] [PATCH 1/3] client: --help should not need platform initialization

2011-01-27 Thread Alon Levy
separate initialization into before command line parsing and after, call later only if command line parsing succeeds (in particular, it "fails" if --help is given). --- client/application.cpp | 90 +++- client/application.h |5 ++- 2 files change

[Spice-devel] [PATCH 0/3] client: fix --help behavior and msvc build

2011-01-27 Thread Alon Levy
First patch splits initilization of client to before parsing and after parsing of command line arguments. fixed calling of cleanup per Uri's comment. Second patch fixes Msvc 2008 build that has been broken since mingw patches were pushed. Third patch makes windows client not allocate a console if

Re: [Spice-devel] [PATCH] Add build script for qxl using the given DDK, configuration and platform

2011-01-27 Thread Arnon Gilboa
NACK, better scripts already found :) Arnon Gilboa wrote: Necessary for build scripting (e.g. from Cygwin) --- make.bat | 10 ++ 1 files changed, 10 insertions(+), 0 deletions(-) create mode 100644 make.bat diff --git a/make.bat b/make.bat new file mode 100644 index 000..80c4075

[Spice-devel] [PATCH] Add build script for qxl using the given DDK, configuration and platform

2011-01-27 Thread Arnon Gilboa
Necessary for build scripting (e.g. from Cygwin) --- make.bat | 10 ++ 1 files changed, 10 insertions(+), 0 deletions(-) create mode 100644 make.bat diff --git a/make.bat b/make.bat new file mode 100644 index 000..80c4075 --- /dev/null +++ b/make.bat @@ -0,0 +1,10 @@ +REM build qxl