[jdk17u-dev] RFR: 8282094: [REDO] Parallel: Refactor PSCardTable::scavenge_contents_parallel [v3]

[Goetz Lindenmaier]

On Wed, 6 Mar 2024 17:28:07 GMT, Richard Reingruber <rrich at openjdk.org> wrote:

>> This backport is a dependency for the backport of https://bugs.openjdk.org/browse/JDK-8310031
>> Applies cleanly. Only a trivial renaming is required.
>> Risk is medium. We've done the downstream backport already many weeks ago.
>> 
>> I've tested on x86_64:
>> jdk:tier1       TEST_VM_OPTS="-XX:+UseParallelGC"
>> langtools:tier1 TEST_VM_OPTS="-XX:+UseParallelGC"
>> 
>> Local CI Testing:
>> The fix passed our CI testing (e.g. 2024-02-24): JTReg tests: tier1-4 of hotspot and jdk. All of Langtools and jaxp. JCK, SPECjvm2008, SPECjbb2015, Renaissance Suite, and SAP specific tests (also with ParallelGC).
>> Testing was done with fastdebug builds on the main platforms and also on Linux/PPC64le.
>
> Richard Reingruber has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains six commits:
> 
>  - Merge branch 'master' into 8282094_17u_backport
>  - Fix build
>  - 8282094: [REDO] Parallel: Refactor PSCardTable::scavenge_contents_parallel
>    
>    Reviewed-by: iwalulya, tschatzl
>  - 8278893: Parallel: Remove GCWorkerDelayMillis
>    
>    Reviewed-by: ayang, mli
>  - Fix build
>  - 8280030: [REDO] Parallel: More precise boundary in ObjectStartArray::object_starts_in_range
>    
>    Reviewed-by: sjohanss, tschatzl

Hi,
this is because you merge two changes, the one that sits under this one and the copy of it that was pushed to jdk17u-dev. They are identical, but Github does not grok this. This is very annoying.  I have seen errors before in the parts merged automatically.

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

PR Comment: https://git.openjdk.org/jdk17u-dev/pull/2228#issuecomment-1981554251