RFR: 8213224: Move code related to GC threads calculation out of AdaptiveSizePolicy

Roman Kennke rkennke at redhat.com
Fri Dec 7 20:00:21 UTC 2018


It is already pushed, but I noticed you have removed the definition of
Abstract_VM_Version::parallel_worker_threads() but the declaration is
still there. One can code against this but then get linker failure.
Maybe also remove:

static unsigned int parallel_worker_threads();

in vm_version.hpp?

Roman

> Could anyone review this cleanup change?
> In addition to straight code movement, it removes unnecessary dependency
> on adaptiveSizePolicy.hpp in G1 code, and removed unnecessary include
> statements in adaptiveSizePolicy.hpp/cpp.
> 
> Webrev: https://cr.openjdk.java.net/~manc/8213224/webrev.00/
> RFE: https://bugs.openjdk.java.net/browse/JDK-8213224
> Tested on submit repo with the help from JC (CCed).
> 
> Thanks,
> Man

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: OpenPGP digital signature
URL: <https://mail.openjdk.org/pipermail/hotspot-gc-dev/attachments/20181207/712f06f7/signature.asc>


More information about the hotspot-gc-dev mailing list