Kevin O'Connor wrote:
On Wed, Jun 16, 2010 at 12:19:24PM +0800, Liu, Jinsong wrote:
Gleb Natapov wrote:
FWIW looks good to me. This patch removes dynamic creation of Processor scope and supports only 15/255 cpus configuration, but given that new code is much more complex that old one I don't see other possibilities (unless we'll add aml compiler to seabios runtime).
I'm okay with the approach (two SSDTs selected by cpu count). The build changes don't look right (15.c is too cryptic). However, more importantly, I want to see where we end up going with the qemu/seabios acpi interface discussion before committing this.
Thanks for comments!
In your last comments, you prefer not to use iasl to compile seabios. Now I update the patch as attached (only need update Makefile): 1. cancel cryptic name like 15.c from Makefile, and we give detailed description for cpu number in new version; 2. user don't need iasl again to compile seabios, except if they want to change dsdt/ssdt itself;
The process to apply patch and build in my desktop is: $ git apply kvm-vcpu-hotplug-seabios.patch $ git add src/acpi-ssdt.dsl $ git add src/mk_ssdt.c $ git add src/acpi-ssdt-15.hex $ git add src/acpi-ssdt-anycpu.hex $ git commit -a $ make
We test the updated patch, it works OK with linux2.6.32 hot-add and hot-remove.