lkml.org 
[lkml]   [2019]   [Dec]   [11]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH AUTOSEL 5.4 038/134] selftests/powerpc: Skip tm-signal-sigreturn-nt if TM not available
    Date
    From: Michael Ellerman <mpe@ellerman.id.au>

    [ Upstream commit 505127068d9b705a6cf335143239db91bfe7bbe2 ]

    On systems where TM (Transactional Memory) is disabled the
    tm-signal-sigreturn-nt test causes a SIGILL:

    test: tm_signal_sigreturn_nt
    tags: git_version:7c202575ef63
    !! child died by signal 4
    failure: tm_signal_sigreturn_nt

    We should skip the test if TM is not available.

    Fixes: 34642d70ac7e ("selftests/powerpc: Add checks for transactional sigreturn")
    Signed-off-by: Michael Ellerman <mpe@ellerman.id.au>
    Link: https://lore.kernel.org/r/20191104233524.24348-1-mpe@ellerman.id.au
    Signed-off-by: Sasha Levin <sashal@kernel.org>
    ---
    tools/testing/selftests/powerpc/tm/tm-signal-sigreturn-nt.c | 4 ++++
    1 file changed, 4 insertions(+)

    diff --git a/tools/testing/selftests/powerpc/tm/tm-signal-sigreturn-nt.c b/tools/testing/selftests/powerpc/tm/tm-signal-sigreturn-nt.c
    index 56fbf9f6bbf30..07c388147b75e 100644
    --- a/tools/testing/selftests/powerpc/tm/tm-signal-sigreturn-nt.c
    +++ b/tools/testing/selftests/powerpc/tm/tm-signal-sigreturn-nt.c
    @@ -10,10 +10,12 @@
    */

    #define _GNU_SOURCE
    +#include <stdio.h>
    #include <stdlib.h>
    #include <signal.h>

    #include "utils.h"
    +#include "tm.h"

    void trap_signal_handler(int signo, siginfo_t *si, void *uc)
    {
    @@ -29,6 +31,8 @@ int tm_signal_sigreturn_nt(void)
    {
    struct sigaction trap_sa;

    + SKIP_IF(!have_htm());
    +
    trap_sa.sa_flags = SA_SIGINFO;
    trap_sa.sa_sigaction = trap_signal_handler;

    --
    2.20.1
    \
     
     \ /
      Last update: 2019-12-11 17:08    [W:4.334 / U:0.016 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site