From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932332AbdJJR1N convert rfc822-to-8bit (ORCPT ); Tue, 10 Oct 2017 13:27:13 -0400 Received: from mout.gmx.net ([212.227.15.18]:50452 "EHLO mout.gmx.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1756543AbdJJR1K (ORCPT ); Tue, 10 Oct 2017 13:27:10 -0400 Message-ID: <1507656420.15259.6.camel@gmx.de> Subject: Re: [ANNOUNCE] v4.11.12-rt13 From: Mike Galbraith To: Sebastian Andrzej Siewior Cc: Thomas Gleixner , LKML , linux-rt-users , Anna-Maria Gleixner , Steven Rostedt Date: Tue, 10 Oct 2017 19:27:00 +0200 In-Reply-To: <20171010164749.apnxbh5kkvkgrmwe@linutronix.de> References: <20170905145244.jro4jywfxugmn7eh@linutronix.de> <1507133279.6934.66.camel@gmx.de> <20171005155441.72mjaac5c42rjilf@linutronix.de> <1507256431.6597.16.camel@gmx.de> <20171006102820.ou4wpm56ed6m3ewr@linutronix.de> <1507296802.6364.14.camel@gmx.de> <1507311488.5480.20.camel@gmx.de> <1507351853.12111.5.camel@gmx.de> <20171010162455.qmvgr476j36nqev7@linutronix.de> <20171010164749.apnxbh5kkvkgrmwe@linutronix.de> Content-Type: text/plain; charset="UTF-8" X-Mailer: Evolution 3.20.5 Mime-Version: 1.0 Content-Transfer-Encoding: 8BIT X-Provags-ID: V03:K0:9WrFiAHYTC0ObMNm9S47dC9FmD31/ePuFCB6kSV1kc006+WfVDV 8Bf2i9l4j0ZWg4O8Et1qFkn0oEB5Jg5Mnhb9/vBV5G+/cbSN+R3xWBrumre4ir9MQ8+9Dzl gsTpiMxkVwtKR73S7MWbp7r5PuOIHBkdzJfO1eDVP5ABSnCcH+RXEPyAsn4zC9x3oDnu9tO Q3b91jPG4RwjNY+eXy+nw== X-UI-Out-Filterresults: notjunk:1;V01:K0:8mjqS7jyp/I=:K6IEmFhrnHWFpI1sh2pfki +47TcMun535ElEmKfkt5CatxebmLt4ph+joJdohOEStbpCgNDw0Dt3kWpDsT5SBQaCK722pQe 4tA8UpPwe+Z8pYx5nx2aMzk7V0fFCYFCiua51K9NNPGYUclMO5kba8OIUoJQaxwjcxitE9gNK n5ZjQRjZjJ+9joHpAvMdjTtLjsaGcDx2XMuUepBX1QXBnD4ia2FzmEUen2m/3DvKimLiruaSm M7ZW99eWvVpUU0hGj+vAAhONBugIl9LYsCLrw6oKKYbEDW8u7Vv9jpMrtEd3JVSA3cedFsxLe UG/1a7Z4a6rFoSn+XGqJXwUQRHNSk4g7+AcEshylpA26Dc/vchcUaw5pvlSBJHuw/OxV5+SMU 5WvUDsf0nJ0YyUzoZz+5iM0cEkTBMeUpDrjxpbQcN0im/VYI6TFCl7JsveSHdtMCqmhqCxQRT QIvcqURRZ24i3AexKiGxG2mkSIjheNaDiC6FhKFtl8QqCv/+FgJPb9C+oJ9KbLNVOLIs3rph5 arHH5fdC3w1Tuud0THdgsq0KM1Tjn+G4VBNIyqeZLKQvVlvMlprO3WWQXoC8ISKCSPnzA5QOA 3s1NTGzS2KjdQUYzgyWGJnHVzAcUUbyfeDmwlvN2zR33dfB/ud1uwod/78fl/8Ml+VPEerao/ L1f8Z1R9jPBqNLEafmPLlutBuloevBR1hYl5eonQep0j+HZbisZtwvfs2uHfwntzd9Qj0xqWu 5L310BaPi9X5fDTGuwISeC1Gj7FGUgfKS9D+DPEqO66WFpvjaS0RO6JzvEqL8hb0A0RB1Pi/E vD+i0siL3x7kPTsV+NMx/CHlcClLfBXUXQKXN9i5ssfOhIRdDs= Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org 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