Commit 7c1d862e authored by Ian Rogers's avatar Ian Rogers Committed by Arnaldo Carvalho de Melo
Browse files

perf test x86: intel-pt-test data is immutable so mark it const 



This allows the movement of 5,808 bytes from .data to .rodata.

Signed-off-by: default avatarIan Rogers <irogers@google.com>
Cc: Adrian Hunter <adrian.hunter@intel.com>
Cc: Alexander Shishkin <alexander.shishkin@linux.intel.com>
Cc: Andi Kleen <ak@linux.intel.com>
Cc: Ingo Molnar <mingo@redhat.com>
Cc: Jiri Olsa <jolsa@kernel.org>
Cc: K Prateek Nayak <kprateek.nayak@amd.com>
Cc: Kan Liang <kan.liang@linux.intel.com>
Cc: Leo Yan <leo.yan@linaro.org>
Cc: Mark Rutland <mark.rutland@arm.com>
Cc: Masami Hiramatsu <mhiramat@kernel.org>
Cc: Namhyung Kim <namhyung@kernel.org>
Cc: Paolo Bonzini <pbonzini@redhat.com>
Cc: Peter Zijlstra <peterz@infradead.org>
Cc: Ravi Bangoria <ravi.bangoria@amd.com>
Cc: Ross Zwisler <zwisler@chromium.org>
Cc: Sean Christopherson <seanjc@google.com>
Cc: Steven Rostedt (VMware) <rostedt@goodmis.org>
Cc: Tiezhu Yang <yangtiezhu@loongson.cn>
Cc: Yang Jihong <yangjihong1@huawei.com>
Link: https://lore.kernel.org/r/20230526183401.2326121-4-irogers@google.com


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

tools/perf/arch/x86/tests/intel-pt-test.c

+7 −7
Original line number Diff line number Diff line
@@ -22,7 +22,7 @@ 
 * @new_ctx: expected new packet context

 * @ctx_unchanged: the packet context must not change

 */

static struct test_data {

static const struct test_data {

	int len;

	u8 bytes[INTEL_PT_PKT_MAX_SZ];

	enum intel_pt_pkt_ctx ctx;
@@ -186,7 +186,7 @@ static struct test_data { 
	{0, {0}, 0, {0, 0, 0}, 0, 0 },

};



static int dump_packet(struct intel_pt_pkt *packet, u8 *bytes, int len)

static int dump_packet(const struct intel_pt_pkt *packet, const u8 *bytes, int len)

{

	char desc[INTEL_PT_PKT_DESC_MAX];

	int ret, i;
@@ -206,14 +206,14 @@ static int dump_packet(struct intel_pt_pkt *packet, u8 *bytes, int len) 
	return TEST_OK;

}



static void decoding_failed(struct test_data *d)

static void decoding_failed(const struct test_data *d)

{

	pr_debug("Decoding failed!\n");

	pr_debug("Decoding:  ");

	dump_packet(&d->packet, d->bytes, d->len);

}



static int fail(struct test_data *d, struct intel_pt_pkt *packet, int len,

static int fail(const struct test_data *d, struct intel_pt_pkt *packet, int len,

		enum intel_pt_pkt_ctx new_ctx)

{

	decoding_failed(d);
@@ -242,7 +242,7 @@ static int fail(struct test_data *d, struct intel_pt_pkt *packet, int len, 
	return TEST_FAIL;

}



static int test_ctx_unchanged(struct test_data *d, struct intel_pt_pkt *packet,

static int test_ctx_unchanged(const struct test_data *d, struct intel_pt_pkt *packet,

			      enum intel_pt_pkt_ctx ctx)

{

	enum intel_pt_pkt_ctx old_ctx = ctx;
@@ -258,7 +258,7 @@ static int test_ctx_unchanged(struct test_data *d, struct intel_pt_pkt *packet, 
	return TEST_OK;

}



static int test_one(struct test_data *d)

static int test_one(const struct test_data *d)

{

	struct intel_pt_pkt packet;

	enum intel_pt_pkt_ctx ctx = d->ctx;
@@ -307,7 +307,7 @@ static int test_one(struct test_data *d) 
 */

int test__intel_pt_pkt_decoder(struct test_suite *test __maybe_unused, int subtest __maybe_unused)

{

	struct test_data *d = data;

	const struct test_data *d = data;

	int ret;



	for (d = data; d->len; d++) {