diff options
author | Sergey Senozhatsky <sergey.senozhatsky@gmail.com> | 2011-05-10 16:58:27 +0300 |
---|---|---|
committer | Arjan van de Ven <arjan@linux.intel.com> | 2011-05-11 00:04:39 -0400 |
commit | 058ad3e50182b45b0bc9ccc8b9b00630208c384d (patch) | |
tree | 01f149ec5951a37f03600bdd52de909ebac8a6fa | |
parent | a2cdd5f37b9704415fdc133ea13e2d22dacf6a7e (diff) | |
download | powertop-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.cpp | 43 |
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(); |