[ofa-general] Re: [PATCH] - [resend] Corrects a race in ipoib_cm_post_receive_nonsrq()

[Hoang-Nam Nguyen]

Thanks, you're much faster than me:-)
Nam

Roland Dreier <rdreier at cisco.com> wrote on 30.07.2008 16:22:51:

> thanks, I'll queue up this patch:
>
> commit e08198169ec5facb3d85bb455efa44a2f8327842
> Author: Roland Dreier <rolandd at cisco.com>
> Date:   Wed Jul 30 07:21:46 2008 -0700
>
>     IPoIB/cm: Set correct SG list in ipoib_cm_init_rx_wr()
>
>     wr->sg_list should be set to the sge pointer passed in, not
>     priv->cm.rx_sge.
>
>     Reported-by: Hoang-Nam Nguyen <HNGUYEN at de.ibm.com>
>     Signed-off-by: Roland Dreier <rolandd at cisco.com>
>
> diff --git a/drivers/infiniband/ulp/ipoib/ipoib_cm.c
> b/drivers/infiniband/ulp/ipoib/ipoib_cm.c
> index 0f2d304..7ebc400 100644
> --- a/drivers/infiniband/ulp/ipoib/ipoib_cm.c
> +++ b/drivers/infiniband/ulp/ipoib/ipoib_cm.c
> @@ -337,7 +337,7 @@ static void ipoib_cm_init_rx_wr(struct net_device
*dev,
>        sge[i].length = PAGE_SIZE;
>
>     wr->next    = NULL;
> -   wr->sg_list = priv->cm.rx_sge;
> +   wr->sg_list = sge;
>     wr->num_sge = priv->cm.num_frags;
>  }
>