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.h | |
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.h')
-rw-r--r-- | runtime/javaheapprof/javaheapsampler.h | 4 |
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}; |