Re: [PATCH] RDMA: update outdated references to hfi1_destroy_qp()

[Leon Romanovsky <leon@kernel.org>]

On Sat, Mar 21, 2026 at 06:58:51PM +0800, Kexin Sun wrote:
> The function hfi1_destroy_qp() was removed in commit
> 75261cc6ab66 ("staging/rdma/hfi1: Remove destroy qp verb") in
> favor of the rdmavt generic rvt_destroy_qp().  Two comments still
> reference hfi1_destroy_qp() as the waiter that rvt_put_qp() will
> wake up; the actual waiter is now rvt_destroy_qp().  Update both
> references.
> 
> Assisted-by: unnamed:deepseek-v3.2 coccinelle
> Signed-off-by: Kexin Sun <kexinsun@smail.nju.edu.cn>
> ---
>  drivers/infiniband/hw/hfi1/qp.c      | 2 +-
>  drivers/infiniband/sw/rdmavt/mcast.c | 2 +-
>  2 files changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/drivers/infiniband/hw/hfi1/qp.c b/drivers/infiniband/hw/hfi1/qp.c
> index f3d8c0c193ac..bbb23c0386ee 100644
> --- a/drivers/infiniband/hw/hfi1/qp.c
> +++ b/drivers/infiniband/hw/hfi1/qp.c
> @@ -404,7 +404,7 @@ void hfi1_qp_wakeup(struct rvt_qp *qp, u32 flag)
>  		hfi1_qp_schedule(qp);
>  	}
>  	spin_unlock_irqrestore(&qp->s_lock, flags);
> -	/* Notify hfi1_destroy_qp() if it is waiting. */
> +	/* Notify rvt_destroy_qp() if it is waiting. */

Just remove these comments, they have no value.

Thanks

>  	rvt_put_qp(qp);
>  }
>  
> diff --git a/drivers/infiniband/sw/rdmavt/mcast.c b/drivers/infiniband/sw/rdmavt/mcast.c
> index 1fda344d2056..c75be2c53b2d 100644
> --- a/drivers/infiniband/sw/rdmavt/mcast.c
> +++ b/drivers/infiniband/sw/rdmavt/mcast.c
> @@ -49,7 +49,7 @@ static void rvt_mcast_qp_free(struct rvt_mcast_qp *mqp)
>  {
>  	struct rvt_qp *qp = mqp->qp;
>  
> -	/* Notify hfi1_destroy_qp() if it is waiting. */
> +	/* Notify rvt_destroy_qp() if it is waiting. */
>  	rvt_put_qp(qp);
>  
>  	kfree(mqp);
> -- 
> 2.25.1
>