Re: [PATCH ethtool-next v2] rxclass: Make output for RSS context action explicit

[Jakub Kicinski <kuba@kernel.org>]

On Mon, 11 Nov 2024 12:05:38 -0700 Daniel Xu wrote:
> -	if (fsp->flow_type & FLOW_RSS)
> -		fprintf(stdout, "\tRSS Context ID: %u\n", rss_context);
> -
>  	if (fsp->ring_cookie == RX_CLS_FLOW_DISC) {
>  		fprintf(stdout, "\tAction: Drop\n");
>  	} else if (fsp->ring_cookie == RX_CLS_FLOW_WAKE) {
>  		fprintf(stdout, "\tAction: Wake-on-LAN\n");
> +	} else if (fsp->flow_type & FLOW_RSS) {
> +		u64 queue = ethtool_get_flow_spec_ring(fsp->ring_cookie);
> +
> +		fprintf(stdout, "\tAction: Direct to RSS context id %u", rss_context);

Do you have strong feelings about the change in formatting?
Looking at Ed's comment on the new test made me wonder if the change 
in capitalization is for the better.

Action: Direct to RSS context id 1 (queue base offset: 2)

vs

Action: Direct to RSS Context ID: 1 (queue base offset: 2)

Given "id" is a word (: I like the ID format, the extra colon is
annoying but OTOH if we retain it your regexp in the other patch
would match before and after..

Actually the best formatting IMHO would be to skip the ID altogether:

Action: Direct to RSS Context 1 (queue base offset: 2)

So please respin this, either set the ID to upper case or skip it.
And depending on the decision here respin the test (feel free to
repost before 24h passes, if you do).