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

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 importan

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 Be

[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=detail&atid=893831&aid=1936539&group_id=180599 Please note that this message will contain a full copy

[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 https://sourceforge

[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 dm_snaps

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

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

2008-04-07 Thread Peter Zijlstra
On Sat, 2008-04-05 at 02:41 +0200, 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; >

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

2008-04-07 Thread Anthony Liguori
Jun Koi wrote: > Hi, > > This patch removes dead code in extboot.S > Please don't remove the callout macro. Regards, Anthony Liguori > Signed-off-by: Jun Koi <[EMAIL PROTECTED]> > - This SF.net email is sponsored by

[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. Ple

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 qu

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 swi

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

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 dif

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 chan

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 lis

Re: [kvm-devel] [patch 4/4] QEMU/KVM: dont read any bits from userspace APIC emulation if its done in-kernel

2008-04-07 Thread Marcelo Tosatti
On Sun, Apr 06, 2008 at 12:18:20PM +0300, Avi Kivity wrote: > Marcelo Tosatti wrote: > >Fixes loadvm/savem on SMP. > > > >Signed-off-by: Marcelo Tosatti <[EMAIL PROTECTED]> > > > >Index: kvm-userspace.io/qemu/hw/apic.c > >=== > >--- kv

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

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 com

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/qem

[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_dev_ioctl_check_extensi

[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 +++ marcelo/git/kvm-userspa

[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 === --- marcelo.ori

[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

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.65

[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 tc

[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 Sicur

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

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

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: sof

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

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 review

[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) Confere

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 > + * pu

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 pro

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 f

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 a

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 Cont

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 o

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(+) > > > > i

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

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,

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

2008-04-07 Thread Arnd Bergmann
On Tuesday 08 April 2008, Hollis Blanchard wrote: > If there is one thing I have learned in my various porting efforts, it's that > using a variable-sized type in an interface is just begging for trouble. > > x86 uses fixed 64-bit variables here (even with x86-32), so that might be the > right s

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

2008-04-07 Thread Nikola Ciprich
On Mon, 7 Apr 2008, Marcelo Tosatti wrote: > I think the logic to wakeup tasks in HLT is racy. Nothing prevents > a timer event from being lost if it fires in between guest exit and > vcpu_block(). > > Please try the patch below. > Hi Marcelo, the problem persists even with patch. [ 1905.899112]