[PATCH v3 1/3] testpmd: Do not enable mbuf fast release TX offload by default

[Ivan Malov]

On Sun, 3 Aug 2025, Morten Brørup wrote:

> Enabling some offload by default may conflict with a manually configured
> offload.
> Specifically, the mbuf fast release TX offload, which conflicts with multi
> segment packet TX offload, was enabled by default.
> Therefore, mbuf fast release TX offload (the only TX offload which was
> enabled by default) is not enabled by default anymore.
>
> Signed-off-by: Morten Brørup <mb at smartsharesystems.com>

Acked-by: Ivan Malov <ivan.malov at arknetworks.am>

> ---
> app/test-pmd/testpmd.c | 11 +----------
> 1 file changed, 1 insertion(+), 10 deletions(-)
>
> diff --git a/app/test-pmd/testpmd.c b/app/test-pmd/testpmd.c
> index bb88555328..f00eae3992 100644
> --- a/app/test-pmd/testpmd.c
> +++ b/app/test-pmd/testpmd.c
> @@ -490,9 +490,7 @@ lcoreid_t latencystats_lcore_id = -1;
>  */
> struct rte_eth_rxmode rx_mode;
>
> -struct rte_eth_txmode tx_mode = {
> -	.offloads = RTE_ETH_TX_OFFLOAD_MBUF_FAST_FREE,
> -};
> +struct rte_eth_txmode tx_mode;
>
> volatile int test_done = 1; /* stop packet forwarding when set to 1. */
>
> @@ -1592,10 +1590,6 @@ init_config_port_offloads(portid_t pid, uint32_t socket_id)
> 	if (ret != 0)
> 		rte_exit(EXIT_FAILURE, "rte_eth_dev_info_get() failed\n");
>
> -	if (!(port->dev_info.tx_offload_capa & RTE_ETH_TX_OFFLOAD_MBUF_FAST_FREE))
> -		port->dev_conf.txmode.offloads &=
> -			~RTE_ETH_TX_OFFLOAD_MBUF_FAST_FREE;
> -
> 	/* Apply Rx offloads configuration */
> 	for (i = 0; i < port->dev_info.max_rx_queues; i++)
> 		port->rxq[i].conf.offloads = port->dev_conf.rxmode.offloads;
> @@ -2857,9 +2851,6 @@ update_bonding_port_dev_conf(portid_t bond_pid)
> 		return;
> 	}
>
> -	if (port->dev_info.tx_offload_capa & RTE_ETH_TX_OFFLOAD_MBUF_FAST_FREE)
> -		port->dev_conf.txmode.offloads |=
> -				RTE_ETH_TX_OFFLOAD_MBUF_FAST_FREE;
> 	/* Apply Tx offloads configuration */
> 	for (i = 0; i < port->dev_info.max_tx_queues; i++)
> 		port->txq[i].conf.offloads = port->dev_conf.txmode.offloads;
> -- 
> 2.43.0
>