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

Ferruh Yigit ferruh.yigit at intel.com
Fri Mar 31 11:23:44 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
> 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.

Can you please send a new version of the patch with suggested updates?
(Keeping the Ack)


> Thanks,
> Yongseok

More information about the dev mailing list