@@ -469,8 +469,6 @@ static void wait_for_events(struct test_output *o)
/* Returned the ellapsed time in us */
static unsigned event_loop(struct test_output *o, unsigned duration_sec)
{
- drmEventContext evctx;
- int ret;
struct timeval start, end;
struct timeval tv_dur;
@@ -481,9 +479,9 @@ static unsigned event_loop(struct test_output *o, unsigned duration_sec)
while (1) {
struct timeval now;
+ run_test_step(o);
wait_for_events(o);
check_all_state(o);
- run_test_step(o);
update_all_state(o);
gettimeofday(&now, NULL);
@@ -494,12 +492,6 @@ static unsigned event_loop(struct test_output *o, unsigned duration_sec)
gettimeofday(&end, NULL);
timersub(&end, &start, &tv_dur);
- /* and drain the event queue */
- memset(&evctx, 0, sizeof evctx);
- evctx.page_flip_handler = NULL;
- ret = drmHandleEvent(drm_fd, &evctx);
- assert(ret == 0);
-
return tv_dur.tv_sec * 1000 * 1000 + tv_dur.tv_usec;
}
@@ -553,8 +545,9 @@ static void flip_mode(struct test_output *o, int crtc, int duration)
fprintf(stderr, "failed to page flip: %s\n", strerror(errno));
exit(4);
}
+ wait_for_events(o);
+
o->current_fb_id = 1;
- o->count = 1; /* for the uncounted tail */
ellapsed = event_loop(o, duration);
At the moment we first check the state then run the test step in our test loop. Swapping the order makes the starting state of each iteration better defined, allowing an easier extension of these steps in the future. Since now it's guaranteed that we exit the event loop with no pending flips, we can also get rid of the final flushing of events. We don't want the first initializing flip to affect the test loop other than setting an initial FB, so before starting the test loop wait for it to complete by calling wait_for_events() and leave the flip event counter at zero. Signed-off-by: Imre Deak <imre.deak@intel.com> --- tests/flip_test.c | 13 +++---------- 1 file changed, 3 insertions(+), 10 deletions(-)