RFR: 8252103: Parallel heap inspection for ParallelScavengeHeap [v12]

Albert Mingkun Yang ayang at openjdk.java.net
Wed Nov 4 08:18:56 UTC 2020


On Wed, 4 Nov 2020 01:57:40 GMT, Lin Zang <lzang at openjdk.org> wrote:

>> src/hotspot/share/gc/parallel/parallelScavengeHeap.cpp line 565:
>> 
>>> 563:     }
>>> 564:     return true;
>>> 565:   }
>> 
>> I wonder if the interface would be cleaner with returning `block_index` directly, i.e. `ssize_t claim_and_get_block()`.  When the index goes out of range, return an `InvalidIndex`, which could be defined as `-3`, like `EdenIndex` above.
>
> Thanks for  your  suggestion. I have thought about it when writing this method, but I was not able to find a value of InvalidIndex. After reconsider it, plus that I will make block_index to be unsigned to avoid signed/unsigned conversion, I think might be the InvalidIndex could be max_size_t (that is (size_t)-1). what do you think?

I think the new version looks very nice; thank you for the change.

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

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



More information about the hotspot-gc-dev mailing list