Request for code review - JDK-8141123 Change int's to size_t in FreeIdSet

Kim Barrett kim.barrett at oracle.com
Wed Dec 2 01:33:35 UTC 2015


On Dec 1, 2015, at 8:23 PM, Kim Barrett <kim.barrett at oracle.com> wrote:
> 
> On Dec 1, 2015, at 6:45 PM, Kim Barrett <kim.barrett at oracle.com> wrote:
>> 
>> However, given that we've apparently never had any callers of
>> set_safepoint, it is perhaps OK to go ahead and remove it, and have
>> additional followups to look into the various issues mentioned here.
> 
> Yet more problems with the existing code.
> 
> A FreeIdSet is constructed with a pointer to a Monitor, which is used
> to protect it.  The Monitor that is provided for the FreeIdSet used by
> DirtyCardQueueSet is the Shared_DirtyCardQ_lock, which is actually a
> Mutex.  

No, I take this back.  Wrong lock…




More information about the hotspot-gc-dev mailing list