lkml.org 
[lkml]   [2016]   [Jun]   [23]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: [LTP] [patch V2 00/20] timer: Refactor the timer wheel
    On Wed, 22 Jun 2016, Cyril Hrubis wrote:
    > Hi!
    > > > rtbox:~ # /usr/local/ltp/conformance/interfaces/sigtimedwait/sigtimedwait_1-1.run-test
    > > > Test FAILED: sigtimedwait() did not return in the required time
    > > > time_elapsed: 1.197057
    > > > ...come on, you can do it...
    > > > rtbox:~ # /usr/local/ltp/conformance/interfaces/sigtimedwait/sigtimedwait_1-1.run-test
    > > > Test PASSED
    > > >
    > > > #define ERRORMARGIN 0.1
    > > > ...
    > > > if ((time_elapsed > SIGTIMEDWAITSEC + ERRORMARGIN)
    > > > || (time_elapsed < SIGTIMEDWAITSEC - ERRORMARGIN)) {
    > > > printf("Test FAILED: sigtimedwait() did not return in "
    > > > "the required time\n");
    > > > printf("time_elapsed: %lf\n", time_elapsed);
    > > > return PTS_FAIL;
    > > > }
    > > >
    > > > Looks hohum to me, but gripe did arrive with patch set, so you get a note.
    > >
    > > hohum is a euphemism. That's completely bogus.
    > >
    > > The only guarantee a syscall with timers has is: timer does not fire early.
    >
    > While this is true, checking with reasonable error margin works just
    > fine 99% of the time. You cannot really test that timer expires, without
    > setting arbitrary margin.

    Err. You know that the timer expired because sigtimedwait() returns
    EAGAIN. And the only thing you can reliably check for is that the timer did
    not expired to early. Anything else is guesswork and voodoo programming.

    Thanks,

    tglx


    \
     
     \ /
      Last update: 2016-06-23 11:01    [W:4.191 / U:0.064 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site