| From 238373b6c2393288186999ae7664e173cbb90cbe Mon Sep 17 00:00:00 2001 |
| From: Numfor Mbiziwo-Tiapo <nums@google.com> |
| Date: Tue, 2 Jul 2019 10:37:15 -0700 |
| Subject: perf test mmap-thread-lookup: Initialize variable to suppress memory |
| sanitizer warning |
| |
| [ Upstream commit 4e4cf62b37da5ff45c904a3acf242ab29ed5881d ] |
| |
| Running the 'perf test' command after building perf with a memory |
| sanitizer causes a warning that says: |
| |
| WARNING: MemorySanitizer: use-of-uninitialized-value... in mmap-thread-lookup.c |
| |
| Initializing the go variable to 0 silences this harmless warning. |
| |
| Committer warning: |
| |
| This was harmless, just a simple test writing whatever was at that |
| sizeof(int) memory area just to signal another thread blocked reading |
| that file created with pipe(). Initialize it tho so that we don't get |
| this warning. |
| |
| Signed-off-by: Numfor Mbiziwo-Tiapo <nums@google.com> |
| Cc: Alexander Shishkin <alexander.shishkin@linux.intel.com> |
| Cc: Ian Rogers <irogers@google.com> |
| Cc: Jiri Olsa <jolsa@redhat.com> |
| Cc: Mark Drayton <mbd@fb.com> |
| Cc: Namhyung Kim <namhyung@kernel.org> |
| Cc: Peter Zijlstra <peterz@infradead.org> |
| Cc: Song Liu <songliubraving@fb.com> |
| Cc: Stephane Eranian <eranian@google.com> |
| Link: http://lkml.kernel.org/r/20190702173716.181223-1-nums@google.com |
| Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com> |
| Signed-off-by: Sasha Levin <sashal@kernel.org> |
| --- |
| tools/perf/tests/mmap-thread-lookup.c | 2 +- |
| 1 file changed, 1 insertion(+), 1 deletion(-) |
| |
| diff --git a/tools/perf/tests/mmap-thread-lookup.c b/tools/perf/tests/mmap-thread-lookup.c |
| index 0c5ce44f723f..e5d6e6584001 100644 |
| --- a/tools/perf/tests/mmap-thread-lookup.c |
| +++ b/tools/perf/tests/mmap-thread-lookup.c |
| @@ -49,7 +49,7 @@ static void *thread_fn(void *arg) |
| { |
| struct thread_data *td = arg; |
| ssize_t ret; |
| - int go; |
| + int go = 0; |
| |
| if (thread_init(td)) |
| return NULL; |
| -- |
| 2.20.1 |
| |