RFR(T): 8234274 [BACKOUT] JDK-8204128 NMT might report incorrect numbers for Compiler area

[Daniel D. Daugherty]

Thanks for the confirmation.

Dan


On 11/15/19 9:17 PM, Zhengyu Gu wrote:
>
>
> On 11/15/19 9:12 PM, Daniel D. Daugherty wrote:
>> No problem. Please confirm you are good with the backout.
> Yes, I am good with backing out.
>
>>
>> Leonid says he can help with testing the [REDO]. I'll morph
>> https://bugs.openjdk.java.net/browse/JDK-8234270 into the REDO bug in 
>> a minute.
>
> I think that the new assertion may actually catch something about 
> original bug. It will be great can verify that.
>
> Thanks,
>
> -Zhengyu
>
>>
>> Dan
>>
>> On 11/15/19 9:10 PM, Zhengyu Gu wrote:
>>> Thanks, Dan.
>>>
>>> -Zhengyu
>>>
>>> On 11/15/19 9:08 PM, Daniel D. Daugherty wrote:
>>>> Greetings,
>>>>
>>>> The following fix needs to be backed out:
>>>>
>>>>      JDK-8204128 NMT might report incorrect numbers for Compiler area
>>>>      https://bugs.openjdk.java.net/browse/JDK-8204128
>>>>
>>>> The fix is causing failures in the stress tiers of the JDK14 CI. 
>>>> Please
>>>> see the info added to the following bug by Leonid:
>>>>
>>>>      JDK-8234270 HugeArenaTracking.java failed assert(_size >= sz) 
>>>> failed: deallocation > allocated
>>>>      https://bugs.openjdk.java.net/browse/JDK-8234270
>>>>
>>>> To keep the noise in the CI down over the weekend, I'm backing out the
>>>> fix using the following bug:
>>>>
>>>>      JDK-8234274 [BACKOUT] JDK-8204128 NMT might report incorrect 
>>>> numbers for Compiler area
>>>>      https://bugs.openjdk.java.net/browse/JDK-8234274
>>>>
>>>> I'm also backing out the following fix:
>>>>
>>>>      JDK-8234272 ProblemList runtime/NMT/HugeArenaTracking.java
>>>>
>>>> Doesn't make sense to ProblemList a test that has been deleted.
>>>>
>>>> Here's the webrev for the backout:
>>>>
>>>> http://cr.openjdk.java.net/~dcubed/8234274-webrev/0-for-jdk14/
>>>>
>>>> I have manually verified the backout against the changeset that 
>>>> precedes
>>>> the changeset for JDK-8204128.
>>>>
>>>> Thanks, in advance, for any comments, questions or suggestions.
>>>>
>>>> Dan
>>>>
>>>>
>>>>
>>>
>>
>
>