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 committee.c: too many arguments to function


-
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2008.
http://clk.atdmt.com/MRT/go/vse012070mrt/direct/01/
___
kvm-devel mailing list
kvm-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/kvm-devel

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 |3 ---
 qemu/hw/vga_int.h|   13 +
 2 files changed, 5 insertions(+), 11 deletions(-)

diff --git a/qemu/hw/cirrus_vga.c b/qemu/hw/cirrus_vga.c
index 43307d8..4bff503 100644
--- a/qemu/hw/cirrus_vga.c
+++ b/qemu/hw/cirrus_vga.c
@@ -240,9 +240,6 @@ typedef struct CirrusVGAState {
 int cirrus_mmio_io_addr;
 unsigned long cirrus_lfb_addr;
 unsigned long cirrus_lfb_end;
-int aliases_enabled;
-uint32_t aliased_bank_base[2];
-uint32_t aliased_bank_limit[2];
 uint32_t cirrus_addr_mask;
 uint32_t linear_mmio_mask;
 uint8_t cirrus_shadow_gr0;
diff --git a/qemu/hw/vga_int.h b/qemu/hw/vga_int.h
index 912d977..c5a3992 100644
--- a/qemu/hw/vga_int.h
+++ b/qemu/hw/vga_int.h
@@ -147,18 +147,15 @@
 uint32_t last_palette[256]; \
 uint32_t last_ch_attr[CH_ATTR_SIZE]; /* XXX: make it dynamic */\
 unsigned long map_addr;\
-unsigned long map_end;
+unsigned long map_end;  \
+int32_t  aliases_enabled;   \
+int32_t  pad1;  \
+uint32_t aliased_bank_base[2];  \
+uint32_t aliased_bank_limit[2];
 
 
 typedef struct VGAState {
 VGA_STATE_COMMON
-
-int32_t  aliases_enabled;
-int32_t  pad1;
-uint32_t aliased_bank_base[2];
-uint32_t aliased_bank_limit[2];
-
-
 } VGAState;
 
 static inline int c6_to_8(int v)
-- 
1.5.3.8


-- 
Soren Hansen
Virtualisation specialist
Ubuntu Server Team
http://www.ubuntu.com/


signature.asc
Description: Digital signature
-
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2008.
http://clk.atdmt.com/MRT/go/vse012070mrt/direct/01/___
kvm-devel mailing list
kvm-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/kvm-devel

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 this an hour or so ago (whenever you last mentioned this).
Actually I used that patch from:


http://people.ubuntu.com/~soren/0001-Move-common-VGAState-attributes-to-VGA_STATE_COMMON.patch

This does not suffice to get X working under linux for me, though it
did get a bit further in that the vga window did resize (but then
nothing).



-
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2008.
http://clk.atdmt.com/MRT/go/vse012070mrt/direct/01/
___
kvm-devel mailing list
kvm-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/kvm-devel

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.

 Signed-off-by: Soren Hansen [EMAIL PROTECTED]
 ---
  qemu/hw/cirrus_vga.c |3 ---
  qemu/hw/vga_int.h|   17 +++--
  2 files changed, 7 insertions(+), 13 deletions(-)

 diff --git a/qemu/hw/cirrus_vga.c b/qemu/hw/cirrus_vga.c
 index 1915c73..35cee6b 100644
 --- a/qemu/hw/cirrus_vga.c
 +++ b/qemu/hw/cirrus_vga.c
 @@ -240,9 +240,6 @@ typedef struct CirrusVGAState {
  int cirrus_mmio_io_addr;
  unsigned long cirrus_lfb_addr;
  unsigned long cirrus_lfb_end;
 -int aliases_enabled;
 -uint32_t aliased_bank_base[2];
 -uint32_t aliased_bank_limit[2];
  uint32_t cirrus_addr_mask;
  uint32_t linear_mmio_mask;
  uint8_t cirrus_shadow_gr0;
 diff --git a/qemu/hw/vga_int.h b/qemu/hw/vga_int.h
 index 912d977..c620ce9 100644
 --- a/qemu/hw/vga_int.h
 +++ b/qemu/hw/vga_int.h
 @@ -145,20 +145,17 @@
  void (*cursor_draw_line)(struct VGAState *s, uint8_t *d, int y);\
  /* tell for each page if it has been updated since the last time */ \
  uint32_t last_palette[256]; \
 -uint32_t last_ch_attr[CH_ATTR_SIZE]; /* XXX: make it dynamic */  \
 -unsigned long map_addr;  \
 -unsigned long map_end;
 +uint32_t last_ch_attr[CH_ATTR_SIZE]; /* XXX: make it dynamic */ \
 +unsigned long map_addr; \
 +unsigned long map_end;  \
 +int32_t  aliases_enabled;   \
 +int32_t  pad1;  \
 +uint32_t aliased_bank_base[2];  \
 +uint32_t aliased_bank_limit[2];
  
  
  typedef struct VGAState {
  VGA_STATE_COMMON
 -
 -int32_t  aliases_enabled;
 -int32_t  pad1;
 -uint32_t aliased_bank_base[2];
 -uint32_t aliased_bank_limit[2];
 -
 -
  } VGAState;
  
  static inline int c6_to_8(int v)
   
 

it does solve the problem, but the question is
does this varibles are really VGA_COMMON varibles?

(well it look like alot of junk went to VGA_COMMON anyway)

-- 
woof.


-
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2008.
http://clk.atdmt.com/MRT/go/vse012070mrt/direct/01/
___
kvm-devel mailing list
kvm-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/kvm-devel