[kvm-devel] [PATCH] Move common VGAState attributes to VGA_STATE_COMMON
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) -- 1.5.3.8 - 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
[kvm-devel] [PATCH] Move common VGAState attributes to VGA_STATE_COMMON
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) -- 1.5.3.8 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
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
Re: [kvm-devel] [PATCH] Move common VGAState attributes to VGA_STATE_COMMON
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]; > The changes to last_ch_attr, map_addr, etc. are gratuitous and will cause merge conflicts in the future. Please change only the relevant lines. -- 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
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
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
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