Message ID | 20210413035046.28578-1-lizh@nvidia.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 mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by inbox.dpdk.org (Postfix) with ESMTP id BC728A0524; Tue, 13 Apr 2021 05:50:52 +0200 (CEST) Received: from [217.70.189.124] (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 8AECB160A2C; Tue, 13 Apr 2021 05:50:52 +0200 (CEST) Received: from mellanox.co.il (mail-il-dmz.mellanox.com [193.47.165.129]) by mails.dpdk.org (Postfix) with ESMTP id 7F893160A2A for <dev@dpdk.org>; Tue, 13 Apr 2021 05:50:50 +0200 (CEST) Received: from Internal Mail-Server by MTLPINE1 (envelope-from lizh@nvidia.com) with SMTP; 13 Apr 2021 06:50:49 +0300 Received: from nvidia.com (c-135-185-1-009.mtl.labs.mlnx [10.135.185.9]) by labmailer.mlnx (8.13.8/8.13.8) with ESMTP id 13D3oniJ027261; Tue, 13 Apr 2021 06:50:49 +0300 From: Li Zhang <lizh@nvidia.com> To: dekelp@nvidia.com, orika@nvidia.com, viacheslavo@nvidia.com, matan@nvidia.com, shahafs@nvidia.com, cristian.dumitrescu@intel.com, lironh@marvell.com, jerinj@marvell.com, ferruh.yigit@intel.com Cc: dev@dpdk.org, thomas@monjalon.net, rasland@nvidia.com, roniba@nvidia.com Date: Tue, 13 Apr 2021 06:50:41 +0300 Message-Id: <20210413035046.28578-1-lizh@nvidia.com> X-Mailer: git-send-email 2.21.0 In-Reply-To: <20210331085405.1445546-1-lizh@nvidia.com> References: <20210331085405.1445546-1-lizh@nvidia.com> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Subject: [dpdk-dev] [PATCH v4 0/4] Support PPS(packet per second) on meter X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.29 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 PPS(packet per second) on meter
|
|
Message
Li Zhang
April 13, 2021, 3:50 a.m. UTC
Currently meter algorithms only supports rate is bytes per second(BPS). Add packet_mode flag in meter profile parameters data structure. So that it can meter traffic by packet per second. When packet_mode is 0, the profile rates and bucket sizes are specified in bytes per second and bytes when packet_mode is not 0, the profile rates and bucket sizes are specified in packets and packets per second. Add the necessary checks to the existing drivers implementing the rte_mtr API to makes sure that profiles with packet_mode set to TRUE are rejected. RFC ("adds support PPS(packet per second) on meter") https://patchwork.dpdk.org/project/dpdk/patch/20210125012023.1769769-2-lizh@nvidia.com/ Depends-on: series=16301 ("Support meter policy API ") https://patchwork.dpdk.org/project/dpdk/list/?series=16301 V2: create a unified patch that contains both the series with the API changes and the series with the necessary error checks in the drivers. V3: Fix comments about commit-log. V4: Fix comments about Depends-on and rebase. Li Zhang (4): ethdev: add packet mode in meter profile structure app/testpmd: add meter profile packet mode option net/softnic: check meter packet mode net/mvpp2: check meter packet mode app/test-pmd/cmdline_mtr.c | 40 ++++++++- doc/guides/rel_notes/release_21_05.rst | 12 +++ doc/guides/testpmd_app_ug/testpmd_funcs.rst | 31 +++---- drivers/net/mvpp2/mrvl_mtr.c | 6 ++ drivers/net/softnic/rte_eth_softnic_meter.c | 8 ++ lib/librte_ethdev/rte_mtr.h | 90 ++++++++++++++++++--- 6 files changed, 159 insertions(+), 28 deletions(-)
Comments
On 4/13/2021 4:50 AM, Li Zhang wrote: > Currently meter algorithms only supports rate is bytes per second(BPS). > Add packet_mode flag in meter profile parameters data structure. > So that it can meter traffic by packet per second. > > When packet_mode is 0, the profile rates and bucket sizes are > specified in bytes per second and bytes > when packet_mode is not 0, the profile rates and bucket sizes are > specified in packets and packets per second. > > Add the necessary checks to the existing drivers implementing > the rte_mtr API to makes sure that profiles with > packet_mode set to TRUE are rejected. > > RFC ("adds support PPS(packet per second) on meter") > https://patchwork.dpdk.org/project/dpdk/patch/20210125012023.1769769-2-lizh@nvidia.com/ > > Depends-on: series=16301 ("Support meter policy API ") > https://patchwork.dpdk.org/project/dpdk/list/?series=16301 > Hi Li, I am not clear with the dependency chain, can you please clarify, 1) Is this set depends to series-16301? Because it compiles fine after conflict resolved, I can see in your repo there is an order, but if there is no functional/logical dependency you can set this patch exactly on top of HEAD (removing the series-16301 in between), so the CI will be enabled. 2) According its cover letter series-16301 depends on mlx ASO patch, this makes all ethdev patches dependent to mlx5 set, I guess that is wrong, can you please confirm? Above (1) is more important, since series-16301 not fully acked, it is blocking me to proceed. > V2: create a unified patch that contains both the series with > the API changes and the series with the necessary error checks in the drivers. > V3: Fix comments about commit-log. > V4: Fix comments about Depends-on and rebase. > > Li Zhang (4): > ethdev: add packet mode in meter profile structure > app/testpmd: add meter profile packet mode option > net/softnic: check meter packet mode > net/mvpp2: check meter packet mode > > app/test-pmd/cmdline_mtr.c | 40 ++++++++- > doc/guides/rel_notes/release_21_05.rst | 12 +++ > doc/guides/testpmd_app_ug/testpmd_funcs.rst | 31 +++---- > drivers/net/mvpp2/mrvl_mtr.c | 6 ++ > drivers/net/softnic/rte_eth_softnic_meter.c | 8 ++ > lib/librte_ethdev/rte_mtr.h | 90 ++++++++++++++++++--- > 6 files changed, 159 insertions(+), 28 deletions(-) >
Hi Ferruh, > -----Original Message----- > From: Ferruh Yigit <ferruh.yigit@intel.com> > Sent: Tuesday, April 13, 2021 6:25 PM > To: Li Zhang <lizh@nvidia.com>; dekelp@nvidia.com; Ori Kam > <orika@nvidia.com>; Slava Ovsiienko <viacheslavo@nvidia.com>; Matan > Azrad <matan@nvidia.com>; Shahaf Shuler <shahafs@nvidia.com>; > cristian.dumitrescu@intel.com; lironh@marvell.com; jerinj@marvell.com > Cc: dev@dpdk.org; NBU-Contact-Thomas Monjalon <thomas@monjalon.net>; > Raslan Darawsheh <rasland@nvidia.com>; Roni Bar Yanai > <roniba@nvidia.com> > Subject: Re: [PATCH v4 0/4] Support PPS(packet per second) on meter > > External email: Use caution opening links or attachments > > > On 4/13/2021 4:50 AM, Li Zhang wrote: > > Currently meter algorithms only supports rate is bytes per second(BPS). > > Add packet_mode flag in meter profile parameters data structure. > > So that it can meter traffic by packet per second. > > > > When packet_mode is 0, the profile rates and bucket sizes are > > specified in bytes per second and bytes when packet_mode is not 0, the > > profile rates and bucket sizes are specified in packets and packets > > per second. > > > > Add the necessary checks to the existing drivers implementing the > > rte_mtr API to makes sure that profiles with packet_mode set to TRUE > > are rejected. > > > > RFC ("adds support PPS(packet per second) on meter") > > https://patchwork.dpdk.org/project/dpdk/patch/20210125012023.1769769- > 2 > > -lizh@nvidia.com/ > > > > Depends-on: series=16301 ("Support meter policy API ") > > https://patchwork.dpdk.org/project/dpdk/list/?series=16301 > > > > Hi Li, > > I am not clear with the dependency chain, can you please clarify, > > 1) Is this set depends to series-16301? Because it compiles fine after conflict > resolved, I can see in your repo there is an order, but if there is no > functional/logical dependency you can set this patch exactly on top of HEAD > (removing the series-16301 in between), so the CI will be enabled. I will delete series-16301. But it will merge conflict when series-16301 merged after it. > 2) According its cover letter series-16301 depends on mlx ASO patch, this > makes all ethdev patches dependent to mlx5 set, I guess that is wrong, can you > please confirm? > > Above (1) is more important, since series-16301 not fully acked, it is blocking > me to proceed. > > > V2: create a unified patch that contains both the series with > > the API changes and the series with the necessary error checks in the > drivers. > > V3: Fix comments about commit-log. > > V4: Fix comments about Depends-on and rebase. > > > > Li Zhang (4): > > ethdev: add packet mode in meter profile structure > > app/testpmd: add meter profile packet mode option > > net/softnic: check meter packet mode > > net/mvpp2: check meter packet mode > > > > app/test-pmd/cmdline_mtr.c | 40 ++++++++- > > doc/guides/rel_notes/release_21_05.rst | 12 +++ > > doc/guides/testpmd_app_ug/testpmd_funcs.rst | 31 +++---- > > drivers/net/mvpp2/mrvl_mtr.c | 6 ++ > > drivers/net/softnic/rte_eth_softnic_meter.c | 8 ++ > > lib/librte_ethdev/rte_mtr.h | 90 ++++++++++++++++++--- > > 6 files changed, 159 insertions(+), 28 deletions(-) > >
On 4/13/2021 12:02 PM, Li Zhang wrote: > Hi Ferruh, > >> -----Original Message----- >> From: Ferruh Yigit <ferruh.yigit@intel.com> >> Sent: Tuesday, April 13, 2021 6:25 PM >> To: Li Zhang <lizh@nvidia.com>; dekelp@nvidia.com; Ori Kam >> <orika@nvidia.com>; Slava Ovsiienko <viacheslavo@nvidia.com>; Matan >> Azrad <matan@nvidia.com>; Shahaf Shuler <shahafs@nvidia.com>; >> cristian.dumitrescu@intel.com; lironh@marvell.com; jerinj@marvell.com >> Cc: dev@dpdk.org; NBU-Contact-Thomas Monjalon <thomas@monjalon.net>; >> Raslan Darawsheh <rasland@nvidia.com>; Roni Bar Yanai >> <roniba@nvidia.com> >> Subject: Re: [PATCH v4 0/4] Support PPS(packet per second) on meter >> >> External email: Use caution opening links or attachments >> >> >> On 4/13/2021 4:50 AM, Li Zhang wrote: >>> Currently meter algorithms only supports rate is bytes per second(BPS). >>> Add packet_mode flag in meter profile parameters data structure. >>> So that it can meter traffic by packet per second. >>> >>> When packet_mode is 0, the profile rates and bucket sizes are >>> specified in bytes per second and bytes when packet_mode is not 0, the >>> profile rates and bucket sizes are specified in packets and packets >>> per second. >>> >>> Add the necessary checks to the existing drivers implementing the >>> rte_mtr API to makes sure that profiles with packet_mode set to TRUE >>> are rejected. >>> >>> RFC ("adds support PPS(packet per second) on meter") >>> https://patchwork.dpdk.org/project/dpdk/patch/20210125012023.1769769- >> 2 >>> -lizh@nvidia.com/ >>> >>> Depends-on: series=16301 ("Support meter policy API ") >>> https://patchwork.dpdk.org/project/dpdk/list/?series=16301 >>> >> >> Hi Li, >> >> I am not clear with the dependency chain, can you please clarify, >> >> 1) Is this set depends to series-16301? Because it compiles fine after conflict >> resolved, I can see in your repo there is an order, but if there is no >> functional/logical dependency you can set this patch exactly on top of HEAD >> (removing the series-16301 in between), so the CI will be enabled. > > I will delete series-16301. > But it will merge conflict when series-16301 merged after it. > Please send both this patch, and series-16301 on top of latest head, this enables CI for both. When merging them we can handle the conflict, based on which one merged first, or can ask you to rebase the second one but for this case it does not look too complex to resolve ourselves. >> 2) According its cover letter series-16301 depends on mlx ASO patch, this >> makes all ethdev patches dependent to mlx5 set, I guess that is wrong, can you >> please confirm? >> >> Above (1) is more important, since series-16301 not fully acked, it is blocking >> me to proceed. >> >>> V2: create a unified patch that contains both the series with >>> the API changes and the series with the necessary error checks in the >> drivers. >>> V3: Fix comments about commit-log. >>> V4: Fix comments about Depends-on and rebase. >>> >>> Li Zhang (4): >>> ethdev: add packet mode in meter profile structure >>> app/testpmd: add meter profile packet mode option >>> net/softnic: check meter packet mode >>> net/mvpp2: check meter packet mode >>> >>> app/test-pmd/cmdline_mtr.c | 40 ++++++++- >>> doc/guides/rel_notes/release_21_05.rst | 12 +++ >>> doc/guides/testpmd_app_ug/testpmd_funcs.rst | 31 +++---- >>> drivers/net/mvpp2/mrvl_mtr.c | 6 ++ >>> drivers/net/softnic/rte_eth_softnic_meter.c | 8 ++ >>> lib/librte_ethdev/rte_mtr.h | 90 ++++++++++++++++++--- >>> 6 files changed, 159 insertions(+), 28 deletions(-) >>> >
Hi Ferruh, > -----Original Message----- > From: Ferruh Yigit <ferruh.yigit@intel.com> > Sent: Tuesday, April 13, 2021 7:06 PM > To: Li Zhang <lizh@nvidia.com>; dekelp@nvidia.com; Ori Kam > <orika@nvidia.com>; Slava Ovsiienko <viacheslavo@nvidia.com>; Matan > Azrad <matan@nvidia.com>; Shahaf Shuler <shahafs@nvidia.com>; > cristian.dumitrescu@intel.com; lironh@marvell.com; jerinj@marvell.com > Cc: dev@dpdk.org; NBU-Contact-Thomas Monjalon <thomas@monjalon.net>; > Raslan Darawsheh <rasland@nvidia.com>; Roni Bar Yanai > <roniba@nvidia.com> > Subject: Re: [PATCH v4 0/4] Support PPS(packet per second) on meter > > External email: Use caution opening links or attachments > > > On 4/13/2021 12:02 PM, Li Zhang wrote: > > Hi Ferruh, > > > >> -----Original Message----- > >> From: Ferruh Yigit <ferruh.yigit@intel.com> > >> Sent: Tuesday, April 13, 2021 6:25 PM > >> To: Li Zhang <lizh@nvidia.com>; dekelp@nvidia.com; Ori Kam > >> <orika@nvidia.com>; Slava Ovsiienko <viacheslavo@nvidia.com>; Matan > >> Azrad <matan@nvidia.com>; Shahaf Shuler <shahafs@nvidia.com>; > >> cristian.dumitrescu@intel.com; lironh@marvell.com; jerinj@marvell.com > >> Cc: dev@dpdk.org; NBU-Contact-Thomas Monjalon > <thomas@monjalon.net>; > >> Raslan Darawsheh <rasland@nvidia.com>; Roni Bar Yanai > >> <roniba@nvidia.com> > >> Subject: Re: [PATCH v4 0/4] Support PPS(packet per second) on meter > >> > >> External email: Use caution opening links or attachments > >> > >> > >> On 4/13/2021 4:50 AM, Li Zhang wrote: > >>> Currently meter algorithms only supports rate is bytes per second(BPS). > >>> Add packet_mode flag in meter profile parameters data structure. > >>> So that it can meter traffic by packet per second. > >>> > >>> When packet_mode is 0, the profile rates and bucket sizes are > >>> specified in bytes per second and bytes when packet_mode is not 0, > >>> the profile rates and bucket sizes are specified in packets and > >>> packets per second. > >>> > >>> Add the necessary checks to the existing drivers implementing the > >>> rte_mtr API to makes sure that profiles with packet_mode set to TRUE > >>> are rejected. > >>> > >>> RFC ("adds support PPS(packet per second) on meter") > >>> > https://patchwork.dpdk.org/project/dpdk/patch/20210125012023.1769769 > >>> - > >> 2 > >>> -lizh@nvidia.com/ > >>> > >>> Depends-on: series=16301 ("Support meter policy API ") > >>> https://patchwork.dpdk.org/project/dpdk/list/?series=16301 > >>> > >> > >> Hi Li, > >> > >> I am not clear with the dependency chain, can you please clarify, > >> > >> 1) Is this set depends to series-16301? Because it compiles fine > >> after conflict resolved, I can see in your repo there is an order, > >> but if there is no functional/logical dependency you can set this > >> patch exactly on top of HEAD (removing the series-16301 in between), so > the CI will be enabled. > > > > I will delete series-16301. > > But it will merge conflict when series-16301 merged after it. > > > > Please send both this patch, and series-16301 on top of latest head, this > enables CI for both. > > When merging them we can handle the conflict, based on which one merged > first, or can ask you to rebase the second one but for this case it does not look > too complex to resolve ourselves. > Got it and will sent it on V5 patch. > >> 2) According its cover letter series-16301 depends on mlx ASO patch, > >> this makes all ethdev patches dependent to mlx5 set, I guess that is > >> wrong, can you please confirm? > >> > >> Above (1) is more important, since series-16301 not fully acked, it > >> is blocking me to proceed. > >> > >>> V2: create a unified patch that contains both the series with > >>> the API changes and the series with the necessary error > >>> checks in the > >> drivers. > >>> V3: Fix comments about commit-log. > >>> V4: Fix comments about Depends-on and rebase. > >>> > >>> Li Zhang (4): > >>> ethdev: add packet mode in meter profile structure > >>> app/testpmd: add meter profile packet mode option > >>> net/softnic: check meter packet mode > >>> net/mvpp2: check meter packet mode > >>> > >>> app/test-pmd/cmdline_mtr.c | 40 ++++++++- > >>> doc/guides/rel_notes/release_21_05.rst | 12 +++ > >>> doc/guides/testpmd_app_ug/testpmd_funcs.rst | 31 +++---- > >>> drivers/net/mvpp2/mrvl_mtr.c | 6 ++ > >>> drivers/net/softnic/rte_eth_softnic_meter.c | 8 ++ > >>> lib/librte_ethdev/rte_mtr.h | 90 ++++++++++++++++++--- > >>> 6 files changed, 159 insertions(+), 28 deletions(-) > >>> > >