Home » Mailing list archives » linux-kernel » 2007 » August » 3

Re: [PATCH] msleep() with hrtimers

!MAILaRCHIVE_VOTE_RePLACE
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
[view in full thread]
From: Arjan van de Ven <arjan@...>
To: Roman Zippel <zippel@...>
Cc: Jonathan Corbet <corbet@...>, <linux-kernel@...>, Thomas Gleixner <tglx@...>, <akpm@...>, Ingo Molnar <mingo@...>
Subject: Re: [PATCH] msleep() with hrtimers
Date: Friday, August 3, 2007 - 7:53 pm

> 

I respectfully disagree. The power of msleep is that the unit of sleep
time is in the name; so in your proposal it would be hr_msleep or
somesuch. I much rather do the opposite in that case; make the "short"
name be the best implementation of the requested behavior, and have
qualifiers for allowing exceptions to that... least surprise and all
that.
-- 
if you want to mail me at work (you don't), use arjan (at) linux.intel.com
Test the interaction between Linux and your BIOS via http://www.linuxfirmwarekit.org

-
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
Messages in current thread:
[PATCH] msleep() with hrtimers, Jonathan Corbet, (Fri Aug 3, 2:37 pm)
Re: [PATCH] msleep() with hrtimers, Andrew Morton, (Wed Nov 28, 6:29 am)
Re: [PATCH] msleep() with hrtimers, Andrew Morton, (Thu Aug 9, 3:16 am)
Re: [linux-usb-devel] [PATCH] msleep() with hrtimers, Alan Stern, (Thu Aug 9, 11:08 am)
Re: [PATCH] msleep() with hrtimers, Roman Zippel, (Fri Aug 3, 3:19 pm)
Re: [PATCH] msleep() with hrtimers, Arjan van de Ven, (Fri Aug 3, 3:46 pm)
Re: [PATCH] msleep() with hrtimers, Roman Zippel, (Fri Aug 3, 3:58 pm)
Re: [PATCH] msleep() with hrtimers, Arjan van de Ven, (Fri Aug 3, 7:53 pm)
Re: [PATCH] msleep() with hrtimers, Roman Zippel, (Fri Aug 3, 11:00 pm)
Re: [PATCH] msleep() with hrtimers, Arjan van de Ven, (Sat Aug 4, 3:19 pm)
Re: [PATCH] msleep() with hrtimers, Roman Zippel, (Sun Aug 5, 8:09 pm)
Re: [PATCH] msleep() with hrtimers, Arjan van de Ven, (Sun Aug 5, 8:43 pm)
Re: [PATCH] msleep() with hrtimers, Roman Zippel, (Sun Aug 5, 9:03 pm)
Re: [PATCH] msleep() with hrtimers, Arjan van de Ven, (Mon Aug 6, 1:39 am)
Re: [PATCH] msleep() with hrtimers, Roman Zippel, (Mon Aug 6, 6:03 am)
Re: [PATCH] msleep() with hrtimers, Manu Abraham, (Mon Aug 6, 6:20 am)
Re: [PATCH] msleep() with hrtimers, Arjan van de Ven, (Mon Aug 6, 11:53 am)
Re: [PATCH] msleep() with hrtimers, Roman Zippel, (Tue Aug 7, 8:45 am)
Re: [PATCH] msleep() with hrtimers, Arjan van de Ven, (Wed Aug 8, 12:15 am)
Re: [PATCH] msleep() with hrtimers, Denis Vlasenko, (Thu Aug 9, 3:31 pm)
Re: [PATCH] msleep() with hrtimers, Denis Vlasenko, (Thu Aug 9, 4:01 pm)
Re: [PATCH] msleep() with hrtimers, Manu Abraham, (Tue Aug 7, 6:40 am)
Re: [PATCH] msleep() with hrtimers, Andrew Morton, (Tue Aug 7, 3:40 pm)
Re: [PATCH] msleep() with hrtimers, Roman Zippel, (Tue Aug 7, 7:16 pm)
Re: [PATCH] msleep() with hrtimers, Andrew Morton, (Tue Aug 7, 7:29 pm)
Re: [PATCH] msleep() with hrtimers, Roman Zippel, (Tue Aug 7, 11:47 pm)
Re: [PATCH] msleep() with hrtimers, Andrew Morton, (Wed Aug 8, 12:14 am)
Re: [PATCH] msleep() with hrtimers, Roman Zippel, (Thu Aug 9, 6:31 pm)
Re: [PATCH] msleep() with hrtimers, Andi Kleen, (Wed Aug 8, 7:55 am)
Re: [PATCH] msleep() with hrtimers, Manu Abraham, (Wed Aug 8, 7:09 am)
Re: [PATCH] msleep() with hrtimers, Andi Kleen, (Wed Aug 8, 7:52 am)
Re: [PATCH] msleep() with hrtimers, Manu Abraham, (Wed Aug 8, 7:59 am)
Re: [PATCH] msleep() with hrtimers, Ingo Molnar, (Fri Aug 3, 2:54 pm)