From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by inbox.dpdk.org (Postfix) with ESMTP id 174C0A0524; Tue, 13 Apr 2021 13:05:42 +0200 (CEST) Received: from [217.70.189.124] (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id D87F6160E3F; Tue, 13 Apr 2021 13:05:41 +0200 (CEST) Received: from mga07.intel.com (mga07.intel.com [134.134.136.100]) by mails.dpdk.org (Postfix) with ESMTP id 769D4160E39 for ; Tue, 13 Apr 2021 13:05:40 +0200 (CEST) IronPort-SDR: Hx73lSKAHMfVUu9zOj6pRiwH8O8M51W88qZ/+1R0dOBRCs0s57H5qSi0kkyPr1bO2g3fPKa3C6 b3UjXXt18JEA== X-IronPort-AV: E=McAfee;i="6200,9189,9952"; a="258361379" X-IronPort-AV: E=Sophos;i="5.82,219,1613462400"; d="scan'208";a="258361379" Received: from orsmga001.jf.intel.com ([10.7.209.18]) by orsmga105.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 13 Apr 2021 04:05:38 -0700 IronPort-SDR: dVrlOWRqzBGyVQpWPULZY2regDpgaKPLSGzRqoD10ZeBu0gOvzG6v7vxVSCMsIahCdD9vwB1dQ rOrFN1LCkRYA== X-IronPort-AV: E=Sophos;i="5.82,219,1613462400"; d="scan'208";a="460540949" Received: from fyigit-mobl1.ger.corp.intel.com (HELO [10.213.204.251]) ([10.213.204.251]) by orsmga001-auth.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 13 Apr 2021 04:05:35 -0700 To: Li Zhang , "dekelp@nvidia.com" , Ori Kam , Slava Ovsiienko , Matan Azrad , Shahaf Shuler , "cristian.dumitrescu@intel.com" , "lironh@marvell.com" , "jerinj@marvell.com" Cc: "dev@dpdk.org" , NBU-Contact-Thomas Monjalon , Raslan Darawsheh , Roni Bar Yanai References: <20210331085405.1445546-1-lizh@nvidia.com> <20210413035046.28578-1-lizh@nvidia.com> From: Ferruh Yigit X-User: ferruhy Message-ID: <1b88ae6e-addf-cfde-1179-d3f4a2506122@intel.com> Date: Tue, 13 Apr 2021 12:05:31 +0100 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=utf-8; format=flowed Content-Language: en-US Content-Transfer-Encoding: 8bit Subject: Re: [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 List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" On 4/13/2021 12:02 PM, Li Zhang wrote: > Hi Ferruh, > >> -----Original Message----- >> From: Ferruh Yigit >> Sent: Tuesday, April 13, 2021 6:25 PM >> To: Li Zhang ; dekelp@nvidia.com; Ori Kam >> ; Slava Ovsiienko ; Matan >> Azrad ; Shahaf Shuler ; >> cristian.dumitrescu@intel.com; lironh@marvell.com; jerinj@marvell.com >> Cc: dev@dpdk.org; NBU-Contact-Thomas Monjalon ; >> Raslan Darawsheh ; Roni Bar Yanai >> >> 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(-) >>> >