From: Stephane Eranian <eran...@google.com>

Add a new perf script filter called code_page_size. There will be more
than 32 filters. Extend the enum perf_output_field from U to UL.
Display sampled code page sizes when PERF_SAMPLE_CODE_PAGE_SIZE was set.

For example,
perf script --fields comm,event,ip,code_page_size
            dtlb mem-loads:uP:            445777 4K
            dtlb mem-loads:uP:            40f724 4K
            dtlb mem-loads:uP:            474926 4K
            dtlb mem-loads:uP:            401075 4K
            dtlb mem-loads:uP:            401095 4K
            dtlb mem-loads:uP:            401095 4K
            dtlb mem-loads:uP:            4010cc 4K
            dtlb mem-loads:uP:            440b6f 4K

Signed-off-by: Stephane Eranian <eran...@google.com>
Signed-off-by: Kan Liang <kan.li...@linux.intel.com>
---

No changes since V2

 tools/perf/Documentation/perf-script.txt |  2 +-
 tools/perf/builtin-script.c              | 76 ++++++++++++++++++--------------
 tools/perf/util/session.c                |  3 ++
 3 files changed, 46 insertions(+), 35 deletions(-)

diff --git a/tools/perf/Documentation/perf-script.txt 
b/tools/perf/Documentation/perf-script.txt
index 14ae84c1..08c6deb 100644
--- a/tools/perf/Documentation/perf-script.txt
+++ b/tools/perf/Documentation/perf-script.txt
@@ -118,7 +118,7 @@ OPTIONS
         comm, tid, pid, time, cpu, event, trace, ip, sym, dso, addr, symoff,
         srcline, period, iregs, uregs, brstack, brstacksym, flags, bpf-output,
         brstackinsn, brstackoff, callindent, insn, insnlen, synth, phys_addr,
-        metric, misc, srccode, data_page_size.
+        metric, misc, srccode, data_page_size, code_page_size.
         Field list can be prepended with the type, trace, sw or hw,
         to indicate to which event type the field list applies.
         e.g., -F sw:comm,tid,time,ip,sym  and -F trace:time,cpu,trace
diff --git a/tools/perf/builtin-script.c b/tools/perf/builtin-script.c
index 0aa55ca..5c8e787 100644
--- a/tools/perf/builtin-script.c
+++ b/tools/perf/builtin-script.c
@@ -67,38 +67,39 @@ static int                  max_blocks;
 unsigned int scripting_max_stack = PERF_MAX_STACK_DEPTH;
 
 enum perf_output_field {
-       PERF_OUTPUT_COMM            = 1U << 0,
-       PERF_OUTPUT_TID             = 1U << 1,
-       PERF_OUTPUT_PID             = 1U << 2,
-       PERF_OUTPUT_TIME            = 1U << 3,
-       PERF_OUTPUT_CPU             = 1U << 4,
-       PERF_OUTPUT_EVNAME          = 1U << 5,
-       PERF_OUTPUT_TRACE           = 1U << 6,
-       PERF_OUTPUT_IP              = 1U << 7,
-       PERF_OUTPUT_SYM             = 1U << 8,
-       PERF_OUTPUT_DSO             = 1U << 9,
-       PERF_OUTPUT_ADDR            = 1U << 10,
-       PERF_OUTPUT_SYMOFFSET       = 1U << 11,
-       PERF_OUTPUT_SRCLINE         = 1U << 12,
-       PERF_OUTPUT_PERIOD          = 1U << 13,
-       PERF_OUTPUT_IREGS           = 1U << 14,
-       PERF_OUTPUT_BRSTACK         = 1U << 15,
-       PERF_OUTPUT_BRSTACKSYM      = 1U << 16,
-       PERF_OUTPUT_DATA_SRC        = 1U << 17,
-       PERF_OUTPUT_WEIGHT          = 1U << 18,
-       PERF_OUTPUT_BPF_OUTPUT      = 1U << 19,
-       PERF_OUTPUT_CALLINDENT      = 1U << 20,
-       PERF_OUTPUT_INSN            = 1U << 21,
-       PERF_OUTPUT_INSNLEN         = 1U << 22,
-       PERF_OUTPUT_BRSTACKINSN     = 1U << 23,
-       PERF_OUTPUT_BRSTACKOFF      = 1U << 24,
-       PERF_OUTPUT_SYNTH           = 1U << 25,
-       PERF_OUTPUT_PHYS_ADDR       = 1U << 26,
-       PERF_OUTPUT_UREGS           = 1U << 27,
-       PERF_OUTPUT_METRIC          = 1U << 28,
-       PERF_OUTPUT_MISC            = 1U << 29,
-       PERF_OUTPUT_SRCCODE         = 1U << 30,
-       PERF_OUTPUT_DATA_PAGE_SIZE  = 1U << 31,
+       PERF_OUTPUT_COMM            = 1ULL << 0,
+       PERF_OUTPUT_TID             = 1ULL << 1,
+       PERF_OUTPUT_PID             = 1ULL << 2,
+       PERF_OUTPUT_TIME            = 1ULL << 3,
+       PERF_OUTPUT_CPU             = 1ULL << 4,
+       PERF_OUTPUT_EVNAME          = 1ULL << 5,
+       PERF_OUTPUT_TRACE           = 1ULL << 6,
+       PERF_OUTPUT_IP              = 1ULL << 7,
+       PERF_OUTPUT_SYM             = 1ULL << 8,
+       PERF_OUTPUT_DSO             = 1ULL << 9,
+       PERF_OUTPUT_ADDR            = 1ULL << 10,
+       PERF_OUTPUT_SYMOFFSET       = 1ULL << 11,
+       PERF_OUTPUT_SRCLINE         = 1ULL << 12,
+       PERF_OUTPUT_PERIOD          = 1ULL << 13,
+       PERF_OUTPUT_IREGS           = 1ULL << 14,
+       PERF_OUTPUT_BRSTACK         = 1ULL << 15,
+       PERF_OUTPUT_BRSTACKSYM      = 1ULL << 16,
+       PERF_OUTPUT_DATA_SRC        = 1ULL << 17,
+       PERF_OUTPUT_WEIGHT          = 1ULL << 18,
+       PERF_OUTPUT_BPF_OUTPUT      = 1ULL << 19,
+       PERF_OUTPUT_CALLINDENT      = 1ULL << 20,
+       PERF_OUTPUT_INSN            = 1ULL << 21,
+       PERF_OUTPUT_INSNLEN         = 1ULL << 22,
+       PERF_OUTPUT_BRSTACKINSN     = 1ULL << 23,
+       PERF_OUTPUT_BRSTACKOFF      = 1ULL << 24,
+       PERF_OUTPUT_SYNTH           = 1ULL << 25,
+       PERF_OUTPUT_PHYS_ADDR       = 1ULL << 26,
+       PERF_OUTPUT_UREGS           = 1ULL << 27,
+       PERF_OUTPUT_METRIC          = 1ULL << 28,
+       PERF_OUTPUT_MISC            = 1ULL << 29,
+       PERF_OUTPUT_SRCCODE         = 1ULL << 30,
+       PERF_OUTPUT_DATA_PAGE_SIZE  = 1ULL << 31,
+       PERF_OUTPUT_CODE_PAGE_SIZE  = 1ULL << 32,
 };
 
 struct output_option {
@@ -137,6 +138,7 @@ struct output_option {
        {.str = "misc", .field = PERF_OUTPUT_MISC},
        {.str = "srccode", .field = PERF_OUTPUT_SRCCODE},
        {.str = "data_page_size", .field = PERF_OUTPUT_DATA_PAGE_SIZE},
+       {.str = "code_page_size", .field = PERF_OUTPUT_CODE_PAGE_SIZE},
 };
 
 enum {
@@ -208,7 +210,7 @@ static struct {
                              PERF_OUTPUT_DSO | PERF_OUTPUT_PERIOD |
                              PERF_OUTPUT_ADDR | PERF_OUTPUT_DATA_SRC |
                              PERF_OUTPUT_WEIGHT | PERF_OUTPUT_PHYS_ADDR |
-                             PERF_OUTPUT_DATA_PAGE_SIZE,
+                             PERF_OUTPUT_DATA_PAGE_SIZE | 
PERF_OUTPUT_CODE_PAGE_SIZE,
 
                .invalid_fields = PERF_OUTPUT_TRACE | PERF_OUTPUT_BPF_OUTPUT,
        },
@@ -475,7 +477,10 @@ static int perf_evsel__check_attr(struct perf_evsel *evsel,
        if (PRINT_FIELD(DATA_PAGE_SIZE) &&
                perf_evsel__check_stype(evsel, PERF_SAMPLE_DATA_PAGE_SIZE, 
"DATA_PAGE_SIZE",
                                        PERF_OUTPUT_DATA_PAGE_SIZE))
-               return -EINVAL;
+       if (PRINT_FIELD(CODE_PAGE_SIZE) &&
+               perf_evsel__check_stype(evsel, PERF_SAMPLE_CODE_PAGE_SIZE, 
"CODE_PAGE_SIZE",
+                                       PERF_OUTPUT_CODE_PAGE_SIZE))
+                       return -EINVAL;
 
        return 0;
 }
@@ -1855,6 +1860,9 @@ static void process_event(struct perf_script *script,
        if (PRINT_FIELD(DATA_PAGE_SIZE))
                fprintf(fp, " %s", get_page_size_name(sample->data_page_size));
 
+       if (PRINT_FIELD(CODE_PAGE_SIZE))
+               fprintf(fp, " %s", get_page_size_name(sample->code_page_size));
+
        fprintf(fp, "\n");
 
        if (PRINT_FIELD(SRCCODE)) {
diff --git a/tools/perf/util/session.c b/tools/perf/util/session.c
index 3c0698d..9690c6d 100644
--- a/tools/perf/util/session.c
+++ b/tools/perf/util/session.c
@@ -1145,6 +1145,9 @@ static void dump_sample(struct perf_evsel *evsel, union 
perf_event *event,
        if (sample_type & PERF_SAMPLE_DATA_PAGE_SIZE)
                printf(" .. data page size: %s\n", 
get_page_size_name(sample->data_page_size));
 
+       if (sample_type & PERF_SAMPLE_CODE_PAGE_SIZE)
+               printf(" .. code page size: %s\n", 
get_page_size_name(sample->code_page_size));
+
        if (sample_type & PERF_SAMPLE_TRANSACTION)
                printf("... transaction: %" PRIx64 "\n", sample->transaction);
 
-- 
2.7.4

Reply via email to