Commit fe96245c authored by Jin Yao's avatar Jin Yao Committed by Arnaldo Carvalho de Melo
Browse files

perf hists: Add block_info in hist_entry 



The block_info contains the program basic block information, i.e,
contains the start address and the end address of this basic block and
how much cycles it takes.

We need to compare, sort and even print out the basic block by some
orders, i.e. sort by cycles.

For this purpose, we add block_info field to hist_entry. In order not to
impact current interface, we creates a new function
hists__add_entry_block.

 v6:
 ---
 Remove the 'ops' argument in hists__add_entry_block

Signed-off-by: default avatarJin Yao <yao.jin@linux.intel.com>
Reviewed-by: default avatarJiri Olsa <jolsa@kernel.org>
Cc: Alexander Shishkin <alexander.shishkin@linux.intel.com>
Cc: Andi Kleen <ak@linux.intel.com>
Cc: Jin Yao <yao.jin@intel.com>
Cc: Kan Liang <kan.liang@linux.intel.com>
Cc: Peter Zijlstra <peterz@infradead.org>
Link: http://lkml.kernel.org/r/1561713784-30533-3-git-send-email-yao.jin@linux.intel.com


Signed-off-by: default avatarArnaldo Carvalho de Melo <acme@redhat.com>
parent 0cec2447

tools/perf/util/hist.c

+18 −2
Original line number Diff line number Diff line
@@ -574,6 +574,8 @@ static struct hist_entry *hists__findnew_entry(struct hists *hists, 
			 */

			mem_info__zput(entry->mem_info);



			block_info__zput(entry->block_info);



			/* If the map of an existing hist_entry has

			 * become out-of-date due to an exec() or

			 * similar, update it.  Otherwise we will
@@ -645,6 +647,7 @@ __hists__add_entry(struct hists *hists, 
		   struct symbol *sym_parent,

		   struct branch_info *bi,

		   struct mem_info *mi,

		   struct block_info *block_info,

		   struct perf_sample *sample,

		   bool sample_self,

		   struct hist_entry_ops *ops)
@@ -677,6 +680,7 @@ __hists__add_entry(struct hists *hists, 
		.hists	= hists,

		.branch_info = bi,

		.mem_info = mi,

		.block_info = block_info,

		.transaction = sample->transaction,

		.raw_data = sample->raw_data,

		.raw_size = sample->raw_size,
@@ -699,7 +703,7 @@ struct hist_entry *hists__add_entry(struct hists *hists, 
				    struct perf_sample *sample,

				    bool sample_self)

{

	return __hists__add_entry(hists, al, sym_parent, bi, mi,

	return __hists__add_entry(hists, al, sym_parent, bi, mi, NULL,

				  sample, sample_self, NULL);

}
@@ -712,10 +716,22 @@ struct hist_entry *hists__add_entry_ops(struct hists *hists, 
					struct perf_sample *sample,

					bool sample_self)

{

	return __hists__add_entry(hists, al, sym_parent, bi, mi,

	return __hists__add_entry(hists, al, sym_parent, bi, mi, NULL,

				  sample, sample_self, ops);

}



struct hist_entry *hists__add_entry_block(struct hists *hists,

					  struct addr_location *al,

					  struct block_info *block_info)

{

	struct hist_entry entry = {

		.block_info = block_info,

		.hists = hists,

	}, *he = hists__findnew_entry(hists, &entry, al, false);



	return he;

}



static int

iter_next_nop_entry(struct hist_entry_iter *iter __maybe_unused,

		    struct addr_location *al __maybe_unused)

tools/perf/util/hist.h

+5 −0
Original line number Diff line number Diff line
@@ -16,6 +16,7 @@ struct addr_location; 
struct map_symbol;

struct mem_info;

struct branch_info;

struct block_info;

struct symbol;



enum hist_filter {
@@ -149,6 +150,10 @@ struct hist_entry *hists__add_entry_ops(struct hists *hists, 
					struct perf_sample *sample,

					bool sample_self);



struct hist_entry *hists__add_entry_block(struct hists *hists,

					  struct addr_location *al,

					  struct block_info *bi);



int hist_entry_iter__add(struct hist_entry_iter *iter, struct addr_location *al,

			 int max_stack_depth, void *arg);

tools/perf/util/sort.h

+1 −0
Original line number Diff line number Diff line
@@ -144,6 +144,7 @@ struct hist_entry { 
	long			time;

	struct hists		*hists;

	struct mem_info		*mem_info;

	struct block_info	*block_info;

	void			*raw_data;

	u32			raw_size;

	int			num_res;

CRA Git | Maintained and supported by SUSTech CRA and CCSE