Re: [kvm-devel] [PATCH] Move common VGAState attributes to VGA_STATE_COMMON

2008-02-23 Thread Avi Kivity
Soren Hansen wrote: vmware_vga.c uses functions in vga.c to do some things. They need to agree on which parts of their state struct is common and which aren't, otherwise they'll overwrite parts of each other's state. This patch makes it so. Applied, thanks. -- error compiling

Re: [kvm-devel] [PATCH] Move common VGAState attributes to VGA_STATE_COMMON

2008-02-22 Thread Soren Hansen
vmware_vga.c uses functions in vga.c to do some things. They need to agree on which parts of their state struct is common and which aren't, otherwise they'll overwrite parts of each other's state. This patch makes it so. Signed-off-by: Soren Hansen [EMAIL PROTECTED] --- qemu/hw/cirrus_vga.c |

Re: [kvm-devel] [PATCH] Move common VGAState attributes to VGA_STATE_COMMON

2008-02-22 Thread Chris Wedgwood
On Fri, Feb 22, 2008 at 11:12:42AM +0100, Soren Hansen wrote: vmware_vga.c uses functions in vga.c to do some things. They need to agree on which parts of their state struct is common and which aren't, otherwise they'll overwrite parts of each other's state. This patch makes it so. I tried

[kvm-devel] [PATCH] Move common VGAState attributes to VGA_STATE_COMMON

2008-02-11 Thread Soren Hansen
I was told to resend this, so here goes: vmware_vga.c uses functions in vga.c to do some things. They need to agree on which parts of their state struct is common and which aren't, otherwise they'll overwrite parts of each other's state. This patch makes it so. Signed-off-by: Soren Hansen [EMAIL

Re: [kvm-devel] [PATCH] Move common VGAState attributes to VGA_STATE_COMMON

2008-02-11 Thread Izik Eidus
Soren Hansen wrote: I was told to resend this, so here goes: vmware_vga.c uses functions in vga.c to do some things. They need to agree on which parts of their state struct is common and which aren't, otherwise they'll overwrite parts of each other's state. This patch makes it so.