On 19/03/2020 08:35, Denghui Dong wrote: > > Make sense, updated. > > Webrev: http://cr.openjdk.java.net/~ddong/8238665/webrev.04/ > Thanks for the updates, looks much better. A minor comment on ManagementFactoryHelper is that bufferPools doesn't have to initialized to null. That will avoid a volatile write. -Alan.