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 026B3A00BE; Fri, 11 Feb 2022 11:48:52 +0100 (CET) Received: from [217.70.189.124] (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id CED1341144; Fri, 11 Feb 2022 11:48:51 +0100 (CET) Received: from mail-io1-f53.google.com (mail-io1-f53.google.com [209.85.166.53]) by mails.dpdk.org (Postfix) with ESMTP id E003140150 for ; Fri, 11 Feb 2022 11:48:50 +0100 (CET) Received: by mail-io1-f53.google.com with SMTP id r144so10783213iod.9 for ; Fri, 11 Feb 2022 02:48:50 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20210112; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :cc; bh=3ipsjZSmq4immM/PrAPOmQSg3ISTExC6LeBN2Ga7rEQ=; b=KVSI+yxSt7mzWBxYwl1C5OQVDx5t+L4C00FyquMOG/8XTe5DdoDU391Kp/sSJGtj5K tMPjcJHYB0SitP1oD0Q3RweTVYhLO96KVo0yeIy7Ewf8IN1DJEleAtIKZc9jIp6d0MCp vFdmAx2OHjfpreqgkG7mF5vKIsH/Pvida6SW0zdCSptoEhBhXVsLz3Ksw1bXuFxJopnR wNFM7ej/EInf4rfxYn8fbbnJMFf3cAMtH7TePFGJ9zcJzD06yp8g3zSZPv63R80znWok pzsJNEUDtbGTxjYNackDFx2KirJkQ6qO5tVwhvVNX4bdUB8eHXOJOMIGG7K5bV01930a qbVA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=3ipsjZSmq4immM/PrAPOmQSg3ISTExC6LeBN2Ga7rEQ=; b=JPbEClW6gseCGcLZeVGkUsCMhBz/Z4bvJOR6ns25Otnv+oJj4aKDveeG00WO4s/95S X3cPHQhHywlV/JD4Tc4klWCS5VOo6jfF+KYqfEq3vpwxR9i3myeElHYEsx6+B2jOgpdW QO1jf5enURcAX4l2qQySxu8Lyk6Yjudi/oKrAWYyuM97nS3BnDZSsKa1LBHpgHD35qMx hgSsRtQBzer88W8Q4pp3GiO75PktLjXYgSRn66rHBXGFXdQQf2p6BWMm4OacPBbMP/j9 Ps15oXNzC77WcKO/tB5o9B4qMH0suWDzgLxyFaS+PbyMxe11cmo4xFSrBaji0xrBK9XW 56Ww== X-Gm-Message-State: AOAM532Q3b5jPnP1PoE3M1okIwFMf5xf7VQdUFtj1G22Za5GS19/8IEy xniSKiwEPUYYGoVbVy6UlP8c1VPXBG/ylBSiv4w= X-Google-Smtp-Source: ABdhPJy5GB9tQw/Ckr2QmIOggbEAehQM19uWolfI1sSkahr562/MHmWwCmqnGRGs02dTpscq5Cz7jKFgQXwh/Oe+hcA= X-Received: by 2002:a05:6638:2105:: with SMTP id n5mr481680jaj.266.1644576530203; Fri, 11 Feb 2022 02:48:50 -0800 (PST) MIME-Version: 1.0 References: <20220209213809.1208269-1-akozyrev@nvidia.com> <245157e3-1948-f207-9cdd-88ce22cad453@intel.com> In-Reply-To: <245157e3-1948-f207-9cdd-88ce22cad453@intel.com> From: Jerin Jacob Date: Fri, 11 Feb 2022 16:18:24 +0530 Message-ID: Subject: Re: [PATCH v4 00/10] ethdev: datapath-focused flow rules management To: Ferruh Yigit Cc: Alexander Kozyrev , dpdk-dev , Ori Kam , Thomas Monjalon , Ivan Malov , Andrew Rybchenko , mohammad.abdul.awal@intel.com, Qi Zhang , Jerin Jacob , Ajit Khaparde , "Richardson, Bruce" Content-Type: text/plain; charset="UTF-8" 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 On Thu, Feb 10, 2022 at 9:30 PM Ferruh Yigit wrote: > > On 2/9/2022 9:37 PM, Alexander Kozyrev wrote: > > Three major changes to a generic RTE Flow API were implemented in order > > to speed up flow rule insertion/destruction and adapt the API to the > > needs of a datapath-focused flow rules management applications: > > > > 1. Pre-configuration hints. > > Application may give us some hints on what type of resources are needed. > > Introduce the configuration routine to prepare all the needed resources > > inside a PMD/HW before any flow rules are created at the init stage. > > > > 2. Flow grouping using templates. > > Use the knowledge about which flow rules are to be used in an application > > and prepare item and action templates for them in advance. Group flow rules > > with common patterns and actions together for better resource management. > > > > 3. Queue-based flow management. > > Perform flow rule insertion/destruction asynchronously to spare the datapath > > from blocking on RTE Flow API and allow it to continue with packet processing. > > Enqueue flow rules operations and poll for the results later. > > > > testpmd examples are part of the patch series. PMD changes will follow. > > > > RFC: https://patchwork.dpdk.org/project/dpdk/cover/20211006044835.3936226-1-akozyrev@nvidia.com/ > > > > Signed-off-by: Alexander Kozyrev > > Acked-by: Ori Kam > > > > --- > > v4: > > - removed structures versioning > > - introduced new rte_flow_port_info structure for rte_flow_info_get API > > - renamed rte_flow_table_create to rte_flow_template_table_create > > > > v3: addressed review comments and updated documentation > > - added API to get info about pre-configurable resources > > - renamed rte_flow_item_template to rte_flow_pattern_template > > - renamed drain operation attribute to postpone > > - renamed rte_flow_q_drain to rte_flow_q_push > > - renamed rte_flow_q_dequeue to rte_flow_q_pull > > > > v2: fixed patch series thread > > > > Alexander Kozyrev (10): > > ethdev: introduce flow pre-configuration hints > > ethdev: add flow item/action templates > > ethdev: bring in async queue-based flow rules operations > > app/testpmd: implement rte flow configuration > > app/testpmd: implement rte flow template management > > app/testpmd: implement rte flow table management > > app/testpmd: implement rte flow queue flow operations > > app/testpmd: implement rte flow push operations > > app/testpmd: implement rte flow pull operations > > app/testpmd: implement rte flow queue indirect actions > > > > Hi Jerin, Ajit, Ivan, > > As far as I can see you did some reviews in the previous versions, > but not ack the patch. > Is there any objection to last version of the patch, if not I will > proceed with it. Personally, I have no strong objections. Based on the top level review, It looks good to me on the application API side. Please feel free to proceed with the series as you see fit. > > Hi Alex, > > As process we require at least one PMD implementation (it can be draft) > to justify the API design. > > If there is no objection from above reviewers and PMD implementation > exists before end of the week, I think we can get the set for -rc1. > > Thanks, > ferruh