All of lore.kernel.org
 help / color / mirror / Atom feed
From: Andrew Morton <akpm@digeo.com>
To: Davide Libenzi <davidel@xmailserver.org>
Cc: Linux Kernel Mailing List <linux-kernel@vger.kernel.org>
Subject: Re: [patch] sys_epoll 0.14 ...
Date: Tue, 29 Oct 2002 19:35:14 -0800	[thread overview]
Message-ID: <3DBF5372.901E3CB3@digeo.com> (raw)
In-Reply-To: Pine.LNX.4.44.0210291643520.1457-100000@blue1.dev.mcafeelabs.com

Davide Libenzi wrote:
> 
> Thanks to Andrew and John suggestions I coded another version of the
> sys_epoll patch ( 0.13 skipped ... superstition :) ). I won't send the
> patch to not waste bandwidth, the patch is available here :
> 
> http://www.xmailserver.org/linux-patches/sys_epoll-2.5.44-last.diff
> 
> Comments are welcome ...
> 

Looking good to me, Davide.  I think you've nailed everything there
except:

- Do we want to introduce new list macros, or change epoll a little
  to use the existing list manipulators (I think the latter)

- Do we keep the vmalloc, replace it with alloc_pages() or go all the
  way and remove the hash table?

  Seems that you're leaning toward the final option but either way,
  it would be best to get that vmalloc out of there - it's basically
  hit-or-miss for anything larger than a single page - for a really
  robust implementation it is best to avoid its use.

  reply	other threads:[~2002-10-30  3:29 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2002-10-30  0:49 [patch] sys_epoll 0.14 Davide Libenzi
2002-10-30  3:35 ` Andrew Morton [this message]
2002-10-30  3:50   ` Davide Libenzi
2002-10-30  3:55     ` Andrew Morton
2002-10-30  4:09       ` Davide Libenzi
2002-10-30  4:12         ` Andrew Morton
2002-10-30  4:29           ` Davide Libenzi
2002-10-30 17:15     ` Mark Hamblin
2002-10-30  3:51   ` Davide Libenzi
2002-10-30 20:14 ` Janet Morgan
2002-10-30 20:37   ` Davide Libenzi
2002-10-30 20:31     ` Linus Torvalds
2002-10-30 20:45       ` Davide Libenzi
2002-10-30 20:38         ` Linus Torvalds

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=3DBF5372.901E3CB3@digeo.com \
    --to=akpm@digeo.com \
    --cc=davidel@xmailserver.org \
    --cc=linux-kernel@vger.kernel.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.