RFR(XXS) 8187629: NMT: Memory miscounting in compiler (C2)

[Vladimir Kozlov]

On 9/19/17 9:28 AM, Andrew Dinn wrote:
> On 19/09/17 17:14, Vladimir Kozlov wrote:
>> Compilers also use a lot thread local ResourceArea -
>> Thread::current()->resource_area() and NEW_RESOURCE_ARRAY() macro.
>> But thread local area is defined as mtThread:
>>
>> http://hg.openjdk.java.net/jdk10/hs/hotspot/file/5ab7a67bc155/src/share/vm/runtime/thread.cpp#l218
> True, although that's arguably part of the fixed cost of creating a thread.
> 
> By contrast, the changes Zhengyu has made are for data areas
> specifically allocated during a compilation to allow that compile to
> proceed.

I totally agree with fix. But I think we should do more to account 
memory used by compilers. Thread local area is huge part of it.

Thanks,
Vladimir

> 
> I don't know for sure which way to sway on that first count. The second
> one definitely needs debiting against the compiler account.
> 
> regards,
> 
> 
> Andrew Dinn
> -----------
> Senior Principal Software Engineer
> Red Hat UK Ltd
> Registered in England and Wales under Company Registration No. 03798903
> Directors: Michael Cunningham, Michael ("Mike") O'Neill, Eric Shander
>