On 08/22, Jakub Kicinski wrote:
> In case changing channel count with provider bound succeeds
> unexpectedly - make sure we return to original settings.

Acked-by: Stanislav Fomichev <[email protected]>

> Signed-off-by: Jakub Kicinski <[email protected]>
> ---
>  .../selftests/drivers/net/hw/ncdevmem.c       | 34 ++++++++++++-------
>  1 file changed, 22 insertions(+), 12 deletions(-)
> 
> diff --git a/tools/testing/selftests/drivers/net/hw/ncdevmem.c 
> b/tools/testing/selftests/drivers/net/hw/ncdevmem.c
> index 8d9d579834b1..580b4459a840 100644
> --- a/tools/testing/selftests/drivers/net/hw/ncdevmem.c
> +++ b/tools/testing/selftests/drivers/net/hw/ncdevmem.c
> @@ -469,7 +469,7 @@ static void reset_rss(void)
>       run_command("ethtool -X %s default >&2", ifname, start_queue);
>  }
>  
> -static int configure_channels(unsigned int rx, unsigned int tx)
> +static int check_changing_channels(unsigned int rx, unsigned int tx)

Good call on renaming!

Reply via email to