[dpdk-dev] [PATCH v1 2/9] app/procinfo: add compare for new options

[Varghese, Vipin]

HI

<snipped>
> 
> > +/**< Enable port debug. */
> In comment , Enable displaying port info?

ok

> 
> > +static uint32_t enable_dbg_port;
> Do you want to consider replacing "debug" to "show" in code here other
> places? As you have already changed the commands to "show."
> 

If the variable naming has to change, I can do

> 				ring_name = optarg;
> How about doing memcpy instead of pointer assignment? Similarly below.
> Though I don't see issue here. But safe to use memcpy.
> 

I will check this internally, I also think this is safe as it is pointer argv elements. I have taken the same design from

/* Print xstat single value given by name*/
if (!strncmp(long_option[option_index].name,
	"xstats-name", MAX_LONG_OPT_SZ)) {
	enable_xstats_name = 1;
	xstats_name = optarg;

<snipped>