Messages in this thread | | | From | Jérémie Galarneau <> | Date | Tue, 18 Sep 2018 14:07:41 -0400 | Subject | Re: [PATCH] perf tools: Initialize perf_data_file fd field |
| |
Hi Arnaldo,
Anything I can do to help you with this patch?
Thanks, Jérémie
On 5 September 2018 at 12:09, Jiri Olsa <jolsa@redhat.com> wrote: > On Wed, Aug 29, 2018 at 04:16:48PM -0400, Jérémie Galarneau wrote: >> Building the perf CTF converter fails with gcc 4.8.4 >> on Ubuntu 14.04 with the following error: >> >> error: missing initializer for field ‘fd’ of ‘struct perf_data_file’ >> [-Werror=missing-field-initializers] >> >> Per 4b838b0d and the ensuing discussion on the mailing >> list, it appears that this affects other distributions >> and gcc versions. >> >> Signed-off-by: Jérémie Galarneau <jeremie.galarneau@efficios.com> > > Acked-by: Jiri Olsa <jolsa@kernel.org> > > thanks, > jirka > >> Cc: Jiri Olsa <jolsa@kernel.org> >> Cc: Arnaldo Carvalho de Melo <acme@redhat.com> >> Cc: Peter Zijlstra <peterz@infradead.org> >> Cc: Ingo Molnar <mingo@redhat.com> >> Cc: Alexander Shishkin <alexander.shishkin@linux.intel.com> >> --- >> tools/perf/util/data-convert-bt.c | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/tools/perf/util/data-convert-bt.c b/tools/perf/util/data-convert-bt.c >> index abd38abf1d91..f75d4aa612c5 100644 >> --- a/tools/perf/util/data-convert-bt.c >> +++ b/tools/perf/util/data-convert-bt.c >> @@ -1578,7 +1578,7 @@ int bt_convert__perf2ctf(const char *input, const char *path, >> { >> struct perf_session *session; >> struct perf_data data = { >> - .file.path = input, >> + .file = { .path = input, .fd = -1 }, >> .mode = PERF_DATA_MODE_READ, >> .force = opts->force, >> }; >> -- >> 2.18.0 >>
-- Jérémie Galarneau EfficiOS Inc. http://www.efficios.com
| |