From: Thomas Gleixner <tglx@linutronix.de>
To: Michael Kerrisk <mtk-manpages@gmx.net>
Cc: Lee.Schermerhorn@hp.com, torvalds@linux-foundation.org,
vda.linux@googlemail.com, rdunlap@xenotime.net, corbet@lwn.net,
hch@lst.de, akpm@linux-foundation.org,
linux-kernel@vger.kernel.org, geoff@gclare.org.uk,
drepper@redhat.com, davidel@xmailserver.org,
"David Härdeman" <david@hardeman.nu>
Subject: Re: RFC: A revised timerfd API
Date: Tue, 18 Sep 2007 11:42:22 +0200 [thread overview]
Message-ID: <1190108542.2995.112.camel@chaos> (raw)
In-Reply-To: <20070918093007.223350@gmx.net>
On Tue, 2007-09-18 at 11:30 +0200, Michael Kerrisk wrote:
> > This way we have it nicely integrated into the posix timer code and keep
> > the existing semantics of posix timers intact.
> >
> > We need to think about the open file descriptor in the timer_delete()
> > case as well, but this should be not too hard to sort out.
>
> This seems like a workable idea also. But note David Härdeman's
> critique of options c & d: the existence of a coupled timerfd
> and a timerid means that the application must maintain a mapping
> between the two, so that after an epoll call (for example) that
> says the timerfd is ready, the timer can be manipulated using
> the corresponding timerfd. This isn't IMO a fatal flaw, but
> it does make the API a little more clumsy.
Hmm, we might do something like:
timer_gettime(fd | POSIX_TIMER_FD, .....);
So the kernel looks up the fd in order to figure out the timer_id, which
needs to be referenced in filep->private_data anyway.
tglx
next prev parent reply other threads:[~2007-09-18 9:43 UTC|newest]
Thread overview: 26+ messages / expand[flat|nested] mbox.gz Atom feed top
2007-09-18 7:27 RFC: A revised timerfd API Michael Kerrisk
2007-09-18 7:30 ` Michael Kerrisk
2007-09-18 8:05 ` David Härdeman
2007-09-18 9:01 ` Michael Kerrisk
2007-09-18 9:27 ` Thomas Gleixner
2007-09-18 9:10 ` Thomas Gleixner
2007-09-18 9:30 ` Michael Kerrisk
2007-09-18 9:42 ` Thomas Gleixner [this message]
2007-09-18 11:08 ` Michael Kerrisk
2007-09-18 11:30 ` Thomas Gleixner
2007-09-18 13:13 ` David Härdeman
2007-09-22 13:03 ` Michael Kerrisk
2007-09-18 16:51 ` Davide Libenzi
2007-09-22 13:12 ` Michael Kerrisk
2007-09-22 14:32 ` Bernd Eckenfels
2007-09-22 16:07 ` Michael Kerrisk
2007-09-22 17:05 ` Thomas Gleixner
2007-09-22 23:37 ` David Härdeman
2007-09-22 17:10 ` Thomas Gleixner
2007-09-22 21:07 ` Davide Libenzi
2007-09-22 21:26 ` Thomas Gleixner
2007-09-22 23:21 ` Davide Libenzi
2007-09-23 17:33 ` Michael Kerrisk
2007-09-23 18:33 ` Davide Libenzi
2007-09-23 18:41 ` Davide Libenzi
2007-09-23 19:03 ` Michael Kerrisk
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=1190108542.2995.112.camel@chaos \
--to=tglx@linutronix.de \
--cc=Lee.Schermerhorn@hp.com \
--cc=akpm@linux-foundation.org \
--cc=corbet@lwn.net \
--cc=david@hardeman.nu \
--cc=davidel@xmailserver.org \
--cc=drepper@redhat.com \
--cc=geoff@gclare.org.uk \
--cc=hch@lst.de \
--cc=linux-kernel@vger.kernel.org \
--cc=mtk-manpages@gmx.net \
--cc=rdunlap@xenotime.net \
--cc=torvalds@linux-foundation.org \
--cc=vda.linux@googlemail.com \
/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