Re: [PATCH] app/testpmd: fix flow transfer proxy port handling

[Ferruh Yigit]

On 11/17/2021 7:41 AM, Slava Ovsiienko wrote:

-----Original Message-----
From: Ivan Malov <ivan.ma...@oktetlabs.ru>
Sent: Tuesday, November 16, 2021 17:38
To: dev@dpdk.org
Cc: David Marchand <david.march...@redhat.com>; Ferruh Yigit
<ferruh.yi...@intel.com>; NBU-Contact-Thomas Monjalon
<tho...@monjalon.net>; Andrew Rybchenko
<andrew.rybche...@oktetlabs.ru>; Xiaoyun Li <xiaoyun...@intel.com>; Ori
Kam <or...@nvidia.com>
Subject: [PATCH] app/testpmd: fix flow transfer proxy port handling

The current approach detects the proxy port on each port (re-)plug and may
spam the log with error messages if the PMD does not support flows.
As testpmd is a debug tool, it must not do such implicit port handling.
Instead, the new API should be called only when the user requests that.

Revoke the existing code. Implement an explicit command-line primitive to let
the user find the proxy port themselves. Provide relevant hints.

Fixes: 1179f05cc9a0 ("ethdev: query proxy port to manage transfer flows")

Signed-off-by: Ivan Malov <ivan.ma...@oktetlabs.ru>
Reviewed-by: Andrew Rybchenko <andrew.rybche...@oktetlabs.ru>

Acked-by: Viacheslav Ovsiienko <viachesl...@nvidia.com>

Applied to dpdk-next-net/main, thanks.