Re: [dpdk-dev] [PATCH v3 0/3] support API to set max LRO packet size

[Dekel Peled <dekelp@mellanox.com>]

Hi Bernard, PSB.

> -----Original Message-----
> From: Iremonger, Bernard <bernard.iremonger@intel.com>
> Sent: Wednesday, November 6, 2019 6:41 PM
> To: Dekel Peled <dekelp@mellanox.com>; Mcnamara, John
> <john.mcnamara@intel.com>; Kovacevic, Marko
> <marko.kovacevic@intel.com>; nhorman@tuxdriver.com;
> ajit.khaparde@broadcom.com; somnath.kotur@broadcom.com; Burakov,
> Anatoly <anatoly.burakov@intel.com>; xuanziyang2@huawei.com;
> cloud.wangxiaoyun@huawei.com; zhouguoyang@huawei.com; Lu, Wenzhuo
> <wenzhuo.lu@intel.com>; Ananyev, Konstantin
> <konstantin.ananyev@intel.com>; Matan Azrad <matan@mellanox.com>;
> Shahaf Shuler <shahafs@mellanox.com>; Slava Ovsiienko
> <viacheslavo@mellanox.com>; rmody@marvell.com;
> shshaikh@marvell.com; maxime.coquelin@redhat.com; Bie, Tiwei
> <tiwei.bie@intel.com>; Wang, Zhihong <zhihong.wang@intel.com>;
> yongwang@vmware.com; Thomas Monjalon <thomas@monjalon.net>; Yigit,
> Ferruh <ferruh.yigit@intel.com>; arybchenko@solarflare.com; Wu, Jingjing
> <jingjing.wu@intel.com>
> Cc: dev@dpdk.org
> Subject: RE: [PATCH v3 0/3] support API to set max LRO packet size
> 
> Hi Dekel,
> 
> > -----Original Message-----
> > From: Dekel Peled <dekelp@mellanox.com>
> > Sent: Wednesday, November 6, 2019 2:28 PM
> > To: Mcnamara, John <john.mcnamara@intel.com>; Kovacevic, Marko
> > <marko.kovacevic@intel.com>; nhorman@tuxdriver.com;
> > ajit.khaparde@broadcom.com; somnath.kotur@broadcom.com; Burakov,
> > Anatoly <anatoly.burakov@intel.com>; xuanziyang2@huawei.com;
> > cloud.wangxiaoyun@huawei.com; zhouguoyang@huawei.com; Lu,
> Wenzhuo
> > <wenzhuo.lu@intel.com>; Ananyev, Konstantin
> > <konstantin.ananyev@intel.com>; matan@mellanox.com;
> > shahafs@mellanox.com; viacheslavo@mellanox.com;
> rmody@marvell.com;
> > shshaikh@marvell.com; maxime.coquelin@redhat.com; Bie, Tiwei
> > <tiwei.bie@intel.com>; Wang, Zhihong <zhihong.wang@intel.com>;
> > yongwang@vmware.com; thomas@monjalon.net; Yigit, Ferruh
> > <ferruh.yigit@intel.com>; arybchenko@solarflare.com; Wu, Jingjing
> > <jingjing.wu@intel.com>; Iremonger, Bernard
> > <bernard.iremonger@intel.com>
> > Cc: dev@dpdk.org
> > Subject: [PATCH v3 0/3] support API to set max LRO packet size
> >
> > This series implements support and use of API for configuration and
> > validation of max size for LRO aggregated packet.
> >
> > v2: Updated ethdev patch per review comments.
> > v3: Updated ethdev and testpmd patches per review comments.
> 
> My comments on the v2 testpmd patch have not been addressed in the v3
> patch.

I accepted your comment about update of function usage(), and added it in v3.
I replied your other comments in v2 reply email, and didn't see additional response.

Regards,
Dekel

> 
> >
> > Dekel Peled (3):
> >   ethdev: support API to set max LRO packet size
> >   net/mlx5: use API to set max LRO packet size
> >   app/testpmd: use API to set max LRO packet size
> >
> >  app/test-pmd/cmdline.c                      | 73
> > +++++++++++++++++++++++++++++
> >  app/test-pmd/config.c                       |  2 +
> >  app/test-pmd/parameters.c                   |  7 +++
> >  app/test-pmd/testpmd.c                      |  1 +
> >  doc/guides/nics/features.rst                |  2 +
> >  doc/guides/nics/mlx5.rst                    |  2 +
> >  doc/guides/rel_notes/deprecation.rst        |  4 --
> >  doc/guides/rel_notes/release_19_11.rst      |  8 ++++
> >  doc/guides/testpmd_app_ug/run_app.rst       |  5 ++
> >  doc/guides/testpmd_app_ug/testpmd_funcs.rst |  9 ++++
> >  drivers/net/bnxt/bnxt_ethdev.c              |  1 +
> >  drivers/net/hinic/hinic_pmd_ethdev.c        |  1 +
> >  drivers/net/ixgbe/ixgbe_ethdev.c            |  2 +
> >  drivers/net/ixgbe/ixgbe_vf_representor.c    |  1 +
> >  drivers/net/mlx5/mlx5.h                     |  3 ++
> >  drivers/net/mlx5/mlx5_ethdev.c              |  1 +
> >  drivers/net/mlx5/mlx5_rxq.c                 |  5 +-
> >  drivers/net/qede/qede_ethdev.c              |  1 +
> >  drivers/net/virtio/virtio_ethdev.c          |  1 +
> >  drivers/net/vmxnet3/vmxnet3_ethdev.c        |  1 +
> >  lib/librte_ethdev/rte_ethdev.c              | 44 +++++++++++++++++
> >  lib/librte_ethdev/rte_ethdev.h              |  4 ++
> >  22 files changed, 172 insertions(+), 6 deletions(-)
> >
> > --
> > 1.8.3.1
> 
> Regards,
> 
> Bernard.