Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(108)

Unified Diff: client/profilers/perf/perf.py

Issue 3541002: Revert "Merge remote branch 'cros/upstream' into tempbranch2" (Closed) Base URL: ssh://git@gitrw.chromium.org:9222/autotest.git
Patch Set: Created 10 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « client/profilers/oprofile/oprofile.py ('k') | client/profilers/powertop/powertop.py » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: client/profilers/perf/perf.py
diff --git a/client/profilers/perf/perf.py b/client/profilers/perf/perf.py
index 211d562bdaea058341afad4fcbda3b65ec0e3664..ff0a32eb2516f4fbc887ae95603f7a77ea12e22e 100644
--- a/client/profilers/perf/perf.py
+++ b/client/profilers/perf/perf.py
@@ -5,8 +5,7 @@ supports functionality similar to oprofile and more.
@see: http://lwn.net/Articles/310260/
"""
-import time, os, stat, subprocess, signal
-import logging
+import time, os, subprocess, signal
from autotest_lib.client.bin import profiler, os_dep, utils
@@ -52,8 +51,3 @@ class perf(profiler.profiler):
p = subprocess.Popen(cmd, shell=True, stdout=outfile,
stderr=subprocess.STDOUT)
p.wait()
- # The raw detailed perf output is HUGE. We cannot store it by default.
- perf_log_size = os.stat(self.logfile)[stat.ST_SIZE]
- logging.info('Removing %s after generating reports (saving %s bytes).',
- self.logfile, perf_log_size)
- os.unlink(self.logfile)
« no previous file with comments | « client/profilers/oprofile/oprofile.py ('k') | client/profilers/powertop/powertop.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698