8292386: jvmti/thread/SuspendThread/suspendthrd03 failed with "FAIL: Status is 2"

Reviewed-by: sspitsyn, dcubed
This commit is contained in:
Leonid Mesnik 2022-10-12 14:59:19 +00:00
parent 760a260fbb
commit 0475c34120
2 changed files with 0 additions and 488 deletions

View File

@ -1,177 +0,0 @@
/*
* Copyright (c) 2003, 2022, 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
* under the terms of the GNU General Public License version 2 only, as
* published by the Free Software Foundation.
*
* This code is distributed in the hope that it will be useful, but WITHOUT
* ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
* FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License
* version 2 for more details (a copy is included in the LICENSE file that
* accompanied this code).
*
* You should have received a copy of the GNU General Public License version
* 2 along with this work; if not, write to the Free Software Foundation,
* Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA.
*
* Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA
* or visit www.oracle.com if you need additional information or have any
* questions.
*/
#include <string.h>
#include "jvmti.h"
#include "jvmti_common.h"
#include "jvmti_thread.h"
extern "C" {
/* scaffold objects */
static jlong timeout = 0;
/* constant names */
#define THREAD_NAME "TestedThread"
#define N_LATE_CALLS 10000
/** Agent algorithm. */
static void JNICALL
agentProc(jvmtiEnv *jvmti, JNIEnv *jni, void *arg) {
jvmtiError err;
/* Original agentProc test block starts here: */
LOG("Wait for thread to start\n");
// SP2.1-n - notify agent is waiting and wait
// SP3.1-w - wait to start test
if (!agent_wait_for_sync(timeout))
return;
/* perform testing */
{
jthread testedThread = NULL;
int late_count;
LOG("Find thread: %s\n", THREAD_NAME);
testedThread = find_thread_by_name(jvmti, jni, THREAD_NAME);
if (testedThread == NULL) {
return;
}
LOG(" ... found thread: %p\n", (void *) testedThread);
LOG("Suspend thread: %p\n", (void *) testedThread);
err = jvmti->SuspendThread(testedThread);
if (err != JVMTI_ERROR_NONE) {
set_agent_fail_status();
return;
}
LOG("Let thread to run and finish\n");
// SP5.1-n - notify suspend done
if (!agent_resume_sync())
return;
LOG("Get state vector for thread: %p\n", (void *) testedThread);
{
jint state = 0;
err = jvmti->GetThreadState(testedThread, &state);
if (err != JVMTI_ERROR_NONE) {
set_agent_fail_status();
return;
}
LOG(" ... got state vector: %s (%d)\n", TranslateState(state), (int) state);
if ((state & JVMTI_THREAD_STATE_SUSPENDED) == 0) {
LOG("SuspendThread() does not turn on flag SUSPENDED:\n"
"# state: %s (%d)\n", TranslateState(state), (int) state);
set_agent_fail_status();
}
}
LOG("Resume thread: %p\n", (void *) testedThread);
err = jvmti->ResumeThread(testedThread);
if (err != JVMTI_ERROR_NONE) {
set_agent_fail_status();
return;
}
/* Original agentProc test block ends here. */
/*
* Using LOG() instead of LOG() in this loop
* in order to slow down the rate of SuspendThread() calls.
*/
for (late_count = 0; late_count < N_LATE_CALLS; late_count++) {
jvmtiError l_err;
LOG("INFO: Late suspend thread: %p\n", (void *) testedThread);
l_err = jvmti->SuspendThread(testedThread);
if (l_err != JVMTI_ERROR_NONE) {
LOG("INFO: Late suspend thread err: %d\n", l_err);
// testedThread has exited so we're done with late calls
break;
}
// Only resume a thread if suspend worked. Using LOG()
// here because we want ResumeThread() to be faster.
LOG("INFO: Late resume thread: %p\n", (void *) testedThread);
err = jvmti->ResumeThread(testedThread);
if (err != JVMTI_ERROR_NONE) {
set_agent_fail_status();
return;
}
}
LOG("INFO: made %d late calls to JVM/TI SuspendThread()\n", late_count);
LOG("INFO: N_LATE_CALLS == %d value is %slarge enough to cause a "
"SuspendThread() call after thread exit.\n", N_LATE_CALLS,
(late_count == N_LATE_CALLS) ? "NOT " : "");
/* Second part of original agentProc test block starts here: */
LOG("Wait for thread to finish\n");
// SP4.1-n - notify agent is waiting and wait
// SP6.1-w - wait to end test
if (!agent_wait_for_sync(timeout))
return;
LOG("Delete thread reference\n");
jni->DeleteGlobalRef(testedThread);
}
LOG("Let debugee to finish\n");
// SP7.1-n - notify agent end
if (!agent_resume_sync())
return;
/* Second part of original agentProc test block ends here. */
}
JNIEXPORT jint JNICALL
Agent_OnLoad(JavaVM *jvm, char *options, void *reserved) {
jvmtiEnv *jvmti = NULL;
timeout = 60 * 1000;
jint res = jvm->GetEnv((void **) &jvmti, JVMTI_VERSION_9);
if (res != JNI_OK || jvmti == NULL) {
LOG("Wrong result of a valid call to GetEnv!\n");
return JNI_ERR;
}
/* add specific capabilities for suspending thread */
jvmtiCapabilities suspendCaps;
memset(&suspendCaps, 0, sizeof(suspendCaps));
suspendCaps.can_suspend = 1;
if (jvmti->AddCapabilities(&suspendCaps) != JVMTI_ERROR_NONE) {
return JNI_ERR;
}
if (init_agent_data(jvmti, &agent_data) != JVMTI_ERROR_NONE) {
return JNI_ERR;
}
/* register agent proc and arg */
if (!set_agent_proc(agentProc, NULL)) {
return JNI_ERR;
}
return JNI_OK;
}
}

View File

@ -1,311 +0,0 @@
/*
* Copyright (c) 2003, 2022, 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
* under the terms of the GNU General Public License version 2 only, as
* published by the Free Software Foundation.
*
* This code is distributed in the hope that it will be useful, but WITHOUT
* ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
* FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License
* version 2 for more details (a copy is included in the LICENSE file that
* accompanied this code).
*
* You should have received a copy of the GNU General Public License version
* 2 along with this work; if not, write to the Free Software Foundation,
* Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA.
*
* Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA
* or visit www.oracle.com if you need additional information or have any
* questions.
*/
/*
* @test
*
* @summary converted from VM Testbase nsk/jvmti/SuspendThread/suspendthrd003.
* VM Testbase keywords: [jpda, jvmti, noras]
* VM Testbase readme:
* DESCRIPTION
* Same test as suspendthrd001 with additional calls to
* SuspendThread() and ResumeThread() while threads are exiting.
* Failing criteria for the test are:
* - failures of used JVMTI functions.
* COMMENTS
* Derived from nsk/jvmti/SuspendThread/suspendthrd001.
* Transaction Diagram for this test:
* main thread agent thread TestedThread-N
* =============================================== ================================================= =============================================
* data->thread_state = NEW
* for N = 0; i < N_THREADS; i++ {
* thr = new TestedThread-N
* thr.start() run()
* # SP1-w - wait for TestedThread-N to be ready :
* thr.checkReady() // SP1-n - tell main we are ready
* : threadReady = true
* // SP2.1-w - wait for agent thread while (!shouldFinish) {
* // SP3.1-n - notify to start test // do work
* // SP5.1-w - wait while testing :
* checkStatus(() { :
* enter(data.monitor) :
* if data.thread_state == NEW { :
* run_agent_thread :
* : // this set is in the agent wrapper: :
* : data.thread_state = RUNNABLE :
* : agentProc() { :
* : // SP2.1-n - notify agent is waiting :
* : // SP3.1-w - wait to start test :
* while (data.thread_state == NEW) { waitForSync() { :
* // SP2.2-w - wait for agent thread enter(data.monitor) :
* wait(data.monitor) : <blocked> :
* : : <enter> :
* : data.thread_state = WAITING :
* : // SP2.2-n - notify agent is waiting and wait :
* : notify(data.monitor) :
* : <notified> while (data.thread_state == WAITING) { :
* : // SP3.2-w - wait to start test :
* : wait(data.monitor) :
* : <reentered> : :
* } : :
* // saw data.thread_state == WAITING : :
* } : :
* // we don't enter loop in first call : :
* while (data.thread_state != WAITING : :
* data.thread_state != TERMINATED) { : :
* // SP4.2-w - second wait for agent thread : :
* wait(data.monitor) : :
* } : :
* if (data.thread_state != TERMINATED) { : :
* data.thread_state = SUSPENDED : :
* // SP3.2-n - notify to start test : :
* notify(data.monitor) : :
* } : <notified> :
* while (data.thread_state == SUSPENDED) { : :
* // SP5.2-w - wait while testing : :
* wait(data.monitor) : :
* : : <reentered> :
* : } :
* : // saw data.thread_state == SUSPENDED :
* : exit(data.monitor) :
* : } // end waitForSync() :
* : SuspendThread(TestedThread-N) :
* : // SP5.1-n - notify suspend done : <thread suspended>
* : resumeSync() { :
* : enter(data.monitor) :
* : if (data.thread_state == SUSPENDED) { :
* : data.thread_state = RUNNABLE :
* : // SP5.2-n - notify suspend done :
* : notify(data.monitor) :
* : <notified> } :
* : exit(data.monitor) :
* : <re-entered> } // end resumeSync() :
* } GetThreadState(TestedThread-N) :
* // saw data.thread_state -= RUNNABLE ResumeThread(TestedThread-N) : <thread resumed>
* : for (1..N_LATE_CALLS) { :
* } // end checkStatus() SuspendThread(TestedThread-N) :
* : : : <thread suspended>
* thr.letFinish() ResumeThread(TestedThread-N) :
* shouldFinish = true; : : <thread resumed>
* thr.join() : : <sees shouldFinish == true>
* : : }
* : : : <thread exits>
* : <join() sees thread exit> : <SuspendThread sees thread exit & breaks>
* // SP4.1-w - second wait for agent thread }
* // SP6.1-n - notify to end test :
* // SP7.1 - wait for agent end :
* checkStatus() // SP4.1-n - notify agent is waiting and wait
* enter(data.monitor) // SP6.1-w - wait to end test
* : waitForSync()
* // we don't enter if-stmt in second call enter(data.monitor)
* if data.thread_state == NEW { : <blocked>
* } :
* while (data.thread_state != WAITING :
* data.thread_state != TERMINATED) { :
* // SP4.2-w - second wait for agent thread :
* wait(data.monitor) :
* : : <enter>
* : data.thread_state = WAITING
* : // SP4.2-n - notify agent is waiting and wait
* : notify(data.monitor)
* : <notified> while (data.thread_state == WAITING) {
* : // SP6.2-w - wait to end test
* : wait(data.monitor)
* : <reentered> :
* } :
* // saw thread_state == WAITING :
* if (data.thread_state != TERMINATED) { :
* data.thread_state = SUSPENDED :
* // SP6.2-n - notify to end test :
* notify(data.monitor) :
* } : <notified>
* while (data.thread_state == SUSPENDED) { :
* // SP7.2 - wait for agent end :
* wait(data.monitor) :
* : : <reentered>
* : }
* : // saw data.thread_state == SUSPENDED
* : exit(data.monitor)
* : } // end waitForSync()
* : // SP7.1-n - notify agent end
* : resumeSync() {
* : enter(data.monitor)
* : if (data.thread_state == SUSPENDED) {
* : data.thread_state = RUNNABLE
* : // SP7.2-n - notify agent end
* : notify(data.monitor)
* : <notified> }
* : exit(data.monitor)
* : <re-entered> } // end resumeSync()
* } } // end agentProc
* // saw data.thread_state -= RUNNABLE // this set is in the wrapper:
* } // end checkStatus() data.thread_state = TERMINATED
* resetAgentData() {
* enter(data.monitor)
* while (data.thread_state != TERMINATED) {
* wait(data.monitor, 10);
* }
* data.thread_state = NEW
* exit(data.monitor)
* } // end resetAgentData()
*
* @library /test/lib
* @run main/othervm/native -Xlog:thread+smr=debug -agentlib:suspendthrd03=-waittime=5 suspendthrd03
*/
import jdk.test.lib.jvmti.DebugeeClass;
public class suspendthrd03 extends DebugeeClass {
final static int N_THREADS = 10;
// load native library if required
static {
System.loadLibrary("suspendthrd03");
}
public static void main(String argv[]) {
int result = new suspendthrd03().runIt();
if (result != 0) {
throw new RuntimeException("check failed with result " + result);
}
}
/* =================================================================== */
long timeout = 0;
int status = DebugeeClass.TEST_PASSED;
// tested thread
suspendthrd03Thread thread = null;
// run debuggee
public int runIt() {
timeout = 60 * 1000; // milliseconds
for (int i = 0; i < N_THREADS; i++) {
System.out.println("Starting TestedThread #" + i + ".");
// Original suspendthrd001 test block starts here:
//
// create tested thread
// Note: Cannot use TestedThread-N for thread name since
// the agent has to know the thread's name.
thread = new suspendthrd03Thread("TestedThread");
// run tested thread
System.out.println("Starting tested thread");
try {
thread.start();
// SP1-w - wait for TestedThread-N to be ready
if (!thread.checkReady()) {
throw new RuntimeException("Unable to prepare tested thread: " + thread);
}
// testing sync
System.out.println("Sync: thread started");
// SP2.1-w - wait for agent thread
// SP3.1-n - notify to start test
// SP5.1-w - wait while testing
status = checkStatus(status);
} finally {
// let thread to finish
thread.letFinish();
}
// wait for thread to finish
System.out.println("Finishing tested thread");
try {
thread.join();
} catch (InterruptedException e) {
throw new RuntimeException(e);
}
// testing sync
System.out.println("Sync: thread finished");
// SP4.1-w - second wait for agent thread
// SP6.1-n - notify to end test
// SP7.1 - wait for agent end
status = checkStatus(status);
// Original suspendthrd001 test block ends here.
if (status != DebugeeClass.TEST_PASSED) {
break;
}
resetAgentData(); // reset for another iteration
}
return status;
}
}
/* =================================================================== */
// basic class for tested threads
class suspendthrd03Thread extends Thread {
private volatile boolean threadReady = false;
private volatile boolean shouldFinish = false;
// make thread with specific name
public suspendthrd03Thread(String name) {
super(name);
}
// run thread continuously
public void run() {
// run in a loop
// SP1-n - tell main we are ready
threadReady = true;
int i = 0;
int n = 1000;
while (!shouldFinish) {
if (n <= 0) {
n = 1000;
}
if (i > n) {
i = 0;
n = n - 1;
}
i = i + 1;
}
}
// check if thread is ready
public boolean checkReady() {
try {
while (!threadReady) {
sleep(1000);
}
} catch (InterruptedException e) {
throw new RuntimeException("Interruption while preparing tested thread: \n\t" + e);
}
return threadReady;
}
// let thread to finish
public void letFinish() {
shouldFinish = true;
}
}