[dpdk-dev] [PATCH] net/mlx5: remove not needed query if and rd from create qp
ferruh.yigit at intel.com
Thu Mar 30 19:08:28 CEST 2017
On 3/30/2017 6:03 PM, Yongseok Koh wrote:
>> 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.
More information about the dev