IB/cm: Drop lockdep assert and WARN when freeing old msg

JIRA: https://issues.redhat.com/browse/RHEL-110100

commit 7590649
Author: Vlad Dumitrescu <vdumitrescu@nvidia.com>
Date:   Mon Apr 28 14:30:18 2025 +0300

    IB/cm: Drop lockdep assert and WARN when freeing old msg

    The send completion handler can run after cm_id has advanced to another
    message.  The cm_id lock is not needed in this case, but a recent change
    re-used cm_free_priv_msg(), which asserts that the lock is held and
    WARNs if the cm_id's currently outstanding msg is different than the one
    being freed.

    Fixes: 1e51592 ("IB/cm: Do not hold reference on cm_id unless needed")
    Signed-off-by: Vlad Dumitrescu <vdumitrescu@nvidia.com>
    Reviewed-by: Sean Hefty <shefty@nvidia.com>
    Link: https://patch.msgid.link/0c364c29142f72b7875fdeba51f3c9bd6ca863ee.1745839788.git.leon@kernel.org
    Signed-off-by: Leon Romanovsky <leon@kernel.org>

Signed-off-by: Kamal Heib <kheib@redhat.com>

Author: Kamalheib

drivers/infiniband/core/cm.c

@@ -3786,7 +3786,8 @@ static void cm_process_send_error(struct cm_id_private *cm_id_priv,
 	spin_lock_irq(&cm_id_priv->lock);
 	if (msg != cm_id_priv->msg) {
 		spin_unlock_irq(&cm_id_priv->lock);
-		cm_free_priv_msg(msg);
+		cm_free_msg(msg);
+		cm_deref_id(cm_id_priv);
 		return;
 	}
 	cm_free_priv_msg(msg);