linux-kernel - Re: [PATCH v4 4/5] selftests/resctrl: Cleanup properly when an error occurs in CAT test

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite for Android: free password hash cracker in your pocket

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <ce549bc3-7e54-b1d3-7ec5-4cde66bd468f@intel.com>
Date:   Wed, 30 Nov 2022 08:33:51 -0800
From:   Reinette Chatre <reinette.chatre@...el.com>
To:     "Shaopeng Tan (Fujitsu)" <tan.shaopeng@...itsu.com>,
        Fenghua Yu <fenghua.yu@...el.com>,
        Shuah Khan <shuah@...nel.org>
CC:     "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
        "linux-kselftest@...r.kernel.org" <linux-kselftest@...r.kernel.org>,
        "Shuah Khan" <skhan@...uxfoundation.org>
Subject: Re: [PATCH v4 4/5] selftests/resctrl: Cleanup properly when an error
 occurs in CAT test

Hi Shaopeng,

On 11/30/2022 12:32 AM, Shaopeng Tan (Fujitsu) wrote:
 
> Removing ctrl_handler() is only part of the fix in the next version(v5).
> All fixes as follows.
> 
> --- a/tools/testing/selftests/resctrl/cat_test.c
> +++ b/tools/testing/selftests/resctrl/cat_test.c
> @@ -98,12 +98,17 @@ void cat_test_cleanup(void)
>         remove(RESULT_FILE_NAME2);
>  }
> 
> +static void ctrlc_handler_child(int signum, siginfo_t *info, void *ptr)
> +{
> +       exit(EXIT_SUCCESS);
> +}
> +

Could you please elaborate why this is necessary?

Reinette