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

[Xen-devel] RE: Resend: RE: enable_ats_device() call site

To: "Allen M Kay" <allen.m.kay@xxxxxxxxx>
Subject: [Xen-devel] RE: Resend: RE: enable_ats_device() call site
From: "Jan Beulich" <JBeulich@xxxxxxxx>
Date: Fri, 21 Oct 2011 08:52:35 +0100
Cc: Kevin Tian <kevin.tian@xxxxxxxxx>, Susie Li <susie.li@xxxxxxxxx>, Haitao Shan <haitao.shan@xxxxxxxxx>, "xen-devel@xxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxx>, Donald D Dugger <donald.d.dugger@xxxxxxxxx>
Delivery-date: Fri, 21 Oct 2011 00:53:32 -0700
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to: <987664A83D2D224EAE907B061CE93D5301F1E5842B@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/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: <4E4E48360200007800051F65@xxxxxxxxxxxxxxxxxxxx> <987664A83D2D224EAE907B061CE93D5301EE709B5E@xxxxxxxxxxxxxxxxxxxxxxxxxxxx> <4E8EC8A80200007800059E3B@xxxxxxxxxxxxxxxxxxxx> <987664A83D2D224EAE907B061CE93D5301EE70A446@xxxxxxxxxxxxxxxxxxxxxxxxxxxx> <4E9458A5020000780005AB99@xxxxxxxxxxxxxxxxxxxx> <987664A83D2D224EAE907B061CE93D5301F19DB855@xxxxxxxxxxxxxxxxxxxxxxxxxxxx> <4E9E8EA1020000780005C155@xxxxxxxxxxxxxxxxxxxx> <987664A83D2D224EAE907B061CE93D5301F1D86476@xxxxxxxxxxxxxxxxxxxxxxxxxxxx> <4E9FE8B6020000780005C661@xxxxxxxxxxxxxxxxxxxx> <987664A83D2D224EAE907B061CE93D5301F1E5842B@xxxxxxxxxxxxxxxxxxxxxxxxxxxx>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
>>> On 21.10.11 at 03:59, "Kay, Allen M" <allen.m.kay@xxxxxxxxx> wrote:
>>  So why does the capability to list individual devices then exist? And why 
> does it matter for DRHDs, but not for ATSRs?
> 
> The difference is ATSR only is meant to communicate PCIe root ports' ATS 
> capability.  If the root port is capable, then downstream endpoints can 
> enable ATS device translation cache.
> 
> acpi_find_matched_drhd_unit() is used to find out which VT-d hardware is 
> servicing the endpoint device.  Given drhd lists either the actually PCI 
> endpoints or include_all, we have to match the actual BDF of the device 
> passed in with devices we recorded for that VT-d HW.
> 
> acpi_find_matched_astr_unit() is used to find out if the endpoint device is 
> under a ATS capable PCIe root port or not.  ASTR information is remembered as 
> secondary and subsidiary bus ranges.  All we have to do is the match the 
> device's bus number with a root ports bus range.  Matching the actual device 
> in this case, will only match the root port device itself, this is what we 
> recorded in acpi_parse_dev_scope(), which should not happen since we don't  
> assign a root port to a guest.  Even if we do, checking for ATS capability is 
> meaningless since root port will not have device translation cache.

Okay, so I'll remove that part then and re-submit both patches.

Jan


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