Re: [Spice-devel] Spice Windows 7 display driver?

2011-08-02 Thread Alon Levy
On Tue, Aug 02, 2011 at 01:17:36PM -0400, Nathan Lager wrote: > -BEGIN PGP SIGNED MESSAGE- > Hash: SHA1 > > Ok, a little less hacky, i still need to tack on -spice... but i virsh > edited my vdi vm, changed the video from "vga" to "qlx", now the command > line generated uses -vga qlx, no m

Re: [Spice-devel] Spice Windows 7 display driver?

2011-08-02 Thread Mario
Hi, that is a bit different to my issue. XP works like a charm but 7 is displaying "Device cannot start (Code10)". My start parameters are as follows: Win XP: /usr/bin/qemu \ -monitor stdio \ -spice port=5930,disable-ticketing \ -vga qxl \ -drive file=/home/VIRUAL/burns/burns.img,index=0,medi

Re: [Spice-devel] Spice Windows 7 display driver?

2011-08-02 Thread Nathan Lager
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 Just an update. I've built an XP vm on the same host, and it's got the exact same issue. Driver installs, but then displays the same "Cannot start" error. On 08/02/2011 12:47 PM, Alon Levy wrote: > On Tue, Aug 02, 2011 at 12:19:37PM -0400, Nathan La

Re: [Spice-devel] Spice Windows 7 display driver?

2011-08-02 Thread Nathan Lager
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 Ok, a little less hacky, i still need to tack on -spice... but i virsh edited my vdi vm, changed the video from "vga" to "qlx", now the command line generated uses -vga qlx, no more -vga std in the command. But the problem persists. On 08/02/2011 12:

Re: [Spice-devel] Spice Windows 7 display driver?

2011-08-02 Thread Nathan Lager
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 The method i use to launch my vm is sort of a hack. This is CentOS6, yes, using libvirt, and qemu-kvm. No multiple displays My command line (as found in ps) follows: [root@kvm ~]# ps awx | grep vdi 31472 ?Sl 0:01 /usr/libexec/qemu-kvm -S

Re: [Spice-devel] Spice Windows 7 display driver?

2011-08-02 Thread Alon Levy
On Tue, Aug 02, 2011 at 12:19:37PM -0400, Nathan Lager wrote: > -BEGIN PGP SIGNED MESSAGE- > Hash: SHA1 > > Ok.. One step further. Its installed, device manager reports my display > adapter as a Red Hat QXL GPU, but it says that "The device cannot start. > (Code 10)" > What are the chan

Re: [Spice-devel] Spice Windows 7 display driver?

2011-08-02 Thread Mario
Hi, that is exact what I experienced. Any ideas? Thanks, Mario On Tue, 02 Aug 2011 12:19:37 -0400, Nathan Lager wrote: -BEGIN PGP SIGNED MESSAGE- Hash: SHA1 Ok.. One step further. Its installed, device manager reports my display adapter as a Red Hat QXL GPU, but it says that "The d

Re: [Spice-devel] Spice Windows 7 display driver?

2011-08-02 Thread Nathan Lager
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 Ok.. One step further. Its installed, device manager reports my display adapter as a Red Hat QXL GPU, but it says that "The device cannot start. (Code 10)" On 08/02/2011 10:05 AM, Arnon Gilboa wrote: > Hi Nathan, > Get the updated test-signed driver

Re: [Spice-devel] Spice Windows 7 display driver?

2011-08-02 Thread Nathan Lager
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 Trying this out now. Seems to have installed. Thanks for the package! On 08/02/2011 10:05 AM, Arnon Gilboa wrote: > Hi Nathan, > Get the updated test-signed driver from the download page, or directly: > http://www.spice-space.org/download/binaries/

Re: [Spice-devel] Spice Windows 7 display driver?

2011-08-02 Thread Arnon Gilboa
Updated the zip with RedHat signed drivers. Enjoy, Arnon Arnon Gilboa wrote: Hi Nathan, Get the updated test-signed driver from the download page, or directly: http://www.spice-space.org/download/binaries/qxl-win_20110802.zip Arnon Nathan Lager wrote: -BEGIN PGP SIGNED MESSAGE- Hash:

Re: [Spice-devel] Spice Windows 7 display driver?

2011-08-02 Thread Arnon Gilboa
Hi Nathan, Get the updated test-signed driver from the download page, or directly: http://www.spice-space.org/download/binaries/qxl-win_20110802.zip Arnon Nathan Lager wrote: -BEGIN PGP SIGNED MESSAGE- Hash: SHA1 I've been running in circles attempting to install a Display driver in Wi

Re: [Spice-devel] Spice Windows 7 display driver?

2011-08-02 Thread Yaniv Kaul
On 02/08/2011 16:49, Nathan Lager wrote: -BEGIN PGP SIGNED MESSAGE- Hash: SHA1 I used the "x64 free build environment" link that wdk added to my start menu. The inf seems to have provisions for xp 32 and 64, and vista 32 and 64. Should there be one specifically for 7? No, you should

Re: [Spice-devel] Spice Windows 7 display driver?

2011-08-02 Thread Nathan Lager
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 I used the "x64 free build environment" link that wdk added to my start menu. The inf seems to have provisions for xp 32 and 64, and vista 32 and 64. Should there be one specifically for 7? On 08/02/2011 09:41 AM, Yaniv Kaul wrote: > On 02/08/2011

Re: [Spice-devel] Spice Windows 7 display driver?

2011-08-02 Thread Yaniv Kaul
On 02/08/2011 16:25, Nathan Lager wrote: -BEGIN PGP SIGNED MESSAGE- Hash: SHA1 I'm not sure what went wrong last time, but this time I was able to get the driver signed with a Test Cert. Followed the instructions for enabling test signing, and attempted to install the driver. It presen

Re: [Spice-devel] Spice Windows 7 display driver?

2011-08-02 Thread Nathan Lager
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 I'm not sure what went wrong last time, but this time I was able to get the driver signed with a Test Cert. Followed the instructions for enabling test signing, and attempted to install the driver. It presents me with the same error. It displays the

Re: [Spice-devel] Spice Windows 7 display driver?

2011-08-02 Thread Nathan Lager
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 Thanks for the link. I had followed this set of instructions. I grabbed the Windows DDK, and built the qxl drivers. However I'm not able to install them. The build went fine, i wasn't able to sign them though. Maybe I'll give that another shot, an

Re: [Spice-devel] [PATCH 2/2] server: init all fields on SpiceMsgDisplayStreamCreate

2011-08-02 Thread Yaniv Kaul
On 08/02/2011 01:54 PM, Alon Levy wrote: On Tue, Aug 02, 2011 at 12:31:57PM +0200, Christophe Fergeau wrote: This patch is an RFC red_display_marshall_stream_start initializes a SpiceMsgDisplayStreamCreate structure before marshalling it and sending it on the wire. However, it never fills Spice

Re: [Spice-devel] [PATCH 2/2] server: init all fields on SpiceMsgDisplayStreamCreate

2011-08-02 Thread Alon Levy
On Tue, Aug 02, 2011 at 12:31:57PM +0200, Christophe Fergeau wrote: > This patch is an RFC > > red_display_marshall_stream_start initializes a > SpiceMsgDisplayStreamCreate structure before marshalling it and > sending it on the wire. However, it never fills > SpiceMsgDisplayStreamCreate::stamp wh

[Spice-devel] [PATCH 2/2] server: init all fields on SpiceMsgDisplayStreamCreate

2011-08-02 Thread Christophe Fergeau
This patch is an RFC red_display_marshall_stream_start initializes a SpiceMsgDisplayStreamCreate structure before marshalling it and sending it on the wire. However, it never fills SpiceMsgDisplayStreamCreate::stamp which then causes a complaint from valgrind. Initializing it is easy enough, howev

[Spice-devel] [PATCH 1/2] fix valgrind warning in test_display__stream

2011-08-02 Thread Christophe Fergeau
create_test_primary_surface::test_display_base.c creates a QXLDevSurfaceCreate structure and initialize it, but doesn't set the position field. Moreover, this structure has 4 bytes of padding to the end (as shown by pahole from dwarves), so initialize the whole structure to 0 before using it. ---

Re: [Spice-devel] [PATCH] server/red_dispatcher: fix memset params

2011-08-02 Thread Alon Levy
On Tue, Aug 02, 2011 at 11:55:53AM +0200, Christophe Fergeau wrote: > Hey Uri, > > Good catch! How did you find it? valgrind? static analysis? Code > review? > > On Tue, Aug 02, 2011 at 11:05:11AM +0300, Uri Lublin wrote: > > update_client_mouse_allowed(); > > -memset(&dispatcher->surfac

Re: [Spice-devel] [PATCH] server/red_dispatcher: fix memset params

2011-08-02 Thread Christophe Fergeau
Hey Uri, Good catch! How did you find it? valgrind? static analysis? Code review? On Tue, Aug 02, 2011 at 11:05:11AM +0300, Uri Lublin wrote: > update_client_mouse_allowed(); > -memset(&dispatcher->surface_create, sizeof(QXLDevSurfaceCreate), 0); > +memset(&dispatcher->surface_create

Re: [Spice-devel] Spice Windows 7 display driver?

2011-08-02 Thread John A. Sullivan III
On Mon, 2011-08-01 at 15:56 -0400, Nathan Lager wrote: > -BEGIN PGP SIGNED MESSAGE- > Hash: SHA1 > > I've been running in circles attempting to install a Display driver in > Windows 7 x64 for use with Spice. > > My host is CentOS6 x64, My guest is Windows 7 x64. Spice is enabled, as > i

Re: [Spice-devel] [Spice-commits] 3 commits - client/cmd_line_parser.cpp client/cmd_line_parser.h client/windows client/x11 server/mjpeg_encoder.c

2011-08-02 Thread Christophe Fergeau
On Tue, Aug 02, 2011 at 02:15:34AM -0700, Christophe Fergau wrote: > New commits: > commit 3582adb989cdb6e1e75bf9341ffcebf35e58b737 > Author: Christophe Fergeau > Date: Fri Jul 29 18:52:15 2011 +0200 > > mjpeg: add missing SPICE_BITMAP_FMT_RGBA > > I forgot to handle SPICE_BITMAP_F

Re: [Spice-devel] [PATCH 2/2] Fix typo: treshold -> threshold

2011-08-02 Thread Christophe Fergeau
On Mon, Jul 25, 2011 at 04:33:45PM +0800, Liang Guo wrote: > I can only checkout from spice git repository, so if it is ok, please > merge these patches > or fell free to change them. I finally committed them, thanks again for the patches, and sorry for the delay! Christophe pgpSwM7ADGUI6.pgp

Re: [Spice-devel] [PATCH] server/red_dispatcher: fix memset params

2011-08-02 Thread Alon Levy
On Tue, Aug 02, 2011 at 11:05:11AM +0300, Uri Lublin wrote: > Replace wrong memset(ptr, size, 0) > With memset(ptr, 0, size) ACK. Thanks for catching this. > --- > server/red_dispatcher.c |2 +- > 1 files changed, 1 insertions(+), 1 deletions(-) > > diff --git a/server/red_dispatcher.c b/se

[Spice-devel] [PATCH] server/red_dispatcher: fix memset params

2011-08-02 Thread Uri Lublin
Replace wrong memset(ptr, size, 0) With memset(ptr, 0, size) --- server/red_dispatcher.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/server/red_dispatcher.c b/server/red_dispatcher.c index 8f4a8a8..f86da81 100644 --- a/server/red_dispatcher.c +++ b/server/red_dispatch

Re: [Spice-devel] Spice Windows 7 display driver?

2011-08-02 Thread Mario
Nathan, did you take a look on this: http://spice-space.org/page/WinQXL Windows will install the default driver if there is no better available. You can verify you are having the qxl device by checking the PCI IDs in your Device Manager. Cheers, Mario On Mon, 01 Aug 2011 15:56:20 -0400, Nat

[Spice-devel] Spice Windows 7 display driver?

2011-08-02 Thread Nathan Lager
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 I've been running in circles attempting to install a Display driver in Windows 7 x64 for use with Spice. My host is CentOS6 x64, My guest is Windows 7 x64. Spice is enabled, as is -vga qxl, Windows see's the display adapter as "standard vga graphics