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] Re: [PATCH 2] HVM vcpu add/remove: setup dsdt and madt i

To: Christoph Egger <Christoph.Egger@xxxxxxx>, "xen-devel@xxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxx>
Subject: Re: [Xen-devel] Re: [PATCH 2] HVM vcpu add/remove: setup dsdt and madt infrastructure for vcpu add/remove
From: Keir Fraser <keir.fraser@xxxxxxxxxxxxx>
Date: Wed, 09 Dec 2009 15:18:54 +0000
Cc: "Liu, Jinsong" <jinsong.liu@xxxxxxxxx>
Delivery-date: Wed, 09 Dec 2009 07:19:27 -0800
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to: <200912091452.49768.Christoph.Egger@xxxxxxx>
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/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=subscribe>
List-unsubscribe: <http://lists.xensource.com/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=unsubscribe>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
Thread-index: Acp41vQwxrPhyiN4QUKNvjS9HS24MgAC/coj
Thread-topic: [Xen-devel] Re: [PATCH 2] HVM vcpu add/remove: setup dsdt and madt infrastructure for vcpu add/remove
User-agent: Microsoft-Entourage/12.23.0.091001
On 09/12/2009 13:52, "Christoph Egger" <Christoph.Egger@xxxxxxx> wrote:

>> Hmm, well, the ACPI spec's example does have the entries shared I think.
>> Perhaps it does make sense then, although I'm still unsure whether
>> maintaining the MADT checksum is required?
> 
> Tools like acpidump or iasl may verify the checksum.

Yes, I think my complaints about the wasy the new ASL code goes about things
was probably unfounded. It's just the interface between hvmloader and the
DSDT that needs cleaning up, and moving the new ASL code patterns into
mk_dsdt.c. I hope Jinsong will re-spin the patches doing it that way, and
then I'll reconsider them.

 -- Keor



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