kselftest/arm64: signal: Skip tests if required features are missing
[ Upstream commit 0e3dbf765fe22060acbcb8eb8c4d256e655a1247 ]
During initialization of a signal testcase, features declared as required
are properly checked against the running system but no action is then taken
to effectively skip such a testcase.
Fix core signals test logic to abort initialization and report such a
testcase as skipped to the KSelfTest framework.
Fixes: f96bf43403
("kselftest: arm64: mangle_pstate_invalid_compat_toggle and common utils")
Signed-off-by: Cristian Marussi <cristian.marussi@arm.com>
Reviewed-by: Mark Brown <broonie@kernel.org>
Link: https://lore.kernel.org/r/20210920121228.35368-1-cristian.marussi@arm.com
Signed-off-by: Catalin Marinas <catalin.marinas@arm.com>
Signed-off-by: Sasha Levin <sashal@kernel.org>
This commit is contained in:

committed by
Greg Kroah-Hartman

parent
91d4da33c3
commit
3aa50241e1
@@ -266,16 +266,19 @@ int test_init(struct tdescr *td)
|
|||||||
td->feats_supported |= FEAT_SSBS;
|
td->feats_supported |= FEAT_SSBS;
|
||||||
if (getauxval(AT_HWCAP) & HWCAP_SVE)
|
if (getauxval(AT_HWCAP) & HWCAP_SVE)
|
||||||
td->feats_supported |= FEAT_SVE;
|
td->feats_supported |= FEAT_SVE;
|
||||||
if (feats_ok(td))
|
if (feats_ok(td)) {
|
||||||
fprintf(stderr,
|
fprintf(stderr,
|
||||||
"Required Features: [%s] supported\n",
|
"Required Features: [%s] supported\n",
|
||||||
feats_to_string(td->feats_required &
|
feats_to_string(td->feats_required &
|
||||||
td->feats_supported));
|
td->feats_supported));
|
||||||
else
|
} else {
|
||||||
fprintf(stderr,
|
fprintf(stderr,
|
||||||
"Required Features: [%s] NOT supported\n",
|
"Required Features: [%s] NOT supported\n",
|
||||||
feats_to_string(td->feats_required &
|
feats_to_string(td->feats_required &
|
||||||
~td->feats_supported));
|
~td->feats_supported));
|
||||||
|
td->result = KSFT_SKIP;
|
||||||
|
return 0;
|
||||||
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
/* Perform test specific additional initialization */
|
/* Perform test specific additional initialization */
|
||||||
|
Reference in New Issue
Block a user