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