• Thomas Betker's avatar
    ARM: zynq: Set bit 22 in PL310 AuxCtrl register (6395/1) · 6632d4fd
    Thomas Betker authored
    This patch is based on the
    commit 1a8e41cd ("ARM: 6395/1: VExpress: Set bit 22 in the PL310
    (cache controller) AuxCtlr register")
    
    Clearing bit 22 in the PL310 Auxiliary Control register (shared
    attribute override enable) has the side effect of transforming Normal
    Shared Non-cacheable reads into Cacheable no-allocate reads.
    
    Coherent DMA buffers in Linux always have a cacheable alias via the
    kernel linear mapping and the processor can speculatively load cache
    lines into the PL310 controller. With bit 22 cleared, Non-cacheable
    reads would unexpectedly hit such cache lines leading to buffer
    corruption.
    
    For Zynq, this fix avoids memory inconsistencies between Gigabit
    Ethernet controller (GEM) and CPU when DMA_CMA is disabled.
    Suggested-by: default avatarPunnaiah Choudary Kalluri <punnaia@xilinx.com>
    Signed-off-by: default avatarThomas Betker <thomas.betker@rohde-schwarz.com>
    Signed-off-by: default avatarMichal Simek <michal.simek@xilinx.com>
    6632d4fd
common.c 5.13 KB