On Dec 14, 2015, at 9:53 AM, Marcus Larsson <marcus.lars...@oracle.com> wrote: > > Hi, > > New webrev: > http://cr.openjdk.java.net/~mlarsson/8145083/webrev.01/ > > Incremental: > http://cr.openjdk.java.net/~mlarsson/8145083/webrev.00-01/
One minor nit: ------------------------------------------------------------------------------ src/share/vm/logging/logConfiguration.hpp 72 class ConfigurationLocker { Shouldn't this be derived from StackObj? ------------------------------------------------------------------------------