public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Jamie Lokier <jamie@shareable.org>
To: Mark Mielke <mark@mark.mielke.cc>
Cc: Davide Libenzi <davidel@xmailserver.org>,
	Lennert Buytenhek <buytenh@math.leidenuniv.nl>,
	Linux Kernel Mailing List <linux-kernel@vger.kernel.org>
Subject: Re: {sys_,/dev/}epoll waiting timeout
Date: Thu, 23 Jan 2003 22:18:58 +0000	[thread overview]
Message-ID: <20030123221858.GA8581@bjl1.asuk.net> (raw)
In-Reply-To: <20030123204056.GC2490@mark.mielke.cc>

Mark Mielke wrote:
> Sorry... not 1 in 1001... almost 100% chance of returning of one
> jiffie too many.

It's curious that select() is different from poll() - almost is if
completely different people wrote the code :)

We have to wonder whether it was a design decision.
Perhaps the unix specifications require it (see below).

> In practice, even on a relatively idle system, the process will not
> be able to wake up as frequently as it might be able to expect.

You're right - it's unfortunate that using poll() lets you sleep and
wake up no faster than every _two_ ticks.  That's actually caused by
poll()'s treating zero differently though, not by +1 as such.

There's a strange contradiction between rounding up the waiting time
to the next number of jiffies, and then rounding it down (in a
time-dependent way) by waiting until the next N'th timer interrupt.

If, as someone said, the appropriate unix specification says that
"wait for 10ms" means to wait for _at minimum_ 10ms, then you do need
the +1.

(Davide), IMHO epoll should decide whether it means "at minimum" (in
which case the +1 is a requirement), or it means "at maximum" (in
which case rounding up is wrong).

The current method of rounding up and then effectively down means that
you get an unpredictable mixture of both.

-- Jamie

ps. I would always prefer an absolute wakeup time anyway - it avoids a
race condition too.  What a shame none of the system calls work that way.

pps. To summarise, all the time APIs are a complete mess in unix, and
there's nothing you can do in user space to make up for the b0rken
system call interface.  Except not duplicate past errors in new interfaces :)

  reply	other threads:[~2003-01-23 22:09 UTC|newest]

Thread overview: 28+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2003-01-22  6:55 {sys_,/dev/}epoll waiting timeout Lennert Buytenhek
2003-01-22  8:03 ` Jamie Lokier
2003-01-22 12:46   ` Ed Tomlinson
2003-01-22 13:20     ` Jamie Lokier
2003-01-22 19:14       ` Randy.Dunlap
2003-01-22 19:34         ` Jamie Lokier
2003-01-22 19:32           ` Randy.Dunlap
2003-01-23 14:07   ` Davide Libenzi
2003-01-23 15:43     ` Jamie Lokier
2003-01-23 17:27       ` Mark Mielke
2003-01-23 18:28         ` Jamie Lokier
2003-01-23 20:40           ` Mark Mielke
2003-01-23 22:18             ` Jamie Lokier [this message]
2003-01-24 14:41               ` Andreas Schwab
2003-01-25  1:08               ` Davide Libenzi
2003-01-27 21:27               ` bug in select() (was Re: {sys_,/dev/}epoll waiting timeout) Bill Rugolsky Jr.
2003-01-27 22:52                 ` Davide Libenzi
2003-01-28  9:45                   ` Jamie Lokier
2003-01-28 10:52                     ` Mark Mielke
2003-01-28 21:39                       ` Jamie Lokier
2003-01-28 22:15                     ` Davide Libenzi
2003-01-28 19:42   ` {sys_,/dev/}epoll waiting timeout Randy.Dunlap
2003-01-28 21:36     ` Jamie Lokier
2003-01-28 21:44       ` David Mosberger
     [not found] <20030122080322.GB3466@bjl1.asuk.net.suse.lists.linux.kernel>
     [not found] ` <Pine.LNX.4.33L2.0301281139570.30636-100000@dragon.pdx.osdl.net.suse.lists.linux.kernel>
     [not found]   ` <20030128213621.GA29036@bjl1.asuk.net.suse.lists.linux.kernel>
2003-01-28 21:55     ` Andi Kleen
2003-01-28 22:24       ` Davide Libenzi
2003-01-28 22:39         ` Andi Kleen
2003-01-28 23:00           ` Davide Libenzi

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=20030123221858.GA8581@bjl1.asuk.net \
    --to=jamie@shareable.org \
    --cc=buytenh@math.leidenuniv.nl \
    --cc=davidel@xmailserver.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mark@mark.mielke.cc \
    /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