linux: remove epoll syscall wrappers#1372
Conversation
|
@bnoordhuis, apologize for noise. Might I ask that what is the relation of glibc requirement here from your patch. I have old server with kernel 2.6.32 and glibc 2.11. Is there any feature blocking to work only that glibc version from your patch. thanks. |
|
@uvguy I believe it should work. glibc's epoll_pwait() wrapper was introduced in 2.6, the system call in kernel 2.6.19. |
|
yes thanks |
ad446dd to
78fba14
Compare
|
As expected, this breaks the centos5 buildbots. Everything else is green. |
|
@bnoordhuis what do you want to do with this one? |
|
Good question. I could retarget it to v2 and back-port to v1 if and when it becomes necessary. I'll look into that this week. |
78fba14 to
60d54bd
Compare
|
Rebased (minor conflict in os390.c) and ready for merge. Given the discussion in #1880 and #1969, I'd say it won't hurt to merge it into v1.x. edit: CI: https://ci.nodejs.org/view/libuv/job/libuv-test-commit/1017/ |
Remove the syscalls wrappers now that the minimum requirements are kernel 2.6.32 + glibc 2.12. This commit should fix the epoll_pwait()-related issues that have been reported against mips/mipsel builds. Fixes: libuv#335 PR-URL: libuv#1372 Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Santiago Gimeno <santiago.gimeno@gmail.com>
Remove the support for kernels that don't support epoll_pwait() now that the minimum requirements are kernel 2.6.32 + glibc 2.12. PR-URL: libuv#1372 Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Santiago Gimeno <santiago.gimeno@gmail.com>
60d54bd to
f43c663
Compare
|
Landed in c0c672e...f43c663. |
This was removed before in 2018 but then reinstated again in 2019 to fix building with old Android SDKs. Well, time marches on; this time it's gone for good. Refs: libuv#1372 Refs: libuv#2358
This was removed before in 2018 but then reinstated again in 2019 to fix building with old Android SDKs. Well, time marches on; this time it's gone for good. Refs: libuv/libuv#1372 Refs: libuv/libuv#2358
This was removed before in 2018 but then reinstated again in 2019 to fix building with old Android SDKs. Well, time marches on; this time it's gone for good. Refs: libuv/libuv#1372 Refs: libuv/libuv#2358
Remove the syscalls wrappers now that the minimum requirements are
kernel 2.6.32 + glibc 2.12.
This commit should fix the epoll_pwait()-related issues that have
been reported against mips/mipsel builds.
Fixes: #335 (longstanding issue)