RE: PCI PT: irq issue

2008-06-22 Thread Han, Weidong
Amit Shah wrote: > On Thursday 19 June 2008 10:17:29 Amit Shah wrote: >> * On Wednesday 18 June 2008 18:26:16 Ben-Ami Yassour wrote: >>> Amit, >>> >>> With the current implementation we have an issue if the driver on >>> the host was never loaded. >>> >>> To be able to run kvm with passthrough w

Re: [PATCH 1/4] KVM: Report hardware virtualization features

2008-06-22 Thread Yang, Sheng
On Monday 23 June 2008 10:40:33 Avi Kivity wrote: > Yang, Sheng wrote: > > On Sunday 22 June 2008 20:21:37 Avi Kivity wrote: > >> Dor Laor wrote: > Yes, this is definitely helpful. However, I think that users will > expect cpu flags under /proc/cpuinfo. > > Perhaps we should ad

Re: [patch 7/7] KVM: in-kernel ACPI C2 idle emulation

2008-06-22 Thread Avi Kivity
Marcelo Tosatti wrote: Some guests fail to process the _CST notification which invalidates the C2 state. Which? Emulate C2 similarly to HLT for those cases. Please split the API change from actual addition of the ACPI emulation. Signed-off-by: Marcelo Tosatti <[EMAIL PROTECTED]>

Re: KVM power usage - high wakeup rates

2008-06-22 Thread Avi Kivity
Anders wrote: Anthony Liguori wrote: With the default options, my idle winxp guest causes about 550 wakeups/sec which is pretty close to what I'd expect. I don't think you'll get much better than that without using a dynticks guest. According to powertop, I had about 15 kvm wakeups/sec with

Re: [PATCH 1/4] KVM: Report hardware virtualization features

2008-06-22 Thread Avi Kivity
Yang, Sheng wrote: On Sunday 22 June 2008 20:21:37 Avi Kivity wrote: Dor Laor wrote: Yes, this is definitely helpful. However, I think that users will expect cpu flags under /proc/cpuinfo. Perhaps we should add a new line 'virt flags' to /proc/cpuinfo? I think all the features are re

Re: [PATCH 0 of 3] [kvm-userspace] Enable KVM TRACE support for PowerPC

2008-06-22 Thread Avi Kivity
Jerone Young wrote: This set of patches adds the ability to compile and use kvm userspace tools with PowerPC. It also adds the ability for other archs to add support. Also added are new events into the format file. These have been submitted for inclusion into events defined in the kernel.

Re: [PATCH 1 of 3] Remove use of bit fields in kvm trace structure

2008-06-22 Thread Avi Kivity
Jerone Young wrote: This patch fixes kvmtrace use on big endian systems. When using bit fields the compiler will lay data out in the wrong order expected when laid down into a file. This fixes it by using one variable instead of using bit fields. Signed-off-by: Jerone Young <[EMAIL PROTECTED]

Re: KVM: x86: move vapic page handling out of fast path

2008-06-22 Thread Avi Kivity
Marcelo Tosatti wrote: On Sun, Jun 22, 2008 at 08:00:11AM +0300, Avi Kivity wrote: Marcelo Tosatti wrote: I fail to see the point of handling the vapic page grab and ref counting in __vcpu_run's heavyweight enter/exit path. It's to avoid pinning the page indefinitely.

Re: [PATCH 1/4] KVM: Report hardware virtualization features

2008-06-22 Thread Yang, Sheng
On Sunday 22 June 2008 20:21:37 Avi Kivity wrote: > Dor Laor wrote: > >> Yes, this is definitely helpful. However, I think that users will > >> expect cpu flags under /proc/cpuinfo. > >> > >> Perhaps we should add a new line 'virt flags' to /proc/cpuinfo? I think > >> all the features are reporte

Re: [PATCH] KVM: VMX: Flush all VPID tagged TLB when VMXON

2008-06-22 Thread Yang, Sheng
On Sunday 22 June 2008 15:35:35 Avi Kivity wrote: > Yang, Sheng wrote: > > From efba05901f8ffca03208217d075f679f78f3469d Mon Sep 17 00:00:00 2001 > > From: Sheng Yang <[EMAIL PROTECTED]> > > Date: Tue, 17 Jun 2008 17:41:10 +0800 > > Subject: [PATCH] KVM: VMX: Flush all VPID tagged TLB when VMXON >

Re: Freeze with KVM_CLOCK, undocumented dependency VIRTIO_BLK -> VIRTIO_PCI

2008-06-22 Thread Bernhard Schmidt
Hi, a) when I enable KVM_CLOCK in the kernel configuration the guest system freezes after PCI: Found IRQ 11 for device :00:04.0 vda: vda1 vda2 TCP cubic registered NET: Registered protocol family 10 NET: Registered protocol family 17 Using IPI Shortcut mode There are some outstanding patc

Re: [Bug] Real mode guests never wake up after an HLT instruction

2008-06-22 Thread Mohammed Gamal
On Sun, Jun 22, 2008 at 10:15 AM, Avi Kivity <[EMAIL PROTECTED]> wrote: > Mohammed Gamal wrote: >> >> CPU Model: Intel(R) Core(TM)2 Duo CPU T7250 @ 2.00GHz >> KVM version: kvm-69-1687-gd660add >> Host Kernel: 2.6.26-rc5 (x86_64) >> Guests: FreeDOS and Minix 3.1.2 32-bit >> Command: qemu-system-x8

Re: RFC: cache_regs in kvm_emulate_pio

2008-06-22 Thread Marcelo Tosatti
On Sun, Jun 22, 2008 at 08:16:19AM +0300, Avi Kivity wrote: > Looks good, but we can aim higher. The cache_regs() API was always > confusing (I usually swap the two parts). If we replace all ->regs > access with accessors, we can make it completely transparent. > > It will be tricky in the em

Re: KVM: x86: move vapic page handling out of fast path

2008-06-22 Thread Marcelo Tosatti
On Sun, Jun 22, 2008 at 08:00:11AM +0300, Avi Kivity wrote: > Marcelo Tosatti wrote: >> I fail to see the point of handling the vapic page grab and ref >> counting in __vcpu_run's heavyweight enter/exit path. >> >> > > It's to avoid pinning the page indefinitely. > >> So move it to kvm_lapic_set

Re: KVM power usage - high wakeup rates

2008-06-22 Thread Anders
Anthony Liguori wrote: With the default options, my idle winxp guest causes about 550 wakeups/sec which is pretty close to what I'd expect. I don't think you'll get much better than that without using a dynticks guest. According to powertop, I had about 15 kvm wakeups/sec with my previous se

[ kvm-Bugs-1998355 ] IO Performance

2008-06-22 Thread SourceForge.net
Bugs item #1998355, was opened at 2008-06-20 00:11 Message generated for change (Comment added) made by bjrosen You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=893831&aid=1998355&group_id=180599 Please note that this message will contain a full copy of the comment t

Re: [PATCH 1/4] KVM: Report hardware virtualization features

2008-06-22 Thread Avi Kivity
Dor Laor wrote: Yes, this is definitely helpful. However, I think that users will expect cpu flags under /proc/cpuinfo. Perhaps we should add a new line 'virt flags' to /proc/cpuinfo? I think all the features are reported using msrs, so it can be done from arch/x86/kernel/cpu/proc.c witho

Re: [PATCH 1/4] KVM: Report hardware virtualization features

2008-06-22 Thread Dor Laor
On Sun, 2008-06-22 at 09:49 +0300, Avi Kivity wrote: > Yang, Sheng wrote: > > From f02d2ccf01e8671d2da517f14a908d1df1cc42ad Mon Sep 17 00:00:00 2001 > > From: Sheng Yang <[EMAIL PROTECTED]> > > Date: Thu, 19 Jun 2008 18:41:26 +0800 > > Subject: [PATCH] KVM: Report hardware virtualization features >

Re: [PATCH] KVM: VMX: Flush all VPID tagged TLB when VMXON

2008-06-22 Thread Avi Kivity
Yang, Sheng wrote: From efba05901f8ffca03208217d075f679f78f3469d Mon Sep 17 00:00:00 2001 From: Sheng Yang <[EMAIL PROTECTED]> Date: Tue, 17 Jun 2008 17:41:10 +0800 Subject: [PATCH] KVM: VMX: Flush all VPID tagged TLB when VMXON Why is this needed? Don't we flush a vpid immediately after a

Re: [Bug] Real mode guests never wake up after an HLT instruction

2008-06-22 Thread Avi Kivity
Mohammed Gamal wrote: CPU Model: Intel(R) Core(TM)2 Duo CPU T7250 @ 2.00GHz KVM version: kvm-69-1687-gd660add Host Kernel: 2.6.26-rc5 (x86_64) Guests: FreeDOS and Minix 3.1.2 32-bit Command: qemu-system-x86_64 -hda /media/sda6/ Real mode guests (namely Minix 3 and FreeDOS with HIMEM XMS driver

Re: kvm-70 + RHEL4 + 2.6.25.7 == invalid module format

2008-06-22 Thread Avi Kivity
(re-adding kvm@) Anoosh Hosseini (anoosh) wrote: BTW I see this issue with kvm-69. But not in kvm-68. Right now use kvm-68's kvm-intel module along with kvm-69 simulation. Works fine. -anoosh I don't think anything special happened between kvm-68 and kvm-69. This might be a toolchain prob