Re: [kvm-devel] [PATCH 0/9] in-kernel APIC v8 (kernel side)

2007-05-24 Thread Gregory Haskins
>>> On Thu, May 24, 2007 at 11:06 AM, in message <[EMAIL PROTECTED]>, Avi Kivity <[EMAIL PROTECTED]> wrote: > Gregory Haskins wrote: >> >> I bet it has something to do with the local_irq_disable vs clgi in the > kvm_run() code. >> > > clgi() overrides local irq, so I don't see how that happen

Re: [kvm-devel] [PATCH 0/9] in-kernel APIC v8 (kernel side)

2007-05-24 Thread Avi Kivity
Gregory Haskins wrote: On Thu, May 24, 2007 at 10:58 AM, in message <[EMAIL PROTECTED]>, > Avi Kivity <[EMAIL PROTECTED]> wrote: > >> Avi Kivity wrote: >> >>> Gregory Haskins wrote: >>> Passes the usual tests, and offers an 11% speedup. There is another

Re: [kvm-devel] [PATCH 0/9] in-kernel APIC v8 (kernel side)

2007-05-24 Thread Gregory Haskins
>>> On Thu, May 24, 2007 at 10:58 AM, in message <[EMAIL PROTECTED]>, Avi Kivity <[EMAIL PROTECTED]> wrote: > Avi Kivity wrote: >> Gregory Haskins wrote: >>> Passes the usual tests, and offers an 11% speedup. There is another >>> enhancement not included here (coming shortly) which exposes anothe

Re: [kvm-devel] [PATCH 0/9] in-kernel APIC v8 (kernel side)

2007-05-24 Thread Avi Kivity
Avi Kivity wrote: > Gregory Haskins wrote: >> Passes the usual tests, and offers an 11% speedup. There is another >> enhancement not included here (coming shortly) which exposes another 12% >> speedup over this as well. Stay tuned >> >> > This fails on amd, locks up completely. Bisecting now.

Re: [kvm-devel] [PATCH 0/9] in-kernel APIC v8 (kernel side)

2007-05-24 Thread Avi Kivity
Gregory Haskins wrote: > Passes the usual tests, and offers an 11% speedup. There is another > enhancement not included here (coming shortly) which exposes another 12% > speedup over this as well. Stay tuned > > This fails on amd, locks up completely. Bisecting now. lapic.c needs a #include

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

2007-05-24 Thread Gregory Haskins
Changelog: v7 plus the following: 1) Rebased on git-head 2) Removed level-senstive patch. I re-read the spec more carefully and we had it right the first time. 3) Got rid of sigprocmask from vcpu_halt per comments Passes the usual tests, and offers an 11% speedup. There is another enhancement