Skip to contentSkip to conversation

Add Intel PT full trace mode basic test case

PATCH Superseded Ammy Yi 3 participants 3 messages Sep 13, 2018 – Oct 1, 2018

v3 of 5 Superseded Latest is v5 →

lore
Share & tools
lore

Ammy Yi

author
#0 lore raw

PATCH v3 1/1 Add Intel PT full trace mode basic test case

This test will do Intel PT full trace mode with test case PID to check if enable/stop/trace generation works with Intel PT.

---
 runtest/tracing                             |   1 +
 testcases/kernel/tracing/pt_test/.gitignore |   1 +
 testcases/kernel/tracing/pt_test/Makefile   |  21 ++++
 testcases/kernel/tracing/pt_test/pt_test.c  | 185 ++++++++++++++++++++++++++++
 4 files changed, 208 insertions(+)
 create mode 100644 testcases/kernel/tracing/pt_test/.gitignore
 create mode 100644 testcases/kernel/tracing/pt_test/Makefile
 create mode 100644 testcases/kernel/tracing/pt_test/pt_test.c
runtest/tracing +1 -0
index 8285eeb43..504132d70 100644
@@ -3,3 +3,4 @@ ftrace_regression01	ftrace_regression01.sh
 ftrace_regression02	ftrace_regression02.sh
 ftrace-stress-test	ftrace_stress_test.sh 90
 dynamic_debug01		dynamic_debug01.sh
+pt_full_trace_basic pt_test
testcases/kernel/tracing/pt_test/.gitignore new file +1 -0
new file mode 100644
index 000000000..96efc142d
@@ -0,0 +1 @@
+/pt_test
testcases/kernel/tracing/pt_test/Makefile new file +21 -0
new file mode 100644
index 000000000..a114fed1b
@@ -0,0 +1,21 @@
+#
+# Copyright (C) 2018 Intel Corporation
+#
+# This program is free software: you can redistribute it and/or modify
+# it under the terms of the GNU General Public License as published by
+# the Free Software Foundation, either version 2 of the License, or
+# (at your option) any later version.
+#
+# This program is distributed in the hope that it will be useful,
+# but WITHOUT ANY WARRANTY; without even the implied warranty of
+# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+# GNU General Public License for more details.
+#
+# You should have received a copy of the GNU General Public License
+# along with this program. If not, see <http://www.gnu.org/licenses/>.
+#
+
+top_srcdir		?= ../../../..
+
+include $(top_srcdir)/include/mk/testcases.mk
+include $(top_srcdir)/include/mk/generic_leaf_target.mk
\ No newline at end of file
testcases/kernel/tracing/pt_test/pt_test.c new file +185 -0

185 lines not on this page · raw message

Signature · 1 line
2.14.1
#1 lore raw

PATCH v3 1/1 Add Intel PT full trace mode basic test case

Hi ,

Is there any other comments for this patch? Thanks.

Best Regards
Ammy Yi

-----Original Message-----
From: Yi, Ammy
Sent: Thursday, September 13, 2018 9:55 AM
To: [email protected]
Cc: Yi, Ammy <[email protected]>
Subject: [PATCH v3 ltp] Add Intel PT full trace mode basic test case

This test will do Intel PT full trace mode with test case PID to check if enable/stop/trace generation works with Intel PT.

---
 runtest/tracing                             |   1 +
 testcases/kernel/tracing/pt_test/.gitignore |   1 +
 testcases/kernel/tracing/pt_test/Makefile   |  21 ++++
 testcases/kernel/tracing/pt_test/pt_test.c  | 185 ++++++++++++++++++++++++++++
 4 files changed, 208 insertions(+)
 create mode 100644 testcases/kernel/tracing/pt_test/.gitignore
 create mode 100644 testcases/kernel/tracing/pt_test/Makefile
 create mode 100644 testcases/kernel/tracing/pt_test/pt_test.c
runtest/tracing +1 -0
@@ -3,3 +3,4 @@ ftrace_regression01	ftrace_regression01.sh
 ftrace_regression02	ftrace_regression02.sh
 ftrace-stress-test	ftrace_stress_test.sh 90
 dynamic_debug01		dynamic_debug01.sh
+pt_full_trace_basic pt_test
testcases/kernel/tracing/pt_test/.gitignore new file +1 -0
new file mode 100644
index 000000000..96efc142d
@@ -0,0 +1 @@
+/pt_test
testcases/kernel/tracing/pt_test/Makefile new file +21 -0
new file mode 100644
index 000000000..a114fed1b
@@ -0,0 +1,21 @@
+#
+# Copyright (C) 2018 Intel Corporation
+#
+# This program is free software: you can redistribute it and/or modify 
+# it under the terms of the GNU General Public License as published by 
+# the Free Software Foundation, either version 2 of the License, or # 
+(at your option) any later version.
+#
+# This program is distributed in the hope that it will be useful, # but 
+WITHOUT ANY WARRANTY; without even the implied warranty of # 
+MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the # GNU 
+General Public License for more details.
+#
+# You should have received a copy of the GNU General Public License # 
+along with this program. If not, see <http://www.gnu.org/licenses/>.
+#
+
+top_srcdir		?= ../../../..
+
+include $(top_srcdir)/include/mk/testcases.mk
+include $(top_srcdir)/include/mk/generic_leaf_target.mk
\ No newline at end of file
testcases/kernel/tracing/pt_test/pt_test.c new file +182 -0

182 lines not on this page · raw message

--
2.14.1

#2 lore raw

PATCH v3 1/1 Add Intel PT full trace mode basic test case

In reply to Ammy Yi

Hi!

Quoted 29 lines
testcases/kernel/tracing/pt_test/pt_test.c new file +29 -0
+#include <linux/perf_event.h>
+#include <sched.h>
+#include <stdlib.h>
+#include <stdio.h>
+#include "tst_test.h"
+#include "lapi/syscalls.h"
+
+#define PAGESIZE 4096
+#define INTEL_PT_MEMSIZE (17*PAGESIZE)
+
+#define BIT(nr)                 (1UL << (nr))
+
+#define INTEL_PT_PMU_TYPE "/sys/devices/intel_pt/type"
+#define INTEL_PT_FORMAT_TSC "/sys/devices/intel_pt/format/tsc"
+#define INTEL_PT_FORMAT_NRT "/sys/devices/intel_pt/format/noretcomp"
+
+//Intel PT event handle
+int FDE = -1;
+//map head and size
+uint64_t **BUFM;
+long BUFSZ;
+
+uint64_t **create_map(int fde, long bufsize)
+{
+	int PROTo;
+	long pbufSz;
+	uint64_t **bufEv;
+	struct perf_event_mmap_page *pc;
+

testcases/kernel/tracing/pt_test/pt_test.c line 64

Quoted 10 lines
testcases/kernel/tracing/pt_test/pt_test.c new file +10 -0
+	bufEv = SAFE_MALLOC(2*sizeof(uint64_t *));
+
+	PROTo = PROT_READ | PROT_WRITE;
+
+	pbufSz = INTEL_PT_MEMSIZE;
+//try to mmap for trace
+	bufEv[0] = SAFE_MMAP(NULL, pbufSz, PROT_READ | PROT_WRITE,
+							MAP_SHARED, fde, 0);
+
+	if (bufEv[0] != MAP_FAILED) {

I think I told you already that SAFE_MMAP() cannot fail. If mmap() called in SAFE_MMAP() fails the test execution stops, that is the whole point of having the SAFE_ variants.

So in this case there is no need to check againts MAP_FAILED, because the bufEv[0] will never have value of MAP_FAILED.

Also bufEv is mixed case, which is frowned upon, so this should rather be buf_ev.

Quoted 23 lines
testcases/kernel/tracing/pt_test/pt_test.c new file +23 -0
+		pc = (struct perf_event_mmap_page *)bufEv[0];
+		pc->aux_offset = pbufSz;
+		pc->aux_size = bufsize;
+		bufEv[1] = SAFE_MMAP(NULL, bufsize, PROTo, MAP_SHARED,
+								fde, pbufSz);
+	}
+	return bufEv;
+}
+
+
+int intel_pt_pmu_value(char *dir)
+{
+	char *value;
+	int val = 0;
+	char delims[] = ":";
+
+	SAFE_FILE_SCANF(dir, "%m[^\n]", &value);
+	if (strstr(value, delims) == NULL) {
+		val = atoi(value);
+	} else {
+		strsep(&value, delims);
+		val = atoi(value);
+	}

testcases/kernel/tracing/pt_test/pt_test.c line 97

Quoted 10 lines
testcases/kernel/tracing/pt_test/pt_test.c new file +10 -0
+	return val;
+}
+
+void del_map(uint64_t **bufEv, long bufsize)
+{
+	long pbufSz;
+
+	pbufSz = INTEL_PT_MEMSIZE;
+
+	munmap(bufEv[0], pbufSz);

^
Why not just INTEL_PT_MEMSIZE?

There is no point in storing the constant in the variable first...

Quoted 14 lines
testcases/kernel/tracing/pt_test/pt_test.c new file +14 -0
+	munmap(bufEv[1], bufsize);
+
+	free(bufEv);
+}
+
+static void intel_pt_full_trace_check(void)
+{
+	uint64_t aux_head = 0;
+	struct perf_event_mmap_page *pmp;
+	/* enable tracing */
+	if (ioctl(FDE, PERF_EVENT_IOC_RESET) != 0) {
+		tst_res(TFAIL, "ioctl with PERF_EVENT_IOC_RESET is failed!");
+		return;
+	}

testcases/kernel/tracing/pt_test/pt_test.c line 121

Quoted 10 lines
testcases/kernel/tracing/pt_test/pt_test.c new file +10 -0
+	if (ioctl(FDE, PERF_EVENT_IOC_ENABLE) != 0) {
+		tst_res(TFAIL, "ioctl with PERF_EVENT_IOC_ENABLE is failed!");
+		return;
+	}
+
+	/* stop tracing */
+	if (ioctl(FDE, PERF_EVENT_IOC_DISABLE) != 0) {
+		tst_res(TFAIL, "ioctl with PERF_EVENT_IOC_DISABLE is failed!");
+		return;
+	}

We do have SAFE_IOCTL() so this should be just:

	SAFE_IOCTL(FDE, PERF_EVENT_IOC_RESET);
	SAFE_IOCTL(FDE, PERF_EVENT_IOC_ENABLE);
	SAFE_IOCTL(FDE, PERF_EVENT_IOC_DISABLE);
Quoted 21 lines
testcases/kernel/tracing/pt_test/pt_test.c new file +21 -0
+	// check if there is some trace generated
+	pmp = (struct perf_event_mmap_page *)BUFM[0];
+	aux_head = *(volatile uint64_t *)&pmp->aux_head;
+	if (aux_head == 0) {
+		tst_res(TFAIL, "There is no trace, so failed!");
+		return;
+	}
+
+	tst_res(TPASS, "perf trace full mode is passed!");
+}
+
+void setup(void)
+{
+	struct perf_event_attr attr = {};
+
+	BUFSZ = 2 * PAGESIZE;
+
+	//set attr for Intel PT trace
+	attr.type	= intel_pt_pmu_value(INTEL_PT_PMU_TYPE);
+	attr.read_format = PERF_FORMAT_ID | PERF_FORMAT_TOTAL_TIME_RUNNING |
+				PERF_FORMAT_TOTAL_TIME_ENABLED;

testcases/kernel/tracing/pt_test/pt_test.c line 153

Quoted 10 lines
testcases/kernel/tracing/pt_test/pt_test.c new file +10 -0
+	attr.disabled	= 1;
+	attr.config	= BIT(intel_pt_pmu_value(INTEL_PT_FORMAT_TSC)) |
+				BIT(intel_pt_pmu_value(INTEL_PT_FORMAT_NRT));
+	attr.size	= sizeof(struct perf_event_attr);
+	attr.exclude_kernel		= 0;
+	attr.exclude_user		= 0;
+	attr.mmap			= 1;
+
+	//only get trace for own pid
+	FDE = tst_syscall(__NR_perf_event_open, &attr, 0, -1, -1, 0);

^
Can we just call this fde? The uppercase names are kind of reserved for macros in C.

testcases/kernel/tracing/pt_test/pt_test.c line 156

Quoted 3 lines
testcases/kernel/tracing/pt_test/pt_test.c new file +3 -0
+	if (FDE < 0) {
+		tst_res(TINFO, "Open Intel PT event failed!");
+		tst_res(TFAIL, "perf trace full mode is failed!");

Why do we print two messages here? Shouldn't tst_res(TFAIL, "Open Intel PT event failed!") suffice?

Also we should probably print errno in this case as well, so it should be tst_res(TFAIL | TERRNO, ...);

testcases/kernel/tracing/pt_test/pt_test.c line 161

Quoted 5 lines
testcases/kernel/tracing/pt_test/pt_test.c new file +5 -0
+		return;
+	}
+	BUFM = NULL;
+	/* allocate memory for trace */
+	BUFM =  create_map(FDE, BUFSZ);

^
Here as well, it would be better to be called bufm.

testcases/kernel/tracing/pt_test/pt_test.c line 166

Quoted 4 lines
testcases/kernel/tracing/pt_test/pt_test.c new file +4 -0
+	if (!BUFM || (BUFM)[0] == MAP_FAILED || (BUFM)[1] == MAP_FAILED) {
+		tst_res(TINFO, "create_map failed!");
+		tst_res(TFAIL, "perf trace full mode is failed!");
+		return;

And as above, since we use SAFE_MMAP() this will never happen.

testcases/kernel/tracing/pt_test/pt_test.c line 175

Quoted 10 lines
testcases/kernel/tracing/pt_test/pt_test.c new file +9 -0
+	}
 
+}
+
+void cleanup(void)
+{
+	if (FDE != -1)
+		close(FDE);
+
+	if (!BUFM || (BUFM)[0] == MAP_FAILED || (BUFM)[1] == MAP_FAILED)

Here as well the MAP_FAILED check will never be true.

Also the cleanup has to be able to cope with partial initialization as
well, for example the second SAFE_MMAP() can exit the test in a state where we allocated bufm and mmapped bufm[0].

So we should probably do in setup:

	bufm = SAFE_MALLOC(...);
	bufm[0] = NULL;
	bufm[1] = NULL:

	bufm[0] = SAFE_MMAP(...);
	bufm[1] = SAFE_MMAP(...);

And in the cleanup (or in the del_map called from cleanup):

	if (bufm) {
		if (bufm[0])
			munmap(bufm[0], ...);

		if (bufm[1])
			munmap(bufm[1], ...);
	}

	free(bufm);
Quoted 16 lines
testcases/kernel/tracing/pt_test/pt_test.c new file +10 -0
+		del_map(BUFM, BUFSZ);
+}
+
+static struct tst_test test = {
+	.test_all = intel_pt_full_trace_check,
+	.min_kver = "4.1",
+	.setup = setup,
+	.cleanup = cleanup,
+	.needs_root = 1,
+};
Signature · 5 lines
2.14.1


-- 
Mailing list info: https://lists.linux.it/listinfo/ltp
Signature · 2 lines
Cyril Hrubis
[email protected]