tree 23ace2424679117bc82f3933c945d599ecdcb710
parent ab86944556c32093d590cdfefebebc7d25f07aec
author Peter Zijlstra <peterz@infradead.org> 1554277190 +0200
committer Paul Lawrence <paullawrence@google.com> 1562775262 -0700

UPSTREAM: x86/uaccess, kasan: Fix KASAN vs SMAP

Upstream commit 57b78a62e7f23c4686fe54091cdc3d12e60d6513.

KASAN inserts extra code for every LOAD/STORE emitted by te compiler.
Much of this code is simple and safe to run with AC=1, however the
kasan_report() function, called on error, is most certainly not safe
to call with AC=1.

Therefore wrap kasan_report() in user_access_{save,restore}; which for
x86 SMAP, saves/restores EFLAGS and clears AC before calling the real
function.

Also ensure all the functions are without __fentry__ hook. The
function tracer is also not safe.

Signed-off-by: Peter Zijlstra (Intel) <peterz@infradead.org>
Cc: Borislav Petkov <bp@alien8.de>
Cc: Dmitry Vyukov <dvyukov@google.com>
Cc: Josh Poimboeuf <jpoimboe@redhat.com>
Cc: Linus Torvalds <torvalds@linux-foundation.org>
Cc: Peter Zijlstra <peterz@infradead.org>
Cc: Thomas Gleixner <tglx@linutronix.de>
Signed-off-by: Ingo Molnar <mingo@kernel.org>
Signed-off-by: Andrey Konovalov <andreyknvl@google.com>
Change-Id: Iad2644e0d40d2ba468c1442fd10c546c5b6dcee3
Bug: 128674696