xen-ia64-devel

[Top] [All Lists]

[Xen-ia64-devel] Bug with dom0 large enough memmap trick

from [Tristan Gingold]

[Permanent Link][Original]

To:	xen-ia64-devel@xxxxxxxxxxxxxxxxxxx
Subject:	[Xen-ia64-devel] Bug with dom0 large enough memmap trick
From:	Tristan Gingold <Tristan.Gingold@xxxxxxxx>
Date:	Tue, 18 Apr 2006 17:41:15 +0200
Delivery-date:	Tue, 18 Apr 2006 08:37:18 -0700
Envelope-to:	www-data@xxxxxxxxxxxxxxxxxx
List-help:	<mailto:xen-ia64-devel-request@lists.xensource.com?subject=help>
List-id:	Discussion of the ia64 port of Xen <xen-ia64-devel.lists.xensource.com>
List-post:	<mailto:xen-ia64-devel@lists.xensource.com>
List-subscribe:	<http://lists.xensource.com/cgi-bin/mailman/listinfo/xen-ia64-devel>, <mailto:xen-ia64-devel-request@lists.xensource.com?subject=subscribe>
List-unsubscribe:	<http://lists.xensource.com/cgi-bin/mailman/listinfo/xen-ia64-devel>, <mailto:xen-ia64-devel-request@lists.xensource.com?subject=unsubscribe>
Sender:	xen-ia64-devel-bounces@xxxxxxxxxxxxxxxxxxx
User-agent:	KMail/1.5

Hi,

this issue will be moot with P2M/VP but it is here.

The last granule is computed from max_page:

   printf("find_memory: efi_memmap_walk returns max_page=%lx\n",max_page);
    /* this is a bad hack.  see dom_fw.c creation of EFI map for dom0 */
    max_page = (GRANULEROUNDDOWN(max_page << PAGE_SHIFT)
        - IA64_GRANULE_SIZE) >> PAGE_SHIFT;
    printf("find_memory: last granule reserved for dom0; xen max_page=%lx\n",
        max_page);

But we don't check there is no reserved data/code within this page.  Because 
this memory
is given to dom0 as conventionnal memory
  /* hack */ 
MAKE_MD(EFI_CONVENTIONAL_MEMORY,EFI_MEMORY_WB,last_start,last_end,1);
this hack may erase PAL/SAL/EFI code or data.

Tristan.



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

[More with this subject...]

<Prev in Thread]	Current Thread	[Next in Thread>
[Xen-ia64-devel] Bug with dom0 large enough memmap trick, Tristan Gingold <=

Previous by Date:	[Xen-ia64-devel] [PATCH][RFC][TAKE5] the P2M/VP patches, Isaku Yamahata
Next by Date:	Re: [Xen-ia64-devel] [PATCH][RFC][TAKE5] the P2M/VP patches, Alex Williamson
Previous by Thread:	[Xen-ia64-devel] [PATCH][RFC][TAKE5] the P2M/VP patches, Isaku Yamahata
Next by Thread:	[Xen-ia64-devel] alt_itlb_miss?, Alex Williamson
Indexes:	[Date] [Thread] [Top] [All Lists]