-
Enhancement
-
Resolution: Migrated
-
P4
-
14
The simple function Arena::inc_bytes_allocated can be found as CPU consuming when profiling the fastdbg build. It is located in a cpp file.
It should better get inlined. Main performance issue seems to be contention on the static variable _bytes_allocated which is updated by several threads.
This function is only needed for PrintMallocStatistics, but the performance impact is also there if the feature is inactive (which is default).
In addition, the implementation could get updated: Atomic 8-Byte load and store functions are available on all platforms, so the "#if defined ..." can get removed or adjusted to only avoid word-tearing on 32 bit.
It should better get inlined. Main performance issue seems to be contention on the static variable _bytes_allocated which is updated by several threads.
This function is only needed for PrintMallocStatistics, but the performance impact is also there if the feature is inactive (which is default).
In addition, the implementation could get updated: Atomic 8-Byte load and store functions are available on all platforms, so the "#if defined ..." can get removed or adjusted to only avoid word-tearing on 32 bit.
- relates to
-
JDK-8227692 Remove develop feature PrintMallocStatistics
-
- Resolved
-