Re: [reSIProcate] epoll prototype patch

[Kennard White <kennard_white@xxxxxxxxxxxx>]

Folks,

This patch is now available as //svn.resiprocate.org/rep/resiprocate/branches/b-epoll-kw-20101115

I would appreciate thoughts on the best way to determine if a given platform supports epoll(). Should I add this as an option to the configure script? Is there any more automatic approach?

Thanks,
Kennard

On Thu, Nov 11, 2010 at 11:27 PM, Kennard White <kennard_white@xxxxxxxxxxxx> wrote:

Hi,

I've added some limited epoll support to ares and resip stacks. I just got it working a couple days ago and it hasn't seen much testing. That said, I'd appreciate feedback on the overall approach. See attached resip-epoll-notes.txt for overview and high-level change summary. As I explain in those notes, my primary goal is to simply to support a very large number of file descriptors, not improve performance. As such, this is a partial implementation that attempts to minimize external API changes and also avoid any behavior changes unless epoll mode is enabled.

Anyways, please see the two attached patch files. This is a patch to apply to SVN8788 (current as of 11/11/2010).

One immediate question I have: what is the best way to determine if code is being compiled on platform that supports epoll? The compilation tree doesn't seem to use autoconf or such. I don't know how many platforms other than Linux (and only relatively Linux versions) support epoll. Any suggestions would be appreciated.

Thanks,
Kennard