RFR: Epsilon GC
Roman Kennke
rkennke at redhat.com
Thu Apr 26 18:32:55 UTC 2018
Hi Aleksey,
I like it! It is a nice testament to the GC interface work indeed :-)
I have nothing to complain about in the changeset, but I am not a
Reviewer either ;-)
Cheers, Roman
> Hi,
>
> This is the review thread for Epsilon GC changes.
>
> JEP, targeted to 11:
> http://openjdk.java.net/jeps/318
> (you can find links to binary builds and sandbox locations there)
>
> Webrev:
> http://cr.openjdk.java.net/~shade/epsilon/webrev.05/
>
> Notes:
>
> *) See how the whole things is _almost_ drop-in to hotspot/share/gc, without having arch-specific
> mess -- thanks to GC interface work done over last years;
>
> *) There are some leftovers due to GC barriers work in progress: templateTable_arm.cpp addition
> goes away after JDK-8201786 [1], c1_LIRGenerator.cpp change should go away after C1 barriers
> modularization is complete, graphKit.cpp change should go away after C2 barriers modularization is
> complete;
>
> *) Half of the webrev are Epsilon-specific tests. They take ~30s in release and ~60s in fastdebug
> on my desktop. They are not in tier1, so smoke testing time is not affected;
>
> *) UseEpsilonGC is experimental. This should be fine after conditional GC compilation [2], so
> vendors who are unwilling to extend whatever small notion [3] of support comes with experimental VM
> option, may choose not to build it.
>
> Testing: all platform builds, gc/epsilon on x86_64
>
> Thanks,
> -Aleksey
>
> [1] https://bugs.openjdk.java.net/browse/JDK-8201786
> [2] https://bugs.openjdk.java.net/browse/JDK-8200729
> [3] http://hg.openjdk.java.net/jdk/jdk/file/3661f31c6df4/src/hotspot/share/runtime/globals.hpp#l150
>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 836 bytes
Desc: OpenPGP digital signature
URL: <https://mail.openjdk.org/pipermail/hotspot-gc-dev/attachments/20180426/152b5a96/signature.asc>
More information about the hotspot-gc-dev
mailing list