Re: [PATCH/RFC] msleep() with hrtimers

Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
From: Jonathan Corbet
Date: Monday, July 16, 2007 - 10:46 am

Roman Zippel <zippel@linux-m68k.org> wrote:


Any additional overhead is clearly small - enough not to disrupt a *high
resolution* timer, after all.  And msleep() is used mostly during
initialization time.  My box had a few hundred calls, almost all during
boot.  Any cost will be bounded by the fact that, well, it sleeps for
milliseconds on every call.

I could run a million-msleep profile if you really want, but I just
can't imagine it's going to have a measurable effect on any real-world
situation.  Is there a specific setup you're worried about?

I'm not *that* attached to this patch; if it causes heartburn we can
just forget it.  But I had thought it might be useful...


drivers/media/video/cafe_ccic.c, and cafe_smbus_write_data() in
particular.  The "hidden problem," though, is that the hardware has
periods where reading the status registers will send it off into its
room where it will hide under its bed and never come out.

I have an alternative which avoids the sleep at the cost of a small,
relatively harmless race; it may be my real solution in the end, we'll
see.


My understanding is that the current dyntick code only turns off the
tick during idle periods; while things are running it's business as
usual.  Perhaps I misunderstood?

jon
-
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
[PATCH/RFC] msleep() with hrtimers, Jonathan Corbet, (Sun Jul 15, 3:42 pm)
Re: [PATCH/RFC] msleep() with hrtimers, Arnd Bergmann, (Sun Jul 15, 4:10 pm)
Re: [PATCH/RFC] msleep() with hrtimers, Ingo Molnar, (Mon Jul 16, 3:48 am)
Re: [PATCH/RFC] msleep() with hrtimers, Roman Zippel, (Mon Jul 16, 4:39 am)
Re: [PATCH/RFC] msleep() with hrtimers, Ingo Molnar, (Mon Jul 16, 4:47 am)
Re: [PATCH/RFC] msleep() with hrtimers, Roman Zippel, (Mon Jul 16, 4:54 am)
Re: [PATCH/RFC] msleep() with hrtimers, Ingo Molnar, (Mon Jul 16, 4:57 am)
Re: [PATCH/RFC] msleep() with hrtimers, Roman Zippel, (Mon Jul 16, 5:05 am)
Re: [PATCH/RFC] msleep() with hrtimers, Ingo Molnar, (Mon Jul 16, 5:19 am)
Re: [PATCH/RFC] msleep() with hrtimers, Roman Zippel, (Mon Jul 16, 6:00 am)
Re: [PATCH/RFC] msleep() with hrtimers, Ingo Molnar, (Mon Jul 16, 7:09 am)
Re: [PATCH/RFC] msleep() with hrtimers, Roman Zippel, (Mon Jul 16, 7:11 am)
Re: [PATCH/RFC] msleep() with hrtimers, Roman Zippel, (Mon Jul 16, 7:32 am)
Re: [PATCH/RFC] msleep() with hrtimers , Jonathan Corbet, (Mon Jul 16, 7:42 am)
Re: [PATCH/RFC] msleep() with hrtimers, Ingo Molnar, (Mon Jul 16, 8:18 am)
Re: [PATCH/RFC] msleep() with hrtimers , Roman Zippel, (Mon Jul 16, 8:43 am)
Re: [PATCH/RFC] msleep() with hrtimers, Ray Lee, (Mon Jul 16, 8:57 am)
Re: [PATCH/RFC] msleep() with hrtimers, Nish Aravamudan, (Mon Jul 16, 9:08 am)
Re: [PATCH/RFC] msleep() with hrtimers, Ingo Molnar, (Mon Jul 16, 9:10 am)
Re: [PATCH/RFC] msleep() with hrtimers, Roman Zippel, (Mon Jul 16, 9:55 am)
Re: [PATCH/RFC] msleep() with hrtimers , Jonathan Corbet, (Mon Jul 16, 10:46 am)
Re: [PATCH/RFC] msleep() with hrtimers, Nick Piggin, (Mon Jul 16, 9:04 pm)
Re: [PATCH/RFC] msleep() with hrtimers, Nish Aravamudan, (Wed Jul 18, 10:53 am)
Re: [PATCH/RFC] msleep() with hrtimers , Roman Zippel, (Fri Jul 20, 5:49 am)