Re: [UNTESTED] KVM: do not call kvm_set_irq from irq disabled section

2010-04-23 Thread Chris Lalancette
this is not scalable anyway. Yes. We also wanted to convert the ioapic/pic to spinlocks so we could queue the interrupt from the PIT directly instead of using KVM_REQ_PENDING_TIMER which keeps confusing me. Chris Lalancette posted a patchset for this a while back but it was never completed

Re: [PATCH] Print a user-friendly message on failed vmentry

2010-05-20 Thread Chris Lalancette
complete analysis of the vmentry failure code to be more certain about the reason for failure. -- Chris Lalancette -- To unsubscribe from this list: send the line unsubscribe kvm in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo

Re: Is AMD rev F the same thing as socket F?

2009-10-19 Thread Chris Lalancette
in hex). -- Chris Lalancette -- To unsubscribe from this list: send the line unsubscribe kvm in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html

[PATCH] Fix up vmx_set_segment for booting older guests.

2009-10-20 Thread Chris Lalancette
that bits 63:32 of this field *must* be 0. Use vmcs_write32() where appropriate. This fixes booting of an absolutely ancient Red Hat Linux 5.2 (not Enterprise Linux!) guest. Signed-off-by: Chris Lalancette clala...@redhat.com --- arch/x86/kvm/vmx.c | 17 - 1 files changed, 16

[PATCH] Print Guest VMCS state on vmexit failure

2009-10-20 Thread Chris Lalancette
If we fail to handle a VMEXIT for some reason, print out a lot more debugging information about the state of the GUEST VMCS area. This does not fix a bug, but helps a lot when trying to track down the cause of a VMEXIT/VMENTRY failure. Signed-off-by: Chris Lalancette clala...@redhat.com

Re: [PATCH] Print Guest VMCS state on vmexit failure

2009-10-20 Thread Chris Lalancette
Avi Kivity wrote: On 10/20/2009 04:50 PM, Chris Lalancette wrote: If we fail to handle a VMEXIT for some reason, print out a lot more debugging information about the state of the GUEST VMCS area. This does not fix a bug, but helps a lot when trying to track down the cause of a VMEXIT/VMENTRY

Re: [PATCH] Print Guest VMCS state on vmexit failure

2009-10-20 Thread Chris Lalancette
the register state to stderr before abort()'ing on an unknown vm exit. Libvirt takes the output from stderr and stores it in /var/log/libvirt/qemu/guestname. So you would still be able to see this output when using libvirt. -- Chris Lalancette -- To unsubscribe from this list: send the line unsubscribe

Re: [PATCH] Fix up vmx_set_segment for booting older guests.

2009-10-20 Thread Chris Lalancette
); if (seg_desc-g) { kvm_desct-limit = 12; -- Chris Lalancette -- To unsubscribe from this list: send the line unsubscribe kvm in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html

[PATCH] Make sure get_user_desc() doesn't sign extend.

2009-10-21 Thread Chris Lalancette
the caller intended. This patch casts the entire thing to unsigned before returning, which generates almost the same assembly as the current code but replaces the final cltq (sign extend) with a mov %eax %eax (zero-extend). This fixes booting certain guests under KVM. Signed-off-by: Chris Lalancette

[PATCH] Make sure get_user_desc() doesn't sign extend.

2009-10-23 Thread Chris Lalancette
time so I guess I sent it to the wrong places) Signed-off-by: Chris Lalancette clala...@redhat.com --- arch/x86/include/asm/desc.h |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/arch/x86/include/asm/desc.h b/arch/x86/include/asm/desc.h index e8de2f6..617bd56 100644

[PATCH 0/5]: Fix kdump under KVM

2009-10-27 Thread Chris Lalancette
This patch series aims to get kdump working inside a KVM guest. The current problem with using kdump is that KVM always delivers PIT interrupts to the BSP, and the BSP only. While this is technically allowed by the MPS spec, most motherboards actually deliver timer interrupts to *any* LAPIC in

[PATCH 1/5] Fix up some comments around the source tree.

2009-10-27 Thread Chris Lalancette
Signed-off-by: Chris Lalancette clala...@redhat.com --- :100644 100644 34b700f... ba61f27... M arch/x86/kvm/svm.c :100644 100644 38a2d20... cd6f92b... M virt/kvm/ioapic.c :100644 100644 bd44fb4... c22bc17... M virt/kvm/kvm_main.c arch/x86/kvm/svm.c |2 +- virt/kvm/ioapic.c |2

[PATCH 3/5] Remove references to VCPU in i8254

2009-10-27 Thread Chris Lalancette
) In __kvm_migrate_pit_timer, we have to migrate the PIT around with the BSP, since hrtimers work on a per-CPU basis. I've added a comment here to clarify why this is needed. Signed-off-by: Chris Lalancette clala...@redhat.com --- :100644 100644 fab7440... d5c08fa... M arch/x86/kvm/i8254.c :100644 100644 d4c1c7f

[PATCH 5/5] Fix kdump under KVM.

2009-10-27 Thread Chris Lalancette
. I don't love the concept of doing kvm_irq_kick_vcpus() from within pit_timer_fn(). A PIT is not connected to a CPU at all, only to a PIC or APIC. However, if a CPU enters idle, this is the only way to wake it up to check for the interrupt. Signed-off-by: Chris Lalancette clala...@redhat.com

[PATCH 4/5] Remove timer.c

2009-10-27 Thread Chris Lalancette
The code in arch/x86/kvm/timer.c is not similar enough between the various implementations to really share it. Move the implementation into the LAPIC code, and then remove timer.c Signed-off-by: Chris Lalancette clala...@redhat.com --- :100644 100644 31a7035... 8d9adf6... M arch/x86/kvm

[PATCH 2/5] Remove KVM_REQ_PENDING_TIMER.

2009-10-27 Thread Chris Lalancette
KVM_REQ_PENDING_TIMER is set and cleared in a couple of places, but it never seems to be actually checked. Remove it. Signed-off-by: Chris Lalancette clala...@redhat.com --- :100644 100644 eea4043... 72b5144... M arch/x86/kvm/timer.c :100644 100644 2ef39062.. 93a65b4... M arch/x86/kvm/x86.c

Re: [PATCH 5/5] Fix kdump under KVM.

2009-10-28 Thread Chris Lalancette
Marcelo Tosatti wrote: On Tue, Oct 27, 2009 at 05:41:07PM +0100, Chris Lalancette wrote: This patch is the main point of the series. In order for kdump to properly work inside a KVM guest, we need to make sure that all VCPUs in virtual wire APIC mode get kicked to try and pick up the timer

[PATCH] x86: Make sure get_user_desc() doesn't sign extend.

2009-10-28 Thread Chris Lalancette
to the last two submissions) Signed-off-by: Chris Lalancette clala...@redhat.com --- arch/x86/include/asm/desc.h |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/arch/x86/include/asm/desc.h b/arch/x86/include/asm/desc.h index e8de2f6..617bd56 100644 --- a/arch/x86/include/asm

Re: [PATCH 0/5]: Fix kdump under KVM

2009-10-29 Thread Chris Lalancette
Avi Kivity wrote: On 10/28/2009 12:13 PM, Chris Lalancette wrote: The kick from i8254 code is pretty bad, as you mention. I forget why it is needed at all - shouldn't kvm_set_irq() end up kicking the correct As I understand it, that's not quite how it works. From what I can see, what

Re: kvm problems on new hardware

2009-10-30 Thread Chris Lalancette
/2009/10/28/201, and see if it helps? -- Chris Lalancette -- To unsubscribe from this list: send the line unsubscribe kvm in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html

Re: [PATCH 5/5] Fix kdump under KVM.

2009-10-30 Thread Chris Lalancette
Marcelo Tosatti wrote: On Wed, Oct 28, 2009 at 11:21:42AM +0100, Chris Lalancette wrote: Marcelo Tosatti wrote: On Tue, Oct 27, 2009 at 05:41:07PM +0100, Chris Lalancette wrote: This patch is the main point of the series. In order for kdump to properly work inside a KVM guest, we need

Re: [PATCH 5/5] Fix kdump under KVM.

2009-10-30 Thread Chris Lalancette
Marcelo Tosatti wrote: On Fri, Oct 30, 2009 at 01:23:57PM +0100, Chris Lalancette wrote: Marcelo Tosatti wrote: On Wed, Oct 28, 2009 at 11:21:42AM +0100, Chris Lalancette wrote: Marcelo Tosatti wrote: On Tue, Oct 27, 2009 at 05:41:07PM +0100, Chris Lalancette wrote: This patch is the main

Re: [PATCH 5/5] Fix kdump under KVM.

2009-11-02 Thread Chris Lalancette
Avi Kivity wrote: On 10/30/2009 02:23 PM, Chris Lalancette wrote: In the meantime, I've gotten the set_irq from IRQ context that Avi suggested working, and the fixing up of this IOAPIC check is the last bit to actually get kdump working. There are two problems

[PATCH] x86: Make sure get_user_desc() doesn't sign extend.

2009-11-04 Thread Chris Lalancette
to the previous 2) Signed-off-by: Chris Lalancette clala...@redhat.com --- arch/x86/include/asm/desc.h |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/arch/x86/include/asm/desc.h b/arch/x86/include/asm/desc.h index e8de2f6..617bd56 100644 --- a/arch/x86/include/asm/desc.h +++ b/arch

[PATCH 0/12]: Fix kdump under KVM

2009-12-01 Thread Chris Lalancette
Another version of the patch series to make kdump work inside KVM guests. The current problem with using kdump is that KVM only delivers PIT interrupts to the BSP. While this is technically allowed by the MPS spec, most motherboards deliver timer interrupts to *any* LAPIC in virtual wire mode.

[PATCH 01/12] Fix up some comments around the source tree.

2009-12-01 Thread Chris Lalancette
Signed-off-by: Chris Lalancette clala...@redhat.com --- arch/x86/kvm/svm.c |2 +- virt/kvm/ioapic.c |4 ++-- virt/kvm/kvm_main.c |2 +- 3 files changed, 4 insertions(+), 4 deletions(-) diff --git a/arch/x86/kvm/svm.c b/arch/x86/kvm/svm.c index 3de0b37..68fe7f2 100644 --- a/arch

[PATCH 02/12] Make kvm_pic_reset static.

2009-12-01 Thread Chris Lalancette
Signed-off-by: Chris Lalancette clala...@redhat.com --- arch/x86/kvm/i8259.c |2 +- arch/x86/kvm/irq.h |2 -- 2 files changed, 1 insertions(+), 3 deletions(-) diff --git a/arch/x86/kvm/i8259.c b/arch/x86/kvm/i8259.c index d057c0c..59a8a68 100644 --- a/arch/x86/kvm/i8259.c +++ b/arch

[PATCH 03/12] Remove KVM_REQ_PENDING_TIMER.

2009-12-01 Thread Chris Lalancette
KVM_REQ_PENDING_TIMER is set and cleared in a couple of places, but it never seems to be actually checked. Remove it. Signed-off-by: Chris Lalancette clala...@redhat.com --- arch/x86/kvm/timer.c |5 + arch/x86/kvm/x86.c |1 - include/linux/kvm_host.h |1 - 3 files

[PATCH 06/12] Make the PIC use interrupt safe locking.

2009-12-01 Thread Chris Lalancette
Since we want to be able to call kvm_pic_set_irq() in interrupt context, convert the uses of spin_lock() to spin_lock_irqsave() as appropriate in i8259.c Signed-off-by: Chris Lalancette clala...@redhat.com --- arch/x86/kvm/i8259.c | 34 ++ 1 files changed, 22

[PATCH 05/12] Make the IOAPIC lock a spinlock.

2009-12-01 Thread Chris Lalancette
In order to be able to call kvm_set_irq from an interrupt context, the IOAPIC lock can't be a (possibly sleeping) mutex. Convert it to a spinlock. Signed-off-by: Chris Lalancette clala...@redhat.com --- virt/kvm/ioapic.c | 40 +--- virt/kvm/ioapic.h |2

[PATCH 04/12] IOAPIC timer interrupt redirect.

2009-12-01 Thread Chris Lalancette
Only redirect IRQ 0 (i.e. timer interrupt) to the BSP if the APIC destination is multiple vcpus. Signed-off-by: Chris Lalancette clala...@redhat.com --- virt/kvm/ioapic.c | 15 +-- 1 files changed, 13 insertions(+), 2 deletions(-) diff --git a/virt/kvm/ioapic.c b/virt/kvm/ioapic.c

[PATCH 08/12] Remove timer.c

2009-12-01 Thread Chris Lalancette
The code in timer.c isn't really similar enough between the i8254 and the lapic to share. Split these into separate functions, and remove timer.c Signed-off-by: Chris Lalancette clala...@redhat.com --- arch/x86/kvm/Makefile|3 +- arch/x86/kvm/i8254.c | 27

[PATCH 07/12] Rename kvm_apic_accept_pic_intr

2009-12-01 Thread Chris Lalancette
Call it kvm_apic_in_virtual_wire_mode, which is more correct. Also change it to not only operate properly on the boot CPU, but on any CPU. Signed-off-by: Chris Lalancette clala...@redhat.com --- arch/x86/kvm/i8259.c |2 +- arch/x86/kvm/irq.c |4 ++-- arch/x86/kvm/lapic.c | 17

[PATCH 11/12] Allow the PIC to signal CPUs other than BSP.

2009-12-01 Thread Chris Lalancette
While generally the PIT is used to signal only the BSP, it's doesn't actually have to do that architecturally. Allow the PIC to signal any VCPU that is in Virtual Wire mode. Signed-off-by: Chris Lalancette clala...@redhat.com --- arch/x86/kvm/i8259.c | 19 ++- 1 files changed

[PATCH 12/12] Kick appropriate CPUs when signalling interrupts.

2009-12-01 Thread Chris Lalancette
Make sure that we kick the appropriate vcpu when delivering an interrupt. This makes sure that we wake any idle cpus to cause a vcpu_run and an interrupt injection to occur. Signed-off-by: Chris Lalancette clala...@redhat.com --- virt/kvm/irq_comm.c |5 - 1 files changed, 4 insertions

[PATCH 09/12] Fix missing spin_lock in PIT timer.

2009-12-01 Thread Chris Lalancette
Changes to the irq_ack variable in the pit_state must be protected by the inject_lock spinlock; otherwise, we can erroneously inject a timer interrupt into a guest. Signed-off-by: Chris Lalancette clala...@redhat.com --- arch/x86/kvm/i8254.c |3 +++ 1 files changed, 3 insertions(+), 0

Re: [PATCH 03/12] Remove KVM_REQ_PENDING_TIMER.

2009-12-02 Thread Chris Lalancette
On 12/01/2009 08:04 PM, Jan Kiszka wrote: Chris Lalancette wrote: KVM_REQ_PENDING_TIMER is set and cleared in a couple of places, but it never seems to be actually checked. Remove it. I would suggest to study the introducing commit 06e05645661211b9eaadaf6344c335d2e80f0ba2. My strong

Re: [PATCH 12/12] Kick appropriate CPUs when signalling interrupts.

2009-12-08 Thread Chris Lalancette
On 12/02/2009 04:44 PM, Gleb Natapov wrote: On Tue, Dec 01, 2009 at 03:36:41PM +0100, Chris Lalancette wrote: Make sure that we kick the appropriate vcpu when delivering an interrupt. This makes sure that we wake any idle cpus to cause a vcpu_run and an interrupt injection to occur. Signed

[PATCH]: Fix silly output for virtio devices in /proc/interrupts

2008-05-21 Thread Chris Lalancette
register_virtio_device() is doing something silly by overwriting what the caller put into .bus_id. This causes the interrupt line for all virtio devices to show up as 0, 1, etc. in /proc/interrupts. The attached patch fixes it. Signed-off-by: Chris Lalancette [EMAIL PROTECTED] commit

Re: [Fwd: [PATCH]: Fix crash in virtio_blk during modprobe ; rmmod ; modprobe]

2008-05-22 Thread Chris Lalancette
Jens Axboe wrote: On Thu, May 22 2008, Rusty Russell wrote: On Wednesday 21 May 2008 23:12:39 Chris Lalancette wrote: diff --git a/drivers/block/virtio_blk.c b/drivers/block/virtio_blk.c index 4962e62..c678ac5 100644 --- a/drivers/block/virtio_blk.c +++ b/drivers/block/virtio_blk.c

[PATCH]: Fake emulate Intel perfctr MSRs

2008-05-29 Thread Chris Lalancette
a GPF kills the guest. Tested by me on RHEL-4 i386 and x86_64 guests, as well as F-9 guests. Signed-off-by: Chris Lalancette [EMAIL PROTECTED] diff --git a/arch/x86/kvm/vmx.c b/arch/x86/kvm/vmx.c index aaa99ed..f28789e 100644 --- a/arch/x86/kvm/vmx.c +++ b/arch/x86/kvm/vmx.c @@ -29,6 +29,7

[PATCH]: Fix i2c_bus_save, which fixes KVM live migration

2008-07-29 Thread Chris Lalancette
to receive 4 bytes. Avi, I don't know when you plan to next rebase KVM to upstream QEMU, but it's probably a good idea to carry this patch so that live migration works at all. Signed-off-by: Chris Lalancette [EMAIL PROTECTED] diff --git a/qemu/hw/i2c.c b/qemu/hw/i2c.c index 5d283fb..f711db7 100644

Re: Live Migration fails

2008-07-30 Thread Chris Lalancette
the i2c save on the sending side was only sending 1 byte, so they eventually timed out while waiting for each other. Chris Lalancette -- To unsubscribe from this list: send the line unsubscribe kvm in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org

Re: Fresh install of Windows XP hangs early in boot?

2008-07-30 Thread Chris Lalancette
under AMD. After installing WinDbg, it told me that it was a Paging Request in Non-Paged memory related to the Video memory area. Does yours look similar to that? I have not had time to track it further than that, though. Chris Lalancette -- To unsubscribe from this list: send the line unsubscribe

[PATCH]: Add a migrate_incoming monitor option

2008-07-31 Thread Chris Lalancette
, and then complete (qemu) info block ...etc. Signed-off-by: Chris Lalancette [EMAIL PROTECTED] diff --git a/qemu/monitor.c b/qemu/monitor.c index 20dcca6..c11b82c 100644 --- a/qemu/monitor.c +++ b/qemu/monitor.c @@ -504,6 +504,25 @@ static void do_cont(void) vm_start(); } +static void

Re: [PATCH]: Add a migrate_incoming monitor option

2008-08-01 Thread Chris Lalancette
command since it seems a little more natural for a management tool, but the nowait is clearly an option as well. I'll also post a cleanup patch with Dan's suggestion for the monitor patch, so both implementations will be available. Chris Lalancette -- To unsubscribe from this list: send the line

[PATCH]: Implement tcp nowait option for migration

2008-08-01 Thread Chris Lalancette
start up the receiving side similar to: qemu-kvm -M pc -S blah blah -incoming tcp://0:,nowait Then you are able to interact with the monitor before the live migration takes place. Signed-off-by: Chris Lalancette [EMAIL PROTECTED] diff --git a/qemu/migration.c b/qemu/migration.c index a64a287

Re: [PATCH]: Add a migrate_incoming monitor option

2008-08-01 Thread Chris Lalancette
to the migrate.c file, instead of polluting vl.c Actually, that won't quite work. We still need to share the incoming_monitor flag between migration.c and monitor.c. However, your suggestion is better in that this is a migration-specific flag, so I'll move it over like you suggest. Chris Lalancette

Re: [PATCH]: Implement tcp nowait option for migration

2008-08-06 Thread Chris Lalancette
Chris Lalancette wrote: Sometimes you want to be able to start up the receiving side of a live migration and actually be able to run monitor commands before you do the migration. Libvirt, in particular, wants to do this for setting up the migration. This patch implements a nowait option

Re: Migrating Vm's from one machine to another...and back

2008-08-11 Thread Chris Lalancette
, and *also* listening for incoming migrate requests. Now you do exactly the same on machine B, and you should be able to migrate back and forth between them at will. Chris Lalancette -- To unsubscribe from this list: send the line unsubscribe kvm in the body of a message to [EMAIL PROTECTED] More

Re: vmport: unknown command 13

2008-08-23 Thread Chris Lalancette
KVM syncs up, the message will disappear. Chris Lalancette -- To unsubscribe from this list: send the line unsubscribe kvm in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html

Re: [Qemu-devel] [PATCH 1/10] Refactor QEMUFile for live migration

2008-09-10 Thread Chris Lalancette
the original semantics of qemu_fopen_bdrv() and qemu_fopen(). Chris Lalancette -- To unsubscribe from this list: send the line unsubscribe kvm in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html

Re: [Qemu-devel] [PATCH 3/10] Add bdrv_flush_all()

2008-09-10 Thread Chris Lalancette
to avoid unnecessary guest down time. Simple enough, and follows the pattern in the KVM migration. Chris Lalancette -- To unsubscribe from this list: send the line unsubscribe kvm in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html

Re: [PATCH 0/2] kvm: disable virtualization on kdump

2008-10-30 Thread Chris Lalancette
bit x86 hardware doesn't usually (ever?) have VT extensions either. -- Chris Lalancette -- To unsubscribe from this list: send the line unsubscribe kvm in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html

[RFC][PATCH 1/3] Introduce a workqueue to deliver PIT timer interrupts.

2010-06-08 Thread Chris Lalancette
We really want to kvm_set_irq during the hrtimer callback, but that is risky because that is during interrupt context. Instead, offload the work to a workqueue, which is a bit safer and should provide most of the same functionality. Signed-off-by: Chris Lalancette clala...@redhat.com --- arch

[RFC][PATCH 2/3] Allow any LAPIC to accept PIC interrupts.

2010-06-08 Thread Chris Lalancette
If the guest wants to accept timer interrupts on a CPU other than the BSP, we need to remove this gate. Signed-off-by: Chris Lalancette clala...@redhat.com --- arch/x86/kvm/lapic.c | 12 +--- 1 files changed, 5 insertions(+), 7 deletions(-) diff --git a/arch/x86/kvm/lapic.c b/arch/x86

[RFC][PATCH 3/3] In DM_LOWEST, only deliver interrupts to vcpus with enabled LAPIC's

2010-06-08 Thread Chris Lalancette
Otherwise we might try to deliver a timer interrupt to a cpu that can't possibly handle it. Signed-off-by: Chris Lalancette clala...@redhat.com --- virt/kvm/irq_comm.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/virt/kvm/irq_comm.c b/virt/kvm/irq_comm.c index

[PATCH 0/3]: Fixes to IRQ routing

2010-06-10 Thread Chris Lalancette
to cancel_work_sync when destroying the PIT Chris Lalancette -- To unsubscribe from this list: send the line unsubscribe kvm in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html

[PATCH 1/3] Introduce a workqueue to deliver PIT timer interrupts.

2010-06-10 Thread Chris Lalancette
We really want to kvm_set_irq during the hrtimer callback, but that is risky because that is during interrupt context. Instead, offload the work to a workqueue, which is a bit safer and should provide most of the same functionality. Signed-off-by: Chris Lalancette clala...@redhat.com --- arch

[PATCH 3/3] In DM_LOWEST, only deliver interrupts to vcpus with enabled LAPIC's

2010-06-10 Thread Chris Lalancette
Otherwise we might try to deliver a timer interrupt to a cpu that can't possibly handle it. Signed-off-by: Chris Lalancette clala...@redhat.com --- virt/kvm/irq_comm.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/virt/kvm/irq_comm.c b/virt/kvm/irq_comm.c index

[PATCH 2/3] Allow any LAPIC to accept PIC interrupts.

2010-06-10 Thread Chris Lalancette
If the guest wants to accept timer interrupts on a CPU other than the BSP, we need to remove this gate. Signed-off-by: Chris Lalancette clala...@redhat.com --- arch/x86/kvm/lapic.c | 12 +--- 1 files changed, 5 insertions(+), 7 deletions(-) diff --git a/arch/x86/kvm/lapic.c b/arch/x86

[PATCH v2 0/3]: Fixes to IRQ routing

2010-06-14 Thread Chris Lalancette
As we've discussed previously, here is a series of patches to fix some of the IRQ routing issues we have in KVM. With this series in place I was able to successfully kdump a RHEL-5 64-bit, and RHEL-6 32- and 64-bit guest on CPU's other than the BSP. RHEL-5 32-bit kdump still does not work; it

[PATCH v2 1/3] Introduce a workqueue to deliver PIT timer interrupts.

2010-06-14 Thread Chris Lalancette
We really want to kvm_set_irq during the hrtimer callback, but that is risky because that is during interrupt context. Instead, offload the work to a workqueue, which is a bit safer and should provide most of the same functionality. Signed-off-by: Chris Lalancette clala...@redhat.com --- arch

[PATCH v2 2/3] Allow any LAPIC to accept PIC interrupts.

2010-06-14 Thread Chris Lalancette
If the guest wants to accept timer interrupts on a CPU other than the BSP, we need to remove this gate. Signed-off-by: Chris Lalancette clala...@redhat.com --- arch/x86/kvm/lapic.c | 12 +--- 1 files changed, 5 insertions(+), 7 deletions(-) diff --git a/arch/x86/kvm/lapic.c b/arch/x86

[PATCH v2 3/3] In DM_LOWEST, only deliver interrupts to vcpus with enabled LAPIC's

2010-06-14 Thread Chris Lalancette
Otherwise we might try to deliver a timer interrupt to a cpu that can't possibly handle it. Signed-off-by: Chris Lalancette clala...@redhat.com --- virt/kvm/irq_comm.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/virt/kvm/irq_comm.c b/virt/kvm/irq_comm.c index

[PATCH v3 0/3]: Fixes to IRQ routing

2010-06-16 Thread Chris Lalancette
As we've discussed previously, here is a series of patches to fix some of the IRQ routing issues we have in KVM. With this series in place I was able to successfully kdump a RHEL-5 64-bit, and RHEL-6 32- and 64-bit guest on CPU's other than the BSP. RHEL-5 32-bit kdump still does not work; it

[PATCH v3 1/3] Introduce a workqueue to deliver PIT timer interrupts.

2010-06-16 Thread Chris Lalancette
We really want to kvm_set_irq during the hrtimer callback, but that is risky because that is during interrupt context. Instead, offload the work to a workqueue, which is a bit safer and should provide most of the same functionality. Signed-off-by: Chris Lalancette clala...@redhat.com --- arch

[PATCH v3 3/3] In DM_LOWEST, only deliver interrupts to vcpus with enabled LAPIC's

2010-06-16 Thread Chris Lalancette
Otherwise we might try to deliver a timer interrupt to a cpu that can't possibly handle it. Signed-off-by: Chris Lalancette clala...@redhat.com --- virt/kvm/irq_comm.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/virt/kvm/irq_comm.c b/virt/kvm/irq_comm.c index

[PATCH v3 2/3] Allow any LAPIC to accept PIC interrupts.

2010-06-16 Thread Chris Lalancette
If the guest wants to accept timer interrupts on a CPU other than the BSP, we need to remove this gate. Signed-off-by: Chris Lalancette clala...@redhat.com --- arch/x86/kvm/lapic.c | 12 +--- 1 files changed, 5 insertions(+), 7 deletions(-) diff --git a/arch/x86/kvm/lapic.c b/arch/x86

[PATCH] Search the LAPIC's for one that will accept a PIC interrupt.

2010-06-21 Thread Chris Lalancette
as today; VCPU0 is always looked at first, so it will always get out of the loop after the first iteration. This will only go through the loop more than once during a kexec/kdump, in which case it will only do it a few times until the kexec'ed kernel programs the LAPIC and IOAPIC. Signed-off-by: Chris

Re: Windows Vista/7 repeatedly prompt to Set Network Location

2010-01-27 Thread Chris Lalancette
be causing the problem. Try setting the UUID in the libvirt XML, and that may solve it. -- Chris Lalancette -- To unsubscribe from this list: send the line unsubscribe kvm in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html

[PATCH]: Trivial qemu/configure fix

2009-01-19 Thread Chris Lalancette
If kvm is *not* detected, then the qemu/configure script gets upset. Add some quotes to make it happier. Signed-off-by: Chris Lalancette clala...@redhat.com diff --git a/qemu/configure b/qemu/configure index ff4a462..107699a 100755 --- a/qemu/configure +++ b/qemu/configure @@ -1642,7 +1642,7

Problem with X on 32 bit guest on 64-bit host

2009-02-05 Thread Chris Lalancette
this down? -- Chris Lalancette -- To unsubscribe from this list: send the line unsubscribe kvm in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html

[PATCH]: Always use KVM_VERSION to build version number

2009-08-07 Thread Chris Lalancette
, to change it to kvm-89, and once right after the release to change it back to kvm-89-devel)? Signed-off-by: Chris Lalancette clala...@redhat.com diff --git a/KVM_VERSION b/KVM_VERSION new file mode 100644 index 000..efd3e0e --- /dev/null +++ b/KVM_VERSION @@ -0,0 +1 @@ +kvm-88-devel

Re: [PATCH 2/2] add sysctl for kvm wallclock sync

2009-09-02 Thread Chris Lalancette
change will fight with ntpd. Also, the command # date 09091323 (or whatever) ceases to work like it does on bare-metal, so I think it has to be an opt-in feature. -- Chris Lalancette -- To unsubscribe from this list: send the line unsubscribe kvm in the body of a message to majord