From:   "Rout, ChandanX" <chandanx.rout@intel.com>
To:     Jason Xing <kerneljasonxing@gmail.com>,
        "Brandeburg, Jesse" <jesse.brandeburg@intel.com>,
        "Nguyen, Anthony L" <anthony.l.nguyen@intel.com>,
        "davem@davemloft.net" <davem@davemloft.net>,
        "edumazet@google.com" <edumazet@google.com>,
        "kuba@kernel.org" <kuba@kernel.org>,
        "pabeni@redhat.com" <pabeni@redhat.com>,
        "richardcochran@gmail.com" <richardcochran@gmail.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>,
        "Lobakin, Alexandr" <alexandr.lobakin@intel.com>,
        "Fijalkowski, Maciej" <maciej.fijalkowski@intel.com>
CC:     "netdev@vger.kernel.org" <netdev@vger.kernel.org>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
        "intel-wired-lan@lists.osuosl.org" <intel-wired-lan@lists.osuosl.org>,
        "bpf@vger.kernel.org" <bpf@vger.kernel.org>,
        Jason Xing <kernelxing@tencent.com>,
        "Kuruvinakunnel, George" <george.kuruvinakunnel@intel.com>,
        "Nagaraju, Shwetha" <shwetha.nagaraju@intel.com>,
        "Nagraj, Shravan" <shravan.nagraj@intel.com>
Subject: RE: [Intel-wired-lan] [PATCH net v2 2/3] i40e: add double of VLAN
 header when computing the max MTU
Thread-Topic: [Intel-wired-lan] [PATCH net v2 2/3] i40e: add double of VLAN
 header when computing the max MTU
Thread-Index: AQHZO2dXF3vsNzyKmkWYiSXcSfkRfq7Nu2Qg
Date:   Tue, 14 Feb 2023 02:13:50 +0000
Message-ID: <MN2PR11MB40457D2433C5155AE212F48FEAA29@MN2PR11MB4045.namprd11.prod.outlook.com>
References: <20230208024333.10465-1-kerneljasonxing@gmail.com>
 <20230208024333.10465-2-kerneljasonxing@gmail.com>
In-Reply-To: <20230208024333.10465-2-kerneljasonxing@gmail.com>
Accept-Language: en-US
Content-Language: en-US
x-ms-exchange-antispam-messagedata-chunkcount: 1
x-ms-exchange-antispam-messagedata-0: =?us-ascii?Q?B5beHWyNnWaOMJo36Gr5FWqQE3YAuVmbmOCtqZMXDQv7XNjV8yL+rvMGyUd3?=
 =?us-ascii?Q?r/tbOL8AuOcP95fNIZMXUpycWJbR3qvWJmSHHSlqR+x/9193GYQib6w1IrA/?=
 =?us-ascii?Q?ZJ0CjO5kUbaqDRbTBkTw0u8AzHW89FqUOpI9NtA84A6uiBiKteQdsZUkFiNA?=
 =?us-ascii?Q?NBaXypPMIwsY/3YiGXEaZ8llsbfshzR0r0WkxUcmUuM1LijcfRSFEUME4T1F?=
 =?us-ascii?Q?xnqW1EymkZ8pa5NyKBnQ6DqRj4JVt45VSK5pV/v3iAhRYqEAyBA1WBJN+oEJ?=
 =?us-ascii?Q?QBdEYRMjOSuHqw59/CoqJuzKQ5N/bCNQFakxtch9EMjvXYBVazfyL/OyVNOc?=
 =?us-ascii?Q?+UMiGi8sLAECgAccIKAJDdApXxs9nS5n5UOyN60spS8VhWnAULsRq74OW1iw?=
 =?us-ascii?Q?KvSprTQopy8oHAxOdJ4YbDDjp2t4y6Is+y9FH4zZm4PaEn9Q5P6aGeJzW2o+?=
 =?us-ascii?Q?cy/DoIW9GQ9/X8IjPll85jc/SlFI129xd4z9KsLT1Z7GPeUA1+3r9dY+IiQ8?=
 =?us-ascii?Q?ftIhjmlKmCqJ4OALBb56lqI0hwd7AW29nH7vtdgeWDP/6VEAx38CWCgc9Utf?=
 =?us-ascii?Q?csu+iNYegoo5YSNqXk4P6mocIHWAmk2fTAM0RKb+JVvOtY14QPNpkKODV5BN?=
 =?us-ascii?Q?QR6NwzrASC5CKqNkjI1FZpTFuVihKemMNdtVEfp3bfuyfGsLbNkcs/ZW6ljJ?=
 =?us-ascii?Q?niG0Ci4P7epGevrrpfcPav9AdB1iQkK3uUIFg3CALUhmsR64niF8Tdf9i/ak?=
 =?us-ascii?Q?W3OYlLceHm34fUu5CjBZCtKHrbeagC986yRwQ/vnYwHEPfYRpV/h0eGInqPI?=
 =?us-ascii?Q?JBeOTNQZZZ7YR4BMQSWBUGGrpt8kY9xWbJP1INFPip8lWTTe1O8HaE9RarrJ?=
 =?us-ascii?Q?cz9bClIcZVFD8AYVyGNjPpp0Hs4Ywl4uPOniOlsPn9x2VQ8439Q4VItKCZyb?=
 =?us-ascii?Q?e34SBrF0eS/afAadeg9VHm6ka5/guMqUre+SAanTKmtKlLYJXzlghLQ7j9sV?=
 =?us-ascii?Q?l7bHHTm/bXi9m8jEszDJECsJvgZRcytlEL2vRPRQtRsm8G6k1H6I+LHA+YdP?=
 =?us-ascii?Q?oLNXvR98hJKoyEHK89qyMYdSVgsrpYRIQMZgrVWKi4v3NKt/E6ArRlOWp6jm?=
 =?us-ascii?Q?PT7EuLDbEHA0T+HiBdUlgPjQyZeRhFrMf6KP8Zcs3q8tukzeZhjeO+fxxSPt?=
 =?us-ascii?Q?JYajEZULXEWfmn/rWTE2YNpvTDdIYmtt1Wi/mNZnZDG9JYH/gKQvXFkHz6/0?=
 =?us-ascii?Q?mfcSawJIqDpZ7WSVKN2FRpyHW+Xlwj8lVxwnjaspHVegw7I+bIJiXRGDUHQA?=
 =?us-ascii?Q?DOk9vcDFQVuz+rNjeS5uFnHDyI0r8nMn28xSFiPO8WngCN2Qtmdc//XYbzqG?=
 =?us-ascii?Q?5RepV7De/1Gkdprr/6rU5g+Hgry7D6G65Pa9R8FV8+EW+yLYaSjjB3Xa/xdA?=
 =?us-ascii?Q?wFVefXSotfg2gSNC6QMZFmarZB88EhUOlVUjbAL21xSbbAUHVJjKXfyvs5Tx?=
 =?us-ascii?Q?Imupk6JBA3gLXRRq3fFvp2gllAeTwKHP49brsx2WzNPajS4GH//3uRoH9VjZ?=
 =?us-ascii?Q?zjqgyXejaDvT6Ajfuye5emCeqlhxMfaWFEoyLW3x?=
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: MN2PR11MB4045.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 5686a2e1-2efe-486c-ca6c-08db0e311d17
X-MS-Exchange-CrossTenant-originalarrivaltime: 14 Feb 2023 02:13:50.7185
 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 46c98d88-e344-4ed4-8496-4ed7712e255d
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: /D9+8ZPaqxUfU4sPI4r02eUJIVUPRd1/1pv2uqSB59eL2ADVImVBsXl4pH9uXcobsMp5FcHm1x8SiFO4ynqQ/g==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: MN0PR11MB5985
X-OriginatorOrg: intel.com
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org


>-----Original Message-----
>From: Intel-wired-lan <intel-wired-lan-bounces@osuosl.org> On Behalf Of
>Jason Xing
>Sent: 08 February 2023 08:14
>To: Brandeburg, Jesse <jesse.brandeburg@intel.com>; Nguyen, Anthony L
><anthony.l.nguyen@intel.com>; davem@davemloft.net;
>edumazet@google.com; kuba@kernel.org; pabeni@redhat.com;
>richardcochran@gmail.com; ast@kernel.org; daniel@iogearbox.net;
>hawk@kernel.org; john.fastabend@gmail.com; Lobakin, Alexandr
><alexandr.lobakin@intel.com>; Fijalkowski, Maciej
><maciej.fijalkowski@intel.com>
>Cc: kerneljasonxing@gmail.com; netdev@vger.kernel.org; linux-
>kernel@vger.kernel.org; intel-wired-lan@lists.osuosl.org;
>bpf@vger.kernel.org; Jason Xing <kernelxing@tencent.com>
>Subject: [Intel-wired-lan] [PATCH net v2 2/3] i40e: add double of VLAN hea=
der
>when computing the max MTU
>
>From: Jason Xing <kernelxing@tencent.com>
>
>Include the second VLAN HLEN into account when computing the maximum
>MTU size as other drivers do.
>
>Fixes: 0c8493d90b6b ("i40e: add XDP support for pass and drop actions")
>Signed-off-by: Jason Xing <kernelxing@tencent.com>
>---
>v2: drop the duplicate definition
>---
> drivers/net/ethernet/intel/i40e/i40e_main.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>

Tested-by: Chandan Kumar Rout <chandanx.rout@intel.com> (A Contingent Worke=
r at Intel)