Re: [PATCH v5 00/12] KVM: introduce readonly memslot

2012-08-15 Thread Xiao Guangrong
On 08/14/2012 11:25 PM, Marcelo Tosatti wrote: > On Tue, Aug 14, 2012 at 10:58:07AM +0800, Xiao Guangrong wrote: >> On 08/14/2012 01:39 AM, Marcelo Tosatti wrote: >>> On Sat, Aug 11, 2012 at 11:36:20AM +0800, Xiao Guangrong wrote: On 08/11/2012 02:14 AM, Marcelo Tosatti wrote: > On Tue, Au

[PATCH] kvm/fpu: Enable fully eager restore kvm FPU

2012-08-15 Thread Xudong Hao
Enable KVM FPU fully eager restore, if there is other FPU state which isn't tracked by CR0.TS bit. Tested with these cases: 1) SpecCPU2000 workload( 1 VM, 2 VMs) 2) Program for floating point caculate Signed-off-by: Xudong Hao --- arch/x86/include/asm/kvm.h |4 arch/x86/kvm/x86.c

[PATCH] Document IACx/DACx registers access using ONE_REG API

2012-08-15 Thread Bharat Bhushan
Patch to access the debug registers (IACx/DACx) using ONE_REG api was sent earlier. But that missed the respective documentation. Also corrected the index number referencing in section 4.69 Signed-off-by: Bharat Bhushan --- Documentation/virtual/kvm/api.txt |9 +++-- 1 files changed, 7

[Bug 42980] BUG in gfn_to_pfn_prot

2012-08-15 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=42980 --- Comment #15 from Luke-Jr 2012-08-15 22:47:39 --- 3.4.0: http://luke.dashjr.org/tmp/code/20120624_002.jpg -- Configure bugmail: https://bugzilla.kernel.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You

[PATCH 2/2] KVM: PPC: 440: Implement mfdcrx

2012-08-15 Thread Alexander Graf
Signed-off-by: Alexander Graf --- arch/powerpc/kvm/44x_emulate.c |3 +++ 1 files changed, 3 insertions(+), 0 deletions(-) diff --git a/arch/powerpc/kvm/44x_emulate.c b/arch/powerpc/kvm/44x_emulate.c index ea57631..30ea3a7 100644 --- a/arch/powerpc/kvm/44x_emulate.c +++ b/arch/powerpc/kvm/44x

[PATCH 1/2] KVM: PPC: 440: Implement mtdcrx

2012-08-15 Thread Alexander Graf
Signed-off-by: Alexander Graf --- arch/powerpc/kvm/44x_emulate.c |3 +++ 1 files changed, 3 insertions(+), 0 deletions(-) diff --git a/arch/powerpc/kvm/44x_emulate.c b/arch/powerpc/kvm/44x_emulate.c index c8c6157..ea57631 100644 --- a/arch/powerpc/kvm/44x_emulate.c +++ b/arch/powerpc/kvm/44x

[Bug 42980] BUG in gfn_to_pfn_prot

2012-08-15 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=42980 --- Comment #14 from Luke-Jr 2012-08-15 22:38:45 --- I'm not sure if it was 3.4.0, 3.4.3, or 3.4.4. Since May 17, I have been building all my kernels (including those) with this patch applied. -- Configure bugmail: https://bugzilla.kernel.

[Bug 42980] BUG in gfn_to_pfn_prot

2012-08-15 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=42980 --- Comment #13 from Alan 2012-08-15 22:34:02 --- Which kernel ? -- Configure bugmail: https://bugzilla.kernel.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are watching the assignee of the bug. -- To

[Bug 42980] BUG in gfn_to_pfn_prot

2012-08-15 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=42980 Luke-Jr changed: What|Removed |Added Status|RESOLVED|REOPENED Resolution|CODE_FIX

[Bug 28402] MacOSX guest has SMP problem without -kvm-irqchip option

2012-08-15 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=28402 Alan changed: What|Removed |Added Status|NEW |NEEDINFO CC|

[Bug 43339] Wrong Pci-Bridge Header Type check.

2012-08-15 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=43339 Alan changed: What|Removed |Added Status|NEW |RESOLVED CC|

[Bug 42980] BUG in gfn_to_pfn_prot

2012-08-15 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=42980 Alan changed: What|Removed |Added Status|NEW |RESOLVED CC|

[Bug 42779] KVM domain hangs after loading initrd with Xenomai kernel

2012-08-15 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=42779 Alan changed: What|Removed |Added Status|NEW |RESOLVED CC|

Re: [PATCH 0/4] [PULL] qemu-kvm.git uq/master queue

2012-08-15 Thread Anthony Liguori
Marcelo Tosatti writes: > The following changes since commit 873359d411eeb380906761e46839a2b705dbcf75: > > Merge branch 'linux-user.next' of > git://git.linaro.org/people/pmaydell/qemu-arm (2012-08-14 19:50:22 +) > > are available in the git repository at: > > git://git.kernel.org/pub/sc

Re: [PATCH v8 0/6] kvm: level irqfd support

2012-08-15 Thread Alex Williamson
On Wed, 2012-08-15 at 22:22 +0300, Michael S. Tsirkin wrote: > On Wed, Aug 15, 2012 at 11:36:31AM -0600, Alex Williamson wrote: > > On Wed, 2012-08-15 at 17:28 +0300, Michael S. Tsirkin wrote: > > > On Fri, Aug 10, 2012 at 04:37:08PM -0600, Alex Williamson wrote: > > > > v8: > > > > > > > > Trying

Re: [PATCH 19/38] KVM: PPC: Add cache flush on page map

2012-08-15 Thread Scott Wood
On 08/15/2012 02:29 PM, Alexander Graf wrote: > > On 15.08.2012, at 21:05, Scott Wood wrote: > >> On 08/15/2012 01:58 PM, Alexander Graf wrote: >>> >>> On 15.08.2012, at 20:56, Scott Wood wrote: >>> On 08/15/2012 01:51 PM, Alexander Graf wrote: > > On 15.08.2012, at 20:33, Scott Wood

Re: [PATCH 19/38] KVM: PPC: Add cache flush on page map

2012-08-15 Thread Alexander Graf
On 15.08.2012, at 21:05, Scott Wood wrote: > On 08/15/2012 01:58 PM, Alexander Graf wrote: >> >> On 15.08.2012, at 20:56, Scott Wood wrote: >> >>> On 08/15/2012 01:51 PM, Alexander Graf wrote: On 15.08.2012, at 20:33, Scott Wood wrote: > On 08/15/2012 01:29 PM, Alexander Gr

Re: [PATCH v8 1/6] kvm: Allow filtering of acked irqs

2012-08-15 Thread Michael S. Tsirkin
On Wed, Aug 15, 2012 at 10:47:38AM -0600, Alex Williamson wrote: > On Wed, 2012-08-15 at 15:27 +0300, Michael S. Tsirkin wrote: > > On Fri, Aug 10, 2012 at 04:37:17PM -0600, Alex Williamson wrote: > > > Registering an kvm_irq_ack_notifier with kian.irq_source_id < 0 > > > retains existing behavior,

Re: [PATCH v8 0/6] kvm: level irqfd support

2012-08-15 Thread Michael S. Tsirkin
On Wed, Aug 15, 2012 at 11:36:31AM -0600, Alex Williamson wrote: > On Wed, 2012-08-15 at 17:28 +0300, Michael S. Tsirkin wrote: > > On Fri, Aug 10, 2012 at 04:37:08PM -0600, Alex Williamson wrote: > > > v8: > > > > > > Trying a new approach. Nobody seems to like the internal IRQ > > > source ID o

Re: [PATCH v2] kvm-all.c: Move init of irqchip_inject_ioctl out of kvm_irqchip_create()

2012-08-15 Thread Marcelo Tosatti
On Wed, Aug 15, 2012 at 12:08:13PM +0100, Peter Maydell wrote: > Move the init of the irqchip_inject_ioctl field of KVMState out of > kvm_irqchip_create() and into kvm_init(), so that kvm_set_irq() > can be used even when no irqchip is created (for architectures > that support async interrupt notif

Re: [PATCH V2] KVM: x86: disable stealtime on reboot to avoid mem corruption

2012-08-15 Thread Marcelo Tosatti
On Wed, Aug 15, 2012 at 04:00:40PM +0200, Florian Westphal wrote: > else, host continues to update stealtime after reboot, > which can corrupt e.g. initramfs area. > found when tracking down initramfs unpack error on initial reboot > (with qemu-kvm -smp 2, no problem with single-core). > > Signed-

Re: [PATCH 16/38] KVM: PPC: BookE: Add check_requests helper function

2012-08-15 Thread Marcelo Tosatti
On Tue, Aug 14, 2012 at 07:10:43PM -0500, Scott Wood wrote: > On 08/14/2012 06:04 PM, Alexander Graf wrote: > > We need a central place to check for pending requests in. Add one that > > only does the timer check we already do in a different place. > > > > Later, this central function can be exten

Re: [PATCH RESEND] KVM: MMU: Fix mmu_shrink() so that it can free mmu pages as intended

2012-08-15 Thread Marcelo Tosatti
On Wed, Aug 15, 2012 at 11:11:51AM +0900, Takuya Yoshikawa wrote: > On Tue, 14 Aug 2012 12:17:12 -0300 > Marcelo Tosatti wrote: > > > - if (kvm->arch.n_used_mmu_pages > 0) { > > - if (!nr_to_scan--) > > - break; > > -- (*1) > > >

Re: [PATCH 19/38] KVM: PPC: Add cache flush on page map

2012-08-15 Thread Scott Wood
On 08/15/2012 01:58 PM, Alexander Graf wrote: > > On 15.08.2012, at 20:56, Scott Wood wrote: > >> On 08/15/2012 01:51 PM, Alexander Graf wrote: >>> >>> On 15.08.2012, at 20:33, Scott Wood wrote: >>> On 08/15/2012 01:29 PM, Alexander Graf wrote: > Ah, if I read Ben's comment correctly we

Re: [PATCH 19/38] KVM: PPC: Add cache flush on page map

2012-08-15 Thread Alexander Graf
On 15.08.2012, at 20:56, Scott Wood wrote: > On 08/15/2012 01:51 PM, Alexander Graf wrote: >> >> On 15.08.2012, at 20:33, Scott Wood wrote: >> >>> On 08/15/2012 01:29 PM, Alexander Graf wrote: On 15.08.2012, at 20:27, Alexander Graf wrote: > > On 15.08.2012, at 20:16,

Re: [PATCH 19/38] KVM: PPC: Add cache flush on page map

2012-08-15 Thread Scott Wood
On 08/15/2012 01:51 PM, Alexander Graf wrote: > > On 15.08.2012, at 20:33, Scott Wood wrote: > >> On 08/15/2012 01:29 PM, Alexander Graf wrote: >>> >>> On 15.08.2012, at 20:27, Alexander Graf wrote: >>> On 15.08.2012, at 20:16, Scott Wood wrote: > On 08/15/2012 01:01 PM, Alexan

Re: [PATCH 19/38] KVM: PPC: Add cache flush on page map

2012-08-15 Thread Alexander Graf
On 15.08.2012, at 20:33, Scott Wood wrote: > On 08/15/2012 01:29 PM, Alexander Graf wrote: >> >> On 15.08.2012, at 20:27, Alexander Graf wrote: >> >>> >>> On 15.08.2012, at 20:16, Scott Wood wrote: >>> On 08/15/2012 01:01 PM, Alexander Graf wrote: > > On 15.08.2012, at 19:47, Sc

Re: [PATCH 19/38] KVM: PPC: Add cache flush on page map

2012-08-15 Thread Scott Wood
On 08/15/2012 01:29 PM, Alexander Graf wrote: > > On 15.08.2012, at 20:27, Alexander Graf wrote: > >> >> On 15.08.2012, at 20:16, Scott Wood wrote: >> >>> On 08/15/2012 01:01 PM, Alexander Graf wrote: On 15.08.2012, at 19:47, Scott Wood wrote: > On 08/15/2012 12:27 PM, Alexande

Re: [PATCH 19/38] KVM: PPC: Add cache flush on page map

2012-08-15 Thread Alexander Graf
On 15.08.2012, at 20:27, Alexander Graf wrote: > > On 15.08.2012, at 20:16, Scott Wood wrote: > >> On 08/15/2012 01:01 PM, Alexander Graf wrote: >>> >>> On 15.08.2012, at 19:47, Scott Wood wrote: >>> On 08/15/2012 12:27 PM, Alexander Graf wrote: > > On 15.08.2012, at 19:26, Scot

Re: [PATCH 19/38] KVM: PPC: Add cache flush on page map

2012-08-15 Thread Alexander Graf
On 15.08.2012, at 20:16, Scott Wood wrote: > On 08/15/2012 01:01 PM, Alexander Graf wrote: >> >> On 15.08.2012, at 19:47, Scott Wood wrote: >> >>> On 08/15/2012 12:27 PM, Alexander Graf wrote: On 15.08.2012, at 19:26, Scott Wood wrote: > On 08/15/2012 04:52 AM, Alexander Gr

Re: [PATCH 19/38] KVM: PPC: Add cache flush on page map

2012-08-15 Thread Scott Wood
On 08/15/2012 01:01 PM, Alexander Graf wrote: > > On 15.08.2012, at 19:47, Scott Wood wrote: > >> On 08/15/2012 12:27 PM, Alexander Graf wrote: >>> >>> On 15.08.2012, at 19:26, Scott Wood wrote: >>> On 08/15/2012 04:52 AM, Alexander Graf wrote: > > On 15.08.2012, at 03:23, Scott Wood

Re: [Qemu-devel] [PATCH 0/4] [PULL] qemu-kvm.git uq/master queue

2012-08-15 Thread Marcelo Tosatti
On Wed, Aug 15, 2012 at 08:46:21AM +0100, Peter Maydell wrote: > On 15 August 2012 00:59, Marcelo Tosatti wrote: > > The following changes since commit 873359d411eeb380906761e46839a2b705dbcf75: > > > > Merge branch 'linux-user.next' of > > git://git.linaro.org/people/pmaydell/qemu-arm (2012-08-

Re: [PATCH 19/38] KVM: PPC: Add cache flush on page map

2012-08-15 Thread Alexander Graf
On 15.08.2012, at 19:47, Scott Wood wrote: > On 08/15/2012 12:27 PM, Alexander Graf wrote: >> >> On 15.08.2012, at 19:26, Scott Wood wrote: >> >>> On 08/15/2012 04:52 AM, Alexander Graf wrote: On 15.08.2012, at 03:23, Scott Wood wrote: > On 08/14/2012 06:04 PM, Alexander Gr

Re: [PATCH 3/5] KVM: PPC: Book3S HV: Handle memory slot deletion and modification correctly

2012-08-15 Thread Marcelo Tosatti
On Wed, Aug 15, 2012 at 12:26:05PM +0300, Avi Kivity wrote: > On 08/14/2012 01:04 AM, Marcelo Tosatti wrote: > > On Mon, Aug 13, 2012 at 01:34:11PM -0300, Marcelo Tosatti wrote: > >> On Sat, Aug 11, 2012 at 10:37:54AM +1000, Paul Mackerras wrote: > >> > On Fri, Aug 10, 2012 at 03:35:53PM -0300, Mar

Re: [PATCH v5 00/12] KVM: introduce readonly memslot

2012-08-15 Thread Marcelo Tosatti
On Wed, Aug 15, 2012 at 01:44:14PM +0300, Avi Kivity wrote: > On 08/14/2012 06:51 PM, Marcelo Tosatti wrote: > >> > >> Userspace may want to modify the ROM (for example, when programming a > >> flash device). It is also possible to map an hva range rw through one > >> slot and ro through another.

Re: [PATCH 19/38] KVM: PPC: Add cache flush on page map

2012-08-15 Thread Scott Wood
On 08/15/2012 12:27 PM, Alexander Graf wrote: > > On 15.08.2012, at 19:26, Scott Wood wrote: > >> On 08/15/2012 04:52 AM, Alexander Graf wrote: >>> >>> On 15.08.2012, at 03:23, Scott Wood wrote: >>> On 08/14/2012 06:04 PM, Alexander Graf wrote: > When we map a page that wasn't icache cle

Re: [PATCH v8 0/6] kvm: level irqfd support

2012-08-15 Thread Alex Williamson
On Wed, 2012-08-15 at 17:28 +0300, Michael S. Tsirkin wrote: > On Fri, Aug 10, 2012 at 04:37:08PM -0600, Alex Williamson wrote: > > v8: > > > > Trying a new approach. Nobody seems to like the internal IRQ > > source ID object and the interactions it implies between irqfd > > and eoifd, so let's g

Re: [PATCH 19/38] KVM: PPC: Add cache flush on page map

2012-08-15 Thread Alexander Graf
On 15.08.2012, at 19:26, Scott Wood wrote: > On 08/15/2012 04:52 AM, Alexander Graf wrote: >> >> On 15.08.2012, at 03:23, Scott Wood wrote: >> >>> On 08/14/2012 06:04 PM, Alexander Graf wrote: When we map a page that wasn't icache cleared before, do so when first mapping it in KVM usi

Re: [PATCH 19/38] KVM: PPC: Add cache flush on page map

2012-08-15 Thread Scott Wood
On 08/15/2012 04:52 AM, Alexander Graf wrote: > > On 15.08.2012, at 03:23, Scott Wood wrote: > >> On 08/14/2012 06:04 PM, Alexander Graf wrote: >>> When we map a page that wasn't icache cleared before, do so when first >>> mapping it in KVM using the same information bits as the Linux mapping >>>

Re: [PATCH v8 6/6] kvm: Add de-assert option to KVM_IRQ_ACKFD

2012-08-15 Thread Alex Williamson
On Wed, 2012-08-15 at 17:11 +0300, Michael S. Tsirkin wrote: > On Fri, Aug 10, 2012 at 04:37:56PM -0600, Alex Williamson wrote: > > It's likely (vfio) that one of the reasons to watch for an IRQ ACK > > is to de-assert and re-enable an interrupt. As the IRQ ACK notfier > > is already watching a GS

Re: [PATCH v8 5/6] kvm: KVM_IRQ_ACKFD

2012-08-15 Thread Alex Williamson
On Wed, 2012-08-15 at 17:05 +0300, Michael S. Tsirkin wrote: > On Fri, Aug 10, 2012 at 04:37:48PM -0600, Alex Williamson wrote: > > Enable a mechanism for IRQ ACKs to be exposed through an eventfd. The > > user can specify the GSI and optionally an IRQ source ID and have the > > provided eventfd t

Re: [PATCH v8 3/6] kvm: Add IRQ source ID option to KVM_IRQFD

2012-08-15 Thread Alex Williamson
On Wed, 2012-08-15 at 16:49 +0300, Michael S. Tsirkin wrote: > On Fri, Aug 10, 2012 at 04:37:33PM -0600, Alex Williamson wrote: > > This allows specifying an IRQ source ID to be used when injecting an > > interrupt. When not specified KVM_USERSPACE_IRQ_SOURCE_ID is used. > > > > Signed-off-by: Al

Re: [PATCH v8 2/6] kvm: Expose IRQ source IDs to userspace

2012-08-15 Thread Alex Williamson
On Wed, 2012-08-15 at 15:59 +0300, Michael S. Tsirkin wrote: > On Fri, Aug 10, 2012 at 04:37:25PM -0600, Alex Williamson wrote: > > Introduce KVM_IRQ_SOURCE_ID and KVM_CAP_NR_IRQ_SOURCE_ID to allow > > user allocation of IRQ source IDs and querying both the capability > > and the total count of IRQ

Re: [PATCH v8 1/6] kvm: Allow filtering of acked irqs

2012-08-15 Thread Alex Williamson
On Wed, 2012-08-15 at 15:27 +0300, Michael S. Tsirkin wrote: > On Fri, Aug 10, 2012 at 04:37:17PM -0600, Alex Williamson wrote: > > Registering an kvm_irq_ack_notifier with kian.irq_source_id < 0 > > retains existing behavior, filling in the actual irq_source_id results > > in the callback only bei

Re: [PATCH v2] KVM: x86 emulator: access GPRs on demand

2012-08-15 Thread Avi Kivity
On 08/15/2012 05:57 PM, Avi Kivity wrote: > Instead of populating the the entire register file, read in registers > as they are accessed, and write back only the modified ones. This > saves a VMREAD and VMWRITE on Intel (for rsp, since it is not usually > used during emulation), and a two 128-byte

[PATCH v2] KVM: x86 emulator: access GPRs on demand

2012-08-15 Thread Avi Kivity
Instead of populating the the entire register file, read in registers as they are accessed, and write back only the modified ones. This saves a VMREAD and VMWRITE on Intel (for rsp, since it is not usually used during emulation), and a two 128-byte copies for the registers. Signed-off-by: Avi Kiv

Re: [PATCH v8 0/6] kvm: level irqfd support

2012-08-15 Thread Michael S. Tsirkin
On Fri, Aug 10, 2012 at 04:37:08PM -0600, Alex Williamson wrote: > v8: > > Trying a new approach. Nobody seems to like the internal IRQ > source ID object and the interactions it implies between irqfd > and eoifd, so let's get rid of it. Instead, simply expose > IRQ source IDs to userspace. Thi

[PATCH] kvm: Fix kvmclock initialization on !CONFIG_KVM_GUEST

2012-08-15 Thread OGAWA Hirofumi
If !CONFIG_KVM_GUEST, kvm_smp_prepare_boot_cpu() is not defined. So, kvm_register_clock("primary cpu clock") in kvm_smp_prepare_boot_cpu() is not called. The detail of problem is hv_clock percpu usage. hv_clock is percpu variable, but kvmclock_init() is called _before_ initializing percpu area, a

Re: [PATCH v8 6/6] kvm: Add de-assert option to KVM_IRQ_ACKFD

2012-08-15 Thread Michael S. Tsirkin
On Fri, Aug 10, 2012 at 04:37:56PM -0600, Alex Williamson wrote: > It's likely (vfio) that one of the reasons to watch for an IRQ ACK > is to de-assert and re-enable an interrupt. As the IRQ ACK notfier > is already watching a GSI for an IRQ source ID we can easily couple > these together. > > Si

Re: [PATCH v8 5/6] kvm: KVM_IRQ_ACKFD

2012-08-15 Thread Michael S. Tsirkin
On Fri, Aug 10, 2012 at 04:37:48PM -0600, Alex Williamson wrote: > Enable a mechanism for IRQ ACKs to be exposed through an eventfd. The > user can specify the GSI and optionally an IRQ source ID and have the > provided eventfd trigger whenever the irqchip resamples it's inputs, > for instance on

[PATCH V2] KVM: x86: disable stealtime on reboot to avoid mem corruption

2012-08-15 Thread Florian Westphal
else, host continues to update stealtime after reboot, which can corrupt e.g. initramfs area. found when tracking down initramfs unpack error on initial reboot (with qemu-kvm -smp 2, no problem with single-core). Signed-off-by: Florian Westphal --- arch/x86/kernel/kvm.c |1 + 1 files changed

Re: [PATCH v8 3/6] kvm: Add IRQ source ID option to KVM_IRQFD

2012-08-15 Thread Michael S. Tsirkin
On Fri, Aug 10, 2012 at 04:37:33PM -0600, Alex Williamson wrote: > This allows specifying an IRQ source ID to be used when injecting an > interrupt. When not specified KVM_USERSPACE_IRQ_SOURCE_ID is used. > > Signed-off-by: Alex Williamson > --- > > Documentation/virtual/kvm/api.txt |5 +++

Re: [Qemu-devel] [PATCH v3 0/4] VFIO-based PCI device assignment for QEMU 1.2

2012-08-15 Thread Avi Kivity
On 08/14/2012 11:32 PM, Alex Williamson wrote: > v3: > - Incorporate feedback from Anthony, Avi, and Jan (Thank you!) > > I've also added a new tag to my github tree, so those who have > already reviewed can check the delta by comparing: > > git://github.com/awilliam/qemu-vfio.git > tags/v

Re: [PATCH] configure: Add --disable-kvm-options

2012-08-15 Thread Cole Robinson
On 08/14/2012 09:40 PM, Marcelo Tosatti wrote: > On Mon, Aug 13, 2012 at 06:49:49PM -0400, Cole Robinson wrote: >> In Fedora, our qemu package is based on qemu-kvm, and we go to convoluted >> lengths to provide a qemu-kvm binary with KVM on by default, but all >> qemu-system-* with KVM off by defau

Re: [PATCH v7 2/2] kvm: KVM_EOIFD, an eventfd for EOIs

2012-08-15 Thread Avi Kivity
On 08/15/2012 02:26 AM, Alex Williamson wrote: > > Yes, I understand. It's simple, it's also very specific to this > problem, and doesn't address generic ack notification. All of which > I've noted before and I continue to note that v8 offers simplifications > while retaining flexibility. Least

Re: [PATCH v8 2/6] kvm: Expose IRQ source IDs to userspace

2012-08-15 Thread Michael S. Tsirkin
On Fri, Aug 10, 2012 at 04:37:25PM -0600, Alex Williamson wrote: > Introduce KVM_IRQ_SOURCE_ID and KVM_CAP_NR_IRQ_SOURCE_ID to allow > user allocation of IRQ source IDs and querying both the capability > and the total count of IRQ source IDs. These will later be used > by interfaces for setting up

Re: [User question] Huge buffer size on KVM host

2012-08-15 Thread Avi Kivity
On 08/15/2012 03:05 PM, Martin Wawro wrote: > > On Aug 15, 2012, at 1:30 PM, Avi Kivity wrote: > >>> We have noticed a strange thing in one of our setups which uses KVM for >>> virtualization. >>> The size of the buffer cache turns out to be rather large >>> [,,,] >>> >> >> How is you storage s

Re: [PATCH v8 1/6] kvm: Allow filtering of acked irqs

2012-08-15 Thread Michael S. Tsirkin
On Fri, Aug 10, 2012 at 04:37:17PM -0600, Alex Williamson wrote: > Registering an kvm_irq_ack_notifier with kian.irq_source_id < 0 > retains existing behavior, filling in the actual irq_source_id results > in the callback only being called when the specified irq_source_id is > asserting the given g

Re: [User question] Huge buffer size on KVM host

2012-08-15 Thread Martin Wawro
On Aug 15, 2012, at 1:30 PM, Avi Kivity wrote: >> We have noticed a strange thing in one of our setups which uses KVM for >> virtualization. >> The size of the buffer cache turns out to be rather large >> [,,,] >> > > How is you storage set up? Files (which format?) or logical volumes? > what'

Re: [Qemu-devel] [PATCH v8] kvm: notify host when the guest is panicked

2012-08-15 Thread Yan Vugenfirer
On Aug 15, 2012, at 12:56 PM, Gleb Natapov wrote: > On Tue, Aug 14, 2012 at 02:35:34PM -0500, Anthony Liguori wrote: >>> Do you consider allowing support for Windows as overengineering? >> >> I don't think there is a way to hook BSOD on Windows so attempting to >> engineer something that works w

Re: [Qemu-devel] [PATCH v8] kvm: notify host when the guest is panicked

2012-08-15 Thread Yan Vugenfirer
On Aug 14, 2012, at 10:35 PM, Anthony Liguori wrote: > Marcelo Tosatti writes: > >> On Tue, Aug 14, 2012 at 01:53:01PM -0500, Anthony Liguori wrote: >>> Marcelo Tosatti writes: >>> On Tue, Aug 14, 2012 at 05:55:54PM +0300, Yan Vugenfirer wrote: > > On Aug 14, 2012, at 1:42 PM, J

Re: [User question] Huge buffer size on KVM host

2012-08-15 Thread Avi Kivity
On 08/15/2012 11:06 AM, Martin Wawro wrote: > Hi all, > > We have noticed a strange thing in one of our setups which uses KVM for > virtualization. > The size of the buffer cache turns out to be rather large, here is what > 'top' on the host > has to say about it: > > Mem: 12274084k total, 12202

[PATCH v2] kvm-all.c: Move init of irqchip_inject_ioctl out of kvm_irqchip_create()

2012-08-15 Thread Peter Maydell
Move the init of the irqchip_inject_ioctl field of KVMState out of kvm_irqchip_create() and into kvm_init(), so that kvm_set_irq() can be used even when no irqchip is created (for architectures that support async interrupt notification even without an in kernel irqchip). Signed-off-by: Peter Mayde

Re: [PATCH v5 00/12] KVM: introduce readonly memslot

2012-08-15 Thread Avi Kivity
On 08/14/2012 06:51 PM, Marcelo Tosatti wrote: >> >> Userspace may want to modify the ROM (for example, when programming a >> flash device). It is also possible to map an hva range rw through one >> slot and ro through another. > > Right, can do that with multiple userspace maps to the same anon

Re: [PATCH] kvm tools: Fix segfault on "lkvm run"

2012-08-15 Thread Pekka Enberg
On Tue, Aug 14, 2012 at 11:47 AM, Paul Neumann wrote: >> Looks good to me. Paul, can you tell how the segfault >> is triggered as >> well? Thanks. >> > The segfault is triggered by just running "lkvm run". On my system, it > does not find any kernel, so kvm_cmd_run_init() returns EINVAL which > f

Re: [PATCH 38/38] ppc: e500_tlb memset clears nothing

2012-08-15 Thread Avi Kivity
On 08/15/2012 01:09 PM, Alexander Graf wrote: > > On 15.08.2012, at 12:07, Avi Kivity wrote: > >> On 08/15/2012 02:04 AM, Alexander Graf wrote: >>> From: Alan Cox >>> >>> Put the parameters the right way around >>> >>> Addresses https://bugzilla.kernel.org/show_bug.cgi?id=44031 >>> >> >> >>

Re: [PATCH 38/38] ppc: e500_tlb memset clears nothing

2012-08-15 Thread Alexander Graf
On 15.08.2012, at 12:07, Avi Kivity wrote: > On 08/15/2012 02:04 AM, Alexander Graf wrote: >> From: Alan Cox >> >> Put the parameters the right way around >> >> Addresses https://bugzilla.kernel.org/show_bug.cgi?id=44031 >> > > > Should this go to 3.6 (and backports etc.)? This one is even

Re: [PATCH 38/38] ppc: e500_tlb memset clears nothing

2012-08-15 Thread Avi Kivity
On 08/15/2012 02:04 AM, Alexander Graf wrote: > From: Alan Cox > > Put the parameters the right way around > > Addresses https://bugzilla.kernel.org/show_bug.cgi?id=44031 > Should this go to 3.6 (and backports etc.)? -- error compiling committee.c: too many arguments to function -- To unsu

Re: [Qemu-devel] [PATCH v8] kvm: notify host when the guest is panicked

2012-08-15 Thread Gleb Natapov
On Tue, Aug 14, 2012 at 02:35:34PM -0500, Anthony Liguori wrote: > > Do you consider allowing support for Windows as overengineering? > > I don't think there is a way to hook BSOD on Windows so attempting to > engineer something that works with Windows seems odd, no? > Yan says in other email tha

[PATCH] KVM: PPC: E500: Remove E500_TLB_DIRTY flag

2012-08-15 Thread Alexander Graf
Since we always mark pages as dirty immediately when mapping them read/write now, there's no need for the dirty flag in our cache. Signed-off-by: Alexander Graf --- arch/powerpc/kvm/e500.h |3 +-- arch/powerpc/kvm/e500_tlb.c |4 +--- 2 files changed, 2 insertions(+), 5 deletions(-)

Re: [PATCH 19/38] KVM: PPC: Add cache flush on page map

2012-08-15 Thread Alexander Graf
On 15.08.2012, at 03:23, Scott Wood wrote: > On 08/14/2012 06:04 PM, Alexander Graf wrote: >> When we map a page that wasn't icache cleared before, do so when first >> mapping it in KVM using the same information bits as the Linux mapping >> logic. That way we are 100% sure that any page we map d

Re: [PATCH 18/38] KVM: PPC: E500: Implement MMU notifiers

2012-08-15 Thread Alexander Graf
On 15.08.2012, at 03:20, Scott Wood wrote: > On 08/14/2012 06:04 PM, Alexander Graf wrote: >> The e500 target has lived without mmu notifiers ever since it got >> introduced, but fails for the user space check on them with hugetlbfs. >> >> So in order to get that one working, implement mmu notif

Re: [PATCH 17/38] KVM: PPC: BookE: Add support for vcpu->mode

2012-08-15 Thread Alexander Graf
On 15.08.2012, at 03:17, Scott Wood wrote: > On 08/14/2012 07:26 PM, Alexander Graf wrote: >> >> On 15.08.2012, at 02:17, Scott Wood wrote: >> >>> On 08/14/2012 06:04 PM, Alexander Graf wrote: Generic KVM code might want to know whether we are inside guest context or outside. It also

Re: [PATCH 3/5] KVM: PPC: Book3S HV: Handle memory slot deletion and modification correctly

2012-08-15 Thread Avi Kivity
On 08/14/2012 01:04 AM, Marcelo Tosatti wrote: > On Mon, Aug 13, 2012 at 01:34:11PM -0300, Marcelo Tosatti wrote: >> On Sat, Aug 11, 2012 at 10:37:54AM +1000, Paul Mackerras wrote: >> > On Fri, Aug 10, 2012 at 03:35:53PM -0300, Marcelo Tosatti wrote: >> > > There's no plan. I just wanted to confirm

Re: [PATCH 3/5] KVM: PPC: Book3S HV: Handle memory slot deletion and modification correctly

2012-08-15 Thread Avi Kivity
On 08/13/2012 07:34 PM, Marcelo Tosatti wrote: > > Avi, Gleb, Alex, do you know why it is necessary to support change of > GPA base again? BAR moving around. > Without taking into consideration backwards compatibility, userspace > can first delete the slot and later create a new one. Current

Re: [PATCH 2/3] vfio: vfio-pci device assignment driver

2012-08-15 Thread Avi Kivity
On 08/14/2012 08:23 PM, Alex Williamson wrote: > >> Unrelated nit: memcmp() doesn't return a boolean or a count, so >> !memcmp() is really unintuitive, at least to me. > > I figure we're all pretty used to it growing up on !strcmp though. I hate that one too. >> > + >> > +/* XXX This should mov

Re: [RFC PATCH 5/5] KVM: PPC: Take the SRCU lock around memslot use

2012-08-15 Thread Benjamin Herrenschmidt
On Fri, 2012-08-10 at 11:27 +0200, Alexander Graf wrote: > > Perhaps... these changes are to areas of the PPC KVM code that I > don't > > use or maintain, so they're really more a suggestion of one way to > fix > > the problem than anything else. That's why I put RFC in the subject > > line. It w

[User question] Huge buffer size on KVM host

2012-08-15 Thread Martin Wawro
Hi all, We have noticed a strange thing in one of our setups which uses KVM for virtualization. The size of the buffer cache turns out to be rather large, here is what 'top' on the host has to say about it: Mem: 12274084k total, 12202860k used,71224k free, 2991728k buffers Swap: 8191992k t

Re: [PATCH 3/4] s390/kvm: Add a channel I/O based virtio transport driver.

2012-08-15 Thread Christian Borntraeger
>> Have you written an appendix for the virtio specification for >> virtio-ccw? I think it would be good to include in this series for the >> purposes of review. > > Might be nice, but don't get fancy about it. Text will be fine and I > can cut & paste it in once it's finalized. There was a pat

Re: [Qemu-devel] [PATCH 0/4] [PULL] qemu-kvm.git uq/master queue

2012-08-15 Thread Peter Maydell
On 15 August 2012 00:59, Marcelo Tosatti wrote: > The following changes since commit 873359d411eeb380906761e46839a2b705dbcf75: > > Merge branch 'linux-user.next' of > git://git.linaro.org/people/pmaydell/qemu-arm (2012-08-14 19:50:22 +) > > are available in the git repository at: > > git:

Re: [PATCH 3/4] s390/kvm: Add a channel I/O based virtio transport driver.

2012-08-15 Thread Rusty Russell
On Tue, 14 Aug 2012 14:56:07 -0500, Anthony Liguori wrote: > Cornelia Huck writes: > > > Add a driver for kvm guests that matches virtual ccw devices provided > > by the host as virtio bridge devices. > > > > These virtio-ccw devices use a special set of channel commands in order > > to perform