[jdk25u-dev] RFR: 8358735: GenShen: block_start() may be incorrect after class unloading [v2]

[Y. Srinivas Ramakrishna]

On Wed, 26 Nov 2025 19:30:06 GMT, Rui Li <duke at openjdk.org> wrote:

>> Backport of JDK-8358735 and its two follow up fixes: JDK-8372110 and JDK-8371854.
>> 
>> The backport of the original issue (JDK-8358735) is almost clean except that atomic.hpp was renamed to atomicAccess.hpp in the later versions, and I had to resolve the conflict by retaining atomic.hpp import.
>> 
>> The two follow up backports are clean.
>> 
>> Our internal application crashed and we identified JDK-8358735 was the root cause. Backporting it for the fix.
>> 
>> Testing: all GHA tests passed.
>
> Rui Li has updated the pull request incrementally with two additional commits since the last revision:
> 
>  - 8371854: Shenandoah: Simplify WALK_FORWARD_IN_BLOCK_START use
>    
>    Backport 713de231a61234632e2f9858b222b5f7fd0bdaf1
>  - 8372110: GenShen: Fix erroneous assert
>    
>    Backport aa7718d1d2008c240c197383c493955d8b654bf3

Thanks for a good description of the backport & conflicts.

LGTM! 🚢

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

Marked as reviewed by ysr (Reviewer).

PR Review: https://git.openjdk.org/jdk25u-dev/pull/19#pullrequestreview-3531742077