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 0/6] MSI-INTx interrupt translation for HVM

To: Qing He <qing.he@xxxxxxxxx>
Subject: Re: [Xen-devel] [PATCH 0/6] MSI-INTx interrupt translation for HVM
From: Shohei Fujiwara <fujiwara-sxa@xxxxxxxxxxxxxxx>
Date: Wed, 14 Jan 2009 15:39:29 +0900
Cc: "xen-devel@xxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxx>
Delivery-date: Tue, 13 Jan 2009 22:40:04 -0800
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to: <20090113092813.GA9499@ub-qhe2>
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: <20090113175932.EA8A.CB716985@xxxxxxxxxxxxxxx> <20090113092813.GA9499@ub-qhe2>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
On Tue, 13 Jan 2009 17:28:13 +0800
Qing He <qing.he@xxxxxxxxx> wrote:

> On Tue, 2009-01-13 at 17:05 +0800, Shohei Fujiwara wrote:
> > What I understand are the followings.
> > Second, it is nice to create blacklist of unsuitable devices for MSI-INTx
> > interrupt, isn't it? The reason is the problem seems device-specific 
> > problem.
> 
> This is reasonable, however, I don't know a real unsuitable at this
> time. Isn't it better to add the blacklist when there is the real need?
> 
> What do you think?

I think adding blacklist now is better, because it is easy to specify
an unsuitable device when it is found. But I don't know what other
developers think about this.


BTW, I found a issue when I read your code.

MSI-INTx translation is enabled when guest domain is started.
When guest OS enables MSI/MSI-X, MSI-INTx translation is disabled.
But when guest OS disables MSI/MSI-X, MSI-INTx translation isn't re-enabled.
This means normal INTx interrupt is used.

I think the results should be the same at all times,
so it is good to re-enable MSI-INTx translation.

Thanks,
--
Shohei Fujiwara


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

<Prev in Thread] Current Thread [Next in Thread>