[kvm-devel] Gentoo livecd amd64 not working?

2007-05-14 Thread Wink Saville
This weekend I tried to get the Gentoo livecd for amd64 and i686 working. The i686 version successfully installed and does "run". The amd64 livecd boots and runs but then during install stops trying to unpack the 50th of 116 files at the beginning of the install process. I'm running on an Intel du

Re: [kvm-devel] Gentoo livecd amd64 not working?

2007-05-14 Thread Avi Kivity
Wink Saville wrote: > This weekend I tried to get the Gentoo livecd for amd64 and i686 working. > The i686 version successfully installed and does "run". The amd64 > livecd boots and > runs but then during install stops trying to unpack the 50th of 116 files at > the beginning of the install proces

Re: [kvm-devel] [PATCH 3/8] KVM: Adds ability to preempt an executing VCPU

2007-05-14 Thread Avi Kivity
Gregory Haskins wrote: > The VCPU executes synchronously w.r.t. userspace today, and therefore > interrupt injection is pretty straight forward. However, we will soon need > to be able to inject interrupts asynchronous to the execution of the VCPU > due to the introduction of SMP, paravirtualized

Re: [kvm-devel] [PATCH 6/8] KVM: Adds support for real NMI injection on VMX processors

2007-05-14 Thread Avi Kivity
Gregory Haskins wrote: > Signed-off-by: Gregory Haskins <[EMAIL PROTECTED]> > --- > > drivers/kvm/vmx.c | 63 > + > drivers/kvm/vmx.h |3 +++ > 2 files changed, 61 insertions(+), 5 deletions(-) > > diff --git a/drivers/kvm/vmx.c b/drivers/

Re: [kvm-devel] [PATCH 8/8] KVM: Adds support for TPR shadowing under VMX processors

2007-05-14 Thread Avi Kivity
Gregory Haskins wrote: > Signed-off-by: Gregory Haskins <[EMAIL PROTECTED]> > --- > > How was this tested? > + printk(KERN_WARNING "KVM: Warning - Host processor does " \ > +"not support TPR-shadow\n"); > KERN_DEBUG. -- error compiling committee.c: too ma

Re: [kvm-devel] [PATCH 09/10] KVM: Add statistics from interrupt subsystem

2007-05-14 Thread Avi Kivity
Gregory Haskins wrote: >* then wake up the vcpu (if necessary) > @@ -2540,6 +2544,7 @@ static void kvm_vcpu_intr(struct kvm_irqsink *this, >*/ > direct_ipi = task_cpu(vcpu->irq.task); > BUG_O

Re: [kvm-devel] [PATCH 2/2] KVM: in-kernel-apic modification to QEMU

2007-05-14 Thread Avi Kivity
Anthony Liguori wrote: >> int intno; >> >> +#ifdef USE_KVM >> +if (!kvm_allowed) { >> +#endif >> intno = apic_get_interrupt(env); >> if (intno >= 0) { >> >> > > Indent the inner block please :-) > > Indenting the inner code would cause merging to be very difficult,

Re: [kvm-devel] [PATCH/RFC 4/9] Basic guest virtual devices infrastructure

2007-05-14 Thread Avi Kivity
Carsten Otte wrote: > From: Carsten Otte <[EMAIL PROTECTED]> > > This patch adds support for a new bus type that manages paravirtualized > devices. The bus uses the s390 diagnose instruction to query devices, and > match them with the corresponding drivers. > Future enhancements should include hot

Re: [kvm-devel] [PATCH/RFC 4/9] Basic guest virtual devices infrastructure

2007-05-14 Thread Carsten Otte
Avi Kivity wrote: > Interesting. We could use a variation this for x86 as well, but I'm not > sure how easy it is to integrate it into closed source OSes (Windows). > The diag instruction could be replaced by a hypercall which would make > the code generic. I think we need to freeze the hyperc

Re: [kvm-devel] [PATCH/RFC 6/9] virtual block device driver

2007-05-14 Thread Avi Kivity
Carsten Otte wrote: > From: Carsten Otte <[EMAIL PROTECTED]> > > This driver provides access to virtual block devices. It does use its own > make_request function which passes the bio to a workqueue thread. The > workqueue > thread does use the diagnose hypervisor call to call the hosting Linux. >

Re: [kvm-devel] [PATCH/RFC 6/9] virtual block device driver

2007-05-14 Thread Avi Kivity
Carsten Otte wrote: > From: Carsten Otte <[EMAIL PROTECTED]> > > This driver provides access to virtual block devices. It does use its own > make_request function which passes the bio to a workqueue thread. The > workqueue > thread does use the diagnose hypervisor call to call the hosting Linux. >

Re: [kvm-devel] [PATCH/RFC 7/9] Virtual network guest device driver

2007-05-14 Thread Avi Kivity
Anthony Liguori wrote: > Dor Laor wrote: > >> Furthermore, >> >> >>> the plan is to completely rearchitect the netback/netfront protocol for >>> the next Xen release (this effort is referred to netchannel2). >>> >>> >> But isn't Jeremy Fitzhardinge is pushing big patch queue

Re: [kvm-devel] [PATCH 00/10] in-kernel APIC v3 (kernel side)

2007-05-14 Thread Gregory Haskins
>>> On Sun, May 13, 2007 at 8:38 AM, in message <[EMAIL PROTECTED]>, Avi Kivity <[EMAIL PROTECTED]> wrote: > Gregory Haskins wrote: >> The load- average on my system is about 1 while XP is idling. qemu seems to > be mostly at "0%" but will bounce up to 1% on occasion. Here is the output > of

Re: [kvm-devel] [PATCH/RFC 4/9] Basic guest virtualdevices infrastructure

2007-05-14 Thread Dor Laor
>Avi Kivity wrote: >> Interesting. We could use a variation this for x86 as well, but I'm not >> sure how easy it is to integrate it into closed source OSes (Windows). >> The diag instruction could be replaced by a hypercall which would make >> the code generic. >I think we need to freeze the hype

Re: [kvm-devel] [PATCH/RFC 7/9] Virtual network guest device driver

2007-05-14 Thread Avi Kivity
ron minnich wrote: > We had hoped to get something like this into Xen. On Xen, for example, > the block device and ethernet device interfaces are as different as > one could imagine. Disk I/O does not steal pages from the guest. The > network does. Disk I/O is in 4k chunks, period, with a bitmap >

Re: [kvm-devel] [PATCH 00/10] in-kernel APIC v3 (kernel side)

2007-05-14 Thread Dor Laor
>> >> A good test is to let Windows boot and idle itself, then compare the >> process cpu time under the TIME+ column with model- 0 and model- 1. > >Will do. One that that I notice that I can't explain yet is as follows: > >When I boot windows + level-1, the point at which windows is running in 16

Re: [kvm-devel] [PATCH 2/4] KVM-USER: Make the kvm_allowed flag always defined so we dont need #ifdefs

2007-05-14 Thread Avi Kivity
Gregory Haskins wrote: > Non-performance critical code is made more awkward by having to always define > both "#ifdef KVM" and "if (kvm_allowed)". Define "kvm_allowed = 0" by > default. Anthony Ligouri is credited with the idea. > > Signed-off-by: Gregory Haskins <[EMAIL PROTECTED]> > --- > > qe

Re: [kvm-devel] [PATCH 00/10] in-kernel APIC v3 (kernel side)

2007-05-14 Thread Avi Kivity
Gregory Haskins wrote: >> A good test is to let Windows boot and idle itself, then compare the >> process cpu time under the TIME+ column with model- 0 and model- 1. >> > > Will do. One that that I notice that I can't explain yet is as follows: > > When I boot windows + level-1, the poi

Re: [kvm-devel] [PATCH 05/10] KVM: Adds ability to signal userspace using a file-descriptor

2007-05-14 Thread Gregory Haskins
>>> On Sun, May 13, 2007 at 9:00 AM, in message <[EMAIL PROTECTED]>, Avi Kivity <[EMAIL PROTECTED]> wrote: > Gregory Haskins wrote: >> Signed- off- by: Gregory Haskins <[EMAIL PROTECTED]> >> > > Please include patch descriptions. Ack. On that topic: Does anyone know how to retroactively cha

Re: [kvm-devel] [PATCH 05/10] KVM: Adds ability to signal userspace using a file-descriptor

2007-05-14 Thread Avi Kivity
Gregory Haskins wrote: >> >> Is having a read() (or a write()) actually necessary? >> > > Based on what I know: yes. It could be a case of ignorance, however ;) > > Heres why I think its necessary: You need poll to simply tell you when > something is pending. You can't clear the pending st

Re: [kvm-devel] [PATCH 00/10] in-kernel APIC v3 (kernel side)

2007-05-14 Thread Gregory Haskins
>>> On Sun, May 13, 2007 at 9:10 AM, in message <[EMAIL PROTECTED]>, Avi Kivity <[EMAIL PROTECTED]> wrote: > Gregory Haskins wrote: >> I am pleased to announce v3, which builds upon v2 by adding: >> > > While I haven't reviewed all this yet, it look like we can merge this > early next week.

Re: [kvm-devel] [PATCH 00/10] in-kernel APIC v3 (kernel side)

2007-05-14 Thread Avi Kivity
Gregory Haskins wrote: > I can handle the other items, except for SVN testing. I currently only have > a Merom-T7600 and Woodcrest-5130 Intel setup. I should have some SVN > hardware at some point soon, but not sure when (probably a few weeks out at > the earliest). Any help also testing the

Re: [kvm-devel] [PATCH/RFC 7/9] Virtual network guest device driver

2007-05-14 Thread Christian Bornträger
On Monday 14 May 2007 14:05, Avi Kivity wrote: > But I agree that the growing code base is a problem. With the block > driver we can probably keep the host side in userspace, but to do the > same for networking is much more work. I do think (now) that it is doable. Interesting. What kind of user

Re: [kvm-devel] [PATCH/RFC 7/9] Virtual network guest device driver

2007-05-14 Thread Avi Kivity
Christian Bornträger wrote: > On Monday 14 May 2007 14:05, Avi Kivity wrote: > >> But I agree that the growing code base is a problem. With the block >> driver we can probably keep the host side in userspace, but to do the >> same for networking is much more work. I do think (now) that it is d

Re: [kvm-devel] [PATCH 00/10] in-kernel APIC v3 (kernel side)

2007-05-14 Thread Gregory Haskins
>>> On Sun, May 13, 2007 at 10:06 AM, in message <[EMAIL PROTECTED]>, "Dor Laor" <[EMAIL PROTECTED]> wrote: >> > >>> Superb results, when I run the old in- kernel apic I got much higher > idle >>> cpu consumtion. I know that's vmenter- vmexit latency was drastically >>> improved but 0- 1% cpu c

Re: [kvm-devel] [PATCH 00/10] in-kernel APIC v3 (kernel side)

2007-05-14 Thread Dor Laor
> >> since several months >> ago when I tested the in- kernel- apic the idle time was much worse. >After >> talking today with Avi, he said his vmentry- exit latency drastically >> improved performance and even the qemu's apic can reach these figures. > >Yeah. At first I was psyched when I saw my

Re: [kvm-devel] [PATCH/RFC 6/9] virtual block device driver

2007-05-14 Thread Carsten Otte
Avi Kivity wrote: > Is this the host file descriptor? If so, we want to use something more > abstract (if the host side is in kernel, there will be no fd, or if the > device is implemented using >1 files (or <1 files)). This is indeed the host file descriptor. Host userland uses sys_open to re

Re: [kvm-devel] [PATCH/RFC 6/9] virtual block device driver

2007-05-14 Thread Carsten Otte
Avi Kivity wrote: > Oh. Why not use Xen's pending block driver? It probably has everything > needed. We're not too eager to have our own device drivers become the solution of choice. I have'nt looked at it so far, will do. so long, Carsten ---

Re: [kvm-devel] [PATCH/RFC 4/9] Basic guest virtualdevices infrastructure

2007-05-14 Thread Carsten Otte
Dor Laor wrote: > I think Avi meant using a virtual bus as an option for HVMs too (windows > especially). Currently we're using the cpi bus. Using a new virtualized > bus might be a good idea, it's easy & clean for open source. The > question is it make life easier for HVMs. For instance, on window

Re: [kvm-devel] [PATCH/RFC 7/9] Virtual network guest device driver

2007-05-14 Thread Carsten Otte
Avi Kivity wrote: > But I agree that the growing code base is a problem. With the block > driver we can probably keep the host side in userspace, but to do the > same for networking is much more work. I do think (now) that it is doable. I agree that networking needs to be handled in the host kern

Re: [kvm-devel] [PATCH 05/10] KVM: Adds ability to signal userspace using a file-descriptor

2007-05-14 Thread Gregory Haskins
>>> On Mon, May 14, 2007 at 8:22 AM, in message <[EMAIL PROTECTED]>, Avi Kivity <[EMAIL PROTECTED]> wrote: >> So based on this, I assume eventfd must be in the kernel already? Cool. > > It is in 2.6.22- rc1. As is the anonymous inodes source which can be used > to retire kvmfs (which will prob

Re: [kvm-devel] [PATCH/PFC 0/2] s390 host support

2007-05-14 Thread Carsten Otte
Avi Kivity wrote: > If the eventfd patchset is merged, then file descriptors will become the > standard Linux handle type, and poll (or rather, epoll) will become the > standard way of waiting for something to happen. But of course if you > come up with something better we'll use that. Triggere

Re: [kvm-devel] [PATCH/RFC 6/9] virtual block device driver

2007-05-14 Thread Avi Kivity
Carsten Otte wrote: > > Avi Kivity wrote: >> Is this the host file descriptor? If so, we want to use something >> more abstract (if the host side is in kernel, there will be no fd, or >> if the device is implemented using >1 files (or <1 files)). > This is indeed the host file descriptor. Host u

Re: [kvm-devel] [PATCH 05/10] KVM: Adds ability to signal userspace using a file-descriptor

2007-05-14 Thread Avi Kivity
Gregory Haskins wrote: On Mon, May 14, 2007 at 8:22 AM, in message <[EMAIL PROTECTED]>, > Avi Kivity <[EMAIL PROTECTED]> wrote: > >>> So based on this, I assume eventfd must be in the kernel already? Cool. >>> >> It is in 2.6.22- rc1. As is the anonymous inodes sourc

Re: [kvm-devel] [PATCH/PFC 0/2] s390 host support

2007-05-14 Thread Avi Kivity
Carsten Otte wrote: > Avi Kivity wrote: >> If the eventfd patchset is merged, then file descriptors will become >> the standard Linux handle type, and poll (or rather, epoll) will >> become the standard way of waiting for something to happen. But of >> course if you come up with something bette

[kvm-devel] [PATCH] lighweight VM Exit (was:RE: guest state leak into host)

2007-05-14 Thread Dong, Eddie
Avi Kivity wrote: > > Some exit handlers (even the #PF handler) can sleep sometimes. They > call kvm_arch_ops->vcpu_put(), do some sleepy thing, then call > kvm_arch_ops->vcpu_load(). The changes in the commit make > sure that if > vcpu_put() is called, the lightweight exit is converted to a > h

Re: [kvm-devel] [PATCH] lighweight VM Exit

2007-05-14 Thread Avi Kivity
Dong, Eddie wrote: > OK, how about this patch which further reduce the light weight VM Exit > MSR save/restore? > > > diff --git a/drivers/kvm/kvm_main.c b/drivers/kvm/kvm_main.c > index 1288cff..ef96fae 100644 > --- a/drivers/kvm/kvm_main.c > +++ b/drivers/kvm/kvm_main.c > @@ -1596,6 +1596,30 @@ v

Re: [kvm-devel] [PATCH 3/8] KVM: Adds ability to preempt an executing VCPU

2007-05-14 Thread Gregory Haskins
>>> On Mon, May 14, 2007 at 5:34 AM, in message <[EMAIL PROTECTED]>, Avi Kivity <[EMAIL PROTECTED]> wrote: > Gregory Haskins wrote: >> The VCPU executes synchronously w.r.t. userspace today, and therefore >> interrupt injection is pretty straight forward. However, we will soon need >> to be able

Re: [kvm-devel] [PATCH/PFC 0/2] s390 host support

2007-05-14 Thread Carsten Otte
Avi Kivity wrote: > kvm doesn't do this directly. A hlt instruction (which is is used on > x86 to signal an idle cpu) is trapped and echoed to userspace, which > then sleeps using select(2). I've read that part. I still don't like this approach, it just does'nt fit our signal processor instruct

Re: [kvm-devel] [PATCH 8/8] KVM: Adds support for TPR shadowing under VMX processors

2007-05-14 Thread Gregory Haskins
>>> On Mon, May 14, 2007 at 7:09 AM, in message <[EMAIL PROTECTED]>, Avi Kivity <[EMAIL PROTECTED]> wrote: > Gregory Haskins wrote: >> Signed- off- by: Gregory Haskins <[EMAIL PROTECTED]> >> --- >> >> > > How was this tested? Its busted. Don't use it ;) Its just for example/comment only.

Re: [kvm-devel] [PATCH/PFC 0/2] s390 host support

2007-05-14 Thread Carsten Otte
Carsten Otte wrote: > - When the target CPU is going idle, but has not yet called signal(), > how can we figure from kernel space if it has masked this interrupt? *Ouch*. Should be select(), not signal(). - This SF.net email

Re: [kvm-devel] [PATCH] lighweight VM Exit

2007-05-14 Thread Christoph Hellwig
On Mon, May 14, 2007 at 06:14:31PM +0300, Avi Kivity wrote: > Dong, Eddie wrote: > > OK, how about this patch which further reduce the light weight VM Exit > > MSR save/restore? > > > > > > diff --git a/drivers/kvm/kvm_main.c b/drivers/kvm/kvm_main.c > > index 1288cff..ef96fae 100644 > > --- a/driv

Re: [kvm-devel] [PATCH 3/8] KVM: Adds ability to preempt an executing VCPU

2007-05-14 Thread Avi Kivity
Gregory Haskins wrote: >>> index 059f074..0f6cc32 100644 >>> --- a/drivers/kvm/kvm.h >>> +++ b/drivers/kvm/kvm.h >>> @@ - 329,6 +329,8 @@ struct kvm_vcpu_irq { >>> struct kvm_irqdevice dev; >>> int pending; >>> int deferred; >>> + struct task_struct

Re: [kvm-devel] [PATCH/PFC 0/2] s390 host support

2007-05-14 Thread Avi Kivity
Carsten Otte wrote: > Avi Kivity wrote: >> kvm doesn't do this directly. A hlt instruction (which is is used on >> x86 to signal an idle cpu) is trapped and echoed to userspace, which >> then sleeps using select(2). > I've read that part. I still don't like this approach, it just does'nt > fit

Re: [kvm-devel] [PATCH/PFC 0/2] s390 host support

2007-05-14 Thread Avi Kivity
Carsten Otte wrote: > Carsten Otte wrote: >> - When the target CPU is going idle, but has not yet called signal(), >> how can we figure from kernel space if it has masked this interrupt? > *Ouch*. Should be select(), not signal(). Ah ok. The kernel signal (or fd readiness) logic takes care of th

Re: [kvm-devel] Gentoo livecd amd64 not working?

2007-05-14 Thread Wink Saville
> Please provide a link to the livecd you used. > http://bouncer.gentoo.org/fetch/gentoo-2007.0-livecd/amd64/ I'm using bridging to connect guest to host and use the following commands: 1) qemu-img create -f qcow gentoo-amd64.img 20G 2) qemu-system-x86_64 . -hda gentoo-amd64.img -cdrom \ livecd

Re: [kvm-devel] [PATCH/RFC 5/9] s390 virtual console for guests

2007-05-14 Thread Christian Bornträger
On Friday 11 May 2007 21:00, Anthony Liguori wrote: > I think it would be better to use hvc_console as Xen now uses it too. I just had a look at hvc_console, and indeed this driver looks appropriate for us. Looking at the xen-frontend driver (~130 lines of code) and the simple interface (get_cha

Re: [kvm-devel] [PATCH/RFC 5/9] s390 virtual console for guests

2007-05-14 Thread Christian Borntraeger
On Monday 14 May 2007 18:23, Christian Bornträger wrote: > On Friday 11 May 2007 21:00, Anthony Liguori wrote: > > I think it would be better to use hvc_console as Xen now uses it too. > I just had a look at hvc_console, and indeed this driver looks appropriate As I started prototyping this front

Re: [kvm-devel] [PATCH 05/10] KVM: Adds ability to signal userspace using a file-descriptor

2007-05-14 Thread Davide Libenzi
On Mon, 14 May 2007, Avi Kivity wrote: > Gregory Haskins wrote: > > > > > > Is having a read() (or a write()) actually necessary? > > > > > > > Based on what I know: yes. It could be a case of ignorance, however ;) > > > > Heres why I think its necessary: You need poll to simply tell you

Re: [kvm-devel] [PATCH 05/10] KVM: Adds ability to signal userspace using a file-descriptor

2007-05-14 Thread Avi Kivity
Davide Libenzi wrote: > On Mon, 14 May 2007, Avi Kivity wrote: > > >> Gregory Haskins wrote: >> Is having a read() (or a write()) actually necessary? >>> Based on what I know: yes. It could be a case of ignorance, however ;) >>> >>> Heres why I think its necess

Re: [kvm-devel] [PATCH 05/10] KVM: Adds ability to signal userspace using a file-descriptor

2007-05-14 Thread Davide Libenzi
On Mon, 14 May 2007, Avi Kivity wrote: > Davide Libenzi wrote: > > On Mon, 14 May 2007, Avi Kivity wrote: > > > > > > > Gregory Haskins wrote: > > > > > > > > Is having a read() (or a write()) actually necessary? > > > > > > > > > Based on what I know: yes. It could be a cas

Re: [kvm-devel] [PATCH 05/10] KVM: Adds ability to signal userspace using a file-descriptor

2007-05-14 Thread Gregory Haskins
>>> On Mon, May 14, 2007 at 10:42 AM, in message <[EMAIL PROTECTED]>, Avi Kivity <[EMAIL PROTECTED]> wrote: > Gregory Haskins wrote: > On Mon, May 14, 2007 at 8:22 AM, in message <[EMAIL PROTECTED]>, > >> Avi Kivity <[EMAIL PROTECTED]> wrote: >> So based on this, I assum

Re: [kvm-devel] [PATCH 05/10] KVM: Adds ability to signal userspace using a file-descriptor

2007-05-14 Thread Avi Kivity
Davide Libenzi wrote: > On Mon, 14 May 2007, Avi Kivity wrote: > > >> Davide Libenzi wrote: >> >>> On Mon, 14 May 2007, Avi Kivity wrote: >>> >>> >>> Gregory Haskins wrote: >> Is having a read() (or a write()) actually necessary? >>

Re: [kvm-devel] [PATCH/RFC 5/9] s390 virtual console for guests

2007-05-14 Thread Anthony Liguori
Christian Borntraeger wrote: > On Monday 14 May 2007 18:23, Christian Bornträger wrote: > >> On Friday 11 May 2007 21:00, Anthony Liguori wrote: >> >>> I think it would be better to use hvc_console as Xen now uses it too. >>> >> I just had a look at hvc_console, and indeed this drive

Re: [kvm-devel] [PATCH 3/8] KVM: Adds ability to preempt an executing VCPU

2007-05-14 Thread Gregory Haskins
>>> On Mon, May 14, 2007 at 11:45 AM, in message <[EMAIL PROTECTED]>, Avi Kivity <[EMAIL PROTECTED]> wrote: > Gregory Haskins wrote: index 059f074..0f6cc32 100644 --- a/drivers/kvm/kvm.h +++ b/drivers/kvm/kvm.h @@ - 329,6 +329,8 @@ struct kvm_vcpu_irq { struct kvm_ir

Re: [kvm-devel] [PATCH 05/10] KVM: Adds ability to signal userspace using a file-descriptor

2007-05-14 Thread Gregory Haskins
>>> On Mon, May 14, 2007 at 1:23 PM, in message <[EMAIL PROTECTED]>, Avi Kivity <[EMAIL PROTECTED]> wrote: > >> Do you close the signaled fd after receiving the signal/event? If you >> don't close it, eventfd will always return ready (POLLIN). >> > > We don't. Anyway, that's what we thought

Re: [kvm-devel] [PATCH 05/10] KVM: Adds ability to signal userspace using a file-descriptor

2007-05-14 Thread Davide Libenzi
On Mon, 14 May 2007, Gregory Haskins wrote: > >>> On Mon, May 14, 2007 at 1:23 PM, in message <[EMAIL PROTECTED]>, > Avi Kivity <[EMAIL PROTECTED]> wrote: > > > >> Do you close the signaled fd after receiving the signal/event? If you > >> don't close it, eventfd will always return ready (POLLIN

Re: [kvm-devel] [PATCH 05/10] KVM: Adds ability to signal userspace using a file-descriptor

2007-05-14 Thread Gregory Haskins
>>> On Mon, May 14, 2007 at 3:12 PM, in message <[EMAIL PROTECTED]>, Davide Libenzi <[EMAIL PROTECTED]> wrote: > On Mon, 14 May 2007, Gregory Haskins wrote: > >> >>> On Mon, May 14, 2007 at 1:23 PM, in message <[EMAIL PROTECTED]>, >> Avi Kivity <[EMAIL PROTECTED]> wrote: >> > >> >> Do you clos

Re: [kvm-devel] [PATCH 05/10] KVM: Adds ability to signal userspace using a file-descriptor

2007-05-14 Thread Davide Libenzi
On Mon, 14 May 2007, Gregory Haskins wrote: > Thanks Davide, > That is very helpful. Is there any reason why we can't export > eventfd_signal() and eventfd_fget() to modules? I'll push a patch for that. - Davide - T

Re: [kvm-devel] [PATCH/RFC 5/9] s390 virtual console for guests

2007-05-14 Thread Arnd Bergmann
On Monday 14 May 2007, Anthony Liguori wrote: > It seems like request_irq is roughly the same as > register_external_interrupt.  I suspect that you could get away with > either patching hvc_console to use register_external_interrupt if > CONFIG_S390 or perhaps providing a common interface. > >

[kvm-devel] [PATCH 0/9] in-kernel APIC v4 (kernel side)

2007-05-14 Thread Gregory Haskins
This release incorporates v3, plus the following: 1) Avi's review comments have been added 2) Minor cleanup to the interrupt handling code 3) Conversion to eventfd userspace signaling mechanism Has been tested on A) 32 bit Windows XP w/ACPI B) 64 bit Linux (2.6.16 based) Userspace will follow

[kvm-devel] [PATCH 1/9] KVM: Adds support for in-kernel mmio handlers

2007-05-14 Thread Gregory Haskins
Signed-off-by: Gregory Haskins <[EMAIL PROTECTED]> --- drivers/kvm/kvm.h | 60 +++ drivers/kvm/kvm_main.c | 94 ++-- 2 files changed, 142 insertions(+), 12 deletions(-) diff --git a/drivers/kvm/kvm.h b/drivers/kvm/k

[kvm-devel] [PATCH 2/9] KVM: VMX - fix interrupt checking on light-exit

2007-05-14 Thread Gregory Haskins
Signed-off-by: Gregory Haskins <[EMAIL PROTECTED]> --- drivers/kvm/vmx.c |6 +++--- 1 files changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/kvm/vmx.c b/drivers/kvm/vmx.c index 804a623..e696d02 100644 --- a/drivers/kvm/vmx.c +++ b/drivers/kvm/vmx.c @@ -1934,13 +1934,13 @@ static

[kvm-devel] [PATCH 3/9] KVM: Add irqdevice object

2007-05-14 Thread Gregory Haskins
The current code is geared towards using a user-mode (A)PIC. This patch adds an "irqdevice" abstraction, and implements a "userint" model to handle the duties of the original code. Later, we can develop other irqdevice models to handle objects like LAPIC, IOAPIC, i8259, etc, as appropriate Signe

[kvm-devel] [PATCH 4/9] KVM: Adds ability to preempt an executing VCPU

2007-05-14 Thread Gregory Haskins
The VCPU executes synchronously w.r.t. userspace today, and therefore interrupt injection is pretty straight forward. However, we will soon need to be able to inject interrupts asynchronous to the execution of the VCPU due to the introduction of SMP, paravirtualized drivers, and asynchronous hyper

[kvm-devel] [PATCH 5/9] KVM: Adds ability to signal userspace using a file-descriptor

2007-05-14 Thread Gregory Haskins
Signed-off-by: Gregory Haskins <[EMAIL PROTECTED]> --- drivers/kvm/kvm.h |1 + drivers/kvm/kvm_main.c | 52 ++-- include/linux/kvm.h|1 + 3 files changed, 48 insertions(+), 6 deletions(-) diff --git a/drivers/kvm/kvm.h b/drivers/kvm/

[kvm-devel] [PATCH 6/9] KVM: Add support for in-kernel LAPIC model

2007-05-14 Thread Gregory Haskins
Signed-off-by: Gregory Haskins <[EMAIL PROTECTED]> --- drivers/kvm/Makefile |2 drivers/kvm/kernint.c | 149 + drivers/kvm/kvm.h | 35 + drivers/kvm/kvm_main.c | 182 ++ drivers/kvm/lapic.c| 1421 drivers/kvm/svm.c

[kvm-devel] [PATCH 7/9] KVM: Adds support for real NMI injection on VMX processors

2007-05-14 Thread Gregory Haskins
Signed-off-by: Gregory Haskins <[EMAIL PROTECTED]> --- drivers/kvm/vmx.c | 57 - drivers/kvm/vmx.h |3 +++ 2 files changed, 55 insertions(+), 5 deletions(-) diff --git a/drivers/kvm/vmx.c b/drivers/kvm/vmx.c index bfc7b2e..a3656b9 100644

[kvm-devel] [PATCH 8/9] KVM: Adds basic plumbing to support TPR shadow features

2007-05-14 Thread Gregory Haskins
Signed-off-by: Gregory Haskins <[EMAIL PROTECTED]> --- drivers/kvm/irqdevice.h |3 +++ drivers/kvm/kvm.h |1 + drivers/kvm/lapic.c | 15 +++ 3 files changed, 19 insertions(+), 0 deletions(-) diff --git a/drivers/kvm/irqdevice.h b/drivers/kvm/irqdevice.h index 097d

[kvm-devel] [PATCH 9/9] KVM: Add statistics from interrupt subsystem

2007-05-14 Thread Gregory Haskins
Signed-off-by: Gregory Haskins <[EMAIL PROTECTED]> --- drivers/kvm/kvm.h |5 + drivers/kvm/kvm_main.c | 17 +++-- drivers/kvm/vmx.c |2 ++ 3 files changed, 22 insertions(+), 2 deletions(-) diff --git a/drivers/kvm/kvm.h b/drivers/kvm/kvm.h index 49ce8c2..b73dd

[kvm-devel] [PATCH 0/5] in-kernel APIC v4 (usermode side)

2007-05-14 Thread Gregory Haskins
Userspace additions to coordinate with v4 kernel patches. Note that the eventfd is supported in the user-library, but QEMU doesnt use it yet. Regards, -Greg - This SF.net email is sponsored by DB2 Express Download DB2 Expres

[kvm-devel] [PATCH 1/5] KVM: Updates for compiling in-kernel APIC support with external-modules

2007-05-14 Thread Gregory Haskins
Signed-off-by: Gregory Haskins <[EMAIL PROTECTED]> --- kernel/Kbuild |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/kernel/Kbuild b/kernel/Kbuild index e9bcda7..103a179 100644 --- a/kernel/Kbuild +++ b/kernel/Kbuild @@ -1,5 +1,5 @@ EXTRA_CFLAGS := -I$(src)/include -inc

[kvm-devel] [PATCH 2/5] KVM-USER: Make the kvm_allowed flag always defined so we dont need #ifdefs

2007-05-14 Thread Gregory Haskins
Non-performance critical code is made more awkward by having to always define both "#ifdef KVM" and "if (kvm_allowed)". Define "kvm_allowed = 0" by default. Anthony Ligouri is credited with the idea. Signed-off-by: Gregory Haskins <[EMAIL PROTECTED]> --- qemu/qemu-kvm.c |9 - 1 fil

[kvm-devel] [PATCH 3/5] KVM-USER: Add ability to specify APIC emulation type from the command-line

2007-05-14 Thread Gregory Haskins
Signed-off-by: Gregory Haskins <[EMAIL PROTECTED]> --- qemu/qemu-kvm.c |1 + qemu/vl.c |5 + qemu/vl.h |1 + 3 files changed, 7 insertions(+), 0 deletions(-) diff --git a/qemu/qemu-kvm.c b/qemu/qemu-kvm.c index d4419a3..faa4684 100644 --- a/qemu/qemu-kvm.c +++ b/qemu/

[kvm-devel] [PATCH 4/5] KVM: in-kernel-apic modification to QEMU

2007-05-14 Thread Gregory Haskins
Signed-off-by: Gregory Haskins <[EMAIL PROTECTED]> --- qemu/hw/apic.c | 20 +++- qemu/hw/pc.c| 29 - qemu/qemu-kvm.c | 49 +++-- qemu/qemu-kvm.h |2 ++ qemu/vl.h |7 ++- user/kvmct

[kvm-devel] [PATCH 5/5] KVM-USER: Add support for listening for kernel-based interrupts

2007-05-14 Thread Gregory Haskins
Signed-off-by: Gregory Haskins <[EMAIL PROTECTED]> --- user/kvmctl.c | 32 user/kvmctl.h |2 ++ 2 files changed, 34 insertions(+), 0 deletions(-) diff --git a/user/kvmctl.c b/user/kvmctl.c index bc6e238..ea86426 100644 --- a/user/kvmctl.c +++ b/user/kvmctl.

[kvm-devel] SDL problem with 2.6.21/kvm-17 on x86_64?

2007-05-14 Thread Joshua Hoblitt
Hello, I'm attempting to get kvm-17 to work on a "server" system that has a kernel built without alsa support and libsdl is built without alsa or oss support. I get this error when attempting to start qemu: -- $ sudo /usr/local/kvm/bin/qemu-system-x86_64 . -hda vdisk.img -cdrom /data/ipp000.0/j

Re: [kvm-devel] [PATCH] lighweight VM Exit

2007-05-14 Thread Dong, Eddie
Avi Kivity wrote: > Why not use hardware autoloading? Is it slower than software? I believe HW is faster than SW, but the problem is that this kind of save/restore is only needed for heavy weight VM Exit in KVM. While HW doesn't provide an easy way to bypass these MSR save/restore for light weig