From: Ingo Molnar <mingo@elte.hu>
To: Roman Zippel <zippel@linux-m68k.org>
Cc: Jonathan Corbet <corbet@lwn.net>,
linux-kernel@vger.kernel.org,
Thomas Gleixner <tglx@linutronix.de>
Subject: Re: [PATCH/RFC] msleep() with hrtimers
Date: Mon, 16 Jul 2007 13:47:34 +0200 [thread overview]
Message-ID: <20070716114734.GA12010@elte.hu> (raw)
In-Reply-To: <Pine.LNX.4.64.0707161325040.1818@scrub.home>
* Roman Zippel <zippel@linux-m68k.org> wrote:
> > Here's another approach: a reimplementation of msleep() and
> > msleep_interruptible() using hrtimers. On a system without real
> > hrtimers this code will at least drop down to single-jiffy delays
> > much of the time (though not deterministically so). On my x86_64
> > system with Thomas's hrtimer/dyntick patch applied, msleep(1) gives
> > almost exactly what was asked for.
>
> One possible problem here is that setting up that timer can be
> considerably more expensive, for a relative timer you have to read the
> current time, which can be quite expensive (e.g. your machine now uses
> the PIT timer, because TSC was deemed unstable).
i dont think there's any significant overhead. The OLPC folks are pretty
sensitive to performance, so if there was any genuine measurable
overhead due to this, i'd expect them to report it. And even if there
_was_ overhead, it would be well worth its price, the legacies of HZ are
clearly biting the OLPC project here. The sooner we get rid of HZ
dependencies and HZ artifacts, the better.
Ingo
next prev parent reply other threads:[~2007-07-16 11:47 UTC|newest]
Thread overview: 24+ messages / expand[flat|nested] mbox.gz Atom feed top
2007-07-15 22:42 [PATCH/RFC] msleep() with hrtimers Jonathan Corbet
2007-07-15 23:10 ` Arnd Bergmann
2007-07-16 10:48 ` Ingo Molnar
2007-07-16 11:39 ` Roman Zippel
2007-07-16 11:47 ` Ingo Molnar [this message]
2007-07-16 11:54 ` Roman Zippel
2007-07-16 11:57 ` Ingo Molnar
2007-07-16 12:05 ` Roman Zippel
2007-07-16 12:19 ` Ingo Molnar
2007-07-16 13:00 ` Roman Zippel
2007-07-16 14:09 ` Ingo Molnar
2007-07-16 14:32 ` Roman Zippel
2007-07-16 14:42 ` Jonathan Corbet
2007-07-16 15:18 ` Ingo Molnar
2007-07-16 15:43 ` Roman Zippel
2007-07-16 15:57 ` Ray Lee
2007-07-16 16:08 ` Nish Aravamudan
2007-07-17 4:04 ` Nick Piggin
2007-07-18 17:53 ` Nish Aravamudan
2007-07-16 16:10 ` Ingo Molnar
2007-07-16 16:55 ` Roman Zippel
2007-07-16 17:46 ` Jonathan Corbet
2007-07-20 12:49 ` Roman Zippel
2007-07-16 14:11 ` Roman Zippel
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20070716114734.GA12010@elte.hu \
--to=mingo@elte.hu \
--cc=corbet@lwn.net \
--cc=linux-kernel@vger.kernel.org \
--cc=tglx@linutronix.de \
--cc=zippel@linux-m68k.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox