RFR(XXS) 8068592 Remove unused code in objectMonitor.hpp

[Daniel D. Daugherty]

Jerry,

Thanks for the quick review! And for the sanity check...

Dan


On 6/7/16 3:12 PM, Gerald Thornbrugh wrote:
> Hi Dan,
>
> I searched for the same patterns and did not find any issues.
>
> Your change looks good to me.
>
> Jerry
>> Greetings,
>>
>> I have a fix ready for the following RFE:
>>
>>     JDK-8068592 Remove unused code in objectMonitor.hpp
>>     https://bugs.openjdk.java.net/browse/JDK-8068592
>>
>> Yes, this an RFE and I'm fixing it now for a few reasons:
>>
>> 1) I promised Claes Redestad back on 04.25 that this would be the first
>>    issue I worked on when I got my Solaris-X64 environment back. mt-haku
>>    is now running Solaris 11.2 SRU5.5 so it is back!
>>
>> 2) This is the simplest change in my queue with little to no risk so it
>>    makes the perfect test case for my upgrade of mt-haku from Solaris
>>    10u11 -> Solaris 11.2 SRU5.5.
>>
>> 3) This change gets rid of unused fields, unused macro assembler 
>> functions
>>    and a couple of declared but not defined functions from 
>> ObjectMonitor.
>>    Slimming down ObjectMonitor is never a bad thing!
>>
>> Yes, I know that the process for getting approval for RFEs is not yet
>> defined. However, I want to get the code review out of the way so that
>> this changeset can go in if it is approved.
>>
>> Sanity check: I searched a JDK9-hs clone for the following patterns:
>>
>>     FreeNext_offset_in_bytes
>>     Responsible_offset_in_bytes
>>     Spinner_offset_in_bytes
>>     TrySpin_Fixed
>>     TrySpin_VaryFrequency
>>     WaitSet_offset_in_bytes
>>     _SpinClock
>>     _SpinFreq
>>     _SpinState
>>
>> and found no mentions outside of the ones that I'm deleting from
>> src/share/vm/runtime/objectMonitor.hpp.
>>
>> Webrev URL: 
>> http://cr.openjdk.java.net/~dcubed/8068592-webrev/0-jdk9-hs-hotspot/
>>
>> Testing: JPRT test builds are in process.
>>
>> Thanks, in advance, for any comments, suggestions or feedback.
>>
>> Dan
>