Re: [PATCH net-next v3 08/13] net/rds: hold connection references in lookup, sockets and c_passive
From: sashiko-bot@kernel.org
Date: 2026-09-15 03:37:39
From: Sharath Srinivasan <redacted> Hand out real references everywhere a struct rds_connection pointer previously escaped bare: - rds_conn_lookup() takes a reference on the connection it returns (kref_get_unless_zero(), so that it only ever hands out a live reference), and __rds_conn_create() returns the connection with a reference held for the caller on every path: lookup hit, fresh creation, lost creation race, and the passive-loopback lookup, which now also holds the parent while it dereferences parent->c_passive. - The rs->rs_conn sendmsg cache owns a reference, which is dropped when the cache is replaced or the socket is released. [ ... ] them synchronously before the final put), and the sendmsg cache is serialized with rs_lock instead of UEK's socket flag; rewrite commit message] Assisted-by: Claude-Code:claude-fable-5 Signed-off-by: Allison Henderson <achender@kernel.org>
Sashiko has reviewed this patch and found no issues. It looks great! -- Sashiko AI review · https://sashiko.dev/#/patchset/20260914033719.138057-1-achender@kernel.org?part=8