diff options
author | 2023-11-13 21:39:26 +0000 | |
---|---|---|
committer | 2023-11-13 21:57:11 +0000 | |
commit | 7bfd80eb74096fc09aafa086efc6f05779ccb710 (patch) | |
tree | 90c217f4b2d654170cf3b4ff77476429385c3cba /runtime/javaheapprof/javaheapsampler.cc | |
parent | ea1529568939cfc97122964375591965a47bedde (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.cc | 4 |
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); } |