[11] 8181855: [Graal] runtime/ReservedStack/ReservedStackTest.java triggers: assert(thread->deopt_mark() == __null) failed: no stack overflow from deopt blob/uncommon trap

[dean.long at oracle.com]

New webrev:

http://cr.openjdk.java.net/~dlong/8181855/webrev.2/

The only change is to test/hotspot/jtreg/ProblemList-graal.txt

The only test failure was an existing bug (JDK-8206241).

dl

On 7/12/18 10:46 AM, dean.long at oracle.com wrote:
> Thanks Tom.  I'll retest with that patch.
>
> dl
>
> On 7/12/18 10:28 AM, Tom Rodriguez wrote:
>>
>>
>> dean.long at oracle.com wrote on 7/12/18 10:07 AM:
>>> Unfortunately, when compiler/graalunit/HotspotTest.java runs my test 
>>> in JDK, org.graalvm.compiler.test.SubprocessUtil thinks the 
>>> command-line is empty, causing a failure.  Vladimir or Katya, is 
>>> there anything special I need to do to run Graal unit tests in JDK?
>>
>> The tests that use SubprocessUtil in the JDK don't work so they are 
>> disabled in ProblemList-graal.txt.  Here's my edit to that file from 
>> the Graal update I was preparing.  It's also restores the regular 
>> ReservedStackTestCompiler test.
>>
>> diff -r e0028bb6dd3d test/hotspot/jtreg/ProblemList-graal.txt
>> --- a/test/hotspot/jtreg/ProblemList-graal.txt
>> +++ b/test/hotspot/jtreg/ProblemList-graal.txt
>> @@ -67,8 +67,6 @@
>>
>>  gc/TestNUMAPageSize.java 8194949   generic-all
>>
>> -runtime/ReservedStack/ReservedStackTestCompiler.java 8181855 
>> generic-all
>> -
>>  serviceability/jvmti/GetModulesInfo/JvmtiGetAllModulesTest.java 
>> 8195156   generic-all
>>
>>  compiler/compilercontrol/directives/LogTest.java 8181753 generic-all
>> @@ -174,6 +172,7 @@
>>  org.graalvm.compiler.hotspot.test.CompilationWrapperTest           
>> 8205081
>>  org.graalvm.compiler.hotspot.test.HsErrLogTest 8205081
>>  org.graalvm.compiler.hotspot.test.OptionsInFileTest 8205081
>> +org.graalvm.compiler.hotspot.test.ReservedStackAccessTest          
>> 8205081
>>
>> org.graalvm.compiler.replacements.test.classfile.ClassfileBytecodeProviderTest 
>>   8205081
>>  org.graalvm.compiler.replacements.test.classfile.RedefineIntrinsicTest 
>>           8205081
>>
>>
>>>
>>> dl
>>>
>>> On 7/11/18 4:20 PM, Vladimir Kozlov wrote:
>>>> Looks good.
>>>>
>>>> Vladimir
>>>>
>>>> On 7/11/18 3:33 PM, dean.long at oracle.com wrote:
>>>>> http://cr.openjdk.java.net/~dlong/8181855/webrev/
>>>>> https://bugs.openjdk.java.net/browse/JDK-8181855
>>>>>
>>>>> This change adds "reserved stack" support to Graal for SPARC, x64, 
>>>>> and AArch64.
>>>>> This change has already gone upstream into Graal, except for the 
>>>>> AOT change in aotCodeHeap.cpp.
>>>>>
>>>>> dl*
>>>>> *
>>>
>