JDK-8171119: Low-Overhead Heap Profiling
Robbin Ehn
robbin.ehn at oracle.com
Mon Jan 29 09:29:51 UTC 2018
Hi JC, thanks!
I'm happy with current state, looks good!
Truncated:
On 01/27/2018 05:01 AM, JC Beyler wrote:
> This is strange but I'm assuming it is because we are not working on
> the same repo?
>
> I used:
> hg clone http://hg.openjdk.java.net/jdk/hs jdkhs-heap
>
> I'll try a new clone on Monday and see. My new version moved hard_end
> back to public so it should work now.
Sorry this compile error was in closed code.
Now the closed part compiles, thanks.
>
> Fair enough, hopefully Thomas will chime in. Are you saying that this
> first version could go in and we can work on a refinement? Or are you
> saying I should work on this now at the same time and fix it before
> this V1 goes in? (Just so I know :))
We may have to change this before integration, but for now keep it as is.
> I'll look at this on Monday then!
Great!
/Robbin
>
> Thanks for the reply and have a great weekend!
> Jc
>
>>
>>>
>>>> ####
>>>> Minor nit, when declaring pointer there is a little mix of having the
>>>> pointer adjacent by type name and data name. (Most hotspot code is by
>>>> type
>>>> name)
>>>> E.g.
>>>> heapMonitoring.cpp:711 jvmtiStackTrace *trace = ....
>>>> heapMonitoring.cpp:733 Method* m = vfst.method();
>>>> (not just this file)
>>>>
>>>
>>> Done!
>>>
>>>> ####
>>>> HeapMonitorThreadOnOffTest.java:77
>>>> I would make g_tmp volatile, otherwise the assignment in loop may
>>>> theoretical be skipped.
>>>>
>>>
>>> Also done!
>>
>>
>> Looks good, thanks!
>>
>> /Robbin
>>
>>>
>>> Thanks again!
>>> Jc
>>>
>>
More information about the serviceability-dev
mailing list