RE: [Xen-devel] [VTD][PATCH] Fix ISA IRQ alias issue

["Han, Weidong" <weidong.han@xxxxxxxxx>]

Keir, attached patch adds ISA IRQ handling for EOI of ISA IRQs via
IO-APIC, does it eliminate your concern? 

-- Weidong

Keir Fraser wrote:
> So what if a PCI-ISA IRQ is delivered via, and EOIed via, the IO-APIC?
> 
>  -- Keir
> 
> On 17/11/07 01:37, "Han, Weidong" <weidong.han@xxxxxxxxx> wrote:
> 
>> This patch doesn't change EOI of PCI-ISA IRQs via IO-APIC, it just
>> uses a separate function to handle EOI of PCI-ISA IRQs via the PIC.
>> Without this patch, assigning multiple devices to guest which uses
>> PIC maybe results in disabling the IRQ. 
>> 
>> -- Weidong
>> 
>> Keir Fraser wrote:
>>> A concerning amount of code churn and I'm not clear it works for EOI
>>> of PCI-ISA IRQs via the IO-APIC. This probably isn't for 3.2.0.
>>> 
>>>  -- Keir
>>> 
>>> On 15/11/07 06:32, "Han, Weidong" <weidong.han@xxxxxxxxx> wrote:
>>> 
>>>> When assign multiple devices to guest which uses PIC, ISA IRQ alias
>>>> may occur. This patch splits ISA IRQ and GSI eoi function. In ISA
>>>> IRQ eoi function, searches all assigned mirqs and does eoi for the
>>>> corresponding mirqs which match the eoi ISA IRQ. Therefore fix ISA
>>>> IRQ alias issue. 
>>>> 
>>>> 
>>>> Signed-off-by: Weidong Han <weidong.han@xxxxxxxxx>
>>>> _______________________________________________
>>>> Xen-devel mailing list
>>>> Xen-devel@xxxxxxxxxxxxxxxxxxx
>>>> http://lists.xensource.com/xen-devel
>>> 
>>> 
>>> 
>>> _______________________________________________
>>> Xen-devel mailing list
>>> Xen-devel@xxxxxxxxxxxxxxxxxxx
>>> http://lists.xensource.com/xen-devel

isapic-2.patch
Description: isapic-2.patch

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel