linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: "Michael Kerrisk" <mtk-manpages@gmx.net>
To: Davide Libenzi <davidel@xmailserver.org>, akpm@linux-foundation.org
Cc: corbet@lwn.net, jengelh@computergmbh.de, hch@lst.de,
	stable@kernel.org, drepper@redhat.com,
	torvalds@linux-foundation.org, linux-kernel@vger.kernel.org,
	tglx@linutronix.de, rdunlap@xenotime.net
Subject: Re: [PATCH] Revised timerfd() interface
Date: Tue, 04 Sep 2007 17:39:40 +0200	[thread overview]
Message-ID: <20070904153940.305730@gmx.net> (raw)
In-Reply-To: <Pine.LNX.4.64.0709040805490.6887@alien.or.mcafeemobile.com>

Hi Davide,

> > <wakes up>
> > 
> > I'd have thought that the existing stuff would be near-useless without
> > the capabilities which you describe?
> 
> Useless like it'd be a motorcycle w/out a cup-holder :)
> Seriously, the ability to get the previous values from "something" could 
> have a meaning if this something is a shared global resource (like 
> signals
> for example). In the timerfd case this makes little sense, since you can 
> create as many timerfd as you like and you do not need to share a single 
> one by changing/restoring the original context.

However, one can have multipe POSIX timers, just as you can 
have multiple timerfd timers; nevertheless POSIX timers provide
the get and get-while-setting functionality.

> On top of that, the cup-holder addition would cost in terms of API 
> clarity

I agree my proposed API is not as clean as it could be, that's why
I would favour:

> (or in terms of two additional system calls in the other case),

Or better still, have timerfd() integrated with POSIX tiemrs (if
this is feasible).  This givesus a simple API, exactly one new
syscall, and all of the functionality of the existing POSIX
timers API.

> and in terms of kernel code footprint.

Not sure what your concern is here.  The total amount of 
new code for all of these options is pretty small.

Cheers,

Michael
-- 
Michael Kerrisk
maintainer of Linux man pages Sections 2, 3, 4, 5, and 7 

Want to help with man page maintenance?  
Grab the latest tarball at
http://www.kernel.org/pub/linux/docs/manpages , 
read the HOWTOHELP file and grep the source 
files for 'FIXME'.


  reply	other threads:[~2007-09-04 15:39 UTC|newest]

Thread overview: 29+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-08-25  6:41 [PATCH] Revised timerfd() interface Michael Kerrisk
2007-08-30 12:01 ` Davide Libenzi
2007-09-04  8:03   ` Michael Kerrisk
2007-09-04  8:18     ` Andrew Morton
2007-09-04  8:24       ` Michael Kerrisk
2007-09-04 15:25       ` Davide Libenzi
2007-09-04 15:39         ` Michael Kerrisk [this message]
2007-09-04 22:41           ` Davide Libenzi
2007-09-04 20:49         ` Michael Kerrisk
2007-09-04 22:44           ` Davide Libenzi
2007-09-05  0:08             ` Michael Kerrisk
2007-09-05 12:02               ` Andrew Morton
2007-09-05 16:14               ` Davide Libenzi
2007-09-05 16:23                 ` Michael Kerrisk
2007-09-05 19:57                   ` Davide Libenzi
2007-09-05 22:50                     ` Michael Kerrisk
2007-09-05 23:45                       ` Davide Libenzi
2007-09-06  6:58                         ` Michael Kerrisk
2007-09-06 23:37                           ` Davide Libenzi
2007-09-10  3:15                             ` Michael Kerrisk
2007-09-05 12:12         ` Denys Vlasenko
2007-09-05 15:32           ` timerfd redux Michael Kerrisk
2007-09-13  2:39             ` Andrew Morton
2007-09-13  6:14               ` Michael Kerrisk
2007-09-13  8:13               ` Michael Kerrisk
2007-09-13  8:20                 ` Andrew Morton
  -- strict thread matches above, loose matches on Subject: below --
2007-07-23  6:32 Problems with timerfd() Michael Kerrisk
2007-07-23  6:38 ` Andrew Morton
2007-07-25 18:18   ` Michael Kerrisk
2007-07-25 22:12     ` Andrew Morton
2007-08-07  6:55       ` Michael Kerrisk
2007-08-09 21:11         ` [PATCH] Revised timerfd() interface Michael Kerrisk
2007-08-13 23:34           ` Randy Dunlap
2007-08-15 14:40           ` Jonathan Corbet

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20070904153940.305730@gmx.net \
    --to=mtk-manpages@gmx.net \
    --cc=akpm@linux-foundation.org \
    --cc=corbet@lwn.net \
    --cc=davidel@xmailserver.org \
    --cc=drepper@redhat.com \
    --cc=hch@lst.de \
    --cc=jengelh@computergmbh.de \
    --cc=linux-kernel@vger.kernel.org \
    --cc=rdunlap@xenotime.net \
    --cc=stable@kernel.org \
    --cc=tglx@linutronix.de \
    --cc=torvalds@linux-foundation.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).