[PATCH v2 0/5] xen/arm: smmuv1: Fix stream match conflict issue

Rahul Singh posted 5 patches 3 years, 1 month ago
Test gitlab-ci passed
Patches applied successfully (tree, apply log)
git fetch https://gitlab.com/xen-project/patchew/xen tags/patchew/cover.1616428314.git.rahul.singh@arm.com
xen/drivers/passthrough/arm/smmu.c | 423 ++++++++++++++++++-----------
1 file changed, 262 insertions(+), 161 deletions(-)
[PATCH v2 0/5] xen/arm: smmuv1: Fix stream match conflict issue
Posted by Rahul Singh 3 years, 1 month ago
This patch is the work to fix the stream match conflict issue when two devices
have the same stream-id.

Approach taken is to merge the below commit from Linux driver to fix the
issue.

1. "iommu/arm-smmu: Handle stream IDs more dynamically"
   commit 21174240e4f4439bb8ed6c116cdbdc03eba2126e
2. "iommu/arm-smmu: Consolidate stream map entry state"
   commit 1f3d5ca43019bff1105838712d55be087d93c0da
3. "iommu/arm-smmu: Keep track of S2CR state"
   commit 8e8b203eabd8b9e96d02d6339e4abce3e5a7ea4b
4. "iommu/arm-smmu: Add a stream map entry iterator"
   commit d3097e39302083d58922a3d1032d7d59a63d263d
5. "iommu/arm-smmu: Intelligent SMR allocation"
   commit 588888a7399db352d2b1a41c9d5b3bf0fd482390

Changes since v2:
 - Updated commit message to show the original author of the patch.
 - Added  Acked-by: Stefano Stabellini <sstabellini@kernel.org>
 - Addded Reviewed-by: Bertrand Marquis <bertrand.marquis@arm.com>
  
Rahul Singh (5):
  xen/arm: smmuv1: Handle stream IDs more dynamically
  xen/arm: smmuv1: Consolidate stream map entry state
  xen/arm: smmuv1: Keep track of S2CR state
  xen/arm: smmuv1: Add a stream map entry iterator
  xen/arm: smmuv1: Intelligent SMR allocation

 xen/drivers/passthrough/arm/smmu.c | 423 ++++++++++++++++++-----------
 1 file changed, 262 insertions(+), 161 deletions(-)

-- 
2.17.1


Re: [PATCH v2 0/5] xen/arm: smmuv1: Fix stream match conflict issue
Posted by Julien Grall 3 years, 1 month ago
Hi Rahul,

On 22/03/2021 16:11, Rahul Singh wrote:
> This patch is the work to fix the stream match conflict issue when two devices
> have the same stream-id.
> 
> Approach taken is to merge the below commit from Linux driver to fix the
> issue.
> 
> 1. "iommu/arm-smmu: Handle stream IDs more dynamically"
>     commit 21174240e4f4439bb8ed6c116cdbdc03eba2126e
> 2. "iommu/arm-smmu: Consolidate stream map entry state"
>     commit 1f3d5ca43019bff1105838712d55be087d93c0da
> 3. "iommu/arm-smmu: Keep track of S2CR state"
>     commit 8e8b203eabd8b9e96d02d6339e4abce3e5a7ea4b
> 4. "iommu/arm-smmu: Add a stream map entry iterator"
>     commit d3097e39302083d58922a3d1032d7d59a63d263d
> 5. "iommu/arm-smmu: Intelligent SMR allocation"
>     commit 588888a7399db352d2b1a41c9d5b3bf0fd482390
> 
> Changes since v2:
>   - Updated commit message to show the original author of the patch.
>   - Added  Acked-by: Stefano Stabellini <sstabellini@kernel.org>
>   - Addded Reviewed-by: Bertrand Marquis <bertrand.marquis@arm.com>
>    
> Rahul Singh (5):
>    xen/arm: smmuv1: Handle stream IDs more dynamically
>    xen/arm: smmuv1: Consolidate stream map entry state
>    xen/arm: smmuv1: Keep track of S2CR state
>    xen/arm: smmuv1: Add a stream map entry iterator
>    xen/arm: smmuv1: Intelligent SMR allocation

I went through the commit messages and they look good to me:

Acked-by: Julien Grall <jgrall@amazon.com>

I have committed the series to my for-next/4.16 branch. They will be 
merged soon in staging.

Cheers,

> 
>   xen/drivers/passthrough/arm/smmu.c | 423 ++++++++++++++++++-----------
>   1 file changed, 262 insertions(+), 161 deletions(-)
> 

-- 
Julien Grall

Re: [PATCH v2 0/5] xen/arm: smmuv1: Fix stream match conflict issue
Posted by Rahul Singh 3 years, 1 month ago
Hi Julien,

> On 24 Mar 2021, at 8:48 pm, Julien Grall <julien@xen.org> wrote:
> 
> Hi Rahul,
> 
> On 22/03/2021 16:11, Rahul Singh wrote:
>> This patch is the work to fix the stream match conflict issue when two devices
>> have the same stream-id.
>> Approach taken is to merge the below commit from Linux driver to fix the
>> issue.
>> 1. "iommu/arm-smmu: Handle stream IDs more dynamically"
>>    commit 21174240e4f4439bb8ed6c116cdbdc03eba2126e
>> 2. "iommu/arm-smmu: Consolidate stream map entry state"
>>    commit 1f3d5ca43019bff1105838712d55be087d93c0da
>> 3. "iommu/arm-smmu: Keep track of S2CR state"
>>    commit 8e8b203eabd8b9e96d02d6339e4abce3e5a7ea4b
>> 4. "iommu/arm-smmu: Add a stream map entry iterator"
>>    commit d3097e39302083d58922a3d1032d7d59a63d263d
>> 5. "iommu/arm-smmu: Intelligent SMR allocation"
>>    commit 588888a7399db352d2b1a41c9d5b3bf0fd482390
>> Changes since v2:
>>  - Updated commit message to show the original author of the patch.
>>  - Added  Acked-by: Stefano Stabellini <sstabellini@kernel.org>
>>  - Addded Reviewed-by: Bertrand Marquis <bertrand.marquis@arm.com>
>>   Rahul Singh (5):
>>   xen/arm: smmuv1: Handle stream IDs more dynamically
>>   xen/arm: smmuv1: Consolidate stream map entry state
>>   xen/arm: smmuv1: Keep track of S2CR state
>>   xen/arm: smmuv1: Add a stream map entry iterator
>>   xen/arm: smmuv1: Intelligent SMR allocation
> 
> I went through the commit messages and they look good to me:
> 
> Acked-by: Julien Grall <jgrall@amazon.com>
> 
> I have committed the series to my for-next/4.16 branch. They will be merged soon in staging.

Thanks.

Regards,
Rahul
> 
> Cheers,
> 
>>  xen/drivers/passthrough/arm/smmu.c | 423 ++++++++++++++++++-----------
>>  1 file changed, 262 insertions(+), 161 deletions(-)
> 
> -- 
> Julien Grall