diff options
| author | Yacan Liu <liuyacan@corp.netease.com> | 2022-08-30 23:23:14 +0800 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2022-09-08 11:11:37 +0200 |
| commit | e416fe7f16c19f223adccd2ca50d2113e57f23e1 (patch) | |
| tree | 1a62dc6ae9b8153cb29ebc4295da7c0384dc73af | |
| parent | d73b89c3b3f751681d927bac8cf7c2e7b93c8d84 (diff) | |
net/smc: Remove redundant refcount increase
[ Upstream commit a8424a9b4522a3ab9f32175ad6d848739079071f ]
For passive connections, the refcount increment has been done in
smc_clcsock_accept()-->smc_sock_alloc().
Fixes: 3b2dec2603d5 ("net/smc: restructure client and server code in af_smc")
Signed-off-by: Yacan Liu <liuyacan@corp.netease.com>
Reviewed-by: Tony Lu <tonylu@linux.alibaba.com>
Link: https://lore.kernel.org/r/20220830152314.838736-1-liuyacan@corp.netease.com
Signed-off-by: Paolo Abeni <pabeni@redhat.com>
Signed-off-by: Sasha Levin <sashal@kernel.org>
| -rw-r--r-- | net/smc/af_smc.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/net/smc/af_smc.c b/net/smc/af_smc.c index 5d7710dd95145..41cbc7c89c9d2 100644 --- a/net/smc/af_smc.c +++ b/net/smc/af_smc.c @@ -1325,7 +1325,6 @@ static void smc_listen_out_connected(struct smc_sock *new_smc) { struct sock *newsmcsk = &new_smc->sk; - sk_refcnt_debug_inc(newsmcsk); if (newsmcsk->sk_state == SMC_INIT) newsmcsk->sk_state = SMC_ACTIVE; |