| Message ID | 20211123082515.65956-2-tonylu@linux.alibaba.com (mailing list archive) |
|---|---|
| State | Accepted |
| Commit | 45c3ff7a9ac195135536057021c1d3ac664f3f62 |
| Delegated to: | Netdev Maintainers |
| Headers | show |
| Series | Fixes for closing process and minor cleanup | expand |
On 23/11/2021 09:25, Tony Lu wrote: > There remains some variables to replace with local struct sock. So clean > them up all. > > Fixes: 3163c5071f25 ("net/smc: use local struct sock variables consistently") > Signed-off-by: Tony Lu <tonylu@linux.alibaba.com> > Reviewed-by: Wen Gu <guwen@linux.alibaba.com> > --- This is a rather cosmetic change, I will pick it up for our next submission to the net-next tree. Thank you.
diff --git a/net/smc/smc_close.c b/net/smc/smc_close.c index 0f9ffba07d26..9b235fbb089a 100644 --- a/net/smc/smc_close.c +++ b/net/smc/smc_close.c @@ -354,9 +354,9 @@ static void smc_close_passive_work(struct work_struct *work) if (rxflags->peer_conn_abort) { /* peer has not received all data */ smc_close_passive_abort_received(smc); - release_sock(&smc->sk); + release_sock(sk); cancel_delayed_work_sync(&conn->tx_work); - lock_sock(&smc->sk); + lock_sock(sk); goto wakeup; }
There remains some variables to replace with local struct sock. So clean them up all. Fixes: 3163c5071f25 ("net/smc: use local struct sock variables consistently") Signed-off-by: Tony Lu <tonylu@linux.alibaba.com> Reviewed-by: Wen Gu <guwen@linux.alibaba.com> --- net/smc/smc_close.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)