[dpdk-dev] [PATCH] net/mlx5: remove not needed query if and rd from create qp

Ferruh Yigit ferruh.yigit at intel.com
Thu Mar 30 19:08:28 CEST 2017


On 3/30/2017 6:03 PM, Yongseok Koh wrote:
> 
> Hi,
> 
>> On Mar 30, 2017, at 1:06 AM, Shahaf Shuler <shahafs at mellanox.com> wrote:
>>
>> Since mlx5 PMD data path is on top of PRM, such verbs calls are
>> no longer needed.
>>
>> Signed-off-by: Shahaf Shuler <shahafs at mellanox.com>
>> Acked-by: Nelio Laranjeiro <nelio.laranjeiro at 6wind.com>
> 
> Acked-by: Yongseok Koh <yskoh at mellanox.com>
> 
> I ack the code change but one small suggestion regarding the commit header.
> It looks like 'if' means interface, 'rd' means the Resource Domain, and
> 'qp' means the Queue Pair. which are all mellanox specific terms and
> acronyms. My suggestion is:
> 
> net/mlx5: remove unnecessary Verbs library calls

Thanks for the suggestion, I was thinking about asking for clarification
on patch subject :)

> 
> And how about moving the details into the message body:
> 
> Remove unnecessary interface queries and the Resource Domain when creating
> the Queue Pair. Since mlx5 PMD data path is on top of native APIs, such
> Verbs library calls are no longer needed.
> 
> Thanks,
> Yongseok
> 



More information about the dev mailing list