RFR: 8253815: Remove unused HeapRegionManager::_num_committed from SA

Chris Plummer cjplummer at openjdk.java.net
Tue Sep 29 21:36:52 UTC 2020


On Tue, 29 Sep 2020 21:28:07 GMT, Stefan Johansson <sjohanss at openjdk.org> wrote:

>> src/hotspot/share/gc/g1/vmStructs_g1.hpp line 56:
>> 
>>> 54:                                                                               \
>>> 55:   nonstatic_field(HeapRegionManager, _regions,          G1HeapRegionTable)    \
>>> 56:   nonstatic_field(HeapRegionManager, _num_committed,    uint)                 \
>> 
>> If this field is remaining in hotspot, it should also remain in vmStructs. Although SA does not need to explicitly know
>> about this field to function properly, SA can be used to dump hotspot objects, and can only dump objects whose types
>> are in vmStructs, and for those types can only display the fields that are also in vmStructs.
>
> The field will remain for now, but the plan is to refactor it into a new class. The new class is not needed by the SA
> and I would really like to avoid having to add unused and untested code to the SA. To simplify this coming change I
> want to remove this from the SA straight away.   Even without this coming change I would vote for removing this type of
> unused code from the SA. I don't think being able to dump the value is worth enough compared to the maintenance cost of
> keeping it in.

Ok. If the field is going away eventually then you can remove it now.

-------------

PR: https://git.openjdk.java.net/jdk/pull/408


More information about the serviceability-dev mailing list