From f92d6d64b1836571e9e4c2323d223c9098b27f76 Mon Sep 17 00:00:00 2001 From: Jihyeon Gim Date: Sat, 5 Sep 2026 21:59:05 +0900 Subject: [PATCH] rdma: carry the session id in the reaped session record The session id only existed as the sessions map key; the session record itself kept an empty id string, so the terminal reap record logged an empty id for every expired, cancelled, or destroyed session. Copy the id into the record at creation time so teardown logs identify the session they describe. --- cuwrapper/rc/rc_server_abi.cpp | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/cuwrapper/rc/rc_server_abi.cpp b/cuwrapper/rc/rc_server_abi.cpp index da2f1938..71e32c2a 100644 --- a/cuwrapper/rc/rc_server_abi.cpp +++ b/cuwrapper/rc/rc_server_abi.cpp @@ -613,6 +613,10 @@ int rc_prepare(rc_server *srv, const rc_prepare_req *req, srv->opts.t_prep_ms ? hipObj::v2::clockSource().nowMs() + srv->opts.t_prep_ms : 0; + /* The session record carries its own id copy: reap logging and the + * terminal teardown record read core.id, while the map key is the + * only other place the id lives. */ + rs.core.id = id; { std::lock_guard g(srv->map_mtx); rs.staging_buf = reinterpret_cast(buf);