diff options
| author | Chuck Lever <chuck.lever@oracle.com> | 2026-06-01 20:54:13 +0300 |
|---|---|---|
| committer | Anna Schumaker <anna.schumaker@hammerspace.com> | 2026-06-08 17:21:56 +0300 |
| commit | bf822d717edecfbf2e6b3eec1bef3813cbeacdd4 (patch) | |
| tree | 505e5373464b8a32d24356473155e7c22c754711 | |
| parent | 04dd9cdbe59a9056b4cc8de07e1585db77661cbc (diff) | |
| download | linux-bf822d717edecfbf2e6b3eec1bef3813cbeacdd4.tar.xz | |
xprtrdma: Remove tautological I2 assertion in rpcrdma_reply_put
rpcrdma_reply_put() sets req->rl_reply to NULL when it is
non-NULL, and skips the block when it is already NULL. The
WARN_ON_ONCE(req->rl_reply) that follows can never fire
because both paths leave rl_reply NULL.
Remove the dead assertion and its comment.
Signed-off-by: Chuck Lever <chuck.lever@oracle.com>
Signed-off-by: Anna Schumaker <anna.schumaker@hammerspace.com>
| -rw-r--r-- | net/sunrpc/xprtrdma/verbs.c | 4 |
1 files changed, 0 insertions, 4 deletions
diff --git a/net/sunrpc/xprtrdma/verbs.c b/net/sunrpc/xprtrdma/verbs.c index da2c6fa44154..92c691d2521f 100644 --- a/net/sunrpc/xprtrdma/verbs.c +++ b/net/sunrpc/xprtrdma/verbs.c @@ -1259,10 +1259,6 @@ void rpcrdma_reply_put(struct rpcrdma_buffer *buffers, struct rpcrdma_req *req) req->rl_reply = NULL; rpcrdma_rep_put(buffers, rep); } - /* I2: rl_reply NULL after the put closes the - * 'rep on rb_free_reps still referenced by req' window. - */ - WARN_ON_ONCE(req->rl_reply); } /** |