Changeset 44228 in vbox for trunk/src

Timestamp:

Jan 2, 2013 12:24:08 PM (12 years ago)

Author:

vboxsync

svn:sync-xref-src-repo-rev:

83036

Message:

tstRTR0Timer.cpp: Attempt at fixing test failures i the CSSD_LOOP.

File:

• trunk/src/VBox/Runtime/testcase/tstRTR0Timer.cpp (modified) (11 diffs)

trunk/src/VBox/Runtime/testcase/tstRTR0Timer.cpp

@@ -400 +400 @@
             do /* break loop */
             {
-                RT_ZERO(State);
+                RT_ZERO(State); ASMAtomicWriteU32(&State.cShots, State.cShots);
                 RTR0TESTR0_CHECK_RC_BREAK(RTTimerStart(pTimer, 0), VINF_SUCCESS);
                 for (uint32_t i = 0; i < 1000 && !ASMAtomicUoReadU32(&State.cShots); i++)
@@ -407 +407 @@
 
                 /* check that it is restartable. */
-                RT_ZERO(State);
+                RT_ZERO(State); ASMAtomicWriteU32(&State.cShots, State.cShots);
                 RTR0TESTR0_CHECK_RC_BREAK(RTTimerStart(pTimer, 0), VINF_SUCCESS);
                 for (uint32_t i = 0; i < 1000 && !ASMAtomicUoReadU32(&State.cShots); i++)
@@ -414 +414 @@
 
                 /* check that it respects the timeout value and can be cancelled. */
-                RT_ZERO(State);
+                RT_ZERO(State); ASMAtomicWriteU32(&State.cShots, State.cShots);
                 RTR0TESTR0_CHECK_RC(RTTimerStart(pTimer, 5*UINT64_C(1000000000)), VINF_SUCCESS);
                 RTR0TESTR0_CHECK_RC(RTTimerStop(pTimer), VINF_SUCCESS);
@@ -421 +421 @@
 
                 /* Check some double starts and stops (shall not assert). */
-                RT_ZERO(State);
+                RT_ZERO(State); ASMAtomicWriteU32(&State.cShots, State.cShots);
                 RTR0TESTR0_CHECK_RC(RTTimerStart(pTimer, 5*UINT64_C(1000000000)), VINF_SUCCESS);
                 RTR0TESTR0_CHECK_RC(RTTimerStart(pTimer, 0), VERR_TIMER_ACTIVE);
@@ -448 +448 @@
                 RT_ZERO(State);
                 State.iActionShot = 0;
+                ASMAtomicWriteU32(&State.cShots, State.cShots);
                 do /* break loop */
                 {
@@ -476 +477 @@
                 State.rc = VERR_IPE_UNINITIALIZED_STATUS;
                 State.iActionShot = 0;
+                ASMAtomicWriteU32(&State.cShots, State.cShots);
                 do /* break loop */
                 {
@@ -504 +506 @@
                     State.rc                = VINF_SUCCESS;
                     State.u.Specific.idCpu  = RTMpCpuIdFromSetIndex(iCpu);
+                    ASMAtomicWriteU32(&State.cShots, State.cShots);
 
                     uint32_t fFlags = TSTRTR0TIMER_IS_HIRES(uOperation) ? RTTIMER_FLAGS_HIGH_RES : 0;
@@ -557 +560 @@
                 RT_ZERO(State);
                 State.fPeriodic = true;
+                ASMAtomicWriteU32(&State.cShots, State.cShots);
+
                 uint64_t uStartNsTS = RTTimeSystemNanoTS();
                 RTR0TESTR0_CHECK_RC_BREAK(RTTimerStart(pTimer, u10HzAsNs), VINF_SUCCESS);
@@ -585 +590 @@
                     RT_ZERO(State);
                     State.fPeriodic = true;
+                    ASMAtomicWriteU32(&State.cShots, State.cShots); /* ordered, necessary? */
+
                     RTR0TESTR0_CHECK_RC_BREAK(RTTimerStart(pTimer, i < 20 ? 0 : cNsSysHz), VINF_SUCCESS);
                     for (uint32_t k = 0; k < 1000 && ASMAtomicUoReadU32(&State.cShots) < 2; k++)
@@ -621 +628 @@
             RTR0TESTR0_CHECK_MSG_BREAK(State.u.ChgInt.cNsMinInterval > 1000, ("%u\n", State.u.ChgInt.cNsMinInterval));
             RTR0TESTR0_CHECK_MSG_BREAK(State.u.ChgInt.cNsMaxInterval > State.u.ChgInt.cNsMinInterval, ("max=%u min=%u\n", State.u.ChgInt.cNsMaxInterval, State.u.ChgInt.cNsMinInterval));
+            ASMAtomicWriteU32(&State.cShots, State.cShots);
 
             /* create the timer and check if RTTimerChangeInterval is supported. */
@@ -664 +672 @@
                     State.fPeriodic         = true;
                     State.u.Specific.idCpu  = RTMpCpuIdFromSetIndex(iCpu);
+                    ASMAtomicWriteU32(&State.cShots, State.cShots);
 
                     uint32_t fFlags = TSTRTR0TIMER_IS_HIRES(uOperation) ? RTTIMER_FLAGS_HIGH_RES : 0;