RFR (XS) 8209802: JFR failed to build when certain garbage collectors are disabled

Thomas Schatzl thomas.schatzl at oracle.com
Thu Nov 8 08:15:15 UTC 2018


Hi Derek,

  some comment in JIRA I accidentally came across that may be helfpul:

On Thu, 2018-11-01 at 16:05 -0700, Derek Thomson wrote:
> > 

[...]
> Thanks for the review! Comments inline. And new webrev at 
> http://cr.openjdk.java.net/~jcbeyler/8209802/webrev.01/
> 
[...]
> >   - in jfr.cpp, I would prefer the ifdef's completely removed the
> > TRACE_REQUEST_FUNC(G1HeapRegionInformation), if possible.
> > Otherwise, I think it is better to remove starting from the body of
> > this method.
> 
> Sorry, did you mean jfrPeriodic.cpp? In which case not sure what
> method you mean here. Can you clarify for me please?
>  
> >   - in jfrType.cpp the question is if compiling without g1 could 

Yes.

> > remove the entire declaration of G1HeapRegionTypeConstant.
> > 
> > Cc'ing hotspot-jfr-dev at openjdk.java.net as the jfr devs might have
> > an idea how to accomplish that.
> 
> Sure.

There has been an answer how to do this (I believe :)) in the comment
for this CR at 
https://bugs.openjdk.java.net/browse/JDK-8209802?focusedCommentId=14221326&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-14221326

"The preferred way is to have the JFR type constant registrations
repatriated to their natural origin. 
The registration mechanism now supports dynamic registrations, where it
originally only had static means to register. 

The ZGC JFR type constants can be referred to as precedents, see: 

ZStatisticsCounterTypeConstant 
ZStatisticsSamplerTypeConstant "

Hth (and does not confuse too much),
  Thomas




More information about the hotspot-jfr-dev mailing list