ObjectSynchronizer iterate only in-use monitors?

[Robbin Ehn]

Re-sending, since Davids reply got sorted into another folder,
so I accidentally dropped GC list and missed David H replies, sorry about that.

/Robbin

On 05/11/2017 10:06 AM, Robbin Ehn wrote:
> On 05/11/2017 10:01 AM, Aleksey Shipilev wrote:
>> On 05/11/2017 09:51 AM, Roman Kennke wrote:
>>> Am 11.05.2017 um 09:44 schrieb Robbin Ehn:
>>>> Let me know how you plan to continue this!
>>>
>>> I can change the patch to hook up thread-local in-use-monitor scanning
>>> to Thread::oops_do() and have ObjectSynchronizer::oops_do() only scan
>>> the gOmInUseList. I'd make it conditional on -XX:+MonitorInUseLists, for
>>> consistency and just in case anybody's ever turning this off, or else we
>>> should remove that flag? This change should transparently enable this
>>> for all GCs. I'll also file a new enhancement bug for it. Does that
>>> sound good?
>>
>> That sounds like a sane tactics to me.
> 
> +1
> 
> I guess first step is to deprecate it ? So we need a CSR for that. (CSR is not yet available)
> (assuming no objections here)
> 
> Yes, enable for all GCs sounds great to me, maybe run some numbers on them just in case.
> 
> Thanks!
> 
> /Robbin
> 
> 
>>
>> -Aleksey
>>
>>