linux-kernel - Re: [ANNOUNCE] v4.11.12-rt13

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: Windows password security audit tool. GUI, reports in PDF.

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

Date:   Tue, 10 Oct 2017 18:24:55 +0200
From:   Sebastian Andrzej Siewior <bigeasy@...utronix.de>
To:     Mike Galbraith <efault@....de>
Cc:     Thomas Gleixner <tglx@...utronix.de>,
        LKML <linux-kernel@...r.kernel.org>,
        linux-rt-users <linux-rt-users@...r.kernel.org>,
        Anna-Maria Gleixner <anna-maria@...utronix.de>,
        Steven Rostedt <rostedt@...dmis.org>
Subject: Re: [ANNOUNCE] v4.11.12-rt13

On 2017-10-07 06:50:53 [+0200], Mike Galbraith wrote:
> > However...
> 
> testcases/open_posix_testsuite/conformance/interfaces/clock_settime/8-1.c:
>  * Steps:
>  * - get time T0
>  * - in child:  set clock_nanosleep() to sleep for SLEEPSEC seconds
>  * - in parent:  sleep SMALLTIME (< SLEEPSEC)
>  * - in parent:  set time back to T0
>  * - in child:  ensure time when clock_nanosleep() expires is within
>  *   ACCEPTABLEDELTA of T0+(SLEEPSEC-SMALLTIME)
> ...
> #define SLEEPSEC 5
> #define SMALLTIME 2
> #define ACCEPTABLEDELTA 1
> 
> homer:..debug/tracing # time chrt -o 0 /usr/local/ltp/conformance/interfaces/clock_settime/clock_settime_8-1.run-test
> Test PASSED
> 
> real    0m5.002s
> user    0m0.000s
> sys     0m0.001s
> homer:..debug/tracing # time chrt -f 1 /usr/local/ltp/conformance/interfaces/clock_settime/clock_settime_8-1.run-test
> Ended too late.  1507351636 >> 1507351634
> Test FAILED
> 
> real    0m7.002s
> user    0m0.000s
> sys     0m0.002s

As per POSIX we should sleep only 5 secs despite the CLOCK_REALTIME
change. In RT case however we sleep 7 so we somehow account it which is
wrong.

Sebastian