Liu, Jinsong wrote:
Kevin O'Connor wrote:
On Sat, Jul 10, 2010 at 12:41:18AM +0800, Liu,
Jinsong wrote:
Kevin O'Connor wrote:
On Thu, Jul 08, 2010 at 09:19:13PM +0800, Liu,
Jinsong wrote:
> Avi Kivity wrote:
>> Very nice. I thought about doing this but abandoned it as
>> unmaintainable. Using external functions and the ID variable,
>> however, reduces the mess to tolerable proportions, and gains us
>> a lot of flexibility. We can now have any combinations of
>> sockets and installed cpus.
>
> Agree, only 1 concern
> will it bring debugable/ scalable issue by hardcode aml code?
I've updated the patch (see below). This version documents how one
can build a new version of the Processor() ssdt snippet.
I've tested this under linux - there were a few bugs in the
previous patch. I also had to replace the dynamically created
CPUS array with a dynamically created NTFY method - which is a bit
more complicated.
Yeah, thanks Kevin.
After you done patch and draft test, our QA may do nightly test.
Hi Jinsong,
Have you had any feedback from tests?
Thanks,
-Kevin
Oh, I misunderstand.
I originally thought our QA will test after your patch send to
upstream, so we wait for it :)
I will talk with our QA this week to arrange test for it.
Thanks,
Jinsong
Kevin,
We have done test for your patch, result are:
1. For linux 2.6.32 hvm guest, it works OK, for both vcpu add and vcpu remove, and re-add,
re-remove, ...
2. For Window 2008 DataCenter hvm guest, it BSOD every time, QA feedback below:
==================
[QA] We test it as following:
1 copy bios.bin to /usr/local/share/qemu/
(BTW, we also test as your method , copy bios.bin to qemu-kvm/pc-bios, then at qemu-kvm
./configure make && make install)
2 create qemu img for Windows 2008 Datacenter
qemu-img create -b /share/xvs/img/Windows/ia32e_win2k8_datacenter.img -f qow2 test.img
3 create guest
qemu-system-x86_64 -hda test.img -m 1024 -smp 2,maxvcpus=16
4 blueScreen
==================
BTW, attached is the latest patch I received from you.
Do you have updated version?
Thanks,
Jinsong