On Thu, 30 Nov 2023 01:43:28 GMT, Jonathan Joo wrote:
>> (I just realized that I made a typo in my previous msg; should be *callee*
>> instead.) That is what I have in mind.
>>
>>
>> void CPUTimeCounters::update_counter(name, total) {
>> auto counter = get_counter(name);
>> auto old_v = co
On Wed, 29 Nov 2023 15:23:55 GMT, Albert Mingkun Yang wrote:
>> We could add a new closure just used by GC that 's a sub-class of
>> `ThreadTotalCPUTimeClosure` and just adds this to the constructor:
>>
>> instance->inc_gc_total_cpu_time(net_cpu_time);
>>
>>
>> That way we could get rid of `
On Thu, 23 Nov 2023 12:43:00 GMT, Albert Mingkun Yang wrote:
>> Jonathan Joo has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> Cleanup and address comments
>
> src/hotspot/share/gc/g1/g1CollectedHeap.cpp line 2433:
>
>> 2431: }
>> 2432:
On Wed, 29 Nov 2023 08:31:37 GMT, Stefan Johansson wrote:
>> src/hotspot/share/runtime/cpuTimeCounters.cpp line 119:
>>
>>> 117: if (CPUTimeGroups::is_gc_counter(_name)) {
>>> 118: instance->inc_gc_total_cpu_time(net_cpu_time);
>>> 119: }
>>
>> I feel much of this is on the wrong
On Thu, 23 Nov 2023 13:46:54 GMT, Albert Mingkun Yang wrote:
>> Jonathan Joo has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> Cleanup and address comments
>
> src/hotspot/share/runtime/cpuTimeCounters.cpp line 119:
>
>> 117: if (CPUT
On Wed, 22 Nov 2023 23:08:36 GMT, Jonathan Joo wrote:
>> 8315149: Add hsperf counters for CPU time of internal GC threads
>
> Jonathan Joo has updated the pull request incrementally with one additional
> commit since the last revision:
>
> Cleanup and address comments
My additional testing f
On Wed, 22 Nov 2023 23:08:36 GMT, Jonathan Joo wrote:
>> 8315149: Add hsperf counters for CPU time of internal GC threads
>
> Jonathan Joo has updated the pull request incrementally with one additional
> commit since the last revision:
>
> Cleanup and address comments
src/hotspot/share/gc/g1
On Wed, 22 Nov 2023 23:08:36 GMT, Jonathan Joo wrote:
>> 8315149: Add hsperf counters for CPU time of internal GC threads
>
> Jonathan Joo has updated the pull request incrementally with one additional
> commit since the last revision:
>
> Cleanup and address comments
I think this looks good
> 8315149: Add hsperf counters for CPU time of internal GC threads
Jonathan Joo has updated the pull request incrementally with one additional
commit since the last revision:
Cleanup and address comments
-
Changes:
- all: https://git.openjdk.org/jdk/pull/15082/files
- new: ht
