Re: [dpdk-dev] [PATCH v3 6/7] app/proc-info: provide way to request info on owned ports

[Thomas Monjalon <thomas@monjalon.net>]

24/04/2022 07:34, Subendu Santra:
> Hi Stephen,
> 
> We were going through the patch set: https://inbox.dpdk.org/dev/20200715212228.28010-7-stephen@networkplumber.org/ and hoping to get clarification on the behaviour if post mask is not specified in the input to `dpdk-proc-info` tool.
> 
> Specifically, In PATCH v3 6/7, we see this:
> +	/* If no port mask was specified, one will be provided */
> +	if (enabled_port_mask == 0) {
> +		RTE_ETH_FOREACH_DEV(i) {
> +			enabled_port_mask |= 1u << i;
> 
> However, in PATCH v4 8/8, we see this:
> +	/* If no port mask was specified, then show non-owned ports */
> +	if (enabled_port_mask == 0) {
> +		RTE_ETH_FOREACH_DEV(i)
> +			enabled_port_mask = 1ul << i;
> +	}
> 
> Was there any specific reason to show just the last non-owned port in case the port mask was not specified?
> Should we show all non-owned ports in case the user doesn’t specify any port mask?

It looks like a bug. It should be |=
Feel free to send a fix.