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] PCI MSI questions

To: Jan Beulich <jbeulich@xxxxxxxxxx>, <xen-devel@xxxxxxxxxxxxxxxxxxx>
Subject: Re: [Xen-devel] PCI MSI questions
From: Keir Fraser <keir.fraser@xxxxxxxxxxxxx>
Date: Thu, 24 Jul 2008 08:20:48 +0100
Cc:
Delivery-date: Thu, 24 Jul 2008 00:21:08 -0700
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to: <4888452A.76E4.0078.0@xxxxxxxxxx>
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>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
Thread-index: AcjtXctiCddWvVlREd2/1AAWy6hiGQ==
Thread-topic: [Xen-devel] PCI MSI questions
User-agent: Microsoft-Entourage/11.4.0.080122
On 24/7/08 08:02, "Jan Beulich" <jbeulich@xxxxxxxxxx> wrote:

> 1) There currently seems to be a hidden requirement of NR_PIRQS in the
> kernel needing to be no smaller than NR_IRQS in the hypervisor.
> Otherwise, the pirq returned from PHYSDEVOP_map_pirq may collide
> with the dynamic IRQs in the kernel or even be out of range altogether.
> Therefore I think that NR_PIRQS has to become a variable defaulting
> to 256 but getting initialized from a hypervisor reported value (perhaps
> in start_info, or else from a new (sub-)hypercall).

Or have the kernel remap the return value of map_pirq into its own PIRQ
namespace, and maintain appropriate translation info? Although, it'd be nice
to have dynamic NR_IRQS sizing anyway -- people who want to run lots of
domUs currently may have to recompile dom0 with more DYNIRQS.

> 4) The hypervisor option "msi_irq_enable" seems to be named pretty
> oddly - both the "irq" and the "enable" in the name are more or less
> redundant. So unless there's a reason for this long a name for an
> option that generally I would expect most people want to set (at
> least on bigger systems), I'd like to change it into "msi" or, if that's
> considered prone for ambiguity, "pci-msi". Also, are there any plans
> when to make have default be on rather than off?

Renaming sounds sensible. I admit I forgot it was turned off by default. I
guess at this point we should turn it on by default immediately after 3.3
has branched.

 -- Keir



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

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