Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
Date:   Tue, 27 Apr 2021 09:53:55 -0300
From:   Arnaldo Carvalho de Melo <acme@kernel.org>
To:     Nicholas Fraser <nfraser@codeweavers.com>
Cc:     Peter Zijlstra <peterz@infradead.org>,
        Ingo Molnar <mingo@redhat.com>,
        Mark Rutland <mark.rutland@arm.com>,
        Alexander Shishkin <alexander.shishkin@linux.intel.com>,
        Jiri Olsa <jolsa@redhat.com>,
        Namhyung Kim <namhyung@kernel.org>,
        Ian Rogers <irogers@google.com>,
        Changbin Du <changbin.du@intel.com>,
        Kan Liang <kan.liang@linux.intel.com>,
        Tan Xiaojun <tanxiaojun@huawei.com>,
        Jin Yao <yao.jin@linux.intel.com>,
        linux-kernel@vger.kernel.org,
        Ulrich Czekalla <uczekalla@codeweavers.com>
Subject: Re: [PATCH v2] perf data: Add JSON export
Message-ID: <YIgJY5jejgvTWqmR@kernel.org>
References: <3884969f-804d-2f53-c648-e2b0bd85edff@codeweavers.com>
 <YIch1aRwL4Lk7iWJ@kernel.org>
 <YIcvjFr/hN2Syybc@kernel.org>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <YIcvjFr/hN2Syybc@kernel.org>
Precedence: bulk

Em Mon, Apr 26, 2021 at 06:24:29PM -0300, Arnaldo Carvalho de Melo escreveu:
> Em Mon, Apr 26, 2021 at 05:25:58PM -0300, Arnaldo Carvalho de Melo escreveu:
> > Em Mon, Apr 26, 2021 at 10:47:16AM -0400, Nicholas Fraser escreveu:
> > > This adds a feature to export perf data to JSON.
> > > 
> > > The resolved symbols are exported into the JSON so that external tools
> > > don't need to load the dsos themselves (or even have access to them at
> > > all.) This makes it easy to load and analyze perf data with standalone
> > > tools where direct perf or libbabeltrace integration is impractical.
> > > 
> > > The exporter uses a minimal inline JSON encoding without any external
> > > dependencies. Currently it only outputs some headers and sample metadata
> > > but it's easily extensible.
> > > 
> > > Use it like this:
> > > 
> > > perf data convert --to-json out.json
> 
> One more, clang complains:
> 
>    7    50.92 alpine:3.10                   : FAIL gcc version 8.3.0 (Alpine 8.3.0)
>     util/data-convert-json.c:126:32: error: address of array 'al->sym->name' will always evaluate to 'true' [-Werror,-Wpointer-bool-conversion]
>             if (al && al->sym && al->sym->name && strlen(al->sym->name) > 0) {
>                               ~~ ~~~~~~~~~^~~~
>     1 error generated.
>     make[3]: *** [/git/perf-5.12.0/tools/build/Makefile.build:139: util] Error 2

So the fix is the one below, applied and tested, thanks.

- Arnaldo

diff --git a/tools/perf/util/data-convert-json.c b/tools/perf/util/data-convert-json.c
index f24593d57f80be1b..355cd1948bdf006a 100644
--- a/tools/perf/util/data-convert-json.c
+++ b/tools/perf/util/data-convert-json.c
@@ -123,7 +123,7 @@ static void output_sample_callchain_entry(struct perf_tool *tool,
 	output_json_format(out, false, 4, "{");
 	output_json_key_format(out, false, 5, "ip", "\"0x%" PRIx64 "\"", ip);
 
-	if (al && al->sym && al->sym->name && strlen(al->sym->name) > 0) {
+	if (al && al->sym && al->sym->namelen) {
 		fputc(',', out);
 		output_json_key_string(out, false, 5, "symbol", al->sym->name);