summaryrefslogtreecommitdiff
path: root/runtime/javaheapprof/javaheapsampler.cc
diff options
context:
space:
mode:
author Lokesh Gidra <lokeshgidra@google.com> 2023-11-13 21:39:26 +0000
committer Lokesh Gidra <lokeshgidra@google.com> 2023-11-13 21:57:11 +0000
commit7bfd80eb74096fc09aafa086efc6f05779ccb710 (patch)
tree90c217f4b2d654170cf3b4ff77476429385c3cba /runtime/javaheapprof/javaheapsampler.cc
parentea1529568939cfc97122964375591965a47bedde (diff)
Initialialize jhp's 'enabled_' field
Otherwise, with jemalloc, which doesn't memset(0), we wrongly enable the sampler, which makes allocations slower due to smaller TLABs. Also move IsEnabled() to the header file as it is invoked quite a few times while allocating TLAB. Bug: 310655393 Test: manual testing on devices using jemalloc Change-Id: I63e819488fb980db95080fa17400712bd8efdbe5
Diffstat (limited to 'runtime/javaheapprof/javaheapsampler.cc')
-rw-r--r--runtime/javaheapprof/javaheapsampler.cc4
1 files changed, 0 insertions, 4 deletions
diff --git a/runtime/javaheapprof/javaheapsampler.cc b/runtime/javaheapprof/javaheapsampler.cc
index a73ed0b719..74671347c7 100644
--- a/runtime/javaheapprof/javaheapsampler.cc
+++ b/runtime/javaheapprof/javaheapsampler.cc
@@ -131,10 +131,6 @@ void HeapSampler::AdjustSampleOffset(size_t adjustment) {
<< " next_bytes_until_sample = " << next_bytes_until_sample;
}
-bool HeapSampler::IsEnabled() {
- return enabled_.load(std::memory_order_acquire);
-}
-
int HeapSampler::GetSamplingInterval() {
return p_sampling_interval_.load(std::memory_order_acquire);
}