Commit 71b634ab authored by Zeng Heng's avatar Zeng Heng Committed by Will Deacon
Browse files

kselftest/arm64: add DEF_SIGHANDLER_FUNC() and DEF_INST_RAISE_SIG() helpers 



Add macro definition functions DEF_SIGHANDLER_FUNC() and
DEF_INST_RAISE_SIG() helpers.

Furthermore, there is no need to modify the default SIGILL handling
function throughout the entire testing lifecycle in the main() function.
It is reasonable to narrow the scope to the context of the sig_fn
function only.

This is a pre-patch for the subsequent SIGBUS handler patch.

Signed-off-by: default avatarZeng Heng <zengheng4@huawei.com>
Reviewed-by: default avatarMark Brown <broonie@kernel.org>
Link: https://lore.kernel.org/r/20230808134036.668954-4-zengheng4@huawei.com


Signed-off-by: default avatarWill Deacon <will@kernel.org>
parent 09d2e95a

tools/testing/selftests/arm64/abi/hwcap.c

+75 −43
Original line number Diff line number Diff line
@@ -438,18 +438,21 @@ static const struct hwcap_data { 
	},

};



static bool seen_sigill;

typedef void (*sighandler_fn)(int, siginfo_t *, void *);



static void handle_sigill(int sig, siginfo_t *info, void *context)

{

	ucontext_t *uc = context;



	seen_sigill = true;



	/* Skip over the offending instruction */

	uc->uc_mcontext.pc += 4;

#define DEF_SIGHANDLER_FUNC(SIG, NUM)					\

static bool seen_##SIG;							\

static void handle_##SIG(int sig, siginfo_t *info, void *context)	\

{									\

	ucontext_t *uc = context;					\

									\

	seen_##SIG = true;						\

	/* Skip over the offending instruction */			\

	uc->uc_mcontext.pc += 4;					\

}



DEF_SIGHANDLER_FUNC(sigill, SIGILL);



bool cpuinfo_present(const char *name)

{

	FILE *f;
@@ -492,25 +495,77 @@ bool cpuinfo_present(const char *name) 
	return false;

}



int main(void)

static int install_sigaction(int signum, sighandler_fn handler)

{

	const struct hwcap_data *hwcap;

	int i, ret;

	bool have_cpuinfo, have_hwcap;

	int ret;

	struct sigaction sa;



	ksft_print_header();

	ksft_set_plan(ARRAY_SIZE(hwcaps) * TESTS_PER_HWCAP);



	memset(&sa, 0, sizeof(sa));

	sa.sa_sigaction = handle_sigill;

	sa.sa_sigaction = handler;

	sa.sa_flags = SA_RESTART | SA_SIGINFO;

	sigemptyset(&sa.sa_mask);

	ret = sigaction(SIGILL, &sa, NULL);

	ret = sigaction(signum, &sa, NULL);

	if (ret < 0)

		ksft_exit_fail_msg("Failed to install SIGILL handler: %s (%d)\n",

				   strerror(errno), errno);



	return ret;

}



static void uninstall_sigaction(int signum)

{

	if (sigaction(signum, NULL, NULL) < 0)

		ksft_exit_fail_msg("Failed to uninstall SIGILL handler: %s (%d)\n",

				   strerror(errno), errno);

}



#define DEF_INST_RAISE_SIG(SIG, NUM)					\

static bool inst_raise_##SIG(const struct hwcap_data *hwcap,		\

				bool have_hwcap)			\

{									\

	if (!hwcap->SIG##_fn) {						\

		ksft_test_result_skip(#SIG"_%s\n", hwcap->name);	\

		/* assume that it would raise exception in default */	\

		return true;						\

	}								\

									\

	install_sigaction(NUM, handle_##SIG);				\

									\

	seen_##SIG = false;						\

	hwcap->SIG##_fn();						\

									\

	if (have_hwcap) {						\

		/* Should be able to use the extension */		\

		ksft_test_result(!seen_##SIG,				\

				#SIG"_%s\n", hwcap->name);		\

	} else if (hwcap->SIG##_reliable) {				\

		/* Guaranteed a SIGNAL */				\

		ksft_test_result(seen_##SIG,				\

				#SIG"_%s\n", hwcap->name);		\

	} else {							\

		/* Missing SIGNAL might be fine */			\

		ksft_print_msg(#SIG"_%sreported for %s\n",		\

				seen_##SIG ? "" : "not ",		\

				hwcap->name);				\

		ksft_test_result_skip(#SIG"_%s\n",			\

					hwcap->name);			\

	}								\

									\

	uninstall_sigaction(NUM);					\

	return seen_##SIG;						\

}



DEF_INST_RAISE_SIG(sigill, SIGILL);



int main(void)

{

	int i;

	const struct hwcap_data *hwcap;

	bool have_cpuinfo, have_hwcap;



	ksft_print_header();

	ksft_set_plan(ARRAY_SIZE(hwcaps) * TESTS_PER_HWCAP);



	for (i = 0; i < ARRAY_SIZE(hwcaps); i++) {

		hwcap = &hwcaps[i];
@@ -523,30 +578,7 @@ int main(void) 
		ksft_test_result(have_hwcap == have_cpuinfo,

				 "cpuinfo_match_%s\n", hwcap->name);



		if (hwcap->sigill_fn) {

			seen_sigill = false;

			hwcap->sigill_fn();



			if (have_hwcap) {

				/* Should be able to use the extension */

				ksft_test_result(!seen_sigill, "sigill_%s\n",

						 hwcap->name);

			} else if (hwcap->sigill_reliable) {

				/* Guaranteed a SIGILL */

				ksft_test_result(seen_sigill, "sigill_%s\n",

						 hwcap->name);

			} else {

				/* Missing SIGILL might be fine */

				ksft_print_msg("SIGILL %sreported for %s\n",

					       seen_sigill ? "" : "not ",

					       hwcap->name);

				ksft_test_result_skip("sigill_%s\n",

						      hwcap->name);

			}

		} else {

			ksft_test_result_skip("sigill_%s\n",

					      hwcap->name);

		}

		inst_raise_sigill(hwcap, have_hwcap);

	}



	ksft_print_cnts();