RFR: 8252656: Replace RegisterArrayForGC mechanism with plain Handles

[Stefan Karlsson]

Thanks, Coleen.

StefanK

On 2020-09-01 15:41, Coleen Phillimore wrote:
> Looks good to me!  I don't see further need for testing as this as this 
> code path is executed for almost everything.
> Coleen
> 
> On 9/1/20 9:13 AM, Stefan Karlsson wrote:
>> Hi all,
>>
>> Please review this patch to replace the RegisterArrayForGC mechanism 
>> with the usage of plain Handles.
>>
>> https://cr.openjdk.java.net/~stefank/8252656/webrev.01/
>> https://bugs.openjdk.java.net/browse/JDK-8252656
>>
>> JVM_GetStackAccessControlContext collects a bunch of oops in array and 
>> then calls into allocating code that might safepoint. To deal with 
>> this, the local array is registered and explicitly visited by the GCs. 
>> This patch gets rid of this and use an array of Handles instead.
>>
>> I've tested this locally with test TEST="test/jdk/java/security". Any 
>> other suggestions on what to test this with? I've also run this 
>> through tier1, as a sanity check.
>>
>> Thanks,
>> StefanK
>