[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-Id: <a883eadab5eac556a56065f088931bc3e6c0d052.1478523828.git.jslaby@suse.cz>
Date: Mon, 7 Nov 2016 14:04:20 +0100
From: Jiri Slaby <jslaby@...e.cz>
To: stable@...r.kernel.org
Cc: linux-kernel@...r.kernel.org, Furquan Shaikh <furquan@...gle.com>,
Enric Balletbo Serra <enric.balletbo@...labora.com>,
Kees Cook <keescook@...omium.org>, Jiri Slaby <jslaby@...e.cz>
Subject: [PATCH 3.12 13/72] pstore/ram: Use memcpy_toio instead of memcpy
From: Furquan Shaikh <furquan@...gle.com>
3.12-stable review patch. If anyone has any objections, please let me know.
===============
commit 7e75678d23167c2527e655658a8ef36a36c8b4d9 upstream.
persistent_ram_update uses vmap / iomap based on whether the buffer is in
memory region or reserved region. However, both map it as non-cacheable
memory. For armv8 specifically, non-cacheable mapping requests use a
memory type that has to be accessed aligned to the request size. memcpy()
doesn't guarantee that.
Signed-off-by: Furquan Shaikh <furquan@...gle.com>
Signed-off-by: Enric Balletbo Serra <enric.balletbo@...labora.com>
Reviewed-by: Aaron Durbin <adurbin@...omium.org>
Reviewed-by: Olof Johansson <olofj@...omium.org>
Tested-by: Furquan Shaikh <furquan@...omium.org>
Signed-off-by: Kees Cook <keescook@...omium.org>
Signed-off-by: Jiri Slaby <jslaby@...e.cz>
---
fs/pstore/ram_core.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/fs/pstore/ram_core.c b/fs/pstore/ram_core.c
index ee3c6ec5348f..eb42483dbb05 100644
--- a/fs/pstore/ram_core.c
+++ b/fs/pstore/ram_core.c
@@ -263,7 +263,7 @@ static void notrace persistent_ram_update(struct persistent_ram_zone *prz,
const void *s, unsigned int start, unsigned int count)
{
struct persistent_ram_buffer *buffer = prz->buffer;
- memcpy(buffer->data + start, s, count);
+ memcpy_toio(buffer->data + start, s, count);
persistent_ram_update_ecc(prz, start, count);
}
--
2.10.2
Powered by blists - more mailing lists