arm64: lib: improve copy_page to deal with 128 bytes at a time
authorWill Deacon <will.deacon@arm.com>
Tue, 2 Feb 2016 12:46:25 +0000 (12:46 +0000)
committerAlex Shi <alex.shi@linaro.org>
Wed, 11 May 2016 14:19:18 +0000 (22:19 +0800)
commit93c384820cf3c1db51073e746980866d2bce8af9
tree5227088e7e2e468a982d63e6bbe92c087901ed17
parent742e490adaa444e9657528ef38dde35f2916c793
arm64: lib: improve copy_page to deal with 128 bytes at a time

We want to avoid lots of different copy_page implementations, settling
for something that is "good enough" everywhere and hopefully easy to
understand and maintain whilst we're at it.

This patch reworks our copy_page implementation based on discussions
with Cavium on the list and benchmarking on Cortex-A processors so that:

  - The loop is unrolled to copy 128 bytes per iteration

  - The reads are offset so that we read from the next 128-byte block
    in the same iteration that we store the previous block

  - Explicit prefetch instructions are removed for now, since they hurt
    performance on CPUs with hardware prefetching

  - The loop exit condition is calculated at the start of the loop

Signed-off-by: Will Deacon <will.deacon@arm.com>
Tested-by: Andrew Pinski <apinski@cavium.com>
Signed-off-by: Catalin Marinas <catalin.marinas@arm.com>
(cherry picked from commit 223e23e8aa26b0bb62c597637e77295e14f6a62c)
Signed-off-by: Alex Shi <alex.shi@linaro.org>
arch/arm64/lib/copy_page.S