CMS parallel initial mark; vm thread sneaking in Thread::try_lock()

Thomas Schatzl thomas.schatzl at
Thu Jun 27 02:15:50 PDT 2013

Hi Hiroshi,

On Tue, 2013-06-25 at 19:05 -0700, Hiroshi Yamauchi wrote:
> Here's an updated
> webrev:
> Let me know if there are more comments.

Minor nitpick:


4516   if (_eden_chunk_array && !CMSEdenChunksRecordAlways) {

The != NULL has been forgotten here.

Thanks for considering to use the mutex in
CMSCollector::sample_eden_chunk(). It looks a lot cleaner now.

Looks good.


More information about the hotspot-runtime-dev mailing list