[dpdk-dev] [PATCH v1 2/9] app/procinfo: add compare for new options
Pattan, Reshma
reshma.pattan at intel.com
Thu Oct 25 17:18:48 CEST 2018
> -----Original Message-----
> From: Varghese, Vipin
> +/**< Enable port debug. */
In comment , Enable displaying port info?
> +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."
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.
> + } else if (!strncmp(long_option[option_index].name,
> + "dbg-mempool",
> MAX_LONG_OPT_SZ)) {
> + enable_dbg_mempool = 1;
> + mempool_name = optarg;
> + }
Thanks,
Reshma
More information about the dev
mailing list