Re: [kvm-devel] [PATCH] mmu notifier #v11

2008-04-07 Thread Andrea Arcangeli
On Sun, Apr 06, 2008 at 10:45:41PM -0700, Christoph Lameter wrote: That depends on what the notifier is being used for. Some serialization with the external mappings has to be done anyways. And its cleaner to have As far as I can tell no, you don't need to serialize against the secondary mmu

Re: [kvm-devel] [patch 02/10] emm: notifier logic

2008-04-07 Thread Andrea Arcangeli
On Sun, Apr 06, 2008 at 10:48:56PM -0700, Christoph Lameter wrote: On Sat, 5 Apr 2008, Andrea Arcangeli wrote: + rcu_assign_pointer(mm-emm_notifier, e); + mm_unlock(mm); My mm_lock solution makes all rcu serialization an unnecessary overhead so you should remove it like I already

Re: [kvm-devel] [patch 02/10] emm: notifier logic

2008-04-07 Thread Christoph Lameter
On Mon, 7 Apr 2008, Andrea Arcangeli wrote: My mm_lock solution makes all rcu serialization an unnecessary overhead so you should remove it like I already did in #v11. If it wasn't the case, then mm_lock wouldn't be a definitive fix for the race. There still could be junk in the

Re: [kvm-devel] [ANN] Bochs bios ported to gcc

2008-04-07 Thread H. Peter Anvin
Avi Kivity wrote: Kevin O'Connor wrote: I have been working on a port of bochs bios to gcc. This port is nearly complete. The new code does not rely on bcc or dev86. Instead, it uses standard gcc and gas. It should compile on any recent Linux distribution. I'm sending this email because

Re: [kvm-devel] [patch 02/10] emm: notifier logic

2008-04-07 Thread Andrea Arcangeli
On Sun, Apr 06, 2008 at 11:20:08PM -0700, Christoph Lameter wrote: On Mon, 7 Apr 2008, Andrea Arcangeli wrote: My mm_lock solution makes all rcu serialization an unnecessary overhead so you should remove it like I already did in #v11. If it wasn't the case, then mm_lock wouldn't be

Re: [kvm-devel] [Qemu-devel] [PATCH 2/6] PCI DMA API (v2)

2008-04-07 Thread andrzej zaborowski
On 07/04/2008, Paul Brook [EMAIL PROTECTED] wrote: As a note, the DMA controllers in the ARM system-on-chip's can byte-swap, do 90deg rotation of 2D arrays, transparency (probably intened for image blitting, but still available on any kind of transfers), etc., and most importantly issue

Re: [kvm-devel] [PATCH] qemu: avoid returning uninitialized value in apic_mem_readl

2008-04-07 Thread Aurelien Jarno
Carlo Marcelo Arenas Belon a écrit : complement 64173d009c1f4d163c425b14aa650df5b982428a to avoid : kvm-65/qemu/hw/apic.c: In function `apic_mem_readl': kvm-65/qemu/hw/apic.c:592: warning: 'val' might be used uninitialized in this function Signed-off-by: Carlo Marcelo Arenas Belon

[kvm-devel] [ kvm-Bugs-1936539 ] Failed to save/restore 32e guests

2008-04-07 Thread SourceForge.net
Bugs item #1936539, was opened at 2008-04-07 16:56 Message generated for change (Tracker Item Submitted) made by Item Submitter You can respond by visiting: https://sourceforge.net/tracker/?func=detailatid=893831aid=1936539group_id=180599 Please note that this message will contain a full copy of

[kvm-devel] KVM Test result, kernel b1afe0f.., userspace 5208ce1.. -- One new issue

2008-04-07 Thread Yunfeng Zhao
Hi All, This is today's KVM test result against kvm.git b1afe0f37aeaa8af6240bf2bf444de1809fe06cc and kvm-userspace.git 5208ce19dca268f84a2b9441c2fbb6129161e44c. Issue list: One new issue: 1. Fails to save/restore ia32e guests

[kvm-devel] paravirt clock stil causing hangs in kvm-65

2008-04-07 Thread Nikola Ciprich
Hi, I also tried paravirt clock again in latest git with kvm-65 patch applied, and problem with cpu-lockups persists: [10813.654806] BUG: soft lockup - CPU#0 stuck for 61s! [swapper:0] [10813.655789] CPU 0: [10813.656624] Modules linked in: virtio_pci virtio_ring virtio_blk virtio piix

Re: [kvm-devel] [ANN] Bochs bios ported to gcc

2008-04-07 Thread Kevin O'Connor
On Sun, Apr 06, 2008 at 11:42:11PM -0700, H. Peter Anvin wrote: bcc is truly horrible. gcc is, of course, best, although it does produce bloated 16-bit code. With gcc, the text size has actually been significantly reduced - I'm not sure if this is because I'm using -fwhole-program and -Os or

[kvm-devel] [PATCH] gfxboot VMX workaround v2

2008-04-07 Thread Alexander Graf
Hi, this is an improved version of the patch I sent several weeks ago to this list. Functionally nothing changed; it still hacks into gfxboot and patches it to work on Intel CPUs on the fly. The big difference is that this version is cleaned up and should work with every future CPU available.

Re: [kvm-devel] performance tuning, problem with paravirtualized clock

2008-04-07 Thread Marcelo Tosatti
On Sun, Apr 06, 2008 at 09:56:39PM +0200, Nikola Ciprich wrote: Hi, I spent some time trying to tune performance of KVM guest using kernel compilation as a kind of benchmark (I'm using virtual machines for compiling a lot, so it's good benchmark for me in general) Host machine: 2x quad

Re: [kvm-devel] [Qemu-devel] [PATCH 2/6] PCI DMA API (v2)

2008-04-07 Thread Paul Brook
+/* Return a new IOVector that's a subset of the passed in IOVector. It should + * be freed with qemu_free when you are done with it. */ +IOVector *iovector_trim(const IOVector *iov, size_t offset, size_t size); Using qemu_free directly seems a bad idea. I guess we're likely to want to

Re: [kvm-devel] [PATCH] gfxboot VMX workaround v2

2008-04-07 Thread Anthony Liguori
Alexander Graf wrote: Hi, this is an improved version of the patch I sent several weeks ago to this list. Functionally nothing changed; it still hacks into gfxboot and patches it to work on Intel CPUs on the fly. The big difference is that this version is cleaned up and should work with

Re: [kvm-devel] [ANN] Bochs bios ported to gcc

2008-04-07 Thread H. Peter Anvin
Kevin O'Connor wrote: On Sun, Apr 06, 2008 at 11:42:11PM -0700, H. Peter Anvin wrote: bcc is truly horrible. gcc is, of course, best, although it does produce bloated 16-bit code. With gcc, the text size has actually been significantly reduced - I'm not sure if this is because I'm using

Re: [kvm-devel] [PATCH] gfxboot VMX workaround v2

2008-04-07 Thread Alexander Graf
On Apr 7, 2008, at 6:05 PM, Anthony Liguori wrote: Alexander Graf wrote: Hi, this is an improved version of the patch I sent several weeks ago to this list. Functionally nothing changed; it still hacks into gfxboot and patches it to work on Intel CPUs on the fly. The big difference is

Re: [kvm-devel] [PATCH] gfxboot VMX workaround v2

2008-04-07 Thread Anthony Liguori
Alexander Graf wrote: On Apr 7, 2008, at 6:05 PM, Anthony Liguori wrote: Alexander Graf wrote: Hi, this is an improved version of the patch I sent several weeks ago to this list. Functionally nothing changed; it still hacks into gfxboot and patches it to work on Intel CPUs on the fly. The

Re: [kvm-devel] [PATCH] gfxboot VMX workaround v2

2008-04-07 Thread Alexander Graf
On Apr 7, 2008, at 6:51 PM, Anthony Liguori wrote: Alexander Graf wrote: On Apr 7, 2008, at 6:05 PM, Anthony Liguori wrote: Alexander Graf wrote: Hi, this is an improved version of the patch I sent several weeks ago to this list. Functionally nothing changed; it still hacks into

Re: [kvm-devel] [PATCH] gfxboot VMX workaround v2

2008-04-07 Thread Anthony Liguori
Alexander Graf wrote: On Apr 7, 2008, at 6:51 PM, Anthony Liguori wrote: Alexander Graf wrote: On Apr 7, 2008, at 6:05 PM, Anthony Liguori wrote: Alexander Graf wrote: Hi, this is an improved version of the patch I sent several weeks ago to this list. Functionally nothing changed; it

Re: [kvm-devel] [patch 01/10] emm: mm_lock: Lock a process against reclaim

2008-04-07 Thread Jeremy Fitzhardinge
Andrea Arcangeli wrote: On Fri, Apr 04, 2008 at 04:12:42PM -0700, Jeremy Fitzhardinge wrote: I think you can break this if() down a bit: if (!(vma-vm_file vma-vm_file-f_mapping)) continue; It makes no difference at runtime, coding

Re: [kvm-devel] [patch 01/10] emm: mm_lock: Lock a process against reclaim

2008-04-07 Thread Andrea Arcangeli
On Mon, Apr 07, 2008 at 12:02:53PM -0700, Jeremy Fitzhardinge wrote: It's per-mm though. How many processes would need to have notifiers? There can be up to hundreds of VM in a single system. Not sure to understand the point of the question though. Well, its definitely going to need more

Re: [kvm-devel] [patch 3/4] QEMU/KVM: ignore SIG_IPI signals in userspace

2008-04-07 Thread Marcelo Tosatti
On Sun, Apr 06, 2008 at 12:15:07PM +0300, Avi Kivity wrote: Marcelo Tosatti wrote: Otherwise a signal can be received in userspace and a vcpu goes back to the kernel while it should stay still. Signed-off-by: Marcelo Tosatti [EMAIL PROTECTED] Index: kvm-userspace.io/qemu/qemu-kvm.c

[kvm-devel] [PATCH] add ioctl's to save/restore mpstate

2008-04-07 Thread Marcelo Tosatti
So userspace can save/restore the mpstate during migration. Signed-off-by: Marcelo Tosatti [EMAIL PROTECTED] diff --git a/arch/x86/kvm/x86.c b/arch/x86/kvm/x86.c index cb57b6a..2ce22d6 100644 --- a/arch/x86/kvm/x86.c +++ b/arch/x86/kvm/x86.c @@ -814,6 +814,7 @@ int

[kvm-devel] [patch 0/4] fix SMP migration and loadvm/savevm (V2)

2008-04-07 Thread Marcelo Tosatti
Now guarded by KVM_CAP and with increased version_id of cpu_save/load as requested. Avi, I prefer not to fold mpstate into kvm_save_registers() as a hidden register because the MPSTATE is only used during migration, whereas save_registers() is not (seems safer). --

[kvm-devel] [patch 1/4] QEMU/KVM: properly copy the in-kernel apicbase value

2008-04-07 Thread Marcelo Tosatti
The MSR_IA32_APICBASE_ENABLE/MSR_IA32_APICBASE_BSP bits in s-apicbase are not initialized if in-kernel APIC emulation is used, so save the actual value passed by cpu_set_apic_base() caller. Signed-off-by: Marcelo Tosatti [EMAIL PROTECTED] Index: marcelo/git/kvm-userspace.io/qemu/hw/apic.c

[kvm-devel] [patch 2/4] QEMU/KVM: get/set mpstate libkvm interface

2008-04-07 Thread Marcelo Tosatti
Add libkvm interface to get/set the mpstate. Signed-off-by: Marcelo Tosatti [EMAIL PROTECTED] Index: marcelo/git/kvm-userspace.io/libkvm/libkvm.c === --- marcelo.orig/git/kvm-userspace.io/libkvm/libkvm.c +++

[kvm-devel] [patch 4/4] QEMU/KVM: ignore SIG_IPI signals in userspace

2008-04-07 Thread Marcelo Tosatti
Otherwise a signal can be received in userspace and a vcpu goes back to the kernel while it should stay still. Signed-off-by: Marcelo Tosatti [EMAIL PROTECTED] Index: marcelo/git/kvm-userspace.io/qemu/qemu-kvm.c === ---

[kvm-devel] [patch 3/4] QEMU/KVM: save and load mp state

2008-04-07 Thread Marcelo Tosatti
Use the new interface to save and restore MP_STATE for all vcpu's. Increase version_id for cpu_load/cpu_save. Fixes SMP migration. Signed-off-by: Marcelo Tosatti [EMAIL PROTECTED] Index: marcelo/git/kvm-userspace.io/qemu/qemu-kvm-x86.c

Re: [kvm-devel] performance tuning, problem with paravirtualized clock

2008-04-07 Thread Nikola Ciprich
Hi Marcelo! Host machine: 2x quad core XEON E5420 @ 2.50GHz, 4GB RAM, 2.6.24 + kvm-64 guest configuration: all 8 cores available, 2GB RAM, 2.6.24 or latest GIT + kvm-64 Can you please provide more details on this? Which kernel version are you running on the host? As I wrote, I use 2.6.24

Re: [kvm-devel] paravirt clock stil causing hangs in kvm-65

2008-04-07 Thread Marcelo Tosatti
On Mon, Apr 07, 2008 at 01:53:36PM +0200, Nikola Ciprich wrote: Hi, I also tried paravirt clock again in latest git with kvm-65 patch applied, and problem with cpu-lockups persists: [10813.654806] BUG: soft lockup - CPU#0 stuck for 61s! [swapper:0] [10813.655789] CPU 0: [10813.656624]

[kvm-devel] Compilation problems with git tree

2008-04-07 Thread Zdenek Kabelac
Hi I've tried to compile git tree for kvm-userspace.git I've used these configure options: --disable-gcc-check --with-patched-kernel using x86-64 platform I've got this error: ar rcs libqemu.a exec.o kqemu.o cpu-exec.o host-utils.o translate-all.o translate.o op.o tcg/tcg.o tcg/tcg-dyngen.o

[kvm-devel] Richiedi l'attivazione di UniCredit Pass

2008-04-07 Thread Banca Di Roma
Gentile CLIENTE, I. I Servizi Online (Internet e telefono) ti consentono di effettuare bonifici, comprare titoli, pagare le bollette e tanto altro ancora, tutto in totale autonomia e comodit. II. UniCredit Banca di Roma ti offre inoltre un adeguato sistema di protezione: la Tessera di

Re: [kvm-devel] [PATCH] gfxboot VMX workaround v2

2008-04-07 Thread Avi Kivity
Anthony Liguori wrote: Yes, but it won't compile as KVM does not have a proper load_seg() function like the Xen's x86_emulate. Izik added something for task switching (see load_segment_descriptor). -- Any sufficiently difficult bug is indistinguishable from a feature.

[kvm-devel] Call for Presentations: KVM Forum 2008

2008-04-07 Thread Avi Kivity
[Note: KVM Forum registration is now open at http://kforum.qumranet.com/KVMForum/about_kvmforum.php] This is the Call for Presentations for the second annual KVM Developer's Forum, to be held on June 10-13, 2008, in Napa, California, USA [1]. We are looking for presentations on KVM development,

Re: [kvm-devel] [patch] remove dead code in extboot.S

2008-04-07 Thread Jun Koi
On 4/7/08, Anthony Liguori [EMAIL PROTECTED] wrote: Jun Koi wrote: Hi, This patch removes dead code in extboot.S Please don't remove the callout macro. I removed it in the patch because even if we keep it, it is unuseable: it relies on a non-registered IO port (0x407). So to use

Re: [kvm-devel] [PATCH 2 of 3] [KVM] Add DCR access information to struct kvm_run

2008-04-07 Thread David Gibson
On Mon, Apr 07, 2008 at 03:53:33PM -0500, Hollis Blanchard wrote: 1 file changed, 7 insertions(+) include/linux/kvm.h |7 +++ Device Control Registers are essentially another address space found on PowerPC 4xx processors, analogous to PIO on x86. DCRs are always 32 bits, and are

Re: [kvm-devel] paravirt clock stil causing hangs in kvm-65

2008-04-07 Thread Marcelo Tosatti
On Mon, Apr 07, 2008 at 06:34:57PM -0300, Marcelo Tosatti wrote: On Mon, Apr 07, 2008 at 01:53:36PM +0200, Nikola Ciprich wrote: Hi, I also tried paravirt clock again in latest git with kvm-65 patch applied, and problem with cpu-lockups persists: [10813.654806] BUG: soft lockup -

Re: [kvm-devel] [patch 3/4] QEMU/KVM: ignore SIG_IPI signals in userspace

2008-04-07 Thread Avi Kivity
Marcelo Tosatti wrote: On Sun, Apr 06, 2008 at 12:15:07PM +0300, Avi Kivity wrote: Marcelo Tosatti wrote: Otherwise a signal can be received in userspace and a vcpu goes back to the kernel while it should stay still. Signed-off-by: Marcelo Tosatti [EMAIL PROTECTED] Index:

Re: [kvm-devel] [PATCH 3 of 3] [KVM POWERPC] PowerPC 440 KVM implementation

2008-04-07 Thread Josh Boyer
On Mon, 07 Apr 2008 15:53:34 -0500 Hollis Blanchard [EMAIL PROTECTED] wrote: Currently supports only PowerPC 440 Linux guests on 440 hosts. (Only tested with 440EP Bamboo guests so far, but with appropriate userspace support other SoC/board combinations should work.) I haven't reviewed the

[kvm-devel] Losing weight is easy now,

2008-04-07 Thread KULESA
I took some pictures of my wife last night. I gave her CIALIS and she went wild. I can show you the pictures later, but you gotta see http://www.ryebopbro.com/ - This SF.net email is sponsored by the 2008 JavaOne(SM)

Re: [kvm-devel] [PATCH 3 of 3] [KVM POWERPC] PowerPC 440 KVM implementation

2008-04-07 Thread Arnd Bergmann
On Monday 07 April 2008, Hollis Blanchard wrote: --- a/include/asm-powerpc/kvm.h +++ b/include/asm-powerpc/kvm.h @@ -1,6 +1,55 @@ +/* + * This program is free software; you can redistribute it and/or modify + * it under the terms of the GNU General Public License, version 2, as + *

Re: [kvm-devel] [PATCH 2 of 3] [KVM] Add DCR access information to struct kvm_run

2008-04-07 Thread Hollis Blanchard
On Monday 07 April 2008 20:11:28 David Gibson wrote: On Mon, Apr 07, 2008 at 03:53:33PM -0500, Hollis Blanchard wrote: 1 file changed, 7 insertions(+) include/linux/kvm.h |7 +++ Device Control Registers are essentially another address space found on PowerPC 4xx processors,

Re: [kvm-devel] [PATCH RFC 1/5]Add some trace enties and define interface for tracing

2008-04-07 Thread Liu, Eric E
Avi Kivity wrote: Liu, Eric E wrote: From d56731ffc6d5742a88a157dfe0e4344d35f7db58 Mon Sep 17 00:00:00 2001 From: Feng(Eric) Liu [EMAIL PROTECTED] Date: Mon, 31 Mar 2008 10:08:55 -0400 Subject: [PATCH] KVM: Add some trace entries in current code and define some interfaces for userspace app

Re: [kvm-devel] [PATCH RFC 2/5] Create relay channels and add trace data

2008-04-07 Thread Liu, Eric E
Avi Kivity wrote: Liu, Eric E wrote: From 41d65b55580d3f07f9f1c50e89e3d64c5d10fbaf Mon Sep 17 00:00:00 2001 From: Feng (Eric) Liu [EMAIL PROTECTED] Date: Tue, 1 Apr 2008 07:26:14 -0400 Subject: [PATCH] KVM: Add kvm trace support. when config KVM_TRACE, it allows userspace app to read the

Re: [kvm-devel] [PATCH 2 of 3] [KVM] Add DCR access information to struct kvm_run

2008-04-07 Thread David Gibson
On Mon, Apr 07, 2008 at 10:25:32PM -0500, Hollis Blanchard wrote: On Monday 07 April 2008 20:11:28 David Gibson wrote: On Mon, Apr 07, 2008 at 03:53:33PM -0500, Hollis Blanchard wrote: 1 file changed, 7 insertions(+) include/linux/kvm.h |7 +++ Device Control Registers are

Re: [kvm-devel] [PATCH 3 of 3] [KVM POWERPC] PowerPC 440 KVM implementation

2008-04-07 Thread Hollis Blanchard
On Monday 07 April 2008 21:12:40 Josh Boyer wrote: On Mon, 07 Apr 2008 15:53:34 -0500 Hollis Blanchard [EMAIL PROTECTED] wrote: Currently supports only PowerPC 440 Linux guests on 440 hosts. (Only tested with 440EP Bamboo guests so far, but with appropriate userspace support other

Re: [kvm-devel] [PATCH 2 of 3] [KVM] Add DCR access information to struct kvm_run

2008-04-07 Thread Hollis Blanchard
On Monday 07 April 2008 22:54:41 David Gibson wrote: On Mon, Apr 07, 2008 at 10:25:32PM -0500, Hollis Blanchard wrote: On Monday 07 April 2008 20:11:28 David Gibson wrote: On Mon, Apr 07, 2008 at 03:53:33PM -0500, Hollis Blanchard wrote: 1 file changed, 7 insertions(+)

Re: [kvm-devel] [PATCH 3 of 3] [KVM POWERPC] PowerPC 440 KVM implementation

2008-04-07 Thread Hollis Blanchard
On Monday 07 April 2008 21:58:17 Arnd Bergmann wrote: On Monday 07 April 2008, Hollis Blanchard wrote: --- a/include/asm-powerpc/kvm.h +++ b/include/asm-powerpc/kvm.h @@ -1,6 +1,55 @@ +/* + * This program is free software; you can redistribute it and/or modify + * it under the terms

Re: [kvm-devel] [PATCH 2 of 3] [KVM] Add DCR access information to struct kvm_run

2008-04-07 Thread David Gibson
On Mon, Apr 07, 2008 at 11:06:10PM -0500, Hollis Blanchard wrote: On Monday 07 April 2008 22:54:41 David Gibson wrote: On Mon, Apr 07, 2008 at 10:25:32PM -0500, Hollis Blanchard wrote: On Monday 07 April 2008 20:11:28 David Gibson wrote: On Mon, Apr 07, 2008 at 03:53:33PM -0500, Hollis