[ofa-general] [PATCH] IB/CM: Remove local write permission enable in QP access flags

[Steve Wise]

Why are you changing this?

I think we set it for a specific reason (but I don't remember why just 
now)...


Sean Hefty wrote:
> Dotan Barak wrote:
>> Remove local write permission enable in QP access flags
>> (this attribute is being used only for remote permissions).
>>
>> Signed-off-by: Dotan Barak <dotanb at mellanox.co.il>
> 
> Acked-by: Sean Hefty <sean.hefty at intel.com>
> 
> Steve, does this look okay to you?
> 
>>
>> ---
>>
>> diff --git a/drivers/infiniband/core/cma.c 
>> b/drivers/infiniband/core/cma.c
>> index 23af7a0..9ffb998 100644
>> --- a/drivers/infiniband/core/cma.c
>> +++ b/drivers/infiniband/core/cma.c
>> @@ -573,7 +573,7 @@ int rdma_init_qp_attr(struct rdma_cm_id *id, 
>> struct ib_qp_attr *qp_attr,
>>          break;
>>      case RDMA_TRANSPORT_IWARP:
>>          if (!id_priv->cm_id.iw) {
>> -            qp_attr->qp_access_flags = IB_ACCESS_LOCAL_WRITE;
>> +            qp_attr->qp_access_flags = 0;
>>              *qp_attr_mask = IB_QP_STATE | IB_QP_ACCESS_FLAGS;
>>          } else
>>              ret = iw_cm_init_qp_attr(id_priv->cm_id.iw, qp_attr,
>> _______________________________________________
>> general mailing list
>> general at lists.openfabrics.org
>> http://lists.openfabrics.org/cgi-bin/mailman/listinfo/general
>>
>> To unsubscribe, please visit 
>> http://openib.org/mailman/listinfo/openib-general
>>