Re: KVM: MMU: Tracking guest writes through EPT entries ?

2012-09-03 Thread Xiao Guangrong
On 09/03/2012 10:09 AM, Hugo wrote: On Sun, Sep 2, 2012 at 8:29 AM, Xiao Guangrong xiaoguangr...@linux.vnet.ibm.com wrote: On 09/01/2012 05:30 AM, Hui Lin (Hugo) wrote: On Thu, Aug 30, 2012 at 9:54 PM, Xiao Guangrong xiaoguangr...@linux.vnet.ibm.com wrote: On 08/31/2012 02:59 AM, Hugo wrote:

Re: [PATCH] Add a page cache-backed balloon device driver.

2012-09-03 Thread Paolo Bonzini
Il 02/07/2012 02:29, Rusty Russell ha scritto: VIRTIO_BALLOON_F_MUST_TELL_HOST implies you should tell the host (eventually). I don't know if any implementations actually care though. This is indeed broken, because it is a negative feature: it tells you that implicit deflate is _not_

Re: virtIO disks not usable

2012-09-03 Thread Paolo Bonzini
Il 31/08/2012 15:19, Andrew Holway ha scritto: I have 4096 block devices from sda..sdx. Do you mean (here and in your other cache=none message) that the host devices have 4k logical blocks? If so, and if you use cache=none, you need to export them as 4k block devices to the guest as well. This

KVM call agenda for Tuesday, September 4th

2012-09-03 Thread Juan Quintela
Hi Please send in any agenda items you are interested in covering. Thanks, Juan. -- To unsubscribe from this list: send the line unsubscribe kvm in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html

Re: [PATCH v7 3/3] KVM: perf: kvm events analysis tool

2012-09-03 Thread don
于 2012年08月31日 02:29, David Ahern 写道: In addition to Andrew's comment about making the stats struct and functions generic... Yes. :-) On 8/27/12 3:51 AM, Dong Hao wrote: ---8--- +static void exit_event_decode_key(struct event_key *key, char decode[20]) +{ + const char *exit_reason =

Re: [PATCH] KVM: VMX: invalidate vpid for invlpg instruction

2012-09-03 Thread Avi Kivity
On 09/03/2012 02:27 AM, Davidlohr Bueso wrote: On Fri, 2012-08-31 at 14:37 -0300, Marcelo Tosatti wrote: On Fri, Aug 31, 2012 at 06:10:48PM +0200, Davidlohr Bueso wrote: For processors that support VPIDs we should invalidate the page table entry specified by the lineal address. For this

Re: NFSoRDMA not working with KVM when cache disabled

2012-09-03 Thread Stefan Hajnoczi
On Fri, Aug 31, 2012 at 6:05 PM, Andrew Holway a.hol...@syseleven.de wrote: I am trying to host KVM machines on an NFSoRDMA mount. This works: -drive file=/mnt/vm001.img,if=none,id=drive-virtio-disk0,format=raw -device

Re: Qcow2 preallocation + backing file

2012-09-03 Thread Stefan Hajnoczi
On Mon, Jul 30, 2012 at 9:10 PM, Mark Moseley moseleym...@gmail.com wrote: While I might've answered my own question, I'm no fio expert. I don't necessarily trust that I'm testing it correctly, which is why I'm asking here. Running the same test multiple times gives me 30-40meg/sec for both

Re: [RFC 0/5] Making KVM_GET_ONE_REG/KVM_SET_ONE_REG generic.

2012-09-03 Thread Avi Kivity
On 09/01/2012 03:35 PM, Rusty Russell wrote: Avi Kivity a...@redhat.com writes: -Capability: basic +Capability: KVM_CAP_REG_LIST Architectures: arm all OK, I guess that's to be true in future. Fixed. Type: vcpu ioctl -Parameters: struct kvm_msr_list (in/out) +Parameters: struct

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

2012-09-03 Thread Avi Kivity
On 08/23/2012 11:51 AM, Hao, Xudong wrote: -Original Message- From: kvm-ow...@vger.kernel.org [mailto:kvm-ow...@vger.kernel.org] On Behalf Of Avi Kivity Sent: Monday, August 20, 2012 6:43 PM To: Roedel, Joerg Cc: Hao, Xudong; kvm@vger.kernel.org; Zhang, Xiantao Subject: Re: [PATCH]

Re: Recommended way access KVM data structures from other kernel subsystems ?

2012-09-03 Thread Avi Kivity
On 08/23/2012 10:49 AM, Pandarathil, Vijaymohan R wrote: Hi, I am looking for the recommended approach for accessing KVM driver data from other kernel components. In my case, I need to set some global variable/state in KVM driver from one of the NMI handlers. I see that using kvm_x86_ops

Re: NFSoRDMA not working with KVM when cache disabled

2012-09-03 Thread Avi Kivity
On 08/31/2012 08:05 PM, Andrew Holway wrote: Hi, I am trying to host KVM machines on an NFSoRDMA mount. This works: -drive file=/mnt/vm001.img,if=none,id=drive-virtio-disk0,format=raw -device virtio-blk-pci,scsi=off,bus=pci.0,addr=0x5,drive=drive-virtio-disk0,id=virtio-disk0 This

Re: [PATCH v7 2/3] KVM: x86: trace mmio begin and complete

2012-09-03 Thread Avi Kivity
On 08/27/2012 12:51 PM, Dong Hao wrote: From: Xiao Guangrong xiaoguangr...@linux.vnet.ibm.com 'perf kvm stat record/report' will use kvm_exit and kvm_mmio(read...) to calculate mmio read emulated time for the old kernel, in order to trace mmio read event more exactly, we add kvm_mmio_begin

Re: [PATCH v7 1/3] KVM: x86: export svm/vmx exit code and vector code to userspace

2012-09-03 Thread Avi Kivity
On 08/27/2012 12:51 PM, Dong Hao wrote: From: Xiao Guangrong xiaoguangr...@linux.vnet.ibm.com Exporting KVM exit information to userspace to be consumed by perf. [ Dong Hao haod...@linux.vnet.ibm.com: rebase it on acme's git tree ] Signed-off-by: Xiao Guangrong

Re: KVM call agenda for Tuesday, September 4th

2012-09-03 Thread Juan Quintela
Juan Quintela quint...@redhat.com wrote: Hi Please send in any agenda items you are interested in covering. Thanks, Juan. Forwarding my own email. Just put a wrong address on the 1st try (kvm@ folks receive this twice). Thanks to Markus for noticing it. Later, Juan. PD. /me learn not to

[PATCH RESEND] KVM: cleanup pic reset

2012-09-03 Thread Gleb Natapov
kvm_pic_reset() is not used anywhere. Move reset logic from pic_ioport_write() there. Signed-off-by: Gleb Natapov g...@redhat.com diff --git a/arch/x86/kvm/i8259.c b/arch/x86/kvm/i8259.c index 90c84f9..848206d 100644 --- a/arch/x86/kvm/i8259.c +++ b/arch/x86/kvm/i8259.c @@ -275,23 +275,20 @@ void

Re: KVM call agenda for Tuesday, September 4th

2012-09-03 Thread Avi Kivity
On 09/03/2012 09:44 AM, Juan Quintela wrote: Hi Please send in any agenda items you are interested in covering. - protecting MemoryRegion::opaque during dispatch I'm guessing Ping won't make it due to timezone problems. Jan, if you will not participate, please remove the topic from the

[PATCHv2] virtio-spec: virtio network device multiqueue support

2012-09-03 Thread Michael S. Tsirkin
At Jason's request, I am trying to help finalize the spec for the new multiqueue feature. Changes from Jason's rfc: - reserved vq 3: this makes all rx vqs even and tx vqs odd, which looks nicer to me. - documented packet steering, added a generalized steering programming command. Current

Re: NFSoRDMA not working with KVM when cache disabled

2012-09-03 Thread Andrew Holway
and report which (if any) of the output files (x1, x2, y1, y2) are corrupted, by comparing them against the original. This will tell us whether O_DIRECT is broken, or 512 byte block size, or neither. Looks like you were directly on the money there. 512, 1K and 2K O_DIRECT looks broken.

Re: NFSoRDMA not working with KVM when cache disabled

2012-09-03 Thread Avi Kivity
On 09/03/2012 02:57 PM, Andrew Holway wrote: and report which (if any) of the output files (x1, x2, y1, y2) are corrupted, by comparing them against the original. This will tell us whether O_DIRECT is broken, or 512 byte block size, or neither. Looks like you were directly on the

[PATCH 2/4] KVM: emulator: make x86 emulation modes enum instead of defines

2012-09-03 Thread Gleb Natapov
Signed-off-by: Gleb Natapov g...@redhat.com --- arch/x86/include/asm/kvm_emulate.h | 22 ++ arch/x86/kvm/emulate.c |4 +++- 2 files changed, 13 insertions(+), 13 deletions(-) diff --git a/arch/x86/include/asm/kvm_emulate.h

[PATCH 3/4] KVM: emulator: string_addr_inc() cleanup

2012-09-03 Thread Gleb Natapov
Remove unneeded segment argument. Address structure already has correct segment which was put there during decode. Signed-off-by: Gleb Natapov g...@redhat.com --- arch/x86/kvm/emulate.c | 11 --- 1 file changed, 4 insertions(+), 7 deletions(-) diff --git a/arch/x86/kvm/emulate.c

[PATCH REBASE RESEND 0/4] improve speed of rep ins emulation

2012-09-03 Thread Gleb Natapov
This series (or rather the last patch of it) takes different approach to rep ins optimization. Instead of writing separate fast path for it do the fast path inside emulator itself. This way nobody can say the code is not reused! Patch 1,2 are now, strictly speaking, not needed, but I think this

[PATCH 4/4] KVM: emulator: optimize rep ins handling.

2012-09-03 Thread Gleb Natapov
Optimize rep ins by allowing emulator to write back more than one datum at a time. Introduce new operand type OP_MEM_STR which tells writeback() that dst contains pointer to an array that should be written back as opposite to just one data element. Signed-off-by: Gleb Natapov g...@redhat.com ---

[PATCH 1/4] Provide userspace IO exit completion callback.

2012-09-03 Thread Gleb Natapov
Current code assumes that IO exit was due to instruction emulation and handles execution back to emulator directly. This patch adds new userspace IO exit completion callback that can be set by any other code that caused IO exit to userspace. Signed-off-by: Gleb Natapov g...@redhat.com ---

Re: [RFC 0/5] Making KVM_GET_ONE_REG/KVM_SET_ONE_REG generic.

2012-09-03 Thread Rusty Russell
Avi Kivity a...@redhat.com writes: On 09/01/2012 03:35 PM, Rusty Russell wrote: Passing an address in a struct is pretty bad, since it involves compatibility wrappers. Right, some s390 thing. Err, no, i386 on x86-64, or ppc32 on ppc64, or arm on arm64 Any time you put a pointer in a

Re: [RFC 0/5] Making KVM_GET_ONE_REG/KVM_SET_ONE_REG generic.

2012-09-03 Thread Peter Maydell
On 3 September 2012 13:33, Rusty Russell ru...@rustcorp.com.au wrote: I have no idea, since I didn't hear the complaints. But any non-fixed size array has issues in C; there's not much we can do about it. x86 manages this fine for msrs, and I didn't have a problem using it for my test

[Bright Cluster Manager Support #2699] NFSoRDMA not working with KVM when cache disabled

2012-09-03 Thread Pawan Jaitly
It looks like this issue has been dealt with, so I'm closing its ticket. * If you would like to reopen the ticket, then please respond to this mail with the same subject header. * If you would like to open a new ticket, then please send a mail with a new subject header. In either case, the mail

Re: KVM on kernel 2.6.16

2012-09-03 Thread Avi Kivity
On 09/02/2012 07:08 PM, Lentes, Bernd wrote: Hi, i have several servers with SLES 10 SP4 and want to run kvm on them. SLES 10 has kernel 2.6.16. Is that possible ? Thanks for any answer. In general no. Please contact your server vendor though. -- error compiling committee.c: too

Re: KVM on kernel 2.6.16

2012-09-03 Thread Avi Kivity
On 09/03/2012 04:24 PM, Avi Kivity wrote: On 09/02/2012 07:08 PM, Lentes, Bernd wrote: Hi, i have several servers with SLES 10 SP4 and want to run kvm on them. SLES 10 has kernel 2.6.16. Is that possible ? Thanks for any answer. In general no. Please contact your server vendor

Re: [libvirt-users] vm pxe fail

2012-09-03 Thread Avi Kivity
On 08/31/2012 05:37 PM, Alex Jia wrote: Hi Andrew, Great, BTW, in fact, you may pxe boot via VF of Intel82576, however, Intel82576 SR-IOV network adapters don't provide a ROM BIOS for the cards virtual functions (VF), but an image of such a ROM is available, and with this ROM visible to

Re: KVM call agenda for Tuesday, September 4th

2012-09-03 Thread Jan Kiszka
On 2012-09-03 13:48, Avi Kivity wrote: On 09/03/2012 09:44 AM, Juan Quintela wrote: Hi Please send in any agenda items you are interested in covering. - protecting MemoryRegion::opaque during dispatch I'm guessing Ping won't make it due to timezone problems. Jan, if you will not

Re: KVM call agenda for Tuesday, September 4th

2012-09-03 Thread Avi Kivity
On 09/03/2012 04:35 PM, Jan Kiszka wrote: On 2012-09-03 13:48, Avi Kivity wrote: On 09/03/2012 09:44 AM, Juan Quintela wrote: Hi Please send in any agenda items you are interested in covering. - protecting MemoryRegion::opaque during dispatch I'm guessing Ping won't make it due to

Re: [PATCH v4 0/8] KVM paravirt remote flush tlb

2012-09-03 Thread Avi Kivity
On 08/21/2012 02:25 PM, Nikunj A. Dadhania wrote: kernbench(lower is better) == base pvflushv4 %improvement 1VM48.5800 46.8513 3.55846 2VM 108.1823 104.6410 3.27346 3VM 183.2733 163.3547 10.86825

Re: [PATCH] Add a page cache-backed balloon device driver.

2012-09-03 Thread Avi Kivity
On 06/26/2012 11:32 PM, Frank Swiderski wrote: This implementation of a virtio balloon driver uses the page cache to store pages that have been released to the host. The communication (outside of target counts) is one way--the guest notifies the host when it adds a page to the page cache,

Re: [PATCH v4 0/8] KVM paravirt remote flush tlb

2012-09-03 Thread Avi Kivity
On 09/03/2012 05:33 PM, Avi Kivity wrote: On 08/21/2012 02:25 PM, Nikunj A. Dadhania wrote: kernbench(lower is better) == base pvflushv4 %improvement 1VM48.5800 46.8513 3.55846 2VM 108.1823 104.6410 3.27346 3VM

Re: [Qemu-devel] [PATCH 4/4] kvm: i386: Add classic PCI device assignment

2012-09-03 Thread Avi Kivity
On 08/29/2012 11:49 AM, Peter Maydell wrote: On 29 August 2012 09:47, Jan Kiszka jan.kis...@siemens.com wrote: On 2012-08-28 23:26, Peter Maydell wrote: Since this is arch-specific we should probably give the resulting device a more specific name than pci-assign, which implies that it is (a)

Re: [PATCH v7 3/3] KVM: perf: kvm events analysis tool

2012-09-03 Thread David Ahern
On 9/3/12 2:48 AM, don wrote: 于 2012年08月31日 02:29, David Ahern 写道: In addition to Andrew's comment about making the stats struct and functions generic... Yes. :-) On 8/27/12 3:51 AM, Dong Hao wrote: ---8--- +static void exit_event_decode_key(struct event_key *key, char decode[20]) +{ +

Re: [Qemu-devel] [PATCH 4/4] kvm: i386: Add classic PCI device assignment

2012-09-03 Thread Avi Kivity
On 08/28/2012 03:30 AM, Jan Kiszka wrote: Maybe add case 8: and default: with abort(), also below. PIO is never 8 bytes long, the generic layer protects us. Note: eventually the pio space will be mapped directly to mmio (instead of being bounced via cpu_inb() in the bridge's mmio handler),

Re: [Qemu-devel] [PATCH 4/4] kvm: i386: Add classic PCI device assignment

2012-09-03 Thread Avi Kivity
On 08/29/2012 11:27 AM, Markus Armbruster wrote: I don't see a point in making contributors avoid non-problems that might conceivably become trivial problems some day. Especially when there's no automated help with the avoiding. -Wpointer-arith -- error compiling committee.c: too many

Sending a VM for restart (after it has been shutdown and saved)

2012-09-03 Thread Michael Johns
Hi list, I have been hacking the KVM-QEMU code, but need some help to be able to perform a particular operation. Currently, I perform some operations on the VM image after it has received a shutdown call, and after the image itself has been fully saved before KVM-QEMU shuts down. I perform these

Re: [Qemu-devel] [PATCH 4/4] kvm: i386: Add classic PCI device assignment

2012-09-03 Thread Blue Swirl
On Mon, Sep 3, 2012 at 4:14 PM, Avi Kivity a...@redhat.com wrote: On 08/29/2012 11:27 AM, Markus Armbruster wrote: I don't see a point in making contributors avoid non-problems that might conceivably become trivial problems some day. Especially when there's no automated help with the

[PATCH] kvm: Fix warning from static code analysis

2012-09-03 Thread Stefan Weil
Report from smatch: kvm-all.c:1373 kvm_init(135) warn: variable dereferenced before check 's' (see line 1360) 's' cannot by NULL (it was alloced using g_malloc0), so there is no need to check it here. Signed-off-by: Stefan Weil s...@weilnetz.de --- kvm-all.c | 12 +--- 1 file

Re: [Qemu-devel] [PATCH] kvm: Fix warning from static code analysis

2012-09-03 Thread Peter Maydell
On 3 September 2012 21:40, Stefan Weil s...@weilnetz.de wrote: Report from smatch: kvm-all.c:1373 kvm_init(135) warn: variable dereferenced before check 's' (see line 1360) 's' cannot by NULL (it was alloced using g_malloc0), so there is no need to check it here. Signed-off-by: Stefan

Re: [PATCH v4 0/8] KVM paravirt remote flush tlb

2012-09-03 Thread Nikunj A Dadhania
On Mon, 03 Sep 2012 17:33:46 +0300, Avi Kivity a...@redhat.com wrote: On 08/21/2012 02:25 PM, Nikunj A. Dadhania wrote: kernbench(lower is better) == base pvflushv4 %improvement 1VM48.5800 46.8513 3.55846 2VM 108.1823

Re: [PATCH 5/5] virtio-scsi: introduce multiqueue support

2012-09-03 Thread Nicholas A. Bellinger
On Tue, 2012-08-28 at 13:54 +0200, Paolo Bonzini wrote: This patch adds queue steering to virtio-scsi. When a target is sent multiple requests, we always drive them to the same queue so that FIFO processing order is kept. However, if a target was idle, we can choose a queue arbitrarily. In

Re: [Qemu-devel] [PATCH 4/4] kvm: i386: Add classic PCI device assignment

2012-09-03 Thread Alex Williamson
On Mon, 2012-09-03 at 18:59 +0300, Avi Kivity wrote: On 08/29/2012 11:49 AM, Peter Maydell wrote: On 29 August 2012 09:47, Jan Kiszka jan.kis...@siemens.com wrote: On 2012-08-28 23:26, Peter Maydell wrote: Since this is arch-specific we should probably give the resulting device a more

Re: [PATCH v7 1/3] KVM: x86: export svm/vmx exit code and vector code to userspace

2012-09-03 Thread Xiao Guangrong
On 09/03/2012 07:13 PM, Avi Kivity wrote: On 08/27/2012 12:51 PM, Dong Hao wrote: From: Xiao Guangrong xiaoguangr...@linux.vnet.ibm.com Exporting KVM exit information to userspace to be consumed by perf. [ Dong Hao haod...@linux.vnet.ibm.com: rebase it on acme's git tree ] Signed-off-by:

Re: [PATCH v7 2/3] KVM: x86: trace mmio begin and complete

2012-09-03 Thread Xiao Guangrong
On 09/03/2012 07:07 PM, Avi Kivity wrote: On 08/27/2012 12:51 PM, Dong Hao wrote: From: Xiao Guangrong xiaoguangr...@linux.vnet.ibm.com 'perf kvm stat record/report' will use kvm_exit and kvm_mmio(read...) to calculate mmio read emulated time for the old kernel, in order to trace mmio read