Feature #2480: request to add GC::Profiler.time method

• Status: Closed
• Assignee: authorNari (Narihiro Nakamura)
• Target version: 2.0.0

[rogerdpack (Roger Pack)]

=begin
currently we have
GC::Profiler.enable

... stuff

report = GC::Profiler.result # report is set to a verbose ascii string

What I would like is to be able to add garbage time to profiling information [i.e. "within this method, of the total time that was used, GC took 42ms")

GC::Profiler.time
=> float of the total time used (ms).

or could call it time_used or total_time or whatever.

Then it would make adding GC time to profiler time much faster, as I wouldn't have to parse the result output each time to calculate my own total (it would just be a matter of calling Profiler.time, seeing if the time has increased since last setting).

Thanks.
-r
=end

[nobu (Nobuyoshi Nakada)]

• Category set to core
• Status changed from Open to Assigned
• Assignee set to authorNari (Narihiro Nakamura)
• Target version set to 2.0.0

=begin

=end

[mame (Yusuke Endoh)]

• Category deleted (core)
• Status changed from Assigned to Open
• Assignee deleted (authorNari (Narihiro Nakamura))
• Target version deleted (2.0.0)

=begin
Hi,

What I would like is to be able to add garbage time to profiling information [i.e. "within this method, of the total time that was used, GC took 42ms")

If anyone (nari?) writes a patch, it can be accepted and committed
because no one has disagreed, I guess.

--
Yusuke Endoh [e-mail]
=end

[mame (Yusuke Endoh)]

• Category set to core
• Status changed from Open to Assigned
• Assignee set to authorNari (Narihiro Nakamura)
• Target version set to 2.0.0

=begin

=end

[wanabe (_ wanabe)]

• File gc_prof_totaltime.patch gc_prof_totaltime.patch added

=begin
I wrote a patch.
If it is OK, I'll commit later.
=end

[mame (Yusuke Endoh)]

=begin
Hi,

2010/3/3 _ wanabe [e-mail]:

File gc_prof_totaltime.patch added

I wrote a patch.
If it is OK, I'll commit later.

+1

The name `total_time' is also good for me.

--
Yusuke ENDOH [e-mail]

=end

[authorNari (Narihiro Nakamura)]

=begin
Hi,

I wrote a patch.
If it is OK, I'll commit later.

+1

The name `total_time' is also good for me.

Me, too.

--
Narihiro Nakamura (nari)

=end

[wanabe (_ wanabe)]

• Status changed from Assigned to Closed

=begin
Applied at r26814. Thank you.
=end

[rogerdpack (Roger Pack)]

=begin
Many thanks. I will use this in the ruby-prof gem most likely, for more intuitive profiling.
=end