mbox

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

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

Message

Tyler Retzlaff Jan. 24, 2023, 5:42 p.m. UTC
  v8:
  * document that no implementation or internal implementation
    is a noop for rte_thread_set_name
  * update commit message to indicate windows now provides an
    implementation for rte_thread_set_name
  * remove '_' from thread_name in comment
  * squish drivers/mlx5 patch into patch 1 as requested.
  * clarify rte_thread_setname anticipated deprecation and removal
    releases in deprecation notice
  * group deprecation notice with other eal deprecation notices

v7:
  * don't dereference thread_name after successful completion
    of mbsrtowcs. (fixes NULL dereference on Windows).

v6:
  * clean up commit descriptions
  * add patch to set worker thread name on windows
  * remove __rte_deprecated from rte_thread_setname

v5:
  * rename rte_thread_getname -> thread_get_name

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

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

Tyler Retzlaff (4):
  eal: add thread set name API operating on rte thread
  eal: remove thread getname API
  eal: set thread name on Windows worker threads
  eal: deprecation notice for rte thread setname API

 doc/guides/rel_notes/deprecation.rst |  5 +++++
 drivers/net/mlx5/mlx5_hws_cnt.c      |  3 ++-
 drivers/vdpa/mlx5/mlx5_vdpa_event.c  |  3 +--
 lib/eal/common/eal_common_thread.c   |  8 ++-----
 lib/eal/common/eal_common_trace.c    | 15 ++++++++++++-
 lib/eal/freebsd/eal.c                |  3 +--
 lib/eal/freebsd/eal_thread.c         | 20 +++++++++--------
 lib/eal/include/rte_lcore.h          | 17 --------------
 lib/eal/include/rte_thread.h         | 20 +++++++++++++++++
 lib/eal/linux/eal.c                  |  6 +----
 lib/eal/linux/eal_thread.c           | 29 +++++++++++++++---------
 lib/eal/version.map                  |  4 +++-
 lib/eal/windows/eal.c                |  7 ++++++
 lib/eal/windows/rte_thread.c         | 43 ++++++++++++++++++++++++++++++++++++
 14 files changed, 128 insertions(+), 55 deletions(-)