Re: [PATCH v8 1/5] iommu/arm-smmu: Destroy domain context in failure path

2018-03-07 Thread Vivek Gautam
On Wed, Mar 7, 2018 at 5:50 PM, Robin Murphy wrote: > On 02/03/18 10:10, Vivek Gautam wrote: >> >> If we fail after initializing domain_context, we should destroy >> the context to free up resources. > > > Have another think about why the "problem" this patch caters for

Re: [PATCH v8 1/5] iommu/arm-smmu: Destroy domain context in failure path

2018-03-07 Thread Robin Murphy
On 02/03/18 10:10, Vivek Gautam wrote: If we fail after initializing domain_context, we should destroy the context to free up resources. Have another think about why the "problem" this patch caters for cannot ever happen (hint: consider how domain->smmu is used in

[PATCH v8 1/5] iommu/arm-smmu: Destroy domain context in failure path

2018-03-02 Thread Vivek Gautam
If we fail after initializing domain_context, we should destroy the context to free up resources. Signed-off-by: Vivek Gautam --- * New patch added in this series. drivers/iommu/arm-smmu.c | 7 ++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git