From mboxrd@z Thu Jan  1 00:00:00 1970
From: Greg KH <greg@kroah.com>
Subject: Re: [PATCH 2/3] iw_cxgb4: drop listen destroy replies if no ep found
Date: Fri, 15 Sep 2017 09:46:27 -0700
Message-ID: <20170915164627.GC27343@kroah.com>
References: <20170915140752.A29BEE0C1D@smtp.ogc.us>
Mime-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Return-path: <stable-owner@vger.kernel.org>
Content-Disposition: inline
In-Reply-To: <20170915140752.A29BEE0C1D@smtp.ogc.us>
Sender: stable-owner@vger.kernel.org
To: Steve Wise <swise@opengridcomputing.com>
Cc: dledford@redhat.com, linux-rdma@vger.kernel.org, stable@vger.kernel.org
List-Id: linux-rdma@vger.kernel.org

On Tue, Sep 05, 2017 at 11:52:33AM -0700, Steve Wise wrote:
> If the thread waiting for a CLOSE_LISTSRV_RPL times out and bails,
> then we need to handle a subsequent CPL if it arrives and the stid has
> been released.  In this case silently drop it.
> 
> Signed-off-by: Steve Wise <swise@opengridcomputing.com>
> ---
>  drivers/infiniband/hw/cxgb4/cm.c | 5 +++++
>  1 file changed, 5 insertions(+)
> 
> diff --git a/drivers/infiniband/hw/cxgb4/cm.c b/drivers/infiniband/hw/cxgb4/cm.c
> index 6d85a5d..d76bfe3 100644
> --- a/drivers/infiniband/hw/cxgb4/cm.c
> +++ b/drivers/infiniband/hw/cxgb4/cm.c
> @@ -2333,9 +2333,14 @@ static int close_listsrv_rpl(struct c4iw_dev *dev, struct sk_buff *skb)
>  	unsigned int stid = GET_TID(rpl);
>  	struct c4iw_listen_ep *ep = get_ep_from_stid(dev, stid);
>  
> +	if (!ep) {
> +		pr_debug("%s stid %d lookup failure!\n", __func__, stid);
> +		goto out;
> +	}
>  	pr_debug("%s ep %p\n", __func__, ep);
>  	c4iw_wake_up(&ep->com.wr_wait, status2errno(rpl->status));
>  	c4iw_put_ep(&ep->com);
> +out:
>  	return 0;
>  }
>  
> -- 
> 1.8.3.1

<formletter>

This is not the correct way to submit patches for inclusion in the
stable kernel tree.  Please read:
    https://www.kernel.org/doc/html/latest/process/stable-kernel-rules.html
for how to do this properly.

</formletter>