From: Kan Liang <kan.li...@linux.intel.com>

Extend sample-parsing test cases to support new sample type
PERF_SAMPLE_PAGE_SIZE.

Signed-off-by: Kan Liang <kan.li...@linux.intel.com>
---
 tools/perf/tests/sample-parsing.c | 7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)

diff --git a/tools/perf/tests/sample-parsing.c 
b/tools/perf/tests/sample-parsing.c
index 0e2d00d..cfba33d 100644
--- a/tools/perf/tests/sample-parsing.c
+++ b/tools/perf/tests/sample-parsing.c
@@ -145,6 +145,9 @@ static bool samples_same(const struct perf_sample *s1,
        if (type & PERF_SAMPLE_PHYS_ADDR)
                COMP(phys_addr);
 
+       if (type & PERF_SAMPLE_PAGE_SIZE)
+               COMP(page_size);
+
        return true;
 }
 
@@ -210,7 +213,9 @@ static int do_test(u64 sample_type, u64 sample_regs, u64 
read_format)
                        .mask   = sample_regs,
                        .regs   = regs,
                },
+
                .phys_addr      = 113,
+               .page_size      = PERF_MEM_PAGE_SIZE_4K,
        };
        struct sample_read_value values[] = {{1, 5}, {9, 3}, {2, 7}, {6, 4},};
        struct perf_sample sample_out;
@@ -310,7 +315,7 @@ int test__sample_parsing(struct test *test __maybe_unused, 
int subtest __maybe_u
         * were added.  Please actually update the test rather than just change
         * the condition below.
         */
-       if (PERF_SAMPLE_MAX > PERF_SAMPLE_PHYS_ADDR << 1) {
+       if (PERF_SAMPLE_MAX > PERF_SAMPLE_PAGE_SIZE << 1) {
                pr_debug("sample format has changed, some new PERF_SAMPLE_ bit 
was introduced - test needs updating\n");
                return -1;
        }
-- 
2.7.4

Reply via email to