Re: [Qemu-devel] Unknown command 0xffffff in SVGA command FIFO

2010-09-09 Thread andrzej zaborowski
Hi, On 16 August 2010 22:26, Janne Huttunen wrote: > Yes, your version works (both on paper and in practice). I'm not > quite sure I like the way it breaches the apparent abstraction > of the FIFO handling routines (if you can call it that) or the > way it first gives FIFO slots back to the guest

Re: [Qemu-devel] Unknown command 0xffffff in SVGA command FIFO

2010-08-16 Thread Janne Huttunen
I came up with this version, it kind of reverses the logic of your patch but reuses the _items function (renamed _length), please see if it looks ok and possibly even works. [sorry about the delay, I was out of office for a while] Yes, your version works (both on paper and in practice). I'm n

Re: [Qemu-devel] Unknown command 0xffffff in SVGA command FIFO

2010-07-22 Thread andrzej zaborowski
Sorry, tried to use get-send-email but haven't tested first. On 23 July 2010 03:35, wrote: > From: Andrzej Zaborowski ...

Re: [Qemu-devel] Unknown command 0xffffff in SVGA command FIFO

2010-07-22 Thread balrog
From: Andrzej Zaborowski Hi Janne, I came up with this version, it kind of reverses the logic of your patch but reuses the _items function (renamed _length), please see if it looks ok and possibly even works. diff --git a/hw/vmware_vga.c b/hw/vmware_vga.c index 12bff48..464f8bc 100644 --- a/hw/v

Re: [Qemu-devel] Unknown command 0xffffff in SVGA command FIFO

2010-07-21 Thread Janne Huttunen
Here's an experiment for sanity checking the lengths and leaving the command in the FIFO if it is not complete. It fixes the problem for me (running it right now), but I agree that it's not very elegant to look at :-/ . And here's another version with couple of stupid bugs removed (it obviousl

Re: [Qemu-devel] Unknown command 0xffffff in SVGA command FIFO

2010-07-21 Thread Janne Huttunen
I see no way to tell whether the guest is currently in the middle of writing a command. So it seems the only way to check is to peek the first word in the fifo (which *is* written entirely before a NEXT_CMD update) and look up the expected command length, and then check whether enough data is i

Re: [Qemu-devel] Unknown command 0xffffff in SVGA command FIFO

2010-07-21 Thread andrzej zaborowski
On 21 July 2010 14:14, Janne Huttunen wrote: >> No, I think that can't happen, but it would be interesting to bisect >> what the guest is doing exactly when this happens.  The guest should >> not move the "next command" pointer before if has written the command >> entirely, this should be enough t

Re: [Qemu-devel] Unknown command 0xffffff in SVGA command FIFO

2010-07-21 Thread Janne Huttunen
No, I think that can't happen, but it would be interesting to bisect what the guest is doing exactly when this happens. The guest should not move the "next command" pointer before if has written the command entirely, this should be enough to guard against executing a partial command. Unless th

Re: [Qemu-devel] Unknown command 0xffffff in SVGA command FIFO

2010-07-21 Thread andrzej zaborowski
Hi, On 21 July 2010 13:17, Janne Huttunen wrote: > Now, correct me if I'm wrong, but isn't vmsvga_fifo_run() called > from an asynchronous context (wrt the guest)? If that indeed is > so, it may very well be, that it is run while the guest is > modifying the FIFO. This means, that a command may f

[Qemu-devel] Unknown command 0xffffff in SVGA command FIFO

2010-07-21 Thread Janne Huttunen
Hi! I'm trying to run a Linux guest on top of QEMU (kvm). The only VGA emulation that seems to give any kind of usable performance is the vmware SVGA adapter, but that in turn is very unstable. It usually freezes the guest display within a minute or two and starts printing an error like "vmsvga_