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

Thanks Vladimir.

dl

On 7/12/18 2:30 PM, Vladimir Kozlov wrote:
> Good.
>
> Thanks,
> Vladimir
>
> On 7/12/18 2:15 PM, dean.long at oracle.com wrote:
>> 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*
>>>>>>> *
>>>>>
>>>
>>