Isaku Yamahata, le Thu 15 Jan 2009 21:22:59 +0900, a écrit :
>     don't use magic number 0xff00000.
>     
> diff --git a/hw/cirrus_vga.c b/hw/cirrus_vga.c
> index d6a9235..f2ec05b 100644
> --- a/hw/cirrus_vga.c
> +++ b/hw/cirrus_vga.c
> @@ -3165,7 +3165,7 @@ static int cirrus_vga_load(QEMUFile *f, void *opaque, 
> int version_id)
>      if (!s->vram_gmfn) {
>          /* Old guest, VRAM is not mapped, we have to restore it
>           * ourselves */
> -        s->vram_gmfn = vga_acc ? s->lfb_addr : 0xff000000;
> +        s->vram_gmfn = vga_acc ? s->lfb_addr : VRAM_RESERVED_ADDRESS;
>          xen_vga_populate_vram(s->vram_gmfn, s->vram_size);
>      }
>      xen_vga_vram_map(s->vram_gmfn, s->vram_size);

Ah, so Stefano's patch actually got applied??  I thought Keir preferred
to stay with allocating video memory from hvmloader.

Note that I do not really care about which way we go, what's important
to keep in mind is that if we revert to qemu populating the video
memory, stubdom-based qemu can _not_ work with the default Xen
parameters, because of memory ballooning.

Samuel

_______________________________________________
Xen-ia64-devel mailing list
Xen-ia64-devel@lists.xensource.com
http://lists.xensource.com/xen-ia64-devel

Reply via email to