From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-wr0-f169.google.com (mail-wr0-f169.google.com [209.85.128.169]) by dpdk.org (Postfix) with ESMTP id D24387EBF for ; Mon, 9 Apr 2018 17:00:47 +0200 (CEST) Received: by mail-wr0-f169.google.com with SMTP id l49so9930356wrl.4 for ; Mon, 09 Apr 2018 08:00:47 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=6wind-com.20150623.gappssmtp.com; s=20150623; h=date:from:to:cc:subject:message-id:references:mime-version :content-disposition:in-reply-to; bh=212ZuiudTyDeecZFqX4Xp5+GXY0G6n1p2KR+MSVEUtA=; b=I1e+9TryPCTZ3PCSXeZw5OAwYYExY5tX9Etc4tnmTrsd2ELT+J2MAdZC/KmjpNXS1b 46TV69ohGCL9DHy3Ksz6sZmcJZf1i5nbgFBjNbIS+uem7nB48IXruH5b/uumnbutUYGP mGwudn3N8Z1Jm1/JCah4inYI8pNXsoxMKpFzESFi+N8X37cx8ZI2sKLifO5ikG3/ak0W ykGPQUu+ZgpX/v3OtAbVVCFzimgDr2SeYjg4NCofKPDEqyNsKi4kjGkqMdocJ5yN7pJH wkTmPumamMkCljkuL9OAHXaPKyuxtKRlt4OF/dJuKtFOhSGvUT06g16hATeS2lc/Jqtg YuvA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:date:from:to:cc:subject:message-id:references :mime-version:content-disposition:in-reply-to; bh=212ZuiudTyDeecZFqX4Xp5+GXY0G6n1p2KR+MSVEUtA=; b=cvSQnNJyS8ag2hFkGCDSV+2BWNLBNDQrjw+aq7Nr/HsmDJZslGVYQw9vz6hWGocaeA 60Dw3gimciHtdPWcdLdfKf4PL/oSPgVOnBq1lnalbfO8dzIwjS7ZQhlPc1lRyJ30I5ya 9flXfKu8aSFa9tyBNN5HtKxLFJkYfMHanT2vbBpiHR2/wdZQzlY2ACTfG+4Aqz3jrD5g STvlIv9Dv0GtL6BiUkE8hYUiMrlAxQCh9JJ9YE7rN/Aqn6Jr0jrhPDoZ+3wIf+8M1Lyj G4d5agVD2U/EUgBmqQz6JsgNal13bZmzMOqsYcAZt4lGYoCHLgaFuomkZk7h5GYrRzPx q6Dg== X-Gm-Message-State: AElRT7G2uoA1YTQEyhD4czBDuCqY68VFwKf0wTx/043XVKDF4O8cyqJz V9xvtcVF2cj0f9VvR26VC1oSOA== X-Google-Smtp-Source: AIpwx4+CJLmFwi+nwgUkIh6c3ipylYjf59dfhU7xnx9kKAMgbTqixw6wYxL7FZ++P41cxvTF7+f6dA== X-Received: by 10.223.176.253 with SMTP id j58mr26153300wra.269.1523286047530; Mon, 09 Apr 2018 08:00:47 -0700 (PDT) Received: from 6wind.com (host.78.145.23.62.rev.coltfrance.com. [62.23.145.78]) by smtp.gmail.com with ESMTPSA id n21sm1185711wmi.37.2018.04.09.08.00.46 (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Mon, 09 Apr 2018 08:00:46 -0700 (PDT) Date: Mon, 9 Apr 2018 17:00:33 +0200 From: Adrien Mazarguil To: Andrew Rybchenko Cc: Thomas Monjalon , Ferruh Yigit , dev@dpdk.org, "Zhang, Qi Z" Message-ID: <20180409150033.GB4957@6wind.com> References: <20180404150312.12304-1-adrien.mazarguil@6wind.com> <20180406131736.19145-1-adrien.mazarguil@6wind.com> <20180406131736.19145-15-adrien.mazarguil@6wind.com> <07a42ecb-3a82-edc7-6763-1fd850ce580b@solarflare.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <07a42ecb-3a82-edc7-6763-1fd850ce580b@solarflare.com> Subject: Re: [dpdk-dev] [PATCH v2 14/15] ethdev: add physical port action to flow 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: , X-List-Received-Date: Mon, 09 Apr 2018 15:00:48 -0000 On Sat, Apr 07, 2018 at 12:51:40PM +0300, Andrew Rybchenko wrote: > On 04/06/2018 04:25 PM, Adrien Mazarguil wrote: > > This patch adds the missing action counterpart to the PHY_PORT pattern > > item, that is, the ability to directly inject matching traffic into a > > physical port of the underlying device. > > Does it mean that if it is applied on ingress (incoming packet from network) > it will simply send packets back to network (specified physical port)? Precisely. > And if it is applied on egress (outgoing from device to network) it will > be directed to possibly different physical port and sent to network. Right, note it gives applications the ability to express that wish, the fact PMDs support it is another matter :) In any case, this action is added for API completeness but should be rarely necessary since we chose to go with port representors. Port representors will expose valid DPDK port IDs, therefore applications will simply have to create ingress/egress flow rules on the right DPDK port targeting different port IDs through the PORT_ID action. > > It breaks ABI compatibility for the following public functions: > > > > - rte_flow_copy() > > - rte_flow_create() > > - rte_flow_query() > > - rte_flow_validate() > > > > Signed-off-by: Adrien Mazarguil > > Cc: "Zhang, Qi Z" > > --- > > app/test-pmd/cmdline_flow.c | 35 ++++++++++++++++++++++++ > > app/test-pmd/config.c | 1 + > > doc/guides/prog_guide/rte_flow.rst | 20 ++++++++++++++ > > doc/guides/testpmd_app_ug/testpmd_funcs.rst | 5 ++++ > > lib/librte_ether/rte_flow.c | 1 + > > lib/librte_ether/rte_flow.h | 22 +++++++++++++++ > > 6 files changed, 84 insertions(+) > > <...> -- Adrien Mazarguil 6WIND