[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: "Xu, Quan" <quan.xu@xxxxxxxxx>, Julien Grall <julien.grall@xxxxxxx>, Jan Beulich <JBeulich@xxxxxxxx>
- From: Julien Grall <julien.grall@xxxxxxx>
- Date: Tue, 26 Apr 2016 10:40:58 +0100
- Cc: "Tian, Kevin" <kevin.tian@xxxxxxxxx>, "Wu, Feng" <feng.wu@xxxxxxxxx>, Liu Jinsong <jinsong.liu@xxxxxxxxxxxxxxx>, "dario.faggioli@xxxxxxxxxx" <dario.faggioli@xxxxxxxxxx>, "xen-devel@xxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxx>, Stefano Stabellini <stefano.stabellini@xxxxxxxxxx>, Suravee Suthikulpanit <suravee.suthikulpanit@xxxxxxx>, Andrew Cooper <andrew.cooper3@xxxxxxxxxx>, Keir Fraser <keir@xxxxxxx>
- Delivery-date: Tue, 26 Apr 2016 09:41:06 +0000
- List-id: Xen developer discussion <xen-devel.lists.xen.org>
Hi Quan,
On 25/04/2016 15:50, Xu, Quan wrote:
On April 25, 2016 9:58pm, <julien.grall@xxxxxxx> wrote:
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.
Julien, A 'tab' ?
Yes, Linux is using hard tabulation for the indentation.
Regards,
--
Julien Grall
_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel
|
