Together FEC driver and parts of IP stack might end up trying to memcpy() small chunks of memory from uncached (that is Device memory) addresses that are not properly aligned, leading to data abort. To prevent such cases, add code to guard unaligned accesses and redirect them to byte-wise implementations which do not have the above problem. Signed-off-by: Andrey Smirnov <andrew.smirnov@xxxxxxxxx> --- arch/arm/lib64/string.c | 11 +++++++++-- 1 file changed, 9 insertions(+), 2 deletions(-) diff --git a/arch/arm/lib64/string.c b/arch/arm/lib64/string.c index cb2633152..1b04f4487 100644 --- a/arch/arm/lib64/string.c +++ b/arch/arm/lib64/string.c @@ -5,9 +5,16 @@ void *__arch_memset(void *dst, int c, __kernel_size_t size); void *__arch_memcpy(void * dest, const void *src, size_t count); +static bool properly_aligned(const void *ptr) +{ + const unsigned long address = (unsigned long)ptr; + + return get_cr() & CR_M && IS_ALIGNED(address, 16); +} + void *memset(void *dst, int c, __kernel_size_t size) { - if (likely(get_cr() & CR_M)) + if (likely(properly_aligned(dst))) return __arch_memset(dst, c, size); return __default_memset(dst, c, size); @@ -15,7 +22,7 @@ void *memset(void *dst, int c, __kernel_size_t size) void *memcpy(void * dest, const void *src, size_t count) { - if (likely(get_cr() & CR_M)) + if (likely(properly_aligned(dest) && properly_aligned(src))) return __arch_memcpy(dest, src, count); return __default_memcpy(dest, src, count); -- 2.17.1 _______________________________________________ barebox mailing list barebox@xxxxxxxxxxxxxxxxxxx http://lists.infradead.org/mailman/listinfo/barebox