From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from dpdk.org (dpdk.org [92.243.14.124]) by inbox.dpdk.org (Postfix) with ESMTP id 38E32A0519; Mon, 22 Jun 2020 17:22:09 +0200 (CEST) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id B4BD91D73D; Mon, 22 Jun 2020 17:22:08 +0200 (CEST) Received: from wout2-smtp.messagingengine.com (wout2-smtp.messagingengine.com [64.147.123.25]) by dpdk.org (Postfix) with ESMTP id D0ACC1D737 for ; Mon, 22 Jun 2020 17:22:06 +0200 (CEST) Received: from compute7.internal (compute7.nyi.internal [10.202.2.47]) by mailout.west.internal (Postfix) with ESMTP id D77CB2CE; Mon, 22 Jun 2020 11:22:05 -0400 (EDT) Received: from mailfrontend2 ([10.202.2.163]) by compute7.internal (MEProxy); Mon, 22 Jun 2020 11:22:06 -0400 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=monjalon.net; h= from:to:cc:subject:date:message-id:in-reply-to:references :mime-version:content-transfer-encoding:content-type; s=fm1; bh= Gu3IdfKoUxOE9yGggvYtte/mTbTXdIbTAHweMSbwNic=; b=rcklq0VrjvY0Q2Co 2wHVrbLTOhFzmkaA3JyY/9A4tQBBskJQXX9txC4b6hG7Os4/dsJ0zZZYBPAthTRi TF7GU8wVsTi9cprPSIlbcBaa9NBtX2YywcipcPaHtH2J0UykAgxvGd72rcB0/8Vl cRCVlFM06Zj7hFcg6aYkttzH5jcKcjLSX/ep97g58f4sps1rJuM71O3QegjOcb7u nf6tUO3GR753JSASAcco+Ss+yUjml+5GN0sNAk9qJ7zI1s+3V1357gfR66hcqang V960mgYiBDA9XgNaEhLIdk/FZMGDtPn33h5qdpsSt+rX+F07t2L3BvIIeNAGh5q0 PnhOWQ== DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d= messagingengine.com; h=cc:content-transfer-encoding:content-type :date:from:in-reply-to:message-id:mime-version:references :subject:to:x-me-proxy:x-me-proxy:x-me-sender:x-me-sender :x-sasl-enc; s=fm3; bh=Gu3IdfKoUxOE9yGggvYtte/mTbTXdIbTAHweMSbwN ic=; b=azLBe75IQbc97zkx8DkAlSwrLWNYVqkmzYFnIA/PIBd48jdRHheqTo+o9 detJsWsdipFQ5gSH70YrkPFDGESwAMeoAHmv0BeIt37pKGl7w8eIYokcktZ2k59M zPY712Dl2P4khf+glEaMwicPO4mXNwtR7r4Qf4i08H95cTIpWd0LSQTZ/zH3WU1u p/UEX9P6AL5DRny6HfN+chABCrszuQ+wz+tNli1k+tRKoLJjuWw2JgJL9mI3BdyQ 0m+WZh9EjQdeZaG0cVLCRX9rGeeapG2J2HT5zDcFgYZZ82ak7kwemzldWXUomr3T cT019WYHcsMJJZ/ZLLaCaKQbyAauw== X-ME-Sender: X-ME-Proxy-Cause: gggruggvucftvghtrhhoucdtuddrgeduhedrudekvddgkeekucetufdoteggodetrfdotf fvucfrrhhofhhilhgvmecuhfgrshhtofgrihhlpdfqfgfvpdfurfetoffkrfgpnffqhgen uceurghilhhouhhtmecufedttdenucesvcftvggtihhpihgvnhhtshculddquddttddmne cujfgurhephffvufffkfgjfhgggfgtsehtufertddttddvnecuhfhrohhmpefvhhhomhgr shcuofhonhhjrghlohhnuceothhhohhmrghssehmohhnjhgrlhhonhdrnhgvtheqnecugg ftrfgrthhtvghrnhepudeggfdvfeduffdtfeeglefghfeukefgfffhueejtdetuedtjeeu ieeivdffgeehnecukfhppeejjedrudefgedrvddtfedrudekgeenucevlhhushhtvghruf hiiigvpedtnecurfgrrhgrmhepmhgrihhlfhhrohhmpehthhhomhgrshesmhhonhhjrghl ohhnrdhnvght X-ME-Proxy: Received: from xps.localnet (184.203.134.77.rev.sfr.net [77.134.203.184]) by mail.messagingengine.com (Postfix) with ESMTPA id C40FB306734C; Mon, 22 Jun 2020 11:22:04 -0400 (EDT) From: Thomas Monjalon To: Andrey Vesnovaty Cc: jerinjacobk@gmail.com, dev@dpdk.org Date: Mon, 22 Jun 2020 17:22:03 +0200 Message-ID: <8953907.gaXCrnZTga@thomas> In-Reply-To: <20200620133231.12355-1-andrey.vesnovaty@gmail.com> References: <20200620133231.12355-1-andrey.vesnovaty@gmail.com> MIME-Version: 1.0 Content-Transfer-Encoding: 7Bit Content-Type: text/plain; charset="us-ascii" Subject: Re: [dpdk-dev] [RFC v2 0/1] add flow action context API X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 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" 20/06/2020 15:32, Andrey Vesnovaty: > Hi, and thanks a lot for your RFC v1 comments. > > RFC v2 emphasize the intent for sharing the flow action: > * The term 'action context' was unclear and replaced with > 'shared action'. > * RFC v2 subject became 'add flow shared action API'. > * all proposed APIs renamed according the above. > > The new shared action is an independent entity decoupled from any flow > while any flow can reuse such an action. Please go over the RFC > description, it was almost entirely rewritten. > > @Jerin Jacob: > Thanks again for your comments, it made me admit that v1 description was > incomplete & unclear. I hope v2 will be better at least in terms of > clarity. > @Thomas Monjalon: > rte_flow_action_ctx_modify() -> rte_flow_action_ctx_modify() I guess it is a typo. I see the name "rte_flow_shared_action_update" in the patch