diff options
author | Dave Allison <dallison@google.com> | 2014-01-28 18:33:52 -0800 |
---|---|---|
committer | Dave Allison <dallison@google.com> | 2014-03-07 13:42:48 -0800 |
commit | 39c3bfbd03d85c63cfbe69f17ce5800ccc7d6c13 (patch) | |
tree | fa777039b3f7c34f3dd322d04307766246526080 /runtime/runtime.cc | |
parent | 0918614b7434783477e8668df7850a7aaf8d5611 (diff) | |
download | art-39c3bfbd03d85c63cfbe69f17ce5800ccc7d6c13.zip art-39c3bfbd03d85c63cfbe69f17ce5800ccc7d6c13.tar.gz art-39c3bfbd03d85c63cfbe69f17ce5800ccc7d6c13.tar.bz2 |
Make use of profiling information for dex2oat
If the profile file exists, the compiler driver will read it
and store the data in an internal map. Then, when we want to work
out whether to compile a method or not, the map is consulted and if
the method shows up with a high enough percentage of use we compile it.
The profile file itself is created by installd and is writeable by the
app. The file is in /data/dalvik-cache/profiles and is named by
the package name.
This also modifies the profiler itself to:
1. Only count runnable threads (not suspended threads) in the profile
2. Use system properties to allow tuning of the profile parameters
3. Merge profiles from multiple processes using file locking.
Bug: 12877748
Change-Id: Iab2f3a327a2860db2a80d5724277d6c626227f2b
Conflicts:
compiler/dex/frontend.cc
compiler/dex/mir_analysis.cc
compiler/dex/verification_results.cc
compiler/driver/compiler_driver.cc
dex2oat/dex2oat.cc
runtime/class_linker.cc
runtime/runtime.cc
runtime/runtime.h
Diffstat (limited to 'runtime/runtime.cc')
-rw-r--r-- | runtime/runtime.cc | 21 |
1 files changed, 17 insertions, 4 deletions
diff --git a/runtime/runtime.cc b/runtime/runtime.cc index fdbf245..d1c8370 100644 --- a/runtime/runtime.cc +++ b/runtime/runtime.cc @@ -27,6 +27,7 @@ #include <cstdlib> #include <limits> #include <vector> +#include <fcntl.h> #include "arch/arm/registers_arm.h" #include "arch/mips/registers_mips.h" @@ -69,6 +70,10 @@ #include "JniConstants.h" // Last to avoid LOG redefinition in ics-mr1-plus-art. +#ifdef HAVE_ANDROID_OS +#include "cutils/properties.h" +#endif + namespace art { Runtime* Runtime::instance_ = NULL; @@ -370,7 +375,12 @@ bool Runtime::Start() { if (profile_) { // User has asked for a profile using -Xprofile - StartProfiler(profile_output_filename_.c_str(), true); + // Create the profile file if it doesn't exist. + int fd = open(profile_output_filename_.c_str(), O_RDWR|O_CREAT|O_EXCL, 0660); + if (fd >= 0) { + close(fd); + } + StartProfiler(profile_output_filename_.c_str(), "", true); } return true; @@ -1055,10 +1065,10 @@ void Runtime::RemoveMethodVerifier(verifier::MethodVerifier* verifier) { method_verifiers_.erase(it); } -void Runtime::StartProfiler(const char *appDir, bool startImmediately) { +void Runtime::StartProfiler(const char* appDir, const char* procName, bool startImmediately) { BackgroundMethodSamplingProfiler::Start(profile_period_s_, profile_duration_s_, appDir, - profile_interval_us_, profile_backoff_coefficient_, - startImmediately); + procName, profile_interval_us_, + profile_backoff_coefficient_, startImmediately); } // Transaction support. @@ -1136,4 +1146,7 @@ void Runtime::SetFaultMessage(const std::string& message) { fault_message_ = message; } +void Runtime::UpdateProfilerState(int state) { + LOG(DEBUG) << "Profiler state updated to " << state; +} } // namespace art |