On 10/12/21 2:53 AM, Ferruh Yigit wrote: > Both 'rte_eth_dev_configure()' & 'rte_eth_dev_set_mtu()' sets MTU but > have slightly different checks. Like one checks min MTU against > RTE_ETHER_MIN_MTU and other RTE_ETHER_MIN_LEN. > > Checks moved into common function to unify the checks. Also this has > benefit to have common error logs. > > Default 'dev_info->min_mtu' (the one set by ethdev if driver doesn't > provide one), changed to ('RTE_ETHER_MIN_LEN' - overhead). Previously it > was 'RTE_ETHER_MIN_MTU' which is min MTU for IPv4 packets. Since the > intention is to provide min MTU corresponding minimum frame size, new > default value suits better. > > Suggested-by: Huisong Li <lihuis...@huawei.com> > Signed-off-by: Ferruh Yigit <ferruh.yi...@intel.com> > Acked-by: Konstantin Ananyev <konstantin.anan...@intel.com>
Acked-by: Andrew Rybchenko <andrew.rybche...@oktetlabs.ru>