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 vcpu hotplug: Fix acpi method NTFY bug

To: "Liu, Jinsong" <jinsong.liu@xxxxxxxxx>, xen-devel <xen-devel@xxxxxxxxxxxxxxxxxxx>
Subject: Re: [Xen-devel] [PATCH] HVM vcpu hotplug: Fix acpi method NTFY bug
From: Keir Fraser <keir.fraser@xxxxxxxxxxxxx>
Date: Sun, 31 Jan 2010 17:06:47 +0000
Cc: "Zheng, Shaohui" <shaohui.zheng@xxxxxxxxx>, "Ke, Liping" <liping.ke@xxxxxxxxx>, "Jiang, Yunhong" <yunhong.jiang@xxxxxxxxx>
Delivery-date: Sun, 31 Jan 2010 09:07:15 -0800
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to: <EB8593BCECAB3D40A8248BE0B6400A3835ACAB98@xxxxxxxxxxxxxxxxxxxxxxxxxxxxx>
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: AcqfEjBdAvKTlDZES9GP92/daWtXOgBn3NcBAABXF2AAAn8xkQBvuV2QAAb4ydA=
Thread-topic: [Xen-devel] [PATCH] HVM vcpu hotplug: Fix acpi method NTFY bug
User-agent: Microsoft-Entourage/12.23.0.091001
On 31/01/2010 14:04, "Liu, Jinsong" <jinsong.liu@xxxxxxxxx> wrote:

> According to your suggestion, I plan to modify the patch as:
> 1. cancel method NTFY, merge it into method PRSC;
> 2. at method PRSC, only scan 'maxvcpus' times. we can transfer 'maxvcpus'
> (which comes from cmdline) from qemu to dsdt side through bios_info;
> 3. at mk_dsdt.c, add some comments to make it clear;
> Do you think it OK?

I don't even mind if you don't do #2. That's up to you. Bear in mind that to
be able to merge NTFY into PRSC, you will have to completely unroll PRSC
(one way to view it is moving the loops out of the DSDT and into mk_dsdt.c,
to be executed at build time and produce straight-line code). That would
mean you emit the code to check against max_vcpus 128 times. Is it worth it?
I don't know.

 -- Keir



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