trace-graph: Have the graph control the start stop records

Move the code from initializing the start cursor locations of the
CPUs.

Also control when a record is passed to the plot based on its
timestamp.

Signed-off-by: Steven Rostedt <rostedt@goodmis.org>

3 files changed, 19 insertions, 26 deletions

diff --git a/trace-graph.c b/trace-graph.c
index 2214141..d62882c 100644
--- a/trace-graph.c
+++ b/trace-graph.c
@@ -1510,6 +1510,9 @@ static void draw_plots(struct graph_info *ginfo, gint new_width)
                 set_color(ginfo->draw, plot->gc, plot->last_color);
         }
 
+        tracecmd_set_all_cpus_to_timestamp(ginfo->handle,
+                                           ginfo->view_start_time);
+
         /* Shortcut if we don't have any task plots */
         if (!ginfo->nr_task_hash && !ginfo->all_recs) {
                 for (cpu = 0; cpu < ginfo->cpus; cpu++) {
@@ -1518,6 +1521,14 @@ static void draw_plots(struct graph_info *ginfo, gint new_width)
                                 continue;
 
                         while ((record = tracecmd_read_data(ginfo->handle, cpu))) {
+                                if (record->ts < ginfo->view_start_time) {
+                                        free_record(record);
+                                        continue;
+                                }
+                                if (record->ts > ginfo->view_end_time) {
+                                        free_record(record);
+                                        break;
+                                }
                                 for (list = hash->plots; list; list = list->next)
                                         draw_plot(ginfo, list->plot, record);
                                 free_record(record);
@@ -1527,6 +1538,14 @@ static void draw_plots(struct graph_info *ginfo, gint new_width)
         }
 
         while ((record = tracecmd_read_next_data(ginfo->handle, &cpu))) {
+                if (record->ts < ginfo->view_start_time) {
+                        free_record(record);
+                        continue;
+                }
+                if (record->ts > ginfo->view_end_time) {
+                        free_record(record);
+                        break;
+                }
                 hash = trace_graph_plot_find_cpu(ginfo, cpu);
                 if (hash) {
                         for (list = hash->plots; list; list = list->next)
diff --git a/trace-plot-cpu.c b/trace-plot-cpu.c
index ccc2bff..5f28e18 100644
--- a/trace-plot-cpu.c
+++ b/trace-plot-cpu.c
@@ -171,35 +171,11 @@ static void cpu_plot_start(struct graph_info *ginfo, struct graph_plot *plot,
                            unsigned long long time)
 {
         struct cpu_plot_info *cpu_info = plot->private;
-        struct record *last_record = NULL;
-        struct record *record;
         int cpu;
 
         cpu = cpu_info->cpu;
         cpu_info->last_time = 0ULL;
         cpu_info->last_pid = -1;
-
-        tracecmd_set_cpu_to_timestamp(ginfo->handle, cpu, time);
-
-        while ((record = tracecmd_read_data(ginfo->handle, cpu))) {
-                if (record->ts >= time)
-                        break;
-
-                free_record(last_record);
-                last_record = record;
-        }
-
-        free_record(record);
-        /* reset so the next record read is the first record */
-        if (last_record) {
-                record = tracecmd_read_at(ginfo->handle,
-                                          last_record->offset,
-                                          NULL);
-                free_record(record);
-                free_record(last_record);
-        } else
-                tracecmd_set_cpu_to_timestamp(ginfo->handle, cpu, time);
-
 }
 
 static int cpu_plot_event(struct graph_info *ginfo,
diff --git a/trace-plot-task.c b/trace-plot-task.c
index 2770384..a42202f 100644
--- a/trace-plot-task.c
+++ b/trace-plot-task.c
@@ -251,8 +251,6 @@ static void task_plot_start(struct graph_info *ginfo, struct graph_plot *plot,
 {
         struct task_plot_info *task_info = plot->private;
 
-        set_cpus_to_time(ginfo, time);
-
         task_info->last_time = 0ULL;
         task_info->last_cpu = -1;
 }