On Mon, Oct 14, 2024 at 9:33 PM Robin Jarry <rja...@redhat.com> wrote:
>
> While invoking telemetry commands (which may happen at any time, out of
> control of the application), an application thread may concurrently
> add/remove ports. The telemetry callbacks may then access partially
> initialized/uninitialised ethdev data.
>
> Reuse the ethdev lock that protects port allocation/destruction and the
> new telemetry callback register api that takes an additional private
> argument. Pass eth_dev_telemetry_do as the main callback and the actual
> endpoint callbacks as private argument.
>
> Fixes: c190daedb9b1 ("ethdev: add telemetry callbacks")
> Cc: sta...@dpdk.org
>
> Signed-off-by: Robin Jarry <rja...@redhat.com>
> Acked-by: Bruce Richardson <bruce.richard...@intel.com>Thanks for the fix Robin, series applied. -- David Marchand
