On 08/22/2016 06:22 AM, Tom Rini wrote:
This series moves the CONFIG_SYS_CACHELINE_SIZE. First, in nearly all cases we are mirroring the values used by the Linux Kernel here. Also, so long as (and in this case, it is true) we implement flushes in hunks that are no larger than the smallest implementation (and given that we mirror the Linux Kernel, again we are fine) it is OK to align higher. The biggest changes here are that we always use 64 bytes for CPU_V7 even if for example the underlying core is only 32 bytes (this mirrors Linux). Second, we say ARM64 uses 64 bytes not 128 (as found in the Linux Kernel) as we do not need multi-platform support (to this degree) and only the Cavium ThunderX 88xx series has a use for such large alignment.
Tested-by: Stephen Warren swarren@nvidia.com
This has the potential to cause a few more cache warnings due to the slightly stricter/larger alignment checking on older ARMv7s with smaller cache lines. However, it works fine on all the Tegra systems my automated tests run on right now, which does include one Cortex A9 with 32-byte cache line, and doesn't seem to actually trigger any additional unaligned buffer warnings.