RFR(XS): 8026708: guarantee(codelet_size > 0 && (size_t)codelet_size > 2*K) failed: not enough space for interpreter generation

[Vladimir Kozlov]

Good. Thank you again, Albert

Vladimir

On 10/17/13 1:46 AM, Albert Noll wrote:
> Christian, Vladimir, thanks for looking at the patch.
>
> Increasing the space that is reserved for the template interpreter leaves 45K (2K reserved for alignment) of availbale
> space if the
> following options are used (64-bit):
>
> java -Xmixed -server -d64 -XX:+UseParallelGC -XX:+IgnoreUnrecognizedVMOptions -XX:+UnlockDiagnosticVMOptions
> -XX:+PrintInterpreter -XX:-UseCompressedOops -Xdebug -Xrunjdwp:transport=dt_socket,server=y,suspend=n,address=8000
> -XX:+VerifyOops
>
> code size        =    208K bytes
> total space      =    255K bytes
> wasted space     =     47K bytes
>
> # of codelets    =    264
> avg codelet size =    808 bytes
>
>
> I was not aware that '-Xdebug -Xrunjdwp:transport=dt_socket,server=y,suspend=n,address=8000' requires so much space
> in the interpreter. Sorry for that!
>
> I am fine with the current solution (256k for 64-bit and 224k for 32-bit).
>
> Best,
> Albert
>
>
> On 16.10.2013 22:58, Vladimir Kozlov wrote:
>> Good.
>>
>> Thanks,
>> Vladimir
>>
>> On 10/16/13 1:06 PM, Albert Noll wrote:
>>> Hi Vladimir,
>>>
>>> thanks for the review. I agree, this gives us more air.
>>> Here is the new webrev:
>>> http://cr.openjdk.java.net/~anoll/8026708/webrev.01/
>>> <http://cr.openjdk.java.net/%7Eanoll/8026708/webrev.01/>
>>>
>>> Best,
>>> Albert
>>>
>>> P.S.: I had the wrong header to the mail. The bug id is actually 8026708.
>>>
>>>
>>>
>>> On 16.10.2013 21:28, Vladimir Kozlov wrote:
>>>> I think we should increase it more for future otherwise you will have
>>>> to always catch up with interpreter changes.
>>>>
>>>> Increase it to 256 * 1024 and 224 * 1024
>>>>
>>>> Vladimir
>>>>
>>>> On 10/16/13 12:22 PM, Albert Noll wrote:
>>>>> Hi,
>>>>>
>>>>> could I have a review for this patch?
>>>>>
>>>>> bug: https://bugs.openjdk.java.net/browse/JDK-8026708
>>>>> webrev: http://cr.openjdk.java.net/~anoll/8026708/webrev.00/
>>>>> <http://cr.openjdk.java.net/%7Eanoll/8026708/webrev.00/>
>>>>>
>>>>> Problem: Not enough room for interpreter. My last patch did not solve
>>>>> the problem for solaris-amd64.
>>>>>                 A local build (solaris-amd64) of the most recent
>>>>> hotspot-comp version requires a template interpreter
>>>>>                 size of 211K (obtained with -XX:+PrintInterpreter).
>>>>> There have been some modifications to the template
>>>>>                 interpreter in the last couple of weeks which might have
>>>>> triggered this error.
>>>>>
>>>>> Solution: Increase interpreter size by 8k (32-bit and 64-bit).
>>>>>
>>>>> Testing: Failing test case in solaris-amd64
>>>>>
>>>>>
>>>>> Many thanks in advance,
>>>>> Albert
>>>
>