ObjectSynchronizer iterate only in-use monitors?

[Robbin Ehn]

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
> 
>