RFR : 8211213: fix aix build after 8196341: Add JFR events for parallel phases of G1
matthias.baesken at sap.com
Fri Sep 28 06:53:26 UTC 2018
Hi Aleksey, sounds great - thanks for looking into this JFR generating stuff and figuring out what to do where !
So I guess we do not need my AIX-related patch any more ,
after your change is in ( and the commit-with parameters calls get usable as well on the non - JFR platforms ).
I think we could simplify then quite a few places in the hs codebase that deal with JFR events ( where currently set_*(param1) + ... set_*(param_n) + commit() is used, this could be just
commit(param1,...param_n) instead ) .
However this is probably out of scope of my (and your ) patch .
Best regards, Matthias
> -----Original Message-----
> From: Aleksey Shipilev <shade at redhat.com>
> Sent: Donnerstag, 27. September 2018 20:03
> To: Baesken, Matthias <matthias.baesken at sap.com>; Thomas Schatzl
> <thomas.schatzl at oracle.com>; 'hotspot-dev at openjdk.java.net' <hotspot-
> dev at openjdk.java.net>; 'build-dev at openjdk.java.net' <build-
> dev at openjdk.java.net>
> Subject: Re: RFR : 8211213: fix aix build after 8196341: Add JFR events for
> parallel phases of G1
> On 09/27/2018 04:39 PM, Baesken, Matthias wrote:
> > Hi Thomas, do you know some fast and helpful person who can bring the
> INCLUDE_JFR and ! INCLUDE_JFR cases in sync ?
> > (or maybe there was a reason to have these differences we observe ? )
> I am not a JFR person, but I think this should do it -- I copied
> printCommitMethod into
> printEmptyEvent and gutted it:
> This fixes x86_64 Zero build.
More information about the hotspot-dev