[PATCH] sched: fix for demo failure in debug mode

[PATCH] sched: fix for demo failure in debug mode

[Megha Ajmera]

This issue is happening due to non-initialization of some
fields in “rte_eth_rxconf” structure in our application.
Doing a memset to zero before initialization in HQoS application.

Signed-off-by: Megha Ajmera <megha.ajmera@intel.com>
---
 examples/qos_sched/init.c | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/examples/qos_sched/init.c b/examples/qos_sched/init.c
index 0709aec10c..d28350e14a 100644
--- a/examples/qos_sched/init.c
+++ b/examples/qos_sched/init.c
@@ -79,6 +79,7 @@ app_init_port(uint16_t portid, struct rte_mempool *mp)
 	if (app_inited_port_mask & (1u << portid))
 		return 0;
 
+	memset(&rx_conf, 0, sizeof(struct rte_eth_rxconf));
 	rx_conf.rx_thresh.pthresh = rx_thresh.pthresh;
 	rx_conf.rx_thresh.hthresh = rx_thresh.hthresh;
 	rx_conf.rx_thresh.wthresh = rx_thresh.wthresh;
@@ -86,6 +87,7 @@ app_init_port(uint16_t portid, struct rte_mempool *mp)
 	rx_conf.rx_drop_en = 0;
 	rx_conf.rx_deferred_start = 0;
 
+	memset(&tx_conf, 0, sizeof(struct rte_eth_txconf));
 	tx_conf.tx_thresh.pthresh = tx_thresh.pthresh;
 	tx_conf.tx_thresh.hthresh = tx_thresh.hthresh;
 	tx_conf.tx_thresh.wthresh = tx_thresh.wthresh;
-- 
2.25.1

RE: [PATCH] sched: fix for demo failure in debug mode

[Dumitrescu, Cristian]

> -----Original Message-----
> From: Ajmera, Megha <megha.ajmera@intel.com>
> Sent: Thursday, January 5, 2023 12:29 PM
> To: dev@dpdk.org; Singh, Jasvinder <jasvinder.singh@intel.com>; Dumitrescu,
> Cristian <cristian.dumitrescu@intel.com>
> Cc: stable@dpdk.org
> Subject: [PATCH] sched: fix for demo failure in debug mode
> 
> This issue is happening due to non-initialization of some
> fields in “rte_eth_rxconf” structure in our application.
> Doing a memset to zero before initialization in HQoS application.
> 
> Signed-off-by: Megha Ajmera <megha.ajmera@intel.com>
> ---
>  examples/qos_sched/init.c | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/examples/qos_sched/init.c b/examples/qos_sched/init.c
> index 0709aec10c..d28350e14a 100644
> --- a/examples/qos_sched/init.c
> +++ b/examples/qos_sched/init.c


Acked-by: Cristian Dumitrescu <cristian.dumitrescu@intel.com>

Re: [PATCH] sched: fix for demo failure in debug mode

[Thomas Monjalon]

> > This issue is happening due to non-initialization of some
> > fields in “rte_eth_rxconf” structure in our application.
> > Doing a memset to zero before initialization in HQoS application.
> > 
> > Signed-off-by: Megha Ajmera <megha.ajmera@intel.com>
> > ---
> >  examples/qos_sched/init.c | 2 ++
> >  1 file changed, 2 insertions(+)
> > 
> > diff --git a/examples/qos_sched/init.c b/examples/qos_sched/init.c
> > index 0709aec10c..d28350e14a 100644
> > --- a/examples/qos_sched/init.c
> > +++ b/examples/qos_sched/init.c
> 
> Acked-by: Cristian Dumitrescu <cristian.dumitrescu@intel.com>

Fixes: de3cfa2c9823 ("sched: initial import")
Cc: stable@dpdk.org

Applied, thanks.