WARNING - OLD ARCHIVES

This is an archived copy of the Xen.org mailing list, which we have preserved to ensure that existing links to archives are not broken. The live archive, which contains the latest emails, can be found at http://lists.xen.org/
   
 
 
Xen 
 
Home Products Support Community News
 
   
 

xen-devel

RE: [Xen-devel] [PATCH]HVM E820 Table cleanup

To: "Keir Fraser" <Keir.Fraser@xxxxxxxxxxxx>, "Wang, Winston L" <winston.l.wang@xxxxxxxxx>, "Ian Pratt" <m+Ian.Pratt@xxxxxxxxxxxx>, <Christian.Limpach@xxxxxxxxxxxx>
Subject: RE: [Xen-devel] [PATCH]HVM E820 Table cleanup
From: "Li, Xin B" <xin.b.li@xxxxxxxxx>
Date: Wed, 18 Oct 2006 16:53:00 +0800
Cc: xen-devel@xxxxxxxxxxxxxxxxxxx
Delivery-date: Wed, 18 Oct 2006 01:53:39 -0700
Envelope-to: www-data@xxxxxxxxxxxxxxxxxx
List-help: <mailto:xen-devel-request@lists.xensource.com?subject=help>
List-id: Xen developer discussion <xen-devel.lists.xensource.com>
List-post: <mailto:xen-devel@lists.xensource.com>
List-subscribe: <http://lists.xensource.com/cgi-bin/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=subscribe>
List-unsubscribe: <http://lists.xensource.com/cgi-bin/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=unsubscribe>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
Thread-index: AcZJ+X/aibN5jDuIR5qaQWxIPo6lRgAAB3DgGcLc9lABXN7R8AD6IxmgAAZ8ITANKTdBwAAApCCwAEQgH2cAl8q5UA==
Thread-topic: [Xen-devel] [PATCH]HVM E820 Table cleanup
>> We found E820 table miss report the ACPI NVS/Data information which
>> trigger the Windows HCT test complain, attached please review for the
>> cleanup patch.
>> 
>> Signed-off-by: Xin Li <xin.b.li@xxxxxxxxx>
>> Signed-off-by: Winston Wang <winston.l.wang@xxxxxxxxx>
>
>Apart from adding a page-size ACPI region in low memory, this patch
>*removes* thirteen pages from the top of memory. Was that 
>region just bogus?
>

Yes, it's bogus :-(

>The hvmloader code which copies the ACPI tables only checks 
>that the tables
>will fit into 0xEA000-0xEFFFF. Shouldn't e820 declare a region 
>of that size,
>therefore (i.e., 0x50000, not 0x10000)?
>

According to ACPI spec, OS may use e820 ACPI_DATA region as RAM after OS
boots up (but acturally Linux doesn't use it).
But for memory ACPI tables reside, OS can not use like that.
-Xin

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

<Prev in Thread] Current Thread [Next in Thread>