RFR: JDK-8145180: Add back PrintGC, PrintGCDetails and -Xloggc

[kirk at kodewerk.com]

Hi Bengt,

I think I commented on the loss of familiar flags earlier on so very happy to see them back. Not sure what prompted you to go through all the effort to put back all this code. I think I would have taken the lazy way out and just provided some mapping in Arguments. Given the size of the change in logging, getting it exactly right isn’t that important. That said, what is done is done and I don’t see any obvious.

Regards,
Kirk

> On Jan 18, 2016, at 4:52 PM, Bengt Rutisson <bengt.rutisson at oracle.com> wrote:
> 
> 
> Hi everyone,
> 
> Could I have a couple of reviews for this follow-up change to the GC logging changes for the unified logging framework?
> 
> http://cr.openjdk.java.net/~brutisso/8145180/webrev.00/
> https://bugs.openjdk.java.net/browse/JDK-8145180
> 
> The change adds back the flags PrintGC and PrintGCDetails as well as -Xloggc. I've tried to keep the behavior similar to before, so if -Xloggc is used the PrintGC and PrintGCDetails settings should affect what is logged to the file not to stdout.
> 
> The change also adds warning messages for all three flags that will let the users know that the flags have been deprecated.
> 
> I've tried to add this code back in similar places where the code used to be. I've put PrintGC and PrintGCDetails back in globals.hpp at the same place where they used to be. I've added a method, handle_deprecated_print_gc_flags(), which is called from the place in Arguments::parse() where we used to enable PrintGC if PrintGCDetails was enabled (http://hg.openjdk.java.net/jdk9/hs-rt/hotspot/rev/f944761a3ce3#l134.259). I also re-introduce the Arguments::_gc_log_filename variable to hold the file name specified by -Xloggc.
> 
> Thanks,
> Bengt

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 496 bytes
Desc: Message signed with OpenPGP using GPGMail
URL: <https://mail.openjdk.org/pipermail/hotspot-gc-dev/attachments/20160119/70e56416/signature.asc>