RFR 8236878: Use atomic instruction to update StringDedupTable's entries and entries_removed counters
zgu at redhat.com
Thu Jan 16 23:37:48 UTC 2020
On 1/16/20 3:51 AM, Aleksey Shipilev wrote:
> On 1/15/20 10:29 PM, Zhengyu Gu wrote:
>> Updated webrev:
> OK, thanks for explaining. I guess that makes sense.
> This comment is outdated then:
> 480 // Delayed update to avoid contention on the table lock
> I'd suggest to rewrite it to:
> // Do atomic update here instead of taking StringDedupTable_lock. This allows concurrent
> // cleanup when multiple workers are cleaning up the table, while the mutators are blocked
> // on StringDedupTable_lock.
Updated as you suggested and pushed.
> ...or some such.
More information about the hotspot-gc-dev