This is the mail archive of the libc-alpha@sourceware.org mailing list for the glibc project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: epoll_pwait broken?


On 1/23/07, Daniel Jacobowitz <drow@false.org> wrote:
On Mon, Jan 22, 2007 at 11:13:03PM -0500, Carlos O'Donell wrote:
> FWIW I think a patch to "sysdeps/unix/sysv/linux/sys/epoll.h" adding
> the 6th argument is the right thing to do.

No, the point is that glibc is supposed to take care of that - I think
the original poster was right.  See e.g. sigsuspend.

Why not export the kernel interface as is, and avoid adding a wrapper? Why does glibc need to take care of this?

Pros:
- User doesn't have to type "sizeof (sigset_t)"

Cons:
- More code in glibc.
- Compiler can't optimize constant load for multiple epoll_pwait calls.
- Kernel and userspace interfaces look different in the documentation.

Cheers,
Carlos.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]