lkml.org 
[lkml]   [2023]   [Mar]   [29]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: [PATCH 08/21] riscv: dma-mapping: only invalidate after DMA, not flush
    On Mon, Mar 27, 2023 at 02:13:04PM +0200, Arnd Bergmann wrote:
    > From: Arnd Bergmann <arnd@arndb.de>
    >
    > No other architecture intentionally writes back dirty cache lines into
    > a buffer that a device has just finished writing into. If the cache is
    > clean, this has no effect at all, but

    > if a cacheline in the buffer has
    > actually been written by the CPU, there is a drive bug that is likely
    > made worse by overwriting that buffer.

    So does this need a
    Fixes: 1631ba1259d6 ("riscv: Add support for non-coherent devices using zicbom extension")
    then, even if the cacheline really should not have been touched by the
    CPU?
    Also, minor typo, s/drive/driver/.

    In the thread we had that sparked this, I went digging for the source of
    the flushes, and it came from a review comment:
    https://lore.kernel.org/linux-riscv/342e3c12-ebb0-badf-7d4c-c444a2b842b2@sholland.org/
    But *surely* if no other arch needs to do that, then we are safe to also
    not do it... Your logic seems right by me at least, especially given the
    lack of flushes elsewhere.
    Reviewed-by: Conor Dooley <conor.dooley@microchip.com>

    Cheers,
    Conor.

    > Signed-off-by: Arnd Bergmann <arnd@arndb.de>
    > ---
    > arch/riscv/mm/dma-noncoherent.c | 2 +-
    > 1 file changed, 1 insertion(+), 1 deletion(-)
    >
    > diff --git a/arch/riscv/mm/dma-noncoherent.c b/arch/riscv/mm/dma-noncoherent.c
    > index d919efab6eba..640f4c496d26 100644
    > --- a/arch/riscv/mm/dma-noncoherent.c
    > +++ b/arch/riscv/mm/dma-noncoherent.c
    > @@ -42,7 +42,7 @@ void arch_sync_dma_for_cpu(phys_addr_t paddr, size_t size,
    > break;
    > case DMA_FROM_DEVICE:
    > case DMA_BIDIRECTIONAL:
    > - ALT_CMO_OP(flush, vaddr, size, riscv_cbom_block_size);
    > + ALT_CMO_OP(inval, vaddr, size, riscv_cbom_block_size);
    > break;
    > default:
    > break;
    > --
    > 2.39.2
    >
    [unhandled content-type:application/pgp-signature]
    \
     
     \ /
      Last update: 2023-03-29 22:48    [W:4.713 / U:0.016 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site