linux-kernel - Re: [tip:timers/core] kselftests: timers: set-timer-lat: Tweak reporting when timer fires early

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: <CALAqxLU+TyaM=Qhtgo4rvNx1HfbR_rTZ4sZZ5DmQnjoykyMB4g@mail.gmail.com>
Date:   Tue, 19 Sep 2017 20:36:21 -0700
From:   John Stultz <john.stultz@...aro.org>
To:     Shuah Khan <shuah@...nel.org>
Cc:     lkml <linux-kernel@...r.kernel.org>,
        Richard Cochran <richardcochran@...il.com>,
        "H. Peter Anvin" <hpa@...or.com>,
        Thomas Gleixner <tglx@...utronix.de>,
        Ingo Molnar <mingo@...nel.org>,
        Prarit Bhargava <prarit@...hat.com>,
        Stephen Boyd <stephen.boyd@...aro.org>,
        Greg Hackmann <ghackmann@...gle.com>,
        Miroslav Lichvar <mlichvar@...hat.com>,
        linux-tip-commits@...r.kernel.org,
        Shuah Khan <shuahkh@....samsung.com>
Subject: Re: [tip:timers/core] kselftests: timers: set-timer-lat: Tweak
 reporting when timer fires early

On Tue, Sep 19, 2017 at 6:02 PM, Shuah Khan <shuah@...nel.org> wrote:
> Hi Greg,
>
> On 08/20/2017 03:56 AM, tip-bot for Greg Hackmann wrote:
>> Commit-ID:  a524b1184b8e86141d689fa78ad150fbf2db9b4c
>> Gitweb:     http://git.kernel.org/tip/a524b1184b8e86141d689fa78ad150fbf2db9b4c
>> Author:     Greg Hackmann <ghackmann@...gle.com>
>> AuthorDate: Tue, 25 Jul 2017 14:36:25 -0700
>> Committer:  John Stultz <john.stultz@...aro.org>
>> CommitDate: Thu, 17 Aug 2017 12:14:54 -0700
>>
>> kselftests: timers: set-timer-lat: Tweak reporting when timer fires early
>>
>> Rather than printing an error inside the alarm signal handler, set a
>> flag that we check later.  This keeps the test from spamming the console
>> every time the alarm fires early.  It also fixes the test exiting with
>> error code 0 if this was the only test failure.
>>
>> Cc: Thomas Gleixner <tglx@...utronix.de>
>> Cc: Ingo Molnar <mingo@...nel.org>
>> Cc: Miroslav Lichvar <mlichvar@...hat.com>
>> Cc: Richard Cochran <richardcochran@...il.com>
>> Cc: Prarit Bhargava <prarit@...hat.com>
>> Cc: Stephen Boyd <stephen.boyd@...aro.org>
>> Cc: Shuah Khan <shuah@...nel.org>
>> Cc: linux-kselftest@...r.kernel.org
>> Signed-off-by: Greg Hackmann <ghackmann@...gle.com>
>> Signed-off-by: John Stultz <john.stultz@...aro.org>
>
> This commit is making the set-timer-lat hang. I reverted this commit
> and there is no hang. Could you please take a look and see if this
> commit should be reverted.
>
> make kselftest as well as make -C tools/testing/selftests/timers run_tests
> hang.

Do you have any specific details as to how the test hangs? (ie: Log
data, as in where it might be when it hangs? Any details about the
machine?)

I'm not seeing it so far in my testing.

thanks
-john