Re: [dpdk-dev] [PATCH] ethdev: forbid MTU set before device configure

2021-11-01 Thread Andrew Rybchenko
Khaparde ; Huisong Li ; Ananyev, Konstantin ; Xu, Rosen Cc: dev@dpdk.org; Ivan Ilchenko Subject: [dpdk-dev] [PATCH] ethdev: forbid MTU set before device configure From: Ivan Ilchenko rte_eth_dev_configure() always sets MTU to either dev_conf.rxmode.mtu or RTE_ETHER_MTU if application doesn&#

Re: [dpdk-dev] [PATCH] ethdev: forbid MTU set before device configure

2021-11-01 Thread Jiang, YuX
Sent: Friday, October 22, 2021 6:18 PM > >>> To: Thomas Monjalon ; Yigit, Ferruh > >>> ; Ajit Khaparde > >>> ; Huisong Li ; > >>> Ananyev, Konstantin ; Xu, Rosen > >>> > >>> Cc: dev@dpdk.org; Ivan Ilchenko > >>>

Re: [dpdk-dev] [PATCH] ethdev: forbid MTU set before device configure

2021-10-29 Thread Ferruh Yigit
: dev@dpdk.org; Ivan Ilchenko Subject: [dpdk-dev] [PATCH] ethdev: forbid MTU set before device configure From: Ivan Ilchenko rte_eth_dev_configure() always sets MTU to either dev_conf.rxmode.mtu or RTE_ETHER_MTU if application doesn't provide the value. So, there is no point to

Re: [dpdk-dev] [PATCH] ethdev: forbid MTU set before device configure

2021-10-28 Thread Andrew Rybchenko
-dev] [PATCH] ethdev: forbid MTU set before device configure From: Ivan Ilchenko rte_eth_dev_configure() always sets MTU to either dev_conf.rxmode.mtu or RTE_ETHER_MTU if application doesn't provide the value. So, there is no point to allow rte_eth_dev_set_mtu() before since set value wi

Re: [dpdk-dev] [PATCH] ethdev: forbid MTU set before device configure

2021-10-28 Thread Jiang, YuX
: [dpdk-dev] [PATCH] ethdev: forbid MTU set before device > configure > > From: Ivan Ilchenko > > rte_eth_dev_configure() always sets MTU to either dev_conf.rxmode.mtu > or RTE_ETHER_MTU if application doesn't provide the value. > So, there is no point to allow rte_eth_dev_s

Re: [dpdk-dev] [PATCH] ethdev: forbid MTU set before device configure

2021-10-22 Thread Ferruh Yigit
On 10/22/2021 2:21 PM, Ferruh Yigit wrote: On 10/22/2021 11:18 AM, Andrew Rybchenko wrote: From: Ivan Ilchenko rte_eth_dev_configure() always sets MTU to either dev_conf.rxmode.mtu or RTE_ETHER_MTU if application doesn't provide the value. So, there is no point to allow rte_eth_dev_set_mtu() b

Re: [dpdk-dev] [PATCH] ethdev: forbid MTU set before device configure

2021-10-22 Thread Ferruh Yigit
On 10/22/2021 11:18 AM, Andrew Rybchenko wrote: From: Ivan Ilchenko rte_eth_dev_configure() always sets MTU to either dev_conf.rxmode.mtu or RTE_ETHER_MTU if application doesn't provide the value. So, there is no point to allow rte_eth_dev_set_mtu() before since set value will be overwritten on

[dpdk-dev] [PATCH] ethdev: forbid MTU set before device configure

2021-10-22 Thread Andrew Rybchenko
From: Ivan Ilchenko rte_eth_dev_configure() always sets MTU to either dev_conf.rxmode.mtu or RTE_ETHER_MTU if application doesn't provide the value. So, there is no point to allow rte_eth_dev_set_mtu() before since set value will be overwritten on configure anyway. Fixes: 1bb4a528c41f ("ethdev: