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.
next prev parent 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 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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
Powered by JetHome