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:   Shenwei Wang <shenwei.wang@nxp.com>
To:     Andrew Lunn <andrew@lunn.ch>
CC:     "David S . Miller" <davem@davemloft.net>,
        Eric Dumazet <edumazet@google.com>,
        Jakub Kicinski <kuba@kernel.org>,
        Paolo Abeni <pabeni@redhat.com>,
        Alexei Starovoitov <ast@kernel.org>,
        Daniel Borkmann <daniel@iogearbox.net>,
        Jesper Dangaard Brouer <hawk@kernel.org>,
        John Fastabend <john.fastabend@gmail.com>,
        Wei Fang <wei.fang@nxp.com>,
        "netdev@vger.kernel.org" <netdev@vger.kernel.org>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
        "imx@lists.linux.dev" <imx@lists.linux.dev>
Subject: RE: [EXT] Re: [PATCH 1/1] net: fec: using page pool to manage RX
 buffers
Thread-Topic: [EXT] Re: [PATCH 1/1] net: fec: using page pool to manage RX
 buffers
Thread-Index: AQHY1QQs3moef3i30kelaD3cHlhvbK34Yi8AgAAAVFCAAAJXgIAAAdEw
Date:   Fri, 30 Sep 2022 20:08:58 +0000
Message-ID: <PAXPR04MB91851D68BDDEAEF25DED506289569@PAXPR04MB9185.eurprd04.prod.outlook.com>
References: <20220930193751.1249054-1-shenwei.wang@nxp.com>
 <YzdI4mDXCKuI/58N@lunn.ch>
 <PAXPR04MB9185E69829540686618987D289569@PAXPR04MB9185.eurprd04.prod.outlook.com>
 <YzdLHx+N4SXaAkUe@lunn.ch>
In-Reply-To: <YzdLHx+N4SXaAkUe@lunn.ch>
Accept-Language: en-US
Content-Language: en-US
x-ms-exchange-antispam-messagedata-chunkcount: 1
x-ms-exchange-antispam-messagedata-0: =?iso-8859-1?Q?3GP9mnNtL+DLP9q/FsiFv/m8RBSXGuXUvotTOJ8OVlLz30FgzDNpcY7dN1?=
 =?iso-8859-1?Q?1HxuRarnCyXPR5L9YrEe3rYwGdvQdTYVxHZIANn8ZrarkSK/qzLFcc/PdU?=
 =?iso-8859-1?Q?EheYHiPQ4TyeNpugHgZTwN6H49+k6ENLIRbenlFf547HlZUZSujaQpkcfQ?=
 =?iso-8859-1?Q?tJBSkbbaL0joqlYk4M7HObPFXSuIWVDhsFf9/l/vseWy14cEoxDIkpAxSo?=
 =?iso-8859-1?Q?Qm66F869D7Ezymwg0mKOPESm7nx2yOzgrW7U1B7xPYwkvfbYlAWERYD3v+?=
 =?iso-8859-1?Q?1D9KrFZvd8L6MaaHl9Lupp9uBhtqsFun0J6I5sJTUGBLbynsxKSaz74zNV?=
 =?iso-8859-1?Q?MP0CHX7dHuXXn0fruGoNyzoffiRrFXpB6SkJzo8C95zhP3K/g7IYuVrBMi?=
 =?iso-8859-1?Q?WNE8uLQUU8xt9V6Vm79zGusXLnfJfR1dS7V/v7vHhAQEXtW/k7ioz7HbJ0?=
 =?iso-8859-1?Q?OWdBX5eqeQup1nsZLhe4w5UzulOcG93aFE8UYduaCBZQYaWeJez8+bouKJ?=
 =?iso-8859-1?Q?Otx+3SxmMKARWmE7jUSqqkiR8mLMTW2JwjSlw98d2nrbUlB8wDxJgb3FBn?=
 =?iso-8859-1?Q?eJ5DmJLEfQJbfqDEYDOimy5kHuzjrewZQ9aVWIeLE93z440PP/3Jqvye4v?=
 =?iso-8859-1?Q?tgw0ayoan3mIDsTTsERv/qG43blo4wWhxaLXfgxPR6OmhImCoIexudNax6?=
 =?iso-8859-1?Q?D2EUCsR50ru6uXvmBIAdd3trXHN/hU2842Jkp6U0jzf26oPSM74rIy2zdv?=
 =?iso-8859-1?Q?UqCbLXKCbhPBjbJRJRVe8j55nWRr3Dg1gdBsyNlsRcQJNgp12tZEfjbkf0?=
 =?iso-8859-1?Q?8CJi9czYvr081qFHJZqPjoeW0xWbyD6K+KVrnT0s2H0j0r31NteMjkIobA?=
 =?iso-8859-1?Q?gs1eGGGFcVoR7t7NtZd3FV+ObOXz9KgzNshgFAIVLaX6bEKR03iJxoJhz3?=
 =?iso-8859-1?Q?XrXca3JWX8GRABw6D77kxFbKsg3TbD82Z549tn2l85J1VGKPH6NcCf2vt+?=
 =?iso-8859-1?Q?bE+xEK+ODuLkNlQ2/zxXgS52YNPCo7U3+zFmECdWJ6a6z/iycheuxbCATJ?=
 =?iso-8859-1?Q?DlnOy62ARfYE+0fPIs7fmE5tFzaXcZbxku2lBwGzV/437LiJbaoxBzk8gu?=
 =?iso-8859-1?Q?cvgDOM6VdChY07fJHS8XPoOkZxQT6zcWDZv9gXvzWVV3W2cTRbzpqXHvhd?=
 =?iso-8859-1?Q?h8GTWE09rXmwAS6mKP1WO/mEsxGSQ06yWO0+uSIZ1ex58nbT6Ti4St90JT?=
 =?iso-8859-1?Q?Lp9BhmnWj5hNtXW5YxKB55TX1HsXdHB0U678jko69S4OuXHG+wotZrbL3m?=
 =?iso-8859-1?Q?FTsr+GphNqVL2q1t+PrtS4+bWvlkMJxx5wlngWrXD88s2eMVEzAp+0ROup?=
 =?iso-8859-1?Q?91iTDiN7jyRWOxmmEcpajVxn7AEdx7wzoUpAED0WUpfBmgte5D39yl5ynI?=
 =?iso-8859-1?Q?+tY54bDbbh44wdZBBBYFyomIIxq0PZ4yh7YbnJS+splub6PZDJmxAFy2w+?=
 =?iso-8859-1?Q?e7HkpLKhAiGqhiTxbpIdNJvd2L6C0XagJ5RjawLrcTHWFiS8zJo9Z3FiPN?=
 =?iso-8859-1?Q?r6lhZ9QRE3AjvxSKKkj2x8Tn/cWzl/NFe6y+0u+lZh0Bx4zu8oySv7nXpt?=
 =?iso-8859-1?Q?Znv2OMRKMqr+ZLlUwTsv/2v18FR5H8tyZv?=
Content-Type: text/plain; charset="iso-8859-1"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-OriginatorOrg: nxp.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: PAXPR04MB9185.eurprd04.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: c40eacee-9f00-411d-c172-08daa31f9c3e
X-MS-Exchange-CrossTenant-originalarrivaltime: 30 Sep 2022 20:08:58.6706
 (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: NF93kGb6JSReev5XrzjieTvpawsFickvia75DT9zghW15gsJqhOXCknvILgPQax+w18nB77d9ZaYofshZfXyXw==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: PA4PR04MB8064
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_DNSWL_NONE,
        RCVD_IN_MSPIKE_H2,SPF_HELO_PASS,SPF_PASS 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


> -----Original Message-----
> From: Andrew Lunn <andrew@lunn.ch>
> Sent: Friday, September 30, 2022 3:02 PM
> To: Shenwei Wang <shenwei.wang@nxp.com>
> Cc: David S . Miller <davem@davemloft.net>; Eric Dumazet
> <edumazet@google.com>; Jakub Kicinski <kuba@kernel.org>; Paolo Abeni
> <pabeni@redhat.com>; Alexei Starovoitov <ast@kernel.org>; Daniel Borkmann
> <daniel@iogearbox.net>; Jesper Dangaard Brouer <hawk@kernel.org>; John
> Fastabend <john.fastabend@gmail.com>; Wei Fang <wei.fang@nxp.com>;
> netdev@vger.kernel.org; linux-kernel@vger.kernel.org; imx@lists.linux.dev
> Subject: Re: [EXT] Re: [PATCH 1/1] net: fec: using page pool to manage RX
> buffers
>=20
> Caution: EXT Email
>=20
> > The testing above was on the imx8 platform. The following are the
> > testing result On the imx6sx board:
> >
> > ######### Original implementation ######
> >
> > shenwei@5810:~/pktgen$ iperf -c 10.81.16.245 -w 2m -i 1
> > ------------------------------------------------------------
> > Client connecting to 10.81.16.245, TCP port 5001 TCP window size:  416
> > KByte (WARNING: requested 1.91 MByte)
> > ------------------------------------------------------------
> > [  1] local 10.81.17.20 port 36486 connected with 10.81.16.245 port 500=
1
> > [ ID] Interval       Transfer     Bandwidth
> > [  1] 0.0000-1.0000 sec  70.5 MBytes   591 Mbits/sec
> > [  1] 1.0000-2.0000 sec  64.5 MBytes   541 Mbits/sec
> > [  1] 2.0000-3.0000 sec  73.6 MBytes   618 Mbits/sec
> > [  1] 3.0000-4.0000 sec  73.6 MBytes   618 Mbits/sec
> > [  1] 4.0000-5.0000 sec  72.9 MBytes   611 Mbits/sec
> > [  1] 5.0000-6.0000 sec  73.4 MBytes   616 Mbits/sec
> > [  1] 6.0000-7.0000 sec  73.5 MBytes   617 Mbits/sec
> > [  1] 7.0000-8.0000 sec  73.4 MBytes   616 Mbits/sec
> > [  1] 8.0000-9.0000 sec  73.4 MBytes   616 Mbits/sec
> > [  1] 9.0000-10.0000 sec  73.9 MBytes   620 Mbits/sec
> > [  1] 0.0000-10.0174 sec   723 MBytes   605 Mbits/sec
> >
> >
> >  ######  Page Pool impl=E9mentation ########
> >
> > shenwei@5810:~/pktgen$ iperf -c 10.81.16.245 -w 2m -i 1
> > ------------------------------------------------------------
> > Client connecting to 10.81.16.245, TCP port 5001 TCP window size:  416
> > KByte (WARNING: requested 1.91 MByte)
> > ------------------------------------------------------------
> > [  1] local 10.81.17.20 port 57288 connected with 10.81.16.245 port 500=
1
> > [ ID] Interval       Transfer     Bandwidth
> > [  1] 0.0000-1.0000 sec  78.8 MBytes   661 Mbits/sec
> > [  1] 1.0000-2.0000 sec  82.5 MBytes   692 Mbits/sec
> > [  1] 2.0000-3.0000 sec  82.4 MBytes   691 Mbits/sec
> > [  1] 3.0000-4.0000 sec  82.4 MBytes   691 Mbits/sec
> > [  1] 4.0000-5.0000 sec  82.5 MBytes   692 Mbits/sec
> > [  1] 5.0000-6.0000 sec  82.4 MBytes   691 Mbits/sec
> > [  1] 6.0000-7.0000 sec  82.5 MBytes   692 Mbits/sec
> > [  1] 7.0000-8.0000 sec  82.4 MBytes   691 Mbits/sec
> > [  1] 8.0000-9.0000 sec  82.4 MBytes   691 Mbits/sec
> > ^C[  1] 9.0000-9.5506 sec  45.0 MBytes   686 Mbits/sec
> > [  1] 0.0000-9.5506 sec   783 MBytes   688 Mbits/sec
>=20
> Cool, so it helps there as well.
>=20
> But you knew i was interested in these numbers. So you should of made the=
m
> part of the commit message so i didn't have to ask...
>=20

Sorry. I am adding it to the commit v2.

Thanks,
Shenwei

>      Andrew