Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
From:   Wei Fang <wei.fang@nxp.com>
To:     Simon Horman <horms@kernel.org>
CC:     "davem@davemloft.net" <davem@davemloft.net>,
        "edumazet@google.com" <edumazet@google.com>,
        "kuba@kernel.org" <kuba@kernel.org>,
        "pabeni@redhat.com" <pabeni@redhat.com>,
        Shenwei Wang <shenwei.wang@nxp.com>,
        Clark Wang <xiaoning.wang@nxp.com>,
        "ast@kernel.org" <ast@kernel.org>,
        "daniel@iogearbox.net" <daniel@iogearbox.net>,
        "hawk@kernel.org" <hawk@kernel.org>,
        "john.fastabend@gmail.com" <john.fastabend@gmail.com>,
        "larysa.zaremba@intel.com" <larysa.zaremba@intel.com>,
        "aleksander.lobakin@intel.com" <aleksander.lobakin@intel.com>,
        "jbrouer@redhat.com" <jbrouer@redhat.com>,
        "netdev@vger.kernel.org" <netdev@vger.kernel.org>,
        dl-linux-imx <linux-imx@nxp.com>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
        "bpf@vger.kernel.org" <bpf@vger.kernel.org>
Subject: RE: [PATCH V5 net-next 1/2] net: fec: add XDP_TX feature support
Thread-Topic: [PATCH V5 net-next 1/2] net: fec: add XDP_TX feature support
Thread-Index: AQHZy1dCnxQwOLKBq0eoj4kIh1yvrq/k/3cAgADLJ1A=
Date:   Sat, 12 Aug 2023 00:06:45 +0000
Message-ID: <AM5PR04MB313962DCC05E4D588C37A6BF8811A@AM5PR04MB3139.eurprd04.prod.outlook.com>
References: <20230810064514.104470-1-wei.fang@nxp.com>
 <20230810064514.104470-2-wei.fang@nxp.com> <ZNYiSEG5/0cJIZ3f@vergenet.net>
In-Reply-To: <ZNYiSEG5/0cJIZ3f@vergenet.net>
Accept-Language: en-US
Content-Language: en-US
x-ms-exchange-antispam-messagedata-chunkcount: 1
x-ms-exchange-antispam-messagedata-0: =?us-ascii?Q?54lZ6+yffRkO/R/j+tiOOhoKJ6lnG16xa6wb5niujyLNbS8Ma6EybX1+O3C3?=
 =?us-ascii?Q?DnEz6SE3U/RZ8CBGMwxiY9gkcAaFkE9vCqg6qTK3bWqdt6HeWT8adVK12wGz?=
 =?us-ascii?Q?dPgBXwbqSfuPUQv/GbQq01kvsV+ncKizJGlBTW/jNn3d7ftmkSWUuIwn9dI/?=
 =?us-ascii?Q?frqgdIKBOo7Frmnf8teRGl/iaJTFttgtl1WjyBpW3fK+4uRRTqKcJzn0S2iU?=
 =?us-ascii?Q?q+Sr+RKlWnoF1FoBrzuBrHmLpPSRP0uiR6La/b+hGeIiww1JH+nCXWTR3VmC?=
 =?us-ascii?Q?qblyTxDChiy0srXs5cxexFxFR3shSEoYh/KuyGU7hgLvZQDIRlJ9GrRrL9lK?=
 =?us-ascii?Q?Rn6cR3SfLtgd04afIhhE1FDV99au1T/V11+IPkU9z5mPn2MuL2BtOGBbmVJk?=
 =?us-ascii?Q?W4Xp83L/LHx0ZoFFEdJq90d+cUbKnqVhTxoEGLMMXvmy0S6LKYmlj5TF3/JR?=
 =?us-ascii?Q?aY7HswGs5Q/P4NQ1U8nypbZUa/Qr2GuVXXEsvd/uSeLK1laAUOOH7IDlpvHT?=
 =?us-ascii?Q?GJ+ZUASr850YB1QyLdXTObCqDKkzI4gi6MI9IfDGRkXmIqJLIbE4T+NuOvj3?=
 =?us-ascii?Q?W4JBojIzv7r3kpNbb6IX1Am4JkpSOC7kaoSs89XmbF0ksu+uV3uHx3q/15MR?=
 =?us-ascii?Q?7huXYhgouiUzcZjQLkv/EuoojU2z31rxvHroQDLX/2h740o1UqbsXui6HkG+?=
 =?us-ascii?Q?/QmT5l4J5p6EflhTQ2w5ah7G4bGv6is1jWNAg7LEJ8CqHejLXI2wtBWJRvjj?=
 =?us-ascii?Q?rQ7DRXJt66jOoT3p2j9ngGDrNUi5c1xGMwvXOoIYKuNLkSz+SvD0BH2KSoM0?=
 =?us-ascii?Q?FonALeZ866hovUyM5jmpe2lWDLuvxz9ibpkbQ1+PH4ElGv8Ip165FCJuAtm7?=
 =?us-ascii?Q?UJKitmwdZP8RorD0CZKMS6gzWogbREWAOGsZpjARsE+eSUANn5LJXr9NexLT?=
 =?us-ascii?Q?5d+71mGkprvn1GKxQAbH6VYQmpOFbJxw/fcljn0Yetf1M94nk+PMVvtW2p09?=
 =?us-ascii?Q?1jtM6vXSwqawwmp/dVtv2Ner0UcdfeuBWDYFb/60xXNUWSonzy3jgX+l/Dzk?=
 =?us-ascii?Q?KxIDP2G51+Ym6wUfFXQ+XnIPatCYcx8FKmD1iYM7GiMQajh/FY1/F+vkHHxR?=
 =?us-ascii?Q?B//RHiEF6xt8f708/qJaD9IWrsuzYMrqy5vRENHe66Th82DZrxTVJfMfBrwH?=
 =?us-ascii?Q?gTzPojNGux7QLgYSkcYB4I9rsmidilqiChc7Ra3fQlrJsLl5vspDqwUqn781?=
 =?us-ascii?Q?h693Cek7Bui3xjKv5XaMKBuaL90ImlnIygBzKoKYhykmE+Kna9JdtbL8pStH?=
 =?us-ascii?Q?LgB9lUXu+EnZE/Jshs13r8QNXnRQ/hZq/wT4iEEvLuF/X/losFpTyxqoFzyR?=
 =?us-ascii?Q?KFJpDdm15HR4APVQcjfNLbCZOyu1G965QcKW8KOg5K0sGlXN7PPLI4SVGVC4?=
 =?us-ascii?Q?fBKJ9vTeyqXRZJTRXEsfCnJpIwLNbrCyH/9AEQziDeqtz1M6bNhov02vD3hx?=
 =?us-ascii?Q?K6QiLrznaGGm1zCjgm6DW7NqsGO1BEKf1/EjLu9a+8lX3DonbyZGwkVnzl1v?=
 =?us-ascii?Q?NO+s5hW7x7z99bOrYoA=3D?=
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-OriginatorOrg: nxp.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: AM5PR04MB3139.eurprd04.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 6f207a9b-86d0-4b28-c06f-08db9ac8042c
X-MS-Exchange-CrossTenant-originalarrivaltime: 12 Aug 2023 00:06:45.6945
 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 686ea1d3-bc2b-4c6f-a92c-d99c5c301635
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: ET9b+4KCvj13J6d2AvTZtshvbv7X7DH/VQG/zkrK20lC5sOe5cpstjykyBF4ZppSpxKe7iqxALNZWQwel7fVxw==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: AS8PR04MB8024
X-Spam-Status: No, score=-2.1 required=5.0 tests=BAYES_00,DKIM_SIGNED,
        DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_MSPIKE_H2,SPF_HELO_NONE,
        T_SPF_TEMPERROR,URIBL_BLOCKED autolearn=ham autolearn_force=no
        version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
        lindbergh.monkeyblade.net
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

> On Thu, Aug 10, 2023 at 02:45:13PM +0800, Wei Fang wrote:
> > The XDP_TX feature is not supported before, and all the frames which
> > are deemed to do XDP_TX action actually do the XDP_DROP action. So
> > this patch adds the XDP_TX support to FEC driver.
> >
> > I tested the performance of XDP_TX in XDP_DRV mode and XDP_SKB mode
> > respectively on i.MX8MP-EVK platform, and as suggested by Jesper, I
> > also tested the performance of XDP_REDIRECT on the same platform. And
> > the test steps and results are as follows.
> >
> > XDP_TX test:
> > Step 1: One board is used as generator and connects to switch,and the
> > FEC port of DUT also connects to the switch. Both boards with flow
> > control off. Then the generator runs the
> > pktgen_sample03_burst_single_flow.sh script to generate and send burst
> > traffic to DUT. Note that the size of packet was set to 64 bytes and
> > the procotol of packet was UDP in my test scenario. In addtion, the
> > SMAC of the packet need to be different from the MAC
>=20
> nit: addtion -> addition
>=20
>      checkpatch.pl --codespell is your friend here.
>=20
Thanks for reminding.