RFR: 8290706: Remove the support for inline contiguous allocations [v2]
Richard Reingruber
rrich at openjdk.org
Mon Jul 25 14:39:07 UTC 2022
On Thu, 21 Jul 2022 16:17:38 GMT, Aleksey Shipilev <shade at openjdk.org> wrote:
>> See the bug for rationale and link to RFC.
>>
>> This removes the 3rd allocation path (first two being TLAB and native GC interface), that is used by Serial/Parallel when TLABs are not available. There is little sense in keeping this code, especially since it requires supporting a bunch of platform-specific assembly.
>>
>> Additional testing:
>> - [x] Linux x86_64 fastdebug `tier1`
>> - [x] Linux x86_32 fastdebug `tier1`
>> - [x] Linux AArch64 fastdebug `tier1`
>> - [x] Linux x86_64 Zero build
>> - [x] Linux AArch64 cross-build (attn @theRealAph, @adinn)
>> - [x] Linux ARM cross-build (attn @bulasevich, @snazarkin)
>> - [x] Linux S390X cross-build (attn @backwaterred, @RealLucy)
>> - [x] Linux PPC64 cross-build (attn @TheRealMDoerr, @reinrich)
>> - [x] Linux RISC-V cross-build (attn @RealFYang)
>>
>> Apart from x86 and AArch64, I only verified the cross-compilation builds pass, no other testing is done.
>>
>> I did not touch the JVMCI interfaces, since I am not sure what is the proper protocol for JVMCI changes.
>
> Aleksey Shipilev has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains two commits:
>
> - Merge branch 'master' into JDK-8290706-remove-inline-contig
> - Work
No findings in our CI testing which includes JCK and JTREG tests on the standard platforms and also on Linux/PPC64le.
I've reviewed the ppc changes. They look good to me.
Thanks, Richard.
-------------
Marked as reviewed by rrich (Reviewer).
PR: https://git.openjdk.org/jdk/pull/9576
More information about the hotspot-dev
mailing list