Re: [kvm-devel] [PATCH] Use a table to dispatch IO requests in kvmctl

2007-10-09 Thread Hollis Blanchard
On Tue, 2007-10-09 at 16:40 -0500, Anthony Liguori wrote: > This patch attempts to clean up kvmctl so that it can be more easily made to > work for multiple architectures and to support more emulation. > > It introduces a io dispatch mechanism. This mechanism supports subpage > granularity but is

Re: [kvm-devel] Windows XP Pro 64 installation issues with kvm-45 onIntel host

2007-10-09 Thread Dong, Eddie
[EMAIL PROTECTED] wrote: > Hello, > > Trying to install Windows XP Pro 64bits, the first phase goes > quite well (except > that it uses 100% of host's cpu), but crashes when trying to > reboot with the > following message: > > unhandled vm exit: 0x8021 > rax 0095b9e5 rbx 0095

[kvm-devel] SMP reboot issue

2007-10-09 Thread Dong, Eddie
SMP reboot failed with various phenomena such as user level "unhandled vm exit: 0x8021 ..." when rebooting SMP RHEL5U or "execption 8 (0)..." when rebooting XP etc. The reason is that we are not correctly reset VCPU. create_vcpu does some initialization work which should be part of reset job.

Re: [kvm-devel] kernel device reset support

2007-10-09 Thread Dong, Eddie
Avi Kivity wrote: > Dong, Eddie wrote: >> Avi Kivity wrote: >> >>> But, for an ungraceful reset, nothing prevents an AP from >>> issuing a reset? >>> >> >> Mmm, Yes, but I think current architecture can't handle this. >> The thread where AP issues "RESET" will continue run, which >> means it bec

Re: [kvm-devel] split ioapic reset API

2007-10-09 Thread Dong, Eddie
Avi Kivity wrote: > Dong, Eddie wrote: >> This one is a preparation for kernel devices reset support. >> >> >> >>Split ioapic reset API and export PIC reset API >> > > Please split into two patches, I need to fold them into the pic patch > and the ioapic patch. > Splited, please che

Re: [kvm-devel] out of memory page request

2007-10-09 Thread David Brown
> The attached patch should fix, though there's probably another problem > lurking in there. > > -- > Any sufficiently difficult bug is indistinguishable from a feature. > Thanks, the patch worked it didn't hoze the host anymore. - David Brown

Re: [kvm-devel] kvm-45 problems

2007-10-09 Thread Zhao, Yunfeng
This fix cannot resolve this issue. Against latest kvm commits, SMP linux with 4 vcpus still cannot boot up. But the issue will not happen , if adding "-smp4 -no-acpi". Here is the error info in dmesg BUG: soft lockup detected on CPU#0! [] softlockup_tick+0x98/0xa6 [] update_process_times+0x39/0

[kvm-devel] Firewire Support for KVM Guest?

2007-10-09 Thread Calin Brabandt
I'm a KVM newb and can't find any information or associated FAQ about KVM guest firewire support. If there's no KVM support, I'm seeking alternative recommendations, if KVM list members are so inclined. My goal is to boot a dedicated and proprietary Linux 2.4 kernel based AVX1 LiveCD under KVM.

Re: [kvm-devel] [PATCH 2/2] Allow memory to be specified in kvmctl

2007-10-09 Thread Anthony Liguori
Muli Ben-Yehuda wrote: > On Tue, Oct 09, 2007 at 05:09:19PM -0500, Anthony Liguori wrote: > > + default: + fprintf(stderr, + "Unrecongized memory suffix: %c\n", + *endp

Re: [kvm-devel] [PATCH 2/2] Allow memory to be specified in kvmctl

2007-10-09 Thread Muli Ben-Yehuda
On Tue, Oct 09, 2007 at 05:09:19PM -0500, Anthony Liguori wrote: > >>+ default: > >>+ fprintf(stderr, > >>+ "Unrecongized memory suffix: %c\n", > >>+ *endptr); > >>+

Re: [kvm-devel] [PATCH] Use a table to dispatch IO requests in kvmctl

2007-10-09 Thread Anthony Liguori
This patch won't apply with the previous two patches applied. I was meant to be independently reviewable. I have a merged patch that I'll send out tomorrow but please still comment on this version :-) Regards, Anthony Liguori Anthony Liguori wrote: > This patch attempts to clean up kvmctl so

Re: [kvm-devel] [PATCH 2/2] Allow memory to be specified in kvmctl

2007-10-09 Thread Anthony Liguori
Muli Ben-Yehuda wrote: >> while ((ch = getopt_long(argc, argv, sopts, lopts, &opt_ind)) != -1) { >> switch (ch) { >> @@ -367,6 +372,24 @@ int main(int argc, char **argv) >> case 'p': >> enter_protected_mode = true; >> break; >

Re: [kvm-devel] [PATCH 2/2] Allow memory to be specified in kvmctl

2007-10-09 Thread Muli Ben-Yehuda
On Tue, Oct 09, 2007 at 04:31:54PM -0500, Anthony Liguori wrote: > This patch adds a --memory option to kvmctl to allow the memory size of the > guest to be specified. > > Signed-off-by: Anthony Liguori <[EMAIL PROTECTED]> > > diff --git a/user/main.c b/user/main.c > index 9a57a24..7fc0924 10064

[kvm-devel] [PATCH] Use a table to dispatch IO requests in kvmctl

2007-10-09 Thread Anthony Liguori
This patch attempts to clean up kvmctl so that it can be more easily made to work for multiple architectures and to support more emulation. It introduces a io dispatch mechanism. This mechanism supports subpage granularity but is optimized to efficiently cover regions of pages too. It's a 64-bit

[kvm-devel] [PATCH 1/2] Remove memory size from linker script

2007-10-09 Thread Anthony Liguori
The memory size is currently hardcoded into the linker script (end_of_memory). This prevents the memory size from being specified dynamically in kvmctl. This patch adds a PIO port that can be used to query the memory size in the tests. Signed-off-by: Anthony Liguori <[EMAIL PROTECTED]> diff --git

[kvm-devel] [PATCH 2/2] Allow memory to be specified in kvmctl

2007-10-09 Thread Anthony Liguori
This patch adds a --memory option to kvmctl to allow the memory size of the guest to be specified. Signed-off-by: Anthony Liguori <[EMAIL PROTECTED]> diff --git a/user/main.c b/user/main.c index 9a57a24..7fc0924 100644 --- a/user/main.c +++ b/user/main.c @@ -57,6 +57,7 @@ static __thread int vcpu

[kvm-devel] [PATCH 1/2] enable user configure script for cross compile even more

2007-10-09 Thread Jerone Young
This patch patch applies on the latest git tree. This adds more cross compile capability to the user diretory configure script. Patch created with git this time :-) Signed-off-by: Jerone Young <[EMAIL PROTECTED]> diff --git a/user/configure b/user/configure index 20b9367..ff10268 100755 --- a/

[kvm-devel] [PATCH 2/2] [FIXED] enable user makefile for cross compile

2007-10-09 Thread Jerone Young
This patch enables enable makefile in user for cross compile. This time is a proper git formatted patch. Signed-off-by: Joerone Young <[EMAIL PROTECTED]> diff --git a/user/Makefile b/user/Makefile index 26eb530..522874b 100644 --- a/user/Makefile +++ b/user/Makefile @@ -1,14 +1,22 @@ include c

Re: [kvm-devel] [PATCH 1/2] [RESEND] Enable user directory configure script for cross compile

2007-10-09 Thread Jerone Young
I believe the problem is that I am not generating my patches in git format. I have been using a mercuial mq tree to generate the patches I have been sending. While they apply fine with patch command, apparently git needs the patches fomatted correctly. I'll get everything together again and use gi

Re: [kvm-devel] [PATCH 1/2] [RESEND] Enable user directory configure script for cross compile

2007-10-09 Thread Jerone Young
The patches are based on an older snapshot. Grabbing todays git tree I see there have been changes in user/Makefile. This would be why it is failing. On Tue, 2007-10-09 at 18:02 +0200, Avi Kivity wrote: > Jerone Young wrote: > > I believe the problem is that I am not generating my patches in git >

Re: [kvm-devel] RFC/patch: a very trivial patch towards portability

2007-10-09 Thread Avi Kivity
Carsten Otte wrote: > A header file named kvm_arch.h is being introduced that contains > prototypes for funtions in kvm_x86.c. > > What's the motivation for the new header? So we have a list of arch-dependent functions? Compiler-wise it could just as well remain in kvm.h. -- error compil

Re: [kvm-devel] RFC/patch: a very trivial patch towards portability

2007-10-09 Thread Avi Kivity
Carsten Otte wrote: > We've seen various big patches regarding portability from Christian and > Xianto, but none was merged to date. Maintaining a large patch set on > top of a quickly moving code base is painful. > I thought it might be cool to try to throw trivial patches at Avi that > obviously

[kvm-devel] RFC/patch: a very trivial patch towards portability

2007-10-09 Thread Carsten Otte
We've seen various big patches regarding portability from Christian and Xianto, but none was merged to date. Maintaining a large patch set on top of a quickly moving code base is painful. I thought it might be cool to try to throw trivial patches at Avi that obviously don't break things and push to

Re: [kvm-devel] [PATCH 1/2] [RESEND] Enable user directory configure script for cross compile

2007-10-09 Thread Avi Kivity
Jerone Young wrote: > I believe the problem is that I am not generating my patches in git > format. I have been using a mercuial mq tree to generate the patches I > have been sending. While they apply fine with patch command, apparently > git needs the patches fomatted correctly. > > git genera

Re: [kvm-devel] [PATCH] Abort if gcc4 was used to compile qemu and kvm is not enabled

2007-10-09 Thread Avi Kivity
Byron Stanoszek wrote: > Even when I first started using > KVM, I had no positive feedback that KVM was even working properly (was the > device node created? am I using a cpu with the correct extensions? etc). I > eventually figured out that if 'lsmod' showed kvm-intel.ko usage > 0, then I > knew i

Re: [kvm-devel] [PATCH 1/2] [RESEND] Enable user directory configure script for cross compile

2007-10-09 Thread Avi Kivity
Jerone Young wrote: > This patch enables configure script in user directory for cross compile. > > Still doesn't apply. What are you generating the patches against? -- error compiling committee.c: too many arguments to function -

Re: [kvm-devel] [PATCH] Abort if gcc4 was used to compile qemu and kvm is not enabled

2007-10-09 Thread Anthony Liguori
Carlo Marcelo Arenas Belon wrote: > On Mon, Oct 08, 2007 at 02:29:32PM -0500, Anthony Liguori wrote: > > 3) We do nothing, add a warning to the compilation for the users to read at > compile time (most of them not aware that they should be using gcc-3.x instead > as they are used to see all the

Re: [kvm-devel] kvm-45 problems

2007-10-09 Thread Zhao, Yunfeng
>> This may be due to NMIs or MCEs. Is it possible your machine is >> receiving them? >> > > >I pushed a fix for the NMI issue. Can you check if it fixes your hangs? [Yunfeng] Sure, the result will be available tomorrow. > > >-- >error compiling committee.c: too many arguments to function ---

Re: [kvm-devel] kernel device reset support

2007-10-09 Thread Avi Kivity
Dong, Eddie wrote: > Avi Kivity wrote: > >>> If BSP mp_state becomes VCPU_MP_STATE_UNINITIALIZED, current code >>> can't wakeup it. We need additional code that I am not aware of now. >>> >>> Current VCPU must be BSP, otherwise the code executing in Qemu will >>> have problem too. >>> >>>

Re: [kvm-devel] kernel device reset support

2007-10-09 Thread Dong, Eddie
Avi Kivity wrote: >> >> If BSP mp_state becomes VCPU_MP_STATE_UNINITIALIZED, current code >> can't wakeup it. We need additional code that I am not aware of now. >> >> Current VCPU must be BSP, otherwise the code executing in Qemu will >> have problem too. >> > > But, for an ungraceful reset,

Re: [kvm-devel] kvm-45 problems

2007-10-09 Thread Avi Kivity
Avi Kivity wrote: > Zhao, Yunfeng wrote: > >> We run some booting multiple guest tests in our nightly testing. >> Like booting two UP windows guests, booting one UP linux guest and one UP >> windows guest, and booting 4 UP linux guests. >> The first two cases have no problem in our testing. >>

Re: [kvm-devel] kernel device reset support

2007-10-09 Thread Avi Kivity
Dong, Eddie wrote: > Avi Kivity wrote: > >>> In native, RESET signal force every processor enter "RESET" status, >>> and then immediately after RESET signal is removed, all CPUs will >>> compete for BSP role, the winner continue execution, and failor will >>> be blocked till INIT/SIPI/SIPI. >>>

Re: [kvm-devel] kernel device reset support

2007-10-09 Thread Dong, Eddie
Avi Kivity wrote: >> In native, RESET signal force every processor enter "RESET" status, >> and then immediately after RESET signal is removed, all CPUs will >> compete for BSP role, the winner continue execution, and failor will >> be blocked till INIT/SIPI/SIPI. >> > > I meant, you could set b

Re: [kvm-devel] kvm-45 problems

2007-10-09 Thread Avi Kivity
Farkas Levente wrote: > Avi Kivity wrote: > >> Farkas Levente wrote: >> >>> Avi Kivity wrote: >>> >>> Zhao, Yunfeng wrote: > We run some booting multiple guest tests in our nightly testing. > Like booting two UP windows guests, booting one UP l

Re: [kvm-devel] kvm-45 problems

2007-10-09 Thread Farkas Levente
Avi Kivity wrote: > Farkas Levente wrote: >> Avi Kivity wrote: >> >>> Zhao, Yunfeng wrote: >>> We run some booting multiple guest tests in our nightly testing. Like booting two UP windows guests, booting one UP linux guest and one UP windows guest, and booting 4 UP linux gue

Re: [kvm-devel] kvm-45 problems

2007-10-09 Thread Avi Kivity
Farkas Levente wrote: > Avi Kivity wrote: > >> Zhao, Yunfeng wrote: >> >>> We run some booting multiple guest tests in our nightly testing. >>> Like booting two UP windows guests, booting one UP linux guest and one UP >>> windows guest, and booting 4 UP linux guests. >>> The first two case

Re: [kvm-devel] kernel device reset support

2007-10-09 Thread Avi Kivity
Avi Kivity wrote: >>> >>> >> For a graceful reboot, this one is not needed since every APs are >> already brought to HALT status before BSP issue RESET signal. But in >> case of non-graceful reboot, it is possible the VCPUs are still >> executing guest instruction, so we kick the VCPU a

Re: [kvm-devel] kvm-45 problems

2007-10-09 Thread Farkas Levente
Avi Kivity wrote: > Zhao, Yunfeng wrote: >> We run some booting multiple guest tests in our nightly testing. >> Like booting two UP windows guests, booting one UP linux guest and one UP >> windows guest, and booting 4 UP linux guests. >> The first two cases have no problem in our testing. >> And b

Re: [kvm-devel] kvm-45 problems

2007-10-09 Thread Avi Kivity
Zhao, Yunfeng wrote: > We run some booting multiple guest tests in our nightly testing. > Like booting two UP windows guests, booting one UP linux guest and one UP > windows guest, and booting 4 UP linux guests. > The first two cases have no problem in our testing. > And booting 4 UP linux guests

Re: [kvm-devel] [PATCH] kvm: bios: decrease the cpu counts in ACPI table to 15

2007-10-09 Thread Avi Kivity
Yang, Sheng wrote: > >From dc4cc1a804d9f7705648bdee3d7152054da8dea8 Mon Sep 17 00:00:00 2001 > From: Sheng Yang <[EMAIL PROTECTED]> > Date: Tue, 9 Oct 2007 16:22:08 +0800 > Subject: [PATCH] kvm: bios: decrease the cpu counts in ACPI table to 15 > > It seems Windows 2000 only support no more than 15

Re: [kvm-devel] global TSS?

2007-10-09 Thread Avi Kivity
Dong, Eddie wrote: > Avi: > It seems we are using global TSS, and init_rmode_tss will modify > the table each time when VCPU is reset. Should we move to per CPU TSS > for correct real mode emulation? > Why is it a problem? But, the real-mode tss can definitely be improved. Right now it

Re: [kvm-devel] out of memory page request

2007-10-09 Thread Avi Kivity
Dong, Eddie wrote: > [EMAIL PROTECTED] wrote: > >> David Brown wrote: >> Can you run 'gdb /path/to/kvm.ko' and post the output of 'disassemble ioapic_mmio_write' please? >>> Sorry for not getting back sooner. >>> >>> (gdb) disassemble ioapic_mmio_write >>>

Re: [kvm-devel] kernel device reset support

2007-10-09 Thread Avi Kivity
Dong, Eddie wrote: >>> >>> + >>> +/* >>> + * Reset VM. >>> + * >>> + */ >>> +int kvm_vm_reset(struct kvm *kvm) >>> +{ >>> + struct kvm_vcpu *vcpu; >>> + int i; >>> + >>> + kvm_reset_devices(kvm); >>> + for (i = 0; i < KVM_MAX_VCPUS; i++) { >>> + vcpu = kvm->vcpus[i]; >>> +

Re: [kvm-devel] Windows XP Pro 64 installation issues with kvm-45 onIntel host

2007-10-09 Thread François Delawarde
Dong, Eddie wrote: > [EMAIL PROTECTED] wrote: > >> Hello, >> >> Trying to install Windows XP Pro 64bits, the first phase goes >> quite well (except >> that it uses 100% of host's cpu), but crashes when trying to >> reboot with the >> following message: >> >> unhandled vm exit: 0x8021 >> rax

[kvm-devel] global TSS?

2007-10-09 Thread Dong, Eddie
Avi: It seems we are using global TSS, and init_rmode_tss will modify the table each time when VCPU is reset. Should we move to per CPU TSS for correct real mode emulation? thx,eddie - This SF.net email is sponsored b

[kvm-devel] [PATCH] kvm: bios: decrease the cpu counts in ACPI table to 15

2007-10-09 Thread Yang, Sheng
>From dc4cc1a804d9f7705648bdee3d7152054da8dea8 Mon Sep 17 00:00:00 2001 From: Sheng Yang <[EMAIL PROTECTED]> Date: Tue, 9 Oct 2007 16:22:08 +0800 Subject: [PATCH] kvm: bios: decrease the cpu counts in ACPI table to 15 It seems Windows 2000 only support no more than 15 cpus in ACPI table, otherwise

Re: [kvm-devel] kvm-45 problems

2007-10-09 Thread Zhao, Yunfeng
We run some booting multiple guest tests in our nightly testing. Like booting two UP windows guests, booting one UP linux guest and one UP windows guest, and booting 4 UP linux guests. The first two cases have no problem in our testing. And booting 4 UP linux guests may crash host, but this proble