mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: Davide Libenzi <davidel@xmailserver.org>
To: Andrew Morton <akpm@linux-foundation.org>
Cc: Michael Kerrisk <mtk-manpages@gmx.net>,
	rdunlap@xenotime.net, tglx@linutronix.de,
	Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
	Linus Torvalds <torvalds@linux-foundation.org>,
	Ulrich Drepper <drepper@redhat.com>,
	stable@kernel.org, hch@lst.de, jengelh@computergmbh.de,
	corbet@lwn.net
Subject: Re: [PATCH] Revised timerfd() interface
Date: Tue, 4 Sep 2007 08:25:34 -0700 (PDT)	[thread overview]
Message-ID: <Pine.LNX.4.64.0709040805490.6887@alien.or.mcafeemobile.com> (raw)
In-Reply-To: <20070904011800.762523a4.akpm@linux-foundation.org>

On Tue, 4 Sep 2007, Andrew Morton wrote:

> > On Tue, 04 Sep 2007 10:03:56 +0200 Michael Kerrisk <mtk-manpages@gmx.net> wrote:
> > Davide,
> > 
> > >> Davide -- ping!  Can you please offer your comments about this change, and
> > >> also thoughts on Jon's and my comments about a more radical API change
> > >> later  in this thread.
> > > 
> > > IMO the complexity of the resulting API (and resulting patch), and the ABI 
> > > change, is not justified by the added value.
> > 
> > Neither of the proposed APIs (either my multiplexed version of timerfd()
> > or Jon's/my idea of using three system calls (like POSIX timers), or
> > the notion of timerfd() integrated with POSIX timers) is more
> > complicated than the existing POSIX timers API.
> > 
> > The ABI change doesn't really matter, since timerfd() was broken in 2.6.22
> > anyway.
> > 
> > Both previous APIs provided the features I have described provide:
> > 
> > * the ability to fetch the old timer value when applying
> >   a new setting
> > 
> > * the ability to non-destructively fetch the amount of time remaining
> >   on a timer.
> > 
> > This is clearly useful for timers -- but you have not explained why
> > you think this is not necessary for timerfd timers.
> 
> <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.
On top of that, the cup-holder addition would cost in terms of API clarity 
(or in terms of two additional system calls in the other case), and in 
terms of kernel code footprint. Costs that IMO are not balanced by the 
added values.



- Davide



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

Thread overview: 29+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-08-25  6:41 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 [this message]
2007-09-04 15:39         ` Michael Kerrisk
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=Pine.LNX.4.64.0709040805490.6887@alien.or.mcafeemobile.com \
    --to=davidel@xmailserver.org \
    --cc=akpm@linux-foundation.org \
    --cc=corbet@lwn.net \
    --cc=drepper@redhat.com \
    --cc=hch@lst.de \
    --cc=jengelh@computergmbh.de \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mtk-manpages@gmx.net \
    --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

all inboxes | Powered by JetHome®