RFR: 8212107: VMThread issues and cleanup [v7]

[Robbin Ehn]

On Fri, 25 Sep 2020 15:15:21 GMT, Claes Redestad <redestad at openjdk.org> wrote:

>> Robbin Ehn has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev
>> excludes the unrelated changes brought in by the merge/rebase. The pull request contains 18 additional commits since
>> the last revision:
>>  - Merge branch 'master' into 8212107-vmthread
>>  - Spelling fixes from Dan
>>  - Removed assert as suggested by David
>>  - Whitespace fixes, thanks to Shipilev
>>  - Update after Shipilev
>>  - Update after Coleen and David
>>  - Merge branch 'master' into 8212107-vmthread
>>  - Fixed nits
>>  - Merge branch 'master' into 8212107-vmthread
>>  - Fixes after review from shipilev.
>>  - ... and 8 more: https://git.openjdk.java.net/jdk/compare/adff7b86...371ac282
>
> src/hotspot/share/runtime/mutexLocker.hpp line 62:
> 
>> 60: extern Mutex*   RetData_lock;                    // a lock on installation of RetData inside method data
>> 61: extern Monitor* VMOperation_lock;                // a lock on queue of vm_operations waiting to execute
>> 62: extern Monitor* VMOperationRequest_lock;         // a lock on Threads waiting for a vm_operation to terminate
> 
> Can the declaration of VMOperationRequest_lock be removed now too? Since it's no longer being defined in mutexLocker.cpp

Fixing, pushing later.

-------------

PR: https://git.openjdk.java.net/jdk/pull/228