> On Nov 9, 2016, at 10:13 AM, David DeHaven wrote:
>
>
> Please review the fix for:
> https://bugs.openjdk.java.net/browse/JDK-8169001
>
> Webrev at:
> http://cr.openjdk.java.net/~ksrini/8169001/webrev.00/
Overall this looks like a
Please review the fix for:
https://bugs.openjdk.java.net/browse/JDK-8169001
Webrev at:
http://cr.openjdk.java.net/~ksrini/8169001/webrev.00/
>>>
>>> Overall this looks like a complete eradication of the launcher
>>> ergonomics. A few specific comments:
>>>
>>>
Hi Kumar,
FYI regarding the test, I was mis-remembering - it was actually an issue
with the VM's notion of is_server_class_machine(), not the launcher.
It's used for GC ergonomic selection.
Cheers,
David
On 9/11/2016 3:28 AM, Kumar Srinivasan wrote:
Hi David,
Thanks for the review please
Hi David,
Thanks for the review please see embedded comments:
Hi Kumar,
On 8/11/2016 4:47 AM, Kumar Srinivasan wrote:
Hello,
Please review the fix for:
https://bugs.openjdk.java.net/browse/JDK-8169001
Webrev at:
http://cr.openjdk.java.net/~ksrini/8169001/webrev.00/
Overall this looks
Build changes look ok.
/Erik
On 2016-11-07 19:47, Kumar Srinivasan wrote:
Hello,
Please review the fix for:
https://bugs.openjdk.java.net/browse/JDK-8169001
Webrev at:
http://cr.openjdk.java.net/~ksrini/8169001/webrev.00/
Background:
Launcher ergonomics was introduced last decade to help
Hi Kumar,
On 8/11/2016 4:47 AM, Kumar Srinivasan wrote:
Hello,
Please review the fix for:
https://bugs.openjdk.java.net/browse/JDK-8169001
Webrev at:
http://cr.openjdk.java.net/~ksrini/8169001/webrev.00/
Overall this looks like a complete eradication of the launcher
ergonomics. A few
Hello,
Please review the fix for:
https://bugs.openjdk.java.net/browse/JDK-8169001
Webrev at:
http://cr.openjdk.java.net/~ksrini/8169001/webrev.00/
Background:
Launcher ergonomics was introduced last decade to help determine
if the execution system is "Server Class", this was necessary to