From: Geliang Tang <tanggeli...@kylinos.cn>

The variable "pm" has been defined in mptcp_pm_fully_established()
and mptcp_pm_data_reset() as "msk->pm", so use "pm" directly instead
of using "msk->pm".

Signed-off-by: Geliang Tang <tanggeli...@kylinos.cn>
Reviewed-by: Matthieu Baerts (NGI0) <matt...@kernel.org>
Signed-off-by: Matthieu Baerts (NGI0) <matt...@kernel.org>
---
 net/mptcp/pm.c | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/net/mptcp/pm.c b/net/mptcp/pm.c
index 
8e6a325a389803196c35175cd5ea7637624d0ee2..04a156395aaddf50e67d10479086591a37063fa3
 100644
--- a/net/mptcp/pm.c
+++ b/net/mptcp/pm.c
@@ -511,13 +511,13 @@ void mptcp_pm_fully_established(struct mptcp_sock *msk, 
const struct sock *ssk)
         * be sure to serve this event only once.
         */
        if (READ_ONCE(pm->work_pending) &&
-           !(msk->pm.status & BIT(MPTCP_PM_ALREADY_ESTABLISHED)))
+           !(pm->status & BIT(MPTCP_PM_ALREADY_ESTABLISHED)))
                mptcp_pm_schedule_work(msk, MPTCP_PM_ESTABLISHED);
 
-       if ((msk->pm.status & BIT(MPTCP_PM_ALREADY_ESTABLISHED)) == 0)
+       if ((pm->status & BIT(MPTCP_PM_ALREADY_ESTABLISHED)) == 0)
                announce = true;
 
-       msk->pm.status |= BIT(MPTCP_PM_ALREADY_ESTABLISHED);
+       pm->status |= BIT(MPTCP_PM_ALREADY_ESTABLISHED);
        spin_unlock_bh(&pm->lock);
 
        if (announce)
@@ -1009,7 +1009,7 @@ void mptcp_pm_data_reset(struct mptcp_sock *msk)
        WRITE_ONCE(pm->addr_signal, 0);
        WRITE_ONCE(pm->remote_deny_join_id0, false);
        pm->status = 0;
-       bitmap_fill(msk->pm.id_avail_bitmap, MPTCP_PM_MAX_ADDR_ID + 1);
+       bitmap_fill(pm->id_avail_bitmap, MPTCP_PM_MAX_ADDR_ID + 1);
 }
 
 void mptcp_pm_data_init(struct mptcp_sock *msk)

-- 
2.48.1


Reply via email to