|
|
|
|
|
|
|
|
|
|
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
|
|
|
|
|