[PATCH v2 1/1] net/af_packet: register supported param fanout mode

Stephen Hemminger stephen at networkplumber.org
Thu Mar 20 18:45:55 CET 2025


On Thu, 20 Mar 2025 16:37:07 +0800
Shaokai Zhang <felix.zhang at jaguarmicro.com> wrote:

> diff --git a/drivers/net/af_packet/rte_eth_af_packet.c b/drivers/net/af_packet/rte_eth_af_packet.c
> index ab34285ed9..55171944cc 100644
> --- a/drivers/net/af_packet/rte_eth_af_packet.c
> +++ b/drivers/net/af_packet/rte_eth_af_packet.c
> @@ -1172,6 +1172,11 @@ rte_eth_from_packet(struct rte_vdev_device *dev,
>  	PMD_LOG(INFO, "%s:\tblock count %d", name, blockcount);
>  	PMD_LOG(INFO, "%s:\tframe size %d", name, framesize);
>  	PMD_LOG(INFO, "%s:\tframe count %d", name, framecount);
> +	PMD_LOG(INFO, "%s:\tqdisc bypass %d", name, qdisc_bypass);
> +	if (fanout_mode)
> +		PMD_LOG(INFO, "%s:\tfanout mode %s", name, fanout_mode);
> +	else
> +		PMD_LOG(INFO, "%s:\tfanout mode %s", name, "default PACKET_FANOUT_HASH")

All these debug messages are too noisy for my taste.
Should all be removed or made DEBUG level.
There is no point in echoing the arguments in a working driver.


More information about the stable mailing list