Staffan,
Yes, I saw trace.xml. But I think this event not specified there, but
implemented )
I will appreciate if you copy it in the message, people might review
that and decide if the event contain all the necessary fields.
And thank you for working on tests!
-- Dima
On 27.08.2014 20:52, Staffan Friberg wrote:
HI Dima,
The event is specified in trace.xml as all other events, please see
the updated webrev as some changes was made to the descriptions.
http://cr.openjdk.java.net/~brutisso/8055845/webrev.01
I'm in the process of writing regression tests, but they will be a
separate webrev as they will be located in a separate repository.
Regards,
Staffan
On 08/27/2014 04:08 AM, Dmitry Fazunenko wrote:
Hi Staffan,
Thanks for submitting RFE and implementing it! This event seems to be
used widely.
So it's very important to design it correctly.
May I ask you to provide some extra information:
- format of event
(this could be included in the CR description)
- regression tests for that event will be also required
- please include hotspot-dev on CC
Thanks,
Dima
On 25.08.2014 20:28, Staffan Friberg wrote:
Hi,
Could I please get reviews for this RFE that adds a trace event for
aging and promotion for young collections in G1, CMS and Parallel GC.
It works similarly to allocation events, and generates the event on
the slow path when either a direct copy occurs or a new PLAB is
request.
Since I'm adding an event I cc:ed the serviceability list in case
anyone has any comments on the event and changes to trace.xml.
RFE: https://bugs.openjdk.java.net/browse/JDK-8055845
Webrev: http://cr.openjdk.java.net/~brutisso/8055845/webrev.00
Bengt has been kind and agreed to both sponsor and host the the webrev.
Thanks,
Staffan
