Re: [PATCH rdma-next 01/13] RDMA: Add EFA related definitions

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 05-Dec-18 21:23, Hefty, Sean wrote:
>>  enum {
>> @@ -119,14 +120,16 @@ enum rdma_transport_type {
>>  	RDMA_TRANSPORT_IB,
>>  	RDMA_TRANSPORT_IWARP,
>>  	RDMA_TRANSPORT_USNIC,
>> -	RDMA_TRANSPORT_USNIC_UDP
>> +	RDMA_TRANSPORT_USNIC_UDP,
>> +	RDMA_TRANSPORT_EFA,
>>  };
>>
>>  enum rdma_protocol_type {
>>  	RDMA_PROTOCOL_IB,
>>  	RDMA_PROTOCOL_IBOE,
>>  	RDMA_PROTOCOL_IWARP,
>> -	RDMA_PROTOCOL_USNIC_UDP
>> +	RDMA_PROTOCOL_USNIC_UDP,
>> +	RDMA_PROTOCOL_EFA,
> 
> EFA is the (marketing?) name of the NIC, not really the transport or protocol.  You called the protocol SRD in the cover letter.  I'm not sure if that would apply as both the transport or protocol, but it seems a better option than EFA.

We support both SRD and UD, we consider EFA as a family of protocols.

> 
> - Sean
> 



[Index of Archives]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Photo]     [Yosemite News]     [Yosemite Photos]     [Linux Kernel]     [Linux SCSI]     [XFree86]

  Powered by Linux