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

[Zhengyu Gu]

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
>>>
>>>
>>>
>>
>