summaryrefslogtreecommitdiff
path: root/runtime/javaheapprof/javaheapsampler.h
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.h
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.h')
-rw-r--r--runtime/javaheapprof/javaheapsampler.h4
1 files changed, 2 insertions, 2 deletions
diff --git a/runtime/javaheapprof/javaheapsampler.h b/runtime/javaheapprof/javaheapsampler.h
index 618893cad0..41514726cd 100644
--- a/runtime/javaheapprof/javaheapsampler.h
+++ b/runtime/javaheapprof/javaheapsampler.h
@@ -68,7 +68,7 @@ class HeapSampler {
// of new Tlab after Reset.
void AdjustSampleOffset(size_t adjustment);
// Is heap sampler enabled?
- bool IsEnabled();
+ bool IsEnabled() { return enabled_.load(std::memory_order_acquire); }
// Set the sampling interval.
void SetSamplingInterval(int sampling_interval) REQUIRES(!geo_dist_rng_lock_);
// Return the sampling interval.
@@ -80,7 +80,7 @@ class HeapSampler {
// possibly decreasing sample intervals by sample_adj_bytes.
size_t PickAndAdjustNextSample(size_t sample_adj_bytes = 0) REQUIRES(!geo_dist_rng_lock_);
- std::atomic<bool> enabled_;
+ std::atomic<bool> enabled_{false};
// Default sampling interval is 4kb.
// Writes guarded by geo_dist_rng_lock_.
std::atomic<int> p_sampling_interval_{4 * 1024};