[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] vga: Fix type of map_addr/end.
From: |
Jan Kiszka |
Subject: |
Re: [Qemu-devel] [PATCH] vga: Fix type of map_addr/end. |
Date: |
Tue, 14 Jun 2011 20:14:10 +0200 |
User-agent: |
Mozilla/5.0 (X11; U; Linux i686 (x86_64); de; rv:1.8.1.12) Gecko/20080226 SUSE/2.0.0.12-1.1 Thunderbird/2.0.0.12 Mnenhy/0.7.5.666 |
On 2011-06-14 19:44, Richard Henderson wrote:
> These addresses have been passed through pci_to_cpu_addr,
> and thus need to be full target_phys_addr_t.
Basically correct, but you also need to change the types of
lfb_addr/end, no?
>
> Signed-off-by: Richard Henderson <address@hidden>
> Cc: Jan Kiszka <address@hidden>
> ---
> hw/vga_int.h | 4 ++--
> 1 files changed, 2 insertions(+), 2 deletions(-)
>
> Jan's recent patch series reminded me that I found a related
> problem in my alpha-softmmu port this past weekend. Note that
> the slight re-ordering of the members avoids extra padding in
> the structure due to alignment.
Yeah, and I just noticed a harmless mistyping in my patches while
thinking about your type changes.
Jan
>
>
> diff --git a/hw/vga_int.h b/hw/vga_int.h
> index d2811bd..b65775b 100644
> --- a/hw/vga_int.h
> +++ b/hw/vga_int.h
> @@ -109,9 +109,9 @@ typedef struct VGACommonState {
> uint32_t vram_size;
> uint32_t lfb_addr;
> uint32_t lfb_end;
> - uint32_t map_addr;
> - uint32_t map_end;
> uint32_t lfb_vram_mapped; /* whether 0xa0000 is mapped as ram */
> + target_phys_addr_t map_addr;
> + target_phys_addr_t map_end;
> uint32_t latch;
> uint8_t sr_index;
> uint8_t sr[256];
--
Siemens AG, Corporate Technology, CT T DE IT 1
Corporate Competence Center Embedded Linux