RFR (S): 8145534: TestRemsetLogging.java takes a long time

[Thomas Schatzl]

Hi Jon,

  thanks for your review.

On Wed, 2015-12-16 at 15:15 -0800, Jon Masamitsu wrote:
> Thomas,
> 
> http://cr.openjdk.java.net/~tschatzl/8145534/webrev/test/gc/g1/TestRemsetLoggingThreads.java.frames.html
> 
> Did you consider adding the -Xint to the execution line above?

The test only runs the VM with -version. No java code is executed.

> 
> http://cr.openjdk.java.net/~tschatzl/8145534/webrev/test/gc/g1/TestRemsetLoggingPerRegion.java.frames.html
> 
> Why is the @requires needed above?  It's telling jtreg that 
> TestRemsetLoggingPerRegion
> needs to be run with G1, right.  Is that really the case?
> 
> Rest looks good.

Yes. The option and log message is G1 only.

New webrev at http://cr.openjdk.java.net/~tschatzl/8145534/webrev.1
(sorry no diff, I messed up)

Thanks,
  Thomas