Re: [PATCH 4/5] select: make select() use schedule_hrtimeout()

!MAILaRCHIVE_VOTE_RePLACE
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
To: Alan Cox <alan@...>
Cc: Arjan van de Ven <arjan@...>, <linux-kernel@...>, <mingo@...>, <tglx@...>
Date: Friday, August 29, 2008 - 2:46 pm

On Fri, 29 Aug 2008, Alan Cox wrote:

Actually, I disagree.

They are _both_ of the exact same type: the precision depends on how long 
the timeout is.

TCP retransmit timers are a perfect example. If the timeout is long (which 
is quite common if you end up having multiple retransmits), you _really_ 
don't care about how precise it is.

And keepalives are 'yeah whatever' exactly because they are so long, not 
because they are inherently uninteresting per se.

I suspect you could find some kernel-generated timer that doesn't fit that 
pattern, but I can't really think of any.


And yes, I do agree that the heuristic could well involve other 
characteristics of the process in question. And probably characteristics 
of the machine itself (ie some general kind of "power mode" where timers 
are simply not considered critical if you want to be in low-power mode).

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

Messages in current thread:
[patch 0/5] Nano/Microsecond resolution for select() and pol..., Arjan van de Ven, (Fri Aug 29, 11:05 am)
Re: [patch 0/5] Nano/Microsecond resolution for select() and..., Arjan van de Ven, (Fri Aug 29, 12:12 pm)
[PATCH 1/5] select: add a timespec version of the timeout to..., Arjan van de Ven, (Fri Aug 29, 11:06 am)
[PATCH 5/5] select: make poll() use schedule_hrtimeout() as ..., Arjan van de Ven, (Fri Aug 29, 11:08 am)
[PATCH 4/5] select: make select() use schedule_hrtimeout(), Arjan van de Ven, (Fri Aug 29, 11:08 am)
Re: [PATCH 4/5] select: make select() use schedule_hrtimeout(), Arjan van de Ven, (Fri Aug 29, 12:30 pm)
Re: [PATCH 4/5] select: make select() use schedule_hrtimeout(), Linus Torvalds, (Fri Aug 29, 2:46 pm)
Re: [PATCH 4/5] select: make select() use schedule_hrtimeout(), Arjan van de Ven, (Sat Aug 30, 11:25 am)
[PATCH 3/5] select: introduce a schedule_hrtimeout() function, Arjan van de Ven, (Fri Aug 29, 11:07 am)
[PATCH 2/5] select: return accurate remainer in select() and..., Arjan van de Ven, (Fri Aug 29, 11:07 am)