lkml.org 
[lkml]   [2017]   [Oct]   [3]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH v2 4/4] Make all print functions receive the fp argument, and opens a dump file in process_event.
On Tue, Sep 26, 2017 at 02:53:17PM +0800, yuzhoujian wrote:

SNIP

> + fprintf(fp, "%16" PRIx64, sample->phys_addr);
> + fprintf(fp, "\n");
> + if (script->tool.per_event_dump)
> + fclose(fp);
> }
>
> static struct scripting_ops *scripting_ops;
> @@ -1574,6 +1589,7 @@ static int process_sample_event(struct perf_tool *tool,
> {
> struct perf_script *scr = container_of(tool, struct perf_script, tool);
> struct addr_location al;
> + FILE *fp;
>
> if (perf_time__skip_sample(&scr->ptime, sample->time))
> return 0;
> @@ -1601,10 +1617,11 @@ static int process_sample_event(struct perf_tool *tool,
> if (cpu_list && !test_bit(sample->cpu, cpu_bitmap))
> goto out_put;
>
> + fp = tool->per_event_dump ? per_event_dump_file : stdout;

I know you explained this already, but I still can't see this one
being set:

[jolsa@krava perf]$ grep -r per_event_dump_file builtin-script.c
static FILE *per_event_dump_file;
fp = tool->per_event_dump ? per_event_dump_file : stdout;
fp = tool->per_event_dump ? per_event_dump_file : stdout;
fp = tool->per_event_dump ? per_event_dump_file : stdout;
fp = tool->per_event_dump ? per_event_dump_file : stdout;
fp = tool->per_event_dump ? per_event_dump_file : stdout;
fp = tool->per_event_dump ? per_event_dump_file : stdout;
fp = tool->per_event_dump ? per_event_dump_file : stdout;
fp = tool->per_event_dump ? per_event_dump_file : stdout;

I think we should use per evsel FILE pointers


jirka

\
 
 \ /
  Last update: 2017-10-03 12:47    [W:0.085 / U:0.904 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site