[PATCH v17] app/procinfo: display eventdev xstats

Sevincer, Abdullah abdullah.sevincer at intel.com
Sat Jul 8 17:11:45 CEST 2023


>+Has new coverity issue.
>+The reason is the boolean is set every time because it gets every time.

>+Looks like code goes over eventdev_var[] even if no eventdevs are present.
>+Should only look for the number of eventdevs

Thanks Stephen, I will add a condition at the top of the function like:

evdevs = rte_event_dev_count();
	if (!evdevs)
		return 0;

This will ensure if there is no eventdev device function returns.

I will also change the for loop to iterate only with the count of evdevs like:
for (i = 0; i < evdevs; i++) {....... instead of for (i = 0; i < RTE_EVENT_MAX_DEVS; i++) {.

I still need that flag to be set when a user sets a value from command line for a queue or port.
The flag is needed to display and exit from the program.  

if (process_eventdev_xstats())
		return 0;



More information about the dev mailing list