Re: [PATCH] Add schedule_timeout_{interruptible,uninterruptible}{,_msecs}() interfaces

Hi,

On Sat, 23 Jul 2005, Nishanth Aravamudan wrote:

> > Keep the thing as simple as possible and jiffies _are_ simple. Please show 
> > me the problem, that needs to be fixed.
> 
> I am not sure why jiffies are any simpler than milliseconds. In the
> millisecond case, conversion happens in common code and only needs to be
> audited once. In the jiffies case, I have to check *every* caller to see
> if they are doing stupid math :)

Jiffies are the basic time unit for kernel timers, hiding that fact gives 
users only wrong expectations about them.
I don't mind using helper functions, but constant conversion can already 
happen at compile time and for variable timeouts the user should seriously 
consider using jiffies directly instead of constantly converting time 
values.

bye, Roman
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

---

• Follow-Ups:
  • Re: [PATCH] Add schedule_timeout_{interruptible,uninterruptible}{,_msecs}() interfaces
    • From: Nishanth Aravamudan <[email protected]>

• References:
  • [patch 1/4] drivers/char/ip2/i2lib.c: replace direct assignment with set_current_state()
    • From: [email protected]
  • Re: [patch 1/4] drivers/char/ip2/i2lib.c: replace direct assignment with set_current_state()
    • From: Andrew Morton <[email protected]>
  • [PATCH] Add schedule_timeout_{interruptible,uninterruptible}{,_msecs}() interfaces
    • From: Nishanth Aravamudan <[email protected]>
  • Re: [PATCH] Add schedule_timeout_{interruptible,uninterruptible}{,_msecs}() interfaces
    • From: Arjan van de Ven <[email protected]>
  • Re: [PATCH] Add schedule_timeout_{interruptible,uninterruptible}{,_msecs}() interfaces
    • From: Roman Zippel <[email protected]>
  • Re: [PATCH] Add schedule_timeout_{interruptible,uninterruptible}{,_msecs}() interfaces
    • From: Arjan van de Ven <[email protected]>
  • Re: [PATCH] Add schedule_timeout_{interruptible,uninterruptible}{,_msecs}() interfaces
    • From: Roman Zippel <[email protected]>
  • Re: [PATCH] Add schedule_timeout_{interruptible,uninterruptible}{,_msecs}() interfaces
    • From: Arjan van de Ven <[email protected]>
  • Re: [PATCH] Add schedule_timeout_{interruptible,uninterruptible}{,_msecs}() interfaces
    • From: Roman Zippel <[email protected]>
  • Re: [PATCH] Add schedule_timeout_{interruptible,uninterruptible}{,_msecs}() interfaces
    • From: Nishanth Aravamudan <[email protected]>

• Prev by Date: Re: [PATCH] 2.6.13rc3: RLIMIT_RTPRIO broken
• Next by Date: Re: [linux-pm] Re: [RFT] solve "swsusp plays yoyo" with disks
• Previous by thread: Re: [PATCH] Add schedule_timeout_{interruptible,uninterruptible}{,_msecs}() interfaces
• Next by thread: Re: [PATCH] Add schedule_timeout_{interruptible,uninterruptible}{,_msecs}() interfaces
• Index(es):
  • Date
  • Thread

[Index of Archives]     [Kernel Newbies]     [Netfilter]     [Bugtraq]     [Photo]     [Gimp]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Video 4 Linux]     [Linux for the blind]