Re: [kvm-devel] [PATCH] KVM-USER: Fix for CPU interrupts in QEMU

2007-07-13 Thread Avi Kivity
Dong, Eddie wrote: > Gregory Haskins wrote: > >> On Thu, 2007-07-12 at 23:39 +0800, Dong, Eddie wrote: >> >>> Greg: >>> I think you need to do the whole patch for this including kernel >>> side. Will u ? >>> >> Ya, I can do that. But I probably wont be able to look at it until >

Re: [kvm-devel] kvm-27 vs 28 I/O speed

2007-07-13 Thread Avi Kivity
Anthony Liguori wrote: >> >>> Dave, can you diff the guest dmesg with kvm-27 and kvm-28 userspace? >>> Maybe that will tell us something. >>> >> >> Here's kvm-27 and kvm-27+55a3212bc2...: >> >> >> diff -ru kvm-fast-dmesg.txt kvm-slow-dmesg.txt >> Linux version 2.6.22 ([EMAIL PROTECTED]) (gc

Re: [kvm-devel] [PATCH] dynamically create vcpus + vmx/svm structures

2007-07-13 Thread Avi Kivity
[added back cc] Paul Turner wrote: > On 7/13/07, Avi Kivity <[EMAIL PROTECTED]> wrote: >> Paul Turner wrote: >> > From: Paul Turner <[EMAIL PROTECTED]> >> > >> > - vcpus now allocated on demand >> > - vmx/svm fields separated into arch specific structures on vcpus >> > - vmx/svm fields now only al

[kvm-devel] [GIT PULL][RESEND] KVM Updates for 2.6.23-rc1

2007-07-13 Thread Avi Kivity
Linus, please do your usual thing from the repository and branch at git://git.kernel.org/pub/scm/linux/kernel/git/avi/kvm.git for-linus This contains kvm updates for the 2.6.23 merge window, including - performance improvements - suspend/resume fixes - guest smp - random fixes and cleanups

[kvm-devel] Minor install fix

2007-07-13 Thread John Lenz
I noticed kvm 29 added a command into make install to move the distro installed kvm.ko modules out of the way. But for me, I had already removed the drivers/kvm directory, so make install was getting a file does not exist error. So I added a test for the directory. John --- kvm-29/kernel/Makef

[kvm-devel] autodesk autocad our price $129 You save: $6590.05

2007-07-13 Thread Lorraine Trent
save $6590 our price - $129.95 Autodesk Autocad 2008 http://zebaratt.com - This SF.net email is sponsored by DB2 Express Download DB2 Express C - the FREE version of DB2 express and take control of your XML. No limits. Just d

Re: [kvm-devel] kvm-27 vs 28 I/O speed

2007-07-13 Thread Anthony Liguori
Dave Hansen wrote: > On Fri, 2007-07-13 at 16:41 +0300, Avi Kivity wrote: > >> I tested FC6 i386 about found no difference with or without the patch. >> > > I've git bisected down the the same patch that you pasted above. I've > also tried just applying that single patch to kvm-27 and repr

Re: [kvm-devel] kvm-27 vs 28 I/O speed

2007-07-13 Thread Dave Hansen
On Fri, 2007-07-13 at 17:39 +0200, Luca wrote: > Dave, can you confirm that backing out my patch fixes your speed > issue? Sure, I'll check current git with and without your patch. I've reproduced that _applying_ it to kvm-27 creates the slowdown, but I'll double-check that backing it out restor

Re: [kvm-devel] kvm-27 vs 28 I/O speed

2007-07-13 Thread Dave Hansen
On Fri, 2007-07-13 at 16:41 +0300, Avi Kivity wrote: > I tested FC6 i386 about found no difference with or without the patch. I've git bisected down the the same patch that you pasted above. I've also tried just applying that single patch to kvm-27 and reproduced the slow behavior. > Dave, can y

Re: [kvm-devel] kvm-27 vs 28 I/O speed

2007-07-13 Thread Gregory Haskins
On Fri, 2007-07-13 at 17:39 +0200, Luca wrote: > Still, I fail to see why enabling the ports should reduce the speed. > Dave, can you confirm that backing out my patch fixes your speed issue? BTW: I said what I said because I thought people had already bisected the problem down to your patch. If

Re: [kvm-devel] kvm-27 vs 28 I/O speed

2007-07-13 Thread Gregory Haskins
On Fri, 2007-07-13 at 17:39 +0200, Luca wrote: > On 7/13/07, Gregory Haskins <[EMAIL PROTECTED]> wrote: > > On Fri, 2007-07-13 at 16:46 +0200, Luca wrote: > > > On 7/13/07, Gregory Haskins <[EMAIL PROTECTED]> wrote: > > > > > Just to clarify: you are suggesting that the "old" IDE driver used to > >

Re: [kvm-devel] kvm-27 vs 28 I/O speed

2007-07-13 Thread Luca
On 7/13/07, Gregory Haskins <[EMAIL PROTECTED]> wrote: > On Fri, 2007-07-13 at 16:46 +0200, Luca wrote: > > On 7/13/07, Gregory Haskins <[EMAIL PROTECTED]> wrote: > > > Just to clarify: you are suggesting that the "old" IDE driver used to > > see that the controller was disabled and reprogrammed it

Re: [kvm-devel] kvm-27 vs 28 I/O speed

2007-07-13 Thread Gregory Haskins
On Fri, 2007-07-13 at 16:46 +0200, Luca wrote: > On 7/13/07, Gregory Haskins <[EMAIL PROTECTED]> wrote: > Just to clarify: you are suggesting that the "old" IDE driver used to > see that the controller was disabled and reprogrammed it by itself > (fast mode); now it sees it's enabled and leaves wh

Re: [kvm-devel] kvm-27 vs 28 I/O speed

2007-07-13 Thread Luca
On 7/13/07, Gregory Haskins <[EMAIL PROTECTED]> wrote: > On Fri, 2007-07-13 at 16:41 +0300, Avi Kivity wrote: > > > > I tested FC6 i386 about found no difference with or without the patch. > > > > Dave, can you diff the guest dmesg with kvm-27 and kvm-28 userspace? > > Maybe that will tell us somet

Re: [kvm-devel] kvm-27 vs 28 I/O speed

2007-07-13 Thread Gregory Haskins
On Fri, 2007-07-13 at 16:41 +0300, Avi Kivity wrote: > I tested FC6 i386 about found no difference with or without the patch. > > Dave, can you diff the guest dmesg with kvm-27 and kvm-28 userspace? > Maybe that will tell us something. > I can say this: It would seem that the IDE enable bits

Re: [kvm-devel] kvm-27 vs 28 I/O speed

2007-07-13 Thread Avi Kivity
Luca wrote: > On 7/13/07, Avi Kivity <[EMAIL PROTECTED]> wrote: >> Dave Hansen wrote: >> > On Thu, 2007-07-12 at 08:37 +0300, Avi Kivity wrote: >> > >> >> Can you confirm it by backing out that one patch? >> >> >> > >> > Do you know the git commit id by chance? >> > >> > -- Dave >> > >> > >> commit

Re: [kvm-devel] kvm-27 vs 28 I/O speed

2007-07-13 Thread Luca
On 7/13/07, Avi Kivity <[EMAIL PROTECTED]> wrote: > Dave Hansen wrote: > > On Thu, 2007-07-12 at 08:37 +0300, Avi Kivity wrote: > > > >> Can you confirm it by backing out that one patch? > >> > > > > Do you know the git commit id by chance? > > > > -- Dave > > > > > commit 55a3212bc2f5ecddcd4c5cdf2

Re: [kvm-devel] apic+ioapiuc patch

2007-07-13 Thread Avi Kivity
Dong, Eddie wrote: > Avi: > Per our discussion, we will only support all user level irqchip > or all kernel level irqchip. > Here is the patch against lapic2 that passed RHEL5U test. Please give > comments. > > > thx,eddie > > General comments: - please split into a hlt patch, lapic p

Re: [kvm-devel] kvm-27 vs 28 I/O speed

2007-07-13 Thread Avi Kivity
Dave Hansen wrote: > On Thu, 2007-07-12 at 08:37 +0300, Avi Kivity wrote: > >> Can you confirm it by backing out that one patch? >> > > Do you know the git commit id by chance? > > -- Dave > > commit 55a3212bc2f5ecddcd4c5cdf2bfb37ad71e45ff2 Author: Luca Tettamanti <[EMAIL PROTECTED]> Dat

[kvm-devel] apic+ioapiuc patch

2007-07-13 Thread Dong, Eddie
Avi: Per our discussion, we will only support all user level irqchip or all kernel level irqchip. Here is the patch against lapic2 that passed RHEL5U test. Please give comments. thx,eddie diff --git a/drivers/kvm/Makefile b/drivers/kvm/Makefile index 952dff3..b29651b 100644 --- a/d

Re: [kvm-devel] [PATCH] Fix QEMU target detection

2007-07-13 Thread Avi Kivity
Jun Koi wrote: > On 7/10/07, Avi Kivity <[EMAIL PROTECTED]> wrote: >> Anthony Liguori wrote: >> > This looks like a typo. I think the intention of the if is to >> normalize >> > i[3456]86 to i386 since that's the target name for QEMU. >> > >> > Signed-off-by: Anthony Liguori <[EMAIL PROTECTED]> >>

Re: [kvm-devel] [PATCH] dynamically create vcpus + vmx/svm structures

2007-07-13 Thread Avi Kivity
Paul Turner wrote: > From: Paul Turner <[EMAIL PROTECTED]> > > - vcpus now allocated on demand > - vmx/svm fields separated into arch specific structures on vcpus > - vmx/svm fields now only allocated on corresponding architectures > > - Paul > > diff --git a/drivers/kvm/kvm.h b/drivers/kvm/kvm.h >