RFR (M) 8246340: Move SystemDictionary GC roots into OopStorage
coleen.phillimore at oracle.com
coleen.phillimore at oracle.com
Thu Jun 4 15:01:32 UTC 2020
Thank you for reviewing, Lois!
Coleen
On 6/4/20 10:45 AM, Lois Foltan wrote:
> +1. Looks good Coleen!
> Lois
>
> On 6/4/2020 10:20 AM, Erik Österlund wrote:
>> Hi Coleen,
>>
>> Looks good! Thanks for the nice cleanup!
>>
>> /Erik
>>
>> On 2020-06-04 13:39, coleen.phillimore at oracle.com wrote:
>>>
>>> Offline, Erik pointed out that I missed a system_dictionary tag in
>>> parallelGC and to not initialize OopStorage static data members, so
>>> here's an incremental that I reran tier1 tests on.
>>>
>>> open webrev at
>>> http://cr.openjdk.java.net/~coleenp/2020/8246340.02.incr/webrev
>>>
>>> Thanks,
>>> Coleen
>>>
>>> On 6/3/20 1:23 PM, coleen.phillimore at oracle.com wrote:
>>>> Summary: use vm_global() OopStorage for system dictionary roots
>>>> removed SystemDictionary::oops_do in all the GCs.
>>>>
>>>> Tested with tier1-6.
>>>>
>>>> open webrev at
>>>> http://cr.openjdk.java.net/~coleenp/2020/8246340.01/webrev
>>>> bug link https://bugs.openjdk.java.net/browse/JDK-8246340
>>>>
>>>> Thanks,
>>>> Coleen
>>>
>>
>
More information about the hotspot-dev
mailing list