[7u60] Request for approval: 7194607, 8005128, 8006542, 8006546, 8006731

[Alejandro E Murillo]

Hi Serguei,
you do not need to get approval to push these changes,
as I do that when I do a bulk integration of the hotspot snapshot,
you just need to get them reviewed.

Also, the repo is: hsx/jdk7u/hotspot (note not dev)
http://hg.openjdk.java.net/hsx/jdk7u/hotspot/

Thanks
Alejandro


On 1/22/2014 5:09 PM, serguei.spitsyn at oracle.com wrote:
> Thanks, Rob!
>
> As these are hotspot fixes, I assume, they must be pushed to 
> hsx/7udev, not jdk7u/jdk7u-dev, right?
>
> Thanks,
> Serguei
>
> On 1/22/14 4:57 AM, Rob McKenna wrote:
>> Approved.
>>
>>     -Rob
>>
>> On 22/01/14 05:31, serguei.spitsyn at oracle.com wrote:
>>> Hello,
>>>
>>> Requesting approval to backport from JDK 8 first chunk (5 of 12 
>>> fixes) of the JSR-292 support
>>> in the JVMTI HotSwap API (includes RedefineClasses, 
>>> RetransformClasses and PopFrame).
>>>
>>> All the fixes below:
>>>  - have a label 7u60-critical-approved
>>>  - have been already integrated into the JDK 8 and tested in the 
>>> hotspot-rt nightly for several months
>>>
>>>
>>>   https://jbs.oracle.com/bugs/browse/JDK-7194607
>>>   jdk8 changeset: 
>>> http://hg.openjdk.java.net/jdk8/jdk8/hotspot/rev/64812523d72e
>>>   7u: 
>>> http://javaweb.sfbay.sun.com/java/svc/ss45998/webrevs/2014/hotspot/7u_port/7u_webrevs/7194607
>>>   Review thread: 
>>> http://mail.openjdk.java.net/pipermail/serviceability-dev/2012-October/007354.html
>>>
>>>  https://jbs.oracle.com/bugs/browse/JDK-8005128
>>>   jdk8 changeset: 
>>> http://hg.openjdk.java.net/hsx/hsx25/hotspot/rev/edd76a5856f7
>>>   7u webrev: 
>>> http://javaweb.sfbay.sun.com/java/svc/ss45998/webrevs/2014/hotspot/7u_port/7u_webrevs/8005128
>>>   Review thread: 
>>> http://mail.openjdk.java.net/pipermail/serviceability-dev/2013-January/008002.html
>>>
>>>  https://jbs.oracle.com/bugs/browse/JDK-8006542
>>>   jdk8 changeset: 
>>> http://hg.openjdk.java.net/hsx/hsx25/hotspot/rev/9bf5f643d1cf
>>>   7u webrev: 
>>> http://javaweb.sfbay.sun.com/java/svc/ss45998/webrevs/2014/hotspot/7u_port/7u_webrevs/8006542
>>>   Review thread: 
>>> http://mail.openjdk.java.net/pipermail/serviceability-dev/2013-January/008080.html
>>>
>>>  https://jbs.oracle.com/bugs/browse/JDK-8006546
>>>   jdk8 changeset: 
>>> http://hg.openjdk.java.net/hsx/hsx25/hotspot/rev/dc31f560d6e7
>>>   7u webrev: 
>>> http://javaweb.sfbay.sun.com/java/svc/ss45998/webrevs/2014/hotspot/7u_port/7u_webrevs/8006546
>>>   Review thread: 
>>> http://mail.openjdk.java.net/pipermail/serviceability-dev/2013-January/008149.html
>>>
>>>  https://jbs.oracle.com/bugs/browse/JDK-8006731
>>>   jdk8 changeset: 
>>> http://hg.openjdk.java.net/hsx/hsx25/hotspot/rev/79c1bb8fce5d
>>>   7u webrev: 
>>> http://javaweb.sfbay.sun.com/java/svc/ss45998/webrevs/2014/hotspot/7u_port/7u_webrevs/8006731
>>>   Review thread: 
>>> http://mail.openjdk.java.net/pipermail/serviceability-dev/2013-January/008150.html
>>>
>>>
>>>  Accumulative webrev based on the hsx/jdk7u-dev repo:
>>> http://javaweb.sfbay.sun.com/java/svc/ss45998/webrevs/2014/hotspot/7u_port/JSR292-chunk1/ 
>>>
>>>
>>>
>>> Thanks,
>>> Serguei
>>
>

-- 
Alejandro