RFR (XS): 8186480: Remove warning when AssumeMP is true and ergonomics determines to use one worker thread

Thomas Schatzl thomas.schatzl at oracle.com
Thu Nov 9 12:17:37 UTC 2017


Hi all,

  can I have reviews for this change that removes some imho useless
warning that confuses some tests?

In particular, if AssumeMP is true, and ergonomics determines that it
only wants to use one thread (e.g. in a container/VM), it gives a
*warning * about that AssumeMP and the ergonomics decision seems to
contradict each other.

The problem is that, some time ago AssumeMP has been made true default,
and already deprecated in 10 (JDK-8185062) and the plan is to obsolete
it, so this message shows up in some tests if run on e.g. a single-
thread VM.

I suggest to remove this message because of this and it does not seem
to be terribly useful: to get only one thread by ergonomics you need to
explicitly run the VM in an environment that only provides a single
thread. It seems logical that in this case the user can expect (given
he relies on ergonomics) that the GC will only use one thread.

And alternative could be to only show this message if the user
explicitly sets AssumeMP to true, but given that AssumeMP is on its way
out and the above reasoning about the usefulness of the message I think
it is okay to just remove that message.

CR:
https://bugs.openjdk.java.net/browse/JDK-8186480
Webrev:
http://cr.openjdk.java.net/~tschatzl/8186480/webrev/
Testing:
compilation

Thanks,
  Thomas




More information about the hotspot-gc-dev mailing list