RFR (simple) 8068352: Move virtualspace.* out of src/share/vm/runtime to memory directory

Coleen Phillimore coleen.phillimore at oracle.com
Tue Apr 28 20:50:43 UTC 2015


Thanks Serguei, the SA tests all pass.
Coleen

On 4/28/15, 4:28 PM, serguei.spitsyn at oracle.com wrote:
> It looks good.
> It was nice catch with the SA.
> I hope, the test results will be good.
>
> Thanks,
> Serguei
>
> On 4/28/15 11:18 AM, Coleen Phillimore wrote:
>>
>> Thanks to Stefan.  Moved and fixed SA VirtualSpace.java also.  I 
>> reran the sajdi tests.
>>
>> open webrev at http://cr.openjdk.java.net/~coleenp/8068352.02/
>>
>> thanks,
>> Coleen
>>
>> On 4/28/15, 1:50 PM, Coleen Phillimore wrote:
>>>
>>> On 4/28/15, 11:29 AM, Stefan Karlsson wrote:
>>>> On 2015-04-28 16:28, Coleen Phillimore wrote:
>>>>> This saves me 6 backspaces every time I try to edit this file.
>>>>>
>>>>> Tested with no precompiled headers (my default now) and JPRT.
>>>>> I'll update the copyrights when I commit the changes.
>>>>>
>>>>> open webrev at http://cr.openjdk.java.net/~coleenp/8068352/
>>>>> bug link https://bugs.openjdk.java.net/browse/JDK-8068352
>>>>
>>>> What about the SA agent? :D
>>>
>>> Oh, Stefan, you know how to make my day.   I'll move that too and 
>>> retest.
>>>
>>> Coleen
>>>>
>>>> agent/src/share/classes/sun/jvm/hotspot/runtime/VirtualSpace.java
>>>>
>>>> Thanks,
>>>> StefanK
>>>>
>>>>>
>>>>> Thanks,
>>>>> Coleen
>>>>
>>>
>>
>



More information about the hotspot-dev mailing list