[dpdk-dev] [PATCH v2 6/6] app/testpmd: add mempool flags parameter

[Ye Xiaolong]

On 03/20, David Marchand wrote:
>On Wed, Mar 20, 2019 at 12:37 AM Jerin Jacob Kollanukkaran <
>jerinj at marvell.com> wrote:
>
>> On Tue, 2019-03-19 at 15:12 +0800, Xiaolong Ye wrote:
>> > When create rte_mempool, flags can be parsed from command line.
>> > Now, it is possible for testpmd to create a af_xdp friendly
>> > mempool (which enable zero copy).
>> >
>> > Signed-off-by: Qi Zhang <qi.z.zhang at intel.com>
>> > Signed-off-by: Xiaolong Ye <xiaolong.ye at intel.com>
>> > ---
>> >  app/test-pmd/parameters.c             | 12 ++++++++++++
>> >  app/test-pmd/testpmd.c                | 17 ++++++++++-------
>> >  app/test-pmd/testpmd.h                |  1 +
>> >  doc/guides/testpmd_app_ug/run_app.rst |  4 ++++
>>
>> If I understand it correctly, The user needs to change all the
>> application in order to avail zero copy feature of XDP.
>>
>> If so,
>>
>> How about creating wrapper mempool driver for xdp at drivers/mempool/?
>> and mempool's best mempool feature to select the required mempool
>> driver for XDP at runtime without changing the apps.
>>
>> see rte_mbuf_best_mempool_ops()
>> see struct eth_dev_ops::pool_ops_supported
>>
>
>Glab to read this, I was under the same impression :-)

We plan to separate this in another patchset and may target for next release.

Thanks,
Xiaolong
>
>
>-- 
>David Marchand