[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Xen-devel] [PATCH v2 09/11] IOMMU: propagate IOMMU Device-TLB flush error up to IOMMU suspending
- To: Jan Beulich <JBeulich@xxxxxxxx>, Quan Xu <quan.xu@xxxxxxxxx>
- From: Julien Grall <julien.grall@xxxxxxx>
- Date: Mon, 25 Apr 2016 14:58:02 +0100
- Cc: Kevin Tian <kevin.tian@xxxxxxxxx>, Feng Wu <feng.wu@xxxxxxxxx>, Liu Jinsong <jinsong.liu@xxxxxxxxxxxxxxx>, dario.faggioli@xxxxxxxxxx, xen-devel@xxxxxxxxxxxxx, Stefano Stabellini <stefano.stabellini@xxxxxxxxxx>, Suravee Suthikulpanit <suravee.suthikulpanit@xxxxxxx>, Andrew Cooper <andrew.cooper3@xxxxxxxxxx>, Keir Fraser <keir@xxxxxxx>
- Delivery-date: Mon, 25 Apr 2016 13:58:16 +0000
- List-id: Xen developer discussion <xen-devel.lists.xen.org>
Hi Jan,
On 25/04/16 12:52, Jan Beulich wrote:
On 18.04.16 at 16:00, <quan.xu@xxxxxxxxx> wrote:
--- a/xen/drivers/passthrough/arm/smmu.c
+++ b/xen/drivers/passthrough/arm/smmu.c
@@ -2540,7 +2540,7 @@ static int force_stage = 2;
*/
static u32 platform_features = ARM_SMMU_FEAT_COHERENT_WALK;
-static void arm_smmu_iotlb_flush_all(struct domain *d)
+static int arm_smmu_iotlb_flush_all(struct domain *d)
{
struct arm_smmu_xen_domain *smmu_domain =
domain_hvm_iommu(d)->arch.priv;
struct iommu_domain *cfg;
@@ -2557,13 +2557,15 @@ static void arm_smmu_iotlb_flush_all(struct domain *d)
arm_smmu_tlb_inv_context(cfg->priv);
}
spin_unlock(&smmu_domain->lock);
+
+ return 0;
}
Even if indentation looks inconsistent in this file, please make your
addition match surrounding code.
The file smmu.c was imported from Linux and supposed to use only Linux
coding style. This is in order to help porting fixes.
Regards,
--
Julien Grall
_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel
|
