xen-devel
Re: [Xen-devel] [PATCH] VT-d: improve RMRR validity checking
To: |
Alex Williamson <alex.williamson@xxxxxx> |
Subject: |
Re: [Xen-devel] [PATCH] VT-d: improve RMRR validity checking |
From: |
Weidong Han <weidong.han@xxxxxxxxx> |
Date: |
Wed, 10 Mar 2010 12:25:59 +0800 |
Cc: |
"xen-devel@xxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxx>, Noboru Iwamatsu <n_iwamatsu@xxxxxxxxxxxxxx>, "Cihula, Joseph" <joseph.cihula@xxxxxxxxx>, "Kay, Allen M" <allen.m.kay@xxxxxxxxx>, "linux@xxxxxxxxxxxxxx" <linux@xxxxxxxxxxxxxx>, "keir.fraser@xxxxxxxxxxxxx" <keir.fraser@xxxxxxxxxxxxx> |
Delivery-date: |
Tue, 09 Mar 2010 20:26:37 -0800 |
Envelope-to: |
www-data@xxxxxxxxxxxxxxxxxxx |
In-reply-to: |
<1268192253.3015.37.camel@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> |
References: |
<C77E162B.6FE6%keir.fraser@xxxxxxxxxxxxx> <4B59098B.6000108@xxxxxxxxx> <4B590FA4.4000008@xxxxxxxxxxxxxx> <4B59132B.40607@xxxxxxxxx> <4B59188C.50901@xxxxxxxxxxxxxx> <4B59660F.4000909@xxxxxxxxx> <7162ab21003091339i4adb8669safd5e074607386a2@xxxxxxxxxxxxxx> <4B9706B3.3050903@xxxxxxxxx> <1268191101.3015.30.camel@xxxxxxxxxx> <4B9711CF.7030908@xxxxxxxxx> <1268192253.3015.37.camel@xxxxxxxxxx> |
Sender: |
xen-devel-bounces@xxxxxxxxxxxxxxxxxxx |
User-agent: |
Thunderbird 2.0.0.21 (Windows/20090302) |
Alex Williamson wrote:
On Wed, 2010-03-10 at 11:28 +0800, Weidong Han wrote:
Alex Williamson wrote:
On Wed, 2010-03-10 at 10:40 +0800, Weidong Han wrote:
Alex Williamson wrote:
I have a system with what I consider to be a valid DRHD that's getting
tripped up on this patch. The problem is that the DRHD includes an
IOAPIC scope, where the IOAPIC is not materialized on the PCI bus. I
think Xen is being overzealous in it's validity checking and that this
is a valid configuration. What do others think? Are IOAPICs a
special case that we can allow to be non-existent on the PCI bus?
Yes, IOAPIC can be not pci-discoverable. IOAPICs are only reported in
the "Include_all" DRHD, and our patch won't check if the device is
pci-discoverable or not for the "Include_all" DRHD. So I think the patch
is no problem unless IOAPIC is not included in the "Include_all" DRHD.
Can you post your boot logs?
Weidong,
That's a very subtle restriction, and I'm not sure how it works in
practice. If I have a multi-IOH system, each with VT-d hardware, each
supporting interrupt remapping, each with one or more IOAPICs below
them, how can interrupt remapping work if we can only associate an
IOAPIC with the "include all" DRHD? I'm confused. Thanks,
Each IOH will have one "include all" DRHD which reports IOAPICs for each
IOH.
Wouldn't that imply multiple PCI segments? The configuration I'm
looking at has multiple IOHs, all on the same PCI segment. By my
reading of the spec, we're only allowed to declare INCLUDE_PCI_ALL for
one DRHD within the segment. Am I incorrect? Thanks,
Currently multiple PCI segments are not supported in Xen yet. So you
encounter issue on multiple PCI segment system. We will support it after
xen 4.0.
Regards,
Weidong
Alex
_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel
|
<Prev in Thread] |
Current Thread |
[Next in Thread>
|
- Re: [Xen-devel] [PATCH] VT-d: improve RMRR validity checking, (continued)
Re: [Xen-devel] [PATCH] VT-d: improve RMRR validity checking, Weidong Han
- Re: [Xen-devel] [PATCH] VT-d: improve RMRR validity checking, Alex Williamson
- Re: [Xen-devel] [PATCH] VT-d: improve RMRR validity checking, Weidong Han
- Re: [Xen-devel] [PATCH] VT-d: improve RMRR validity checking, Alex Williamson
- Re: [Xen-devel] [PATCH] VT-d: improve RMRR validity checking,
Weidong Han <=
- Re: [Xen-devel] [PATCH] VT-d: improve RMRR validity checking, Alex Williamson
- Re: [Xen-devel] [PATCH] VT-d: improve RMRR validity checking, Weidong Han
- Re: [Xen-devel] [PATCH] VT-d: improve RMRR validity checking, Alex Williamson
- Re: [Xen-devel] [PATCH] VT-d: improve RMRR validity checking, Alex Williamson
- Re: [Xen-devel] [PATCH] VT-d: improve RMRR validity checking, Weidong Han
- Re: [Xen-devel] [PATCH] VT-d: improve RMRR validity checking, Alex Williamson
- Re: [Xen-devel] [PATCH] VT-d: improve RMRR validity checking, Weidong Han
- Re: [Xen-devel] [PATCH] VT-d: improve RMRR validity checking, Alex Williamson
- Re: [Xen-devel] [PATCH] VT-d: improve RMRR validity checking, Weidong Han
|
|
|