X-Git-Url: https://git.cworth.org/git?a=blobdiff_plain;f=metrics.c;h=ad7ea3a330cc27b6354e13a8070209b0b406bd13;hb=refs%2Fheads%2Fframe-timings;hp=c52bf2ee6c3557dd6b8b41dda820ac24bced4e5b;hpb=851d05230af70b4a2f1f5754607d13f84d371887;p=fips

diff --git a/metrics.c b/metrics.c
index c52bf2e..ad7ea3a 100644
--- 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;
@@ -180,7 +185,6 @@ metrics_fini (metrics_t *metrics)
 
 	/* Discard and cleanup any outstanding queries. */
 	if (metrics->timer_begun_id) {
-		glEndQuery (GL_TIME_ELAPSED);
 		glDeleteQueries (1, &metrics->timer_begun_id);
 		metrics->timer_begun_id = 0;
 	}
@@ -199,7 +203,6 @@ metrics_fini (metrics_t *metrics)
 	if (metrics->info->have_perfmon) {
 
 		if (metrics->monitor_begun_id) {
-			glEndPerfMonitorAMD (metrics->monitor_begun_id);
 			glDeletePerfMonitorsAMD (1, &metrics->monitor_begun_id);
 			metrics->monitor_begun_id = 0;
 		}
@@ -555,6 +558,7 @@ print_per_stage_metrics (metrics_t *metrics,
 	const char *op_string;
 	unsigned group_index, counter;
 	double value;
+	int program_name_length = 0;
 
 	/* Don't print anything for stages with no alloted time. */
 	if (per_stage->time_ns == 0.0)
@@ -562,13 +566,29 @@ print_per_stage_metrics (metrics_t *metrics,
 
 	op_string = metrics_op_string (op_metrics->op);
 
-	printf ("%21s", op_string);
-
 	if (op_metrics->op >= METRICS_OP_SHADER) {
-		printf (" %3d", op_metrics->op - METRICS_OP_SHADER);
-	} else {
-		printf ("    ");
+		int program = op_metrics->op - METRICS_OP_SHADER;
+		glGetObjectLabel (GL_PROGRAM, program, 0,
+				  &program_name_length, NULL);
+		if (program_name_length) {
+			char *program_name;
+
+			program_name = malloc(program_name_length + 1);
+			if (program_name == 0) {
+				fprintf (stderr, "Out of memory.\n");
+				exit (1);
+			}
+			glGetObjectLabel (GL_PROGRAM, program,
+					  program_name_length + 1,
+					  NULL, program_name);
 
+			printf ("%21s    ", program_name);
+		} else {
+			printf ("%21s %3d", op_string, program);
+		}
+	}
+	else {
+		printf ("%21s    ", op_string);
 	}
 
 	if (per_stage->stage)
@@ -819,9 +839,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 +872,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 +886,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
@@ -921,7 +944,7 @@ metrics_end_frame_post_swap (metrics_t *metrics)
 
 	frames++;
 
-	if (frames % 15 == 0) {
+	if (frames) {
 		double fps;
 
 		gettimeofday (&tv_now, NULL);