RFR: 8285710: Miscalculation of G1CardSetAllocator unused memory size [v4]

[Albert Mingkun Yang]

On Sat, 7 May 2022 02:41:27 GMT, tqxia <duke at openjdk.java.net> wrote:

>> when calculating the wasted memory size of G1CardSetAllocator, the code erroneously substracted both _segmented_array.num_allocated_slots() and _free_slots_list.pending_count() from _segmented_array.num_available_slots().
>> 
>> The correct formula should be: num_wasted_slots = _segmented_array.num_available_slots() - (_segmented_array.num_allocated_slots() - (uint)_free_slots_list.pending_count()).
>> 
>> This can potentially leads to an arithmetic overflow and misleading information will be displayed when G1SummarizeRSetStatsPeriod is set.
>
> tqxia has refreshed the contents of this pull request, and previous commits have been removed. The incremental views will show differences compared to the previous content of the PR. The pull request contains one new commit since the last revision:
> 
>   8285710: Miscalculation of G1CardSetAllocator unused memory size

I suggest the following, `<not-yet-allocated> + <recycled>`.


... = (total - allocated) +
      free_count;

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

Marked as reviewed by ayang (Reviewer).

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