2020-07-21 10:31:24

by Marco Elver

Subject: [PATCH 2/8] objtool, kcsan: Add __tsan_read_write to uaccess whitelist

Adds the new __tsan_read_write compound instrumentation to objtool's
uaccess whitelist.

Signed-off-by: Marco Elver <[email protected]>
---
tools/objtool/check.c | 5 +++++
1 file changed, 5 insertions(+)

diff --git a/tools/objtool/check.c b/tools/objtool/check.c
index 63d8b630c67a..38d82e705c93 100644
--- a/tools/objtool/check.c
+++ b/tools/objtool/check.c
@@ -528,6 +528,11 @@ static const char *uaccess_safe_builtin[] = {
"__tsan_write4",
"__tsan_write8",
"__tsan_write16",
+ "__tsan_read_write1",
+ "__tsan_read_write2",
+ "__tsan_read_write4",
+ "__tsan_read_write8",
+ "__tsan_read_write16",
"__tsan_atomic8_load",
"__tsan_atomic16_load",
"__tsan_atomic32_load",
--
2.28.0.rc0.105.gf9edc3c819-goog

2020-07-21 14:07:48

by Peter Zijlstra

Subject: Re: [PATCH 2/8] objtool, kcsan: Add __tsan_read_write to uaccess whitelist

On Tue, Jul 21, 2020 at 12:30:10PM +0200, Marco Elver wrote:
> Adds the new __tsan_read_write compound instrumentation to objtool's
> uaccess whitelist.
>
> Signed-off-by: Marco Elver <[email protected]>

Acked-by: Peter Zijlstra (Intel) <[email protected]>

> ---
> tools/objtool/check.c | 5 +++++
> 1 file changed, 5 insertions(+)
>
> diff --git a/tools/objtool/check.c b/tools/objtool/check.c
> index 63d8b630c67a..38d82e705c93 100644
> --- a/tools/objtool/check.c
> +++ b/tools/objtool/check.c
> @@ -528,6 +528,11 @@ static const char *uaccess_safe_builtin[] = {
> "__tsan_write4",
> "__tsan_write8",
> "__tsan_write16",
> + "__tsan_read_write1",
> + "__tsan_read_write2",
> + "__tsan_read_write4",
> + "__tsan_read_write8",
> + "__tsan_read_write16",
> "__tsan_atomic8_load",
> "__tsan_atomic16_load",
> "__tsan_atomic32_load",
> --
> 2.28.0.rc0.105.gf9edc3c819-goog
>