From mboxrd@z Thu Jan 1 00:00:00 1970 From: Thomas Gleixner Date: Tue, 26 May 2009 23:39:53 +0000 Subject: Re: [PATCH] sched: Support current clocksource handling in fallback Message-Id: List-Id: References: <20090526061532.GD9188@linux-sh.org> <63386a3d0905260731m655bfee3q82a6f52d71fa3cef@mail.gmail.com> <1243348681.23657.14.camel@twins> <1243369423.3275.5.camel@localhost> <1243369815.6600.2.camel@laptop> <1243370439.3275.13.camel@localhost> <1243371309.6600.8.camel@laptop> <1243378835.3275.30.camel@localhost> In-Reply-To: <1243378835.3275.30.camel@localhost> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: john stultz Cc: Peter Zijlstra , Linus Walleij , Paul Mundt , Ingo Molnar , Andrew Victor , Haavard Skinnemoen , Andrew Morton , linux-kernel@vger.kernel.org, linux-sh@vger.kernel.org, linux-arm-kernel@lists.arm.linux.org.uk, John Stultz On Tue, 26 May 2009, john stultz wrote: w> > > Is latency all we care about? How bad would > > > the TSC have to be before we wouldn't want to use it? > > > > Anything better than jiffies ;-) > > Except HPET thought, right? :) Eeek. HPET access serializes across cores. That's orders of magnitudes worse than the granularity loss of jiffies. > > For sched_clock() we want something high-res that is monotonic per cpu > > and has a bounded drift between cpus in the order of jiffies. > > > > Look at kernel/sched_clock.c for what we do to make really shitty TSC > > conform to the above requirements. > > Sure, I guess what I'm trying to pull out here is that should we try to > create some OK_FOR_SCHED_CLOCK flag for clocksources, and then we try to > make this generic so other arches can add that flag and be done, what is > the guidance we want to give to arch maintainers for setting that flag? > > 1) Has to be very very fast. Can we put a number on this? 50ns to read? > > 2) How long does it have to be monotonic for? Is it ok if it wraps every > few seconds? > > If get_cycles() || jiffies is what we want, then lets leave it there. I > just want to avoid mixing the clocksource code into the sched clock code > until we really get this sort of definition sorted. The criterion is simple. If arch maintainer decides that the access to the particular clock source is the best compromise between granularity and access speed vs. jiffies then he can express that by setting the flag. Where is the difference between that flag and an arch specific sched_clock() implementation which overrides the weak generic one ? That arch specific function will probably do the same thing: return cyc2ns(whathever_clocksource_the_maintainer_thinks_is_the_best); So let's get rid of those sched_clock() overrrides which do nothing else than the generic version. Thanks, tglx