[8u] RFA (S) 8059677: Thread.getName() instantiates Strings
Aleksey Shipilev
aleksey.shipilev at oracle.com
Tue Feb 2 12:20:18 UTC 2016
Thank you, pushed to jdk8-dev:
https://bugs.openjdk.java.net/browse/JDK-8148842
http://hg.openjdk.java.net/jdk8u/jdk8u-dev/hotspot/rev/efe013052465
http://hg.openjdk.java.net/jdk8u/jdk8u-dev/jdk/rev/2cdb8c0719ad
But, did I push it right? It got "Fix Versions: 8u82", while I expected
8u76. Have we forked 8u76 already?
Cheers,
-Aleksey
On 02/01/2016 05:38 PM, Seán Coffey wrote:
> Approved for jdk8u-dev.
>
> Regards,
> Sean.
>
> On 01/02/16 14:26, Roger Riggs wrote:
>> +1
>>
>> On 2/1/2016 7:05 AM, Aleksey Shipilev wrote:
>>> Hi,
>>>
>>> I would like to backport a simple performance optimization to 8u:
>>> https://bugs.openjdk.java.net/browse/JDK-8059677
>>>
>>> This change was in JDK 9 for at least a year now, and no problems were
>>> apparently discovered. The change comes in two parts:
>>> http://hg.openjdk.java.net/jdk9/jdk9/hotspot/rev/9dc4d4fc73ca
>>> http://hg.openjdk.java.net/jdk9/jdk9/jdk/rev/0bddfc90d139
>>>
>>> Review thread:
>>>
>>> http://mail.openjdk.java.net/pipermail/core-libs-dev/2014-November/029569.html
>>>
>>>
>>> Note that Roger had specific concerns about backporting, which seem
>>> unproblematic:
>>>
>>> http://mail.openjdk.java.net/pipermail/core-libs-dev/2014-November/029589.html
>>>
>>>
>>> JDK part applies without problems, and HS part needs some fuzz in
>>> vm/runtime/thread.cpp. These are the proposed webrevs against 8u, for
>>> convenience:
>>> http://cr.openjdk.java.net/~shade/8059677/8u/webrev.01.hs/
>>> http://cr.openjdk.java.net/~shade/8059677/8u/webrev.01.jdk/
>>>
>>> I re-ran full JPRT against 8u with this change, and it was clean.
>>>
>>> Cheers,
>>> -Aleksey
>>>
>>>
>>
>
More information about the jdk8u-dev
mailing list