Sree Kuchibhotla | f448c34 | 2016-05-19 10:51:24 -0700 | [diff] [blame] | 1 | /* |
| 2 | * |
Jan Tattermusch | 7897ae9 | 2017-06-07 22:57:36 +0200 | [diff] [blame] | 3 | * Copyright 2016 gRPC authors. |
Sree Kuchibhotla | f448c34 | 2016-05-19 10:51:24 -0700 | [diff] [blame] | 4 | * |
Jan Tattermusch | 7897ae9 | 2017-06-07 22:57:36 +0200 | [diff] [blame] | 5 | * Licensed under the Apache License, Version 2.0 (the "License"); |
| 6 | * you may not use this file except in compliance with the License. |
| 7 | * You may obtain a copy of the License at |
Sree Kuchibhotla | f448c34 | 2016-05-19 10:51:24 -0700 | [diff] [blame] | 8 | * |
Jan Tattermusch | 7897ae9 | 2017-06-07 22:57:36 +0200 | [diff] [blame] | 9 | * http://www.apache.org/licenses/LICENSE-2.0 |
Sree Kuchibhotla | f448c34 | 2016-05-19 10:51:24 -0700 | [diff] [blame] | 10 | * |
Jan Tattermusch | 7897ae9 | 2017-06-07 22:57:36 +0200 | [diff] [blame] | 11 | * Unless required by applicable law or agreed to in writing, software |
| 12 | * distributed under the License is distributed on an "AS IS" BASIS, |
| 13 | * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. |
| 14 | * See the License for the specific language governing permissions and |
| 15 | * limitations under the License. |
Sree Kuchibhotla | f448c34 | 2016-05-19 10:51:24 -0700 | [diff] [blame] | 16 | * |
| 17 | */ |
| 18 | |
murgatroid99 | 5407089 | 2016-08-08 17:01:18 -0700 | [diff] [blame] | 19 | #include "src/core/lib/iomgr/port.h" |
Sree Kuchibhotla | f448c34 | 2016-05-19 10:51:24 -0700 | [diff] [blame] | 20 | |
Yash Tibrewal | 4e0fe52 | 2017-10-08 18:07:15 -0700 | [diff] [blame] | 21 | #include <grpc/grpc_posix.h> |
yang-g | ceb2475 | 2017-11-07 12:06:37 -0800 | [diff] [blame] | 22 | #include <grpc/support/log.h> |
Yash Tibrewal | 4e0fe52 | 2017-10-08 18:07:15 -0700 | [diff] [blame] | 23 | |
Sree Kuchibhotla | 76a0795 | 2016-06-22 15:09:06 -0700 | [diff] [blame] | 24 | /* This polling engine is only relevant on linux kernels supporting epoll() */ |
Mehrdad Afshari | fb66900 | 2018-01-17 15:37:56 -0800 | [diff] [blame^] | 25 | #ifdef GRPC_LINUX_EPOLL_CREATE1 |
Sree Kuchibhotla | f448c34 | 2016-05-19 10:51:24 -0700 | [diff] [blame] | 26 | |
Craig Tiller | 4509c47 | 2017-04-27 19:05:13 +0000 | [diff] [blame] | 27 | #include "src/core/lib/iomgr/ev_epollsig_linux.h" |
Sree Kuchibhotla | f448c34 | 2016-05-19 10:51:24 -0700 | [diff] [blame] | 28 | |
| 29 | #include <assert.h> |
| 30 | #include <errno.h> |
Craig Tiller | 2039779 | 2017-07-18 11:35:27 -0700 | [diff] [blame] | 31 | #include <limits.h> |
Sree Kuchibhotla | f448c34 | 2016-05-19 10:51:24 -0700 | [diff] [blame] | 32 | #include <poll.h> |
Sree Kuchibhotla | 4998e30 | 2016-08-16 07:26:31 -0700 | [diff] [blame] | 33 | #include <pthread.h> |
Sree Kuchibhotla | f448c34 | 2016-05-19 10:51:24 -0700 | [diff] [blame] | 34 | #include <signal.h> |
| 35 | #include <string.h> |
| 36 | #include <sys/epoll.h> |
| 37 | #include <sys/socket.h> |
| 38 | #include <unistd.h> |
| 39 | |
| 40 | #include <grpc/support/alloc.h> |
Sree Kuchibhotla | f448c34 | 2016-05-19 10:51:24 -0700 | [diff] [blame] | 41 | #include <grpc/support/string_util.h> |
| 42 | #include <grpc/support/tls.h> |
| 43 | #include <grpc/support/useful.h> |
| 44 | |
Craig Tiller | b4bb1cd | 2017-07-20 14:18:17 -0700 | [diff] [blame] | 45 | #include "src/core/lib/debug/stats.h" |
Craig Tiller | 6b7c1fb | 2017-07-19 15:45:03 -0700 | [diff] [blame] | 46 | #include "src/core/lib/iomgr/block_annotate.h" |
Sree Kuchibhotla | f448c34 | 2016-05-19 10:51:24 -0700 | [diff] [blame] | 47 | #include "src/core/lib/iomgr/ev_posix.h" |
| 48 | #include "src/core/lib/iomgr/iomgr_internal.h" |
Craig Tiller | 376887d | 2017-04-06 08:27:03 -0700 | [diff] [blame] | 49 | #include "src/core/lib/iomgr/lockfree_event.h" |
Craig Tiller | 185f6c9 | 2017-03-17 08:33:19 -0700 | [diff] [blame] | 50 | #include "src/core/lib/iomgr/timer.h" |
Sree Kuchibhotla | f448c34 | 2016-05-19 10:51:24 -0700 | [diff] [blame] | 51 | #include "src/core/lib/iomgr/wakeup_fd_posix.h" |
| 52 | #include "src/core/lib/profiling/timers.h" |
Craig Tiller | fbf61bb | 2017-11-08 11:50:14 -0800 | [diff] [blame] | 53 | #include "src/core/lib/support/manual_constructor.h" |
Sree Kuchibhotla | f448c34 | 2016-05-19 10:51:24 -0700 | [diff] [blame] | 54 | |
Craig Tiller | baa14a9 | 2017-11-03 09:09:36 -0700 | [diff] [blame] | 55 | #define GRPC_POLLSET_KICK_BROADCAST ((grpc_pollset_worker*)1) |
Craig Tiller | e24b24d | 2017-04-06 16:05:45 -0700 | [diff] [blame] | 56 | |
Noah Eisen | c384d81 | 2017-11-12 20:14:27 -0800 | [diff] [blame] | 57 | #define GRPC_POLLING_TRACE(...) \ |
ncteisen | 9ffb149 | 2017-11-10 14:00:49 -0800 | [diff] [blame] | 58 | if (grpc_polling_trace.enabled()) { \ |
Noah Eisen | c384d81 | 2017-11-12 20:14:27 -0800 | [diff] [blame] | 59 | gpr_log(GPR_INFO, __VA_ARGS__); \ |
Sree Kuchibhotla | 1e77668 | 2016-06-28 14:09:26 -0700 | [diff] [blame] | 60 | } |
| 61 | |
Sree Kuchibhotla | c7be7c6 | 2016-06-09 17:08:50 -0700 | [diff] [blame] | 62 | static int grpc_wakeup_signal = -1; |
| 63 | static bool is_grpc_wakeup_signal_initialized = false; |
Sree Kuchibhotla | f448c34 | 2016-05-19 10:51:24 -0700 | [diff] [blame] | 64 | |
Sree Kuchibhotla | c7be7c6 | 2016-06-09 17:08:50 -0700 | [diff] [blame] | 65 | /* Implements the function defined in grpc_posix.h. This function might be |
| 66 | * called before even calling grpc_init() to set either a different signal to |
| 67 | * use. If signum == -1, then the use of signals is disabled */ |
| 68 | void grpc_use_signal(int signum) { |
| 69 | grpc_wakeup_signal = signum; |
| 70 | is_grpc_wakeup_signal_initialized = true; |
| 71 | |
| 72 | if (grpc_wakeup_signal < 0) { |
| 73 | gpr_log(GPR_INFO, |
| 74 | "Use of signals is disabled. Epoll engine will not be used"); |
| 75 | } else { |
| 76 | gpr_log(GPR_INFO, "epoll engine will be using signal: %d", |
| 77 | grpc_wakeup_signal); |
| 78 | } |
| 79 | } |
| 80 | |
| 81 | struct polling_island; |
Sree Kuchibhotla | 5855c47 | 2016-06-08 12:56:56 -0700 | [diff] [blame] | 82 | |
Sree Kuchibhotla | f6f33d7 | 2016-11-18 11:38:52 -0800 | [diff] [blame] | 83 | typedef enum { |
| 84 | POLL_OBJ_FD, |
| 85 | POLL_OBJ_POLLSET, |
| 86 | POLL_OBJ_POLLSET_SET |
| 87 | } poll_obj_type; |
| 88 | |
| 89 | typedef struct poll_obj { |
ncteisen | e9cd8a8 | 2017-06-29 06:03:52 -0400 | [diff] [blame] | 90 | #ifndef NDEBUG |
Sree Kuchibhotla | a0749a6 | 2016-11-18 20:22:09 -0800 | [diff] [blame] | 91 | poll_obj_type obj_type; |
| 92 | #endif |
Sree Kuchibhotla | f6f33d7 | 2016-11-18 11:38:52 -0800 | [diff] [blame] | 93 | gpr_mu mu; |
Craig Tiller | baa14a9 | 2017-11-03 09:09:36 -0700 | [diff] [blame] | 94 | struct polling_island* pi; |
Sree Kuchibhotla | f6f33d7 | 2016-11-18 11:38:52 -0800 | [diff] [blame] | 95 | } poll_obj; |
| 96 | |
Craig Tiller | baa14a9 | 2017-11-03 09:09:36 -0700 | [diff] [blame] | 97 | const char* poll_obj_string(poll_obj_type po_type) { |
Sree Kuchibhotla | f6f33d7 | 2016-11-18 11:38:52 -0800 | [diff] [blame] | 98 | switch (po_type) { |
| 99 | case POLL_OBJ_FD: |
| 100 | return "fd"; |
| 101 | case POLL_OBJ_POLLSET: |
| 102 | return "pollset"; |
| 103 | case POLL_OBJ_POLLSET_SET: |
| 104 | return "pollset_set"; |
| 105 | } |
| 106 | |
| 107 | GPR_UNREACHABLE_CODE(return "UNKNOWN"); |
| 108 | } |
| 109 | |
Craig Tiller | baa14a9 | 2017-11-03 09:09:36 -0700 | [diff] [blame] | 110 | /******************************************************************************* |
| 111 | * Fd Declarations |
| 112 | */ |
Sree Kuchibhotla | f6f33d7 | 2016-11-18 11:38:52 -0800 | [diff] [blame] | 113 | |
Craig Tiller | baa14a9 | 2017-11-03 09:09:36 -0700 | [diff] [blame] | 114 | #define FD_FROM_PO(po) ((grpc_fd*)(po)) |
Sree Kuchibhotla | f6f33d7 | 2016-11-18 11:38:52 -0800 | [diff] [blame] | 115 | |
Sree Kuchibhotla | f448c34 | 2016-05-19 10:51:24 -0700 | [diff] [blame] | 116 | struct grpc_fd { |
Sree Kuchibhotla | f6f33d7 | 2016-11-18 11:38:52 -0800 | [diff] [blame] | 117 | poll_obj po; |
| 118 | |
Sree Kuchibhotla | f448c34 | 2016-05-19 10:51:24 -0700 | [diff] [blame] | 119 | int fd; |
| 120 | /* refst format: |
Sree Kuchibhotla | 5098f91 | 2016-05-31 10:58:17 -0700 | [diff] [blame] | 121 | bit 0 : 1=Active / 0=Orphaned |
| 122 | bits 1-n : refcount |
Sree Kuchibhotla | 0bcbd79 | 2016-06-01 15:43:03 -0700 | [diff] [blame] | 123 | Ref/Unref by two to avoid altering the orphaned bit */ |
Sree Kuchibhotla | f448c34 | 2016-05-19 10:51:24 -0700 | [diff] [blame] | 124 | gpr_atm refst; |
| 125 | |
Sree Kuchibhotla | 9998338 | 2017-02-12 17:03:27 -0800 | [diff] [blame] | 126 | /* The fd is either closed or we relinquished control of it. In either |
| 127 | cases, this indicates that the 'fd' on this structure is no longer |
| 128 | valid */ |
Sree Kuchibhotla | 79a6233 | 2016-06-04 14:01:03 -0700 | [diff] [blame] | 129 | bool orphaned; |
Sree Kuchibhotla | f448c34 | 2016-05-19 10:51:24 -0700 | [diff] [blame] | 130 | |
Craig Tiller | fbf61bb | 2017-11-08 11:50:14 -0800 | [diff] [blame] | 131 | grpc_core::ManualConstructor<grpc_core::LockfreeEvent> read_closure; |
| 132 | grpc_core::ManualConstructor<grpc_core::LockfreeEvent> write_closure; |
Sree Kuchibhotla | f448c34 | 2016-05-19 10:51:24 -0700 | [diff] [blame] | 133 | |
Craig Tiller | baa14a9 | 2017-11-03 09:09:36 -0700 | [diff] [blame] | 134 | struct grpc_fd* freelist_next; |
| 135 | grpc_closure* on_done_closure; |
Sree Kuchibhotla | f448c34 | 2016-05-19 10:51:24 -0700 | [diff] [blame] | 136 | |
Sree Kuchibhotla | 82d7341 | 2017-02-09 18:27:45 -0800 | [diff] [blame] | 137 | /* The pollset that last noticed that the fd is readable. The actual type |
| 138 | * stored in this is (grpc_pollset *) */ |
| 139 | gpr_atm read_notifier_pollset; |
Sree Kuchibhotla | 5855c47 | 2016-06-08 12:56:56 -0700 | [diff] [blame] | 140 | |
Sree Kuchibhotla | f448c34 | 2016-05-19 10:51:24 -0700 | [diff] [blame] | 141 | grpc_iomgr_object iomgr_object; |
| 142 | }; |
| 143 | |
Sree Kuchibhotla | f448c34 | 2016-05-19 10:51:24 -0700 | [diff] [blame] | 144 | /* Reference counting for fds */ |
ncteisen | d39010e | 2017-06-08 17:08:07 -0700 | [diff] [blame] | 145 | #ifndef NDEBUG |
Craig Tiller | baa14a9 | 2017-11-03 09:09:36 -0700 | [diff] [blame] | 146 | static void fd_ref(grpc_fd* fd, const char* reason, const char* file, int line); |
| 147 | static void fd_unref(grpc_fd* fd, const char* reason, const char* file, |
Sree Kuchibhotla | f448c34 | 2016-05-19 10:51:24 -0700 | [diff] [blame] | 148 | int line); |
| 149 | #define GRPC_FD_REF(fd, reason) fd_ref(fd, reason, __FILE__, __LINE__) |
| 150 | #define GRPC_FD_UNREF(fd, reason) fd_unref(fd, reason, __FILE__, __LINE__) |
| 151 | #else |
Craig Tiller | baa14a9 | 2017-11-03 09:09:36 -0700 | [diff] [blame] | 152 | static void fd_ref(grpc_fd* fd); |
| 153 | static void fd_unref(grpc_fd* fd); |
Sree Kuchibhotla | f448c34 | 2016-05-19 10:51:24 -0700 | [diff] [blame] | 154 | #define GRPC_FD_REF(fd, reason) fd_ref(fd) |
| 155 | #define GRPC_FD_UNREF(fd, reason) fd_unref(fd) |
| 156 | #endif |
| 157 | |
| 158 | static void fd_global_init(void); |
| 159 | static void fd_global_shutdown(void); |
| 160 | |
Sree Kuchibhotla | f448c34 | 2016-05-19 10:51:24 -0700 | [diff] [blame] | 161 | /******************************************************************************* |
Sree Kuchibhotla | 2f8ade0 | 2016-06-17 13:28:38 -0700 | [diff] [blame] | 162 | * Polling island Declarations |
Sree Kuchibhotla | f448c34 | 2016-05-19 10:51:24 -0700 | [diff] [blame] | 163 | */ |
Sree Kuchibhotla | 2f8ade0 | 2016-06-17 13:28:38 -0700 | [diff] [blame] | 164 | |
ncteisen | a135485 | 2017-06-08 16:25:53 -0700 | [diff] [blame] | 165 | #ifndef NDEBUG |
Sree Kuchibhotla | 2f8ade0 | 2016-06-17 13:28:38 -0700 | [diff] [blame] | 166 | |
Sree Kuchibhotla | 6a29545 | 2016-06-23 15:53:10 -0700 | [diff] [blame] | 167 | #define PI_ADD_REF(p, r) pi_add_ref_dbg((p), (r), __FILE__, __LINE__) |
Yash Tibrewal | 8cf1470 | 2017-12-06 09:47:54 -0800 | [diff] [blame] | 168 | #define PI_UNREF(p, r) pi_unref_dbg((p), (r), __FILE__, __LINE__) |
Sree Kuchibhotla | 2f8ade0 | 2016-06-17 13:28:38 -0700 | [diff] [blame] | 169 | |
ncteisen | d39010e | 2017-06-08 17:08:07 -0700 | [diff] [blame] | 170 | #else |
Sree Kuchibhotla | 2f8ade0 | 2016-06-17 13:28:38 -0700 | [diff] [blame] | 171 | |
Sree Kuchibhotla | 6a29545 | 2016-06-23 15:53:10 -0700 | [diff] [blame] | 172 | #define PI_ADD_REF(p, r) pi_add_ref((p)) |
Yash Tibrewal | 8cf1470 | 2017-12-06 09:47:54 -0800 | [diff] [blame] | 173 | #define PI_UNREF(p, r) pi_unref((p)) |
Sree Kuchibhotla | 2f8ade0 | 2016-06-17 13:28:38 -0700 | [diff] [blame] | 174 | |
ncteisen | a135485 | 2017-06-08 16:25:53 -0700 | [diff] [blame] | 175 | #endif |
Sree Kuchibhotla | 2f8ade0 | 2016-06-17 13:28:38 -0700 | [diff] [blame] | 176 | |
Craig Tiller | 460502e | 2016-10-13 10:02:08 -0700 | [diff] [blame] | 177 | /* This is also used as grpc_workqueue (by directly casing it) */ |
Sree Kuchibhotla | f448c34 | 2016-05-19 10:51:24 -0700 | [diff] [blame] | 178 | typedef struct polling_island { |
| 179 | gpr_mu mu; |
Sree Kuchibhotla | 2f8ade0 | 2016-06-17 13:28:38 -0700 | [diff] [blame] | 180 | /* Ref count. Use PI_ADD_REF() and PI_UNREF() macros to increment/decrement |
| 181 | the refcount. |
| 182 | Once the ref count becomes zero, this structure is destroyed which means |
| 183 | we should ensure that there is never a scenario where a PI_ADD_REF() is |
| 184 | racing with a PI_UNREF() that just made the ref_count zero. */ |
Craig Tiller | 1500761 | 2016-07-06 09:36:16 -0700 | [diff] [blame] | 185 | gpr_atm ref_count; |
Sree Kuchibhotla | f448c34 | 2016-05-19 10:51:24 -0700 | [diff] [blame] | 186 | |
Sree Kuchibhotla | 2f8ade0 | 2016-06-17 13:28:38 -0700 | [diff] [blame] | 187 | /* Pointer to the polling_island this merged into. |
| 188 | * merged_to value is only set once in polling_island's lifetime (and that too |
| 189 | * only if the island is merged with another island). Because of this, we can |
| 190 | * use gpr_atm type here so that we can do atomic access on this and reduce |
| 191 | * lock contention on 'mu' mutex. |
| 192 | * |
| 193 | * Note that if this field is not NULL (i.e not 0), all the remaining fields |
| 194 | * (except mu and ref_count) are invalid and must be ignored. */ |
| 195 | gpr_atm merged_to; |
Sree Kuchibhotla | f448c34 | 2016-05-19 10:51:24 -0700 | [diff] [blame] | 196 | |
Craig Tiller | 460502e | 2016-10-13 10:02:08 -0700 | [diff] [blame] | 197 | /* Number of threads currently polling on this island */ |
Craig Tiller | d8a3c04 | 2016-09-09 12:42:37 -0700 | [diff] [blame] | 198 | gpr_atm poller_count; |
Craig Tiller | b39307d | 2016-06-30 15:39:13 -0700 | [diff] [blame] | 199 | |
Sree Kuchibhotla | f448c34 | 2016-05-19 10:51:24 -0700 | [diff] [blame] | 200 | /* The fd of the underlying epoll set */ |
| 201 | int epoll_fd; |
| 202 | |
| 203 | /* The file descriptors in the epoll set */ |
| 204 | size_t fd_cnt; |
| 205 | size_t fd_capacity; |
Craig Tiller | baa14a9 | 2017-11-03 09:09:36 -0700 | [diff] [blame] | 206 | grpc_fd** fds; |
Sree Kuchibhotla | f448c34 | 2016-05-19 10:51:24 -0700 | [diff] [blame] | 207 | } polling_island; |
| 208 | |
Sree Kuchibhotla | 0bcbd79 | 2016-06-01 15:43:03 -0700 | [diff] [blame] | 209 | /******************************************************************************* |
| 210 | * Pollset Declarations |
| 211 | */ |
Sree Kuchibhotla | 0bcbd79 | 2016-06-01 15:43:03 -0700 | [diff] [blame] | 212 | struct grpc_pollset_worker { |
Sree Kuchibhotla | 3421724 | 2016-06-29 00:19:07 -0700 | [diff] [blame] | 213 | /* Thread id of this worker */ |
| 214 | pthread_t pt_id; |
| 215 | |
| 216 | /* Used to prevent a worker from getting kicked multiple times */ |
| 217 | gpr_atm is_kicked; |
Craig Tiller | baa14a9 | 2017-11-03 09:09:36 -0700 | [diff] [blame] | 218 | struct grpc_pollset_worker* next; |
| 219 | struct grpc_pollset_worker* prev; |
Sree Kuchibhotla | 0bcbd79 | 2016-06-01 15:43:03 -0700 | [diff] [blame] | 220 | }; |
| 221 | |
| 222 | struct grpc_pollset { |
Sree Kuchibhotla | f6f33d7 | 2016-11-18 11:38:52 -0800 | [diff] [blame] | 223 | poll_obj po; |
| 224 | |
Sree Kuchibhotla | 0bcbd79 | 2016-06-01 15:43:03 -0700 | [diff] [blame] | 225 | grpc_pollset_worker root_worker; |
| 226 | bool kicked_without_pollers; |
| 227 | |
| 228 | bool shutting_down; /* Is the pollset shutting down ? */ |
| 229 | bool finish_shutdown_called; /* Is the 'finish_shutdown_locked()' called ? */ |
Craig Tiller | baa14a9 | 2017-11-03 09:09:36 -0700 | [diff] [blame] | 230 | grpc_closure* shutdown_done; /* Called after after shutdown is complete */ |
Sree Kuchibhotla | 0bcbd79 | 2016-06-01 15:43:03 -0700 | [diff] [blame] | 231 | }; |
| 232 | |
| 233 | /******************************************************************************* |
| 234 | * Pollset-set Declarations |
| 235 | */ |
| 236 | struct grpc_pollset_set { |
Sree Kuchibhotla | f6f33d7 | 2016-11-18 11:38:52 -0800 | [diff] [blame] | 237 | poll_obj po; |
Sree Kuchibhotla | 0bcbd79 | 2016-06-01 15:43:03 -0700 | [diff] [blame] | 238 | }; |
| 239 | |
| 240 | /******************************************************************************* |
Sree Kuchibhotla | 20d0a16 | 2016-06-23 15:14:03 -0700 | [diff] [blame] | 241 | * Common helpers |
| 242 | */ |
| 243 | |
Craig Tiller | baa14a9 | 2017-11-03 09:09:36 -0700 | [diff] [blame] | 244 | static bool append_error(grpc_error** composite, grpc_error* error, |
| 245 | const char* desc) { |
Craig Tiller | f975f74 | 2016-07-01 14:56:27 -0700 | [diff] [blame] | 246 | if (error == GRPC_ERROR_NONE) return true; |
Sree Kuchibhotla | 20d0a16 | 2016-06-23 15:14:03 -0700 | [diff] [blame] | 247 | if (*composite == GRPC_ERROR_NONE) { |
Noah Eisen | 3005ce8 | 2017-03-14 13:38:41 -0700 | [diff] [blame] | 248 | *composite = GRPC_ERROR_CREATE_FROM_COPIED_STRING(desc); |
Sree Kuchibhotla | 20d0a16 | 2016-06-23 15:14:03 -0700 | [diff] [blame] | 249 | } |
| 250 | *composite = grpc_error_add_child(*composite, error); |
Craig Tiller | f975f74 | 2016-07-01 14:56:27 -0700 | [diff] [blame] | 251 | return false; |
Sree Kuchibhotla | 20d0a16 | 2016-06-23 15:14:03 -0700 | [diff] [blame] | 252 | } |
| 253 | |
| 254 | /******************************************************************************* |
Sree Kuchibhotla | 24b1062 | 2016-06-08 15:20:17 -0700 | [diff] [blame] | 255 | * Polling island Definitions |
Sree Kuchibhotla | 0bcbd79 | 2016-06-01 15:43:03 -0700 | [diff] [blame] | 256 | */ |
| 257 | |
Sree Kuchibhotla | 24b1062 | 2016-06-08 15:20:17 -0700 | [diff] [blame] | 258 | /* The wakeup fd that is used to wake up all threads in a Polling island. This |
| 259 | is useful in the polling island merge operation where we need to wakeup all |
| 260 | the threads currently polling the smaller polling island (so that they can |
| 261 | start polling the new/merged polling island) |
| 262 | |
| 263 | NOTE: This fd is initialized to be readable and MUST NOT be consumed i.e the |
| 264 | threads that woke up MUST NOT call grpc_wakeup_fd_consume_wakeup() */ |
| 265 | static grpc_wakeup_fd polling_island_wakeup_fd; |
| 266 | |
Craig Tiller | 2e62013 | 2016-10-10 15:27:44 -0700 | [diff] [blame] | 267 | /* The polling island being polled right now. |
| 268 | See comments in workqueue_maybe_wakeup for why this is tracked. */ |
Craig Tiller | baa14a9 | 2017-11-03 09:09:36 -0700 | [diff] [blame] | 269 | static __thread polling_island* g_current_thread_polling_island; |
Craig Tiller | d8a3c04 | 2016-09-09 12:42:37 -0700 | [diff] [blame] | 270 | |
Craig Tiller | b39307d | 2016-06-30 15:39:13 -0700 | [diff] [blame] | 271 | /* Forward declaration */ |
Yash Tibrewal | 8cf1470 | 2017-12-06 09:47:54 -0800 | [diff] [blame] | 272 | static void polling_island_delete(polling_island* pi); |
Sree Kuchibhotla | 2f8ade0 | 2016-06-17 13:28:38 -0700 | [diff] [blame] | 273 | |
Sree Kuchibhotla | ad2c477 | 2016-06-13 19:06:54 -0700 | [diff] [blame] | 274 | #ifdef GRPC_TSAN |
Sree Kuchibhotla | 41622a8 | 2016-06-13 16:43:14 -0700 | [diff] [blame] | 275 | /* Currently TSAN may incorrectly flag data races between epoll_ctl and |
| 276 | epoll_wait for any grpc_fd structs that are added to the epoll set via |
| 277 | epoll_ctl and are returned (within a very short window) via epoll_wait(). |
| 278 | |
| 279 | To work-around this race, we establish a happens-before relation between |
| 280 | the code just-before epoll_ctl() and the code after epoll_wait() by using |
| 281 | this atomic */ |
| 282 | gpr_atm g_epoll_sync; |
Sree Kuchibhotla | ad2c477 | 2016-06-13 19:06:54 -0700 | [diff] [blame] | 283 | #endif /* defined(GRPC_TSAN) */ |
Sree Kuchibhotla | 41622a8 | 2016-06-13 16:43:14 -0700 | [diff] [blame] | 284 | |
Craig Tiller | baa14a9 | 2017-11-03 09:09:36 -0700 | [diff] [blame] | 285 | static void pi_add_ref(polling_island* pi); |
Yash Tibrewal | 8cf1470 | 2017-12-06 09:47:54 -0800 | [diff] [blame] | 286 | static void pi_unref(polling_island* pi); |
Sree Kuchibhotla | 2f8ade0 | 2016-06-17 13:28:38 -0700 | [diff] [blame] | 287 | |
ncteisen | a135485 | 2017-06-08 16:25:53 -0700 | [diff] [blame] | 288 | #ifndef NDEBUG |
Craig Tiller | baa14a9 | 2017-11-03 09:09:36 -0700 | [diff] [blame] | 289 | static void pi_add_ref_dbg(polling_island* pi, const char* reason, |
| 290 | const char* file, int line) { |
ncteisen | 9ffb149 | 2017-11-10 14:00:49 -0800 | [diff] [blame] | 291 | if (grpc_polling_trace.enabled()) { |
ncteisen | 3ac64f8 | 2017-06-19 17:35:44 -0700 | [diff] [blame] | 292 | gpr_atm old_cnt = gpr_atm_acq_load(&pi->ref_count); |
Craig Tiller | baa14a9 | 2017-11-03 09:09:36 -0700 | [diff] [blame] | 293 | gpr_log(GPR_DEBUG, |
| 294 | "Add ref pi: %p, old:%" PRIdPTR " -> new:%" PRIdPTR |
| 295 | " (%s) - (%s, %d)", |
ncteisen | 3ac64f8 | 2017-06-19 17:35:44 -0700 | [diff] [blame] | 296 | pi, old_cnt, old_cnt + 1, reason, file, line); |
ncteisen | a135485 | 2017-06-08 16:25:53 -0700 | [diff] [blame] | 297 | } |
| 298 | pi_add_ref(pi); |
Sree Kuchibhotla | 2f8ade0 | 2016-06-17 13:28:38 -0700 | [diff] [blame] | 299 | } |
| 300 | |
Yash Tibrewal | 8cf1470 | 2017-12-06 09:47:54 -0800 | [diff] [blame] | 301 | static void pi_unref_dbg(polling_island* pi, const char* reason, |
| 302 | const char* file, int line) { |
ncteisen | 9ffb149 | 2017-11-10 14:00:49 -0800 | [diff] [blame] | 303 | if (grpc_polling_trace.enabled()) { |
ncteisen | 3ac64f8 | 2017-06-19 17:35:44 -0700 | [diff] [blame] | 304 | gpr_atm old_cnt = gpr_atm_acq_load(&pi->ref_count); |
Craig Tiller | baa14a9 | 2017-11-03 09:09:36 -0700 | [diff] [blame] | 305 | gpr_log(GPR_DEBUG, |
| 306 | "Unref pi: %p, old:%" PRIdPTR " -> new:%" PRIdPTR |
| 307 | " (%s) - (%s, %d)", |
ncteisen | 3ac64f8 | 2017-06-19 17:35:44 -0700 | [diff] [blame] | 308 | pi, old_cnt, (old_cnt - 1), reason, file, line); |
ncteisen | a135485 | 2017-06-08 16:25:53 -0700 | [diff] [blame] | 309 | } |
Yash Tibrewal | 8cf1470 | 2017-12-06 09:47:54 -0800 | [diff] [blame] | 310 | pi_unref(pi); |
Sree Kuchibhotla | 2f8ade0 | 2016-06-17 13:28:38 -0700 | [diff] [blame] | 311 | } |
| 312 | #endif |
| 313 | |
Craig Tiller | baa14a9 | 2017-11-03 09:09:36 -0700 | [diff] [blame] | 314 | static void pi_add_ref(polling_island* pi) { |
Craig Tiller | 1500761 | 2016-07-06 09:36:16 -0700 | [diff] [blame] | 315 | gpr_atm_no_barrier_fetch_add(&pi->ref_count, 1); |
| 316 | } |
Sree Kuchibhotla | 2f8ade0 | 2016-06-17 13:28:38 -0700 | [diff] [blame] | 317 | |
Yash Tibrewal | 8cf1470 | 2017-12-06 09:47:54 -0800 | [diff] [blame] | 318 | static void pi_unref(polling_island* pi) { |
Craig Tiller | d8a3c04 | 2016-09-09 12:42:37 -0700 | [diff] [blame] | 319 | /* If ref count went to zero, delete the polling island. |
Sree Kuchibhotla | 6a29545 | 2016-06-23 15:53:10 -0700 | [diff] [blame] | 320 | Note that this deletion not be done under a lock. Once the ref count goes |
| 321 | to zero, we are guaranteed that no one else holds a reference to the |
| 322 | polling island (and that there is no racing pi_add_ref() call either). |
Sree Kuchibhotla | 2f8ade0 | 2016-06-17 13:28:38 -0700 | [diff] [blame] | 323 | |
| 324 | Also, if we are deleting the polling island and the merged_to field is |
| 325 | non-empty, we should remove a ref to the merged_to polling island |
| 326 | */ |
Craig Tiller | d8a3c04 | 2016-09-09 12:42:37 -0700 | [diff] [blame] | 327 | if (1 == gpr_atm_full_fetch_add(&pi->ref_count, -1)) { |
Craig Tiller | baa14a9 | 2017-11-03 09:09:36 -0700 | [diff] [blame] | 328 | polling_island* next = (polling_island*)gpr_atm_acq_load(&pi->merged_to); |
Yash Tibrewal | 8cf1470 | 2017-12-06 09:47:54 -0800 | [diff] [blame] | 329 | polling_island_delete(pi); |
Craig Tiller | 4782d92 | 2017-11-10 09:53:21 -0800 | [diff] [blame] | 330 | if (next != nullptr) { |
Yash Tibrewal | 8cf1470 | 2017-12-06 09:47:54 -0800 | [diff] [blame] | 331 | PI_UNREF(next, "pi_delete"); /* Recursive call */ |
Sree Kuchibhotla | 2f8ade0 | 2016-06-17 13:28:38 -0700 | [diff] [blame] | 332 | } |
| 333 | } |
Sree Kuchibhotla | 2f8ade0 | 2016-06-17 13:28:38 -0700 | [diff] [blame] | 334 | } |
| 335 | |
Sree Kuchibhotla | 9442bab | 2016-05-20 17:54:06 -0700 | [diff] [blame] | 336 | /* The caller is expected to hold pi->mu lock before calling this function */ |
Craig Tiller | baa14a9 | 2017-11-03 09:09:36 -0700 | [diff] [blame] | 337 | static void polling_island_add_fds_locked(polling_island* pi, grpc_fd** fds, |
Sree Kuchibhotla | 20d0a16 | 2016-06-23 15:14:03 -0700 | [diff] [blame] | 338 | size_t fd_count, bool add_fd_refs, |
Craig Tiller | baa14a9 | 2017-11-03 09:09:36 -0700 | [diff] [blame] | 339 | grpc_error** error) { |
Sree Kuchibhotla | f448c34 | 2016-05-19 10:51:24 -0700 | [diff] [blame] | 340 | int err; |
Sree Kuchibhotla | 9442bab | 2016-05-20 17:54:06 -0700 | [diff] [blame] | 341 | size_t i; |
Sree Kuchibhotla | f448c34 | 2016-05-19 10:51:24 -0700 | [diff] [blame] | 342 | struct epoll_event ev; |
Craig Tiller | baa14a9 | 2017-11-03 09:09:36 -0700 | [diff] [blame] | 343 | char* err_msg; |
| 344 | const char* err_desc = "polling_island_add_fds"; |
Sree Kuchibhotla | f448c34 | 2016-05-19 10:51:24 -0700 | [diff] [blame] | 345 | |
Sree Kuchibhotla | ad2c477 | 2016-06-13 19:06:54 -0700 | [diff] [blame] | 346 | #ifdef GRPC_TSAN |
Sree Kuchibhotla | 41622a8 | 2016-06-13 16:43:14 -0700 | [diff] [blame] | 347 | /* See the definition of g_epoll_sync for more context */ |
Sree Kuchibhotla | 0224dcc | 2016-06-22 18:04:00 -0700 | [diff] [blame] | 348 | gpr_atm_rel_store(&g_epoll_sync, (gpr_atm)0); |
Sree Kuchibhotla | ad2c477 | 2016-06-13 19:06:54 -0700 | [diff] [blame] | 349 | #endif /* defined(GRPC_TSAN) */ |
Sree Kuchibhotla | 41622a8 | 2016-06-13 16:43:14 -0700 | [diff] [blame] | 350 | |
Sree Kuchibhotla | 9442bab | 2016-05-20 17:54:06 -0700 | [diff] [blame] | 351 | for (i = 0; i < fd_count; i++) { |
| 352 | ev.events = (uint32_t)(EPOLLIN | EPOLLOUT | EPOLLET); |
| 353 | ev.data.ptr = fds[i]; |
| 354 | err = epoll_ctl(pi->epoll_fd, EPOLL_CTL_ADD, fds[i]->fd, &ev); |
Sree Kuchibhotla | f448c34 | 2016-05-19 10:51:24 -0700 | [diff] [blame] | 355 | |
Sree Kuchibhotla | 0bcbd79 | 2016-06-01 15:43:03 -0700 | [diff] [blame] | 356 | if (err < 0) { |
| 357 | if (errno != EEXIST) { |
Sree Kuchibhotla | 20d0a16 | 2016-06-23 15:14:03 -0700 | [diff] [blame] | 358 | gpr_asprintf( |
| 359 | &err_msg, |
| 360 | "epoll_ctl (epoll_fd: %d) add fd: %d failed with error: %d (%s)", |
| 361 | pi->epoll_fd, fds[i]->fd, errno, strerror(errno)); |
| 362 | append_error(error, GRPC_OS_ERROR(errno, err_msg), err_desc); |
| 363 | gpr_free(err_msg); |
Sree Kuchibhotla | 0bcbd79 | 2016-06-01 15:43:03 -0700 | [diff] [blame] | 364 | } |
| 365 | |
Sree Kuchibhotla | 9442bab | 2016-05-20 17:54:06 -0700 | [diff] [blame] | 366 | continue; |
| 367 | } |
| 368 | |
| 369 | if (pi->fd_cnt == pi->fd_capacity) { |
| 370 | pi->fd_capacity = GPR_MAX(pi->fd_capacity + 8, pi->fd_cnt * 3 / 2); |
Yash Tibrewal | ca3c1c0 | 2017-09-07 22:47:16 -0700 | [diff] [blame] | 371 | pi->fds = |
Craig Tiller | baa14a9 | 2017-11-03 09:09:36 -0700 | [diff] [blame] | 372 | (grpc_fd**)gpr_realloc(pi->fds, sizeof(grpc_fd*) * pi->fd_capacity); |
Sree Kuchibhotla | 9442bab | 2016-05-20 17:54:06 -0700 | [diff] [blame] | 373 | } |
| 374 | |
| 375 | pi->fds[pi->fd_cnt++] = fds[i]; |
Sree Kuchibhotla | 0bcbd79 | 2016-06-01 15:43:03 -0700 | [diff] [blame] | 376 | if (add_fd_refs) { |
| 377 | GRPC_FD_REF(fds[i], "polling_island"); |
| 378 | } |
Sree Kuchibhotla | f448c34 | 2016-05-19 10:51:24 -0700 | [diff] [blame] | 379 | } |
Sree Kuchibhotla | f448c34 | 2016-05-19 10:51:24 -0700 | [diff] [blame] | 380 | } |
| 381 | |
Sree Kuchibhotla | 24b1062 | 2016-06-08 15:20:17 -0700 | [diff] [blame] | 382 | /* The caller is expected to hold pi->mu before calling this */ |
Craig Tiller | baa14a9 | 2017-11-03 09:09:36 -0700 | [diff] [blame] | 383 | static void polling_island_add_wakeup_fd_locked(polling_island* pi, |
| 384 | grpc_wakeup_fd* wakeup_fd, |
| 385 | grpc_error** error) { |
Sree Kuchibhotla | 24b1062 | 2016-06-08 15:20:17 -0700 | [diff] [blame] | 386 | struct epoll_event ev; |
| 387 | int err; |
Craig Tiller | baa14a9 | 2017-11-03 09:09:36 -0700 | [diff] [blame] | 388 | char* err_msg; |
| 389 | const char* err_desc = "polling_island_add_wakeup_fd"; |
Sree Kuchibhotla | 24b1062 | 2016-06-08 15:20:17 -0700 | [diff] [blame] | 390 | |
| 391 | ev.events = (uint32_t)(EPOLLIN | EPOLLET); |
| 392 | ev.data.ptr = wakeup_fd; |
| 393 | err = epoll_ctl(pi->epoll_fd, EPOLL_CTL_ADD, |
| 394 | GRPC_WAKEUP_FD_GET_READ_FD(wakeup_fd), &ev); |
Sree Kuchibhotla | 20d0a16 | 2016-06-23 15:14:03 -0700 | [diff] [blame] | 395 | if (err < 0 && errno != EEXIST) { |
| 396 | gpr_asprintf(&err_msg, |
| 397 | "epoll_ctl (epoll_fd: %d) add wakeup fd: %d failed with " |
| 398 | "error: %d (%s)", |
Craig Tiller | c357179 | 2017-05-02 12:33:38 -0700 | [diff] [blame] | 399 | pi->epoll_fd, GRPC_WAKEUP_FD_GET_READ_FD(wakeup_fd), errno, |
| 400 | strerror(errno)); |
Sree Kuchibhotla | 20d0a16 | 2016-06-23 15:14:03 -0700 | [diff] [blame] | 401 | append_error(error, GRPC_OS_ERROR(errno, err_msg), err_desc); |
| 402 | gpr_free(err_msg); |
Sree Kuchibhotla | 24b1062 | 2016-06-08 15:20:17 -0700 | [diff] [blame] | 403 | } |
| 404 | } |
| 405 | |
Sree Kuchibhotla | 9442bab | 2016-05-20 17:54:06 -0700 | [diff] [blame] | 406 | /* The caller is expected to hold pi->mu lock before calling this function */ |
Craig Tiller | baa14a9 | 2017-11-03 09:09:36 -0700 | [diff] [blame] | 407 | static void polling_island_remove_all_fds_locked(polling_island* pi, |
Sree Kuchibhotla | 20d0a16 | 2016-06-23 15:14:03 -0700 | [diff] [blame] | 408 | bool remove_fd_refs, |
Craig Tiller | baa14a9 | 2017-11-03 09:09:36 -0700 | [diff] [blame] | 409 | grpc_error** error) { |
Sree Kuchibhotla | 9442bab | 2016-05-20 17:54:06 -0700 | [diff] [blame] | 410 | int err; |
| 411 | size_t i; |
Craig Tiller | baa14a9 | 2017-11-03 09:09:36 -0700 | [diff] [blame] | 412 | char* err_msg; |
| 413 | const char* err_desc = "polling_island_remove_fds"; |
Sree Kuchibhotla | 9442bab | 2016-05-20 17:54:06 -0700 | [diff] [blame] | 414 | |
| 415 | for (i = 0; i < pi->fd_cnt; i++) { |
Craig Tiller | 4782d92 | 2017-11-10 09:53:21 -0800 | [diff] [blame] | 416 | err = epoll_ctl(pi->epoll_fd, EPOLL_CTL_DEL, pi->fds[i]->fd, nullptr); |
Sree Kuchibhotla | 9442bab | 2016-05-20 17:54:06 -0700 | [diff] [blame] | 417 | if (err < 0 && errno != ENOENT) { |
Sree Kuchibhotla | 20d0a16 | 2016-06-23 15:14:03 -0700 | [diff] [blame] | 418 | gpr_asprintf(&err_msg, |
| 419 | "epoll_ctl (epoll_fd: %d) delete fds[%zu]: %d failed with " |
| 420 | "error: %d (%s)", |
| 421 | pi->epoll_fd, i, pi->fds[i]->fd, errno, strerror(errno)); |
| 422 | append_error(error, GRPC_OS_ERROR(errno, err_msg), err_desc); |
| 423 | gpr_free(err_msg); |
Sree Kuchibhotla | ad162ba | 2016-06-06 16:23:37 -0700 | [diff] [blame] | 424 | } |
| 425 | |
| 426 | if (remove_fd_refs) { |
| 427 | GRPC_FD_UNREF(pi->fds[i], "polling_island"); |
Sree Kuchibhotla | 9442bab | 2016-05-20 17:54:06 -0700 | [diff] [blame] | 428 | } |
| 429 | } |
| 430 | |
| 431 | pi->fd_cnt = 0; |
| 432 | } |
| 433 | |
Sree Kuchibhotla | 9442bab | 2016-05-20 17:54:06 -0700 | [diff] [blame] | 434 | /* The caller is expected to hold pi->mu lock before calling this function */ |
Craig Tiller | baa14a9 | 2017-11-03 09:09:36 -0700 | [diff] [blame] | 435 | static void polling_island_remove_fd_locked(polling_island* pi, grpc_fd* fd, |
Sree Kuchibhotla | 20d0a16 | 2016-06-23 15:14:03 -0700 | [diff] [blame] | 436 | bool is_fd_closed, |
Craig Tiller | baa14a9 | 2017-11-03 09:09:36 -0700 | [diff] [blame] | 437 | grpc_error** error) { |
Sree Kuchibhotla | 9442bab | 2016-05-20 17:54:06 -0700 | [diff] [blame] | 438 | int err; |
| 439 | size_t i; |
Craig Tiller | baa14a9 | 2017-11-03 09:09:36 -0700 | [diff] [blame] | 440 | char* err_msg; |
| 441 | const char* err_desc = "polling_island_remove_fd"; |
Sree Kuchibhotla | 0bcbd79 | 2016-06-01 15:43:03 -0700 | [diff] [blame] | 442 | |
Sree Kuchibhotla | 79a6233 | 2016-06-04 14:01:03 -0700 | [diff] [blame] | 443 | /* If fd is already closed, then it would have been automatically been removed |
| 444 | from the epoll set */ |
| 445 | if (!is_fd_closed) { |
Craig Tiller | 4782d92 | 2017-11-10 09:53:21 -0800 | [diff] [blame] | 446 | err = epoll_ctl(pi->epoll_fd, EPOLL_CTL_DEL, fd->fd, nullptr); |
Sree Kuchibhotla | 0bcbd79 | 2016-06-01 15:43:03 -0700 | [diff] [blame] | 447 | if (err < 0 && errno != ENOENT) { |
Sree Kuchibhotla | 20d0a16 | 2016-06-23 15:14:03 -0700 | [diff] [blame] | 448 | gpr_asprintf( |
| 449 | &err_msg, |
| 450 | "epoll_ctl (epoll_fd: %d) del fd: %d failed with error: %d (%s)", |
| 451 | pi->epoll_fd, fd->fd, errno, strerror(errno)); |
| 452 | append_error(error, GRPC_OS_ERROR(errno, err_msg), err_desc); |
| 453 | gpr_free(err_msg); |
Sree Kuchibhotla | 0bcbd79 | 2016-06-01 15:43:03 -0700 | [diff] [blame] | 454 | } |
Sree Kuchibhotla | 9442bab | 2016-05-20 17:54:06 -0700 | [diff] [blame] | 455 | } |
| 456 | |
| 457 | for (i = 0; i < pi->fd_cnt; i++) { |
| 458 | if (pi->fds[i] == fd) { |
| 459 | pi->fds[i] = pi->fds[--pi->fd_cnt]; |
Sree Kuchibhotla | 79a6233 | 2016-06-04 14:01:03 -0700 | [diff] [blame] | 460 | GRPC_FD_UNREF(fd, "polling_island"); |
Sree Kuchibhotla | 9442bab | 2016-05-20 17:54:06 -0700 | [diff] [blame] | 461 | break; |
| 462 | } |
| 463 | } |
| 464 | } |
| 465 | |
Sree Kuchibhotla | 20d0a16 | 2016-06-23 15:14:03 -0700 | [diff] [blame] | 466 | /* Might return NULL in case of an error */ |
Yash Tibrewal | 8cf1470 | 2017-12-06 09:47:54 -0800 | [diff] [blame] | 467 | static polling_island* polling_island_create(grpc_fd* initial_fd, |
Craig Tiller | baa14a9 | 2017-11-03 09:09:36 -0700 | [diff] [blame] | 468 | grpc_error** error) { |
Craig Tiller | 4782d92 | 2017-11-10 09:53:21 -0800 | [diff] [blame] | 469 | polling_island* pi = nullptr; |
Craig Tiller | baa14a9 | 2017-11-03 09:09:36 -0700 | [diff] [blame] | 470 | const char* err_desc = "polling_island_create"; |
Sree Kuchibhotla | 0bcbd79 | 2016-06-01 15:43:03 -0700 | [diff] [blame] | 471 | |
Craig Tiller | b39307d | 2016-06-30 15:39:13 -0700 | [diff] [blame] | 472 | *error = GRPC_ERROR_NONE; |
Sree Kuchibhotla | f448c34 | 2016-05-19 10:51:24 -0700 | [diff] [blame] | 473 | |
Craig Tiller | baa14a9 | 2017-11-03 09:09:36 -0700 | [diff] [blame] | 474 | pi = (polling_island*)gpr_malloc(sizeof(*pi)); |
Craig Tiller | b39307d | 2016-06-30 15:39:13 -0700 | [diff] [blame] | 475 | gpr_mu_init(&pi->mu); |
| 476 | pi->fd_cnt = 0; |
| 477 | pi->fd_capacity = 0; |
Craig Tiller | 4782d92 | 2017-11-10 09:53:21 -0800 | [diff] [blame] | 478 | pi->fds = nullptr; |
Craig Tiller | b39307d | 2016-06-30 15:39:13 -0700 | [diff] [blame] | 479 | pi->epoll_fd = -1; |
Craig Tiller | d8a3c04 | 2016-09-09 12:42:37 -0700 | [diff] [blame] | 480 | |
Craig Tiller | 1500761 | 2016-07-06 09:36:16 -0700 | [diff] [blame] | 481 | gpr_atm_rel_store(&pi->ref_count, 0); |
Craig Tiller | d8a3c04 | 2016-09-09 12:42:37 -0700 | [diff] [blame] | 482 | gpr_atm_rel_store(&pi->poller_count, 0); |
Yash Tibrewal | 8cf1470 | 2017-12-06 09:47:54 -0800 | [diff] [blame] | 483 | gpr_atm_rel_store(&pi->merged_to, (gpr_atm) nullptr); |
Sree Kuchibhotla | 2f8ade0 | 2016-06-17 13:28:38 -0700 | [diff] [blame] | 484 | |
Sree Kuchibhotla | 0bcbd79 | 2016-06-01 15:43:03 -0700 | [diff] [blame] | 485 | pi->epoll_fd = epoll_create1(EPOLL_CLOEXEC); |
Sree Kuchibhotla | 41622a8 | 2016-06-13 16:43:14 -0700 | [diff] [blame] | 486 | |
Sree Kuchibhotla | 0bcbd79 | 2016-06-01 15:43:03 -0700 | [diff] [blame] | 487 | if (pi->epoll_fd < 0) { |
Craig Tiller | b39307d | 2016-06-30 15:39:13 -0700 | [diff] [blame] | 488 | append_error(error, GRPC_OS_ERROR(errno, "epoll_create1"), err_desc); |
| 489 | goto done; |
Sree Kuchibhotla | f448c34 | 2016-05-19 10:51:24 -0700 | [diff] [blame] | 490 | } |
| 491 | |
Craig Tiller | 4782d92 | 2017-11-10 09:53:21 -0800 | [diff] [blame] | 492 | if (initial_fd != nullptr) { |
Craig Tiller | b39307d | 2016-06-30 15:39:13 -0700 | [diff] [blame] | 493 | polling_island_add_fds_locked(pi, &initial_fd, 1, true, error); |
Craig Tiller | b39307d | 2016-06-30 15:39:13 -0700 | [diff] [blame] | 494 | } |
| 495 | |
Craig Tiller | b39307d | 2016-06-30 15:39:13 -0700 | [diff] [blame] | 496 | done: |
| 497 | if (*error != GRPC_ERROR_NONE) { |
Yash Tibrewal | 8cf1470 | 2017-12-06 09:47:54 -0800 | [diff] [blame] | 498 | polling_island_delete(pi); |
Craig Tiller | 4782d92 | 2017-11-10 09:53:21 -0800 | [diff] [blame] | 499 | pi = nullptr; |
Craig Tiller | b39307d | 2016-06-30 15:39:13 -0700 | [diff] [blame] | 500 | } |
Sree Kuchibhotla | f448c34 | 2016-05-19 10:51:24 -0700 | [diff] [blame] | 501 | return pi; |
| 502 | } |
| 503 | |
Yash Tibrewal | 8cf1470 | 2017-12-06 09:47:54 -0800 | [diff] [blame] | 504 | static void polling_island_delete(polling_island* pi) { |
Sree Kuchibhotla | 9442bab | 2016-05-20 17:54:06 -0700 | [diff] [blame] | 505 | GPR_ASSERT(pi->fd_cnt == 0); |
| 506 | |
Craig Tiller | 0a06cd7 | 2016-07-14 13:21:24 -0700 | [diff] [blame] | 507 | if (pi->epoll_fd >= 0) { |
| 508 | close(pi->epoll_fd); |
| 509 | } |
Craig Tiller | b39307d | 2016-06-30 15:39:13 -0700 | [diff] [blame] | 510 | gpr_mu_destroy(&pi->mu); |
| 511 | gpr_free(pi->fds); |
| 512 | gpr_free(pi); |
Sree Kuchibhotla | 9442bab | 2016-05-20 17:54:06 -0700 | [diff] [blame] | 513 | } |
| 514 | |
Sree Kuchibhotla | 229533b1 | 2016-06-21 20:42:52 -0700 | [diff] [blame] | 515 | /* Attempts to gets the last polling island in the linked list (liked by the |
| 516 | * 'merged_to' field). Since this does not lock the polling island, there are no |
| 517 | * guarantees that the island returned is the last island */ |
Craig Tiller | baa14a9 | 2017-11-03 09:09:36 -0700 | [diff] [blame] | 518 | static polling_island* polling_island_maybe_get_latest(polling_island* pi) { |
| 519 | polling_island* next = (polling_island*)gpr_atm_acq_load(&pi->merged_to); |
Craig Tiller | 4782d92 | 2017-11-10 09:53:21 -0800 | [diff] [blame] | 520 | while (next != nullptr) { |
Sree Kuchibhotla | 229533b1 | 2016-06-21 20:42:52 -0700 | [diff] [blame] | 521 | pi = next; |
Craig Tiller | baa14a9 | 2017-11-03 09:09:36 -0700 | [diff] [blame] | 522 | next = (polling_island*)gpr_atm_acq_load(&pi->merged_to); |
Sree Kuchibhotla | 229533b1 | 2016-06-21 20:42:52 -0700 | [diff] [blame] | 523 | } |
| 524 | |
| 525 | return pi; |
| 526 | } |
| 527 | |
Sree Kuchibhotla | 2f8ade0 | 2016-06-17 13:28:38 -0700 | [diff] [blame] | 528 | /* Gets the lock on the *latest* polling island i.e the last polling island in |
Sree Kuchibhotla | 229533b1 | 2016-06-21 20:42:52 -0700 | [diff] [blame] | 529 | the linked list (linked by the 'merged_to' field). Call gpr_mu_unlock on the |
Sree Kuchibhotla | 2f8ade0 | 2016-06-17 13:28:38 -0700 | [diff] [blame] | 530 | returned polling island's mu. |
| 531 | Usage: To lock/unlock polling island "pi", do the following: |
| 532 | polling_island *pi_latest = polling_island_lock(pi); |
| 533 | ... |
| 534 | ... critical section .. |
| 535 | ... |
Sree Kuchibhotla | 229533b1 | 2016-06-21 20:42:52 -0700 | [diff] [blame] | 536 | gpr_mu_unlock(&pi_latest->mu); // NOTE: use pi_latest->mu. NOT pi->mu */ |
Craig Tiller | baa14a9 | 2017-11-03 09:09:36 -0700 | [diff] [blame] | 537 | static polling_island* polling_island_lock(polling_island* pi) { |
Craig Tiller | 4782d92 | 2017-11-10 09:53:21 -0800 | [diff] [blame] | 538 | polling_island* next = nullptr; |
Sree Kuchibhotla | 229533b1 | 2016-06-21 20:42:52 -0700 | [diff] [blame] | 539 | |
Sree Kuchibhotla | 2f8ade0 | 2016-06-17 13:28:38 -0700 | [diff] [blame] | 540 | while (true) { |
Craig Tiller | baa14a9 | 2017-11-03 09:09:36 -0700 | [diff] [blame] | 541 | next = (polling_island*)gpr_atm_acq_load(&pi->merged_to); |
Craig Tiller | 4782d92 | 2017-11-10 09:53:21 -0800 | [diff] [blame] | 542 | if (next == nullptr) { |
Sree Kuchibhotla | 229533b1 | 2016-06-21 20:42:52 -0700 | [diff] [blame] | 543 | /* Looks like 'pi' is the last node in the linked list but unless we check |
| 544 | this by holding the pi->mu lock, we cannot be sure (i.e without the |
| 545 | pi->mu lock, we don't prevent island merges). |
| 546 | To be absolutely sure, check once more by holding the pi->mu lock */ |
Sree Kuchibhotla | 2f8ade0 | 2016-06-17 13:28:38 -0700 | [diff] [blame] | 547 | gpr_mu_lock(&pi->mu); |
Craig Tiller | baa14a9 | 2017-11-03 09:09:36 -0700 | [diff] [blame] | 548 | next = (polling_island*)gpr_atm_acq_load(&pi->merged_to); |
Craig Tiller | 4782d92 | 2017-11-10 09:53:21 -0800 | [diff] [blame] | 549 | if (next == nullptr) { |
Sree Kuchibhotla | 229533b1 | 2016-06-21 20:42:52 -0700 | [diff] [blame] | 550 | /* pi is infact the last node and we have the pi->mu lock. we're done */ |
Sree Kuchibhotla | 2f8ade0 | 2016-06-17 13:28:38 -0700 | [diff] [blame] | 551 | break; |
| 552 | } |
Sree Kuchibhotla | 229533b1 | 2016-06-21 20:42:52 -0700 | [diff] [blame] | 553 | |
| 554 | /* pi->merged_to is not NULL i.e pi isn't the last node anymore. pi->mu |
| 555 | * isn't the lock we are interested in. Continue traversing the list */ |
Sree Kuchibhotla | 2f8ade0 | 2016-06-17 13:28:38 -0700 | [diff] [blame] | 556 | gpr_mu_unlock(&pi->mu); |
| 557 | } |
| 558 | |
Sree Kuchibhotla | 9442bab | 2016-05-20 17:54:06 -0700 | [diff] [blame] | 559 | pi = next; |
Sree Kuchibhotla | 9442bab | 2016-05-20 17:54:06 -0700 | [diff] [blame] | 560 | } |
| 561 | |
Sree Kuchibhotla | 9442bab | 2016-05-20 17:54:06 -0700 | [diff] [blame] | 562 | return pi; |
| 563 | } |
| 564 | |
Sree Kuchibhotla | 20d0a16 | 2016-06-23 15:14:03 -0700 | [diff] [blame] | 565 | /* Gets the lock on the *latest* polling islands in the linked lists pointed by |
| 566 | *p and *q (and also updates *p and *q to point to the latest polling islands) |
| 567 | |
Sree Kuchibhotla | 2f8ade0 | 2016-06-17 13:28:38 -0700 | [diff] [blame] | 568 | This function is needed because calling the following block of code to obtain |
| 569 | locks on polling islands (*p and *q) is prone to deadlocks. |
| 570 | { |
Sree Kuchibhotla | 229533b1 | 2016-06-21 20:42:52 -0700 | [diff] [blame] | 571 | polling_island_lock(*p, true); |
| 572 | polling_island_lock(*q, true); |
Sree Kuchibhotla | 2f8ade0 | 2016-06-17 13:28:38 -0700 | [diff] [blame] | 573 | } |
| 574 | |
Sree Kuchibhotla | 229533b1 | 2016-06-21 20:42:52 -0700 | [diff] [blame] | 575 | Usage/example: |
Sree Kuchibhotla | 2f8ade0 | 2016-06-17 13:28:38 -0700 | [diff] [blame] | 576 | polling_island *p1; |
| 577 | polling_island *p2; |
| 578 | .. |
| 579 | polling_island_lock_pair(&p1, &p2); |
| 580 | .. |
| 581 | .. Critical section with both p1 and p2 locked |
| 582 | .. |
Sree Kuchibhotla | 20d0a16 | 2016-06-23 15:14:03 -0700 | [diff] [blame] | 583 | // Release locks: Always call polling_island_unlock_pair() to release locks |
| 584 | polling_island_unlock_pair(p1, p2); |
Sree Kuchibhotla | 2f8ade0 | 2016-06-17 13:28:38 -0700 | [diff] [blame] | 585 | */ |
Craig Tiller | baa14a9 | 2017-11-03 09:09:36 -0700 | [diff] [blame] | 586 | static void polling_island_lock_pair(polling_island** p, polling_island** q) { |
| 587 | polling_island* pi_1 = *p; |
| 588 | polling_island* pi_2 = *q; |
Craig Tiller | 4782d92 | 2017-11-10 09:53:21 -0800 | [diff] [blame] | 589 | polling_island* next_1 = nullptr; |
| 590 | polling_island* next_2 = nullptr; |
Sree Kuchibhotla | 9442bab | 2016-05-20 17:54:06 -0700 | [diff] [blame] | 591 | |
Sree Kuchibhotla | 2f8ade0 | 2016-06-17 13:28:38 -0700 | [diff] [blame] | 592 | /* The algorithm is simple: |
| 593 | - Go to the last polling islands in the linked lists *pi_1 and *pi_2 (and |
| 594 | keep updating pi_1 and pi_2) |
| 595 | - Then obtain locks on the islands by following a lock order rule of |
| 596 | locking polling_island with lower address first |
| 597 | Special case: Before obtaining the locks, check if pi_1 and pi_2 are |
| 598 | pointing to the same island. If that is the case, we can just call |
| 599 | polling_island_lock() |
| 600 | - After obtaining both the locks, double check that the polling islands |
| 601 | are still the last polling islands in their respective linked lists |
| 602 | (this is because there might have been polling island merges before |
| 603 | we got the lock) |
| 604 | - If the polling islands are the last islands, we are done. If not, |
| 605 | release the locks and continue the process from the first step */ |
| 606 | while (true) { |
Craig Tiller | baa14a9 | 2017-11-03 09:09:36 -0700 | [diff] [blame] | 607 | next_1 = (polling_island*)gpr_atm_acq_load(&pi_1->merged_to); |
Craig Tiller | 4782d92 | 2017-11-10 09:53:21 -0800 | [diff] [blame] | 608 | while (next_1 != nullptr) { |
Sree Kuchibhotla | 2f8ade0 | 2016-06-17 13:28:38 -0700 | [diff] [blame] | 609 | pi_1 = next_1; |
Craig Tiller | baa14a9 | 2017-11-03 09:09:36 -0700 | [diff] [blame] | 610 | next_1 = (polling_island*)gpr_atm_acq_load(&pi_1->merged_to); |
Sree Kuchibhotla | 9442bab | 2016-05-20 17:54:06 -0700 | [diff] [blame] | 611 | } |
| 612 | |
Craig Tiller | baa14a9 | 2017-11-03 09:09:36 -0700 | [diff] [blame] | 613 | next_2 = (polling_island*)gpr_atm_acq_load(&pi_2->merged_to); |
Craig Tiller | 4782d92 | 2017-11-10 09:53:21 -0800 | [diff] [blame] | 614 | while (next_2 != nullptr) { |
Sree Kuchibhotla | 2f8ade0 | 2016-06-17 13:28:38 -0700 | [diff] [blame] | 615 | pi_2 = next_2; |
Craig Tiller | baa14a9 | 2017-11-03 09:09:36 -0700 | [diff] [blame] | 616 | next_2 = (polling_island*)gpr_atm_acq_load(&pi_2->merged_to); |
Sree Kuchibhotla | 2f8ade0 | 2016-06-17 13:28:38 -0700 | [diff] [blame] | 617 | } |
Sree Kuchibhotla | 9442bab | 2016-05-20 17:54:06 -0700 | [diff] [blame] | 618 | |
Sree Kuchibhotla | 2f8ade0 | 2016-06-17 13:28:38 -0700 | [diff] [blame] | 619 | if (pi_1 == pi_2) { |
| 620 | pi_1 = pi_2 = polling_island_lock(pi_1); |
| 621 | break; |
| 622 | } |
| 623 | |
| 624 | if (pi_1 < pi_2) { |
Sree Kuchibhotla | 9442bab | 2016-05-20 17:54:06 -0700 | [diff] [blame] | 625 | gpr_mu_lock(&pi_1->mu); |
Sree Kuchibhotla | 2f8ade0 | 2016-06-17 13:28:38 -0700 | [diff] [blame] | 626 | gpr_mu_lock(&pi_2->mu); |
| 627 | } else { |
| 628 | gpr_mu_lock(&pi_2->mu); |
| 629 | gpr_mu_lock(&pi_1->mu); |
Sree Kuchibhotla | 9442bab | 2016-05-20 17:54:06 -0700 | [diff] [blame] | 630 | } |
| 631 | |
Craig Tiller | baa14a9 | 2017-11-03 09:09:36 -0700 | [diff] [blame] | 632 | next_1 = (polling_island*)gpr_atm_acq_load(&pi_1->merged_to); |
| 633 | next_2 = (polling_island*)gpr_atm_acq_load(&pi_2->merged_to); |
Craig Tiller | 4782d92 | 2017-11-10 09:53:21 -0800 | [diff] [blame] | 634 | if (next_1 == nullptr && next_2 == nullptr) { |
Sree Kuchibhotla | 2f8ade0 | 2016-06-17 13:28:38 -0700 | [diff] [blame] | 635 | break; |
Sree Kuchibhotla | 9442bab | 2016-05-20 17:54:06 -0700 | [diff] [blame] | 636 | } |
Sree Kuchibhotla | 9442bab | 2016-05-20 17:54:06 -0700 | [diff] [blame] | 637 | |
Sree Kuchibhotla | 2f8ade0 | 2016-06-17 13:28:38 -0700 | [diff] [blame] | 638 | gpr_mu_unlock(&pi_1->mu); |
| 639 | gpr_mu_unlock(&pi_2->mu); |
Sree Kuchibhotla | 9442bab | 2016-05-20 17:54:06 -0700 | [diff] [blame] | 640 | } |
| 641 | |
| 642 | *p = pi_1; |
| 643 | *q = pi_2; |
| 644 | } |
| 645 | |
Craig Tiller | baa14a9 | 2017-11-03 09:09:36 -0700 | [diff] [blame] | 646 | static void polling_island_unlock_pair(polling_island* p, polling_island* q) { |
Sree Kuchibhotla | 20d0a16 | 2016-06-23 15:14:03 -0700 | [diff] [blame] | 647 | if (p == q) { |
| 648 | gpr_mu_unlock(&p->mu); |
| 649 | } else { |
| 650 | gpr_mu_unlock(&p->mu); |
| 651 | gpr_mu_unlock(&q->mu); |
| 652 | } |
| 653 | } |
| 654 | |
Craig Tiller | baa14a9 | 2017-11-03 09:09:36 -0700 | [diff] [blame] | 655 | static polling_island* polling_island_merge(polling_island* p, |
| 656 | polling_island* q, |
| 657 | grpc_error** error) { |
Sree Kuchibhotla | 0bcbd79 | 2016-06-01 15:43:03 -0700 | [diff] [blame] | 658 | /* Get locks on both the polling islands */ |
Sree Kuchibhotla | 2f8ade0 | 2016-06-17 13:28:38 -0700 | [diff] [blame] | 659 | polling_island_lock_pair(&p, &q); |
Sree Kuchibhotla | 9442bab | 2016-05-20 17:54:06 -0700 | [diff] [blame] | 660 | |
Sree Kuchibhotla | 20d0a16 | 2016-06-23 15:14:03 -0700 | [diff] [blame] | 661 | if (p != q) { |
| 662 | /* Make sure that p points to the polling island with fewer fds than q */ |
| 663 | if (p->fd_cnt > q->fd_cnt) { |
Craig Tiller | baa14a9 | 2017-11-03 09:09:36 -0700 | [diff] [blame] | 664 | GPR_SWAP(polling_island*, p, q); |
Sree Kuchibhotla | 20d0a16 | 2016-06-23 15:14:03 -0700 | [diff] [blame] | 665 | } |
| 666 | |
| 667 | /* Merge p with q i.e move all the fds from p (The one with fewer fds) to q |
| 668 | Note that the refcounts on the fds being moved will not change here. |
| 669 | This is why the last param in the following two functions is 'false') */ |
| 670 | polling_island_add_fds_locked(q, p->fds, p->fd_cnt, false, error); |
| 671 | polling_island_remove_all_fds_locked(p, false, error); |
| 672 | |
| 673 | /* Wakeup all the pollers (if any) on p so that they pickup this change */ |
| 674 | polling_island_add_wakeup_fd_locked(p, &polling_island_wakeup_fd, error); |
| 675 | |
| 676 | /* Add the 'merged_to' link from p --> q */ |
| 677 | gpr_atm_rel_store(&p->merged_to, (gpr_atm)q); |
| 678 | PI_ADD_REF(q, "pi_merge"); /* To account for the new incoming ref from p */ |
Sree Kuchibhotla | 9442bab | 2016-05-20 17:54:06 -0700 | [diff] [blame] | 679 | } |
Sree Kuchibhotla | 20d0a16 | 2016-06-23 15:14:03 -0700 | [diff] [blame] | 680 | /* else if p == q, nothing needs to be done */ |
Sree Kuchibhotla | 9442bab | 2016-05-20 17:54:06 -0700 | [diff] [blame] | 681 | |
Sree Kuchibhotla | 20d0a16 | 2016-06-23 15:14:03 -0700 | [diff] [blame] | 682 | polling_island_unlock_pair(p, q); |
Sree Kuchibhotla | 9442bab | 2016-05-20 17:54:06 -0700 | [diff] [blame] | 683 | |
Sree Kuchibhotla | 20d0a16 | 2016-06-23 15:14:03 -0700 | [diff] [blame] | 684 | /* Return the merged polling island (Note that no merge would have happened |
| 685 | if p == q which is ok) */ |
Sree Kuchibhotla | 0bcbd79 | 2016-06-01 15:43:03 -0700 | [diff] [blame] | 686 | return q; |
Sree Kuchibhotla | 9442bab | 2016-05-20 17:54:06 -0700 | [diff] [blame] | 687 | } |
| 688 | |
Craig Tiller | baa14a9 | 2017-11-03 09:09:36 -0700 | [diff] [blame] | 689 | static grpc_error* polling_island_global_init() { |
| 690 | grpc_error* error = GRPC_ERROR_NONE; |
Sree Kuchibhotla | 3131c26 | 2016-06-21 17:28:28 -0700 | [diff] [blame] | 691 | |
Sree Kuchibhotla | 3131c26 | 2016-06-21 17:28:28 -0700 | [diff] [blame] | 692 | error = grpc_wakeup_fd_init(&polling_island_wakeup_fd); |
| 693 | if (error == GRPC_ERROR_NONE) { |
| 694 | error = grpc_wakeup_fd_wakeup(&polling_island_wakeup_fd); |
| 695 | } |
| 696 | |
| 697 | return error; |
Sree Kuchibhotla | f448c34 | 2016-05-19 10:51:24 -0700 | [diff] [blame] | 698 | } |
| 699 | |
Sree Kuchibhotla | d627c10 | 2016-06-06 15:49:32 -0700 | [diff] [blame] | 700 | static void polling_island_global_shutdown() { |
Sree Kuchibhotla | 24b1062 | 2016-06-08 15:20:17 -0700 | [diff] [blame] | 701 | grpc_wakeup_fd_destroy(&polling_island_wakeup_fd); |
Sree Kuchibhotla | d627c10 | 2016-06-06 15:49:32 -0700 | [diff] [blame] | 702 | } |
| 703 | |
Sree Kuchibhotla | f448c34 | 2016-05-19 10:51:24 -0700 | [diff] [blame] | 704 | /******************************************************************************* |
Sree Kuchibhotla | 0bcbd79 | 2016-06-01 15:43:03 -0700 | [diff] [blame] | 705 | * Fd Definitions |
Sree Kuchibhotla | f448c34 | 2016-05-19 10:51:24 -0700 | [diff] [blame] | 706 | */ |
| 707 | |
Sree Kuchibhotla | 0bcbd79 | 2016-06-01 15:43:03 -0700 | [diff] [blame] | 708 | /* We need to keep a freelist not because of any concerns of malloc performance |
Sree Kuchibhotla | f448c34 | 2016-05-19 10:51:24 -0700 | [diff] [blame] | 709 | * but instead so that implementations with multiple threads in (for example) |
| 710 | * epoll_wait deal with the race between pollset removal and incoming poll |
| 711 | * notifications. |
| 712 | * |
| 713 | * The problem is that the poller ultimately holds a reference to this |
| 714 | * object, so it is very difficult to know when is safe to free it, at least |
| 715 | * without some expensive synchronization. |
| 716 | * |
| 717 | * If we keep the object freelisted, in the worst case losing this race just |
| 718 | * becomes a spurious read notification on a reused fd. |
| 719 | */ |
Sree Kuchibhotla | 0bcbd79 | 2016-06-01 15:43:03 -0700 | [diff] [blame] | 720 | |
| 721 | /* The alarm system needs to be able to wakeup 'some poller' sometimes |
| 722 | * (specifically when a new alarm needs to be triggered earlier than the next |
| 723 | * alarm 'epoch'). This wakeup_fd gives us something to alert on when such a |
| 724 | * case occurs. */ |
Sree Kuchibhotla | 9bc3d2d | 2016-06-06 10:27:56 -0700 | [diff] [blame] | 725 | |
Craig Tiller | 4782d92 | 2017-11-10 09:53:21 -0800 | [diff] [blame] | 726 | static grpc_fd* fd_freelist = nullptr; |
Sree Kuchibhotla | f448c34 | 2016-05-19 10:51:24 -0700 | [diff] [blame] | 727 | static gpr_mu fd_freelist_mu; |
| 728 | |
ncteisen | d39010e | 2017-06-08 17:08:07 -0700 | [diff] [blame] | 729 | #ifndef NDEBUG |
Sree Kuchibhotla | f448c34 | 2016-05-19 10:51:24 -0700 | [diff] [blame] | 730 | #define REF_BY(fd, n, reason) ref_by(fd, n, reason, __FILE__, __LINE__) |
| 731 | #define UNREF_BY(fd, n, reason) unref_by(fd, n, reason, __FILE__, __LINE__) |
Craig Tiller | baa14a9 | 2017-11-03 09:09:36 -0700 | [diff] [blame] | 732 | static void ref_by(grpc_fd* fd, int n, const char* reason, const char* file, |
Sree Kuchibhotla | f448c34 | 2016-05-19 10:51:24 -0700 | [diff] [blame] | 733 | int line) { |
ncteisen | 9ffb149 | 2017-11-10 14:00:49 -0800 | [diff] [blame] | 734 | if (grpc_trace_fd_refcount.enabled()) { |
ncteisen | 973863d | 2017-06-12 10:28:50 -0700 | [diff] [blame] | 735 | gpr_log(GPR_DEBUG, |
| 736 | "FD %d %p ref %d %" PRIdPTR " -> %" PRIdPTR " [%s; %s:%d]", |
ncteisen | 3ac64f8 | 2017-06-19 17:35:44 -0700 | [diff] [blame] | 737 | fd->fd, fd, n, gpr_atm_no_barrier_load(&fd->refst), |
ncteisen | d39010e | 2017-06-08 17:08:07 -0700 | [diff] [blame] | 738 | gpr_atm_no_barrier_load(&fd->refst) + n, reason, file, line); |
| 739 | } |
Sree Kuchibhotla | f448c34 | 2016-05-19 10:51:24 -0700 | [diff] [blame] | 740 | #else |
| 741 | #define REF_BY(fd, n, reason) ref_by(fd, n) |
| 742 | #define UNREF_BY(fd, n, reason) unref_by(fd, n) |
Craig Tiller | baa14a9 | 2017-11-03 09:09:36 -0700 | [diff] [blame] | 743 | static void ref_by(grpc_fd* fd, int n) { |
Sree Kuchibhotla | f448c34 | 2016-05-19 10:51:24 -0700 | [diff] [blame] | 744 | #endif |
| 745 | GPR_ASSERT(gpr_atm_no_barrier_fetch_add(&fd->refst, n) > 0); |
| 746 | } |
| 747 | |
ncteisen | d39010e | 2017-06-08 17:08:07 -0700 | [diff] [blame] | 748 | #ifndef NDEBUG |
Craig Tiller | baa14a9 | 2017-11-03 09:09:36 -0700 | [diff] [blame] | 749 | static void unref_by(grpc_fd* fd, int n, const char* reason, const char* file, |
Sree Kuchibhotla | f448c34 | 2016-05-19 10:51:24 -0700 | [diff] [blame] | 750 | int line) { |
ncteisen | 9ffb149 | 2017-11-10 14:00:49 -0800 | [diff] [blame] | 751 | if (grpc_trace_fd_refcount.enabled()) { |
ncteisen | 973863d | 2017-06-12 10:28:50 -0700 | [diff] [blame] | 752 | gpr_log(GPR_DEBUG, |
| 753 | "FD %d %p unref %d %" PRIdPTR " -> %" PRIdPTR " [%s; %s:%d]", |
ncteisen | 3ac64f8 | 2017-06-19 17:35:44 -0700 | [diff] [blame] | 754 | fd->fd, fd, n, gpr_atm_no_barrier_load(&fd->refst), |
ncteisen | d39010e | 2017-06-08 17:08:07 -0700 | [diff] [blame] | 755 | gpr_atm_no_barrier_load(&fd->refst) - n, reason, file, line); |
| 756 | } |
Sree Kuchibhotla | f448c34 | 2016-05-19 10:51:24 -0700 | [diff] [blame] | 757 | #else |
Craig Tiller | baa14a9 | 2017-11-03 09:09:36 -0700 | [diff] [blame] | 758 | static void unref_by(grpc_fd* fd, int n) { |
Sree Kuchibhotla | f448c34 | 2016-05-19 10:51:24 -0700 | [diff] [blame] | 759 | #endif |
Noah Eisen | 264879f | 2017-06-20 17:14:47 -0700 | [diff] [blame] | 760 | gpr_atm old = gpr_atm_full_fetch_add(&fd->refst, -n); |
Sree Kuchibhotla | f448c34 | 2016-05-19 10:51:24 -0700 | [diff] [blame] | 761 | if (old == n) { |
Sree Kuchibhotla | 0bcbd79 | 2016-06-01 15:43:03 -0700 | [diff] [blame] | 762 | /* Add the fd to the freelist */ |
| 763 | gpr_mu_lock(&fd_freelist_mu); |
| 764 | fd->freelist_next = fd_freelist; |
| 765 | fd_freelist = fd; |
| 766 | grpc_iomgr_unregister_object(&fd->iomgr_object); |
Sree Kuchibhotla | 82d7341 | 2017-02-09 18:27:45 -0800 | [diff] [blame] | 767 | |
yang-g | ed49fe5 | 2017-11-20 13:49:54 -0800 | [diff] [blame] | 768 | fd->read_closure->DestroyEvent(); |
| 769 | fd->write_closure->DestroyEvent(); |
Sree Kuchibhotla | 2f8ade0 | 2016-06-17 13:28:38 -0700 | [diff] [blame] | 770 | |
Sree Kuchibhotla | 0bcbd79 | 2016-06-01 15:43:03 -0700 | [diff] [blame] | 771 | gpr_mu_unlock(&fd_freelist_mu); |
Sree Kuchibhotla | f448c34 | 2016-05-19 10:51:24 -0700 | [diff] [blame] | 772 | } else { |
| 773 | GPR_ASSERT(old > n); |
| 774 | } |
| 775 | } |
| 776 | |
Sree Kuchibhotla | 0bcbd79 | 2016-06-01 15:43:03 -0700 | [diff] [blame] | 777 | /* Increment refcount by two to avoid changing the orphan bit */ |
ncteisen | d39010e | 2017-06-08 17:08:07 -0700 | [diff] [blame] | 778 | #ifndef NDEBUG |
Craig Tiller | baa14a9 | 2017-11-03 09:09:36 -0700 | [diff] [blame] | 779 | static void fd_ref(grpc_fd* fd, const char* reason, const char* file, |
Sree Kuchibhotla | f448c34 | 2016-05-19 10:51:24 -0700 | [diff] [blame] | 780 | int line) { |
| 781 | ref_by(fd, 2, reason, file, line); |
| 782 | } |
| 783 | |
Craig Tiller | baa14a9 | 2017-11-03 09:09:36 -0700 | [diff] [blame] | 784 | static void fd_unref(grpc_fd* fd, const char* reason, const char* file, |
Sree Kuchibhotla | f448c34 | 2016-05-19 10:51:24 -0700 | [diff] [blame] | 785 | int line) { |
| 786 | unref_by(fd, 2, reason, file, line); |
| 787 | } |
| 788 | #else |
Craig Tiller | baa14a9 | 2017-11-03 09:09:36 -0700 | [diff] [blame] | 789 | static void fd_ref(grpc_fd* fd) { ref_by(fd, 2); } |
| 790 | static void fd_unref(grpc_fd* fd) { unref_by(fd, 2); } |
Sree Kuchibhotla | f448c34 | 2016-05-19 10:51:24 -0700 | [diff] [blame] | 791 | #endif |
| 792 | |
Sree Kuchibhotla | 0bcbd79 | 2016-06-01 15:43:03 -0700 | [diff] [blame] | 793 | static void fd_global_init(void) { gpr_mu_init(&fd_freelist_mu); } |
| 794 | |
| 795 | static void fd_global_shutdown(void) { |
| 796 | gpr_mu_lock(&fd_freelist_mu); |
| 797 | gpr_mu_unlock(&fd_freelist_mu); |
Craig Tiller | 4782d92 | 2017-11-10 09:53:21 -0800 | [diff] [blame] | 798 | while (fd_freelist != nullptr) { |
Craig Tiller | baa14a9 | 2017-11-03 09:09:36 -0700 | [diff] [blame] | 799 | grpc_fd* fd = fd_freelist; |
Sree Kuchibhotla | 0bcbd79 | 2016-06-01 15:43:03 -0700 | [diff] [blame] | 800 | fd_freelist = fd_freelist->freelist_next; |
Sree Kuchibhotla | f6f33d7 | 2016-11-18 11:38:52 -0800 | [diff] [blame] | 801 | gpr_mu_destroy(&fd->po.mu); |
Sree Kuchibhotla | 0bcbd79 | 2016-06-01 15:43:03 -0700 | [diff] [blame] | 802 | gpr_free(fd); |
| 803 | } |
| 804 | gpr_mu_destroy(&fd_freelist_mu); |
| 805 | } |
| 806 | |
Craig Tiller | baa14a9 | 2017-11-03 09:09:36 -0700 | [diff] [blame] | 807 | static grpc_fd* fd_create(int fd, const char* name) { |
Craig Tiller | 4782d92 | 2017-11-10 09:53:21 -0800 | [diff] [blame] | 808 | grpc_fd* new_fd = nullptr; |
Sree Kuchibhotla | 0bcbd79 | 2016-06-01 15:43:03 -0700 | [diff] [blame] | 809 | |
| 810 | gpr_mu_lock(&fd_freelist_mu); |
Craig Tiller | 4782d92 | 2017-11-10 09:53:21 -0800 | [diff] [blame] | 811 | if (fd_freelist != nullptr) { |
Sree Kuchibhotla | 0bcbd79 | 2016-06-01 15:43:03 -0700 | [diff] [blame] | 812 | new_fd = fd_freelist; |
| 813 | fd_freelist = fd_freelist->freelist_next; |
| 814 | } |
| 815 | gpr_mu_unlock(&fd_freelist_mu); |
| 816 | |
Craig Tiller | 4782d92 | 2017-11-10 09:53:21 -0800 | [diff] [blame] | 817 | if (new_fd == nullptr) { |
Craig Tiller | baa14a9 | 2017-11-03 09:09:36 -0700 | [diff] [blame] | 818 | new_fd = (grpc_fd*)gpr_malloc(sizeof(grpc_fd)); |
Sree Kuchibhotla | f6f33d7 | 2016-11-18 11:38:52 -0800 | [diff] [blame] | 819 | gpr_mu_init(&new_fd->po.mu); |
yang-g | 26521b3 | 2017-11-17 17:15:37 -0800 | [diff] [blame] | 820 | new_fd->read_closure.Init(); |
| 821 | new_fd->write_closure.Init(); |
Sree Kuchibhotla | 0bcbd79 | 2016-06-01 15:43:03 -0700 | [diff] [blame] | 822 | } |
| 823 | |
Sree Kuchibhotla | f6f33d7 | 2016-11-18 11:38:52 -0800 | [diff] [blame] | 824 | /* Note: It is not really needed to get the new_fd->po.mu lock here. If this |
| 825 | * is a newly created fd (or an fd we got from the freelist), no one else |
| 826 | * would be holding a lock to it anyway. */ |
| 827 | gpr_mu_lock(&new_fd->po.mu); |
Craig Tiller | 4782d92 | 2017-11-10 09:53:21 -0800 | [diff] [blame] | 828 | new_fd->po.pi = nullptr; |
ncteisen | e9cd8a8 | 2017-06-29 06:03:52 -0400 | [diff] [blame] | 829 | #ifndef NDEBUG |
Sree Kuchibhotla | a0749a6 | 2016-11-18 20:22:09 -0800 | [diff] [blame] | 830 | new_fd->po.obj_type = POLL_OBJ_FD; |
| 831 | #endif |
Sree Kuchibhotla | 0bcbd79 | 2016-06-01 15:43:03 -0700 | [diff] [blame] | 832 | |
Sree Kuchibhotla | 0224dcc | 2016-06-22 18:04:00 -0700 | [diff] [blame] | 833 | gpr_atm_rel_store(&new_fd->refst, (gpr_atm)1); |
Sree Kuchibhotla | 5855c47 | 2016-06-08 12:56:56 -0700 | [diff] [blame] | 834 | new_fd->fd = fd; |
Sree Kuchibhotla | 5855c47 | 2016-06-08 12:56:56 -0700 | [diff] [blame] | 835 | new_fd->orphaned = false; |
yang-g | ed49fe5 | 2017-11-20 13:49:54 -0800 | [diff] [blame] | 836 | new_fd->read_closure->InitEvent(); |
| 837 | new_fd->write_closure->InitEvent(); |
Sree Kuchibhotla | 61fe094 | 2017-02-14 12:26:56 -0800 | [diff] [blame] | 838 | gpr_atm_no_barrier_store(&new_fd->read_notifier_pollset, (gpr_atm)NULL); |
Sree Kuchibhotla | 82d7341 | 2017-02-09 18:27:45 -0800 | [diff] [blame] | 839 | |
Craig Tiller | 4782d92 | 2017-11-10 09:53:21 -0800 | [diff] [blame] | 840 | new_fd->freelist_next = nullptr; |
| 841 | new_fd->on_done_closure = nullptr; |
Sree Kuchibhotla | 0bcbd79 | 2016-06-01 15:43:03 -0700 | [diff] [blame] | 842 | |
Sree Kuchibhotla | f6f33d7 | 2016-11-18 11:38:52 -0800 | [diff] [blame] | 843 | gpr_mu_unlock(&new_fd->po.mu); |
Sree Kuchibhotla | 0bcbd79 | 2016-06-01 15:43:03 -0700 | [diff] [blame] | 844 | |
Craig Tiller | baa14a9 | 2017-11-03 09:09:36 -0700 | [diff] [blame] | 845 | char* fd_name; |
Sree Kuchibhotla | 0bcbd79 | 2016-06-01 15:43:03 -0700 | [diff] [blame] | 846 | gpr_asprintf(&fd_name, "%s fd=%d", name, fd); |
| 847 | grpc_iomgr_register_object(&new_fd->iomgr_object, fd_name); |
Sree Kuchibhotla | 6a29545 | 2016-06-23 15:53:10 -0700 | [diff] [blame] | 848 | gpr_free(fd_name); |
Sree Kuchibhotla | 0bcbd79 | 2016-06-01 15:43:03 -0700 | [diff] [blame] | 849 | return new_fd; |
| 850 | } |
| 851 | |
Craig Tiller | baa14a9 | 2017-11-03 09:09:36 -0700 | [diff] [blame] | 852 | static int fd_wrapped_fd(grpc_fd* fd) { |
Sree Kuchibhotla | 0bcbd79 | 2016-06-01 15:43:03 -0700 | [diff] [blame] | 853 | int ret_fd = -1; |
Sree Kuchibhotla | f6f33d7 | 2016-11-18 11:38:52 -0800 | [diff] [blame] | 854 | gpr_mu_lock(&fd->po.mu); |
Sree Kuchibhotla | 79a6233 | 2016-06-04 14:01:03 -0700 | [diff] [blame] | 855 | if (!fd->orphaned) { |
Sree Kuchibhotla | 0bcbd79 | 2016-06-01 15:43:03 -0700 | [diff] [blame] | 856 | ret_fd = fd->fd; |
| 857 | } |
Sree Kuchibhotla | f6f33d7 | 2016-11-18 11:38:52 -0800 | [diff] [blame] | 858 | gpr_mu_unlock(&fd->po.mu); |
Sree Kuchibhotla | 0bcbd79 | 2016-06-01 15:43:03 -0700 | [diff] [blame] | 859 | |
| 860 | return ret_fd; |
| 861 | } |
| 862 | |
Yash Tibrewal | 8cf1470 | 2017-12-06 09:47:54 -0800 | [diff] [blame] | 863 | static void fd_orphan(grpc_fd* fd, grpc_closure* on_done, int* release_fd, |
Craig Tiller | baa14a9 | 2017-11-03 09:09:36 -0700 | [diff] [blame] | 864 | bool already_closed, const char* reason) { |
| 865 | grpc_error* error = GRPC_ERROR_NONE; |
Craig Tiller | 4782d92 | 2017-11-10 09:53:21 -0800 | [diff] [blame] | 866 | polling_island* unref_pi = nullptr; |
Sree Kuchibhotla | 20d0a16 | 2016-06-23 15:14:03 -0700 | [diff] [blame] | 867 | |
Sree Kuchibhotla | f6f33d7 | 2016-11-18 11:38:52 -0800 | [diff] [blame] | 868 | gpr_mu_lock(&fd->po.mu); |
Sree Kuchibhotla | 0bcbd79 | 2016-06-01 15:43:03 -0700 | [diff] [blame] | 869 | fd->on_done_closure = on_done; |
| 870 | |
Sree Kuchibhotla | 79a6233 | 2016-06-04 14:01:03 -0700 | [diff] [blame] | 871 | /* Remove the active status but keep referenced. We want this grpc_fd struct |
| 872 | to be alive (and not added to freelist) until the end of this function */ |
| 873 | REF_BY(fd, 1, reason); |
Sree Kuchibhotla | 0bcbd79 | 2016-06-01 15:43:03 -0700 | [diff] [blame] | 874 | |
| 875 | /* Remove the fd from the polling island: |
Sree Kuchibhotla | 2f8ade0 | 2016-06-17 13:28:38 -0700 | [diff] [blame] | 876 | - Get a lock on the latest polling island (i.e the last island in the |
Sree Kuchibhotla | f6f33d7 | 2016-11-18 11:38:52 -0800 | [diff] [blame] | 877 | linked list pointed by fd->po.pi). This is the island that |
Sree Kuchibhotla | 2f8ade0 | 2016-06-17 13:28:38 -0700 | [diff] [blame] | 878 | would actually contain the fd |
| 879 | - Remove the fd from the latest polling island |
| 880 | - Unlock the latest polling island |
Sree Kuchibhotla | f6f33d7 | 2016-11-18 11:38:52 -0800 | [diff] [blame] | 881 | - Set fd->po.pi to NULL (but remove the ref on the polling island |
Sree Kuchibhotla | 2f8ade0 | 2016-06-17 13:28:38 -0700 | [diff] [blame] | 882 | before doing this.) */ |
Craig Tiller | 4782d92 | 2017-11-10 09:53:21 -0800 | [diff] [blame] | 883 | if (fd->po.pi != nullptr) { |
Craig Tiller | baa14a9 | 2017-11-03 09:09:36 -0700 | [diff] [blame] | 884 | polling_island* pi_latest = polling_island_lock(fd->po.pi); |
Yuchen Zeng | d40a7ae | 2017-07-12 15:59:56 -0700 | [diff] [blame] | 885 | polling_island_remove_fd_locked(pi_latest, fd, already_closed, &error); |
Sree Kuchibhotla | 2f8ade0 | 2016-06-17 13:28:38 -0700 | [diff] [blame] | 886 | gpr_mu_unlock(&pi_latest->mu); |
Sree Kuchibhotla | 79a6233 | 2016-06-04 14:01:03 -0700 | [diff] [blame] | 887 | |
Sree Kuchibhotla | f6f33d7 | 2016-11-18 11:38:52 -0800 | [diff] [blame] | 888 | unref_pi = fd->po.pi; |
Craig Tiller | 4782d92 | 2017-11-10 09:53:21 -0800 | [diff] [blame] | 889 | fd->po.pi = nullptr; |
Sree Kuchibhotla | 88ee12f | 2016-06-03 19:26:48 -0700 | [diff] [blame] | 890 | } |
Sree Kuchibhotla | 0bcbd79 | 2016-06-01 15:43:03 -0700 | [diff] [blame] | 891 | |
Yuchen Zeng | 5b48dea | 2017-07-12 19:00:35 -0700 | [diff] [blame] | 892 | /* If release_fd is not NULL, we should be relinquishing control of the file |
| 893 | descriptor fd->fd (but we still own the grpc_fd structure). */ |
Craig Tiller | 4782d92 | 2017-11-10 09:53:21 -0800 | [diff] [blame] | 894 | if (release_fd != nullptr) { |
Yuchen Zeng | 5b48dea | 2017-07-12 19:00:35 -0700 | [diff] [blame] | 895 | *release_fd = fd->fd; |
| 896 | } else { |
| 897 | close(fd->fd); |
| 898 | } |
| 899 | |
| 900 | fd->orphaned = true; |
| 901 | |
Yash Tibrewal | 8cf1470 | 2017-12-06 09:47:54 -0800 | [diff] [blame] | 902 | GRPC_CLOSURE_SCHED(fd->on_done_closure, GRPC_ERROR_REF(error)); |
Sree Kuchibhotla | 0bcbd79 | 2016-06-01 15:43:03 -0700 | [diff] [blame] | 903 | |
Sree Kuchibhotla | f6f33d7 | 2016-11-18 11:38:52 -0800 | [diff] [blame] | 904 | gpr_mu_unlock(&fd->po.mu); |
Sree Kuchibhotla | 0bcbd79 | 2016-06-01 15:43:03 -0700 | [diff] [blame] | 905 | UNREF_BY(fd, 2, reason); /* Drop the reference */ |
Craig Tiller | 4782d92 | 2017-11-10 09:53:21 -0800 | [diff] [blame] | 906 | if (unref_pi != nullptr) { |
Craig Tiller | 0a06cd7 | 2016-07-14 13:21:24 -0700 | [diff] [blame] | 907 | /* Unref stale polling island here, outside the fd lock above. |
| 908 | The polling island owns a workqueue which owns an fd, and unreffing |
| 909 | inside the lock can cause an eventual lock loop that makes TSAN very |
| 910 | unhappy. */ |
Yash Tibrewal | 8cf1470 | 2017-12-06 09:47:54 -0800 | [diff] [blame] | 911 | PI_UNREF(unref_pi, "fd_orphan"); |
Craig Tiller | 1500761 | 2016-07-06 09:36:16 -0700 | [diff] [blame] | 912 | } |
Yuchen Zeng | 4ebace7 | 2017-06-05 17:24:06 -0700 | [diff] [blame] | 913 | if (error != GRPC_ERROR_NONE) { |
Craig Tiller | baa14a9 | 2017-11-03 09:09:36 -0700 | [diff] [blame] | 914 | const char* msg = grpc_error_string(error); |
Yuchen Zeng | 4ebace7 | 2017-06-05 17:24:06 -0700 | [diff] [blame] | 915 | gpr_log(GPR_DEBUG, "fd_orphan: %s", msg); |
| 916 | } |
Yuchen Zeng | a0399f2 | 2016-08-04 17:52:53 -0700 | [diff] [blame] | 917 | GRPC_ERROR_UNREF(error); |
Sree Kuchibhotla | 0bcbd79 | 2016-06-01 15:43:03 -0700 | [diff] [blame] | 918 | } |
| 919 | |
Yash Tibrewal | 8cf1470 | 2017-12-06 09:47:54 -0800 | [diff] [blame] | 920 | static grpc_pollset* fd_get_read_notifier_pollset(grpc_fd* fd) { |
Sree Kuchibhotla | fb7ced6 | 2017-02-15 18:31:57 -0800 | [diff] [blame] | 921 | gpr_atm notifier = gpr_atm_acq_load(&fd->read_notifier_pollset); |
Craig Tiller | baa14a9 | 2017-11-03 09:09:36 -0700 | [diff] [blame] | 922 | return (grpc_pollset*)notifier; |
Sree Kuchibhotla | 5855c47 | 2016-06-08 12:56:56 -0700 | [diff] [blame] | 923 | } |
| 924 | |
Craig Tiller | baa14a9 | 2017-11-03 09:09:36 -0700 | [diff] [blame] | 925 | static bool fd_is_shutdown(grpc_fd* fd) { |
Craig Tiller | fbf61bb | 2017-11-08 11:50:14 -0800 | [diff] [blame] | 926 | return fd->read_closure->IsShutdown(); |
Sree Kuchibhotla | 24b6eae | 2016-06-21 18:01:14 -0700 | [diff] [blame] | 927 | } |
| 928 | |
Sree Kuchibhotla | 0100b2f | 2016-06-21 17:38:13 -0700 | [diff] [blame] | 929 | /* Might be called multiple times */ |
Yash Tibrewal | 8cf1470 | 2017-12-06 09:47:54 -0800 | [diff] [blame] | 930 | static void fd_shutdown(grpc_fd* fd, grpc_error* why) { |
| 931 | if (fd->read_closure->SetShutdown(GRPC_ERROR_REF(why))) { |
Sree Kuchibhotla | 0100b2f | 2016-06-21 17:38:13 -0700 | [diff] [blame] | 932 | shutdown(fd->fd, SHUT_RDWR); |
Yash Tibrewal | 8cf1470 | 2017-12-06 09:47:54 -0800 | [diff] [blame] | 933 | fd->write_closure->SetShutdown(GRPC_ERROR_REF(why)); |
Sree Kuchibhotla | 0100b2f | 2016-06-21 17:38:13 -0700 | [diff] [blame] | 934 | } |
Craig Tiller | 376887d | 2017-04-06 08:27:03 -0700 | [diff] [blame] | 935 | GRPC_ERROR_UNREF(why); |
Sree Kuchibhotla | f448c34 | 2016-05-19 10:51:24 -0700 | [diff] [blame] | 936 | } |
| 937 | |
Yash Tibrewal | 8cf1470 | 2017-12-06 09:47:54 -0800 | [diff] [blame] | 938 | static void fd_notify_on_read(grpc_fd* fd, grpc_closure* closure) { |
| 939 | fd->read_closure->NotifyOn(closure); |
Sree Kuchibhotla | f448c34 | 2016-05-19 10:51:24 -0700 | [diff] [blame] | 940 | } |
| 941 | |
Yash Tibrewal | 8cf1470 | 2017-12-06 09:47:54 -0800 | [diff] [blame] | 942 | static void fd_notify_on_write(grpc_fd* fd, grpc_closure* closure) { |
| 943 | fd->write_closure->NotifyOn(closure); |
Sree Kuchibhotla | f448c34 | 2016-05-19 10:51:24 -0700 | [diff] [blame] | 944 | } |
| 945 | |
| 946 | /******************************************************************************* |
Sree Kuchibhotla | 0bcbd79 | 2016-06-01 15:43:03 -0700 | [diff] [blame] | 947 | * Pollset Definitions |
Sree Kuchibhotla | f448c34 | 2016-05-19 10:51:24 -0700 | [diff] [blame] | 948 | */ |
Sree Kuchibhotla | 8e4926c | 2016-06-08 20:33:19 -0700 | [diff] [blame] | 949 | GPR_TLS_DECL(g_current_thread_pollset); |
| 950 | GPR_TLS_DECL(g_current_thread_worker); |
Craig Tiller | 1919699 | 2016-06-27 18:45:56 -0700 | [diff] [blame] | 951 | static __thread bool g_initialized_sigmask; |
| 952 | static __thread sigset_t g_orig_sigmask; |
Sree Kuchibhotla | f448c34 | 2016-05-19 10:51:24 -0700 | [diff] [blame] | 953 | |
Sree Kuchibhotla | 0bcbd79 | 2016-06-01 15:43:03 -0700 | [diff] [blame] | 954 | static void sig_handler(int sig_num) { |
Sree Kuchibhotla | d627c10 | 2016-06-06 15:49:32 -0700 | [diff] [blame] | 955 | #ifdef GRPC_EPOLL_DEBUG |
Sree Kuchibhotla | 0bcbd79 | 2016-06-01 15:43:03 -0700 | [diff] [blame] | 956 | gpr_log(GPR_INFO, "Received signal %d", sig_num); |
Sree Kuchibhotla | 9bc3d2d | 2016-06-06 10:27:56 -0700 | [diff] [blame] | 957 | #endif |
Sree Kuchibhotla | 0bcbd79 | 2016-06-01 15:43:03 -0700 | [diff] [blame] | 958 | } |
Sree Kuchibhotla | f448c34 | 2016-05-19 10:51:24 -0700 | [diff] [blame] | 959 | |
Sree Kuchibhotla | c7be7c6 | 2016-06-09 17:08:50 -0700 | [diff] [blame] | 960 | static void poller_kick_init() { signal(grpc_wakeup_signal, sig_handler); } |
Sree Kuchibhotla | 5855c47 | 2016-06-08 12:56:56 -0700 | [diff] [blame] | 961 | |
Sree Kuchibhotla | 0bcbd79 | 2016-06-01 15:43:03 -0700 | [diff] [blame] | 962 | /* Global state management */ |
Craig Tiller | baa14a9 | 2017-11-03 09:09:36 -0700 | [diff] [blame] | 963 | static grpc_error* pollset_global_init(void) { |
Sree Kuchibhotla | 8e4926c | 2016-06-08 20:33:19 -0700 | [diff] [blame] | 964 | gpr_tls_init(&g_current_thread_pollset); |
| 965 | gpr_tls_init(&g_current_thread_worker); |
Sree Kuchibhotla | 5855c47 | 2016-06-08 12:56:56 -0700 | [diff] [blame] | 966 | poller_kick_init(); |
Craig Tiller | c357179 | 2017-05-02 12:33:38 -0700 | [diff] [blame] | 967 | return GRPC_ERROR_NONE; |
Sree Kuchibhotla | 0bcbd79 | 2016-06-01 15:43:03 -0700 | [diff] [blame] | 968 | } |
| 969 | |
| 970 | static void pollset_global_shutdown(void) { |
Sree Kuchibhotla | 8e4926c | 2016-06-08 20:33:19 -0700 | [diff] [blame] | 971 | gpr_tls_destroy(&g_current_thread_pollset); |
| 972 | gpr_tls_destroy(&g_current_thread_worker); |
Sree Kuchibhotla | 0bcbd79 | 2016-06-01 15:43:03 -0700 | [diff] [blame] | 973 | } |
| 974 | |
Craig Tiller | baa14a9 | 2017-11-03 09:09:36 -0700 | [diff] [blame] | 975 | static grpc_error* pollset_worker_kick(grpc_pollset_worker* worker) { |
| 976 | grpc_error* err = GRPC_ERROR_NONE; |
Sree Kuchibhotla | 3421724 | 2016-06-29 00:19:07 -0700 | [diff] [blame] | 977 | |
| 978 | /* Kick the worker only if it was not already kicked */ |
| 979 | if (gpr_atm_no_barrier_cas(&worker->is_kicked, (gpr_atm)0, (gpr_atm)1)) { |
| 980 | GRPC_POLLING_TRACE( |
| 981 | "pollset_worker_kick: Kicking worker: %p (thread id: %ld)", |
Craig Tiller | baa14a9 | 2017-11-03 09:09:36 -0700 | [diff] [blame] | 982 | (void*)worker, (long int)worker->pt_id); |
Sree Kuchibhotla | 3421724 | 2016-06-29 00:19:07 -0700 | [diff] [blame] | 983 | int err_num = pthread_kill(worker->pt_id, grpc_wakeup_signal); |
| 984 | if (err_num != 0) { |
| 985 | err = GRPC_OS_ERROR(err_num, "pthread_kill"); |
| 986 | } |
Sree Kuchibhotla | 3131c26 | 2016-06-21 17:28:28 -0700 | [diff] [blame] | 987 | } |
| 988 | return err; |
Sree Kuchibhotla | 5855c47 | 2016-06-08 12:56:56 -0700 | [diff] [blame] | 989 | } |
| 990 | |
Sree Kuchibhotla | 0bcbd79 | 2016-06-01 15:43:03 -0700 | [diff] [blame] | 991 | /* Return 1 if the pollset has active threads in pollset_work (pollset must |
| 992 | * be locked) */ |
Craig Tiller | baa14a9 | 2017-11-03 09:09:36 -0700 | [diff] [blame] | 993 | static int pollset_has_workers(grpc_pollset* p) { |
Sree Kuchibhotla | 0bcbd79 | 2016-06-01 15:43:03 -0700 | [diff] [blame] | 994 | return p->root_worker.next != &p->root_worker; |
| 995 | } |
Sree Kuchibhotla | f448c34 | 2016-05-19 10:51:24 -0700 | [diff] [blame] | 996 | |
Craig Tiller | baa14a9 | 2017-11-03 09:09:36 -0700 | [diff] [blame] | 997 | static void remove_worker(grpc_pollset* p, grpc_pollset_worker* worker) { |
Sree Kuchibhotla | f448c34 | 2016-05-19 10:51:24 -0700 | [diff] [blame] | 998 | worker->prev->next = worker->next; |
| 999 | worker->next->prev = worker->prev; |
| 1000 | } |
| 1001 | |
Craig Tiller | baa14a9 | 2017-11-03 09:09:36 -0700 | [diff] [blame] | 1002 | static grpc_pollset_worker* pop_front_worker(grpc_pollset* p) { |
Sree Kuchibhotla | f448c34 | 2016-05-19 10:51:24 -0700 | [diff] [blame] | 1003 | if (pollset_has_workers(p)) { |
Craig Tiller | baa14a9 | 2017-11-03 09:09:36 -0700 | [diff] [blame] | 1004 | grpc_pollset_worker* w = p->root_worker.next; |
Sree Kuchibhotla | f448c34 | 2016-05-19 10:51:24 -0700 | [diff] [blame] | 1005 | remove_worker(p, w); |
| 1006 | return w; |
| 1007 | } else { |
Craig Tiller | 4782d92 | 2017-11-10 09:53:21 -0800 | [diff] [blame] | 1008 | return nullptr; |
Sree Kuchibhotla | f448c34 | 2016-05-19 10:51:24 -0700 | [diff] [blame] | 1009 | } |
| 1010 | } |
| 1011 | |
Craig Tiller | baa14a9 | 2017-11-03 09:09:36 -0700 | [diff] [blame] | 1012 | static void push_back_worker(grpc_pollset* p, grpc_pollset_worker* worker) { |
Sree Kuchibhotla | f448c34 | 2016-05-19 10:51:24 -0700 | [diff] [blame] | 1013 | worker->next = &p->root_worker; |
| 1014 | worker->prev = worker->next->prev; |
| 1015 | worker->prev->next = worker->next->prev = worker; |
| 1016 | } |
| 1017 | |
Craig Tiller | baa14a9 | 2017-11-03 09:09:36 -0700 | [diff] [blame] | 1018 | static void push_front_worker(grpc_pollset* p, grpc_pollset_worker* worker) { |
Sree Kuchibhotla | f448c34 | 2016-05-19 10:51:24 -0700 | [diff] [blame] | 1019 | worker->prev = &p->root_worker; |
| 1020 | worker->next = worker->prev->next; |
| 1021 | worker->prev->next = worker->next->prev = worker; |
| 1022 | } |
| 1023 | |
Sree Kuchibhotla | 0bcbd79 | 2016-06-01 15:43:03 -0700 | [diff] [blame] | 1024 | /* p->mu must be held before calling this function */ |
Yash Tibrewal | 8cf1470 | 2017-12-06 09:47:54 -0800 | [diff] [blame] | 1025 | static grpc_error* pollset_kick(grpc_pollset* p, |
Craig Tiller | baa14a9 | 2017-11-03 09:09:36 -0700 | [diff] [blame] | 1026 | grpc_pollset_worker* specific_worker) { |
Sree Kuchibhotla | 0bcbd79 | 2016-06-01 15:43:03 -0700 | [diff] [blame] | 1027 | GPR_TIMER_BEGIN("pollset_kick", 0); |
Craig Tiller | baa14a9 | 2017-11-03 09:09:36 -0700 | [diff] [blame] | 1028 | grpc_error* error = GRPC_ERROR_NONE; |
Yash Tibrewal | 8cf1470 | 2017-12-06 09:47:54 -0800 | [diff] [blame] | 1029 | GRPC_STATS_INC_POLLSET_KICK(); |
Craig Tiller | baa14a9 | 2017-11-03 09:09:36 -0700 | [diff] [blame] | 1030 | const char* err_desc = "Kick Failure"; |
| 1031 | grpc_pollset_worker* worker = specific_worker; |
Craig Tiller | 4782d92 | 2017-11-10 09:53:21 -0800 | [diff] [blame] | 1032 | if (worker != nullptr) { |
Sree Kuchibhotla | 0bcbd79 | 2016-06-01 15:43:03 -0700 | [diff] [blame] | 1033 | if (worker == GRPC_POLLSET_KICK_BROADCAST) { |
Sree Kuchibhotla | 0bcbd79 | 2016-06-01 15:43:03 -0700 | [diff] [blame] | 1034 | if (pollset_has_workers(p)) { |
Sree Kuchibhotla | 79a6233 | 2016-06-04 14:01:03 -0700 | [diff] [blame] | 1035 | GPR_TIMER_BEGIN("pollset_kick.broadcast", 0); |
Sree Kuchibhotla | 0bcbd79 | 2016-06-01 15:43:03 -0700 | [diff] [blame] | 1036 | for (worker = p->root_worker.next; worker != &p->root_worker; |
| 1037 | worker = worker->next) { |
Sree Kuchibhotla | 8e4926c | 2016-06-08 20:33:19 -0700 | [diff] [blame] | 1038 | if (gpr_tls_get(&g_current_thread_worker) != (intptr_t)worker) { |
Sree Kuchibhotla | 20d0a16 | 2016-06-23 15:14:03 -0700 | [diff] [blame] | 1039 | append_error(&error, pollset_worker_kick(worker), err_desc); |
Sree Kuchibhotla | 8e4926c | 2016-06-08 20:33:19 -0700 | [diff] [blame] | 1040 | } |
Sree Kuchibhotla | f448c34 | 2016-05-19 10:51:24 -0700 | [diff] [blame] | 1041 | } |
Craig Tiller | a218a06 | 2016-06-26 09:58:37 -0700 | [diff] [blame] | 1042 | GPR_TIMER_END("pollset_kick.broadcast", 0); |
Sree Kuchibhotla | 0bcbd79 | 2016-06-01 15:43:03 -0700 | [diff] [blame] | 1043 | } else { |
| 1044 | p->kicked_without_pollers = true; |
Sree Kuchibhotla | f448c34 | 2016-05-19 10:51:24 -0700 | [diff] [blame] | 1045 | } |
Sree Kuchibhotla | 0bcbd79 | 2016-06-01 15:43:03 -0700 | [diff] [blame] | 1046 | } else { |
| 1047 | GPR_TIMER_MARK("kicked_specifically", 0); |
Sree Kuchibhotla | 8e4926c | 2016-06-08 20:33:19 -0700 | [diff] [blame] | 1048 | if (gpr_tls_get(&g_current_thread_worker) != (intptr_t)worker) { |
Sree Kuchibhotla | 20d0a16 | 2016-06-23 15:14:03 -0700 | [diff] [blame] | 1049 | append_error(&error, pollset_worker_kick(worker), err_desc); |
Sree Kuchibhotla | 8e4926c | 2016-06-08 20:33:19 -0700 | [diff] [blame] | 1050 | } |
Sree Kuchibhotla | 0bcbd79 | 2016-06-01 15:43:03 -0700 | [diff] [blame] | 1051 | } |
Sree Kuchibhotla | 8e4926c | 2016-06-08 20:33:19 -0700 | [diff] [blame] | 1052 | } else if (gpr_tls_get(&g_current_thread_pollset) != (intptr_t)p) { |
| 1053 | /* Since worker == NULL, it means that we can kick "any" worker on this |
| 1054 | pollset 'p'. If 'p' happens to be the same pollset this thread is |
| 1055 | currently polling (i.e in pollset_work() function), then there is no need |
| 1056 | to kick any other worker since the current thread can just absorb the |
| 1057 | kick. This is the reason why we enter this case only when |
| 1058 | g_current_thread_pollset is != p */ |
| 1059 | |
Sree Kuchibhotla | 0bcbd79 | 2016-06-01 15:43:03 -0700 | [diff] [blame] | 1060 | GPR_TIMER_MARK("kick_anonymous", 0); |
| 1061 | worker = pop_front_worker(p); |
Craig Tiller | 4782d92 | 2017-11-10 09:53:21 -0800 | [diff] [blame] | 1062 | if (worker != nullptr) { |
Sree Kuchibhotla | 0bcbd79 | 2016-06-01 15:43:03 -0700 | [diff] [blame] | 1063 | GPR_TIMER_MARK("finally_kick", 0); |
| 1064 | push_back_worker(p, worker); |
Sree Kuchibhotla | 20d0a16 | 2016-06-23 15:14:03 -0700 | [diff] [blame] | 1065 | append_error(&error, pollset_worker_kick(worker), err_desc); |
Sree Kuchibhotla | f448c34 | 2016-05-19 10:51:24 -0700 | [diff] [blame] | 1066 | } else { |
| 1067 | GPR_TIMER_MARK("kicked_no_pollers", 0); |
Sree Kuchibhotla | 0bcbd79 | 2016-06-01 15:43:03 -0700 | [diff] [blame] | 1068 | p->kicked_without_pollers = true; |
Sree Kuchibhotla | f448c34 | 2016-05-19 10:51:24 -0700 | [diff] [blame] | 1069 | } |
| 1070 | } |
| 1071 | |
Sree Kuchibhotla | 0bcbd79 | 2016-06-01 15:43:03 -0700 | [diff] [blame] | 1072 | GPR_TIMER_END("pollset_kick", 0); |
Sree Kuchibhotla | 3131c26 | 2016-06-21 17:28:28 -0700 | [diff] [blame] | 1073 | GRPC_LOG_IF_ERROR("pollset_kick", GRPC_ERROR_REF(error)); |
| 1074 | return error; |
Sree Kuchibhotla | f448c34 | 2016-05-19 10:51:24 -0700 | [diff] [blame] | 1075 | } |
| 1076 | |
Craig Tiller | baa14a9 | 2017-11-03 09:09:36 -0700 | [diff] [blame] | 1077 | static void pollset_init(grpc_pollset* pollset, gpr_mu** mu) { |
Sree Kuchibhotla | f6f33d7 | 2016-11-18 11:38:52 -0800 | [diff] [blame] | 1078 | gpr_mu_init(&pollset->po.mu); |
| 1079 | *mu = &pollset->po.mu; |
Craig Tiller | 4782d92 | 2017-11-10 09:53:21 -0800 | [diff] [blame] | 1080 | pollset->po.pi = nullptr; |
ncteisen | e9cd8a8 | 2017-06-29 06:03:52 -0400 | [diff] [blame] | 1081 | #ifndef NDEBUG |
Sree Kuchibhotla | a0749a6 | 2016-11-18 20:22:09 -0800 | [diff] [blame] | 1082 | pollset->po.obj_type = POLL_OBJ_POLLSET; |
| 1083 | #endif |
Sree Kuchibhotla | 0bcbd79 | 2016-06-01 15:43:03 -0700 | [diff] [blame] | 1084 | |
Sree Kuchibhotla | f448c34 | 2016-05-19 10:51:24 -0700 | [diff] [blame] | 1085 | pollset->root_worker.next = pollset->root_worker.prev = &pollset->root_worker; |
Sree Kuchibhotla | 0bcbd79 | 2016-06-01 15:43:03 -0700 | [diff] [blame] | 1086 | pollset->kicked_without_pollers = false; |
| 1087 | |
| 1088 | pollset->shutting_down = false; |
| 1089 | pollset->finish_shutdown_called = false; |
Craig Tiller | 4782d92 | 2017-11-10 09:53:21 -0800 | [diff] [blame] | 1090 | pollset->shutdown_done = nullptr; |
Sree Kuchibhotla | f448c34 | 2016-05-19 10:51:24 -0700 | [diff] [blame] | 1091 | } |
| 1092 | |
Yash Tibrewal | 8cf1470 | 2017-12-06 09:47:54 -0800 | [diff] [blame] | 1093 | static int poll_deadline_to_millis_timeout(grpc_millis millis) { |
Craig Tiller | 2039779 | 2017-07-18 11:35:27 -0700 | [diff] [blame] | 1094 | if (millis == GRPC_MILLIS_INF_FUTURE) return -1; |
Yash Tibrewal | 8cf1470 | 2017-12-06 09:47:54 -0800 | [diff] [blame] | 1095 | grpc_millis delta = millis - grpc_core::ExecCtx::Get()->Now(); |
Craig Tiller | 2039779 | 2017-07-18 11:35:27 -0700 | [diff] [blame] | 1096 | if (delta > INT_MAX) |
| 1097 | return INT_MAX; |
| 1098 | else if (delta < 0) |
Sree Kuchibhotla | f448c34 | 2016-05-19 10:51:24 -0700 | [diff] [blame] | 1099 | return 0; |
Craig Tiller | 2039779 | 2017-07-18 11:35:27 -0700 | [diff] [blame] | 1100 | else |
| 1101 | return (int)delta; |
Sree Kuchibhotla | f448c34 | 2016-05-19 10:51:24 -0700 | [diff] [blame] | 1102 | } |
| 1103 | |
Yash Tibrewal | 8cf1470 | 2017-12-06 09:47:54 -0800 | [diff] [blame] | 1104 | static void fd_become_readable(grpc_fd* fd, grpc_pollset* notifier) { |
| 1105 | fd->read_closure->SetReady(); |
Sree Kuchibhotla | 82d7341 | 2017-02-09 18:27:45 -0800 | [diff] [blame] | 1106 | |
Sree Kuchibhotla | 4db8c82 | 2017-02-12 17:07:31 -0800 | [diff] [blame] | 1107 | /* Note, it is possible that fd_become_readable might be called twice with |
Sree Kuchibhotla | 4c60d0d | 2017-02-12 17:09:08 -0800 | [diff] [blame] | 1108 | different 'notifier's when an fd becomes readable and it is in two epoll |
| 1109 | sets (This can happen briefly during polling island merges). In such cases |
| 1110 | it does not really matter which notifer is set as the read_notifier_pollset |
| 1111 | (They would both point to the same polling island anyway) */ |
Sree Kuchibhotla | fb7ced6 | 2017-02-15 18:31:57 -0800 | [diff] [blame] | 1112 | /* Use release store to match with acquire load in fd_get_read_notifier */ |
| 1113 | gpr_atm_rel_store(&fd->read_notifier_pollset, (gpr_atm)notifier); |
Sree Kuchibhotla | f448c34 | 2016-05-19 10:51:24 -0700 | [diff] [blame] | 1114 | } |
| 1115 | |
Yash Tibrewal | 8cf1470 | 2017-12-06 09:47:54 -0800 | [diff] [blame] | 1116 | static void fd_become_writable(grpc_fd* fd) { fd->write_closure->SetReady(); } |
Sree Kuchibhotla | f448c34 | 2016-05-19 10:51:24 -0700 | [diff] [blame] | 1117 | |
Yash Tibrewal | 8cf1470 | 2017-12-06 09:47:54 -0800 | [diff] [blame] | 1118 | static void pollset_release_polling_island(grpc_pollset* ps, |
Craig Tiller | baa14a9 | 2017-11-03 09:09:36 -0700 | [diff] [blame] | 1119 | const char* reason) { |
Craig Tiller | 4782d92 | 2017-11-10 09:53:21 -0800 | [diff] [blame] | 1120 | if (ps->po.pi != nullptr) { |
Yash Tibrewal | 8cf1470 | 2017-12-06 09:47:54 -0800 | [diff] [blame] | 1121 | PI_UNREF(ps->po.pi, reason); |
Sree Kuchibhotla | 8e4926c | 2016-06-08 20:33:19 -0700 | [diff] [blame] | 1122 | } |
Craig Tiller | 4782d92 | 2017-11-10 09:53:21 -0800 | [diff] [blame] | 1123 | ps->po.pi = nullptr; |
Sree Kuchibhotla | 8e4926c | 2016-06-08 20:33:19 -0700 | [diff] [blame] | 1124 | } |
| 1125 | |
Yash Tibrewal | 8cf1470 | 2017-12-06 09:47:54 -0800 | [diff] [blame] | 1126 | static void finish_shutdown_locked(grpc_pollset* pollset) { |
Sree Kuchibhotla | 8e4926c | 2016-06-08 20:33:19 -0700 | [diff] [blame] | 1127 | /* The pollset cannot have any workers if we are at this stage */ |
| 1128 | GPR_ASSERT(!pollset_has_workers(pollset)); |
| 1129 | |
| 1130 | pollset->finish_shutdown_called = true; |
Sree Kuchibhotla | 8e4926c | 2016-06-08 20:33:19 -0700 | [diff] [blame] | 1131 | |
Sree Kuchibhotla | f6f33d7 | 2016-11-18 11:38:52 -0800 | [diff] [blame] | 1132 | /* Release the ref and set pollset->po.pi to NULL */ |
Yash Tibrewal | 8cf1470 | 2017-12-06 09:47:54 -0800 | [diff] [blame] | 1133 | pollset_release_polling_island(pollset, "ps_shutdown"); |
| 1134 | GRPC_CLOSURE_SCHED(pollset->shutdown_done, GRPC_ERROR_NONE); |
Sree Kuchibhotla | 8e4926c | 2016-06-08 20:33:19 -0700 | [diff] [blame] | 1135 | } |
| 1136 | |
Sree Kuchibhotla | f6f33d7 | 2016-11-18 11:38:52 -0800 | [diff] [blame] | 1137 | /* pollset->po.mu lock must be held by the caller before calling this */ |
Yash Tibrewal | 8cf1470 | 2017-12-06 09:47:54 -0800 | [diff] [blame] | 1138 | static void pollset_shutdown(grpc_pollset* pollset, grpc_closure* closure) { |
Sree Kuchibhotla | 8e4926c | 2016-06-08 20:33:19 -0700 | [diff] [blame] | 1139 | GPR_TIMER_BEGIN("pollset_shutdown", 0); |
| 1140 | GPR_ASSERT(!pollset->shutting_down); |
| 1141 | pollset->shutting_down = true; |
| 1142 | pollset->shutdown_done = closure; |
Yash Tibrewal | 8cf1470 | 2017-12-06 09:47:54 -0800 | [diff] [blame] | 1143 | pollset_kick(pollset, GRPC_POLLSET_KICK_BROADCAST); |
Sree Kuchibhotla | 8e4926c | 2016-06-08 20:33:19 -0700 | [diff] [blame] | 1144 | |
| 1145 | /* If the pollset has any workers, we cannot call finish_shutdown_locked() |
| 1146 | because it would release the underlying polling island. In such a case, we |
| 1147 | let the last worker call finish_shutdown_locked() from pollset_work() */ |
| 1148 | if (!pollset_has_workers(pollset)) { |
| 1149 | GPR_ASSERT(!pollset->finish_shutdown_called); |
| 1150 | GPR_TIMER_MARK("pollset_shutdown.finish_shutdown_locked", 0); |
Yash Tibrewal | 8cf1470 | 2017-12-06 09:47:54 -0800 | [diff] [blame] | 1151 | finish_shutdown_locked(pollset); |
Sree Kuchibhotla | 8e4926c | 2016-06-08 20:33:19 -0700 | [diff] [blame] | 1152 | } |
| 1153 | GPR_TIMER_END("pollset_shutdown", 0); |
| 1154 | } |
| 1155 | |
| 1156 | /* pollset_shutdown is guaranteed to be called before pollset_destroy. So other |
| 1157 | * than destroying the mutexes, there is nothing special that needs to be done |
| 1158 | * here */ |
Yash Tibrewal | 8cf1470 | 2017-12-06 09:47:54 -0800 | [diff] [blame] | 1159 | static void pollset_destroy(grpc_pollset* pollset) { |
Sree Kuchibhotla | 8e4926c | 2016-06-08 20:33:19 -0700 | [diff] [blame] | 1160 | GPR_ASSERT(!pollset_has_workers(pollset)); |
Sree Kuchibhotla | f6f33d7 | 2016-11-18 11:38:52 -0800 | [diff] [blame] | 1161 | gpr_mu_destroy(&pollset->po.mu); |
Sree Kuchibhotla | 8e4926c | 2016-06-08 20:33:19 -0700 | [diff] [blame] | 1162 | } |
| 1163 | |
Craig Tiller | 84ea341 | 2016-09-08 14:57:56 -0700 | [diff] [blame] | 1164 | #define GRPC_EPOLL_MAX_EVENTS 100 |
Sree Kuchibhotla | 20d0a16 | 2016-06-23 15:14:03 -0700 | [diff] [blame] | 1165 | /* Note: sig_mask contains the signal mask to use *during* epoll_wait() */ |
Yash Tibrewal | 8cf1470 | 2017-12-06 09:47:54 -0800 | [diff] [blame] | 1166 | static void pollset_work_and_unlock(grpc_pollset* pollset, |
Craig Tiller | baa14a9 | 2017-11-03 09:09:36 -0700 | [diff] [blame] | 1167 | grpc_pollset_worker* worker, int timeout_ms, |
| 1168 | sigset_t* sig_mask, grpc_error** error) { |
Sree Kuchibhotla | 0bcbd79 | 2016-06-01 15:43:03 -0700 | [diff] [blame] | 1169 | struct epoll_event ep_ev[GRPC_EPOLL_MAX_EVENTS]; |
Sree Kuchibhotla | 88ee12f | 2016-06-03 19:26:48 -0700 | [diff] [blame] | 1170 | int epoll_fd = -1; |
Sree Kuchibhotla | 0bcbd79 | 2016-06-01 15:43:03 -0700 | [diff] [blame] | 1171 | int ep_rv; |
Craig Tiller | 4782d92 | 2017-11-10 09:53:21 -0800 | [diff] [blame] | 1172 | polling_island* pi = nullptr; |
Craig Tiller | baa14a9 | 2017-11-03 09:09:36 -0700 | [diff] [blame] | 1173 | char* err_msg; |
| 1174 | const char* err_desc = "pollset_work_and_unlock"; |
Sree Kuchibhotla | 0bcbd79 | 2016-06-01 15:43:03 -0700 | [diff] [blame] | 1175 | GPR_TIMER_BEGIN("pollset_work_and_unlock", 0); |
| 1176 | |
| 1177 | /* We need to get the epoll_fd to wait on. The epoll_fd is in inside the |
Sree Kuchibhotla | f6f33d7 | 2016-11-18 11:38:52 -0800 | [diff] [blame] | 1178 | latest polling island pointed by pollset->po.pi |
Sree Kuchibhotla | 229533b1 | 2016-06-21 20:42:52 -0700 | [diff] [blame] | 1179 | |
| 1180 | Since epoll_fd is immutable, we can read it without obtaining the polling |
| 1181 | island lock. There is however a possibility that the polling island (from |
| 1182 | which we got the epoll_fd) got merged with another island while we are |
| 1183 | in this function. This is still okay because in such a case, we will wakeup |
| 1184 | right-away from epoll_wait() and pick up the latest polling_island the next |
Sree Kuchibhotla | 20d0a16 | 2016-06-23 15:14:03 -0700 | [diff] [blame] | 1185 | this function (i.e pollset_work_and_unlock()) is called */ |
Sree Kuchibhotla | 0bcbd79 | 2016-06-01 15:43:03 -0700 | [diff] [blame] | 1186 | |
Craig Tiller | 4782d92 | 2017-11-10 09:53:21 -0800 | [diff] [blame] | 1187 | if (pollset->po.pi == nullptr) { |
Yash Tibrewal | 8cf1470 | 2017-12-06 09:47:54 -0800 | [diff] [blame] | 1188 | pollset->po.pi = polling_island_create(nullptr, error); |
Craig Tiller | 4782d92 | 2017-11-10 09:53:21 -0800 | [diff] [blame] | 1189 | if (pollset->po.pi == nullptr) { |
Sree Kuchibhotla | 20d0a16 | 2016-06-23 15:14:03 -0700 | [diff] [blame] | 1190 | GPR_TIMER_END("pollset_work_and_unlock", 0); |
| 1191 | return; /* Fatal error. We cannot continue */ |
| 1192 | } |
| 1193 | |
Sree Kuchibhotla | f6f33d7 | 2016-11-18 11:38:52 -0800 | [diff] [blame] | 1194 | PI_ADD_REF(pollset->po.pi, "ps"); |
Sree Kuchibhotla | 1e77668 | 2016-06-28 14:09:26 -0700 | [diff] [blame] | 1195 | GRPC_POLLING_TRACE("pollset_work: pollset: %p created new pi: %p", |
Craig Tiller | baa14a9 | 2017-11-03 09:09:36 -0700 | [diff] [blame] | 1196 | (void*)pollset, (void*)pollset->po.pi); |
Sree Kuchibhotla | 88ee12f | 2016-06-03 19:26:48 -0700 | [diff] [blame] | 1197 | } |
Sree Kuchibhotla | 0bcbd79 | 2016-06-01 15:43:03 -0700 | [diff] [blame] | 1198 | |
Sree Kuchibhotla | f6f33d7 | 2016-11-18 11:38:52 -0800 | [diff] [blame] | 1199 | pi = polling_island_maybe_get_latest(pollset->po.pi); |
Sree Kuchibhotla | 24b1062 | 2016-06-08 15:20:17 -0700 | [diff] [blame] | 1200 | epoll_fd = pi->epoll_fd; |
| 1201 | |
Sree Kuchibhotla | f6f33d7 | 2016-11-18 11:38:52 -0800 | [diff] [blame] | 1202 | /* Update the pollset->po.pi since the island being pointed by |
| 1203 | pollset->po.pi maybe older than the one pointed by pi) */ |
| 1204 | if (pollset->po.pi != pi) { |
Sree Kuchibhotla | 2f8ade0 | 2016-06-17 13:28:38 -0700 | [diff] [blame] | 1205 | /* Always do PI_ADD_REF before PI_UNREF because PI_UNREF may cause the |
| 1206 | polling island to be deleted */ |
| 1207 | PI_ADD_REF(pi, "ps"); |
Yash Tibrewal | 8cf1470 | 2017-12-06 09:47:54 -0800 | [diff] [blame] | 1208 | PI_UNREF(pollset->po.pi, "ps"); |
Sree Kuchibhotla | f6f33d7 | 2016-11-18 11:38:52 -0800 | [diff] [blame] | 1209 | pollset->po.pi = pi; |
Sree Kuchibhotla | 2f8ade0 | 2016-06-17 13:28:38 -0700 | [diff] [blame] | 1210 | } |
Sree Kuchibhotla | d627c10 | 2016-06-06 15:49:32 -0700 | [diff] [blame] | 1211 | |
Sree Kuchibhotla | 2f8ade0 | 2016-06-17 13:28:38 -0700 | [diff] [blame] | 1212 | /* Add an extra ref so that the island does not get destroyed (which means |
| 1213 | the epoll_fd won't be closed) while we are are doing an epoll_wait() on the |
| 1214 | epoll_fd */ |
| 1215 | PI_ADD_REF(pi, "ps_work"); |
Sree Kuchibhotla | f6f33d7 | 2016-11-18 11:38:52 -0800 | [diff] [blame] | 1216 | gpr_mu_unlock(&pollset->po.mu); |
Sree Kuchibhotla | 0bcbd79 | 2016-06-01 15:43:03 -0700 | [diff] [blame] | 1217 | |
Craig Tiller | 61f96c1 | 2017-05-12 13:36:39 -0700 | [diff] [blame] | 1218 | gpr_atm_no_barrier_fetch_add(&pi->poller_count, 1); |
| 1219 | g_current_thread_polling_island = pi; |
Craig Tiller | d8a3c04 | 2016-09-09 12:42:37 -0700 | [diff] [blame] | 1220 | |
Craig Tiller | 61f96c1 | 2017-05-12 13:36:39 -0700 | [diff] [blame] | 1221 | GRPC_SCHEDULING_START_BLOCKING_REGION; |
Yash Tibrewal | 8cf1470 | 2017-12-06 09:47:54 -0800 | [diff] [blame] | 1222 | GRPC_STATS_INC_SYSCALL_POLL(); |
Craig Tiller | 61f96c1 | 2017-05-12 13:36:39 -0700 | [diff] [blame] | 1223 | ep_rv = |
| 1224 | epoll_pwait(epoll_fd, ep_ev, GRPC_EPOLL_MAX_EVENTS, timeout_ms, sig_mask); |
Yash Tibrewal | 8cf1470 | 2017-12-06 09:47:54 -0800 | [diff] [blame] | 1225 | GRPC_SCHEDULING_END_BLOCKING_REGION; |
Craig Tiller | 61f96c1 | 2017-05-12 13:36:39 -0700 | [diff] [blame] | 1226 | if (ep_rv < 0) { |
| 1227 | if (errno != EINTR) { |
| 1228 | gpr_asprintf(&err_msg, |
| 1229 | "epoll_wait() epoll fd: %d failed with error: %d (%s)", |
| 1230 | epoll_fd, errno, strerror(errno)); |
| 1231 | append_error(error, GRPC_OS_ERROR(errno, err_msg), err_desc); |
| 1232 | } else { |
| 1233 | /* We were interrupted. Save an interation by doing a zero timeout |
| 1234 | epoll_wait to see if there are any other events of interest */ |
| 1235 | GRPC_POLLING_TRACE("pollset_work: pollset: %p, worker: %p received kick", |
Craig Tiller | baa14a9 | 2017-11-03 09:09:36 -0700 | [diff] [blame] | 1236 | (void*)pollset, (void*)worker); |
Craig Tiller | 61f96c1 | 2017-05-12 13:36:39 -0700 | [diff] [blame] | 1237 | ep_rv = epoll_wait(epoll_fd, ep_ev, GRPC_EPOLL_MAX_EVENTS, 0); |
Sree Kuchibhotla | e5012ba | 2016-06-06 16:01:45 -0700 | [diff] [blame] | 1238 | } |
Craig Tiller | 61f96c1 | 2017-05-12 13:36:39 -0700 | [diff] [blame] | 1239 | } |
Sree Kuchibhotla | 79a6233 | 2016-06-04 14:01:03 -0700 | [diff] [blame] | 1240 | |
Sree Kuchibhotla | ad2c477 | 2016-06-13 19:06:54 -0700 | [diff] [blame] | 1241 | #ifdef GRPC_TSAN |
Craig Tiller | 61f96c1 | 2017-05-12 13:36:39 -0700 | [diff] [blame] | 1242 | /* See the definition of g_poll_sync for more details */ |
| 1243 | gpr_atm_acq_load(&g_epoll_sync); |
Sree Kuchibhotla | ad2c477 | 2016-06-13 19:06:54 -0700 | [diff] [blame] | 1244 | #endif /* defined(GRPC_TSAN) */ |
Sree Kuchibhotla | 41622a8 | 2016-06-13 16:43:14 -0700 | [diff] [blame] | 1245 | |
Craig Tiller | 61f96c1 | 2017-05-12 13:36:39 -0700 | [diff] [blame] | 1246 | for (int i = 0; i < ep_rv; ++i) { |
Craig Tiller | baa14a9 | 2017-11-03 09:09:36 -0700 | [diff] [blame] | 1247 | void* data_ptr = ep_ev[i].data.ptr; |
Craig Tiller | 61f96c1 | 2017-05-12 13:36:39 -0700 | [diff] [blame] | 1248 | if (data_ptr == &polling_island_wakeup_fd) { |
| 1249 | GRPC_POLLING_TRACE( |
| 1250 | "pollset_work: pollset: %p, worker: %p polling island (epoll_fd: " |
| 1251 | "%d) got merged", |
Craig Tiller | baa14a9 | 2017-11-03 09:09:36 -0700 | [diff] [blame] | 1252 | (void*)pollset, (void*)worker, epoll_fd); |
Craig Tiller | 61f96c1 | 2017-05-12 13:36:39 -0700 | [diff] [blame] | 1253 | /* This means that our polling island is merged with a different |
| 1254 | island. We do not have to do anything here since the subsequent call |
| 1255 | to the function pollset_work_and_unlock() will pick up the correct |
| 1256 | epoll_fd */ |
| 1257 | } else { |
Craig Tiller | baa14a9 | 2017-11-03 09:09:36 -0700 | [diff] [blame] | 1258 | grpc_fd* fd = (grpc_fd*)data_ptr; |
Craig Tiller | 61f96c1 | 2017-05-12 13:36:39 -0700 | [diff] [blame] | 1259 | int cancel = ep_ev[i].events & (EPOLLERR | EPOLLHUP); |
| 1260 | int read_ev = ep_ev[i].events & (EPOLLIN | EPOLLPRI); |
| 1261 | int write_ev = ep_ev[i].events & EPOLLOUT; |
| 1262 | if (read_ev || cancel) { |
Yash Tibrewal | 8cf1470 | 2017-12-06 09:47:54 -0800 | [diff] [blame] | 1263 | fd_become_readable(fd, pollset); |
Craig Tiller | 61f96c1 | 2017-05-12 13:36:39 -0700 | [diff] [blame] | 1264 | } |
| 1265 | if (write_ev || cancel) { |
Yash Tibrewal | 8cf1470 | 2017-12-06 09:47:54 -0800 | [diff] [blame] | 1266 | fd_become_writable(fd); |
Sree Kuchibhotla | 0bcbd79 | 2016-06-01 15:43:03 -0700 | [diff] [blame] | 1267 | } |
Sree Kuchibhotla | e5012ba | 2016-06-06 16:01:45 -0700 | [diff] [blame] | 1268 | } |
Craig Tiller | d8a3c04 | 2016-09-09 12:42:37 -0700 | [diff] [blame] | 1269 | } |
Sree Kuchibhotla | 24b1062 | 2016-06-08 15:20:17 -0700 | [diff] [blame] | 1270 | |
Craig Tiller | 4782d92 | 2017-11-10 09:53:21 -0800 | [diff] [blame] | 1271 | g_current_thread_polling_island = nullptr; |
Craig Tiller | 61f96c1 | 2017-05-12 13:36:39 -0700 | [diff] [blame] | 1272 | gpr_atm_no_barrier_fetch_add(&pi->poller_count, -1); |
| 1273 | |
Craig Tiller | 4782d92 | 2017-11-10 09:53:21 -0800 | [diff] [blame] | 1274 | GPR_ASSERT(pi != nullptr); |
Sree Kuchibhotla | 24b1062 | 2016-06-08 15:20:17 -0700 | [diff] [blame] | 1275 | |
Sree Kuchibhotla | 2f8ade0 | 2016-06-17 13:28:38 -0700 | [diff] [blame] | 1276 | /* Before leaving, release the extra ref we added to the polling island. It |
Sree Kuchibhotla | f6f33d7 | 2016-11-18 11:38:52 -0800 | [diff] [blame] | 1277 | is important to use "pi" here (i.e our old copy of pollset->po.pi |
Sree Kuchibhotla | 2f8ade0 | 2016-06-17 13:28:38 -0700 | [diff] [blame] | 1278 | that we got before releasing the polling island lock). This is because |
Sree Kuchibhotla | f6f33d7 | 2016-11-18 11:38:52 -0800 | [diff] [blame] | 1279 | pollset->po.pi pointer might get udpated in other parts of the |
Sree Kuchibhotla | 2f8ade0 | 2016-06-17 13:28:38 -0700 | [diff] [blame] | 1280 | code when there is an island merge while we are doing epoll_wait() above */ |
Yash Tibrewal | 8cf1470 | 2017-12-06 09:47:54 -0800 | [diff] [blame] | 1281 | PI_UNREF(pi, "ps_work"); |
Sree Kuchibhotla | 24b1062 | 2016-06-08 15:20:17 -0700 | [diff] [blame] | 1282 | |
Sree Kuchibhotla | 0bcbd79 | 2016-06-01 15:43:03 -0700 | [diff] [blame] | 1283 | GPR_TIMER_END("pollset_work_and_unlock", 0); |
| 1284 | } |
| 1285 | |
Sree Kuchibhotla | f6f33d7 | 2016-11-18 11:38:52 -0800 | [diff] [blame] | 1286 | /* pollset->po.mu lock must be held by the caller before calling this. |
| 1287 | The function pollset_work() may temporarily release the lock (pollset->po.mu) |
Sree Kuchibhotla | 0bcbd79 | 2016-06-01 15:43:03 -0700 | [diff] [blame] | 1288 | during the course of its execution but it will always re-acquire the lock and |
| 1289 | ensure that it is held by the time the function returns */ |
Yash Tibrewal | 8cf1470 | 2017-12-06 09:47:54 -0800 | [diff] [blame] | 1290 | static grpc_error* pollset_work(grpc_pollset* pollset, |
Craig Tiller | baa14a9 | 2017-11-03 09:09:36 -0700 | [diff] [blame] | 1291 | grpc_pollset_worker** worker_hdl, |
Craig Tiller | 2039779 | 2017-07-18 11:35:27 -0700 | [diff] [blame] | 1292 | grpc_millis deadline) { |
Sree Kuchibhotla | 0bcbd79 | 2016-06-01 15:43:03 -0700 | [diff] [blame] | 1293 | GPR_TIMER_BEGIN("pollset_work", 0); |
Craig Tiller | baa14a9 | 2017-11-03 09:09:36 -0700 | [diff] [blame] | 1294 | grpc_error* error = GRPC_ERROR_NONE; |
Yash Tibrewal | 8cf1470 | 2017-12-06 09:47:54 -0800 | [diff] [blame] | 1295 | int timeout_ms = poll_deadline_to_millis_timeout(deadline); |
Sree Kuchibhotla | 0bcbd79 | 2016-06-01 15:43:03 -0700 | [diff] [blame] | 1296 | |
| 1297 | sigset_t new_mask; |
Sree Kuchibhotla | 0bcbd79 | 2016-06-01 15:43:03 -0700 | [diff] [blame] | 1298 | |
| 1299 | grpc_pollset_worker worker; |
Craig Tiller | 4782d92 | 2017-11-10 09:53:21 -0800 | [diff] [blame] | 1300 | worker.next = worker.prev = nullptr; |
Sree Kuchibhotla | 0bcbd79 | 2016-06-01 15:43:03 -0700 | [diff] [blame] | 1301 | worker.pt_id = pthread_self(); |
Sree Kuchibhotla | 3421724 | 2016-06-29 00:19:07 -0700 | [diff] [blame] | 1302 | gpr_atm_no_barrier_store(&worker.is_kicked, (gpr_atm)0); |
Sree Kuchibhotla | 0bcbd79 | 2016-06-01 15:43:03 -0700 | [diff] [blame] | 1303 | |
Craig Tiller | 557c88c | 2017-04-05 17:20:18 -0700 | [diff] [blame] | 1304 | if (worker_hdl) *worker_hdl = &worker; |
Sree Kuchibhotla | 20d0a16 | 2016-06-23 15:14:03 -0700 | [diff] [blame] | 1305 | |
Sree Kuchibhotla | 8e4926c | 2016-06-08 20:33:19 -0700 | [diff] [blame] | 1306 | gpr_tls_set(&g_current_thread_pollset, (intptr_t)pollset); |
| 1307 | gpr_tls_set(&g_current_thread_worker, (intptr_t)&worker); |
Sree Kuchibhotla | 0bcbd79 | 2016-06-01 15:43:03 -0700 | [diff] [blame] | 1308 | |
| 1309 | if (pollset->kicked_without_pollers) { |
| 1310 | /* If the pollset was kicked without pollers, pretend that the current |
| 1311 | worker got the kick and skip polling. A kick indicates that there is some |
| 1312 | work that needs attention like an event on the completion queue or an |
| 1313 | alarm */ |
| 1314 | GPR_TIMER_MARK("pollset_work.kicked_without_pollers", 0); |
| 1315 | pollset->kicked_without_pollers = 0; |
| 1316 | } else if (!pollset->shutting_down) { |
Sree Kuchibhotla | 20d0a16 | 2016-06-23 15:14:03 -0700 | [diff] [blame] | 1317 | /* We use the posix-signal with number 'grpc_wakeup_signal' for waking up |
Sree Kuchibhotla | 3421724 | 2016-06-29 00:19:07 -0700 | [diff] [blame] | 1318 | (i.e 'kicking') a worker in the pollset. A 'kick' is a way to inform the |
| 1319 | worker that there is some pending work that needs immediate attention |
| 1320 | (like an event on the completion queue, or a polling island merge that |
| 1321 | results in a new epoll-fd to wait on) and that the worker should not |
| 1322 | spend time waiting in epoll_pwait(). |
Sree Kuchibhotla | 20d0a16 | 2016-06-23 15:14:03 -0700 | [diff] [blame] | 1323 | |
Sree Kuchibhotla | 3421724 | 2016-06-29 00:19:07 -0700 | [diff] [blame] | 1324 | A worker can be kicked anytime from the point it is added to the pollset |
| 1325 | via push_front_worker() (or push_back_worker()) to the point it is |
| 1326 | removed via remove_worker(). |
| 1327 | If the worker is kicked before/during it calls epoll_pwait(), it should |
| 1328 | immediately exit from epoll_wait(). If the worker is kicked after it |
| 1329 | returns from epoll_wait(), then nothing really needs to be done. |
Sree Kuchibhotla | 20d0a16 | 2016-06-23 15:14:03 -0700 | [diff] [blame] | 1330 | |
Sree Kuchibhotla | 3421724 | 2016-06-29 00:19:07 -0700 | [diff] [blame] | 1331 | To accomplish this, we mask 'grpc_wakeup_signal' on this thread at all |
Sree Kuchibhotla | 20d0a16 | 2016-06-23 15:14:03 -0700 | [diff] [blame] | 1332 | times *except* when it is in epoll_pwait(). This way, the worker never |
| 1333 | misses acting on a kick */ |
| 1334 | |
Craig Tiller | 1919699 | 2016-06-27 18:45:56 -0700 | [diff] [blame] | 1335 | if (!g_initialized_sigmask) { |
| 1336 | sigemptyset(&new_mask); |
| 1337 | sigaddset(&new_mask, grpc_wakeup_signal); |
| 1338 | pthread_sigmask(SIG_BLOCK, &new_mask, &g_orig_sigmask); |
| 1339 | sigdelset(&g_orig_sigmask, grpc_wakeup_signal); |
| 1340 | g_initialized_sigmask = true; |
| 1341 | /* new_mask: The new thread mask which blocks 'grpc_wakeup_signal'. |
| 1342 | This is the mask used at all times *except during |
| 1343 | epoll_wait()*" |
| 1344 | g_orig_sigmask: The thread mask which allows 'grpc_wakeup_signal' and |
Craig Tiller | 510ff69 | 2016-06-27 20:31:49 -0700 | [diff] [blame] | 1345 | this is the mask to use *during epoll_wait()* |
Sree Kuchibhotla | 0bcbd79 | 2016-06-01 15:43:03 -0700 | [diff] [blame] | 1346 | |
Craig Tiller | 1919699 | 2016-06-27 18:45:56 -0700 | [diff] [blame] | 1347 | The new_mask is set on the worker before it is added to the pollset |
Craig Tiller | 510ff69 | 2016-06-27 20:31:49 -0700 | [diff] [blame] | 1348 | (i.e before it can be kicked) */ |
Craig Tiller | 1919699 | 2016-06-27 18:45:56 -0700 | [diff] [blame] | 1349 | } |
Sree Kuchibhotla | 0bcbd79 | 2016-06-01 15:43:03 -0700 | [diff] [blame] | 1350 | |
Sree Kuchibhotla | 20d0a16 | 2016-06-23 15:14:03 -0700 | [diff] [blame] | 1351 | push_front_worker(pollset, &worker); /* Add worker to pollset */ |
| 1352 | |
Yash Tibrewal | 8cf1470 | 2017-12-06 09:47:54 -0800 | [diff] [blame] | 1353 | pollset_work_and_unlock(pollset, &worker, timeout_ms, &g_orig_sigmask, |
| 1354 | &error); |
| 1355 | grpc_core::ExecCtx::Get()->Flush(); |
Sree Kuchibhotla | 0bcbd79 | 2016-06-01 15:43:03 -0700 | [diff] [blame] | 1356 | |
Sree Kuchibhotla | f6f33d7 | 2016-11-18 11:38:52 -0800 | [diff] [blame] | 1357 | gpr_mu_lock(&pollset->po.mu); |
Sree Kuchibhotla | 3421724 | 2016-06-29 00:19:07 -0700 | [diff] [blame] | 1358 | |
| 1359 | /* Note: There is no need to reset worker.is_kicked to 0 since we are no |
| 1360 | longer going to use this worker */ |
Sree Kuchibhotla | 0bcbd79 | 2016-06-01 15:43:03 -0700 | [diff] [blame] | 1361 | remove_worker(pollset, &worker); |
| 1362 | } |
| 1363 | |
| 1364 | /* If we are the last worker on the pollset (i.e pollset_has_workers() is |
| 1365 | false at this point) and the pollset is shutting down, we may have to |
| 1366 | finish the shutdown process by calling finish_shutdown_locked(). |
| 1367 | See pollset_shutdown() for more details. |
| 1368 | |
| 1369 | Note: Continuing to access pollset here is safe; it is the caller's |
| 1370 | responsibility to not destroy a pollset when it has outstanding calls to |
| 1371 | pollset_work() */ |
| 1372 | if (pollset->shutting_down && !pollset_has_workers(pollset) && |
| 1373 | !pollset->finish_shutdown_called) { |
| 1374 | GPR_TIMER_MARK("pollset_work.finish_shutdown_locked", 0); |
Yash Tibrewal | 8cf1470 | 2017-12-06 09:47:54 -0800 | [diff] [blame] | 1375 | finish_shutdown_locked(pollset); |
Sree Kuchibhotla | 0bcbd79 | 2016-06-01 15:43:03 -0700 | [diff] [blame] | 1376 | |
Sree Kuchibhotla | f6f33d7 | 2016-11-18 11:38:52 -0800 | [diff] [blame] | 1377 | gpr_mu_unlock(&pollset->po.mu); |
Yash Tibrewal | 8cf1470 | 2017-12-06 09:47:54 -0800 | [diff] [blame] | 1378 | grpc_core::ExecCtx::Get()->Flush(); |
Sree Kuchibhotla | f6f33d7 | 2016-11-18 11:38:52 -0800 | [diff] [blame] | 1379 | gpr_mu_lock(&pollset->po.mu); |
Sree Kuchibhotla | 0bcbd79 | 2016-06-01 15:43:03 -0700 | [diff] [blame] | 1380 | } |
| 1381 | |
Craig Tiller | 4782d92 | 2017-11-10 09:53:21 -0800 | [diff] [blame] | 1382 | if (worker_hdl) *worker_hdl = nullptr; |
Sree Kuchibhotla | 20d0a16 | 2016-06-23 15:14:03 -0700 | [diff] [blame] | 1383 | |
Sree Kuchibhotla | 8e4926c | 2016-06-08 20:33:19 -0700 | [diff] [blame] | 1384 | gpr_tls_set(&g_current_thread_pollset, (intptr_t)0); |
| 1385 | gpr_tls_set(&g_current_thread_worker, (intptr_t)0); |
Sree Kuchibhotla | 20d0a16 | 2016-06-23 15:14:03 -0700 | [diff] [blame] | 1386 | |
Sree Kuchibhotla | 0bcbd79 | 2016-06-01 15:43:03 -0700 | [diff] [blame] | 1387 | GPR_TIMER_END("pollset_work", 0); |
Sree Kuchibhotla | 20d0a16 | 2016-06-23 15:14:03 -0700 | [diff] [blame] | 1388 | |
Sree Kuchibhotla | 3131c26 | 2016-06-21 17:28:28 -0700 | [diff] [blame] | 1389 | GRPC_LOG_IF_ERROR("pollset_work", GRPC_ERROR_REF(error)); |
| 1390 | return error; |
Sree Kuchibhotla | 0bcbd79 | 2016-06-01 15:43:03 -0700 | [diff] [blame] | 1391 | } |
| 1392 | |
Yash Tibrewal | 8cf1470 | 2017-12-06 09:47:54 -0800 | [diff] [blame] | 1393 | static void add_poll_object(poll_obj* bag, poll_obj_type bag_type, |
| 1394 | poll_obj* item, poll_obj_type item_type) { |
Sree Kuchibhotla | f6f33d7 | 2016-11-18 11:38:52 -0800 | [diff] [blame] | 1395 | GPR_TIMER_BEGIN("add_poll_object", 0); |
| 1396 | |
ncteisen | e9cd8a8 | 2017-06-29 06:03:52 -0400 | [diff] [blame] | 1397 | #ifndef NDEBUG |
Sree Kuchibhotla | a0749a6 | 2016-11-18 20:22:09 -0800 | [diff] [blame] | 1398 | GPR_ASSERT(item->obj_type == item_type); |
| 1399 | GPR_ASSERT(bag->obj_type == bag_type); |
| 1400 | #endif |
Craig Tiller | 57726ca | 2016-09-12 11:59:45 -0700 | [diff] [blame] | 1401 | |
Craig Tiller | baa14a9 | 2017-11-03 09:09:36 -0700 | [diff] [blame] | 1402 | grpc_error* error = GRPC_ERROR_NONE; |
Craig Tiller | 4782d92 | 2017-11-10 09:53:21 -0800 | [diff] [blame] | 1403 | polling_island* pi_new = nullptr; |
Sree Kuchibhotla | f448c34 | 2016-05-19 10:51:24 -0700 | [diff] [blame] | 1404 | |
Sree Kuchibhotla | f6f33d7 | 2016-11-18 11:38:52 -0800 | [diff] [blame] | 1405 | gpr_mu_lock(&bag->mu); |
| 1406 | gpr_mu_lock(&item->mu); |
| 1407 | |
Craig Tiller | 7212c23 | 2016-07-06 13:11:09 -0700 | [diff] [blame] | 1408 | retry: |
Sree Kuchibhotla | f6f33d7 | 2016-11-18 11:38:52 -0800 | [diff] [blame] | 1409 | /* |
| 1410 | * 1) If item->pi and bag->pi are both non-NULL and equal, do nothing |
| 1411 | * 2) If item->pi and bag->pi are both NULL, create a new polling island (with |
| 1412 | * a refcount of 2) and point item->pi and bag->pi to the new island |
| 1413 | * 3) If exactly one of item->pi or bag->pi is NULL, update it to point to |
| 1414 | * the other's non-NULL pi |
| 1415 | * 4) Finally if item->pi and bag-pi are non-NULL and not-equal, merge the |
| 1416 | * polling islands and update item->pi and bag->pi to point to the new |
| 1417 | * island |
Sree Kuchibhotla | 0bcbd79 | 2016-06-01 15:43:03 -0700 | [diff] [blame] | 1418 | */ |
Craig Tiller | 8e8027b | 2016-07-07 10:42:09 -0700 | [diff] [blame] | 1419 | |
Sree Kuchibhotla | f6f33d7 | 2016-11-18 11:38:52 -0800 | [diff] [blame] | 1420 | /* Early out if we are trying to add an 'fd' to a 'bag' but the fd is already |
| 1421 | * orphaned */ |
| 1422 | if (item_type == POLL_OBJ_FD && (FD_FROM_PO(item))->orphaned) { |
| 1423 | gpr_mu_unlock(&item->mu); |
| 1424 | gpr_mu_unlock(&bag->mu); |
Craig Tiller | 42ac6db | 2016-07-06 17:13:56 -0700 | [diff] [blame] | 1425 | return; |
| 1426 | } |
| 1427 | |
Sree Kuchibhotla | f6f33d7 | 2016-11-18 11:38:52 -0800 | [diff] [blame] | 1428 | if (item->pi == bag->pi) { |
| 1429 | pi_new = item->pi; |
Craig Tiller | 4782d92 | 2017-11-10 09:53:21 -0800 | [diff] [blame] | 1430 | if (pi_new == nullptr) { |
Sree Kuchibhotla | f6f33d7 | 2016-11-18 11:38:52 -0800 | [diff] [blame] | 1431 | /* GPR_ASSERT(item->pi == bag->pi == NULL) */ |
Sree Kuchibhotla | a129adf | 2016-10-26 16:44:44 -0700 | [diff] [blame] | 1432 | |
Sree Kuchibhotla | f6f33d7 | 2016-11-18 11:38:52 -0800 | [diff] [blame] | 1433 | /* If we are adding an fd to a bag (i.e pollset or pollset_set), then |
| 1434 | * we need to do some extra work to make TSAN happy */ |
| 1435 | if (item_type == POLL_OBJ_FD) { |
| 1436 | /* Unlock before creating a new polling island: the polling island will |
| 1437 | create a workqueue which creates a file descriptor, and holding an fd |
| 1438 | lock here can eventually cause a loop to appear to TSAN (making it |
| 1439 | unhappy). We don't think it's a real loop (there's an epoch point |
| 1440 | where that loop possibility disappears), but the advantages of |
| 1441 | keeping TSAN happy outweigh any performance advantage we might have |
| 1442 | by keeping the lock held. */ |
| 1443 | gpr_mu_unlock(&item->mu); |
Yash Tibrewal | 8cf1470 | 2017-12-06 09:47:54 -0800 | [diff] [blame] | 1444 | pi_new = polling_island_create(FD_FROM_PO(item), &error); |
Sree Kuchibhotla | f6f33d7 | 2016-11-18 11:38:52 -0800 | [diff] [blame] | 1445 | gpr_mu_lock(&item->mu); |
Sree Kuchibhotla | a129adf | 2016-10-26 16:44:44 -0700 | [diff] [blame] | 1446 | |
Sree Kuchibhotla | f6f33d7 | 2016-11-18 11:38:52 -0800 | [diff] [blame] | 1447 | /* Need to reverify any assumptions made between the initial lock and |
| 1448 | getting to this branch: if they've changed, we need to throw away our |
| 1449 | work and figure things out again. */ |
Craig Tiller | 4782d92 | 2017-11-10 09:53:21 -0800 | [diff] [blame] | 1450 | if (item->pi != nullptr) { |
Sree Kuchibhotla | 499b94b | 2016-11-18 14:35:47 -0800 | [diff] [blame] | 1451 | GRPC_POLLING_TRACE( |
| 1452 | "add_poll_object: Raced creating new polling island. pi_new: %p " |
| 1453 | "(fd: %d, %s: %p)", |
Craig Tiller | baa14a9 | 2017-11-03 09:09:36 -0700 | [diff] [blame] | 1454 | (void*)pi_new, FD_FROM_PO(item)->fd, poll_obj_string(bag_type), |
| 1455 | (void*)bag); |
Sree Kuchibhotla | f6f33d7 | 2016-11-18 11:38:52 -0800 | [diff] [blame] | 1456 | /* No need to lock 'pi_new' here since this is a new polling island |
Sree Kuchibhotla | 8214b87 | 2017-02-23 12:49:48 -0800 | [diff] [blame] | 1457 | and no one has a reference to it yet */ |
Sree Kuchibhotla | f6f33d7 | 2016-11-18 11:38:52 -0800 | [diff] [blame] | 1458 | polling_island_remove_all_fds_locked(pi_new, true, &error); |
| 1459 | |
| 1460 | /* Ref and unref so that the polling island gets deleted during unref |
| 1461 | */ |
| 1462 | PI_ADD_REF(pi_new, "dance_of_destruction"); |
Yash Tibrewal | 8cf1470 | 2017-12-06 09:47:54 -0800 | [diff] [blame] | 1463 | PI_UNREF(pi_new, "dance_of_destruction"); |
Sree Kuchibhotla | f6f33d7 | 2016-11-18 11:38:52 -0800 | [diff] [blame] | 1464 | goto retry; |
| 1465 | } |
Craig Tiller | 27da642 | 2016-07-06 13:14:46 -0700 | [diff] [blame] | 1466 | } else { |
Yash Tibrewal | 8cf1470 | 2017-12-06 09:47:54 -0800 | [diff] [blame] | 1467 | pi_new = polling_island_create(nullptr, &error); |
Craig Tiller | 7212c23 | 2016-07-06 13:11:09 -0700 | [diff] [blame] | 1468 | } |
Sree Kuchibhotla | 499b94b | 2016-11-18 14:35:47 -0800 | [diff] [blame] | 1469 | |
| 1470 | GRPC_POLLING_TRACE( |
| 1471 | "add_poll_object: Created new polling island. pi_new: %p (%s: %p, " |
| 1472 | "%s: %p)", |
Craig Tiller | baa14a9 | 2017-11-03 09:09:36 -0700 | [diff] [blame] | 1473 | (void*)pi_new, poll_obj_string(item_type), (void*)item, |
| 1474 | poll_obj_string(bag_type), (void*)bag); |
Sree Kuchibhotla | 499b94b | 2016-11-18 14:35:47 -0800 | [diff] [blame] | 1475 | } else { |
| 1476 | GRPC_POLLING_TRACE( |
| 1477 | "add_poll_object: Same polling island. pi: %p (%s, %s)", |
Craig Tiller | baa14a9 | 2017-11-03 09:09:36 -0700 | [diff] [blame] | 1478 | (void*)pi_new, poll_obj_string(item_type), poll_obj_string(bag_type)); |
Sree Kuchibhotla | f448c34 | 2016-05-19 10:51:24 -0700 | [diff] [blame] | 1479 | } |
Craig Tiller | 4782d92 | 2017-11-10 09:53:21 -0800 | [diff] [blame] | 1480 | } else if (item->pi == nullptr) { |
Sree Kuchibhotla | f6f33d7 | 2016-11-18 11:38:52 -0800 | [diff] [blame] | 1481 | /* GPR_ASSERT(bag->pi != NULL) */ |
| 1482 | /* Make pi_new point to latest pi*/ |
| 1483 | pi_new = polling_island_lock(bag->pi); |
Sree Kuchibhotla | 1e77668 | 2016-06-28 14:09:26 -0700 | [diff] [blame] | 1484 | |
Sree Kuchibhotla | f6f33d7 | 2016-11-18 11:38:52 -0800 | [diff] [blame] | 1485 | if (item_type == POLL_OBJ_FD) { |
Craig Tiller | baa14a9 | 2017-11-03 09:09:36 -0700 | [diff] [blame] | 1486 | grpc_fd* fd = FD_FROM_PO(item); |
Sree Kuchibhotla | f6f33d7 | 2016-11-18 11:38:52 -0800 | [diff] [blame] | 1487 | polling_island_add_fds_locked(pi_new, &fd, 1, true, &error); |
| 1488 | } |
Sree Kuchibhotla | 1e77668 | 2016-06-28 14:09:26 -0700 | [diff] [blame] | 1489 | |
Sree Kuchibhotla | f6f33d7 | 2016-11-18 11:38:52 -0800 | [diff] [blame] | 1490 | gpr_mu_unlock(&pi_new->mu); |
Sree Kuchibhotla | 1e77668 | 2016-06-28 14:09:26 -0700 | [diff] [blame] | 1491 | GRPC_POLLING_TRACE( |
Sree Kuchibhotla | 499b94b | 2016-11-18 14:35:47 -0800 | [diff] [blame] | 1492 | "add_poll_obj: item->pi was NULL. pi_new: %p (item(%s): %p, " |
| 1493 | "bag(%s): %p)", |
Craig Tiller | baa14a9 | 2017-11-03 09:09:36 -0700 | [diff] [blame] | 1494 | (void*)pi_new, poll_obj_string(item_type), (void*)item, |
| 1495 | poll_obj_string(bag_type), (void*)bag); |
Craig Tiller | 4782d92 | 2017-11-10 09:53:21 -0800 | [diff] [blame] | 1496 | } else if (bag->pi == nullptr) { |
Sree Kuchibhotla | f6f33d7 | 2016-11-18 11:38:52 -0800 | [diff] [blame] | 1497 | /* GPR_ASSERT(item->pi != NULL) */ |
| 1498 | /* Make pi_new to point to latest pi */ |
| 1499 | pi_new = polling_island_lock(item->pi); |
| 1500 | gpr_mu_unlock(&pi_new->mu); |
Sree Kuchibhotla | 499b94b | 2016-11-18 14:35:47 -0800 | [diff] [blame] | 1501 | GRPC_POLLING_TRACE( |
| 1502 | "add_poll_obj: bag->pi was NULL. pi_new: %p (item(%s): %p, " |
| 1503 | "bag(%s): %p)", |
Craig Tiller | baa14a9 | 2017-11-03 09:09:36 -0700 | [diff] [blame] | 1504 | (void*)pi_new, poll_obj_string(item_type), (void*)item, |
| 1505 | poll_obj_string(bag_type), (void*)bag); |
Sree Kuchibhotla | 5098f91 | 2016-05-31 10:58:17 -0700 | [diff] [blame] | 1506 | } else { |
Sree Kuchibhotla | f6f33d7 | 2016-11-18 11:38:52 -0800 | [diff] [blame] | 1507 | pi_new = polling_island_merge(item->pi, bag->pi, &error); |
Sree Kuchibhotla | 1e77668 | 2016-06-28 14:09:26 -0700 | [diff] [blame] | 1508 | GRPC_POLLING_TRACE( |
Sree Kuchibhotla | 499b94b | 2016-11-18 14:35:47 -0800 | [diff] [blame] | 1509 | "add_poll_obj: polling islands merged. pi_new: %p (item(%s): %p, " |
| 1510 | "bag(%s): %p)", |
Craig Tiller | baa14a9 | 2017-11-03 09:09:36 -0700 | [diff] [blame] | 1511 | (void*)pi_new, poll_obj_string(item_type), (void*)item, |
| 1512 | poll_obj_string(bag_type), (void*)bag); |
Sree Kuchibhotla | f448c34 | 2016-05-19 10:51:24 -0700 | [diff] [blame] | 1513 | } |
| 1514 | |
Sree Kuchibhotla | f6f33d7 | 2016-11-18 11:38:52 -0800 | [diff] [blame] | 1515 | /* At this point, pi_new is the polling island that both item->pi and bag->pi |
| 1516 | MUST be pointing to */ |
Sree Kuchibhotla | 20d0a16 | 2016-06-23 15:14:03 -0700 | [diff] [blame] | 1517 | |
Sree Kuchibhotla | f6f33d7 | 2016-11-18 11:38:52 -0800 | [diff] [blame] | 1518 | if (item->pi != pi_new) { |
| 1519 | PI_ADD_REF(pi_new, poll_obj_string(item_type)); |
Craig Tiller | 4782d92 | 2017-11-10 09:53:21 -0800 | [diff] [blame] | 1520 | if (item->pi != nullptr) { |
Yash Tibrewal | 8cf1470 | 2017-12-06 09:47:54 -0800 | [diff] [blame] | 1521 | PI_UNREF(item->pi, poll_obj_string(item_type)); |
Sree Kuchibhotla | 2f8ade0 | 2016-06-17 13:28:38 -0700 | [diff] [blame] | 1522 | } |
Sree Kuchibhotla | f6f33d7 | 2016-11-18 11:38:52 -0800 | [diff] [blame] | 1523 | item->pi = pi_new; |
Sree Kuchibhotla | 2f8ade0 | 2016-06-17 13:28:38 -0700 | [diff] [blame] | 1524 | } |
| 1525 | |
Sree Kuchibhotla | f6f33d7 | 2016-11-18 11:38:52 -0800 | [diff] [blame] | 1526 | if (bag->pi != pi_new) { |
| 1527 | PI_ADD_REF(pi_new, poll_obj_string(bag_type)); |
Craig Tiller | 4782d92 | 2017-11-10 09:53:21 -0800 | [diff] [blame] | 1528 | if (bag->pi != nullptr) { |
Yash Tibrewal | 8cf1470 | 2017-12-06 09:47:54 -0800 | [diff] [blame] | 1529 | PI_UNREF(bag->pi, poll_obj_string(bag_type)); |
Sree Kuchibhotla | 2f8ade0 | 2016-06-17 13:28:38 -0700 | [diff] [blame] | 1530 | } |
Sree Kuchibhotla | f6f33d7 | 2016-11-18 11:38:52 -0800 | [diff] [blame] | 1531 | bag->pi = pi_new; |
Sree Kuchibhotla | 2f8ade0 | 2016-06-17 13:28:38 -0700 | [diff] [blame] | 1532 | } |
Sree Kuchibhotla | f448c34 | 2016-05-19 10:51:24 -0700 | [diff] [blame] | 1533 | |
Sree Kuchibhotla | f6f33d7 | 2016-11-18 11:38:52 -0800 | [diff] [blame] | 1534 | gpr_mu_unlock(&item->mu); |
| 1535 | gpr_mu_unlock(&bag->mu); |
Craig Tiller | 1500761 | 2016-07-06 09:36:16 -0700 | [diff] [blame] | 1536 | |
Sree Kuchibhotla | f6f33d7 | 2016-11-18 11:38:52 -0800 | [diff] [blame] | 1537 | GRPC_LOG_IF_ERROR("add_poll_object", error); |
| 1538 | GPR_TIMER_END("add_poll_object", 0); |
| 1539 | } |
Craig Tiller | 57726ca | 2016-09-12 11:59:45 -0700 | [diff] [blame] | 1540 | |
Yash Tibrewal | 8cf1470 | 2017-12-06 09:47:54 -0800 | [diff] [blame] | 1541 | static void pollset_add_fd(grpc_pollset* pollset, grpc_fd* fd) { |
| 1542 | add_poll_object(&pollset->po, POLL_OBJ_POLLSET, &fd->po, POLL_OBJ_FD); |
Sree Kuchibhotla | f448c34 | 2016-05-19 10:51:24 -0700 | [diff] [blame] | 1543 | } |
| 1544 | |
Sree Kuchibhotla | f448c34 | 2016-05-19 10:51:24 -0700 | [diff] [blame] | 1545 | /******************************************************************************* |
Sree Kuchibhotla | 0bcbd79 | 2016-06-01 15:43:03 -0700 | [diff] [blame] | 1546 | * Pollset-set Definitions |
Sree Kuchibhotla | f448c34 | 2016-05-19 10:51:24 -0700 | [diff] [blame] | 1547 | */ |
| 1548 | |
Craig Tiller | baa14a9 | 2017-11-03 09:09:36 -0700 | [diff] [blame] | 1549 | static grpc_pollset_set* pollset_set_create(void) { |
| 1550 | grpc_pollset_set* pss = (grpc_pollset_set*)gpr_malloc(sizeof(*pss)); |
Sree Kuchibhotla | 2385fd7 | 2016-11-18 16:30:41 -0800 | [diff] [blame] | 1551 | gpr_mu_init(&pss->po.mu); |
Craig Tiller | 4782d92 | 2017-11-10 09:53:21 -0800 | [diff] [blame] | 1552 | pss->po.pi = nullptr; |
ncteisen | e9cd8a8 | 2017-06-29 06:03:52 -0400 | [diff] [blame] | 1553 | #ifndef NDEBUG |
Sree Kuchibhotla | a0749a6 | 2016-11-18 20:22:09 -0800 | [diff] [blame] | 1554 | pss->po.obj_type = POLL_OBJ_POLLSET_SET; |
| 1555 | #endif |
Sree Kuchibhotla | 2385fd7 | 2016-11-18 16:30:41 -0800 | [diff] [blame] | 1556 | return pss; |
Sree Kuchibhotla | f448c34 | 2016-05-19 10:51:24 -0700 | [diff] [blame] | 1557 | } |
| 1558 | |
Yash Tibrewal | 8cf1470 | 2017-12-06 09:47:54 -0800 | [diff] [blame] | 1559 | static void pollset_set_destroy(grpc_pollset_set* pss) { |
Sree Kuchibhotla | 2385fd7 | 2016-11-18 16:30:41 -0800 | [diff] [blame] | 1560 | gpr_mu_destroy(&pss->po.mu); |
| 1561 | |
Craig Tiller | 4782d92 | 2017-11-10 09:53:21 -0800 | [diff] [blame] | 1562 | if (pss->po.pi != nullptr) { |
Yash Tibrewal | 8cf1470 | 2017-12-06 09:47:54 -0800 | [diff] [blame] | 1563 | PI_UNREF(pss->po.pi, "pss_destroy"); |
Sree Kuchibhotla | f448c34 | 2016-05-19 10:51:24 -0700 | [diff] [blame] | 1564 | } |
Sree Kuchibhotla | 2385fd7 | 2016-11-18 16:30:41 -0800 | [diff] [blame] | 1565 | |
| 1566 | gpr_free(pss); |
Sree Kuchibhotla | f448c34 | 2016-05-19 10:51:24 -0700 | [diff] [blame] | 1567 | } |
| 1568 | |
Yash Tibrewal | 8cf1470 | 2017-12-06 09:47:54 -0800 | [diff] [blame] | 1569 | static void pollset_set_add_fd(grpc_pollset_set* pss, grpc_fd* fd) { |
| 1570 | add_poll_object(&pss->po, POLL_OBJ_POLLSET_SET, &fd->po, POLL_OBJ_FD); |
Sree Kuchibhotla | 0bcbd79 | 2016-06-01 15:43:03 -0700 | [diff] [blame] | 1571 | } |
| 1572 | |
Yash Tibrewal | 8cf1470 | 2017-12-06 09:47:54 -0800 | [diff] [blame] | 1573 | static void pollset_set_del_fd(grpc_pollset_set* pss, grpc_fd* fd) { |
Sree Kuchibhotla | 2385fd7 | 2016-11-18 16:30:41 -0800 | [diff] [blame] | 1574 | /* Nothing to do */ |
Sree Kuchibhotla | 0bcbd79 | 2016-06-01 15:43:03 -0700 | [diff] [blame] | 1575 | } |
| 1576 | |
Yash Tibrewal | 8cf1470 | 2017-12-06 09:47:54 -0800 | [diff] [blame] | 1577 | static void pollset_set_add_pollset(grpc_pollset_set* pss, grpc_pollset* ps) { |
| 1578 | add_poll_object(&pss->po, POLL_OBJ_POLLSET_SET, &ps->po, POLL_OBJ_POLLSET); |
Sree Kuchibhotla | f448c34 | 2016-05-19 10:51:24 -0700 | [diff] [blame] | 1579 | } |
| 1580 | |
Yash Tibrewal | 8cf1470 | 2017-12-06 09:47:54 -0800 | [diff] [blame] | 1581 | static void pollset_set_del_pollset(grpc_pollset_set* pss, grpc_pollset* ps) { |
Sree Kuchibhotla | 2385fd7 | 2016-11-18 16:30:41 -0800 | [diff] [blame] | 1582 | /* Nothing to do */ |
Sree Kuchibhotla | f448c34 | 2016-05-19 10:51:24 -0700 | [diff] [blame] | 1583 | } |
| 1584 | |
Yash Tibrewal | 8cf1470 | 2017-12-06 09:47:54 -0800 | [diff] [blame] | 1585 | static void pollset_set_add_pollset_set(grpc_pollset_set* bag, |
Craig Tiller | baa14a9 | 2017-11-03 09:09:36 -0700 | [diff] [blame] | 1586 | grpc_pollset_set* item) { |
Yash Tibrewal | 8cf1470 | 2017-12-06 09:47:54 -0800 | [diff] [blame] | 1587 | add_poll_object(&bag->po, POLL_OBJ_POLLSET_SET, &item->po, |
Sree Kuchibhotla | 2385fd7 | 2016-11-18 16:30:41 -0800 | [diff] [blame] | 1588 | POLL_OBJ_POLLSET_SET); |
Sree Kuchibhotla | f448c34 | 2016-05-19 10:51:24 -0700 | [diff] [blame] | 1589 | } |
| 1590 | |
Yash Tibrewal | 8cf1470 | 2017-12-06 09:47:54 -0800 | [diff] [blame] | 1591 | static void pollset_set_del_pollset_set(grpc_pollset_set* bag, |
Craig Tiller | baa14a9 | 2017-11-03 09:09:36 -0700 | [diff] [blame] | 1592 | grpc_pollset_set* item) { |
Sree Kuchibhotla | 2385fd7 | 2016-11-18 16:30:41 -0800 | [diff] [blame] | 1593 | /* Nothing to do */ |
Sree Kuchibhotla | f448c34 | 2016-05-19 10:51:24 -0700 | [diff] [blame] | 1594 | } |
| 1595 | |
Sree Kuchibhotla | 2e12db9 | 2016-06-16 16:53:59 -0700 | [diff] [blame] | 1596 | /* Test helper functions |
| 1597 | * */ |
Craig Tiller | baa14a9 | 2017-11-03 09:09:36 -0700 | [diff] [blame] | 1598 | void* grpc_fd_get_polling_island(grpc_fd* fd) { |
| 1599 | polling_island* pi; |
Sree Kuchibhotla | 2e12db9 | 2016-06-16 16:53:59 -0700 | [diff] [blame] | 1600 | |
Sree Kuchibhotla | f6f33d7 | 2016-11-18 11:38:52 -0800 | [diff] [blame] | 1601 | gpr_mu_lock(&fd->po.mu); |
| 1602 | pi = fd->po.pi; |
| 1603 | gpr_mu_unlock(&fd->po.mu); |
Sree Kuchibhotla | 2e12db9 | 2016-06-16 16:53:59 -0700 | [diff] [blame] | 1604 | |
| 1605 | return pi; |
| 1606 | } |
| 1607 | |
Craig Tiller | baa14a9 | 2017-11-03 09:09:36 -0700 | [diff] [blame] | 1608 | void* grpc_pollset_get_polling_island(grpc_pollset* ps) { |
| 1609 | polling_island* pi; |
Sree Kuchibhotla | 2e12db9 | 2016-06-16 16:53:59 -0700 | [diff] [blame] | 1610 | |
Sree Kuchibhotla | f6f33d7 | 2016-11-18 11:38:52 -0800 | [diff] [blame] | 1611 | gpr_mu_lock(&ps->po.mu); |
| 1612 | pi = ps->po.pi; |
| 1613 | gpr_mu_unlock(&ps->po.mu); |
Sree Kuchibhotla | 2e12db9 | 2016-06-16 16:53:59 -0700 | [diff] [blame] | 1614 | |
| 1615 | return pi; |
| 1616 | } |
| 1617 | |
Craig Tiller | baa14a9 | 2017-11-03 09:09:36 -0700 | [diff] [blame] | 1618 | bool grpc_are_polling_islands_equal(void* p, void* q) { |
| 1619 | polling_island* p1 = (polling_island*)p; |
| 1620 | polling_island* p2 = (polling_island*)q; |
Sree Kuchibhotla | 2f8ade0 | 2016-06-17 13:28:38 -0700 | [diff] [blame] | 1621 | |
Sree Kuchibhotla | 20d0a16 | 2016-06-23 15:14:03 -0700 | [diff] [blame] | 1622 | /* Note: polling_island_lock_pair() may change p1 and p2 to point to the |
| 1623 | latest polling islands in their respective linked lists */ |
Sree Kuchibhotla | 2f8ade0 | 2016-06-17 13:28:38 -0700 | [diff] [blame] | 1624 | polling_island_lock_pair(&p1, &p2); |
Sree Kuchibhotla | 20d0a16 | 2016-06-23 15:14:03 -0700 | [diff] [blame] | 1625 | polling_island_unlock_pair(p1, p2); |
Sree Kuchibhotla | 2f8ade0 | 2016-06-17 13:28:38 -0700 | [diff] [blame] | 1626 | |
| 1627 | return p1 == p2; |
Sree Kuchibhotla | 2e12db9 | 2016-06-16 16:53:59 -0700 | [diff] [blame] | 1628 | } |
| 1629 | |
Sree Kuchibhotla | f448c34 | 2016-05-19 10:51:24 -0700 | [diff] [blame] | 1630 | /******************************************************************************* |
Sree Kuchibhotla | 0bcbd79 | 2016-06-01 15:43:03 -0700 | [diff] [blame] | 1631 | * Event engine binding |
Sree Kuchibhotla | f448c34 | 2016-05-19 10:51:24 -0700 | [diff] [blame] | 1632 | */ |
| 1633 | |
| 1634 | static void shutdown_engine(void) { |
| 1635 | fd_global_shutdown(); |
| 1636 | pollset_global_shutdown(); |
Sree Kuchibhotla | d627c10 | 2016-06-06 15:49:32 -0700 | [diff] [blame] | 1637 | polling_island_global_shutdown(); |
Sree Kuchibhotla | f448c34 | 2016-05-19 10:51:24 -0700 | [diff] [blame] | 1638 | } |
| 1639 | |
| 1640 | static const grpc_event_engine_vtable vtable = { |
Yash Tibrewal | 533d118 | 2017-09-18 10:48:22 -0700 | [diff] [blame] | 1641 | sizeof(grpc_pollset), |
Sree Kuchibhotla | f448c34 | 2016-05-19 10:51:24 -0700 | [diff] [blame] | 1642 | |
Yash Tibrewal | 533d118 | 2017-09-18 10:48:22 -0700 | [diff] [blame] | 1643 | fd_create, |
| 1644 | fd_wrapped_fd, |
| 1645 | fd_orphan, |
| 1646 | fd_shutdown, |
| 1647 | fd_notify_on_read, |
| 1648 | fd_notify_on_write, |
| 1649 | fd_is_shutdown, |
| 1650 | fd_get_read_notifier_pollset, |
Sree Kuchibhotla | f448c34 | 2016-05-19 10:51:24 -0700 | [diff] [blame] | 1651 | |
Yash Tibrewal | 533d118 | 2017-09-18 10:48:22 -0700 | [diff] [blame] | 1652 | pollset_init, |
| 1653 | pollset_shutdown, |
| 1654 | pollset_destroy, |
| 1655 | pollset_work, |
| 1656 | pollset_kick, |
| 1657 | pollset_add_fd, |
Sree Kuchibhotla | f448c34 | 2016-05-19 10:51:24 -0700 | [diff] [blame] | 1658 | |
Yash Tibrewal | 533d118 | 2017-09-18 10:48:22 -0700 | [diff] [blame] | 1659 | pollset_set_create, |
| 1660 | pollset_set_destroy, |
| 1661 | pollset_set_add_pollset, |
| 1662 | pollset_set_del_pollset, |
| 1663 | pollset_set_add_pollset_set, |
| 1664 | pollset_set_del_pollset_set, |
| 1665 | pollset_set_add_fd, |
| 1666 | pollset_set_del_fd, |
Sree Kuchibhotla | f448c34 | 2016-05-19 10:51:24 -0700 | [diff] [blame] | 1667 | |
Yash Tibrewal | 533d118 | 2017-09-18 10:48:22 -0700 | [diff] [blame] | 1668 | shutdown_engine, |
Sree Kuchibhotla | f448c34 | 2016-05-19 10:51:24 -0700 | [diff] [blame] | 1669 | }; |
| 1670 | |
Sree Kuchibhotla | 7274402 | 2016-06-09 09:42:06 -0700 | [diff] [blame] | 1671 | /* It is possible that GLIBC has epoll but the underlying kernel doesn't. |
| 1672 | * Create a dummy epoll_fd to make sure epoll support is available */ |
| 1673 | static bool is_epoll_available() { |
| 1674 | int fd = epoll_create1(EPOLL_CLOEXEC); |
| 1675 | if (fd < 0) { |
| 1676 | gpr_log( |
| 1677 | GPR_ERROR, |
| 1678 | "epoll_create1 failed with error: %d. Not using epoll polling engine", |
| 1679 | fd); |
| 1680 | return false; |
| 1681 | } |
| 1682 | close(fd); |
| 1683 | return true; |
| 1684 | } |
| 1685 | |
Craig Tiller | baa14a9 | 2017-11-03 09:09:36 -0700 | [diff] [blame] | 1686 | const grpc_event_engine_vtable* grpc_init_epollsig_linux( |
Craig Tiller | f8382b8 | 2017-04-27 15:09:48 -0700 | [diff] [blame] | 1687 | bool explicit_request) { |
Sree Kuchibhotla | c7be7c6 | 2016-06-09 17:08:50 -0700 | [diff] [blame] | 1688 | /* If use of signals is disabled, we cannot use epoll engine*/ |
| 1689 | if (is_grpc_wakeup_signal_initialized && grpc_wakeup_signal < 0) { |
yang-g | 30101b0 | 2017-11-06 14:35:30 -0800 | [diff] [blame] | 1690 | gpr_log(GPR_ERROR, "Skipping epollsig because use of signals is disabled."); |
Craig Tiller | 4782d92 | 2017-11-10 09:53:21 -0800 | [diff] [blame] | 1691 | return nullptr; |
Sree Kuchibhotla | c7be7c6 | 2016-06-09 17:08:50 -0700 | [diff] [blame] | 1692 | } |
| 1693 | |
Ken Payson | cd7d047 | 2016-10-11 12:24:20 -0700 | [diff] [blame] | 1694 | if (!grpc_has_wakeup_fd()) { |
yang-g | 30101b0 | 2017-11-06 14:35:30 -0800 | [diff] [blame] | 1695 | gpr_log(GPR_ERROR, "Skipping epollsig because of no wakeup fd."); |
Craig Tiller | 4782d92 | 2017-11-10 09:53:21 -0800 | [diff] [blame] | 1696 | return nullptr; |
Ken Payson | bc544be | 2016-10-06 19:23:47 -0700 | [diff] [blame] | 1697 | } |
| 1698 | |
Sree Kuchibhotla | 7274402 | 2016-06-09 09:42:06 -0700 | [diff] [blame] | 1699 | if (!is_epoll_available()) { |
yang-g | 30101b0 | 2017-11-06 14:35:30 -0800 | [diff] [blame] | 1700 | gpr_log(GPR_ERROR, "Skipping epollsig because epoll is unavailable."); |
Craig Tiller | 4782d92 | 2017-11-10 09:53:21 -0800 | [diff] [blame] | 1701 | return nullptr; |
Sree Kuchibhotla | 7274402 | 2016-06-09 09:42:06 -0700 | [diff] [blame] | 1702 | } |
| 1703 | |
Sree Kuchibhotla | c7be7c6 | 2016-06-09 17:08:50 -0700 | [diff] [blame] | 1704 | if (!is_grpc_wakeup_signal_initialized) { |
Sree Kuchibhotla | 0fda880 | 2017-08-30 20:34:51 -0700 | [diff] [blame] | 1705 | if (explicit_request) { |
Craig Tiller | f8382b8 | 2017-04-27 15:09:48 -0700 | [diff] [blame] | 1706 | grpc_use_signal(SIGRTMIN + 6); |
| 1707 | } else { |
yang-g | 30101b0 | 2017-11-06 14:35:30 -0800 | [diff] [blame] | 1708 | gpr_log(GPR_ERROR, |
| 1709 | "Skipping epollsig because uninitialized wakeup signal."); |
Craig Tiller | 4782d92 | 2017-11-10 09:53:21 -0800 | [diff] [blame] | 1710 | return nullptr; |
Craig Tiller | f8382b8 | 2017-04-27 15:09:48 -0700 | [diff] [blame] | 1711 | } |
Sree Kuchibhotla | c7be7c6 | 2016-06-09 17:08:50 -0700 | [diff] [blame] | 1712 | } |
| 1713 | |
Sree Kuchibhotla | f448c34 | 2016-05-19 10:51:24 -0700 | [diff] [blame] | 1714 | fd_global_init(); |
Sree Kuchibhotla | 3131c26 | 2016-06-21 17:28:28 -0700 | [diff] [blame] | 1715 | |
| 1716 | if (!GRPC_LOG_IF_ERROR("pollset_global_init", pollset_global_init())) { |
Craig Tiller | 4782d92 | 2017-11-10 09:53:21 -0800 | [diff] [blame] | 1717 | return nullptr; |
Sree Kuchibhotla | 3131c26 | 2016-06-21 17:28:28 -0700 | [diff] [blame] | 1718 | } |
| 1719 | |
| 1720 | if (!GRPC_LOG_IF_ERROR("polling_island_global_init", |
| 1721 | polling_island_global_init())) { |
Craig Tiller | 4782d92 | 2017-11-10 09:53:21 -0800 | [diff] [blame] | 1722 | return nullptr; |
Sree Kuchibhotla | 3131c26 | 2016-06-21 17:28:28 -0700 | [diff] [blame] | 1723 | } |
| 1724 | |
Sree Kuchibhotla | f448c34 | 2016-05-19 10:51:24 -0700 | [diff] [blame] | 1725 | return &vtable; |
| 1726 | } |
| 1727 | |
Mehrdad Afshari | fb66900 | 2018-01-17 15:37:56 -0800 | [diff] [blame^] | 1728 | #else /* defined(GRPC_LINUX_EPOLL_CREATE1) */ |
murgatroid99 | 623dd4f | 2016-08-08 17:31:27 -0700 | [diff] [blame] | 1729 | #if defined(GRPC_POSIX_SOCKET) |
Yash Tibrewal | 1cac223 | 2017-09-26 11:31:11 -0700 | [diff] [blame] | 1730 | #include "src/core/lib/iomgr/ev_epollsig_linux.h" |
Mehrdad Afshari | fb66900 | 2018-01-17 15:37:56 -0800 | [diff] [blame^] | 1731 | /* If GRPC_LINUX_EPOLL_CREATE1 is not defined, it means |
| 1732 | epoll_create1 is not available. Return NULL */ |
Craig Tiller | baa14a9 | 2017-11-03 09:09:36 -0700 | [diff] [blame] | 1733 | const grpc_event_engine_vtable* grpc_init_epollsig_linux( |
Craig Tiller | f8382b8 | 2017-04-27 15:09:48 -0700 | [diff] [blame] | 1734 | bool explicit_request) { |
Yash Tibrewal | 8cf1470 | 2017-12-06 09:47:54 -0800 | [diff] [blame] | 1735 | return nullptr; |
Craig Tiller | f8382b8 | 2017-04-27 15:09:48 -0700 | [diff] [blame] | 1736 | } |
murgatroid99 | 623dd4f | 2016-08-08 17:31:27 -0700 | [diff] [blame] | 1737 | #endif /* defined(GRPC_POSIX_SOCKET) */ |
Sree Kuchibhotla | 5855c47 | 2016-06-08 12:56:56 -0700 | [diff] [blame] | 1738 | |
Sree Kuchibhotla | c7be7c6 | 2016-06-09 17:08:50 -0700 | [diff] [blame] | 1739 | void grpc_use_signal(int signum) {} |
Mehrdad Afshari | fb66900 | 2018-01-17 15:37:56 -0800 | [diff] [blame^] | 1740 | #endif /* !defined(GRPC_LINUX_EPOLL_CREATE1) */ |