Steven Moreland | bdb53ab | 2021-05-05 17:57:41 +0000 | [diff] [blame] | 1 | /* |
| 2 | * Copyright (C) 2020 The Android Open Source Project |
| 3 | * |
| 4 | * Licensed under the Apache License, Version 2.0 (the "License"); |
| 5 | * you may not use this file except in compliance with the License. |
| 6 | * You may obtain a copy of the License at |
| 7 | * |
| 8 | * http://www.apache.org/licenses/LICENSE-2.0 |
| 9 | * |
| 10 | * Unless required by applicable law or agreed to in writing, software |
| 11 | * distributed under the License is distributed on an "AS IS" BASIS, |
| 12 | * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. |
| 13 | * See the License for the specific language governing permissions and |
| 14 | * limitations under the License. |
| 15 | */ |
| 16 | |
| 17 | #define LOG_TAG "RpcSession" |
| 18 | |
| 19 | #include <binder/RpcSession.h> |
| 20 | |
Yifan Hong | 194acf2 | 2021-06-29 18:44:56 -0700 | [diff] [blame] | 21 | #include <dlfcn.h> |
Steven Moreland | bdb53ab | 2021-05-05 17:57:41 +0000 | [diff] [blame] | 22 | #include <inttypes.h> |
Steven Moreland | 4ec3c43 | 2021-05-20 00:32:47 +0000 | [diff] [blame] | 23 | #include <poll.h> |
Yifan Hong | 194acf2 | 2021-06-29 18:44:56 -0700 | [diff] [blame] | 24 | #include <pthread.h> |
Steven Moreland | bdb53ab | 2021-05-05 17:57:41 +0000 | [diff] [blame] | 25 | #include <unistd.h> |
| 26 | |
| 27 | #include <string_view> |
| 28 | |
Steven Moreland | 826367f | 2021-09-10 14:05:31 -0700 | [diff] [blame^] | 29 | #include <android-base/hex.h> |
Steven Moreland | 4ec3c43 | 2021-05-20 00:32:47 +0000 | [diff] [blame] | 30 | #include <android-base/macros.h> |
Yifan Hong | 194acf2 | 2021-06-29 18:44:56 -0700 | [diff] [blame] | 31 | #include <android_runtime/vm.h> |
Steven Moreland | bdb53ab | 2021-05-05 17:57:41 +0000 | [diff] [blame] | 32 | #include <binder/Parcel.h> |
Steven Moreland | ee78e76 | 2021-05-05 21:12:51 +0000 | [diff] [blame] | 33 | #include <binder/RpcServer.h> |
Yifan Hong | 702115c | 2021-06-24 15:39:18 -0700 | [diff] [blame] | 34 | #include <binder/RpcTransportRaw.h> |
Steven Moreland | bdb53ab | 2021-05-05 17:57:41 +0000 | [diff] [blame] | 35 | #include <binder/Stability.h> |
Yifan Hong | 194acf2 | 2021-06-29 18:44:56 -0700 | [diff] [blame] | 36 | #include <jni.h> |
Steven Moreland | bdb53ab | 2021-05-05 17:57:41 +0000 | [diff] [blame] | 37 | #include <utils/String8.h> |
| 38 | |
Yifan Hong | 8c95042 | 2021-08-05 17:13:55 -0700 | [diff] [blame] | 39 | #include "FdTrigger.h" |
Steven Moreland | bdb53ab | 2021-05-05 17:57:41 +0000 | [diff] [blame] | 40 | #include "RpcSocketAddress.h" |
| 41 | #include "RpcState.h" |
| 42 | #include "RpcWireFormat.h" |
Yifan Hong | b675ffe | 2021-08-05 16:37:17 -0700 | [diff] [blame] | 43 | #include "Utils.h" |
Steven Moreland | bdb53ab | 2021-05-05 17:57:41 +0000 | [diff] [blame] | 44 | |
| 45 | #ifdef __GLIBC__ |
| 46 | extern "C" pid_t gettid(); |
| 47 | #endif |
| 48 | |
| 49 | namespace android { |
| 50 | |
| 51 | using base::unique_fd; |
| 52 | |
Yifan Hong | ecf937d | 2021-08-11 17:29:28 -0700 | [diff] [blame] | 53 | RpcSession::RpcSession(std::unique_ptr<RpcTransportCtx> ctx) : mCtx(std::move(ctx)) { |
Steven Moreland | bdb53ab | 2021-05-05 17:57:41 +0000 | [diff] [blame] | 54 | LOG_RPC_DETAIL("RpcSession created %p", this); |
| 55 | |
| 56 | mState = std::make_unique<RpcState>(); |
| 57 | } |
| 58 | RpcSession::~RpcSession() { |
| 59 | LOG_RPC_DETAIL("RpcSession destroyed %p", this); |
| 60 | |
| 61 | std::lock_guard<std::mutex> _l(mMutex); |
Steven Moreland | 19fc9f7 | 2021-06-10 03:57:30 +0000 | [diff] [blame] | 62 | LOG_ALWAYS_FATAL_IF(mIncomingConnections.size() != 0, |
Steven Moreland | bdb53ab | 2021-05-05 17:57:41 +0000 | [diff] [blame] | 63 | "Should not be able to destroy a session with servers in use."); |
| 64 | } |
| 65 | |
Yifan Hong | ecf937d | 2021-08-11 17:29:28 -0700 | [diff] [blame] | 66 | sp<RpcSession> RpcSession::make() { |
Yifan Hong | 702115c | 2021-06-24 15:39:18 -0700 | [diff] [blame] | 67 | // Default is without TLS. |
Yifan Hong | ecf937d | 2021-08-11 17:29:28 -0700 | [diff] [blame] | 68 | return make(RpcTransportCtxFactoryRaw::make(), std::nullopt, std::nullopt); |
| 69 | } |
| 70 | |
| 71 | sp<RpcSession> RpcSession::make(std::unique_ptr<RpcTransportCtxFactory> rpcTransportCtxFactory, |
| 72 | std::optional<CertificateFormat> serverCertificateFormat, |
| 73 | std::optional<std::string> serverCertificate) { |
| 74 | auto ctx = rpcTransportCtxFactory->newClientCtx(); |
| 75 | if (ctx == nullptr) return nullptr; |
| 76 | LOG_ALWAYS_FATAL_IF(serverCertificateFormat.has_value() != serverCertificate.has_value()); |
| 77 | if (serverCertificateFormat.has_value() && serverCertificate.has_value()) { |
| 78 | status_t status = |
| 79 | ctx->addTrustedPeerCertificate(*serverCertificateFormat, *serverCertificate); |
| 80 | if (status != OK) { |
| 81 | ALOGE("Cannot add trusted server certificate: %s", statusToString(status).c_str()); |
| 82 | return nullptr; |
| 83 | } |
| 84 | } |
| 85 | return sp<RpcSession>::make(std::move(ctx)); |
Steven Moreland | bdb53ab | 2021-05-05 17:57:41 +0000 | [diff] [blame] | 86 | } |
| 87 | |
Steven Moreland | 103424e | 2021-06-02 18:16:19 +0000 | [diff] [blame] | 88 | void RpcSession::setMaxThreads(size_t threads) { |
| 89 | std::lock_guard<std::mutex> _l(mMutex); |
Steven Moreland | 19fc9f7 | 2021-06-10 03:57:30 +0000 | [diff] [blame] | 90 | LOG_ALWAYS_FATAL_IF(!mOutgoingConnections.empty() || !mIncomingConnections.empty(), |
Steven Moreland | 103424e | 2021-06-02 18:16:19 +0000 | [diff] [blame] | 91 | "Must set max threads before setting up connections, but has %zu client(s) " |
| 92 | "and %zu server(s)", |
Steven Moreland | 19fc9f7 | 2021-06-10 03:57:30 +0000 | [diff] [blame] | 93 | mOutgoingConnections.size(), mIncomingConnections.size()); |
Steven Moreland | 103424e | 2021-06-02 18:16:19 +0000 | [diff] [blame] | 94 | mMaxThreads = threads; |
| 95 | } |
| 96 | |
| 97 | size_t RpcSession::getMaxThreads() { |
| 98 | std::lock_guard<std::mutex> _l(mMutex); |
| 99 | return mMaxThreads; |
Steven Moreland | 659416d | 2021-05-11 00:47:50 +0000 | [diff] [blame] | 100 | } |
| 101 | |
Steven Moreland | bf57bce | 2021-07-26 15:26:12 -0700 | [diff] [blame] | 102 | bool RpcSession::setProtocolVersion(uint32_t version) { |
| 103 | if (version >= RPC_WIRE_PROTOCOL_VERSION_NEXT && |
| 104 | version != RPC_WIRE_PROTOCOL_VERSION_EXPERIMENTAL) { |
| 105 | ALOGE("Cannot start RPC session with version %u which is unknown (current protocol version " |
| 106 | "is %u).", |
| 107 | version, RPC_WIRE_PROTOCOL_VERSION); |
| 108 | return false; |
| 109 | } |
| 110 | |
| 111 | std::lock_guard<std::mutex> _l(mMutex); |
Steven Moreland | 40b736e | 2021-07-30 14:37:10 -0700 | [diff] [blame] | 112 | if (mProtocolVersion && version > *mProtocolVersion) { |
| 113 | ALOGE("Cannot upgrade explicitly capped protocol version %u to newer version %u", |
| 114 | *mProtocolVersion, version); |
| 115 | return false; |
| 116 | } |
| 117 | |
Steven Moreland | bf57bce | 2021-07-26 15:26:12 -0700 | [diff] [blame] | 118 | mProtocolVersion = version; |
| 119 | return true; |
| 120 | } |
| 121 | |
| 122 | std::optional<uint32_t> RpcSession::getProtocolVersion() { |
| 123 | std::lock_guard<std::mutex> _l(mMutex); |
| 124 | return mProtocolVersion; |
| 125 | } |
| 126 | |
Steven Moreland | 2372f9d | 2021-08-05 15:42:01 -0700 | [diff] [blame] | 127 | status_t RpcSession::setupUnixDomainClient(const char* path) { |
Steven Moreland | bdb53ab | 2021-05-05 17:57:41 +0000 | [diff] [blame] | 128 | return setupSocketClient(UnixSocketAddress(path)); |
| 129 | } |
| 130 | |
Steven Moreland | 2372f9d | 2021-08-05 15:42:01 -0700 | [diff] [blame] | 131 | status_t RpcSession::setupVsockClient(unsigned int cid, unsigned int port) { |
Steven Moreland | bdb53ab | 2021-05-05 17:57:41 +0000 | [diff] [blame] | 132 | return setupSocketClient(VsockSocketAddress(cid, port)); |
| 133 | } |
| 134 | |
Steven Moreland | 2372f9d | 2021-08-05 15:42:01 -0700 | [diff] [blame] | 135 | status_t RpcSession::setupInetClient(const char* addr, unsigned int port) { |
Steven Moreland | bdb53ab | 2021-05-05 17:57:41 +0000 | [diff] [blame] | 136 | auto aiStart = InetSocketAddress::getAddrInfo(addr, port); |
Steven Moreland | 2372f9d | 2021-08-05 15:42:01 -0700 | [diff] [blame] | 137 | if (aiStart == nullptr) return UNKNOWN_ERROR; |
Steven Moreland | bdb53ab | 2021-05-05 17:57:41 +0000 | [diff] [blame] | 138 | for (auto ai = aiStart.get(); ai != nullptr; ai = ai->ai_next) { |
| 139 | InetSocketAddress socketAddress(ai->ai_addr, ai->ai_addrlen, addr, port); |
Steven Moreland | 2372f9d | 2021-08-05 15:42:01 -0700 | [diff] [blame] | 140 | if (status_t status = setupSocketClient(socketAddress); status == OK) return OK; |
Steven Moreland | bdb53ab | 2021-05-05 17:57:41 +0000 | [diff] [blame] | 141 | } |
| 142 | ALOGE("None of the socket address resolved for %s:%u can be added as inet client.", addr, port); |
Steven Moreland | 2372f9d | 2021-08-05 15:42:01 -0700 | [diff] [blame] | 143 | return NAME_NOT_FOUND; |
Steven Moreland | bdb53ab | 2021-05-05 17:57:41 +0000 | [diff] [blame] | 144 | } |
| 145 | |
Steven Moreland | 2372f9d | 2021-08-05 15:42:01 -0700 | [diff] [blame] | 146 | status_t RpcSession::setupPreconnectedClient(unique_fd fd, std::function<unique_fd()>&& request) { |
Steven Moreland | 826367f | 2021-09-10 14:05:31 -0700 | [diff] [blame^] | 147 | return setupClient([&](const std::vector<uint8_t>& sessionId, bool incoming) -> status_t { |
Steven Moreland | 4198a12 | 2021-08-03 17:37:58 -0700 | [diff] [blame] | 148 | // std::move'd from fd becomes -1 (!ok()) |
| 149 | if (!fd.ok()) { |
| 150 | fd = request(); |
Steven Moreland | 2372f9d | 2021-08-05 15:42:01 -0700 | [diff] [blame] | 151 | if (!fd.ok()) return BAD_VALUE; |
Steven Moreland | 4198a12 | 2021-08-03 17:37:58 -0700 | [diff] [blame] | 152 | } |
Yifan Hong | b675ffe | 2021-08-05 16:37:17 -0700 | [diff] [blame] | 153 | if (auto res = setNonBlocking(fd); !res.ok()) { |
| 154 | ALOGE("setupPreconnectedClient: %s", res.error().message().c_str()); |
| 155 | return res.error().code() == 0 ? UNKNOWN_ERROR : -res.error().code(); |
| 156 | } |
Steven Moreland | 4198a12 | 2021-08-03 17:37:58 -0700 | [diff] [blame] | 157 | return initAndAddConnection(std::move(fd), sessionId, incoming); |
| 158 | }); |
| 159 | } |
| 160 | |
Steven Moreland | 2372f9d | 2021-08-05 15:42:01 -0700 | [diff] [blame] | 161 | status_t RpcSession::addNullDebuggingClient() { |
Yifan Hong | 702115c | 2021-06-24 15:39:18 -0700 | [diff] [blame] | 162 | // Note: only works on raw sockets. |
Yifan Hong | 832521e | 2021-08-05 14:55:40 -0700 | [diff] [blame] | 163 | if (auto status = initShutdownTrigger(); status != OK) return status; |
| 164 | |
Steven Moreland | bdb53ab | 2021-05-05 17:57:41 +0000 | [diff] [blame] | 165 | unique_fd serverFd(TEMP_FAILURE_RETRY(open("/dev/null", O_WRONLY | O_CLOEXEC))); |
| 166 | |
| 167 | if (serverFd == -1) { |
Steven Moreland | 2372f9d | 2021-08-05 15:42:01 -0700 | [diff] [blame] | 168 | int savedErrno = errno; |
| 169 | ALOGE("Could not connect to /dev/null: %s", strerror(savedErrno)); |
| 170 | return -savedErrno; |
Steven Moreland | bdb53ab | 2021-05-05 17:57:41 +0000 | [diff] [blame] | 171 | } |
| 172 | |
Yifan Hong | ecf937d | 2021-08-11 17:29:28 -0700 | [diff] [blame] | 173 | auto server = mCtx->newTransport(std::move(serverFd), mShutdownTrigger.get()); |
Yifan Hong | 702115c | 2021-06-24 15:39:18 -0700 | [diff] [blame] | 174 | if (server == nullptr) { |
| 175 | ALOGE("Unable to set up RpcTransport"); |
Steven Moreland | 2372f9d | 2021-08-05 15:42:01 -0700 | [diff] [blame] | 176 | return UNKNOWN_ERROR; |
Yifan Hong | 702115c | 2021-06-24 15:39:18 -0700 | [diff] [blame] | 177 | } |
| 178 | return addOutgoingConnection(std::move(server), false); |
Steven Moreland | bdb53ab | 2021-05-05 17:57:41 +0000 | [diff] [blame] | 179 | } |
| 180 | |
| 181 | sp<IBinder> RpcSession::getRootObject() { |
Steven Moreland | 195edb8 | 2021-06-08 02:44:39 +0000 | [diff] [blame] | 182 | ExclusiveConnection connection; |
| 183 | status_t status = ExclusiveConnection::find(sp<RpcSession>::fromExisting(this), |
| 184 | ConnectionUse::CLIENT, &connection); |
| 185 | if (status != OK) return nullptr; |
Steven Moreland | 5ae6256 | 2021-06-10 03:21:42 +0000 | [diff] [blame] | 186 | return state()->getRootObject(connection.get(), sp<RpcSession>::fromExisting(this)); |
Steven Moreland | bdb53ab | 2021-05-05 17:57:41 +0000 | [diff] [blame] | 187 | } |
| 188 | |
Steven Moreland | 1be9135 | 2021-05-11 22:12:15 +0000 | [diff] [blame] | 189 | status_t RpcSession::getRemoteMaxThreads(size_t* maxThreads) { |
Steven Moreland | 195edb8 | 2021-06-08 02:44:39 +0000 | [diff] [blame] | 190 | ExclusiveConnection connection; |
| 191 | status_t status = ExclusiveConnection::find(sp<RpcSession>::fromExisting(this), |
| 192 | ConnectionUse::CLIENT, &connection); |
| 193 | if (status != OK) return status; |
Steven Moreland | 5ae6256 | 2021-06-10 03:21:42 +0000 | [diff] [blame] | 194 | return state()->getMaxThreads(connection.get(), sp<RpcSession>::fromExisting(this), maxThreads); |
Steven Moreland | bdb53ab | 2021-05-05 17:57:41 +0000 | [diff] [blame] | 195 | } |
| 196 | |
Steven Moreland | c9d7b53 | 2021-06-04 20:57:41 +0000 | [diff] [blame] | 197 | bool RpcSession::shutdownAndWait(bool wait) { |
Steven Moreland | 659416d | 2021-05-11 00:47:50 +0000 | [diff] [blame] | 198 | std::unique_lock<std::mutex> _l(mMutex); |
Steven Moreland | 659416d | 2021-05-11 00:47:50 +0000 | [diff] [blame] | 199 | LOG_ALWAYS_FATAL_IF(mShutdownTrigger == nullptr, "Shutdown trigger not installed"); |
Steven Moreland | 659416d | 2021-05-11 00:47:50 +0000 | [diff] [blame] | 200 | |
| 201 | mShutdownTrigger->trigger(); |
Steven Moreland | 659416d | 2021-05-11 00:47:50 +0000 | [diff] [blame] | 202 | |
Steven Moreland | c9d7b53 | 2021-06-04 20:57:41 +0000 | [diff] [blame] | 203 | if (wait) { |
| 204 | LOG_ALWAYS_FATAL_IF(mShutdownListener == nullptr, "Shutdown listener not installed"); |
| 205 | mShutdownListener->waitForShutdown(_l); |
Steven Moreland | dd67b94 | 2021-07-23 17:15:41 -0700 | [diff] [blame] | 206 | |
Steven Moreland | c9d7b53 | 2021-06-04 20:57:41 +0000 | [diff] [blame] | 207 | LOG_ALWAYS_FATAL_IF(!mThreads.empty(), "Shutdown failed"); |
| 208 | } |
| 209 | |
| 210 | _l.unlock(); |
| 211 | mState->clear(); |
| 212 | |
Steven Moreland | 659416d | 2021-05-11 00:47:50 +0000 | [diff] [blame] | 213 | return true; |
| 214 | } |
| 215 | |
Steven Moreland | f517427 | 2021-05-25 00:39:28 +0000 | [diff] [blame] | 216 | status_t RpcSession::transact(const sp<IBinder>& binder, uint32_t code, const Parcel& data, |
Steven Moreland | bdb53ab | 2021-05-05 17:57:41 +0000 | [diff] [blame] | 217 | Parcel* reply, uint32_t flags) { |
Steven Moreland | 195edb8 | 2021-06-08 02:44:39 +0000 | [diff] [blame] | 218 | ExclusiveConnection connection; |
| 219 | status_t status = |
| 220 | ExclusiveConnection::find(sp<RpcSession>::fromExisting(this), |
| 221 | (flags & IBinder::FLAG_ONEWAY) ? ConnectionUse::CLIENT_ASYNC |
| 222 | : ConnectionUse::CLIENT, |
| 223 | &connection); |
| 224 | if (status != OK) return status; |
Steven Moreland | 5ae6256 | 2021-06-10 03:21:42 +0000 | [diff] [blame] | 225 | return state()->transact(connection.get(), binder, code, data, |
Steven Moreland | bdb53ab | 2021-05-05 17:57:41 +0000 | [diff] [blame] | 226 | sp<RpcSession>::fromExisting(this), reply, flags); |
| 227 | } |
| 228 | |
| 229 | status_t RpcSession::sendDecStrong(const RpcAddress& address) { |
Steven Moreland | 195edb8 | 2021-06-08 02:44:39 +0000 | [diff] [blame] | 230 | ExclusiveConnection connection; |
| 231 | status_t status = ExclusiveConnection::find(sp<RpcSession>::fromExisting(this), |
| 232 | ConnectionUse::CLIENT_REFCOUNT, &connection); |
| 233 | if (status != OK) return status; |
Steven Moreland | 5ae6256 | 2021-06-10 03:21:42 +0000 | [diff] [blame] | 234 | return state()->sendDecStrong(connection.get(), sp<RpcSession>::fromExisting(this), address); |
Steven Moreland | bdb53ab | 2021-05-05 17:57:41 +0000 | [diff] [blame] | 235 | } |
| 236 | |
| 237 | status_t RpcSession::readId() { |
| 238 | { |
| 239 | std::lock_guard<std::mutex> _l(mMutex); |
| 240 | LOG_ALWAYS_FATAL_IF(mForServer != nullptr, "Can only update ID for client."); |
| 241 | } |
| 242 | |
Steven Moreland | 195edb8 | 2021-06-08 02:44:39 +0000 | [diff] [blame] | 243 | ExclusiveConnection connection; |
| 244 | status_t status = ExclusiveConnection::find(sp<RpcSession>::fromExisting(this), |
| 245 | ConnectionUse::CLIENT, &connection); |
| 246 | if (status != OK) return status; |
| 247 | |
Steven Moreland | 826367f | 2021-09-10 14:05:31 -0700 | [diff] [blame^] | 248 | status = state()->getSessionId(connection.get(), sp<RpcSession>::fromExisting(this), &mId); |
Steven Moreland | bdb53ab | 2021-05-05 17:57:41 +0000 | [diff] [blame] | 249 | if (status != OK) return status; |
| 250 | |
Steven Moreland | 826367f | 2021-09-10 14:05:31 -0700 | [diff] [blame^] | 251 | LOG_RPC_DETAIL("RpcSession %p has id %s", this, |
| 252 | base::HexString(mId.data(), mId.size()).c_str()); |
Steven Moreland | bdb53ab | 2021-05-05 17:57:41 +0000 | [diff] [blame] | 253 | return OK; |
| 254 | } |
| 255 | |
Steven Moreland | dd67b94 | 2021-07-23 17:15:41 -0700 | [diff] [blame] | 256 | void RpcSession::WaitForShutdownListener::onSessionAllIncomingThreadsEnded( |
Steven Moreland | 659416d | 2021-05-11 00:47:50 +0000 | [diff] [blame] | 257 | const sp<RpcSession>& session) { |
| 258 | (void)session; |
| 259 | mShutdown = true; |
| 260 | } |
| 261 | |
Steven Moreland | 19fc9f7 | 2021-06-10 03:57:30 +0000 | [diff] [blame] | 262 | void RpcSession::WaitForShutdownListener::onSessionIncomingThreadEnded() { |
Steven Moreland | 659416d | 2021-05-11 00:47:50 +0000 | [diff] [blame] | 263 | mCv.notify_all(); |
| 264 | } |
| 265 | |
| 266 | void RpcSession::WaitForShutdownListener::waitForShutdown(std::unique_lock<std::mutex>& lock) { |
| 267 | while (!mShutdown) { |
| 268 | if (std::cv_status::timeout == mCv.wait_for(lock, std::chrono::seconds(1))) { |
| 269 | ALOGE("Waiting for RpcSession to shut down (1s w/o progress)."); |
| 270 | } |
| 271 | } |
| 272 | } |
| 273 | |
Steven Moreland | c88b7fc | 2021-06-10 00:40:39 +0000 | [diff] [blame] | 274 | void RpcSession::preJoinThreadOwnership(std::thread thread) { |
Steven Moreland | a63ff93 | 2021-05-12 00:03:15 +0000 | [diff] [blame] | 275 | LOG_ALWAYS_FATAL_IF(thread.get_id() != std::this_thread::get_id(), "Must own this thread"); |
Steven Moreland | bdb53ab | 2021-05-05 17:57:41 +0000 | [diff] [blame] | 276 | |
Steven Moreland | a63ff93 | 2021-05-12 00:03:15 +0000 | [diff] [blame] | 277 | { |
| 278 | std::lock_guard<std::mutex> _l(mMutex); |
| 279 | mThreads[thread.get_id()] = std::move(thread); |
| 280 | } |
Steven Moreland | 5802c2b | 2021-05-12 20:13:04 +0000 | [diff] [blame] | 281 | } |
Steven Moreland | a63ff93 | 2021-05-12 00:03:15 +0000 | [diff] [blame] | 282 | |
Yifan Hong | 702115c | 2021-06-24 15:39:18 -0700 | [diff] [blame] | 283 | RpcSession::PreJoinSetupResult RpcSession::preJoinSetup( |
| 284 | std::unique_ptr<RpcTransport> rpcTransport) { |
Steven Moreland | bdb53ab | 2021-05-05 17:57:41 +0000 | [diff] [blame] | 285 | // must be registered to allow arbitrary client code executing commands to |
| 286 | // be able to do nested calls (we can't only read from it) |
Yifan Hong | 702115c | 2021-06-24 15:39:18 -0700 | [diff] [blame] | 287 | sp<RpcConnection> connection = assignIncomingConnectionToThisThread(std::move(rpcTransport)); |
Steven Moreland | bdb53ab | 2021-05-05 17:57:41 +0000 | [diff] [blame] | 288 | |
Steven Moreland | dd67b94 | 2021-07-23 17:15:41 -0700 | [diff] [blame] | 289 | status_t status; |
| 290 | |
| 291 | if (connection == nullptr) { |
| 292 | status = DEAD_OBJECT; |
| 293 | } else { |
| 294 | status = mState->readConnectionInit(connection, sp<RpcSession>::fromExisting(this)); |
| 295 | } |
Steven Moreland | bdb53ab | 2021-05-05 17:57:41 +0000 | [diff] [blame] | 296 | |
Steven Moreland | c88b7fc | 2021-06-10 00:40:39 +0000 | [diff] [blame] | 297 | return PreJoinSetupResult{ |
| 298 | .connection = std::move(connection), |
| 299 | .status = status, |
| 300 | }; |
| 301 | } |
| 302 | |
Yifan Hong | 194acf2 | 2021-06-29 18:44:56 -0700 | [diff] [blame] | 303 | namespace { |
| 304 | // RAII object for attaching / detaching current thread to JVM if Android Runtime exists. If |
| 305 | // Android Runtime doesn't exist, no-op. |
| 306 | class JavaThreadAttacher { |
| 307 | public: |
| 308 | JavaThreadAttacher() { |
| 309 | // Use dlsym to find androidJavaAttachThread because libandroid_runtime is loaded after |
| 310 | // libbinder. |
| 311 | auto vm = getJavaVM(); |
| 312 | if (vm == nullptr) return; |
| 313 | |
| 314 | char threadName[16]; |
| 315 | if (0 != pthread_getname_np(pthread_self(), threadName, sizeof(threadName))) { |
| 316 | constexpr const char* defaultThreadName = "UnknownRpcSessionThread"; |
| 317 | memcpy(threadName, defaultThreadName, |
| 318 | std::min<size_t>(sizeof(threadName), strlen(defaultThreadName) + 1)); |
| 319 | } |
| 320 | LOG_RPC_DETAIL("Attaching current thread %s to JVM", threadName); |
| 321 | JavaVMAttachArgs args; |
| 322 | args.version = JNI_VERSION_1_2; |
| 323 | args.name = threadName; |
| 324 | args.group = nullptr; |
| 325 | JNIEnv* env; |
| 326 | |
| 327 | LOG_ALWAYS_FATAL_IF(vm->AttachCurrentThread(&env, &args) != JNI_OK, |
| 328 | "Cannot attach thread %s to JVM", threadName); |
| 329 | mAttached = true; |
| 330 | } |
| 331 | ~JavaThreadAttacher() { |
| 332 | if (!mAttached) return; |
| 333 | auto vm = getJavaVM(); |
| 334 | LOG_ALWAYS_FATAL_IF(vm == nullptr, |
| 335 | "Unable to detach thread. No JavaVM, but it was present before!"); |
| 336 | |
| 337 | LOG_RPC_DETAIL("Detaching current thread from JVM"); |
| 338 | if (vm->DetachCurrentThread() != JNI_OK) { |
| 339 | mAttached = false; |
| 340 | } else { |
| 341 | ALOGW("Unable to detach current thread from JVM"); |
| 342 | } |
| 343 | } |
| 344 | |
| 345 | private: |
| 346 | DISALLOW_COPY_AND_ASSIGN(JavaThreadAttacher); |
| 347 | bool mAttached = false; |
| 348 | |
| 349 | static JavaVM* getJavaVM() { |
| 350 | static auto fn = reinterpret_cast<decltype(&AndroidRuntimeGetJavaVM)>( |
| 351 | dlsym(RTLD_DEFAULT, "AndroidRuntimeGetJavaVM")); |
| 352 | if (fn == nullptr) return nullptr; |
| 353 | return fn(); |
| 354 | } |
| 355 | }; |
| 356 | } // namespace |
| 357 | |
Steven Moreland | c88b7fc | 2021-06-10 00:40:39 +0000 | [diff] [blame] | 358 | void RpcSession::join(sp<RpcSession>&& session, PreJoinSetupResult&& setupResult) { |
| 359 | sp<RpcConnection>& connection = setupResult.connection; |
| 360 | |
| 361 | if (setupResult.status == OK) { |
Steven Moreland | dd67b94 | 2021-07-23 17:15:41 -0700 | [diff] [blame] | 362 | LOG_ALWAYS_FATAL_IF(!connection, "must have connection if setup succeeded"); |
Yifan Hong | 194acf2 | 2021-06-29 18:44:56 -0700 | [diff] [blame] | 363 | JavaThreadAttacher javaThreadAttacher; |
Steven Moreland | c88b7fc | 2021-06-10 00:40:39 +0000 | [diff] [blame] | 364 | while (true) { |
Steven Moreland | 5ae6256 | 2021-06-10 03:21:42 +0000 | [diff] [blame] | 365 | status_t status = session->state()->getAndExecuteCommand(connection, session, |
Steven Moreland | c88b7fc | 2021-06-10 00:40:39 +0000 | [diff] [blame] | 366 | RpcState::CommandType::ANY); |
| 367 | if (status != OK) { |
| 368 | LOG_RPC_DETAIL("Binder connection thread closing w/ status %s", |
| 369 | statusToString(status).c_str()); |
| 370 | break; |
| 371 | } |
Steven Moreland | bdb53ab | 2021-05-05 17:57:41 +0000 | [diff] [blame] | 372 | } |
Steven Moreland | c88b7fc | 2021-06-10 00:40:39 +0000 | [diff] [blame] | 373 | } else { |
| 374 | ALOGE("Connection failed to init, closing with status %s", |
| 375 | statusToString(setupResult.status).c_str()); |
Steven Moreland | bdb53ab | 2021-05-05 17:57:41 +0000 | [diff] [blame] | 376 | } |
| 377 | |
Steven Moreland | 659416d | 2021-05-11 00:47:50 +0000 | [diff] [blame] | 378 | sp<RpcSession::EventListener> listener; |
Steven Moreland | a63ff93 | 2021-05-12 00:03:15 +0000 | [diff] [blame] | 379 | { |
Steven Moreland | 659416d | 2021-05-11 00:47:50 +0000 | [diff] [blame] | 380 | std::lock_guard<std::mutex> _l(session->mMutex); |
| 381 | auto it = session->mThreads.find(std::this_thread::get_id()); |
| 382 | LOG_ALWAYS_FATAL_IF(it == session->mThreads.end()); |
Steven Moreland | a63ff93 | 2021-05-12 00:03:15 +0000 | [diff] [blame] | 383 | it->second.detach(); |
Steven Moreland | 659416d | 2021-05-11 00:47:50 +0000 | [diff] [blame] | 384 | session->mThreads.erase(it); |
Steven Moreland | ee3f466 | 2021-05-22 01:07:33 +0000 | [diff] [blame] | 385 | |
Steven Moreland | 659416d | 2021-05-11 00:47:50 +0000 | [diff] [blame] | 386 | listener = session->mEventListener.promote(); |
Steven Moreland | ee3f466 | 2021-05-22 01:07:33 +0000 | [diff] [blame] | 387 | } |
| 388 | |
Steven Moreland | dd67b94 | 2021-07-23 17:15:41 -0700 | [diff] [blame] | 389 | // done after all cleanup, since session shutdown progresses via callbacks here |
| 390 | if (connection != nullptr) { |
| 391 | LOG_ALWAYS_FATAL_IF(!session->removeIncomingConnection(connection), |
| 392 | "bad state: connection object guaranteed to be in list"); |
| 393 | } |
| 394 | |
Steven Moreland | 659416d | 2021-05-11 00:47:50 +0000 | [diff] [blame] | 395 | session = nullptr; |
| 396 | |
| 397 | if (listener != nullptr) { |
Steven Moreland | 19fc9f7 | 2021-06-10 03:57:30 +0000 | [diff] [blame] | 398 | listener->onSessionIncomingThreadEnded(); |
Steven Moreland | ee78e76 | 2021-05-05 21:12:51 +0000 | [diff] [blame] | 399 | } |
| 400 | } |
| 401 | |
Steven Moreland | 7b8bc4c | 2021-06-10 22:50:27 +0000 | [diff] [blame] | 402 | sp<RpcServer> RpcSession::server() { |
| 403 | RpcServer* unsafeServer = mForServer.unsafe_get(); |
| 404 | sp<RpcServer> server = mForServer.promote(); |
| 405 | |
| 406 | LOG_ALWAYS_FATAL_IF((unsafeServer == nullptr) != (server == nullptr), |
| 407 | "wp<> is to avoid strong cycle only"); |
| 408 | return server; |
Steven Moreland | bdb53ab | 2021-05-05 17:57:41 +0000 | [diff] [blame] | 409 | } |
| 410 | |
Steven Moreland | 826367f | 2021-09-10 14:05:31 -0700 | [diff] [blame^] | 411 | status_t RpcSession::setupClient(const std::function<status_t(const std::vector<uint8_t>& sessionId, |
| 412 | bool incoming)>& connectAndInit) { |
Steven Moreland | bdb53ab | 2021-05-05 17:57:41 +0000 | [diff] [blame] | 413 | { |
| 414 | std::lock_guard<std::mutex> _l(mMutex); |
Steven Moreland | 19fc9f7 | 2021-06-10 03:57:30 +0000 | [diff] [blame] | 415 | LOG_ALWAYS_FATAL_IF(mOutgoingConnections.size() != 0, |
Steven Moreland | bdb53ab | 2021-05-05 17:57:41 +0000 | [diff] [blame] | 416 | "Must only setup session once, but already has %zu clients", |
Steven Moreland | 19fc9f7 | 2021-06-10 03:57:30 +0000 | [diff] [blame] | 417 | mOutgoingConnections.size()); |
Steven Moreland | bdb53ab | 2021-05-05 17:57:41 +0000 | [diff] [blame] | 418 | } |
Yifan Hong | 832521e | 2021-08-05 14:55:40 -0700 | [diff] [blame] | 419 | if (auto status = initShutdownTrigger(); status != OK) return status; |
Steven Moreland | bdb53ab | 2021-05-05 17:57:41 +0000 | [diff] [blame] | 420 | |
Steven Moreland | 826367f | 2021-09-10 14:05:31 -0700 | [diff] [blame^] | 421 | if (status_t status = connectAndInit({}, false /*incoming*/); status != OK) return status; |
Steven Moreland | bdb53ab | 2021-05-05 17:57:41 +0000 | [diff] [blame] | 422 | |
Steven Moreland | bf57bce | 2021-07-26 15:26:12 -0700 | [diff] [blame] | 423 | { |
| 424 | ExclusiveConnection connection; |
Steven Moreland | 2372f9d | 2021-08-05 15:42:01 -0700 | [diff] [blame] | 425 | if (status_t status = ExclusiveConnection::find(sp<RpcSession>::fromExisting(this), |
| 426 | ConnectionUse::CLIENT, &connection); |
| 427 | status != OK) |
| 428 | return status; |
Steven Moreland | bf57bce | 2021-07-26 15:26:12 -0700 | [diff] [blame] | 429 | |
| 430 | uint32_t version; |
Steven Moreland | 2372f9d | 2021-08-05 15:42:01 -0700 | [diff] [blame] | 431 | if (status_t status = |
| 432 | state()->readNewSessionResponse(connection.get(), |
| 433 | sp<RpcSession>::fromExisting(this), &version); |
| 434 | status != OK) |
| 435 | return status; |
| 436 | if (!setProtocolVersion(version)) return BAD_VALUE; |
Steven Moreland | bf57bce | 2021-07-26 15:26:12 -0700 | [diff] [blame] | 437 | } |
| 438 | |
Steven Moreland | a5036f0 | 2021-06-08 02:26:57 +0000 | [diff] [blame] | 439 | // TODO(b/189955605): we should add additional sessions dynamically |
Steven Moreland | bdb53ab | 2021-05-05 17:57:41 +0000 | [diff] [blame] | 440 | // instead of all at once. |
Steven Moreland | bdb53ab | 2021-05-05 17:57:41 +0000 | [diff] [blame] | 441 | size_t numThreadsAvailable; |
Steven Moreland | 1be9135 | 2021-05-11 22:12:15 +0000 | [diff] [blame] | 442 | if (status_t status = getRemoteMaxThreads(&numThreadsAvailable); status != OK) { |
Steven Moreland | 4198a12 | 2021-08-03 17:37:58 -0700 | [diff] [blame] | 443 | ALOGE("Could not get max threads after initial session setup: %s", |
Steven Moreland | bdb53ab | 2021-05-05 17:57:41 +0000 | [diff] [blame] | 444 | statusToString(status).c_str()); |
Steven Moreland | 2372f9d | 2021-08-05 15:42:01 -0700 | [diff] [blame] | 445 | return status; |
Steven Moreland | bdb53ab | 2021-05-05 17:57:41 +0000 | [diff] [blame] | 446 | } |
| 447 | |
| 448 | if (status_t status = readId(); status != OK) { |
Steven Moreland | 4198a12 | 2021-08-03 17:37:58 -0700 | [diff] [blame] | 449 | ALOGE("Could not get session id after initial session setup: %s", |
Steven Moreland | bdb53ab | 2021-05-05 17:57:41 +0000 | [diff] [blame] | 450 | statusToString(status).c_str()); |
Steven Moreland | 2372f9d | 2021-08-05 15:42:01 -0700 | [diff] [blame] | 451 | return status; |
Steven Moreland | bdb53ab | 2021-05-05 17:57:41 +0000 | [diff] [blame] | 452 | } |
| 453 | |
Steven Moreland | a5036f0 | 2021-06-08 02:26:57 +0000 | [diff] [blame] | 454 | // TODO(b/189955605): we should add additional sessions dynamically |
Steven Moreland | 659416d | 2021-05-11 00:47:50 +0000 | [diff] [blame] | 455 | // instead of all at once - the other side should be responsible for setting |
| 456 | // up additional connections. We need to create at least one (unless 0 are |
| 457 | // requested to be set) in order to allow the other side to reliably make |
| 458 | // any requests at all. |
| 459 | |
Steven Moreland | 4198a12 | 2021-08-03 17:37:58 -0700 | [diff] [blame] | 460 | // we've already setup one client |
| 461 | for (size_t i = 0; i + 1 < numThreadsAvailable; i++) { |
Steven Moreland | 826367f | 2021-09-10 14:05:31 -0700 | [diff] [blame^] | 462 | if (status_t status = connectAndInit(mId, false /*incoming*/); status != OK) return status; |
Steven Moreland | 4198a12 | 2021-08-03 17:37:58 -0700 | [diff] [blame] | 463 | } |
| 464 | |
Steven Moreland | 103424e | 2021-06-02 18:16:19 +0000 | [diff] [blame] | 465 | for (size_t i = 0; i < mMaxThreads; i++) { |
Steven Moreland | 826367f | 2021-09-10 14:05:31 -0700 | [diff] [blame^] | 466 | if (status_t status = connectAndInit(mId, true /*incoming*/); status != OK) return status; |
Steven Moreland | bdb53ab | 2021-05-05 17:57:41 +0000 | [diff] [blame] | 467 | } |
| 468 | |
Steven Moreland | 2372f9d | 2021-08-05 15:42:01 -0700 | [diff] [blame] | 469 | return OK; |
Steven Moreland | bdb53ab | 2021-05-05 17:57:41 +0000 | [diff] [blame] | 470 | } |
| 471 | |
Steven Moreland | 2372f9d | 2021-08-05 15:42:01 -0700 | [diff] [blame] | 472 | status_t RpcSession::setupSocketClient(const RpcSocketAddress& addr) { |
Steven Moreland | 826367f | 2021-09-10 14:05:31 -0700 | [diff] [blame^] | 473 | return setupClient([&](const std::vector<uint8_t>& sessionId, bool incoming) { |
Steven Moreland | 4198a12 | 2021-08-03 17:37:58 -0700 | [diff] [blame] | 474 | return setupOneSocketConnection(addr, sessionId, incoming); |
| 475 | }); |
| 476 | } |
| 477 | |
Steven Moreland | 2372f9d | 2021-08-05 15:42:01 -0700 | [diff] [blame] | 478 | status_t RpcSession::setupOneSocketConnection(const RpcSocketAddress& addr, |
Steven Moreland | 826367f | 2021-09-10 14:05:31 -0700 | [diff] [blame^] | 479 | const std::vector<uint8_t>& sessionId, |
| 480 | bool incoming) { |
Steven Moreland | 76d2c1f | 2021-05-05 20:28:58 +0000 | [diff] [blame] | 481 | for (size_t tries = 0; tries < 5; tries++) { |
| 482 | if (tries > 0) usleep(10000); |
| 483 | |
Yifan Hong | b675ffe | 2021-08-05 16:37:17 -0700 | [diff] [blame] | 484 | unique_fd serverFd(TEMP_FAILURE_RETRY( |
| 485 | socket(addr.addr()->sa_family, SOCK_STREAM | SOCK_CLOEXEC | SOCK_NONBLOCK, 0))); |
Steven Moreland | 76d2c1f | 2021-05-05 20:28:58 +0000 | [diff] [blame] | 486 | if (serverFd == -1) { |
| 487 | int savedErrno = errno; |
| 488 | ALOGE("Could not create socket at %s: %s", addr.toString().c_str(), |
| 489 | strerror(savedErrno)); |
Steven Moreland | 2372f9d | 2021-08-05 15:42:01 -0700 | [diff] [blame] | 490 | return -savedErrno; |
Steven Moreland | 76d2c1f | 2021-05-05 20:28:58 +0000 | [diff] [blame] | 491 | } |
| 492 | |
| 493 | if (0 != TEMP_FAILURE_RETRY(connect(serverFd.get(), addr.addr(), addr.addrSize()))) { |
Yifan Hong | 95d15e5 | 2021-08-25 17:15:15 -0700 | [diff] [blame] | 494 | int connErrno = errno; |
| 495 | if (connErrno == EAGAIN || connErrno == EINPROGRESS) { |
| 496 | // For non-blocking sockets, connect() may return EAGAIN (for unix domain socket) or |
| 497 | // EINPROGRESS (for others). Call poll() and getsockopt() to get the error. |
| 498 | status_t pollStatus = mShutdownTrigger->triggerablePoll(serverFd, POLLOUT); |
| 499 | if (pollStatus != OK) { |
| 500 | ALOGE("Could not POLLOUT after connect() on non-blocking socket: %s", |
| 501 | statusToString(pollStatus).c_str()); |
| 502 | return pollStatus; |
| 503 | } |
| 504 | // Set connErrno to the errno that connect() would have set if the fd were blocking. |
| 505 | socklen_t connErrnoLen = sizeof(connErrno); |
| 506 | int ret = |
| 507 | getsockopt(serverFd.get(), SOL_SOCKET, SO_ERROR, &connErrno, &connErrnoLen); |
| 508 | if (ret == -1) { |
| 509 | int savedErrno = errno; |
| 510 | ALOGE("Could not getsockopt() after connect() on non-blocking socket: %s. " |
| 511 | "(Original error from connect() is: %s)", |
| 512 | strerror(savedErrno), strerror(connErrno)); |
| 513 | return -savedErrno; |
| 514 | } |
| 515 | // Retrieved the real connErrno as if connect() was called with a blocking socket |
| 516 | // fd. Continue checking connErrno. |
| 517 | } |
| 518 | if (connErrno == ECONNRESET) { |
Steven Moreland | 76d2c1f | 2021-05-05 20:28:58 +0000 | [diff] [blame] | 519 | ALOGW("Connection reset on %s", addr.toString().c_str()); |
| 520 | continue; |
| 521 | } |
Yifan Hong | 95d15e5 | 2021-08-25 17:15:15 -0700 | [diff] [blame] | 522 | // connErrno could be zero if getsockopt determines so. Hence zero-check again. |
| 523 | if (connErrno != 0) { |
Yifan Hong | d9f8cef | 2021-08-05 15:17:31 -0700 | [diff] [blame] | 524 | ALOGE("Could not connect socket at %s: %s", addr.toString().c_str(), |
Yifan Hong | 95d15e5 | 2021-08-25 17:15:15 -0700 | [diff] [blame] | 525 | strerror(connErrno)); |
| 526 | return -connErrno; |
Yifan Hong | d9f8cef | 2021-08-05 15:17:31 -0700 | [diff] [blame] | 527 | } |
Steven Moreland | 76d2c1f | 2021-05-05 20:28:58 +0000 | [diff] [blame] | 528 | } |
Yifan Hong | 702115c | 2021-06-24 15:39:18 -0700 | [diff] [blame] | 529 | LOG_RPC_DETAIL("Socket at %s client with fd %d", addr.toString().c_str(), serverFd.get()); |
| 530 | |
Steven Moreland | 4198a12 | 2021-08-03 17:37:58 -0700 | [diff] [blame] | 531 | return initAndAddConnection(std::move(serverFd), sessionId, incoming); |
Steven Moreland | bdb53ab | 2021-05-05 17:57:41 +0000 | [diff] [blame] | 532 | } |
| 533 | |
Steven Moreland | 76d2c1f | 2021-05-05 20:28:58 +0000 | [diff] [blame] | 534 | ALOGE("Ran out of retries to connect to %s", addr.toString().c_str()); |
Steven Moreland | 2372f9d | 2021-08-05 15:42:01 -0700 | [diff] [blame] | 535 | return UNKNOWN_ERROR; |
Steven Moreland | bdb53ab | 2021-05-05 17:57:41 +0000 | [diff] [blame] | 536 | } |
| 537 | |
Steven Moreland | 826367f | 2021-09-10 14:05:31 -0700 | [diff] [blame^] | 538 | status_t RpcSession::initAndAddConnection(unique_fd fd, const std::vector<uint8_t>& sessionId, |
Steven Moreland | 2372f9d | 2021-08-05 15:42:01 -0700 | [diff] [blame] | 539 | bool incoming) { |
Yifan Hong | 8c95042 | 2021-08-05 17:13:55 -0700 | [diff] [blame] | 540 | LOG_ALWAYS_FATAL_IF(mShutdownTrigger == nullptr); |
Yifan Hong | ecf937d | 2021-08-11 17:29:28 -0700 | [diff] [blame] | 541 | auto server = mCtx->newTransport(std::move(fd), mShutdownTrigger.get()); |
Steven Moreland | 4198a12 | 2021-08-03 17:37:58 -0700 | [diff] [blame] | 542 | if (server == nullptr) { |
Yifan Hong | ecf937d | 2021-08-11 17:29:28 -0700 | [diff] [blame] | 543 | ALOGE("%s: Unable to set up RpcTransport", __PRETTY_FUNCTION__); |
Steven Moreland | 2372f9d | 2021-08-05 15:42:01 -0700 | [diff] [blame] | 544 | return UNKNOWN_ERROR; |
Steven Moreland | 4198a12 | 2021-08-03 17:37:58 -0700 | [diff] [blame] | 545 | } |
| 546 | |
| 547 | LOG_RPC_DETAIL("Socket at client with RpcTransport %p", server.get()); |
| 548 | |
Steven Moreland | 826367f | 2021-09-10 14:05:31 -0700 | [diff] [blame^] | 549 | if (sessionId.size() > std::numeric_limits<uint16_t>::max()) { |
| 550 | ALOGE("Session ID too big %zu", sessionId.size()); |
| 551 | return BAD_VALUE; |
| 552 | } |
| 553 | |
Steven Moreland | 4198a12 | 2021-08-03 17:37:58 -0700 | [diff] [blame] | 554 | RpcConnectionHeader header{ |
| 555 | .version = mProtocolVersion.value_or(RPC_WIRE_PROTOCOL_VERSION), |
| 556 | .options = 0, |
Steven Moreland | 826367f | 2021-09-10 14:05:31 -0700 | [diff] [blame^] | 557 | .sessionIdSize = static_cast<uint16_t>(sessionId.size()), |
Steven Moreland | 4198a12 | 2021-08-03 17:37:58 -0700 | [diff] [blame] | 558 | }; |
Steven Moreland | 4198a12 | 2021-08-03 17:37:58 -0700 | [diff] [blame] | 559 | |
Steven Moreland | 826367f | 2021-09-10 14:05:31 -0700 | [diff] [blame^] | 560 | if (incoming) { |
| 561 | header.options |= RPC_CONNECTION_OPTION_INCOMING; |
| 562 | } |
Steven Moreland | 4198a12 | 2021-08-03 17:37:58 -0700 | [diff] [blame] | 563 | |
Yifan Hong | 8c95042 | 2021-08-05 17:13:55 -0700 | [diff] [blame] | 564 | auto sendHeaderStatus = |
| 565 | server->interruptableWriteFully(mShutdownTrigger.get(), &header, sizeof(header)); |
| 566 | if (sendHeaderStatus != OK) { |
Steven Moreland | 4198a12 | 2021-08-03 17:37:58 -0700 | [diff] [blame] | 567 | ALOGE("Could not write connection header to socket: %s", |
Yifan Hong | 8c95042 | 2021-08-05 17:13:55 -0700 | [diff] [blame] | 568 | statusToString(sendHeaderStatus).c_str()); |
| 569 | return sendHeaderStatus; |
Steven Moreland | 4198a12 | 2021-08-03 17:37:58 -0700 | [diff] [blame] | 570 | } |
| 571 | |
Steven Moreland | 826367f | 2021-09-10 14:05:31 -0700 | [diff] [blame^] | 572 | if (sessionId.size() > 0) { |
| 573 | auto sendSessionIdStatus = |
| 574 | server->interruptableWriteFully(mShutdownTrigger.get(), sessionId.data(), |
| 575 | sessionId.size()); |
| 576 | if (sendSessionIdStatus != OK) { |
| 577 | ALOGE("Could not write session ID ('%s') to socket: %s", |
| 578 | base::HexString(sessionId.data(), sessionId.size()).c_str(), |
| 579 | statusToString(sendSessionIdStatus).c_str()); |
| 580 | return sendSessionIdStatus; |
| 581 | } |
| 582 | } |
| 583 | |
Steven Moreland | 4198a12 | 2021-08-03 17:37:58 -0700 | [diff] [blame] | 584 | LOG_RPC_DETAIL("Socket at client: header sent"); |
| 585 | |
| 586 | if (incoming) { |
| 587 | return addIncomingConnection(std::move(server)); |
| 588 | } else { |
| 589 | return addOutgoingConnection(std::move(server), true /*init*/); |
| 590 | } |
| 591 | } |
| 592 | |
Steven Moreland | 2372f9d | 2021-08-05 15:42:01 -0700 | [diff] [blame] | 593 | status_t RpcSession::addIncomingConnection(std::unique_ptr<RpcTransport> rpcTransport) { |
Steven Moreland | fba6f77 | 2021-07-15 22:45:09 +0000 | [diff] [blame] | 594 | std::mutex mutex; |
| 595 | std::condition_variable joinCv; |
| 596 | std::unique_lock<std::mutex> lock(mutex); |
| 597 | std::thread thread; |
| 598 | sp<RpcSession> thiz = sp<RpcSession>::fromExisting(this); |
| 599 | bool ownershipTransferred = false; |
| 600 | thread = std::thread([&]() { |
| 601 | std::unique_lock<std::mutex> threadLock(mutex); |
Yifan Hong | 702115c | 2021-06-24 15:39:18 -0700 | [diff] [blame] | 602 | std::unique_ptr<RpcTransport> movedRpcTransport = std::move(rpcTransport); |
Steven Moreland | fba6f77 | 2021-07-15 22:45:09 +0000 | [diff] [blame] | 603 | // NOLINTNEXTLINE(performance-unnecessary-copy-initialization) |
| 604 | sp<RpcSession> session = thiz; |
| 605 | session->preJoinThreadOwnership(std::move(thread)); |
| 606 | |
| 607 | // only continue once we have a response or the connection fails |
Yifan Hong | 702115c | 2021-06-24 15:39:18 -0700 | [diff] [blame] | 608 | auto setupResult = session->preJoinSetup(std::move(movedRpcTransport)); |
Steven Moreland | fba6f77 | 2021-07-15 22:45:09 +0000 | [diff] [blame] | 609 | |
| 610 | ownershipTransferred = true; |
| 611 | threadLock.unlock(); |
| 612 | joinCv.notify_one(); |
| 613 | // do not use & vars below |
| 614 | |
| 615 | RpcSession::join(std::move(session), std::move(setupResult)); |
| 616 | }); |
| 617 | joinCv.wait(lock, [&] { return ownershipTransferred; }); |
| 618 | LOG_ALWAYS_FATAL_IF(!ownershipTransferred); |
Steven Moreland | 2372f9d | 2021-08-05 15:42:01 -0700 | [diff] [blame] | 619 | return OK; |
Steven Moreland | fba6f77 | 2021-07-15 22:45:09 +0000 | [diff] [blame] | 620 | } |
| 621 | |
Yifan Hong | 832521e | 2021-08-05 14:55:40 -0700 | [diff] [blame] | 622 | status_t RpcSession::initShutdownTrigger() { |
| 623 | // first client connection added, but setForServer not called, so |
| 624 | // initializaing for a client. |
| 625 | if (mShutdownTrigger == nullptr) { |
| 626 | mShutdownTrigger = FdTrigger::make(); |
| 627 | mEventListener = mShutdownListener = sp<WaitForShutdownListener>::make(); |
| 628 | if (mShutdownTrigger == nullptr) return INVALID_OPERATION; |
| 629 | } |
| 630 | return OK; |
| 631 | } |
| 632 | |
Steven Moreland | 2372f9d | 2021-08-05 15:42:01 -0700 | [diff] [blame] | 633 | status_t RpcSession::addOutgoingConnection(std::unique_ptr<RpcTransport> rpcTransport, bool init) { |
Steven Moreland | c88b7fc | 2021-06-10 00:40:39 +0000 | [diff] [blame] | 634 | sp<RpcConnection> connection = sp<RpcConnection>::make(); |
| 635 | { |
| 636 | std::lock_guard<std::mutex> _l(mMutex); |
Yifan Hong | 702115c | 2021-06-24 15:39:18 -0700 | [diff] [blame] | 637 | connection->rpcTransport = std::move(rpcTransport); |
Steven Moreland | c88b7fc | 2021-06-10 00:40:39 +0000 | [diff] [blame] | 638 | connection->exclusiveTid = gettid(); |
Steven Moreland | 19fc9f7 | 2021-06-10 03:57:30 +0000 | [diff] [blame] | 639 | mOutgoingConnections.push_back(connection); |
Steven Moreland | ee3f466 | 2021-05-22 01:07:33 +0000 | [diff] [blame] | 640 | } |
| 641 | |
Steven Moreland | b86e26b | 2021-06-12 00:35:58 +0000 | [diff] [blame] | 642 | status_t status = OK; |
| 643 | if (init) { |
| 644 | mState->sendConnectionInit(connection, sp<RpcSession>::fromExisting(this)); |
| 645 | } |
Steven Moreland | c88b7fc | 2021-06-10 00:40:39 +0000 | [diff] [blame] | 646 | |
| 647 | { |
| 648 | std::lock_guard<std::mutex> _l(mMutex); |
| 649 | connection->exclusiveTid = std::nullopt; |
| 650 | } |
| 651 | |
Steven Moreland | 2372f9d | 2021-08-05 15:42:01 -0700 | [diff] [blame] | 652 | return status; |
Steven Moreland | bdb53ab | 2021-05-05 17:57:41 +0000 | [diff] [blame] | 653 | } |
| 654 | |
Steven Moreland | a8b4429 | 2021-06-08 01:27:53 +0000 | [diff] [blame] | 655 | bool RpcSession::setForServer(const wp<RpcServer>& server, const wp<EventListener>& eventListener, |
Steven Moreland | 826367f | 2021-09-10 14:05:31 -0700 | [diff] [blame^] | 656 | const std::vector<uint8_t>& sessionId) { |
Steven Moreland | 659416d | 2021-05-11 00:47:50 +0000 | [diff] [blame] | 657 | LOG_ALWAYS_FATAL_IF(mForServer != nullptr); |
| 658 | LOG_ALWAYS_FATAL_IF(server == nullptr); |
| 659 | LOG_ALWAYS_FATAL_IF(mEventListener != nullptr); |
| 660 | LOG_ALWAYS_FATAL_IF(eventListener == nullptr); |
Steven Moreland | ee3f466 | 2021-05-22 01:07:33 +0000 | [diff] [blame] | 661 | LOG_ALWAYS_FATAL_IF(mShutdownTrigger != nullptr); |
Steven Moreland | a8b4429 | 2021-06-08 01:27:53 +0000 | [diff] [blame] | 662 | |
| 663 | mShutdownTrigger = FdTrigger::make(); |
| 664 | if (mShutdownTrigger == nullptr) return false; |
Steven Moreland | ee3f466 | 2021-05-22 01:07:33 +0000 | [diff] [blame] | 665 | |
Steven Moreland | bdb53ab | 2021-05-05 17:57:41 +0000 | [diff] [blame] | 666 | mId = sessionId; |
| 667 | mForServer = server; |
Steven Moreland | 659416d | 2021-05-11 00:47:50 +0000 | [diff] [blame] | 668 | mEventListener = eventListener; |
Steven Moreland | a8b4429 | 2021-06-08 01:27:53 +0000 | [diff] [blame] | 669 | return true; |
Steven Moreland | bdb53ab | 2021-05-05 17:57:41 +0000 | [diff] [blame] | 670 | } |
| 671 | |
Yifan Hong | 702115c | 2021-06-24 15:39:18 -0700 | [diff] [blame] | 672 | sp<RpcSession::RpcConnection> RpcSession::assignIncomingConnectionToThisThread( |
| 673 | std::unique_ptr<RpcTransport> rpcTransport) { |
Steven Moreland | bdb53ab | 2021-05-05 17:57:41 +0000 | [diff] [blame] | 674 | std::lock_guard<std::mutex> _l(mMutex); |
Steven Moreland | dd67b94 | 2021-07-23 17:15:41 -0700 | [diff] [blame] | 675 | |
Steven Moreland | 132d5bf | 2021-08-03 16:13:24 -0700 | [diff] [blame] | 676 | if (mIncomingConnections.size() >= mMaxThreads) { |
| 677 | ALOGE("Cannot add thread to session with %zu threads (max is set to %zu)", |
| 678 | mIncomingConnections.size(), mMaxThreads); |
| 679 | return nullptr; |
| 680 | } |
| 681 | |
Steven Moreland | dd67b94 | 2021-07-23 17:15:41 -0700 | [diff] [blame] | 682 | // Don't accept any more connections, some have shutdown. Usually this |
| 683 | // happens when new connections are still being established as part of a |
| 684 | // very short-lived session which shuts down after it already started |
| 685 | // accepting new connections. |
| 686 | if (mIncomingConnections.size() < mMaxIncomingConnections) { |
| 687 | return nullptr; |
| 688 | } |
| 689 | |
Steven Moreland | bdb53ab | 2021-05-05 17:57:41 +0000 | [diff] [blame] | 690 | sp<RpcConnection> session = sp<RpcConnection>::make(); |
Yifan Hong | 702115c | 2021-06-24 15:39:18 -0700 | [diff] [blame] | 691 | session->rpcTransport = std::move(rpcTransport); |
Steven Moreland | bdb53ab | 2021-05-05 17:57:41 +0000 | [diff] [blame] | 692 | session->exclusiveTid = gettid(); |
Steven Moreland | dd67b94 | 2021-07-23 17:15:41 -0700 | [diff] [blame] | 693 | |
Steven Moreland | 19fc9f7 | 2021-06-10 03:57:30 +0000 | [diff] [blame] | 694 | mIncomingConnections.push_back(session); |
Steven Moreland | dd67b94 | 2021-07-23 17:15:41 -0700 | [diff] [blame] | 695 | mMaxIncomingConnections = mIncomingConnections.size(); |
Steven Moreland | bdb53ab | 2021-05-05 17:57:41 +0000 | [diff] [blame] | 696 | |
| 697 | return session; |
| 698 | } |
| 699 | |
Steven Moreland | 19fc9f7 | 2021-06-10 03:57:30 +0000 | [diff] [blame] | 700 | bool RpcSession::removeIncomingConnection(const sp<RpcConnection>& connection) { |
Steven Moreland | dd67b94 | 2021-07-23 17:15:41 -0700 | [diff] [blame] | 701 | std::unique_lock<std::mutex> _l(mMutex); |
Steven Moreland | 19fc9f7 | 2021-06-10 03:57:30 +0000 | [diff] [blame] | 702 | if (auto it = std::find(mIncomingConnections.begin(), mIncomingConnections.end(), connection); |
| 703 | it != mIncomingConnections.end()) { |
| 704 | mIncomingConnections.erase(it); |
| 705 | if (mIncomingConnections.size() == 0) { |
Steven Moreland | 659416d | 2021-05-11 00:47:50 +0000 | [diff] [blame] | 706 | sp<EventListener> listener = mEventListener.promote(); |
| 707 | if (listener) { |
Steven Moreland | dd67b94 | 2021-07-23 17:15:41 -0700 | [diff] [blame] | 708 | _l.unlock(); |
| 709 | listener->onSessionAllIncomingThreadsEnded(sp<RpcSession>::fromExisting(this)); |
Steven Moreland | a86e8fe | 2021-05-26 22:52:35 +0000 | [diff] [blame] | 710 | } |
Steven Moreland | ee78e76 | 2021-05-05 21:12:51 +0000 | [diff] [blame] | 711 | } |
Steven Moreland | bdb53ab | 2021-05-05 17:57:41 +0000 | [diff] [blame] | 712 | return true; |
| 713 | } |
| 714 | return false; |
| 715 | } |
| 716 | |
Yifan Hong | ecf937d | 2021-08-11 17:29:28 -0700 | [diff] [blame] | 717 | std::string RpcSession::getCertificate(CertificateFormat format) { |
| 718 | return mCtx->getCertificate(format); |
| 719 | } |
| 720 | |
Steven Moreland | 195edb8 | 2021-06-08 02:44:39 +0000 | [diff] [blame] | 721 | status_t RpcSession::ExclusiveConnection::find(const sp<RpcSession>& session, ConnectionUse use, |
| 722 | ExclusiveConnection* connection) { |
| 723 | connection->mSession = session; |
| 724 | connection->mConnection = nullptr; |
| 725 | connection->mReentrant = false; |
Steven Moreland | bdb53ab | 2021-05-05 17:57:41 +0000 | [diff] [blame] | 726 | |
Steven Moreland | 195edb8 | 2021-06-08 02:44:39 +0000 | [diff] [blame] | 727 | pid_t tid = gettid(); |
| 728 | std::unique_lock<std::mutex> _l(session->mMutex); |
| 729 | |
| 730 | session->mWaitingThreads++; |
Steven Moreland | bdb53ab | 2021-05-05 17:57:41 +0000 | [diff] [blame] | 731 | while (true) { |
| 732 | sp<RpcConnection> exclusive; |
| 733 | sp<RpcConnection> available; |
| 734 | |
| 735 | // CHECK FOR DEDICATED CLIENT SOCKET |
| 736 | // |
Steven Moreland | 85e067b | 2021-05-26 17:43:53 +0000 | [diff] [blame] | 737 | // A server/looper should always use a dedicated connection if available |
Steven Moreland | 19fc9f7 | 2021-06-10 03:57:30 +0000 | [diff] [blame] | 738 | findConnection(tid, &exclusive, &available, session->mOutgoingConnections, |
| 739 | session->mOutgoingConnectionsOffset); |
Steven Moreland | bdb53ab | 2021-05-05 17:57:41 +0000 | [diff] [blame] | 740 | |
| 741 | // WARNING: this assumes a server cannot request its client to send |
Steven Moreland | 19fc9f7 | 2021-06-10 03:57:30 +0000 | [diff] [blame] | 742 | // a transaction, as mIncomingConnections is excluded below. |
Steven Moreland | bdb53ab | 2021-05-05 17:57:41 +0000 | [diff] [blame] | 743 | // |
| 744 | // Imagine we have more than one thread in play, and a single thread |
| 745 | // sends a synchronous, then an asynchronous command. Imagine the |
| 746 | // asynchronous command is sent on the first client connection. Then, if |
| 747 | // we naively send a synchronous command to that same connection, the |
| 748 | // thread on the far side might be busy processing the asynchronous |
| 749 | // command. So, we move to considering the second available thread |
| 750 | // for subsequent calls. |
| 751 | if (use == ConnectionUse::CLIENT_ASYNC && (exclusive != nullptr || available != nullptr)) { |
Steven Moreland | 19fc9f7 | 2021-06-10 03:57:30 +0000 | [diff] [blame] | 752 | session->mOutgoingConnectionsOffset = (session->mOutgoingConnectionsOffset + 1) % |
| 753 | session->mOutgoingConnections.size(); |
Steven Moreland | bdb53ab | 2021-05-05 17:57:41 +0000 | [diff] [blame] | 754 | } |
| 755 | |
Steven Moreland | c7d4013 | 2021-06-10 03:42:11 +0000 | [diff] [blame] | 756 | // USE SERVING SOCKET (e.g. nested transaction) |
Steven Moreland | bdb53ab | 2021-05-05 17:57:41 +0000 | [diff] [blame] | 757 | if (use != ConnectionUse::CLIENT_ASYNC) { |
Steven Moreland | 19fc9f7 | 2021-06-10 03:57:30 +0000 | [diff] [blame] | 758 | sp<RpcConnection> exclusiveIncoming; |
Steven Moreland | bdb53ab | 2021-05-05 17:57:41 +0000 | [diff] [blame] | 759 | // server connections are always assigned to a thread |
Steven Moreland | 19fc9f7 | 2021-06-10 03:57:30 +0000 | [diff] [blame] | 760 | findConnection(tid, &exclusiveIncoming, nullptr /*available*/, |
| 761 | session->mIncomingConnections, 0 /* index hint */); |
Steven Moreland | c7d4013 | 2021-06-10 03:42:11 +0000 | [diff] [blame] | 762 | |
| 763 | // asynchronous calls cannot be nested, we currently allow ref count |
| 764 | // calls to be nested (so that you can use this without having extra |
| 765 | // threads). Note 'drainCommands' is used so that these ref counts can't |
| 766 | // build up. |
Steven Moreland | 19fc9f7 | 2021-06-10 03:57:30 +0000 | [diff] [blame] | 767 | if (exclusiveIncoming != nullptr) { |
| 768 | if (exclusiveIncoming->allowNested) { |
Steven Moreland | c7d4013 | 2021-06-10 03:42:11 +0000 | [diff] [blame] | 769 | // guaranteed to be processed as nested command |
Steven Moreland | 19fc9f7 | 2021-06-10 03:57:30 +0000 | [diff] [blame] | 770 | exclusive = exclusiveIncoming; |
Steven Moreland | c7d4013 | 2021-06-10 03:42:11 +0000 | [diff] [blame] | 771 | } else if (use == ConnectionUse::CLIENT_REFCOUNT && available == nullptr) { |
| 772 | // prefer available socket, but if we don't have one, don't |
| 773 | // wait for one |
Steven Moreland | 19fc9f7 | 2021-06-10 03:57:30 +0000 | [diff] [blame] | 774 | exclusive = exclusiveIncoming; |
Steven Moreland | c7d4013 | 2021-06-10 03:42:11 +0000 | [diff] [blame] | 775 | } |
| 776 | } |
Steven Moreland | bdb53ab | 2021-05-05 17:57:41 +0000 | [diff] [blame] | 777 | } |
| 778 | |
Steven Moreland | 85e067b | 2021-05-26 17:43:53 +0000 | [diff] [blame] | 779 | // if our thread is already using a connection, prioritize using that |
Steven Moreland | bdb53ab | 2021-05-05 17:57:41 +0000 | [diff] [blame] | 780 | if (exclusive != nullptr) { |
Steven Moreland | 195edb8 | 2021-06-08 02:44:39 +0000 | [diff] [blame] | 781 | connection->mConnection = exclusive; |
| 782 | connection->mReentrant = true; |
Steven Moreland | bdb53ab | 2021-05-05 17:57:41 +0000 | [diff] [blame] | 783 | break; |
| 784 | } else if (available != nullptr) { |
Steven Moreland | 195edb8 | 2021-06-08 02:44:39 +0000 | [diff] [blame] | 785 | connection->mConnection = available; |
| 786 | connection->mConnection->exclusiveTid = tid; |
Steven Moreland | bdb53ab | 2021-05-05 17:57:41 +0000 | [diff] [blame] | 787 | break; |
| 788 | } |
| 789 | |
Steven Moreland | 19fc9f7 | 2021-06-10 03:57:30 +0000 | [diff] [blame] | 790 | if (session->mOutgoingConnections.size() == 0) { |
Steven Moreland | 195edb8 | 2021-06-08 02:44:39 +0000 | [diff] [blame] | 791 | ALOGE("Session has no client connections. This is required for an RPC server to make " |
| 792 | "any non-nested (e.g. oneway or on another thread) calls. Use: %d. Server " |
| 793 | "connections: %zu", |
Steven Moreland | 19fc9f7 | 2021-06-10 03:57:30 +0000 | [diff] [blame] | 794 | static_cast<int>(use), session->mIncomingConnections.size()); |
Steven Moreland | 195edb8 | 2021-06-08 02:44:39 +0000 | [diff] [blame] | 795 | return WOULD_BLOCK; |
| 796 | } |
Steven Moreland | bdb53ab | 2021-05-05 17:57:41 +0000 | [diff] [blame] | 797 | |
Steven Moreland | 85e067b | 2021-05-26 17:43:53 +0000 | [diff] [blame] | 798 | LOG_RPC_DETAIL("No available connections (have %zu clients and %zu servers). Waiting...", |
Steven Moreland | 19fc9f7 | 2021-06-10 03:57:30 +0000 | [diff] [blame] | 799 | session->mOutgoingConnections.size(), session->mIncomingConnections.size()); |
Steven Moreland | 195edb8 | 2021-06-08 02:44:39 +0000 | [diff] [blame] | 800 | session->mAvailableConnectionCv.wait(_l); |
Steven Moreland | bdb53ab | 2021-05-05 17:57:41 +0000 | [diff] [blame] | 801 | } |
Steven Moreland | 195edb8 | 2021-06-08 02:44:39 +0000 | [diff] [blame] | 802 | session->mWaitingThreads--; |
| 803 | |
| 804 | return OK; |
Steven Moreland | bdb53ab | 2021-05-05 17:57:41 +0000 | [diff] [blame] | 805 | } |
| 806 | |
| 807 | void RpcSession::ExclusiveConnection::findConnection(pid_t tid, sp<RpcConnection>* exclusive, |
| 808 | sp<RpcConnection>* available, |
| 809 | std::vector<sp<RpcConnection>>& sockets, |
| 810 | size_t socketsIndexHint) { |
| 811 | LOG_ALWAYS_FATAL_IF(sockets.size() > 0 && socketsIndexHint >= sockets.size(), |
| 812 | "Bad index %zu >= %zu", socketsIndexHint, sockets.size()); |
| 813 | |
| 814 | if (*exclusive != nullptr) return; // consistent with break below |
| 815 | |
| 816 | for (size_t i = 0; i < sockets.size(); i++) { |
| 817 | sp<RpcConnection>& socket = sockets[(i + socketsIndexHint) % sockets.size()]; |
| 818 | |
Steven Moreland | 85e067b | 2021-05-26 17:43:53 +0000 | [diff] [blame] | 819 | // take first available connection (intuition = caching) |
Steven Moreland | bdb53ab | 2021-05-05 17:57:41 +0000 | [diff] [blame] | 820 | if (available && *available == nullptr && socket->exclusiveTid == std::nullopt) { |
| 821 | *available = socket; |
| 822 | continue; |
| 823 | } |
| 824 | |
Steven Moreland | 85e067b | 2021-05-26 17:43:53 +0000 | [diff] [blame] | 825 | // though, prefer to take connection which is already inuse by this thread |
Steven Moreland | bdb53ab | 2021-05-05 17:57:41 +0000 | [diff] [blame] | 826 | // (nested transactions) |
| 827 | if (exclusive && socket->exclusiveTid == tid) { |
| 828 | *exclusive = socket; |
| 829 | break; // consistent with return above |
| 830 | } |
| 831 | } |
| 832 | } |
| 833 | |
| 834 | RpcSession::ExclusiveConnection::~ExclusiveConnection() { |
Steven Moreland | 85e067b | 2021-05-26 17:43:53 +0000 | [diff] [blame] | 835 | // reentrant use of a connection means something less deep in the call stack |
Steven Moreland | bdb53ab | 2021-05-05 17:57:41 +0000 | [diff] [blame] | 836 | // is using this fd, and it retains the right to it. So, we don't give up |
| 837 | // exclusive ownership, and no thread is freed. |
Steven Moreland | 195edb8 | 2021-06-08 02:44:39 +0000 | [diff] [blame] | 838 | if (!mReentrant && mConnection != nullptr) { |
Steven Moreland | bdb53ab | 2021-05-05 17:57:41 +0000 | [diff] [blame] | 839 | std::unique_lock<std::mutex> _l(mSession->mMutex); |
| 840 | mConnection->exclusiveTid = std::nullopt; |
| 841 | if (mSession->mWaitingThreads > 0) { |
| 842 | _l.unlock(); |
| 843 | mSession->mAvailableConnectionCv.notify_one(); |
| 844 | } |
| 845 | } |
| 846 | } |
| 847 | |
| 848 | } // namespace android |