RFR(S): 8195109: ServiceUtil::visible_oop is not needed anymore

[Stefan Karlsson]

Looks good.

StefanK

On 2018-03-20 20:39, Chris Plummer wrote:
> Hi,
> 
> New webrev:
> 
> http://cr.openjdk.java.net/~cjplummer/8195109/webrev.01/index.html
> 
> There was a build failure on solaris-sparc in threadSMR.cpp. References 
> to the Copy class were producing "unresolved symbol" errors. 
> threadSMR.cpp includes threadService.hpp, which no longer includes 
> serviceUtil.hpp (because it was removed). It looks like serviceUtil.hpp 
> indirectly included "utilities/copy.hpp", so now I include it directly 
> in threadSMR.cpp. The problem was only on solaris-sparc, so I assume on 
> other platforms there was platform dependent code indirectly pulling in 
> copy.hpp. In any case, it's now directly pulled in on all platforms.
> 
> thanks,
> 
> Chris
> 
> On 3/19/18 5:48 PM, Chris Plummer wrote:
>> Hello,
>>
>> Please review the following:
>>
>> https://bugs.openjdk.java.net/browse/JDK-8195109
>> http://cr.openjdk.java.net/~cjplummer/8195109/webrev.00/index.html
>>
>> The assert I added to make sure this is safe has been in place in 
>> jdk/jdk for almost 3 weeks with no issues (longer in jdk/hs).
>>
>> The webrev is missing the copyright update for threadService.hpp. I 
>> fixed it after noticing that.
>>
>> Testing is in progress. Running hs tiers 1, 2, and 3, and jdk tiers 1 
>> and 2. Also making sure all serviceability tests are run.
>>
>> thanks,
>>
>> Chris
> 
>