WARNING - OLD ARCHIVES

This is an archived copy of the Xen.org mailing list, which we have preserved to ensure that existing links to archives are not broken. The live archive, which contains the latest emails, can be found at http://lists.xen.org/
   
 
 
Xen 
 
Home Products Support Community News
 
   
 

xen-devel

Re: [Xen-devel] [PATCH] [RFC] MSI and interrupt mapping

To: Keir Fraser <keir.fraser@xxxxxxxxxxxxx>
Subject: Re: [Xen-devel] [PATCH] [RFC] MSI and interrupt mapping
From: Espen Skoglund <espen.skoglund@xxxxxxxxxxxxx>
Date: Wed, 4 Feb 2009 18:20:51 +0000
Cc: xen-devel@xxxxxxxxxxxxxxxxxxx, Espen Skoglund <espen.skoglund@xxxxxxxxxxxxx>
Delivery-date: Wed, 04 Feb 2009 10:22:13 -0800
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to: <C5AF88B3.BEC%keir.fraser@xxxxxxxxxxxxx>
List-help: <mailto:xen-devel-request@lists.xensource.com?subject=help>
List-id: Xen developer discussion <xen-devel.lists.xensource.com>
List-post: <mailto:xen-devel@lists.xensource.com>
List-subscribe: <http://lists.xensource.com/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=subscribe>
List-unsubscribe: <http://lists.xensource.com/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=unsubscribe>
References: <18825.54424.7532.479107@xxxxxxxxxxxxxxxxxx> <C5AF88B3.BEC%keir.fraser@xxxxxxxxxxxxx>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
[Keir Fraser]
> On 04/02/2009 17:47, "Espen Skoglund" <espen.skoglund@xxxxxxxxxxxxx> wrote:
>> More generally speaking, the current state of affairs with regards
>> to interrupt management in Xen can be a bit confusing from the
>> source code point of view.  Essentially, all irqs except some
>> legacy interrupts are assumed to be IOAPIC irqs.  This assumption
>> breaks down with the introduction of MSIs and per-domain pirq
>> tables.  It might therefore be worthwile to rename a few things and
>> slightly restructure the Xen code to make the distinction between
>> IOAPIC irqs and other interrupts clearer.  I could volunteer to
>> have a stab at such a cleanup if this is something people want.
>> 
>> Comments?

> Sketch out what you mean in a bit more detail and I'll think about
> it. The stuff in arch/x86/irq.c at least makes reasonable sense to
> me right now (of course ;-).

Get rid of the global irq_vector[] and vector_irq[] arrays.  Where
needed (e.g., in the IOAPIC code) use the dom0 pirq arrays instead.
>From a quick glance, the stuff in arch/x86/irq.c makes sense as it is;
the exception perhaps being the dump_irqs() function which does not
take MSIs into account.

        eSk



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