summaryrefslogtreecommitdiff
path: root/tools/testing/selftests/powerpc/ptrace/ptrace-tm-spr.c
diff options
context:
space:
mode:
authorBenjamin Gray <bgray@linux.ibm.com>2023-07-25 10:58:39 +1000
committerMichael Ellerman <mpe@ellerman.id.au>2023-08-02 22:22:19 +1000
commit68877ff20a7f4f773069784cfe4f6fe9c7b9a841 (patch)
treeed3e75c33d80cb5f0c115d71d1d26d4b3fba8d88 /tools/testing/selftests/powerpc/ptrace/ptrace-tm-spr.c
parent81d7cac4d11cc65f29be68c72759429d5194347a (diff)
selftests/powerpc/ptrace: Explain why tests are skipped
Many tests require specific hardware features/configurations that a typical machine might not have. As a result, it's common to see a test is skipped. But it is tedious to find out why a test is skipped when all it gives is the file location of the skip macro. Convert SKIP_IF() to SKIP_IF_MSG(), with appropriate descriptions of why the test is being skipped. This gives a general idea of why a test is skipped, which can be looked into further if it doesn't make sense. Signed-off-by: Benjamin Gray <bgray@linux.ibm.com> Signed-off-by: Michael Ellerman <mpe@ellerman.id.au> Link: https://msgid.link/20230725005841.28854-3-bgray@linux.ibm.com
Diffstat (limited to 'tools/testing/selftests/powerpc/ptrace/ptrace-tm-spr.c')
-rw-r--r--tools/testing/selftests/powerpc/ptrace/ptrace-tm-spr.c4
1 files changed, 2 insertions, 2 deletions
diff --git a/tools/testing/selftests/powerpc/ptrace/ptrace-tm-spr.c b/tools/testing/selftests/powerpc/ptrace/ptrace-tm-spr.c
index 880ba6a29a48..e64cdb04cecf 100644
--- a/tools/testing/selftests/powerpc/ptrace/ptrace-tm-spr.c
+++ b/tools/testing/selftests/powerpc/ptrace/ptrace-tm-spr.c
@@ -113,8 +113,8 @@ int ptrace_tm_spr(void)
pid_t pid;
int ret, status;
- SKIP_IF(!have_htm());
- SKIP_IF(htm_is_synthetic());
+ SKIP_IF_MSG(!have_htm(), "Don't have transactional memory");
+ SKIP_IF_MSG(htm_is_synthetic(), "Transactional memory is synthetic");
shm_id = shmget(IPC_PRIVATE, sizeof(struct shared), 0777|IPC_CREAT);
shm_id1 = shmget(IPC_PRIVATE, sizeof(int), 0777|IPC_CREAT);
pid = fork();