Skip to content

Commit

Permalink
8321938: java/foreign/critical/TestCriticalUpcall.java does not need …
Browse files Browse the repository at this point in the history
…a core file

Reviewed-by: dholmes
  • Loading branch information
Daniel D. Daugherty committed Jan 18, 2024
1 parent 806ffb1 commit a22ae90
Showing 1 changed file with 3 additions and 2 deletions.
5 changes: 3 additions & 2 deletions test/jdk/java/foreign/critical/TestCriticalUpcall.java
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
/*
* Copyright (c) 2023, Oracle and/or its affiliates. All rights reserved.
* Copyright (c) 2023, 2024, Oracle and/or its affiliates. All rights reserved.
* DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
*
* This code is free software; you can redistribute it and/or modify it
Expand Down Expand Up @@ -35,6 +35,7 @@
import java.lang.foreign.Linker;
import java.lang.foreign.MemorySegment;
import java.lang.invoke.MethodHandle;
import java.util.List;

import static org.testng.Assert.fail;

Expand All @@ -43,7 +44,7 @@ public class TestCriticalUpcall extends UpcallTestHelper {
@Test
public void testUpcallFailure() throws IOException, InterruptedException {
// test to see if we catch a trivial downcall doing an upcall
runInNewProcess(Runner.class, true)
runInNewProcess(Runner.class, true, List.of("-XX:-CreateCoredumpOnCrash"), List.of())
.shouldNotHaveExitValue(0)
.stdoutShouldContain("wrong thread state for upcall");
}
Expand Down

0 comments on commit a22ae90

Please sign in to comment.