From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751425AbdKUULZ (ORCPT ); Tue, 21 Nov 2017 15:11:25 -0500 Received: from mout.web.de ([212.227.17.12]:63564 "EHLO mout.web.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751067AbdKUULX (ORCPT ); Tue, 21 Nov 2017 15:11:23 -0500 To: linux-kselftest@vger.kernel.org, Frederic Weisbecker , John Stultz , Shuah Khan , Stephen Boyd , Thomas Gleixner Cc: LKML , kernel-janitors@vger.kernel.org, Andrew Morton , Kosaki Motohiro , Olivier Langlois , Peter Zijlstra , Stanislaw Gruszka , Steven Rostedt From: SF Markus Elfring Subject: [PATCH] selftests: posix_timers: Make signal handling safer Message-ID: <7ba0d855-8b55-cb38-2740-c2a3495f3083@users.sourceforge.net> Date: Tue, 21 Nov 2017 21:10:56 +0100 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.4.0 MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Language: en-GB Content-Transfer-Encoding: 7bit X-Provags-ID: V03:K0:KSC5X2jGi6dSEO4NvjNx85stuq2b2euYyBpvmwJI4YRO7kCNcUm 1b5pFGDjTLOEjEOXr4+4DdEDc2I4V5+fBnXZDlfHEp52gKiW3lbbphjin8ucNziuuHwyxm+ ViDi+0tFhfM31nNC0kf8pfYAzfq4Sy7zKzr+l6mxcihO5rUR+lpbkDjSS6UJrRRPM0iEvxV VYCI+UY2MZXWAuzX+EL7w== X-UI-Out-Filterresults: notjunk:1;V01:K0:BSS+GRMhJFM=:+J8kJOW8/cMEMHPRHRuXRY vSTgboCUS9Yn6B8seDuEA7R0vfAzyqTcrxVyy8uzcgGxUQ/n0blV3ukdVucQGsarIGTixFPA1 C97hsIkbmTkxmnepeIJx6Ek8kfq9Z4mUP0YifuRp6xnLoWM/RQWfdC4j/580e7/bB/q6jCR+q 4IgR0r+Rjy6ojD1c6MkVZ2phebgtQjPLp4cofyZDt9Q/tcx86OoiNsYbKk5UHp8aYYj18RVVg LaJq8RYXB5mihqbSy41w9Wumm7DdH9ZwJ4luVCVVtCFYFvxelqjfk/LvqU+nZcFAQ5U8CZ5cs KsVJZzMqn51ziwqrRbaUebePoHG2b2P0iu87JFyDovrDnrwJB1eerqPTgVAGsFKkH8M/j7V+v qMJmWnWVotcs2QPBaNRE4A9fzTdeyOU/VGTZPs+vIyFz/38/FmzpnmzGwebX1FvLAgPuN3+0W OdwaoigqZe8hrpSLRR25BdpA4enN1/JcP5e0u2TRJomvFSHi4/aTEt65CkBtGgpFi6VbJ3sxD QboFZjPMfffR105qZCyIMD126H2coSXndASxrzUR3jNMiYATDOkkbLAtw/F7IViCZJCVmeydU mKWoEiTTQncSqdkBIGITsQtppLtFctDns2A/fu6zUmzzL0kcuJ7H+QN+pHnkGCryQ0XoGJzO+ axOhqiotyuFlcVp5wmxO/PEjnF9UuWbb0LyvH78rQeNzvYeQw+Y7epUI6U4P7X2IsitSQH5Xp E9tJJBtV/u6HZDJeuDQvdZmW7qiZ48jphwmR8J6UyonfFEBhho2VqNb0raHi/EQwc2/XMgfYU UMC+1Qbr7qt23ixw8Y8+2HQyqo1yBzJba5OmJv9FuF7AjXcTvs= Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org From: Markus Elfring Date: Tue, 21 Nov 2017 20:56:51 +0100 Use the data type "sig_atomic_t" for the variable "done" so that it can be safely modified by a signal handler. Fixes: 0bc4b0cf15708fca04095232c4e448634e94d029 ("selftests: add basic posix timers selftests") Signed-off-by: Markus Elfring --- tools/testing/selftests/timers/posix_timers.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tools/testing/selftests/timers/posix_timers.c b/tools/testing/selftests/timers/posix_timers.c index 15cf56d32155..d64732c8b69a 100644 --- a/tools/testing/selftests/timers/posix_timers.c +++ b/tools/testing/selftests/timers/posix_timers.c @@ -20,7 +20,7 @@ #define DELAY 2 #define USECS_PER_SEC 1000000 -static volatile int done; +static sig_atomic_t done; /* Busy loop in userspace to elapse ITIMER_VIRTUAL */ static void user_loop(void) -- 2.15.0