[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [Patch] Expand shadow memory area for using option ROM

  • To: Akio Takebe <takebe_akio@xxxxxxxxxxxxxx>, xen-devel <xen-devel@xxxxxxxxxxxxxxxxxxx>
  • From: Keir Fraser <keir.fraser@xxxxxxxxxxxxx>
  • Date: Mon, 29 Dec 2008 14:27:12 +0000
  • Cc:
  • Delivery-date: Mon, 29 Dec 2008 06:27:28 -0800
  • List-id: Xen developer discussion <xen-devel.lists.xensource.com>
  • Thread-index: AclpwYnnBbKliOPLgEmen4Tm+JkpcQ==
  • Thread-topic: [Xen-devel] [Patch] Expand shadow memory area for using option ROM

On 26/12/2008 04:46, "Akio Takebe" <takebe_akio@xxxxxxxxxxxxxx> wrote:

> This patch is:
> - replace e820 table from 0xe0000 to 0xeb000.
> - expand shadowr from 0xc0000-0xdffff to 0xc0000-0xe8fff.
> - fix scan size of VGA BIOS.
> after this patch, BIOS map is below.
>  BIOS map:
>   c0000-c8fff: VGA BIOS
>   c9000-d57ff: Etherboot ROM
>   d5800-e0fff: PCI Option ROMs
>   e9000-e919c: SMBIOS tables
>   f0000-fffff: Main BIOS


1. Why does the VGA BIOS 'scan size' need to be fixed? What's wrong with
scanning only up to 0xc8000 for it, and letting option ROMs continue to
reside from 0xc8000? Indeed, won't the VGABIOS always be at 0xc0000, so we
could actually only probe that address for it, and let option ROMs load as
low as 0xc0800?

2. It'd be nice to give E820_PHYSICAL_ADDRESS a definition in config.h and
then use that instead of hardcoding the address throughout rombios.c.

 -- Keir

Xen-devel mailing list



Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.