diff options
| author | Steven Rostedt <srostedt@redhat.com> | 2009-12-29 14:03:57 -0500 |
|---|---|---|
| committer | Steven Rostedt <rostedt@goodmis.org> | 2009-12-29 14:15:54 -0500 |
| commit | ff8ebb0ecd19766c9a8d72cd17e622156f7ca723 (patch) | |
| tree | 526df3f17113545ded1dd18802c215d5cf48101a | |
| parent | fb1e0b1e12f928760c6c80b43090b3c08e39693f (diff) | |
trace-graph: Use free_record() instead of free() to free records
Now that a record can pin a page to memory, it is imperative that we
use free_record() to free any records.
Signed-off-by: Steven Rostedt <rostedt@goodmis.org>
| -rw-r--r-- | trace-graph.c | 7 |
1 files changed, 3 insertions, 4 deletions
diff --git a/trace-graph.c b/trace-graph.c index f7d98c1..99b7140 100644 --- a/trace-graph.c +++ b/trace-graph.c | |||
| @@ -415,7 +415,7 @@ static void draw_cpu_info(struct graph_info *ginfo, gint cpu, gint x, gint y) | |||
| 415 | else | 415 | else |
| 416 | trace_seq_puts(&s, " <idle>"); | 416 | trace_seq_puts(&s, " <idle>"); |
| 417 | 417 | ||
| 418 | free(record); | 418 | free_record(record); |
| 419 | 419 | ||
| 420 | } else | 420 | } else |
| 421 | trace_seq_printf(&s, "%lu.%06lu", sec, usec); | 421 | trace_seq_printf(&s, "%lu.%06lu", sec, usec); |
| @@ -969,7 +969,7 @@ static void draw_cpu(struct graph_info *ginfo, gint cpu, | |||
| 969 | p1 = p2; | 969 | p1 = p2; |
| 970 | p2 = p3; | 970 | p2 = p3; |
| 971 | last_event_id = event_id; | 971 | last_event_id = event_id; |
| 972 | free(record); | 972 | free_record(record); |
| 973 | } | 973 | } |
| 974 | 974 | ||
| 975 | if (p2) | 975 | if (p2) |
| @@ -988,8 +988,7 @@ static void draw_cpu(struct graph_info *ginfo, gint cpu, | |||
| 988 | x, CPU_BOTTOM(cpu)); | 988 | x, CPU_BOTTOM(cpu)); |
| 989 | } | 989 | } |
| 990 | 990 | ||
| 991 | if (record) | 991 | free_record(record); |
| 992 | free(record); | ||
| 993 | } | 992 | } |
| 994 | 993 | ||
| 995 | 994 | ||
