On Thu, 20 Mar 2025 16:37:07 +0800 Shaokai Zhang <felix.zh...@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.
