On 10/22/2021 4:20 PM, Ferruh Yigit wrote:
On 10/22/2021 3:10 PM, Jiang, YuX wrote:
-----Original Message-----
From: dev <dev-boun...@dpdk.org> On Behalf Of Ferruh Yigit
Sent: 2021年10月22日 20:57
To: Wang, Haiyue <haiyue.w...@intel.com>; Xing, Beilei
<beilei.x...@intel.com>; Wu, Jingjing <jingjing...@intel.com>; Yang, Qiming
<qiming.y...@intel.com>; Zhang, Qi Z <qi.z.zh...@intel.com>; Hyong Youb
Kim <hyon...@cisco.com>; Michal Krawczyk <m...@semihalf.com>; Andrew
Rybchenko <andrew.rybche...@oktetlabs.ru>; Somnath Kotur
<somnath.ko...@broadcom.com>
Cc: Yigit, Ferruh <ferruh.yi...@intel.com>; dev@dpdk.org
Subject: [dpdk-dev] [PATCH v2 2/2] drivers/net: fix removing jumbo offload
flag
After DEV_RX_OFFLOAD_JUMBO_FRAME flag removed, drivers give jumbo
frame decisions based on MTU value checks, but some of the checks were
wrong by mistake, causing device initialization to fail, fixing them.
Fixes: b563c1421282 ("ethdev: remove jumbo offload flag")
Signed-off-by: Ferruh Yigit <ferruh.yi...@intel.com>
Tested-by: Yu Jiang <yux.ji...@intel.com>
Verify patchset v2
https://patches.dpdk.org/project/dpdk/patch/20211022125716.2706937-2-ferruh.yi...@intel.com/
on dpdk main branch(f4acb429d0 (HEAD, origin/main) hash: promote some
functions to stable)
Tested on i40e(iavf) and it looks good, will test more on others.
Thanks Yu,
I can see it is not fully validated, and most likely it won't be
before -rc1, but it would be good to have this fix for -rc1 to not
block testing.
Since the previous code is clearly wrong, and fixes are similar
between drivers, if one is good I will take risk and merge
the fix for -rc1.
Series applied to dpdk-next-net/main, thanks.
