From dbb69deea72e18a5f77521734ad8b6fb36ef8faa Mon Sep 17 00:00:00 2001
From: Huo Yaoyuan <huoyaoyuan@hotmail.com>
Date: Thu, 22 May 2025 14:01:09 +0800
Subject: [PATCH 1/2] Use minipal_lowres_ticks for paltests

---
 .../tests/palsuite/threading/Sleep/test1/Sleep.cpp | 14 ++++----------
 .../tests/palsuite/threading/Sleep/test2/sleep.cpp | 14 ++++----------
 .../palsuite/threading/SleepEx/test1/test1.cpp     | 14 ++++----------
 .../palsuite/threading/SleepEx/test2/test2.cpp     | 14 ++++----------
 .../WaitForMultipleObjectsEx/test2/test2.cpp       | 14 ++++----------
 .../WFSOExMutexTest/WFSOExMutexTest.cpp            | 14 ++++----------
 .../WFSOExSemaphoreTest/WFSOExSemaphoreTest.cpp    | 14 ++++----------
 .../WFSOExThreadTest/WFSOExThreadTest.cpp          | 14 ++++----------
 8 files changed, 32 insertions(+), 80 deletions(-)

diff --git a/src/coreclr/pal/tests/palsuite/threading/Sleep/test1/Sleep.cpp b/src/coreclr/pal/tests/palsuite/threading/Sleep/test1/Sleep.cpp
index 3cc3d92448282d..765c1bd6dd0a37 100644
--- a/src/coreclr/pal/tests/palsuite/threading/Sleep/test1/Sleep.cpp
+++ b/src/coreclr/pal/tests/palsuite/threading/Sleep/test1/Sleep.cpp
@@ -32,8 +32,8 @@ PALTEST(threading_Sleep_test1_paltest_sleep_test1, "threading/Sleep/test1/paltes
     /* Milliseconds of error which are acceptable Function execution time, etc. */
     DWORD AcceptableTimeError = 150;
 
-    UINT64 OldTimeStamp;
-    UINT64 NewTimeStamp;
+    int64_t OldTimeStamp;
+    int64_t NewTimeStamp;
     DWORD MaxDelta;
     DWORD TimeDelta;
     DWORD i;
@@ -43,17 +43,11 @@ PALTEST(threading_Sleep_test1_paltest_sleep_test1, "threading/Sleep/test1/paltes
         return ( FAIL );
     }
 
-    LARGE_INTEGER performanceFrequency;
-    if (!QueryPerformanceFrequency(&performanceFrequency))
-    {
-        return FAIL;
-    }
-
     for( i = 0; i < sizeof(SleepTimes) / sizeof(DWORD); i++)
     {
-        OldTimeStamp = GetHighPrecisionTimeStamp(performanceFrequency);
+        OldTimeStamp = minipal_lowres_ticks();
         Sleep(SleepTimes[i]);
-        NewTimeStamp = GetHighPrecisionTimeStamp(performanceFrequency);
+        NewTimeStamp = minipal_lowres_ticks();
 
         TimeDelta = NewTimeStamp - OldTimeStamp;
 
diff --git a/src/coreclr/pal/tests/palsuite/threading/Sleep/test2/sleep.cpp b/src/coreclr/pal/tests/palsuite/threading/Sleep/test2/sleep.cpp
index 6fb148b681f2fd..838f8ab5c96dd1 100644
--- a/src/coreclr/pal/tests/palsuite/threading/Sleep/test2/sleep.cpp
+++ b/src/coreclr/pal/tests/palsuite/threading/Sleep/test2/sleep.cpp
@@ -31,8 +31,8 @@ PALTEST(threading_Sleep_test2_paltest_sleep_test2, "threading/Sleep/test2/paltes
     /* Milliseconds of error which are acceptable Function execution time, etc. */
     DWORD AcceptableTimeError = 150;
 
-    UINT64 OldTimeStamp;
-    UINT64 NewTimeStamp;
+    int64_t OldTimeStamp;
+    int64_t NewTimeStamp;
     DWORD MaxDelta;
     DWORD TimeDelta;
     DWORD i;
@@ -42,17 +42,11 @@ PALTEST(threading_Sleep_test2_paltest_sleep_test2, "threading/Sleep/test2/paltes
         return ( FAIL );
     }
 
-    LARGE_INTEGER performanceFrequency;
-    if (!QueryPerformanceFrequency(&performanceFrequency))
-    {
-        return FAIL;
-    }
-
     for( i = 0; i < sizeof(SleepTimes) / sizeof(DWORD); i++)
     {
-        OldTimeStamp = GetHighPrecisionTimeStamp(performanceFrequency);
+        OldTimeStamp = minipal_lowres_ticks();
         Sleep(SleepTimes[i]);
-        NewTimeStamp = GetHighPrecisionTimeStamp(performanceFrequency);
+        NewTimeStamp = minipal_lowres_ticks();
 
         TimeDelta = NewTimeStamp - OldTimeStamp;
 
diff --git a/src/coreclr/pal/tests/palsuite/threading/SleepEx/test1/test1.cpp b/src/coreclr/pal/tests/palsuite/threading/SleepEx/test1/test1.cpp
index 61bdf136b8f8a3..94abcf5b0b38de 100644
--- a/src/coreclr/pal/tests/palsuite/threading/SleepEx/test1/test1.cpp
+++ b/src/coreclr/pal/tests/palsuite/threading/SleepEx/test1/test1.cpp
@@ -39,8 +39,8 @@ PALTEST(threading_SleepEx_test1_paltest_sleepex_test1, "threading/SleepEx/test1/
         {2000, TRUE},
     };
 
-    UINT64 OldTimeStamp;
-    UINT64 NewTimeStamp;
+    int64_t OldTimeStamp;
+    int64_t NewTimeStamp;
     DWORD MaxDelta;
     DWORD TimeDelta;
     DWORD i;
@@ -50,19 +50,13 @@ PALTEST(threading_SleepEx_test1_paltest_sleepex_test1, "threading/SleepEx/test1/
         return FAIL;
     }
 
-    LARGE_INTEGER performanceFrequency;
-    if (!QueryPerformanceFrequency(&performanceFrequency))
-    {
-        return FAIL;
-    }
-
     for (i = 0; i<sizeof(testCases) / sizeof(testCases[0]); i++)
     {
-        OldTimeStamp = GetHighPrecisionTimeStamp(performanceFrequency);
+        OldTimeStamp = minipal_lowres_ticks();
 
         SleepEx(testCases[i].SleepTime, testCases[i].Alertable);
 
-        NewTimeStamp = GetHighPrecisionTimeStamp(performanceFrequency);
+        NewTimeStamp = minipal_lowres_ticks();
 
         TimeDelta = NewTimeStamp - OldTimeStamp;
 
diff --git a/src/coreclr/pal/tests/palsuite/threading/SleepEx/test2/test2.cpp b/src/coreclr/pal/tests/palsuite/threading/SleepEx/test2/test2.cpp
index b139e2d85bc76a..61132f0e6021f4 100644
--- a/src/coreclr/pal/tests/palsuite/threading/SleepEx/test2/test2.cpp
+++ b/src/coreclr/pal/tests/palsuite/threading/SleepEx/test2/test2.cpp
@@ -155,25 +155,19 @@ VOID PALAPI APCFunc_SleepEx_test2(ULONG_PTR dwParam)
 /* Entry Point for child thread. */
 DWORD PALAPI SleeperProc_SleepEx_test2(LPVOID lpParameter)
 {
-    UINT64 OldTimeStamp;
-    UINT64 NewTimeStamp;
+    int64_t OldTimeStamp;
+    int64_t NewTimeStamp;
     BOOL Alertable;
     DWORD ret;
 
     Alertable = (BOOL)(SIZE_T) lpParameter;
 
-    LARGE_INTEGER performanceFrequency;
-    if (!QueryPerformanceFrequency(&performanceFrequency))
-    {
-        return FAIL;
-    }
-
-    OldTimeStamp = GetHighPrecisionTimeStamp(performanceFrequency);
+    OldTimeStamp = minipal_lowres_ticks();
     s_preWaitTimestampRecorded = true;
 
     ret = SleepEx(ChildThreadSleepTime, Alertable);
     
-    NewTimeStamp = GetHighPrecisionTimeStamp(performanceFrequency);
+    NewTimeStamp = minipal_lowres_ticks();
 
     if (Alertable && ret != WAIT_IO_COMPLETION)
     {
diff --git a/src/coreclr/pal/tests/palsuite/threading/WaitForMultipleObjectsEx/test2/test2.cpp b/src/coreclr/pal/tests/palsuite/threading/WaitForMultipleObjectsEx/test2/test2.cpp
index 96124434e768f4..ca8941458c25db 100644
--- a/src/coreclr/pal/tests/palsuite/threading/WaitForMultipleObjectsEx/test2/test2.cpp
+++ b/src/coreclr/pal/tests/palsuite/threading/WaitForMultipleObjectsEx/test2/test2.cpp
@@ -140,8 +140,8 @@ VOID PALAPI APCFunc_WFMO_test2(ULONG_PTR dwParam)
 DWORD PALAPI WaiterProc_WFMO_test2(LPVOID lpParameter)
 {
     HANDLE Semaphore;
-    UINT64 OldTimeStamp;
-    UINT64 NewTimeStamp;
+    int64_t OldTimeStamp;
+    int64_t NewTimeStamp;
     BOOL Alertable;
     DWORD ret;
 
@@ -156,19 +156,13 @@ DWORD PALAPI WaiterProc_WFMO_test2(LPVOID lpParameter)
 
     Alertable = (BOOL)(SIZE_T) lpParameter;
 
-    LARGE_INTEGER performanceFrequency;
-    if (!QueryPerformanceFrequency(&performanceFrequency))
-    {
-        Fail("Failed to query performance frequency!");
-    }
-
-    OldTimeStamp = GetHighPrecisionTimeStamp(performanceFrequency);
+    OldTimeStamp = minipal_lowres_ticks();
     s_preWaitTimestampRecorded = true;
 
     ret = WaitForMultipleObjectsEx(1, &Semaphore, FALSE, ChildThreadWaitTime,
         Alertable);
 
-    NewTimeStamp = GetHighPrecisionTimeStamp(performanceFrequency);
+    NewTimeStamp = minipal_lowres_ticks();
 
 
     if (Alertable && ret != WAIT_IO_COMPLETION)
diff --git a/src/coreclr/pal/tests/palsuite/threading/WaitForSingleObject/WFSOExMutexTest/WFSOExMutexTest.cpp b/src/coreclr/pal/tests/palsuite/threading/WaitForSingleObject/WFSOExMutexTest/WFSOExMutexTest.cpp
index c98659db838e25..764bac05729039 100644
--- a/src/coreclr/pal/tests/palsuite/threading/WaitForSingleObject/WFSOExMutexTest/WFSOExMutexTest.cpp
+++ b/src/coreclr/pal/tests/palsuite/threading/WaitForSingleObject/WFSOExMutexTest/WFSOExMutexTest.cpp
@@ -177,27 +177,21 @@ VOID PALAPI APCFunc_WFSOExMutexTest(ULONG_PTR dwParam)
 /* Entry Point for child thread. */
 DWORD PALAPI WaiterProc_WFSOExMutexTest(LPVOID lpParameter)
 {
-    UINT64 OldTimeStamp;
-    UINT64 NewTimeStamp;
+    int64_t OldTimeStamp;
+    int64_t NewTimeStamp;
     BOOL Alertable;
     DWORD ret;
 
     Alertable = (BOOL)(SIZE_T) lpParameter;
 
-    LARGE_INTEGER performanceFrequency;
-    if (!QueryPerformanceFrequency(&performanceFrequency))
-    {
-        Fail("Failed to query performance frequency!");
-    }
-
-    OldTimeStamp = GetHighPrecisionTimeStamp(performanceFrequency);
+    OldTimeStamp = minipal_lowres_ticks();
     s_preWaitTimestampRecorded = true;
 
     ret = WaitForSingleObjectEx(	hMutex_WFSOExMutexTest, 
 								ChildThreadWaitTime, 
         							Alertable);
     
-    NewTimeStamp = GetHighPrecisionTimeStamp(performanceFrequency);
+    NewTimeStamp = minipal_lowres_ticks();
 
     if (Alertable && ret != WAIT_IO_COMPLETION)
     {
diff --git a/src/coreclr/pal/tests/palsuite/threading/WaitForSingleObject/WFSOExSemaphoreTest/WFSOExSemaphoreTest.cpp b/src/coreclr/pal/tests/palsuite/threading/WaitForSingleObject/WFSOExSemaphoreTest/WFSOExSemaphoreTest.cpp
index 3859e4abd0717d..12ac1ffd37a7a7 100644
--- a/src/coreclr/pal/tests/palsuite/threading/WaitForSingleObject/WFSOExSemaphoreTest/WFSOExSemaphoreTest.cpp
+++ b/src/coreclr/pal/tests/palsuite/threading/WaitForSingleObject/WFSOExSemaphoreTest/WFSOExSemaphoreTest.cpp
@@ -133,17 +133,11 @@ VOID PALAPI APCFunc_WFSOExSemaphoreTest(ULONG_PTR dwParam)
 DWORD PALAPI WaiterProc_WFSOExSemaphoreTest(LPVOID lpParameter)
 {
     HANDLE hSemaphore;
-    UINT64 OldTimeStamp;
-    UINT64 NewTimeStamp;
+    int64_t OldTimeStamp;
+    int64_t NewTimeStamp;
     BOOL Alertable;
     DWORD ret;
 
-    LARGE_INTEGER performanceFrequency;
-    if (!QueryPerformanceFrequency(&performanceFrequency))
-    {
-        Fail("Failed to query performance frequency!");
-    }
-
     /* Create a semaphore that is not in the signalled state */
     hSemaphore = CreateSemaphoreExW(NULL, 0, 1, NULL, 0, 0);
 
@@ -155,14 +149,14 @@ DWORD PALAPI WaiterProc_WFSOExSemaphoreTest(LPVOID lpParameter)
 
     Alertable = (BOOL)(SIZE_T) lpParameter;
 
-    OldTimeStamp = GetHighPrecisionTimeStamp(performanceFrequency);
+    OldTimeStamp = minipal_lowres_ticks();
     s_preWaitTimestampRecorded = true;
 
     ret = WaitForSingleObjectEx(	hSemaphore,
 								ChildThreadWaitTime,
         							Alertable);
 
-    NewTimeStamp = GetHighPrecisionTimeStamp(performanceFrequency);
+    NewTimeStamp = minipal_lowres_ticks();
 
 
     if (Alertable && ret != WAIT_IO_COMPLETION)
diff --git a/src/coreclr/pal/tests/palsuite/threading/WaitForSingleObject/WFSOExThreadTest/WFSOExThreadTest.cpp b/src/coreclr/pal/tests/palsuite/threading/WaitForSingleObject/WFSOExThreadTest/WFSOExThreadTest.cpp
index 078ae531353ac7..5f308951f0e32e 100644
--- a/src/coreclr/pal/tests/palsuite/threading/WaitForSingleObject/WFSOExThreadTest/WFSOExThreadTest.cpp
+++ b/src/coreclr/pal/tests/palsuite/threading/WaitForSingleObject/WFSOExThreadTest/WFSOExThreadTest.cpp
@@ -136,8 +136,8 @@ VOID PALAPI APCFunc_WFSOExThreadTest(ULONG_PTR dwParam)
 DWORD PALAPI WaiterProc_WFSOExThreadTest(LPVOID lpParameter)
 {
     HANDLE hWaitThread;
-    UINT64 OldTimeStamp;
-    UINT64 NewTimeStamp;
+    int64_t OldTimeStamp;
+    int64_t NewTimeStamp;
     BOOL Alertable;
     DWORD ret;
     DWORD dwThreadId = 0;
@@ -163,20 +163,14 @@ satisfying any threads that were waiting on the object.
 
     Alertable = (BOOL)(SIZE_T) lpParameter;
 
-    LARGE_INTEGER performanceFrequency;
-    if (!QueryPerformanceFrequency(&performanceFrequency))
-    {
-        Fail("Failed to query performance frequency!");
-    }
-
-    OldTimeStamp = GetHighPrecisionTimeStamp(performanceFrequency);
+    OldTimeStamp = minipal_lowres_ticks();
     s_preWaitTimestampRecorded = true;
 
     ret = WaitForSingleObjectEx(	hWaitThread, 
 								ChildThreadWaitTime, 
         							Alertable);
     
-    NewTimeStamp = GetHighPrecisionTimeStamp(performanceFrequency);
+    NewTimeStamp = minipal_lowres_ticks();
 
 
     if (Alertable && ret != WAIT_IO_COMPLETION)

From 8a876cec08d37d8e78aa1e99ced5fa617020e5ae Mon Sep 17 00:00:00 2001
From: Huo Yaoyuan <huoyaoyuan@hotmail.com>
Date: Thu, 22 May 2025 14:01:28 +0800
Subject: [PATCH 2/2] Delete unused GetHighPrecisionTimeStamp

---
 src/coreclr/pal/tests/palsuite/common/palsuite.cpp | 5 -----
 src/coreclr/pal/tests/palsuite/common/palsuite.h   | 1 -
 2 files changed, 6 deletions(-)

diff --git a/src/coreclr/pal/tests/palsuite/common/palsuite.cpp b/src/coreclr/pal/tests/palsuite/common/palsuite.cpp
index 030d8cc13cd6d6..914aa6b9cacb34 100644
--- a/src/coreclr/pal/tests/palsuite/common/palsuite.cpp
+++ b/src/coreclr/pal/tests/palsuite/common/palsuite.cpp
@@ -52,11 +52,6 @@ char* convertC(const WCHAR * wString)
     return MultiBuffer;
 }
 
-UINT64 GetHighPrecisionTimeStamp(LARGE_INTEGER performanceFrequency)
-{
-    return (UINT64)minipal_hires_ticks();
-}
-
 static const char* rgchPathDelim = "/";
 
 
diff --git a/src/coreclr/pal/tests/palsuite/common/palsuite.h b/src/coreclr/pal/tests/palsuite/common/palsuite.h
index ed5766963533e8..f80107e33ee545 100644
--- a/src/coreclr/pal/tests/palsuite/common/palsuite.h
+++ b/src/coreclr/pal/tests/palsuite/common/palsuite.h
@@ -163,7 +163,6 @@ inline ULONG   VAL32(ULONG x)
 
 WCHAR* convert(const char * aString);
 char* convertC(const WCHAR * wString);
-UINT64 GetHighPrecisionTimeStamp(LARGE_INTEGER performanceFrequency);
 
 extern const char* szTextFile;