test(regression): removed stale Windows accommodations

Removed Windows-specific timer-thread cleanup from testcontrol after the port handled stale host-thread teardown directly.

Restored stricter event flag, sleep, and timer expectations where port fixes made the previous Windows accommodations unnecessary.

Co-authored-by: Codex (gpt 5.5) <codex@openai.com>

Author: fdesbiens

test/smp/regression/threadx_event_flag_isr_set_clear_test.c

@@ -254,7 +254,7 @@ ULONG   actual;
         {
 
             /* Test error!  */
-            printf("ERROR #8 (%u %lu %lu)\n", status, condition_count, event_flags_set_counter);
+            printf("ERROR #8\n");
             test_control_return(1);
         }
 

test/smp/regression/threadx_event_flag_suspension_timeout_test.c

@@ -179,9 +179,7 @@ UINT    status;
     tx_thread_sleep(63);
 
     /* Check the run counters.  */
-    /* Host scheduling jitter can shift one extra round in these timeout loops.  */
-    if (((thread_1_counter < 32UL) || (thread_1_counter > 33UL)) ||
-        ((thread_2_counter < 13UL) || (thread_2_counter > 14UL)))
+    if ((thread_1_counter != 33) || (thread_2_counter != 13))
     {
 
         /* Event flag error.  */

test/tx/regression/testcontrol.c

@@ -430,37 +430,8 @@ static void   init_timer_entry(ULONG timer_input)
 
 void delete_timer_thread(void)
 {
-#if defined(_WIN32)
-HANDLE  threadhandle;
-HANDLE  threadrunsemaphore;
-#endif
 
     _tx_thread_terminate(&_tx_timer_thread);
-
-#if defined(_WIN32)
-    if (_tx_thread_current_ptr == TX_NULL)
-    {
-
-        /* The Windows simulator creates the host thread before the scheduler can run it.
-           Tear it down directly so the regression hook can force the retry path safely.  */
-        threadhandle =       _tx_timer_thread.tx_thread_win32_thread_handle;
-        threadrunsemaphore = _tx_timer_thread.tx_thread_win32_thread_run_semaphore;
-
-        if (threadhandle != ((HANDLE) 0))
-        {
-            (void)TerminateThread(threadhandle, 0U);
-            (void)CloseHandle(threadhandle);
-            _tx_timer_thread.tx_thread_win32_thread_handle = ((HANDLE) 0);
-        }
-
-        if (threadrunsemaphore != ((HANDLE) 0))
-        {
-            (void)CloseHandle(threadrunsemaphore);
-            _tx_timer_thread.tx_thread_win32_thread_run_semaphore = ((HANDLE) 0);
-        }
-    }
-#endif
-
     _tx_thread_delete(&_tx_timer_thread);
 }
 

test/tx/regression/threadx_event_flag_isr_set_clear_test.c

@@ -250,14 +250,14 @@ ULONG   actual;
     {
 
         /* Suspend on the event_flags that is going to be set via the ISR.  */
-        status =  tx_event_flags_get(&event_flags_0, 2, TX_OR_CLEAR, &actual, 100);
+        status =  tx_event_flags_get(&event_flags_0, 2, TX_OR_CLEAR, &actual, 4);
 
         /* Determine if we have an unexpected result.  */
         if (status != TX_SUCCESS)
         {
 
             /* Test error!  */
-            printf("ERROR #8 (%u %lu %lu)\n", status, condition_count, event_flags_set_counter);
+            printf("ERROR #8\n");
             test_control_return(1);
         }
 
@@ -316,7 +316,7 @@ ULONG   actual;
     {
 
         /* Suspend on the event_flags that is going to be set via the ISR.  */
-        status =  tx_event_flags_get(&event_flags_0, 1, TX_OR_CLEAR, &actual, 100);
+        status =  tx_event_flags_get(&event_flags_0, 1, TX_OR_CLEAR, &actual, 4);
 
         /* Determine if we have an unexpected result.  */
         if (status != TX_SUCCESS)

test/tx/regression/threadx_thread_sleep_for_100ticks_test.c

@@ -406,10 +406,9 @@ volatile ULONG       value = 0;
     /* Sleep for 100 ticks.  */
     status =  tx_thread_sleep(100);
 
-    /* The Windows host simulator can occasionally resume the sleeping thread
-       one additional tick later under heavier host-side scheduling jitter.  */
+    /* Determine if the sleep was accurate.  */
     if ((status != TX_SUCCESS) || (tx_time_get() < 100) ||
-        (tx_time_get() > 102))
+        (tx_time_get() > 101))
     {
 
         /* Thread Simple Sleep error.  */
@@ -433,4 +432,3 @@ volatile ULONG       value = 0;
         test_control_return(0);
     }
 }
-

test/tx/regression/threadx_timer_multiple_test.c

@@ -236,17 +236,13 @@ UINT    status;
     /* Sleep for 200.  */
     tx_thread_sleep(200);
 
-    /* The multiple timer accuracy test validates exact cadence already.
-       On the Windows host simulator, the waiting test thread can resume
-       just before or just after the neighboring tick is processed on the
-       shortest timer periods.  */
-    if ((timer_0_counter < 102) || (timer_0_counter > 104) ||
-        (timer_1_counter < 52) || (timer_1_counter > 54) ||
-        (timer_2_counter < 36) || (timer_2_counter > 37))
+    /* Insure that each timer haven't run again.  */
+    if ((timer_0_counter != 103) || (timer_1_counter != 53) ||
+        (timer_2_counter != 36))
     {
 
         /* Application timer error.  */
-        printf("ERROR #16 (%lu, %lu, %lu)\n", timer_0_counter, timer_1_counter, timer_2_counter);
+        printf("ERROR #16\n");
         test_control_return(1);
     }