diff options
| author |  Sree Kuchibhotla <sreek@google.com> | 2018-07-16 18:10:05 -0700 |
|---|---|---|
| committer | Sree Kuchibhotla <sreek@google.com> | 2018-07-16 18:10:05 -0700 |
| commit | c55026eee95ce24137c81194abdda2c8845e414d (patch) | |
| tree | f4e91f4715f85c53cd294f3968bfaeb4daa1ee5b /src/core/lib | |
| parent | 00476fd2b8876f0e33f5bec4f14cd73edbc9fe8b (diff) | |
| parent | 31c7ab1aafcd00c7407c84c3838ef0bb356ffeb1 (diff) | |
Merge branch 'master' into fix-dns-job-2
Diffstat (limited to 'src/core/lib')
| -rw-r--r-- | src/core/lib/iomgr/ev_epollex_linux.cc | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/src/core/lib/iomgr/ev_epollex_linux.cc b/src/core/lib/iomgr/ev_epollex_linux.cc index 5ffabdc665..7b368410cf 100644 --- a/src/core/lib/iomgr/ev_epollex_linux.cc +++ b/src/core/lib/iomgr/ev_epollex_linux.cc @@ -494,8 +494,9 @@ static void fd_orphan(grpc_fd* fd, grpc_closure* on_done, int* release_fd, is_fd_closed = true; } + // TODO(sreek): handle fd removal (where is_fd_closed=false) if (!is_fd_closed) { - gpr_log(GPR_DEBUG, "TODO: handle fd removal?"); + GRPC_FD_TRACE("epoll_fd %p (%d) was orphaned but not closed.", fd, fd->fd); } /* Remove the active status but keep referenced. We want this grpc_fd struct @@ -1564,7 +1565,7 @@ static void pollset_set_add_pollset_set(grpc_pollset_set* a, gpr_mu_unlock(b_mu); } // try to do the least copying possible - // TODO(ctiller): there's probably a better heuristic here + // TODO(sreek): there's probably a better heuristic here const size_t a_size = a->fd_count + a->pollset_count; const size_t b_size = b->fd_count + b->pollset_count; if (b_size > a_size) { |