You may commit it IMO. 

> -----Original Message-----
> From: [email protected] 
> [mailto:[email protected]] On Behalf Of Sean Hefty
> Sent: Friday, August 07, 2009 9:43 PM
> To: [email protected]
> Subject: [ofw] [PATCH] ibal/cm: allow RDMA writes for QP 
> attributes bydefault
> 
> Enable remote RDMA writes as the default QP attributes for 
> INIT transitions.  Previously, RDMA writes were only enabled 
> if the CM messages carried a responder resources > 0.  
> However, responder resources indicates whether RDMA 
> reads/atomics should be enabled, not RDMA writes.
> 
> Without this patch, users of winverbs that connect in-band 
> that want to perform RDMA writes without performing RDMA 
> reads will see QP errors when performing writes.
> 
> Signed-off-by: Sean Hefty <[email protected]>
> ---
> This patch needs to be included with WinOF 2.1.
> 
> diff -up -r -X \mshefty\scm\winof\trunk\docs\dontdiff.txt -I 
> '\$Id:' trunk\core\al/kernel/al_cm_cep.c 
> branches\winverbs\core\al/kernel/al_cm_cep.c
> --- trunk\core\al/kernel/al_cm_cep.c  2009-06-11 
> 10:32:59.346125000 -0700
> +++ branches\winverbs\core\al/kernel/al_cm_cep.c      
> 2009-08-07 11:21:30.867045100 -0700
> @@ -6198,12 +6198,10 @@ al_cep_get_init_attr(
>                       p_cep->av[p_cep->idx_primary].attr.port_num;
>               p_init->state.init.pkey_index =
>                       p_cep->av[p_cep->idx_primary].pkey_index;
> -             p_init->state.init.access_ctrl = IB_AC_LOCAL_WRITE;
> +             p_init->state.init.access_ctrl = IB_AC_LOCAL_WRITE | 
> +IB_AC_RDMA_WRITE;
>               if ( p_cep->resp_res )
>               {
> -                     p_init->state.init.access_ctrl |= 
> IB_AC_RDMA_READ |
> -                                                             
>                         IB_AC_RDMA_WRITE |
> -                                                             
>                         IB_AC_ATOMIC;
> +                     p_init->state.init.access_ctrl |= 
> IB_AC_RDMA_READ | IB_AC_ATOMIC;
>               }
>               status = IB_SUCCESS;
>               break;
> 
> 
> _______________________________________________
> ofw mailing list
> [email protected]
> http://lists.openfabrics.org/cgi-bin/mailman/listinfo/ofw
> 
_______________________________________________
ofw mailing list
[email protected]
http://lists.openfabrics.org/cgi-bin/mailman/listinfo/ofw

Reply via email to