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

[Tom Rodriguez]

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