Commit f1cbf78d authored Dec 18, 2013 by Namhyung Kim Committed by Arnaldo Carvalho de Melo Dec 18, 2013

perf hists: Do not pass period and weight to add_hist_entry()



The @entry argument already has the info so no need to pass them.

Signed-off-by: Namhyung Kim <namhyung@kernel.org>
Acked-by: Jiri Olsa <jolsa@redhat.com>
Cc: Arun Sharma <asharma@fb.com>
Cc: Frederic Weisbecker <fweisbec@gmail.com>
Cc: Ingo Molnar <mingo@kernel.org>
Cc: Jiri Olsa <jolsa@redhat.com>
Cc: Paul Mackerras <paulus@samba.org>
Cc: Peter Zijlstra <a.p.zijlstra@chello.nl>
Cc: Rodrigo Campos <rodrigo@sdfg.com.ar>
Link: http://lkml.kernel.org/r/1387344086-12744-4-git-send-email-namhyung@kernel.org


Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>

parent 68f6d022

tools/perf/util/hist.c

+5 −5

Original line number	Diff line number	Diff line
		@@ -343,14 +343,14 @@ static u8 symbol__parent_filter(const struct symbol *parent)

		static struct hist_entry add_hist_entry(struct hists hists,
		struct hist_entry *entry,
		struct addr_location *al,
		u64 period,
		u64 weight)
		struct addr_location *al)
		{
		struct rb_node **p;
		struct rb_node *parent = NULL;
		struct hist_entry *he;
		int64_t cmp;
		u64 period = entry->stat.period;
		u64 weight = entry->stat.weight;

		p = &hists->entries_in->rb_node;

		@@ -437,7 +437,7 @@ struct hist_entry __hists__add_entry(struct hists hists,
		.transaction = transaction,
		};

		return add_hist_entry(hists, &entry, al, period, weight);
		return add_hist_entry(hists, &entry, al);
		}

		int64_t