aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorSergey Senozhatsky <sergey.senozhatsky@gmail.com>2011-05-10 16:58:27 +0300
committerArjan van de Ven <arjan@linux.intel.com>2011-05-11 00:04:39 -0400
commit058ad3e50182b45b0bc9ccc8b9b00630208c384d (patch)
tree01f149ec5951a37f03600bdd52de909ebac8a6fa
parenta2cdd5f37b9704415fdc133ea13e2d22dacf6a7e (diff)
downloadpowertop-058ad3e50182b45b0bc9ccc8b9b00630208c384d.tar.gz
process/do_process: Use introduced clear_* methods
Use introduced clear_* methods instead of iterating through corresponding containers in process_process_data() and end_process_data(). Remove 3 iterations to null processes' wake_ups, accumulated_runtime and child_runtime from end_process_data(), because the very next step to do is to delete all of them. Signed-off-by: Sergey Senozhatsky <sergey.senozhatsky@gmail.com>
-rw-r--r--process/do_process.cpp43
1 files changed, 8 insertions, 35 deletions
diff --git a/process/do_process.cpp b/process/do_process.cpp
index ff8a826..156baab 100644
--- a/process/do_process.cpp
+++ b/process/do_process.cpp
@@ -865,18 +865,10 @@ void process_process_data(void)
if (!perf_events)
return;
+ clear_processes();
+ clear_interrupts();
- /* clean out old data */
- for (i = 0; i < all_processes.size() ; i++)
- delete all_processes[i];
-
- all_processes.erase(all_processes.begin(), all_processes.end());;
-
- for (i = 0; i < all_interrupts.size() ; i++)
- delete all_interrupts[i];
-
- all_interrupts.resize(0);
- all_power.resize(0);
+ all_power.erase(all_power.begin(), all_power.end());
clear_consumers();
@@ -936,31 +928,12 @@ void end_process_data(void)
report_utilization("disk-operations", total_disk_hits());
report_utilization("disk-operations-hard", total_hard_disk_hits());
- /* clean out old data */
- for (i = 0; i < all_processes.size() ; i++)
- all_processes[i]->wake_ups = 0;
- for (i = 0; i < all_processes.size() ; i++)
- all_processes[i]->accumulated_runtime = 0;
- for (i = 0; i < all_processes.size() ; i++)
- all_processes[i]->child_runtime = 0;
-
- for (i = 0; i < all_processes.size() ; i++)
- delete all_processes[i];
-
- for (i = 0; i < all_proc_devices.size() ; i++)
- delete all_proc_devices[i];
-
- all_processes.erase(all_processes.begin(), all_processes.end());;
-
- for (i = 0; i < all_interrupts.size() ; i++)
- delete all_interrupts[i];
-
- all_interrupts.resize(0);
- all_power.resize(0);
- all_proc_devices.resize(0);
+ all_power.erase(all_power.begin(), all_power.end());
+ clear_processes();
+ clear_proc_devices();
+ clear_interrupts();
clear_timers();
- delete_all_work();
-
+ clear_work();
clear_consumers();
perf_events->clear();