From mboxrd@z Thu Jan 1 00:00:00 1970 From: Julien Grall Subject: Re: [RFC][PATCH 07/13] xen/passthrough: extend hypercall to support rdm reservation policy Date: Mon, 11 May 2015 10:51:29 +0100 Message-ID: <55507BA1.7000008@citrix.com> References: <1428657724-3498-1-git-send-email-tiejun.chen@intel.com> <1428657724-3498-8-git-send-email-tiejun.chen@intel.com> <554CDF5E.8050806@citrix.com> <55506B8C.2070903@intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <55506B8C.2070903@intel.com> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xen.org Errors-To: xen-devel-bounces@lists.xen.org To: "Chen, Tiejun" , Julien Grall , JBeulich@suse.com, tim@xen.org, konrad.wilk@oracle.com, andrew.cooper3@citrix.com, kevin.tian@intel.com, yang.z.zhang@intel.com, ian.campbell@citrix.com, wei.liu2@citrix.com, Ian.Jackson@eu.citrix.com, stefano.stabellini@citrix.com Cc: xen-devel@lists.xen.org List-Id: xen-devel@lists.xenproject.org Hi, On 11/05/15 09:42, Chen, Tiejun wrote: > diff --git a/xen/drivers/passthrough/arm/smmu.c > b/xen/drivers/passthrough/arm/smmu.c > index 8a9b58b..a3e6383 100644 > --- a/xen/drivers/passthrough/arm/smmu.c > +++ b/xen/drivers/passthrough/arm/smmu.c > @@ -2599,7 +2599,7 @@ static void arm_smmu_destroy_iommu_domain(struct > iommu_domain *domain) > } > > static int arm_smmu_assign_dev(struct domain *d, u8 devfn, > - struct device *dev) > + struct device *dev, u32 flag) > { > struct iommu_domain *domain; > struct arm_smmu_xen_domain *xen_domain; > diff --git a/xen/drivers/passthrough/device_tree.c > b/xen/drivers/passthrough/device_tree.c > index 377d41d..97e7fc5 100644 > --- a/xen/drivers/passthrough/device_tree.c > +++ b/xen/drivers/passthrough/device_tree.c > @@ -41,7 +41,8 @@ int iommu_assign_dt_device(struct domain *d, struct > dt_device_node *dev) > if ( !list_empty(&dev->domain_list) ) > goto fail; > > - rc = hd->platform_ops->assign_device(d, 0, dt_to_dev(dev)); > + rc = hd->platform_ops->assign_device(d, 0, dt_to_dev(dev), > + XEN_DOMCTL_PCIDEV_RDM_TRY); On ARM we can passthrough 2 different types of device: PCI device and platform device described in the device tree (it's a tree representation of the hardware). This assign_device callback deals with the latter. So from the name the value doesn't look right. Regards, -- Julien Grall