[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] pc: Clean up PIC-to-APIC IRQ path
From: |
Jan Kiszka |
Subject: |
Re: [Qemu-devel] [PATCH] pc: Clean up PIC-to-APIC IRQ path |
Date: |
Mon, 29 Aug 2011 23:06:25 +0200 |
User-agent: |
Mozilla/5.0 (X11; U; Linux i686 (x86_64); de; rv:1.8.1.12) Gecko/20080226 SUSE/2.0.0.12-1.1 Thunderbird/2.0.0.12 Mnenhy/0.7.5.666 |
On 2011-08-29 21:25, Anthony Liguori wrote:
> On 08/27/2011 09:16 AM, Jan Kiszka wrote:
>> From: Jan Kiszka<address@hidden>
>>
>> The master PIC is connected to the LINTIN0 of the APICs. As the APIC
>> currently does not track the state of that line, we have to ask the PIC
>> to re-inject its IRQ after the CPU picked up an event from the APIC.
>>
>> Adds the proper state tracking so that we can already re-assert the CPU
>> IRQ at APIC level if there is a pending PIC IRQ. This allows to remove
>> all the old workarounds.
>>
>> The patch also fixes some failures of the kvm unit tests apic and
>> eventinj by enabling a proper CPU IRQ deassert when the guest masks some
>> pending IRQs at PIC level.
>>
>> Signed-off-by: Jan Kiszka<address@hidden>
>> ---
>>
>> It turned out that this patch from a larger cleanup series has no
>> dependencies and can be applied directly to master to fix the observed
>> bug.
>>
>> hw/apic.c | 4 +++-
>> hw/i8259.c | 10 ++--------
>> hw/pc.c | 3 ---
>> hw/pc.h | 1 -
>> 4 files changed, 5 insertions(+), 13 deletions(-)
>>
>> diff --git a/hw/apic.c b/hw/apic.c
>> index d8f56c8..22ad635 100644
>> --- a/hw/apic.c
>> +++ b/hw/apic.c
>> @@ -104,6 +104,7 @@ struct APICState {
>> QEMUTimer *timer;
>> int sipi_vector;
>> int wait_for_sipi;
>> + int pic_level;
>> };
>
> Does this need to be save/restored for migration?
Nope, but we need some other measure. I thought to remember the pic was
refreshing this after load, but I do not find any traces of this now. We
likely need a post_load handler in the i8259 that re-asserts the IRQ as
required.
Jan
signature.asc
Description: OpenPGP digital signature
- [Qemu-devel] [PATCH] pc: Clean up PIC-to-APIC IRQ path, Jan Kiszka, 2011/08/27
- Re: [Qemu-devel] [PATCH] pc: Clean up PIC-to-APIC IRQ path, Blue Swirl, 2011/08/28
- Re: [Qemu-devel] [PATCH] pc: Clean up PIC-to-APIC IRQ path, Anthony Liguori, 2011/08/29
- Re: [Qemu-devel] [PATCH] pc: Clean up PIC-to-APIC IRQ path,
Jan Kiszka <=
- Re: [Qemu-devel] [PATCH] pc: Clean up PIC-to-APIC IRQ path, Avi Kivity, 2011/08/29
- Re: [Qemu-devel] [PATCH] pc: Clean up PIC-to-APIC IRQ path, Jan Kiszka, 2011/08/29
- Re: [Qemu-devel] [PATCH] pc: Clean up PIC-to-APIC IRQ path, Blue Swirl, 2011/08/30
- Re: [Qemu-devel] [PATCH] pc: Clean up PIC-to-APIC IRQ path, Jan Kiszka, 2011/08/30
- Re: [Qemu-devel] [PATCH] pc: Clean up PIC-to-APIC IRQ path, Blue Swirl, 2011/08/30
- Re: [Qemu-devel] [PATCH] pc: Clean up PIC-to-APIC IRQ path, Peter Maydell, 2011/08/31
- Re: [Qemu-devel] [PATCH] pc: Clean up PIC-to-APIC IRQ path, Jan Kiszka, 2011/08/31
- Re: [Qemu-devel] [PATCH] pc: Clean up PIC-to-APIC IRQ path, Blue Swirl, 2011/08/31
- Re: [Qemu-devel] [PATCH] pc: Clean up PIC-to-APIC IRQ path, Jan Kiszka, 2011/08/31
- Re: [Qemu-devel] [PATCH] pc: Clean up PIC-to-APIC IRQ path, Blue Swirl, 2011/08/31