Server will overwrite this anyways (with also a boosted main thread priority/QoS class).
-- v2: ntdll: Do not set main thread QoS class on macOS.
From: Marc-Aurel Zent mzent@codeweavers.com
Server already does so for us. This reverts the following two commits:
Revert "ntdll: Fix runtime availability check for pthread_attr_set_qos_class_np."
This reverts commit e605bc17a5657814a8f9a99e2a94d3222e03ba9d.
Revert "ntdll: Set the QoS class of the main Wine thread on macOS."
This reverts commit ed666db8d8d6afd740a266584a38e08622474821. --- dlls/ntdll/unix/loader.c | 3 --- 1 file changed, 3 deletions(-)
diff --git a/dlls/ntdll/unix/loader.c b/dlls/ntdll/unix/loader.c index effd15c8fe3..34e9c0d87e5 100644 --- a/dlls/ntdll/unix/loader.c +++ b/dlls/ntdll/unix/loader.c @@ -2065,9 +2065,6 @@ static void apple_create_wine_thread( void *arg )
pthread_attr_init( &attr ); pthread_attr_setdetachstate( &attr, PTHREAD_CREATE_JOINABLE ); - /* Use the same QoS class as the process main thread (user-interactive). */ - if (&pthread_attr_set_qos_class_np) - pthread_attr_set_qos_class_np( &attr, QOS_CLASS_USER_INTERACTIVE, 0 ); if (pthread_create( &thread, &attr, apple_wine_thread, NULL )) exit(1); pthread_attr_destroy( &attr ); }
On Thu Oct 23 12:59:14 2025 +0000, Huw Davies wrote:
Any reason not to squash these together? As it stands you'll be leaving things in a broken state after the first commit.
Not really, should be fixed now.