[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <Pine.LNX.4.64.0703101230010.29503@alien.or.mcafeemobile.com>
Date: Sat, 10 Mar 2007 12:41:35 -0800 (PST)
From: Davide Libenzi <davidel@...ilserver.org>
To: Nicholas Miell <nmiell@...cast.net>
cc: Linux Kernel Mailing List <linux-kernel@...r.kernel.org>,
Andrew Morton <akpm@...ux-foundation.org>,
Linus Torvalds <torvalds@...ux-foundation.org>
Subject: Re: [patch 6/9] signalfd/timerfd v1 - timerfd core ...
On Sat, 10 Mar 2007, Nicholas Miell wrote:
> Try reading the timer_create man page.
>
> In short, you're limited to a single clock, so you can't set timers
> based on wall-clock time (subject to NTP correction), monotomic time
> (not subject to NTP, will not ever go backwards or skip ticks), the
> high-res versions of the previous two clocks, per-thread or per-process
> CPU usage time, or any other clocks that may get introduced in the
> future.
One timer per fd yes. So?
The real-time and monotonic selection can be added.
If you look at the posix timers code, that's a bunch of code over the real
meat of it, that is hrtimer.c. The timerfd interface goes straight to
that, without adding yet another meaning to the sigevent structure, and
yet another case inside the posix timers trigger functions. That will be
as unstandard as timerfd is, and even more, since you cannot use that
interface and hope to be portable in any case.
On top of that, handing over files to the posix timers will creates
problems with references kept around.
The timerfd code is just a *really* thin layer (if you exclude the
includes, the structure definitions and the fd setup code, there's
basically *nothing*) over hrtimer.c and does not mess up with other kernel
code in any way, and offers the same functionalities. I'd like to keep it
that way.
- Davide
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists