On Tue, 2017-10-10 at 18:47 +0200, Sebastian Andrzej Siewior wrote: > On 2017-10-10 18:24:55 [+0200], To Mike Galbraith wrote: > > > 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. > > This should cure it: Yup, nailed it. I hadn't gotten around to chasing this one (4 weeks vacation === baaaackloooog), but did do the promised full ltp run with fixes applied, and there were no shiny new failures. So tree should now be perfect.. modulo the pile of bugs nobody has as yet noticed :) > diff --git a/kernel/time/hrtimer.c b/kernel/time/hrtimer.c > --- a/kernel/time/hrtimer.c > +++ b/kernel/time/hrtimer.c > @@ -1258,6 +1258,8 @@ static void __hrtimer_init(struct hrtimer *timer, clockid_t clock_id, > clock_id = CLOCK_MONOTONIC; > else if (clock_id == CLOCK_REALTIME_SOFT) > clock_id = CLOCK_MONOTONIC_SOFT; > + else if (clock_id == CLOCK_REALTIME_HARD) > + clock_id = CLOCK_MONOTONIC; > } > > base = hrtimer_clockid_to_base(clock_id); > > Sebastian -- To unsubscribe from this list: send the line "unsubscribe linux-rt-users" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
