RFR(S): 8195109: ServiceUtil::visible_oop is not needed anymore
chris.plummer at oracle.com
Tue Mar 20 19:39:47 UTC 2018
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.
On 3/19/18 5:48 PM, Chris Plummer wrote:
> Please review the following:
> 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.
More information about the hotspot-runtime-dev