[PATCH v8 0/2] Fix dma mapping when the cache is coherent

Gregory CLEMENT gregory.clement at free-electrons.com
Wed Apr 20 16:02:29 PDT 2016


Hi Marcin,
 
 On sam., avril 16 2016, Marcin Wojtas <mw at semihalf.com> wrote:

> Hi Gregory,
>
> You can re-add my tested-by. Hang is no longer a problem with your
> patch during my test of neta.

The patch was already submitted in the Russell King's Patch Tracking
System, so I can't add the tested-by on them, but I am sure it will help
these pacthes to be merged.

Thanks,

Gregory

>
> Best regards,
> Marcin
>
> 2016-04-12 17:31 GMT+02:00 Gregory CLEMENT <gregory.clement at free-electrons.com>:
>> Hi,
>>
>> These two patches fixes the dma mapping functions when the system is
>> cache coherent. The first one allows to fix an issue we have on Armada
>> 375/38x with the PL310 that's why it is tagged for stable too.
>>
>> Thanks,
>>
>> Gregory
>>
>> PS: the mailing list was missing in the recipient list of the v7. It
>> was a mistake and I realized it only now. If needed I can repost it
>> with the few exchange we had with Russell King.
>>
>>
>> Changelog
>> v7 -> v8:
>>  - Use a flag instead of a boolean for checking the coherency. It
>>    improve the readability of the code. Suggested by Russell King.
>>  - Consider that when coherency is set, it's for both L1 and L2 caches
>>  - Use the arm_dma_alloc_args struct to pass the coherency status when
>>    possible. Suggested by Rabin Vincent
>>  - Remove the Tested-by flag from Marcin because of the modifications
>>    around the L1 cache management in this series.
>>
>> v6 -> v7:
>>  - Renamed is coherent by l2_coherent as suggested by Russell
>>
>> v5 -> v6:
>>  - Rebased on v4.6-rc1
>>
>> v4 -> v5
>>  - Keep the dmac_* function outside the !is_coherent case.
>>
>> v3 -> v4:
>>  - Rebased on v4.3-rc1
>>  - Fix conflict with commit "21caf3a765b0 ARM: 8398/1: arm DMA: Fix
>>    allocation from CMA for coherent DMA"
>>
>> v2 -> v3:
>>
>>  - Fix comments in patch 1 as suggested by Catalin.
>>  - Fix build issues in patch 2 (by using the multi_v7_defconfig +
>>    CONFIG_ROCKCHIP_IOMMU).
>>  - Add the arm_coherent_iommu_mmap_attrs function.
>>
>>
>> Gregory CLEMENT (2):
>>   ARM: dma-mapping: Don't use outer_flush_range when the L2C is coherent
>>   ARM: dma-mapping: Fix the coherent case when iommu is used
>>
>>  arch/arm/mm/dma-mapping.c | 144 +++++++++++++++++++++++++++++++++-------------
>>  1 file changed, 105 insertions(+), 39 deletions(-)
>>
>> --
>> 2.5.0
>>

-- 
Gregory Clement, Free Electrons
Kernel, drivers, real-time and embedded Linux
development, consulting, training and support.
http://free-electrons.com



More information about the linux-arm-kernel mailing list