* [PATCH v1] perf build: Pretend scandirat is missing with msan
@ 2024-03-20 16:32 Ian Rogers
2024-03-20 16:36 ` Ian Rogers
2024-04-25 18:45 ` Arnaldo Carvalho de Melo
0 siblings, 2 replies; 4+ messages in thread
From: Ian Rogers @ 2024-03-20 16:32 UTC (permalink / raw)
To: Peter Zijlstra, Ingo Molnar, Arnaldo Carvalho de Melo,
Namhyung Kim, Mark Rutland, Alexander Shishkin, Jiri Olsa,
Ian Rogers, Adrian Hunter, linux-perf-users, linux-kernel
Memory sanitizer lacks an interceptor for scandirat, reporting all
memory it allocates as uninitialized. Memory sanitizer has a scandir
interceptor so use the fallback function in this case. This allows
perf test to run under memory sanitizer.
Signed-off-by: Ian Rogers <irogers@google.com>
---
tools/perf/Makefile.config | 5 ++++-
1 file changed, 4 insertions(+), 1 deletion(-)
diff --git a/tools/perf/Makefile.config b/tools/perf/Makefile.config
index 1fe8df97fe88..74e0b17050b5 100644
--- a/tools/perf/Makefile.config
+++ b/tools/perf/Makefile.config
@@ -486,7 +486,10 @@ ifdef NO_DWARF
endif
ifeq ($(feature-scandirat), 1)
- CFLAGS += -DHAVE_SCANDIRAT_SUPPORT
+ # Ignore having scandirat with memory sanitizer that lacks an interceptor.
+ ifeq ($(filter s% -fsanitize=memory%,$(EXTRA_CFLAGS),),)
+ CFLAGS += -DHAVE_SCANDIRAT_SUPPORT
+ endif
endif
ifeq ($(feature-sched_getcpu), 1)
--
2.44.0.291.gc1ea87d7ee-goog
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH v1] perf build: Pretend scandirat is missing with msan
2024-03-20 16:32 [PATCH v1] perf build: Pretend scandirat is missing with msan Ian Rogers
@ 2024-03-20 16:36 ` Ian Rogers
2024-04-25 18:47 ` Arnaldo Carvalho de Melo
2024-04-25 18:45 ` Arnaldo Carvalho de Melo
1 sibling, 1 reply; 4+ messages in thread
From: Ian Rogers @ 2024-03-20 16:36 UTC (permalink / raw)
To: Peter Zijlstra, Ingo Molnar, Arnaldo Carvalho de Melo,
Namhyung Kim, Mark Rutland, Alexander Shishkin, Jiri Olsa,
Ian Rogers, Adrian Hunter, linux-perf-users, linux-kernel
On Wed, Mar 20, 2024 at 9:32 AM Ian Rogers <irogers@google.com> wrote:
>
> Memory sanitizer lacks an interceptor for scandirat, reporting all
> memory it allocates as uninitialized. Memory sanitizer has a scandir
> interceptor so use the fallback function in this case. This allows
> perf test to run under memory sanitizer.
>
> Signed-off-by: Ian Rogers <irogers@google.com>
Note, as msan needs to instrument memory allocations libraries need to
be compiled with it. I lacked the msan built libraries and so built
with:
```
$ make -C tools/perf O=/tmp/perf DEBUG=1 EXTRA_CFLAGS="-O0 -g
-fno-omit-frame-pointer -fsanitize=memory
-fsanitize-memory-track-origins" CC=clang CXX=clang++ HOSTCC=clang
NO_LIBTRACEEVENT=1 NO_LIBELF=1 BUILD_BPF_SKEL=0 NO_LIBPFM=1
```
oh, I disabled libbpf here as the bpf system call also lacks msan interceptors.
Thanks,
Ian
> ---
> tools/perf/Makefile.config | 5 ++++-
> 1 file changed, 4 insertions(+), 1 deletion(-)
>
> diff --git a/tools/perf/Makefile.config b/tools/perf/Makefile.config
> index 1fe8df97fe88..74e0b17050b5 100644
> --- a/tools/perf/Makefile.config
> +++ b/tools/perf/Makefile.config
> @@ -486,7 +486,10 @@ ifdef NO_DWARF
> endif
>
> ifeq ($(feature-scandirat), 1)
> - CFLAGS += -DHAVE_SCANDIRAT_SUPPORT
> + # Ignore having scandirat with memory sanitizer that lacks an interceptor.
> + ifeq ($(filter s% -fsanitize=memory%,$(EXTRA_CFLAGS),),)
> + CFLAGS += -DHAVE_SCANDIRAT_SUPPORT
> + endif
> endif
>
> ifeq ($(feature-sched_getcpu), 1)
> --
> 2.44.0.291.gc1ea87d7ee-goog
>
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH v1] perf build: Pretend scandirat is missing with msan
2024-03-20 16:32 [PATCH v1] perf build: Pretend scandirat is missing with msan Ian Rogers
2024-03-20 16:36 ` Ian Rogers
@ 2024-04-25 18:45 ` Arnaldo Carvalho de Melo
1 sibling, 0 replies; 4+ messages in thread
From: Arnaldo Carvalho de Melo @ 2024-04-25 18:45 UTC (permalink / raw)
To: Ian Rogers
Cc: Peter Zijlstra, Ingo Molnar, Namhyung Kim, Mark Rutland,
Alexander Shishkin, Jiri Olsa, Adrian Hunter, linux-perf-users,
linux-kernel
On Wed, Mar 20, 2024 at 09:32:44AM -0700, Ian Rogers wrote:
> Memory sanitizer lacks an interceptor for scandirat, reporting all
> memory it allocates as uninitialized. Memory sanitizer has a scandir
> interceptor so use the fallback function in this case. This allows
> perf test to run under memory sanitizer.
Thanks, applied to perf-tools-next,
- Arnaldo
> Signed-off-by: Ian Rogers <irogers@google.com>
> ---
> tools/perf/Makefile.config | 5 ++++-
> 1 file changed, 4 insertions(+), 1 deletion(-)
>
> diff --git a/tools/perf/Makefile.config b/tools/perf/Makefile.config
> index 1fe8df97fe88..74e0b17050b5 100644
> --- a/tools/perf/Makefile.config
> +++ b/tools/perf/Makefile.config
> @@ -486,7 +486,10 @@ ifdef NO_DWARF
> endif
>
> ifeq ($(feature-scandirat), 1)
> - CFLAGS += -DHAVE_SCANDIRAT_SUPPORT
> + # Ignore having scandirat with memory sanitizer that lacks an interceptor.
> + ifeq ($(filter s% -fsanitize=memory%,$(EXTRA_CFLAGS),),)
> + CFLAGS += -DHAVE_SCANDIRAT_SUPPORT
> + endif
> endif
>
> ifeq ($(feature-sched_getcpu), 1)
> --
> 2.44.0.291.gc1ea87d7ee-goog
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH v1] perf build: Pretend scandirat is missing with msan
2024-03-20 16:36 ` Ian Rogers
@ 2024-04-25 18:47 ` Arnaldo Carvalho de Melo
0 siblings, 0 replies; 4+ messages in thread
From: Arnaldo Carvalho de Melo @ 2024-04-25 18:47 UTC (permalink / raw)
To: Ian Rogers
Cc: Peter Zijlstra, Ingo Molnar, Namhyung Kim, Mark Rutland,
Alexander Shishkin, Jiri Olsa, Adrian Hunter, linux-perf-users,
linux-kernel
On Wed, Mar 20, 2024 at 09:36:19AM -0700, Ian Rogers wrote:
> On Wed, Mar 20, 2024 at 9:32 AM Ian Rogers <irogers@google.com> wrote:
> >
> > Memory sanitizer lacks an interceptor for scandirat, reporting all
> > memory it allocates as uninitialized. Memory sanitizer has a scandir
> > interceptor so use the fallback function in this case. This allows
> > perf test to run under memory sanitizer.
> >
> > Signed-off-by: Ian Rogers <irogers@google.com>
>
> Note, as msan needs to instrument memory allocations libraries need to
> be compiled with it. I lacked the msan built libraries and so built
> with:
> ```
> $ make -C tools/perf O=/tmp/perf DEBUG=1 EXTRA_CFLAGS="-O0 -g
> -fno-omit-frame-pointer -fsanitize=memory
> -fsanitize-memory-track-origins" CC=clang CXX=clang++ HOSTCC=clang
> NO_LIBTRACEEVENT=1 NO_LIBELF=1 BUILD_BPF_SKEL=0 NO_LIBPFM=1
> ```
> oh, I disabled libbpf here as the bpf system call also lacks msan interceptors.
I added the above notes to the cset message.
- Arnaldo
> Thanks,
> Ian
>
> > ---
> > tools/perf/Makefile.config | 5 ++++-
> > 1 file changed, 4 insertions(+), 1 deletion(-)
> >
> > diff --git a/tools/perf/Makefile.config b/tools/perf/Makefile.config
> > index 1fe8df97fe88..74e0b17050b5 100644
> > --- a/tools/perf/Makefile.config
> > +++ b/tools/perf/Makefile.config
> > @@ -486,7 +486,10 @@ ifdef NO_DWARF
> > endif
> >
> > ifeq ($(feature-scandirat), 1)
> > - CFLAGS += -DHAVE_SCANDIRAT_SUPPORT
> > + # Ignore having scandirat with memory sanitizer that lacks an interceptor.
> > + ifeq ($(filter s% -fsanitize=memory%,$(EXTRA_CFLAGS),),)
> > + CFLAGS += -DHAVE_SCANDIRAT_SUPPORT
> > + endif
> > endif
> >
> > ifeq ($(feature-sched_getcpu), 1)
> > --
> > 2.44.0.291.gc1ea87d7ee-goog
> >
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2024-04-25 18:47 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2024-03-20 16:32 [PATCH v1] perf build: Pretend scandirat is missing with msan Ian Rogers
2024-03-20 16:36 ` Ian Rogers
2024-04-25 18:47 ` Arnaldo Carvalho de Melo
2024-04-25 18:45 ` Arnaldo Carvalho de Melo
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).