LKML Archive on lore.kernel.org help / color / mirror / Atom feed
From: Evgeniy Polyakov <johnpol@2ka.mipt.ru> To: Ulrich Drepper <drepper@redhat.com> Cc: Ulrich Drepper <drepper@gmail.com>, lkml <linux-kernel@vger.kernel.org>, David Miller <davem@davemloft.net>, Andrew Morton <akpm@osdl.org>, netdev <netdev@vger.kernel.org>, Zach Brown <zach.brown@oracle.com>, Christoph Hellwig <hch@infradead.org>, Chase Venters <chase.venters@clientec.com> Subject: Re: [take19 0/4] kevent: Generic event handling mechanism. Date: Thu, 5 Oct 2006 13:02:15 +0400 [thread overview] Message-ID: <20061005090214.GB1015@2ka.mipt.ru> (raw) In-Reply-To: <4523ED6C.9080902@redhat.com> On Wed, Oct 04, 2006 at 10:20:44AM -0700, Ulrich Drepper (drepper@redhat.com) wrote: > Evgeniy Polyakov wrote: > > It is completely possible to do what you describe without special > > syscall parameters. > > First of all, I don't see how this is efficiently possible. The mask > might change from call to call. And you can add/remove signal events using existing kevent api between calls. > Second, hasn't it sunk in that inventing new ways to pass parameters is > bad? Programmers don't want to learn new ways for every new interface. > Reuse is good! And creating special cases for usual events is bad. There is unified way to deal with events in kevent - add/remove/modify/wait on them, signals are just usual events. > This applies to the signal mask here. > > But there is another parameter falling into that category and I meant to > mention it before: the timeout value. All other calls except poll and > especially all modern interfaces use a timespec pointer. This is the > way times are kept in userland code. Don't try to force people to do > something else. > > Using a timespec also has the advantage that we can add an absolute > timeout value mode (optional) instead of the relative timeout value. > > In this context, we should/must be able to specify which clock the > timeout is for (not as part of the wait call, but another control > operation perhaps). It's important to distinguish between > CLOCK_REALTIME and CLOCK_MONOTONE. Both have their use. I think you wanted to say, that 'all event mechanism except the most commonly used poll/select/epoll use timespec'. I designed it to be similar to poll(), it is really good interface. Nature of the waiting is to wait for some time, so I put there that 'some time'. > -- > ➧ Ulrich Drepper ➧ Red Hat, Inc. ➧ 444 Castro St ➧ Mountain View, CA ❖ > -- Evgeniy Polyakov
next prev parent reply other threads:[~2006-10-05 9:02 UTC|newest] Thread overview: 68+ messages / expand[flat|nested] mbox.gz Atom feed top [not found] <115a6230591036@2ka.mipt.ru> 2006-09-12 8:41 ` [take18 0/4] kevent: Generic event handling mechanism Evgeniy Polyakov 2006-09-12 8:41 ` [take18 1/4] kevent: Core files Evgeniy Polyakov 2006-09-12 8:41 ` [take18 2/4] kevent: poll/select() notifications Evgeniy Polyakov 2006-09-12 8:41 ` [take18 3/4] kevent: Socket notifications Evgeniy Polyakov 2006-09-12 8:41 ` [take18 4/4] kevent: Timer notifications Evgeniy Polyakov 2006-09-20 9:35 ` [take19 0/4] kevent: Generic event handling mechanism Evgeniy Polyakov 2006-09-20 9:35 ` [take19 1/4] kevent: Core files Evgeniy Polyakov 2006-09-20 9:35 ` [take19 2/4] kevent: poll/select() notifications Evgeniy Polyakov 2006-09-20 9:35 ` [take19 3/4] kevent: Socket notifications Evgeniy Polyakov 2006-09-20 9:35 ` [take19 4/4] kevent: Timer notifications Evgeniy Polyakov 2006-10-04 6:34 ` [take19 1/4] kevent: Core files Ulrich Drepper 2006-10-04 6:48 ` Evgeniy Polyakov 2006-10-04 17:57 ` Ulrich Drepper 2006-10-05 8:57 ` Evgeniy Polyakov 2006-10-05 9:56 ` Eric Dumazet 2006-10-05 10:21 ` Evgeniy Polyakov 2006-10-05 10:45 ` Eric Dumazet 2006-10-05 10:55 ` Evgeniy Polyakov 2006-10-05 12:09 ` Eric Dumazet 2006-10-05 12:37 ` Evgeniy Polyakov 2006-10-15 23:22 ` Ulrich Drepper 2006-10-16 7:33 ` Evgeniy Polyakov 2006-10-16 10:16 ` Ulrich Drepper 2006-10-16 11:23 ` Evgeniy Polyakov 2006-10-17 5:10 ` Johann Borck 2006-10-17 5:59 ` Chase Venters 2006-10-17 10:42 ` Evgeniy Polyakov 2006-10-17 13:12 ` Chase Venters 2006-10-17 13:35 ` Evgeniy Polyakov 2006-10-17 10:39 ` Evgeniy Polyakov 2006-10-17 13:19 ` Eric Dumazet 2006-10-17 13:42 ` Evgeniy Polyakov 2006-10-17 13:52 ` Eric Dumazet 2006-10-17 14:07 ` Evgeniy Polyakov 2006-10-17 14:25 ` Eric Dumazet 2006-10-17 15:09 ` Evgeniy Polyakov 2006-10-17 15:32 ` Eric Dumazet 2006-10-17 16:01 ` Evgeniy Polyakov 2006-10-17 16:26 ` Eric Dumazet 2006-10-17 16:35 ` Evgeniy Polyakov 2006-10-17 16:45 ` Eric Dumazet 2006-10-18 4:10 ` Evgeniy Polyakov 2006-10-18 4:45 ` Eric Dumazet 2006-10-17 15:33 ` Hans Henrik Happe 2006-10-05 14:01 ` Hans Henrik Happe 2006-10-05 14:15 ` Evgeniy Polyakov 2006-10-05 15:07 ` Hans Henrik Happe 2006-09-22 19:22 ` [take19 0/4] kevent: Generic event handling mechanism Andrew Morton 2006-09-23 4:23 ` Evgeniy Polyakov 2006-10-04 6:09 ` Ulrich Drepper 2006-10-04 6:10 ` Ulrich Drepper 2006-10-04 6:27 ` Evgeniy Polyakov 2006-10-04 6:24 ` Evgeniy Polyakov 2006-09-26 15:54 ` Christoph Hellwig 2006-09-27 4:46 ` Evgeniy Polyakov 2006-09-27 15:09 ` Evgeniy Polyakov 2006-10-04 4:50 ` Ulrich Drepper 2006-10-04 4:55 ` Evgeniy Polyakov 2006-10-04 7:33 ` Ulrich Drepper 2006-10-04 7:48 ` Evgeniy Polyakov 2006-10-04 17:20 ` Ulrich Drepper 2006-10-05 9:02 ` Evgeniy Polyakov [this message] 2006-10-05 14:45 ` Ulrich Drepper 2006-10-06 8:36 ` Evgeniy Polyakov 2006-10-15 22:43 ` Ulrich Drepper 2006-10-16 7:23 ` Evgeniy Polyakov 2006-10-16 9:59 ` Ulrich Drepper 2006-10-16 10:38 ` Evgeniy Polyakov
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=20061005090214.GB1015@2ka.mipt.ru \ --to=johnpol@2ka.mipt.ru \ --cc=akpm@osdl.org \ --cc=chase.venters@clientec.com \ --cc=davem@davemloft.net \ --cc=drepper@gmail.com \ --cc=drepper@redhat.com \ --cc=hch@infradead.org \ --cc=linux-kernel@vger.kernel.org \ --cc=netdev@vger.kernel.org \ --cc=zach.brown@oracle.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: linkBe 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).