RFR: 8308387: CLD created and unloading list sharing _next node pointer leads to concurrent YC missing CLD roots [v4]

[Axel Boldt-Christmas]

On Thu, 1 Jun 2023 02:37:40 GMT, David Holmes <dholmes at openjdk.org> wrote:

>> Axel Boldt-Christmas has updated the pull request incrementally with three additional commits since the last revision:
>> 
>>  - Spelling
>>  - Tidy up comment
>>  - Include unlink_next() in comment and remove whitespace
>
> src/hotspot/share/classfile/classLoaderData.hpp line 171:
> 
>> 169:   // After all dead CLDs have been moved to the unloading list, there's a
>> 170:   // synchronization point (handshake) to ensure that all threads reading these
>> 171:   // CLDs finish their work. This ensures that we don't have a user-after-free
> 
> use-after-free ?

Fixed

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

PR Review Comment: https://git.openjdk.org/jdk/pull/14241#discussion_r1212627600