mbox

[v4,0/4] add rte_thread_set_name API for rte_thread_t

Message ID 1673635923-12432-1-git-send-email-roretzla@linux.microsoft.com (mailing list archive)
Headers

Message

Tyler Retzlaff Jan. 13, 2023, 6:51 p.m. UTC 
  Replace the rte_thread_setname API which operates on pthread_t with
rte_thread_set_name that operates on rte_thread_t.

We should try to align tracing output from the EAL for all platforms
but in this case we are retaining an exception for linux as requested
from the community.

v4:
  * retain and move rte_thread_getname function to the
    single site of use

v3:
  * fix coding style error
  * move remove of rte_thread_getname to patch #2

v2:
  * initial series provided get/set for lcore thread id, those
    additions have been removed as per discussion. including
    unit test
  * add a single api rte_thread_set_name does not fail but emits
    debug logging if the internal implementation is aware of
    in-exact use of the name or failure to set the name
  * adapt mlx5 driver to avoid use of deprecated API


Tyler Retzlaff (4):
  eal: add thread set name API operating on rte thread
  eal: remove thread getname API
  drivers: mlx5 use rte thread set name
  eal: deprecate rte thread setname API

 doc/guides/rel_notes/deprecation.rst |  4 ++++
 drivers/net/mlx5/mlx5_hws_cnt.c      |  3 ++-
 drivers/vdpa/mlx5/mlx5_vdpa_event.c  |  3 +--
 lib/eal/common/eal_common_thread.c   |  9 +++----
 lib/eal/common/eal_common_trace.c    | 15 +++++++++++-
 lib/eal/freebsd/eal.c                |  4 +++-
 lib/eal/freebsd/eal_thread.c         | 20 +++++++++-------
 lib/eal/include/rte_lcore.h          | 19 ++-------------
 lib/eal/include/rte_thread.h         | 17 +++++++++++++
 lib/eal/linux/eal.c                  |  8 +++----
 lib/eal/linux/eal_thread.c           | 29 ++++++++++++++---------
 lib/eal/version.map                  |  4 +++-
 lib/eal/windows/rte_thread.c         | 46 ++++++++++++++++++++++++++++++++++++
 13 files changed, 127 insertions(+), 54 deletions(-)

Series-acked-by: Morten Brørup <mb@smartsharesystems.com>