Add a timer API to libdimension.

7 files changed, 137 insertions, 21 deletions

diff --git a/libdimension/Makefile.am b/libdimension/Makefile.am
index 16ab286..73fb010 100644
--- a/libdimension/Makefile.am
+++ b/libdimension/Makefile.am
@@ -43,7 +43,8 @@ nobase_include_HEADERS = dimension.h                                           \
                          dimension/progress.h                                  \
                          dimension/raytrace.h                                  \
                          dimension/scene.h                                     \
-                         dimension/texture.h
+                         dimension/texture.h                                   \
+                         dimension/timer.h
 
 lib_LTLIBRARIES = libdimension.la
 
@@ -84,7 +85,8 @@ libdimension_la_SOURCES = $(nobase_include_HEADERS)                            \
                           sphere.c                                             \
                           texture.c                                            \
                           threads.c                                            \
-                          threads.h
+                          threads.h                                            \
+                          timer.c
 libdimension_la_CFLAGS  = $(AM_CFLAGS) -pthread
 libdimension_la_LDFLAGS = -version-info 0:0:0 $(AM_LDFLAGS)
 libdimension_la_LIBADD  = -lm
diff --git a/libdimension/dimension.h b/libdimension/dimension.h
index 57e7c90..c58eb0e 100644
--- a/libdimension/dimension.h
+++ b/libdimension/dimension.h
@@ -65,6 +65,7 @@ typedef void dmnsn_free_fn(void *ptr);
 #include <dimension/array.h>
 #include <dimension/list.h>
 #include <dimension/progress.h>
+#include <dimension/timer.h>
 #include <dimension/geometry.h>
 #include <dimension/color.h>
 #include <dimension/canvas.h>
diff --git a/libdimension/dimension/scene.h b/libdimension/dimension/scene.h
index 7c53991..7e52859 100644
--- a/libdimension/dimension/scene.h
+++ b/libdimension/dimension/scene.h
@@ -64,6 +64,10 @@ typedef struct {
 
   /* Number of parallel threads */
   unsigned int nthreads;
+
+  /* Timers */
+  dmnsn_timer *bounding_timer;
+  dmnsn_timer *render_timer;
 } dmnsn_scene;
 
 /* Create a scene */
diff --git a/libdimension/dimension/timer.h b/libdimension/dimension/timer.h
new file mode 100644
index 0000000..113cb5f
--- /dev/null
+++ b/libdimension/dimension/timer.h
@@ -0,0 +1,34 @@
+/*************************************************************************
+ * Copyright (C) 2010 Tavian Barnes <tavianator@gmail.com>               *
+ *                                                                       *
+ * This file is part of The Dimension Library.                           *
+ *                                                                       *
+ * The Dimension Library is free software; you can redistribute it and/  *
+ * or modify it under the terms of the GNU Lesser General Public License *
+ * as published by the Free Software Foundation; either version 3 of the *
+ * License, or (at your option) any later version.                       *
+ *                                                                       *
+ * The Dimension Library is distributed in the hope that it will be      *
+ * useful, but WITHOUT ANY WARRANTY; without even the implied warranty   *
+ * of MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU  *
+ * Lesser General Public License for more details.                       *
+ *                                                                       *
+ * You should have received a copy of the GNU Lesser General Public      *
+ * License along with this program.  If not, see                         *
+ * <http://www.gnu.org/licenses/>.                                       *
+ *************************************************************************/
+
+/*
+ * A platform-agnostic timer abstraction
+ */
+
+typedef struct dmnsn_timer {
+  double real, user, system;
+} dmnsn_timer;
+
+#define DMNSN_TIMER_FORMAT "%.2fs (user: %.2fs; system: %.2fs)"
+#define DMNSN_TIMER_PRINTF(t) (t)->real, (t)->user, (t)->system
+
+dmnsn_timer *dmnsn_new_timer();
+void dmnsn_complete_timer(dmnsn_timer *timer);
+void dmnsn_delete_timer(dmnsn_timer *timer);
diff --git a/libdimension/raytrace.c b/libdimension/raytrace.c
index 3430678..9822c8c 100644
--- a/libdimension/raytrace.c
+++ b/libdimension/raytrace.c
@@ -76,12 +76,15 @@ dmnsn_raytrace_scene_thread(void *ptr)
 {
   dmnsn_raytrace_payload *payload = ptr;
 
-  /* Pre-calculate bounding box transformations, etc. */
-  DMNSN_ARRAY_FOREACH (dmnsn_object **, object, payload->scene->objects) {
-    dmnsn_object_init(*object);
-  }
+  /* Time the bounding tree construction */
+  payload->scene->bounding_timer = dmnsn_new_timer();
+    /* Pre-calculate bounding box transformations, etc. */
+    DMNSN_ARRAY_FOREACH (dmnsn_object **, object, payload->scene->objects) {
+      dmnsn_object_init(*object);
+    }
 
-  payload->prtree = dmnsn_new_prtree(payload->scene->objects);
+    payload->prtree = dmnsn_new_prtree(payload->scene->objects);
+  dmnsn_complete_timer(payload->scene->bounding_timer);
 
   dmnsn_raytrace_payload *payloads;
   pthread_t *threads;
@@ -105,23 +108,26 @@ dmnsn_raytrace_scene_thread(void *ptr)
     payloads[i].threads = nthreads;
   }
 
-  /* Create the threads */
-  for (int i = 0; i < nthreads; ++i) {
-    if (pthread_create(&threads[i], NULL,
-                       &dmnsn_raytrace_scene_multithread_thread,
-                       &payloads[i]) != 0)
-    {
-      dmnsn_error(DMNSN_SEVERITY_HIGH,
-                  "Couldn't start worker thread in raytrace engine.");
+  /* Time the render itself */
+  payload->scene->render_timer = dmnsn_new_timer();
+    /* Create the threads */
+    for (int i = 0; i < nthreads; ++i) {
+      if (pthread_create(&threads[i], NULL,
+                         &dmnsn_raytrace_scene_multithread_thread,
+                         &payloads[i]) != 0)
+        {
+          dmnsn_error(DMNSN_SEVERITY_HIGH,
+                      "Couldn't start worker thread in raytrace engine.");
+        }
     }
-  }
 
-  for (int i = 0; i < nthreads; ++i) {
-    if (pthread_join(threads[i], NULL)) {
-      dmnsn_error(DMNSN_SEVERITY_MEDIUM,
-                  "Couldn't join worker thread in raytrace engine.");
+    for (int i = 0; i < nthreads; ++i) {
+      if (pthread_join(threads[i], NULL)) {
+        dmnsn_error(DMNSN_SEVERITY_MEDIUM,
+                    "Couldn't join worker thread in raytrace engine.");
+      }
     }
-  }
+  dmnsn_complete_timer(payload->scene->render_timer);
 
   dmnsn_free(threads);
   dmnsn_free(payloads);
diff --git a/libdimension/scene.c b/libdimension/scene.c
index 6d17cea..f56236f 100644
--- a/libdimension/scene.c
+++ b/libdimension/scene.c
@@ -35,6 +35,8 @@ dmnsn_new_scene()
   scene->quality         = DMNSN_RENDER_FULL;
   scene->reclimit        = 5;
   scene->nthreads        = dmnsn_ncpus();
+  scene->bounding_timer  = NULL;
+  scene->render_timer    = NULL;
 
   return scene;
 }
@@ -44,6 +46,9 @@ void
 dmnsn_delete_scene(dmnsn_scene *scene)
 {
   if (scene) {
+    dmnsn_delete_timer(scene->render_timer);
+    dmnsn_delete_timer(scene->bounding_timer);
+
     DMNSN_ARRAY_FOREACH (dmnsn_light **, light, scene->lights) {
       dmnsn_delete_light(*light);
     }
diff --git a/libdimension/timer.c b/libdimension/timer.c
new file mode 100644
index 0000000..eb15357
--- /dev/null
+++ b/libdimension/timer.c
@@ -0,0 +1,64 @@
+/*************************************************************************
+ * Copyright (C) 2009-2010 Tavian Barnes <tavianator@gmail.com>          *
+ *                                                                       *
+ * This file is part of The Dimension Library.                           *
+ *                                                                       *
+ * The Dimension Library is free software; you can redistribute it and/  *
+ * or modify it under the terms of the GNU Lesser General Public License *
+ * as published by the Free Software Foundation; either version 3 of the *
+ * License, or (at your option) any later version.                       *
+ *                                                                       *
+ * The Dimension Library is distributed in the hope that it will be      *
+ * useful, but WITHOUT ANY WARRANTY; without even the implied warranty   *
+ * of MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU  *
+ * Lesser General Public License for more details.                       *
+ *                                                                       *
+ * You should have received a copy of the GNU Lesser General Public      *
+ * License along with this program.  If not, see                         *
+ * <http://www.gnu.org/licenses/>.                                       *
+ *************************************************************************/
+
+#include "dimension.h"
+#include <sys/times.h>
+#include <unistd.h>
+
+static long clk_tck = 0;
+
+dmnsn_timer *
+dmnsn_new_timer()
+{
+  /* Figure out the clock ticks per second */
+  if (!clk_tck) {
+    clk_tck = sysconf(_SC_CLK_TCK);
+    if (clk_tck == -1) {
+      dmnsn_error(DMNSN_SEVERITY_MEDIUM, "sysconf(_SC_CLK_TCK) failed.");
+      clk_tck = 1000000L;
+    }
+  }
+
+  dmnsn_timer *timer = dmnsn_malloc(sizeof(dmnsn_timer));
+
+  struct tms buf;
+  clock_t real = times(&buf);
+  timer->real   = (double)real/clk_tck;
+  timer->user   = (double)buf.tms_utime/clk_tck;
+  timer->system = (double)buf.tms_stime/clk_tck;
+
+  return timer;
+}
+
+void
+dmnsn_complete_timer(dmnsn_timer *timer)
+{
+  struct tms buf;
+  clock_t real = times(&buf);
+  timer->real   = (double)real/clk_tck - timer->real;
+  timer->user   = (double)buf.tms_utime/clk_tck - timer->user;
+  timer->system = (double)buf.tms_stime/clk_tck - timer->system;
+}
+
+void
+dmnsn_delete_timer(dmnsn_timer *timer)
+{
+  dmnsn_free(timer);
+}