From:   "Rout, ChandanX" <chandanx.rout@intel.com>
To:     Jason Xing <kerneljasonxing@gmail.com>,
        "alexander.duyck@gmail.com" <alexander.duyck@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>,
        Alexander Duyck <alexanderduyck@fb.com>,
        "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 3/3] ixgbe: add double of VLAN
 header when computing the max MTU
Thread-Topic: [Intel-wired-lan] [PATCH net v2 3/3] ixgbe: add double of VLAN
 header when computing the max MTU
Thread-Index: AQHZPDA3iJ4ob7BPJkG8UKvmi2cxcq7NvMqQ
Date:   Tue, 14 Feb 2023 02:19:42 +0000
Message-ID: <MN2PR11MB4045A74C446C281EEB2ECA96EAA29@MN2PR11MB4045.namprd11.prod.outlook.com>
References: <20230209024128.4695-1-kerneljasonxing@gmail.com>
In-Reply-To: <20230209024128.4695-1-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?/MlKeFM1UsxdZoQrWhJ7lAB2eV5xzkfsMPDV+5yzszPAkUz0Cx02RMCAh81o?=
 =?us-ascii?Q?+9Nmlfb1/EFnAY2CDykTI+3jBQvcmc0lnUJWP/8aym3SqiiYKwph2Te7smGb?=
 =?us-ascii?Q?gIxYT9h635Efi3yj5GrX8slt2PVT4WksbK+W2B1izPrNpIjd7aLMvk0z2D7K?=
 =?us-ascii?Q?EJLBJF/WckZ20AAHoXywcPMwN2BGH+kKvcL6GdYvVvV0yCRHPRDLbKRV9APV?=
 =?us-ascii?Q?9NlVQeXzp328fF3TuEaWkOnKk1AbvcK/zTcwrGdavxoZdtdnMNPFVhsFTjET?=
 =?us-ascii?Q?hxZlgJMgIsi5hXqPhYGtltjEVJ9X/aSU/65UxVa02fm//54QprVXm0W9ufyL?=
 =?us-ascii?Q?Sq2+6JntaJk0JY7cfYczJCaqTVAH+PlbpV4GtU4w+CdHr1BJagg6Orc4UJhO?=
 =?us-ascii?Q?0sDe8XyNK4RJJo8CAgOOWW4BFQhBC2g4kVbYNrNU3Wkb6P64mtJdh1OUjowi?=
 =?us-ascii?Q?nf8Vaie1GsGzHfXk7F7XqEhhcmEQOT268w+yIjWNnKe3sotcPYuWaVbi7N8m?=
 =?us-ascii?Q?VPnpCdj44KJobKyEa4ujl6ClohwVYbI6bZ/dB+9o90Lt3tp0qlP6+ep6h/SE?=
 =?us-ascii?Q?1a2fl6L+rRN3rI6ZmSYOrkpdpHTN3Njb9eAet45XJ155bmU4FGdTu2mShCkf?=
 =?us-ascii?Q?ulOQJ43CUswiXMas8ol7cK7PwviAEPL3F7vGYVTNMx+qMTZidjXuFH8WDOw+?=
 =?us-ascii?Q?L2YuzQsjFpgRC065GhyOxr/IKCpJTNwV/h3/sDflA1JTzn9XyLUaGNmMS0x4?=
 =?us-ascii?Q?OL8NTpv9yxm2QPW0xpNY560TS/2d+86cl9zpKvMK3nUrvXyRUVWO7DXEepVC?=
 =?us-ascii?Q?Vyb1q0RTKj+AN7TrAkzEUMJFlzo1Z8KkfuqNIAVema2bWVpYJSA9D6bbfsVl?=
 =?us-ascii?Q?NLbxgs1w98/KIzde592SAMq8EktpgGgj0y6q5eXLE60lPa5EvZFL/srVytMK?=
 =?us-ascii?Q?bDz2+knppJokAZVz4aLLsiWnlTPd0cr8COZlc4VeeuSEFiW1Qv9QCnj2vQW7?=
 =?us-ascii?Q?/mLisCvPm1YCWvBju+UVhp1iJ0Cb2R8ovRisN9FCJ+7ai7SutDVfsB+6Kp0w?=
 =?us-ascii?Q?eL/iBfPIwYBq81IuAqUgW7Mwz/s5E8SQVMcGx0glumSsoHEU5T1hR1h1e4x6?=
 =?us-ascii?Q?CIpbI42W4mccQGUN2t5B6O94jKQ5FVCfqytb9Tws3E6CMeI7VG46tbX/Xvrf?=
 =?us-ascii?Q?CmXI12mhNuJvkaRykgAN6Yt1ScUefnGpKzzg7bV/L3HQhj38nPmChI6AJ//f?=
 =?us-ascii?Q?DrQ13l1xwO4dvljO2Rpp7VFEQuqBBJlPgAaMExwPHWf5ldVyejvonOPRYxVV?=
 =?us-ascii?Q?UWNWHAvnRb7THylTm+4rKi0JNq2a9DOnrTFC6+hLwWaGe9R4y1Ak6Xat6cVm?=
 =?us-ascii?Q?DX9y02eVzkk3/JoIT5/bab2t10AVvVXpGThr1qYx+8SVexCZ3sl2EDOay/Jp?=
 =?us-ascii?Q?6TjZMSbJdyAzs4EdPsOmCpbz1dVsJ6MDGfHY9Mt0mT/mdwenbBUPXMmhNm0x?=
 =?us-ascii?Q?3QiTir16JItCCjgHNGPNpptue4IGHkHijgPb/dLYtVLDVaPxANB7Rwqqo7Fm?=
 =?us-ascii?Q?UdsEm6I0lNAlEFIJA1hjs2lHUEGS5/psBICE4GDC?=
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: c980b725-63dc-4c13-10eb-08db0e31eef7
X-MS-Exchange-CrossTenant-originalarrivaltime: 14 Feb 2023 02:19:42.7787
 (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: KO8IyDegfKR8dLHh2ut0BGXFY8eDMQ13j06gcHUls8PGnRqCvx3QtofX+HAwtyha3Y/5F5CILkX3XZa/1oHe6Q==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DS0PR11MB6327
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: 09 February 2023 08:11
>To: alexander.duyck@gmail.com; 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; Alexander Duyck
><alexanderduyck@fb.com>; 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 3/3] ixgbe: add double of VLAN
>header 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: fabf1bce103a ("ixgbe: Prevent unsupported configurations with XDP")
>Signed-off-by: Jason Xing <kernelxing@tencent.com>
>Reviewed-by: Alexander Duyck <alexanderduyck@fb.com>
>---
>v2: add reviewed-by label.
>---
> drivers/net/ethernet/intel/ixgbe/ixgbe.h      | 2 ++
> drivers/net/ethernet/intel/ixgbe/ixgbe_main.c | 3 +--
> 2 files changed, 3 insertions(+), 2 deletions(-)
>

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