Re: [PATCH] IB/ipoib: Fix CONFIG_INFINIBAND_IPOIB_CM

[Doug Ledford <dledford-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>]

[-- Attachment #1: Type: text/plain, Size: 1375 bytes --]

On 07/22/2015 04:30 PM, Jason Gunthorpe wrote:
> If the above is turned off then ipoib_cm_dev_init unconditionally
> returns ENOSYS, and the newly added error handling in
> 0b3957 prevents ipoib from coming up at all:
> 
> kernel: mlx4_0: ipoib_transport_dev_init failed
> kernel: mlx4_0: failed to initialize port 1 (ret = -12)
> 
> Fixes: 0b39578bcde4 (IB/ipoib: Use dedicated workqueues per interface)
> Signed-off-by: Jason Gunthorpe <jgunthorpe-ePGOBjL8dl3ta4EC/59zMFaTQe2KTcn/@public.gmane.org>
> ---
>  drivers/infiniband/ulp/ipoib/ipoib_verbs.c | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/drivers/infiniband/ulp/ipoib/ipoib_verbs.c b/drivers/infiniband/ulp/ipoib/ipoib_verbs.c
> index 9e6ee82a8fd7..851c8219d501 100644
> --- a/drivers/infiniband/ulp/ipoib/ipoib_verbs.c
> +++ b/drivers/infiniband/ulp/ipoib/ipoib_verbs.c
> @@ -177,7 +177,8 @@ int ipoib_transport_dev_init(struct net_device *dev, struct ib_device *ca)
>  		else
>  			size += ipoib_recvq_size * ipoib_max_conn_qp;
>  	} else
> -		goto out_free_wq;
> +		if (ret != -ENOSYS)
> +			goto out_free_wq;
>  
>  	cq_attr.cqe = size;
>  	priv->recv_cq = ib_create_cq(priv->ca, ipoib_ib_completion, NULL,
> 

Thanks, applied for 4.2-rc.

-- 
Doug Ledford <dledford-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
              GPG KeyID: 0E572FDD



[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 884 bytes --]