c++ - gRPC async_client 中的内存泄漏

Question

我正在以与 example 类似的方式使用 gRPC async client .

在这个例子中(发布在gRPC官方github)客户端为要发送的消息分配内存，使用地址作为标签对于 completion queue，当消息在监听器线程中被应答时，内存(由 tag- 地址已知)是空闲的。

我担心服务器没有响应消息并且内存永远不会空闲的情况。

• gRPC 会保护我免受这种情况的影响吗？
• 我应该以不同的方式实现它吗？ (使用智能指针/将指针保存在数据结构中/等等...)

异步客户端发送函数

void SayHello(const std::string& user) {
    // Data we are sending to the server.
    HelloRequest request;
    request.set_name(user);

    // Call object to store rpc data
    AsyncClientCall* call = new AsyncClientCall;

    // Because we are using the asynchronous API, we need to hold on to
    // the "call" instance in order to get updates on the ongoing RPC.
    call->response_reader =
        stub_->PrepareAsyncSayHello(&call->context, request, &cq_);

    // StartCall initiates the RPC call
    call->response_reader->StartCall();

    call->response_reader->Finish(&call->reply, &call->status, (void*)call);

}

线程的异步客户端接收函数

void AsyncCompleteRpc() {
    void* got_tag;
    bool ok = false;

    // Block until the next result is available in the completion queue "cq".
    while (cq_.Next(&got_tag, &ok)) {
        // The tag in this example is the memory location of the call object
        AsyncClientCall* call = static_cast<AsyncClientCall*>(got_tag);

        // Verify that the request was completed successfully. Note that "ok"
        // corresponds solely to the request for updates introduced by Finish().
        GPR_ASSERT(ok);

        if (call->status.ok())
            std::cout << "Greeter received: " << call->reply.message() << std::endl;
        else
            std::cout << "RPC failed" << std::endl;

        // Once we're complete, deallocate the call object.
        delete call;
    }
}

主要

int main(int argc, char** argv) {


    GreeterClient greeter(grpc::CreateChannel(
            "localhost:50051", grpc::InsecureChannelCredentials()));

    // Spawn reader thread that loops indefinitely
    std::thread thread_ = std::thread(&GreeterClient::AsyncCompleteRpc, &greeter);

    for (int i = 0; i < 100; i++) {
        std::string user("world " + std::to_string(i));
        greeter.SayHello(user);  // The actual RPC call!
    }

    std::cout << "Press control-c to quit" << std::endl << std::endl;
    thread_.join();  //blocks forever

    return 0;
}

Answer 1

Does the gRPC protect me from this situation?

有点。 gRPC 保证所有排队的操作迟早会在它们匹配的完成队列中结束。所以你的代码是可以的，只要:

• 在不幸的时候不会抛出异常。
• 您不会更改创建不包括排队操作或删除调用的代码路径的代码。

换句话说:没问题，但很脆弱。

选项 A:

如果你想变得真正健壮，方法是std::shared_ptr<> .但是，它们可能会以意想不到的方式影响多线程性能。因此，它是否值得取决于您的应用在性能与稳健性范围内的位置。

这样的重构看起来像:

1. 有AsyncClientCall继承自 std::enable_shared_from_this
2. 更改 call 的构造至 std::make_shared<AsyncClientCall>()
3. 在完成队列处理程序中，增加引用计数:

while (cq_.Next(&got_tag, &ok)) {
    auto call = static_cast<AsyncClientCall*>(got_tag)->shared_from_this();

并去掉 delete , 显然。

选项 B:

您还可以使用 unique_ptr<> 获得一个不错的中途测量值:

    auto call = std::make_unique<AsyncClientCall>();
    ...
    call->response_reader->Finish(&call->reply, &call->status, (void*)call.release());

和

    std::unique_ptr<AsyncClientCall> call{static_cast<AsyncClientCall*>(got_tag)};

这可以在维护其他所有内容的同时防止重构和异常。但是，这仅适用于产生单个完成事件的一元 rpc。流式 rpc 或交换元数据的 rpc 将需要完全不同的处理方式。