Re: [PATCH v2] parisc: fix a possible DMA corruption

[Helge Deller <deller@gmx.de>]

On 7/27/24 20:22, Mikulas Patocka wrote:
> ARCH_DMA_MINALIGN was defined as 16 - this is too small - it may be
> possible that two unrelated 16-byte allocations share a cache line. If
> one of these allocations is written using DMA and the other is written
> using cached write, the value that was written with DMA may be
> corrupted.
>
> This commit changes ARCH_DMA_MINALIGN to be 128 on PA20 and 32 on PA1.1 -
> that's the largest possible cache line size.
>
> As different parisc microarchitectures have different cache line size, we
> define arch_slab_minalign(), cache_line_size() and
> dma_get_cache_alignment() so that the kernel may tune slab cache
> parameters dynamically, based on the detected cache line size.
>
> Signed-off-by: Mikulas Patocka <mpatocka@redhat.com>
> Cc: stable@vger.kernel.org

applied.

Thanks!
Helge

>
> ---
>   arch/parisc/Kconfig             |    1 +
>   arch/parisc/include/asm/cache.h |   11 ++++++++++-
>   2 files changed, 11 insertions(+), 1 deletion(-)
>
> Index: linux-6.10/arch/parisc/include/asm/cache.h
> ===================================================================
> --- linux-6.10.orig/arch/parisc/include/asm/cache.h	2023-09-18 11:33:40.000000000 +0200
> +++ linux-6.10/arch/parisc/include/asm/cache.h	2024-07-27 19:28:18.000000000 +0200
> @@ -20,7 +20,16 @@
>
>   #define SMP_CACHE_BYTES L1_CACHE_BYTES
>
> -#define ARCH_DMA_MINALIGN	L1_CACHE_BYTES
> +#ifndef CONFIG_PA20
> +#define ARCH_DMA_MINALIGN	32
> +#else
> +#define ARCH_DMA_MINALIGN	128
> +#endif
> +#define ARCH_KMALLOC_MINALIGN	16	/* ldcw requires 16-byte alignment */
> +
> +#define arch_slab_minalign()	((unsigned)dcache_stride)
> +#define cache_line_size()	dcache_stride
> +#define dma_get_cache_alignment cache_line_size
>
>   #define __read_mostly __section(".data..read_mostly")
>
> Index: linux-6.10/arch/parisc/Kconfig
> ===================================================================
> --- linux-6.10.orig/arch/parisc/Kconfig	2024-07-23 20:35:34.000000000 +0200
> +++ linux-6.10/arch/parisc/Kconfig	2024-07-26 19:41:15.000000000 +0200
> @@ -20,6 +20,7 @@ config PARISC
>   	select ARCH_SUPPORTS_HUGETLBFS if PA20
>   	select ARCH_SUPPORTS_MEMORY_FAILURE
>   	select ARCH_STACKWALK
> +	select ARCH_HAS_CACHE_LINE_SIZE
>   	select ARCH_HAS_DEBUG_VM_PGTABLE
>   	select HAVE_RELIABLE_STACKTRACE
>   	select DMA_OPS
>