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