[prev in list] [next in list] [prev in thread] [next in thread]
List: openjdk-serviceability-dev
Subject: Re: RFR(S): 8195109: ServiceUtil::visible_oop is not needed anymore
From: "serguei.spitsyn () oracle ! com" <serguei ! spitsyn () oracle ! com>
Date: 2018-03-21 18:54:34
Message-ID: 754fc246-b7b8-9caf-d0ae-2546bb25acd5 () oracle ! com
[Download RAW message or body]
Hi Chris,
It looks good.
Thanks,
Serguei
On 3/20/18 12: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
>
>
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic