mbox series

[v4,0/3] iommu/arm-smmu-v3: permit users to disable msi polling

Message ID 20200818233827.21452-1-song.bao.hua@hisilicon.com (mailing list archive)
Headers show
Series iommu/arm-smmu-v3: permit users to disable msi polling | expand

Message

Song Bao Hua (Barry Song) Aug. 18, 2020, 11:38 p.m. UTC
patch 1/3 and patch 2/3 are the preparation of patch 3/3 which permits users
to disable MSI-based polling by cmd line.

-v4:
  with respect to Robin's comments
  * cleanup the code of the existing module parameter disable_bypass
  * add ARM_SMMU_OPT_MSIPOLL flag. on the other hand, we only need to check
    a bit in options rather than two bits in features

Barry Song (3):
  iommu/arm-smmu-v3: replace symbolic permissions by octal permissions
    for module parameter
  iommu/arm-smmu-v3: replace module_param_named by module_param for
    disable_bypass
  iommu/arm-smmu-v3: permit users to disable msi polling

 drivers/iommu/arm/arm-smmu-v3/arm-smmu-v3.c | 19 +++++++++++++------
 1 file changed, 13 insertions(+), 6 deletions(-)

Comments

Robin Murphy Aug. 20, 2020, 1:10 p.m. UTC | #1
On 2020-08-19 00:38, Barry Song wrote:
> patch 1/3 and patch 2/3 are the preparation of patch 3/3 which permits users
> to disable MSI-based polling by cmd line.
> 
> -v4:
>    with respect to Robin's comments
>    * cleanup the code of the existing module parameter disable_bypass
>    * add ARM_SMMU_OPT_MSIPOLL flag. on the other hand, we only need to check
>      a bit in options rather than two bits in features

Thanks Barry - for all 3 patches,

Reviewed-by: Robin Murphy <robin.murphy@arm.com>

I'd be inclined to squash #2 into #1, but I'll leave that up to Will.

Cheers,
Robin.

> 
> Barry Song (3):
>    iommu/arm-smmu-v3: replace symbolic permissions by octal permissions
>      for module parameter
>    iommu/arm-smmu-v3: replace module_param_named by module_param for
>      disable_bypass
>    iommu/arm-smmu-v3: permit users to disable msi polling
> 
>   drivers/iommu/arm/arm-smmu-v3/arm-smmu-v3.c | 19 +++++++++++++------
>   1 file changed, 13 insertions(+), 6 deletions(-)
>