| Message ID | cover.1573050133.git.dekelp@mellanox.com (mailing list archive) |
|---|---|
| Headers |
Return-Path: <dev-bounces@dpdk.org> X-Original-To: patchwork@inbox.dpdk.org Delivered-To: patchwork@inbox.dpdk.org Received: from dpdk.org (dpdk.org [92.243.14.124]) by inbox.dpdk.org (Postfix) with ESMTP id 35E27A04AB; Wed, 6 Nov 2019 15:28:29 +0100 (CET) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id F17A61C213; Wed, 6 Nov 2019 15:28:28 +0100 (CET) Received: from mellanox.co.il (mail-il-dmz.mellanox.com [193.47.165.129]) by dpdk.org (Postfix) with ESMTP id 7D5A21C210 for <dev@dpdk.org>; Wed, 6 Nov 2019 15:28:27 +0100 (CET) Received: from Internal Mail-Server by MTLPINE1 (envelope-from dekelp@mellanox.com) with ESMTPS (AES256-SHA encrypted); 6 Nov 2019 16:28:23 +0200 Received: from mtl-vdi-280.wap.labs.mlnx. (mtl-vdi-280.wap.labs.mlnx [10.128.130.87]) by labmailer.mlnx (8.13.8/8.13.8) with ESMTP id xA6ESNjq014921; Wed, 6 Nov 2019 16:28:23 +0200 From: Dekel Peled <dekelp@mellanox.com> To: john.mcnamara@intel.com, marko.kovacevic@intel.com, nhorman@tuxdriver.com, ajit.khaparde@broadcom.com, somnath.kotur@broadcom.com, anatoly.burakov@intel.com, xuanziyang2@huawei.com, cloud.wangxiaoyun@huawei.com, zhouguoyang@huawei.com, wenzhuo.lu@intel.com, konstantin.ananyev@intel.com, matan@mellanox.com, shahafs@mellanox.com, viacheslavo@mellanox.com, rmody@marvell.com, shshaikh@marvell.com, maxime.coquelin@redhat.com, tiwei.bie@intel.com, zhihong.wang@intel.com, yongwang@vmware.com, thomas@monjalon.net, ferruh.yigit@intel.com, arybchenko@solarflare.com, jingjing.wu@intel.com, bernard.iremonger@intel.com Cc: dev@dpdk.org Date: Wed, 6 Nov 2019 16:28:15 +0200 Message-Id: <cover.1573050133.git.dekelp@mellanox.com> X-Mailer: git-send-email 1.7.1 In-Reply-To: <cover.1573039465.git.dekelp@mellanox.com> References: <cover.1573039465.git.dekelp@mellanox.com> Subject: [dpdk-dev] [PATCH v3 0/3] support API to set max LRO packet size X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: DPDK patches and discussions <dev.dpdk.org> List-Unsubscribe: <https://mails.dpdk.org/options/dev>, <mailto:dev-request@dpdk.org?subject=unsubscribe> List-Archive: <http://mails.dpdk.org/archives/dev/> List-Post: <mailto:dev@dpdk.org> List-Help: <mailto:dev-request@dpdk.org?subject=help> List-Subscribe: <https://mails.dpdk.org/listinfo/dev>, <mailto:dev-request@dpdk.org?subject=subscribe> Errors-To: dev-bounces@dpdk.org Sender: "dev" <dev-bounces@dpdk.org> |
| Series |
support API to set max LRO packet size
|
|
Message
Dekel Peled
Nov. 6, 2019, 2:28 p.m. UTC
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. 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(-)
Comments
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. > > 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.
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.