[PATCH v2 0/1] MR5167: quartz: Set the name of internal threads.

[Brendan Shanks (＠bshanks)]

--
  v2: quartz: Set the name of internal threads.
 
https://gitlab.winehq.org/wine/wine/-/merge_requests/5167

[PATCH v2 1/1] quartz: Set the name of internal threads.

[Brendan Shanks]

From: Brendan Shanks <bshanks(a)codeweavers.com>

---
 dlls/quartz/filesource.c  | 2 ++
 dlls/quartz/filtergraph.c | 2 ++
 dlls/quartz/systemclock.c | 1 +
 3 files changed, 5 insertions(+)

diff --git a/dlls/quartz/filesource.c b/dlls/quartz/filesource.c
index 7d6292df579..f96555ff78c 100644
--- a/dlls/quartz/filesource.c
+++ b/dlls/quartz/filesource.c
@@ -381,6 +381,8 @@ static DWORD CALLBACK io_thread(void *arg)
     DWORD size;
     BOOL ret;
 
+    SetThreadDescription(GetCurrentThread(), L"wine_qz_async_reader_io");
+
     for (;;)
     {
         ret = GetQueuedCompletionStatus(filter->port, &size, &key, &ovl, INFINITE);
diff --git a/dlls/quartz/filtergraph.c b/dlls/quartz/filtergraph.c
index 4100e71be3a..9b2b2b4e93b 100644
--- a/dlls/quartz/filtergraph.c
+++ b/dlls/quartz/filtergraph.c
@@ -1033,6 +1033,8 @@ static DWORD WINAPI message_thread_run(void *ctx)
 {
     MSG msg;
 
+    SetThreadDescription(GetCurrentThread(), L"wine_qz_graph_worker");
+
     /* Make sure we have a message queue. */
     PeekMessageW(&msg, NULL, 0, 0, PM_NOREMOVE);
     SetEvent(message_thread_ret);
diff --git a/dlls/quartz/systemclock.c b/dlls/quartz/systemclock.c
index 01a8e9fa1ee..244bfd294b6 100644
--- a/dlls/quartz/systemclock.c
+++ b/dlls/quartz/systemclock.c
@@ -144,6 +144,7 @@ static DWORD WINAPI SystemClockAdviseThread(void *param)
     REFERENCE_TIME current_time;
 
     TRACE("Starting advise thread for clock %p.\n", clock);
+    SetThreadDescription(GetCurrentThread(), L"wine_qz_clock_advise");
 
     for (;;)
     {
-- 
GitLab

https://gitlab.winehq.org/wine/wine/-/merge_requests/5167

[Brendan Shanks (＠bshanks)]

On Thu Mar  7 00:26:11 2024 +0000, Zebediah Figura wrote:
> Hm. We don't use the abbreviation elsewhere, but maybe "wine_qtz" or "wine_qz"?
> Simplifying to "wine_qz_async_reader_io" and "wine_qz_graph_worker"
> would also put more semantically interesting parts within the 15
> character limit.
> Thanks, I went with "wine_qz" and also simplified them

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/5167#note_63791

[PATCH v2 0/1] MR5167: quartz: Set the name of internal threads. - approved

[Zebediah Figura (＠zfigura)]

This merge request was approved by Zebediah Figura.
-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/5167