Hi Ferruh, > From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of Ferruh Yigit > Sent: Thursday, January 11, 2018 9:50 AM > > On 11/8/2017 12:41 AM, Thomas Monjalon wrote: > > The compile-time option CONFIG_RTE_LIBRTE_QEDE_VF_TX_SWITCH > > has been added just before the 17.11 release. > > Such new options are forbidden and must be replaced by a run-time > option. > > If it is not replaced in 18.02-rc1, this removal will be applied. > > I hope this is not forgotten!
We'll submit this shortly. Thanks! -Rasesh > > > > > Signed-off-by: Thomas Monjalon <tho...@monjalon.net> > > --- > > config/common_base | 1 - > > drivers/net/qede/qede_ethdev.c | 2 -- > > 2 files changed, 3 deletions(-) > > > > diff --git a/config/common_base b/config/common_base index > > 34f04a9fe..9ae5ec30f 100644 > > --- a/config/common_base > > +++ b/config/common_base > > @@ -415,7 +415,6 @@ CONFIG_RTE_LIBRTE_QEDE_DEBUG_INFO=n > > CONFIG_RTE_LIBRTE_QEDE_DEBUG_DRIVER=n > > CONFIG_RTE_LIBRTE_QEDE_DEBUG_TX=n > > CONFIG_RTE_LIBRTE_QEDE_DEBUG_RX=n > > -CONFIG_RTE_LIBRTE_QEDE_VF_TX_SWITCH=n > > #Provides abs path/name of the firmware file. > > #Empty string denotes driver will use default firmware > > CONFIG_RTE_LIBRTE_QEDE_FW="" > > diff --git a/drivers/net/qede/qede_ethdev.c > > b/drivers/net/qede/qede_ethdev.c index 883214515..0de3a2760 100644 > > --- a/drivers/net/qede/qede_ethdev.c > > +++ b/drivers/net/qede/qede_ethdev.c > > @@ -453,12 +453,10 @@ int qede_activate_vport(struct rte_eth_dev > *eth_dev, bool flg) > > params.update_vport_active_tx_flg = 1; > > params.vport_active_rx_flg = flg; > > params.vport_active_tx_flg = flg; > > -#ifndef RTE_LIBRTE_QEDE_VF_TX_SWITCH > > if (IS_VF(edev)) { > > params.update_tx_switching_flg = 1; > > params.tx_switching_flg = !flg; > > } > > -#endif > > for_each_hwfn(edev, i) { > > p_hwfn = &edev->hwfns[i]; > > params.opaque_fid = p_hwfn->hw_info.opaque_fid; > >