aboutsummaryrefslogtreecommitdiffhomepage
diff options
context:
space:
mode:
authorGravatar Sree Kuchibhotla <sreecha@users.noreply.github.com>2017-07-25 13:45:10 -0700
committerGravatar GitHub <noreply@github.com>2017-07-25 13:45:10 -0700
commitad5a9c2a0db1926eaec110a7fe573875840c6ce3 (patch)
tree20b14000f93f1dfaf8034d37ffef7360d67254b8
parent196d3d6ee8778e1d08310cbb798d5500001ffdab (diff)
parent9bf8342a93a158f0c5c78eb2b3f347d518025b11 (diff)
Merge pull request #11925 from sreecha/disable-epoll1
Disable epoll1 unless explicitly requested
-rw-r--r--src/core/lib/iomgr/ev_epoll1_linux.c6
-rw-r--r--src/core/lib/iomgr/ev_epollsig_linux.c2
2 files changed, 7 insertions, 1 deletions
diff --git a/src/core/lib/iomgr/ev_epoll1_linux.c b/src/core/lib/iomgr/ev_epoll1_linux.c
index 8db4a92453..dc48d73df9 100644
--- a/src/core/lib/iomgr/ev_epoll1_linux.c
+++ b/src/core/lib/iomgr/ev_epoll1_linux.c
@@ -1030,6 +1030,12 @@ static const grpc_event_engine_vtable vtable = {
/* It is possible that GLIBC has epoll but the underlying kernel doesn't.
* Create a dummy epoll_fd to make sure epoll support is available */
const grpc_event_engine_vtable *grpc_init_epoll1_linux(bool explicit_request) {
+ /* TODO(sreek): Temporarily disable this poller unless explicitly requested
+ * via GRPC_POLL_STRATEGY */
+ if (!explicit_request) {
+ return NULL;
+ }
+
if (!grpc_has_wakeup_fd()) {
return NULL;
}
diff --git a/src/core/lib/iomgr/ev_epollsig_linux.c b/src/core/lib/iomgr/ev_epollsig_linux.c
index 11067e3959..070d75e42a 100644
--- a/src/core/lib/iomgr/ev_epollsig_linux.c
+++ b/src/core/lib/iomgr/ev_epollsig_linux.c
@@ -1730,7 +1730,7 @@ const grpc_event_engine_vtable *grpc_init_epollsig_linux(
if (!is_grpc_wakeup_signal_initialized) {
/* TODO(ctiller): when other epoll engines are ready, remove the true || to
* force this to be explitly chosen if needed */
- if (explicit_request) {
+ if (true || explicit_request) {
grpc_use_signal(SIGRTMIN + 6);
} else {
return NULL;