]> git.cworth.org Git - fips/blobdiff - metrics.c
Don't print a frame-time report for the first frame after glXMakeCurrent
[fips] / metrics.c
index c52bf2ee6c3557dd6b8b41dda820ac24bced4e5b..8f961331707569e0759bb635e3826af94bb1697a 100644 (file)
--- a/metrics.c
+++ b/metrics.c
@@ -74,6 +74,9 @@ typedef struct op_metrics
 
 struct metrics
 {
+       /* Flag to indicate first frame is being processed. */
+       bool first_frame;
+
        /* Description of all available peformance counters, counter
         * groups, their names and IDs, etc. */
        metrics_info_t *info;
@@ -144,6 +147,8 @@ metrics_create (metrics_info_t *info)
 
        metrics->info = info;
 
+       metrics->first_frame = true;
+
        metrics->op = 0;
 
        metrics->timer_begun_id = 0;
@@ -819,9 +824,9 @@ metrics_collect_available (metrics_t *metrics)
        }
 }
 
-/* Return the difference from ts0 to ts1 as floating-point seconds. */
-#define TIMESPEC_DIFF(ts0, ts1) (double) (ts1.tv_sec - ts0.tv_sec + \
-                                         (ts1.tv_nsec - ts0.tv_nsec) / 1e9)
+/* Subtract timespec values: Return (a - b) in seconds as a double. */
+#define SUBTRACT_TIMESPEC(a, b) (double) (a.tv_sec - b.tv_sec + \
+                                         (a.tv_nsec - b.tv_nsec) / 1e9)
 
 void
 metrics_end_frame_pre_swap (metrics_t *metrics)
@@ -852,10 +857,12 @@ metrics_end_frame_pre_swap (metrics_t *metrics)
 
        glGetQueryObjectui64v (metrics->swap_end_timestamp_id,
                               GL_QUERY_RESULT, &swap_end_timestamp);
-
-       if (frames == 0) {
+       if (metrics->first_frame) {
                /* Print header */
                printf ("# frame: Frame_Number Frame_Time_milliseconds Frame_latency_milliseconds CPU_load GPU_load\n");
+
+               metrics->first_frame = false;
        } else {
                /* Subtract previous frame's times to get frame times. */
                frame_time_ns = subtract_timestamp (swap_end_timestamp,
@@ -864,7 +871,8 @@ metrics_end_frame_pre_swap (metrics_t *metrics)
                latency_ns = subtract_timestamp (swap_end_timestamp,
                                                 metrics->swap_begin_timestamp);
 
-               cpu_time = TIMESPEC_DIFF (metrics->previous_cpu_time_ts, metrics->cpu_time_ts);
+               cpu_time = SUBTRACT_TIMESPEC (metrics->cpu_time_ts,
+                                             metrics->previous_cpu_time_ts);
 
                /* We've waited one frame to ensure we have a timestamp
                 * result. So the time we've actually measured here is