RFR: 8254999: Move G1RemSetSamplingTask to more appropriate location [v3]

Thomas Schatzl tschatzl at openjdk.java.net
Tue Nov 24 12:32:05 UTC 2020


On Tue, 24 Nov 2020 12:04:19 GMT, Albert Mingkun Yang <ayang at openjdk.org> wrote:

>> Stefan Johansson has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Remove new scheduling logic
>
> src/hotspot/share/gc/g1/g1RemSet.cpp line 499:
> 
>> 497: G1RemSet::~G1RemSet() {
>> 498:   delete _scan_state;
>> 499:   if (_sampling_task != NULL) {
> 
> Could you add a comment to explain when `_sampling_task` would be null?

Actually the check can be removed because the operator implementation handles NULL (by ignoring it).

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

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



More information about the hotspot-gc-dev mailing list