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 1/5] Add MSI support to XEN

To: "Tian, Kevin" <kevin.tian@xxxxxxxxx>, "Shan, Haitao" <haitao.shan@xxxxxxxxx>, Keir Fraser <Keir.Fraser@xxxxxxxxxxxx>, xen-devel <xen-devel@xxxxxxxxxxxxxxxxxxx>
Subject: Re: [Xen-devel] [PATCH 1/5] Add MSI support to XEN
From: Keir Fraser <keir.fraser@xxxxxxxxxxxxx>
Date: Fri, 28 Mar 2008 09:33:02 +0000
Cc: "Jiang, Yunhong" <yunhong.jiang@xxxxxxxxx>, "Li, Xin B" <xin.b.li@xxxxxxxxx>
Delivery-date: Fri, 28 Mar 2008 02:40:08 -0700
Envelope-to: www-data@xxxxxxxxxxxxxxxxxx
In-reply-to: <D470B4E54465E3469E2ABBC5AFAC390F024D9145@xxxxxxxxxxxxxxxxxxxxxxxxxxxx>
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/cgi-bin/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=subscribe>
List-unsubscribe: <http://lists.xensource.com/cgi-bin/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=unsubscribe>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
Thread-index: AciP14w6vOjg7j6MRbadB0OfiPPBYwAAB/ewADZXTlgAAM4IUAAAnY8Y
Thread-topic: [Xen-devel] [PATCH 1/5] Add MSI support to XEN
User-agent: Microsoft-Entourage/11.4.0.080122
On 28/3/08 09:23, "Tian, Kevin" <kevin.tian@xxxxxxxxx> wrote:

> I think the reverse. :-) Here IRQ is just a namespace which is allocatable
> and not bound to platform hard-wired logic. Each MSI just requires one
> IRQ placeholder to gear to evtchn core with the latter on top of IRQ name-
> space. However GSI or ISA IRQ more indicates platform attribute which
> doesn't fit the purpose here, though GSI can be also tweaked in some
> version of Linux kernel.

I don't understand you. Do you mean *PIRQ* is just a namespace which is
allocatable? I fully agree with that. I was talking about the
MAO_IRQ_TYPE_IRQ binding type -- here I believe 'IRQ' does refer to a real
platform resource, but 'IRQ' is not a well-defined architectural namespace
like 'GSI' or 'ISA IRQ'. So the interface should be fixed imo.

> This should work, and may solve the issue Yunhong described in
> another mail by giving Xen ability to mask device directly upon
> spurious interrupts. And... seems like less change to Linux code?
> The only concern is how complex the interface may finally go,
> and in this case Xen still needs to sync PCI config space access
> for port I/O style.

Yes, the synchronisation is pretty easy though. We just have to add a layer
of emulation to PV guest accesses to 0xcf8/0xcfc.

 -- Keir



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