# HG changeset patch # User phh # Date 1569450153 25200 # Node ID c29e49148be7ffa797430a5521c4d4df15c3e6ec # Parent ac24594d2c8cd3d324a72b9d51b92ed4df6a15d9 8231209: [REDO] ThreadMXBean::getThreadAllocatedBytes() can be quicker for self thread Summary: Add com.sun.management.getCurrentThreadAllocatedBytes, implement getThreadAllocatedBytes(long) independent of getThreadAllocatedBytes(long[]) Reviewed-by: mchung, dholmes, sspitsyn diff -r ac24594d2c8c -r c29e49148be7 src/hotspot/share/include/jmm.h --- a/src/hotspot/share/include/jmm.h Wed Sep 25 14:33:44 2019 -0700 +++ b/src/hotspot/share/include/jmm.h Wed Sep 25 15:22:33 2019 -0700 @@ -1,5 +1,5 @@ /* - * Copyright (c) 2003, 2017, Oracle and/or its affiliates. All rights reserved. + * Copyright (c) 2003, 2019, 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 @@ -50,8 +50,9 @@ JMM_VERSION_1_2 = 0x20010200, // JDK 7 JMM_VERSION_1_2_1 = 0x20010201, // JDK 7 GA JMM_VERSION_1_2_2 = 0x20010202, - JMM_VERSION_2 = 0x20020000, // JDK 10 - JMM_VERSION = 0x20020000 + JMM_VERSION_2 = 0x20020000, // JDK 10 + JMM_VERSION_3 = 0x20030000, // JDK 14 + JMM_VERSION = JMM_VERSION_3 }; typedef struct { @@ -239,6 +240,9 @@ jobject (JNICALL *GetMemoryPoolUsage) (JNIEnv* env, jobject pool); jobject (JNICALL *GetPeakMemoryPoolUsage) (JNIEnv* env, jobject pool); + jlong (JNICALL *GetOneThreadAllocatedMemory) + (JNIEnv *env, + jlong thread_id); void (JNICALL *GetThreadAllocatedMemory) (JNIEnv *env, jlongArray ids, diff -r ac24594d2c8c -r c29e49148be7 src/hotspot/share/services/management.cpp --- a/src/hotspot/share/services/management.cpp Wed Sep 25 14:33:44 2019 -0700 +++ b/src/hotspot/share/services/management.cpp Wed Sep 25 15:22:33 2019 -0700 @@ -2068,6 +2068,31 @@ } #endif // INCLUDE_MANAGEMENT +// Gets the amount of memory allocated on the Java heap for a single thread. +// Returns -1 if the thread does not exist or has terminated. +JVM_ENTRY(jlong, jmm_GetOneThreadAllocatedMemory(JNIEnv *env, jlong thread_id)) + if (thread_id < 0) { + THROW_MSG_(vmSymbols::java_lang_IllegalArgumentException(), + "Invalid thread ID", -1); + } + + if (thread_id == 0) { + // current thread + if (THREAD->is_Java_thread()) { + return ((JavaThread*)THREAD)->cooked_allocated_bytes(); + } + return -1; + } + + ThreadsListHandle tlh; + JavaThread* java_thread = tlh.list()->find_JavaThread_from_java_tid(thread_id); + + if (java_thread != NULL) { + return java_thread->cooked_allocated_bytes(); + } + return -1; +JVM_END + // Gets an array containing the amount of memory allocated on the Java // heap for a set of threads (in bytes). Each element of the array is // the amount of memory allocated for the thread ID specified in the @@ -2192,6 +2217,7 @@ jmm_GetMemoryManagers, jmm_GetMemoryPoolUsage, jmm_GetPeakMemoryPoolUsage, + jmm_GetOneThreadAllocatedMemory, jmm_GetThreadAllocatedMemory, jmm_GetMemoryUsage, jmm_GetLongAttribute, diff -r ac24594d2c8c -r c29e49148be7 src/java.management/share/classes/java/lang/management/ThreadMXBean.java --- a/src/java.management/share/classes/java/lang/management/ThreadMXBean.java Wed Sep 25 14:33:44 2019 -0700 +++ b/src/java.management/share/classes/java/lang/management/ThreadMXBean.java Wed Sep 25 15:22:33 2019 -0700 @@ -160,7 +160,7 @@ * * @return an array of {@code long}, each is a thread ID. * - * @throws java.lang.SecurityException if a security manager + * @throws SecurityException if a security manager * exists and the caller does not have * ManagementPermission("monitor"). */ @@ -199,7 +199,7 @@ * it does not exist. * * @throws IllegalArgumentException if {@code id <= 0}. - * @throws java.lang.SecurityException if a security manager + * @throws SecurityException if a security manager * exists and the caller does not have * ManagementPermission("monitor"). */ @@ -237,7 +237,7 @@ * * @throws IllegalArgumentException if any element in the input array * {@code ids} is {@code <= 0}. - * @throws java.lang.SecurityException if a security manager + * @throws SecurityException if a security manager * exists and the caller does not have * ManagementPermission("monitor"). */ @@ -284,7 +284,7 @@ * * @throws IllegalArgumentException if {@code id <= 0}. * @throws IllegalArgumentException if {@code maxDepth is negative}. - * @throws java.lang.SecurityException if a security manager + * @throws SecurityException if a security manager * exists and the caller does not have * ManagementPermission("monitor"). * @@ -337,7 +337,7 @@ * @throws IllegalArgumentException if {@code maxDepth is negative}. * @throws IllegalArgumentException if any element in the input array * {@code ids} is {@code <= 0}. - * @throws java.lang.SecurityException if a security manager + * @throws SecurityException if a security manager * exists and the caller does not have * ManagementPermission("monitor"). * @@ -360,7 +360,7 @@ * @return {@code true} if thread contention monitoring is enabled; * {@code false} otherwise. * - * @throws java.lang.UnsupportedOperationException if the Java virtual + * @throws UnsupportedOperationException if the Java virtual * machine does not support thread contention monitoring. * * @see #isThreadContentionMonitoringSupported @@ -374,10 +374,10 @@ * @param enable {@code true} to enable; * {@code false} to disable. * - * @throws java.lang.UnsupportedOperationException if the Java + * @throws UnsupportedOperationException if the Java * virtual machine does not support thread contention monitoring. * - * @throws java.lang.SecurityException if a security manager + * @throws SecurityException if a security manager * exists and the caller does not have * ManagementPermission("control"). * @@ -394,7 +394,7 @@ * the current thread has executed in user mode or system mode. * *

- * This is a convenient method for local management use and is + * This is a convenience method for local management use and is * equivalent to calling: *

      *   {@link #getThreadCpuTime getThreadCpuTime}(Thread.currentThread().getId());
@@ -403,7 +403,7 @@
      * @return the total CPU time for the current thread if CPU time
      * measurement is enabled; {@code -1} otherwise.
      *
-     * @throws java.lang.UnsupportedOperationException if the Java
+     * @throws UnsupportedOperationException if the Java
      * virtual machine does not support CPU time measurement for
      * the current thread.
      *
@@ -421,7 +421,7 @@
      * not necessarily nanoseconds accuracy.
      *
      * 

- * This is a convenient method for local management use and is + * This is a convenience method for local management use and is * equivalent to calling: *

      *   {@link #getThreadUserTime getThreadUserTime}(Thread.currentThread().getId());
@@ -430,7 +430,7 @@
      * @return the user-level CPU time for the current thread if CPU time
      * measurement is enabled; {@code -1} otherwise.
      *
-     * @throws java.lang.UnsupportedOperationException if the Java
+     * @throws UnsupportedOperationException if the Java
      * virtual machine does not support CPU time measurement for
      * the current thread.
      *
@@ -467,7 +467,7 @@
      * {@code -1} otherwise.
      *
      * @throws IllegalArgumentException if {@code id <= 0}.
-     * @throws java.lang.UnsupportedOperationException if the Java
+     * @throws UnsupportedOperationException if the Java
      * virtual machine does not support CPU time measurement for
      * other threads.
      *
@@ -502,7 +502,7 @@
      * {@code -1} otherwise.
      *
      * @throws IllegalArgumentException if {@code id <= 0}.
-     * @throws java.lang.UnsupportedOperationException if the Java
+     * @throws UnsupportedOperationException if the Java
      * virtual machine does not support CPU time measurement for
      * other threads.
      *
@@ -548,7 +548,7 @@
      * @return {@code true} if thread CPU time measurement is enabled;
      *         {@code false} otherwise.
      *
-     * @throws java.lang.UnsupportedOperationException if the Java virtual
+     * @throws UnsupportedOperationException if the Java virtual
      * machine does not support CPU time measurement for other threads
      * nor for the current thread.
      *
@@ -564,11 +564,11 @@
      * @param enable {@code true} to enable;
      *               {@code false} to disable.
      *
-     * @throws java.lang.UnsupportedOperationException if the Java
+     * @throws UnsupportedOperationException if the Java
      * virtual machine does not support CPU time measurement for
      * any threads nor for the current thread.
      *
-     * @throws java.lang.SecurityException if a security manager
+     * @throws SecurityException if a security manager
      *         exists and the caller does not have
      *         ManagementPermission("control").
      *
@@ -604,7 +604,7 @@
      * @return an array of IDs of the threads that are monitor
      * deadlocked, if any; {@code null} otherwise.
      *
-     * @throws java.lang.SecurityException if a security manager
+     * @throws SecurityException if a security manager
      *         exists and the caller does not have
      *         ManagementPermission("monitor").
      *
@@ -616,7 +616,7 @@
      * Resets the peak thread count to the current number of
      * live threads.
      *
-     * @throws java.lang.SecurityException if a security manager
+     * @throws SecurityException if a security manager
      *         exists and the caller does not have
      *         ManagementPermission("control").
      *
@@ -642,10 +642,10 @@
      * deadlocked waiting for object monitors or ownable synchronizers, if any;
      * {@code null} otherwise.
      *
-     * @throws java.lang.SecurityException if a security manager
+     * @throws SecurityException if a security manager
      *         exists and the caller does not have
      *         ManagementPermission("monitor").
-     * @throws java.lang.UnsupportedOperationException if the Java virtual
+     * @throws UnsupportedOperationException if the Java virtual
      * machine does not support monitoring of ownable synchronizer usage.
      *
      * @see #isSynchronizerUsageSupported
@@ -704,10 +704,10 @@
      * information about a thread whose ID is in the corresponding
      * element of the input array of IDs.
      *
-     * @throws java.lang.SecurityException if a security manager
+     * @throws SecurityException if a security manager
      *         exists and the caller does not have
      *         ManagementPermission("monitor").
-     * @throws java.lang.UnsupportedOperationException
+     * @throws UnsupportedOperationException
      *         
    *
  • if {@code lockedMonitors} is {@code true} but * the Java virtual machine does not support monitoring @@ -794,10 +794,10 @@ * element of the input array of IDs. * * @throws IllegalArgumentException if {@code maxDepth} is negative. - * @throws java.lang.SecurityException if a security manager + * @throws SecurityException if a security manager * exists and the caller does not have * ManagementPermission("monitor"). - * @throws java.lang.UnsupportedOperationException + * @throws UnsupportedOperationException *
      *
    • if {@code lockedMonitors} is {@code true} but * the Java virtual machine does not support monitoring @@ -835,10 +835,10 @@ * * @return an array of {@link ThreadInfo} for all live threads. * - * @throws java.lang.SecurityException if a security manager + * @throws SecurityException if a security manager * exists and the caller does not have * ManagementPermission("monitor"). - * @throws java.lang.UnsupportedOperationException + * @throws UnsupportedOperationException *
        *
      • if {@code lockedMonitors} is {@code true} but * the Java virtual machine does not support monitoring @@ -884,10 +884,10 @@ * @return an array of {@link ThreadInfo} for all live threads. * * @throws IllegalArgumentException if {@code maxDepth} is negative. - * @throws java.lang.SecurityException if a security manager + * @throws SecurityException if a security manager * exists and the caller does not have * ManagementPermission("monitor"). - * @throws java.lang.UnsupportedOperationException + * @throws UnsupportedOperationException *
          *
        • if {@code lockedMonitors} is {@code true} but * the Java virtual machine does not support monitoring diff -r ac24594d2c8c -r c29e49148be7 src/java.management/share/classes/sun/management/ThreadImpl.java --- a/src/java.management/share/classes/sun/management/ThreadImpl.java Wed Sep 25 14:33:44 2019 -0700 +++ b/src/java.management/share/classes/sun/management/ThreadImpl.java Wed Sep 25 15:22:33 2019 -0700 @@ -29,6 +29,7 @@ import java.lang.management.ThreadInfo; import java.lang.management.ThreadMXBean; import javax.management.ObjectName; +import java.util.Objects; /** * Implementation for java.lang.management.ThreadMXBean as well as providing the @@ -112,11 +113,15 @@ return cpuTimeEnabled; } - protected boolean isThreadAllocatedMemoryEnabled() { + private void ensureThreadAllocatedMemorySupported() { if (!isThreadAllocatedMemorySupported()) { throw new UnsupportedOperationException( - "Thread allocated memory measurement is not supported"); + "Thread allocated memory measurement is not supported."); } + } + + protected boolean isThreadAllocatedMemoryEnabled() { + ensureThreadAllocatedMemorySupported(); return allocatedMemoryEnabled; } @@ -155,16 +160,18 @@ return getThreadInfo(ids, 0); } + private void verifyThreadId(long id) { + if (id <= 0) { + throw new IllegalArgumentException( + "Invalid thread ID parameter: " + id); + } + } + private void verifyThreadIds(long[] ids) { - if (ids == null) { - throw new NullPointerException("Null ids parameter."); - } + Objects.requireNonNull(ids); for (int i = 0; i < ids.length; i++) { - if (ids[i] <= 0) { - throw new IllegalArgumentException( - "Invalid thread ID parameter: " + ids[i]); - } + verifyThreadId(ids[i]); } } @@ -342,26 +349,41 @@ } } + protected long getCurrentThreadAllocatedBytes() { + if (isThreadAllocatedMemoryEnabled()) { + return getThreadAllocatedMemory0(0); + } + return -1; + } + + private boolean verifyThreadAllocatedMemory(long id) { + verifyThreadId(id); + return isThreadAllocatedMemoryEnabled(); + } + protected long getThreadAllocatedBytes(long id) { - long[] ids = new long[1]; - ids[0] = id; - final long[] sizes = getThreadAllocatedBytes(ids); - return sizes[0]; + boolean verified = verifyThreadAllocatedMemory(id); + + if (verified) { + return getThreadAllocatedMemory0( + Thread.currentThread().getId() == id ? 0 : id); + } + return -1; } private boolean verifyThreadAllocatedMemory(long[] ids) { verifyThreadIds(ids); - - // check if Thread allocated memory measurement is supported. - if (!isThreadAllocatedMemorySupported()) { - throw new UnsupportedOperationException( - "Thread allocated memory measurement is not supported."); - } - return isThreadAllocatedMemoryEnabled(); } protected long[] getThreadAllocatedBytes(long[] ids) { + Objects.requireNonNull(ids); + + if (ids.length == 1) { + long size = getThreadAllocatedBytes(ids[0]); + return new long[] { size }; + } + boolean verified = verifyThreadAllocatedMemory(ids); long[] sizes = new long[ids.length]; @@ -374,10 +396,7 @@ } protected void setThreadAllocatedMemoryEnabled(boolean enable) { - if (!isThreadAllocatedMemorySupported()) { - throw new UnsupportedOperationException( - "Thread allocated memory measurement is not supported."); - } + ensureThreadAllocatedMemorySupported(); Util.checkControlAccess(); synchronized (this) { @@ -511,6 +530,7 @@ private static native void getThreadTotalCpuTime1(long[] ids, long[] result); private static native long getThreadUserCpuTime0(long id); private static native void getThreadUserCpuTime1(long[] ids, long[] result); + private static native long getThreadAllocatedMemory0(long id); private static native void getThreadAllocatedMemory1(long[] ids, long[] result); private static native void setThreadCpuTimeEnabled0(boolean enable); private static native void setThreadAllocatedMemoryEnabled0(boolean enable); diff -r ac24594d2c8c -r c29e49148be7 src/java.management/share/native/libmanagement/ThreadImpl.c --- a/src/java.management/share/native/libmanagement/ThreadImpl.c Wed Sep 25 14:33:44 2019 -0700 +++ b/src/java.management/share/native/libmanagement/ThreadImpl.c Wed Sep 25 15:22:33 2019 -0700 @@ -1,5 +1,5 @@ /* - * Copyright (c) 2003, 2017, Oracle and/or its affiliates. All rights reserved. + * Copyright (c) 2003, 2019, 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 @@ -94,6 +94,13 @@ JNI_FALSE /* user */); } +JNIEXPORT jlong JNICALL +Java_sun_management_ThreadImpl_getThreadAllocatedMemory0 + (JNIEnv *env, jclass cls, jlong tid) +{ + return jmm_interface->GetOneThreadAllocatedMemory(env, tid); +} + JNIEXPORT void JNICALL Java_sun_management_ThreadImpl_getThreadAllocatedMemory1 (JNIEnv *env, jclass cls, jlongArray ids, jlongArray sizeArray) diff -r ac24594d2c8c -r c29e49148be7 src/jdk.management/share/classes/com/sun/management/ThreadMXBean.java --- a/src/jdk.management/share/classes/com/sun/management/ThreadMXBean.java Wed Sep 25 14:33:44 2019 -0700 +++ b/src/jdk.management/share/classes/com/sun/management/ThreadMXBean.java Wed Sep 25 15:22:33 2019 -0700 @@ -1,5 +1,5 @@ /* - * Copyright (c) 2011, 2013, Oracle and/or its affiliates. All rights reserved. + * Copyright (c) 2011, 2019, 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 @@ -61,7 +61,7 @@ * @throws NullPointerException if {@code ids} is {@code null} * @throws IllegalArgumentException if any element in the input array * {@code ids} is {@code <=} {@code 0}. - * @throws java.lang.UnsupportedOperationException if the Java + * @throws UnsupportedOperationException if the Java * virtual machine implementation does not support CPU time * measurement. * @@ -95,7 +95,7 @@ * @throws NullPointerException if {@code ids} is {@code null} * @throws IllegalArgumentException if any element in the input array * {@code ids} is {@code <=} {@code 0}. - * @throws java.lang.UnsupportedOperationException if the Java + * @throws UnsupportedOperationException if the Java * virtual machine implementation does not support CPU time * measurement. * @@ -109,13 +109,50 @@ /** * Returns an approximation of the total amount of memory, in bytes, - * allocated in heap memory for the thread of the specified ID. + * allocated in heap memory for the current thread. + * The returned value is an approximation because some Java virtual machine + * implementations may use object allocation mechanisms that result in a + * delay between the time an object is allocated and the time its size is + * recorded. + * + *

          + * This is a convenience method for local management use and is + * equivalent to calling: + *

          +     *   {@link #getThreadAllocatedBytes getThreadAllocatedBytes}(Thread.currentThread().getId());
          +     * 
          + * + * @implSpec The default implementation throws + * {@code UnsupportedOperationException}. + * + * @return an approximation of the total memory allocated, in bytes, in + * heap memory for the current thread + * if thread memory allocation measurement is enabled; + * {@code -1} otherwise. + * + * @throws UnsupportedOperationException if the Java virtual + * machine implementation does not support thread memory allocation + * measurement. + * + * @see #isThreadAllocatedMemorySupported + * @see #isThreadAllocatedMemoryEnabled + * @see #setThreadAllocatedMemoryEnabled + * + * @since 14 + */ + public default long getCurrentThreadAllocatedBytes() { + throw new UnsupportedOperationException(); + } + + /** + * Returns an approximation of the total amount of memory, in bytes, + * allocated in heap memory for the thread with the specified ID. * The returned value is an approximation because some Java virtual machine * implementations may use object allocation mechanisms that result in a * delay between the time an object is allocated and the time its size is * recorded. *

          - * If the thread of the specified ID is not alive or does not exist, + * If the thread with the specified ID is not alive or does not exist, * this method returns {@code -1}. If thread memory allocation measurement * is disabled, this method returns {@code -1}. * A thread is alive if it has been started and has not yet died. @@ -127,13 +164,13 @@ * * @param id the thread ID of a thread * @return an approximation of the total memory allocated, in bytes, in - * heap memory for a thread of the specified ID - * if the thread of the specified ID exists, the thread is alive, + * heap memory for the thread with the specified ID + * if the thread with the specified ID exists, the thread is alive, * and thread memory allocation measurement is enabled; * {@code -1} otherwise. * * @throws IllegalArgumentException if {@code id} {@code <=} {@code 0}. - * @throws java.lang.UnsupportedOperationException if the Java virtual + * @throws UnsupportedOperationException if the Java virtual * machine implementation does not support thread memory allocation * measurement. * @@ -165,7 +202,7 @@ * @throws NullPointerException if {@code ids} is {@code null} * @throws IllegalArgumentException if any element in the input array * {@code ids} is {@code <=} {@code 0}. - * @throws java.lang.UnsupportedOperationException if the Java virtual + * @throws UnsupportedOperationException if the Java virtual * machine implementation does not support thread memory allocation * measurement. * @@ -194,7 +231,7 @@ * @return {@code true} if thread memory allocation measurement is enabled; * {@code false} otherwise. * - * @throws java.lang.UnsupportedOperationException if the Java virtual + * @throws UnsupportedOperationException if the Java virtual * machine does not support thread memory allocation measurement. * * @see #isThreadAllocatedMemorySupported @@ -208,10 +245,10 @@ * @param enable {@code true} to enable; * {@code false} to disable. * - * @throws java.lang.UnsupportedOperationException if the Java virtual + * @throws UnsupportedOperationException if the Java virtual * machine does not support thread memory allocation measurement. * - * @throws java.lang.SecurityException if a security manager + * @throws SecurityException if a security manager * exists and the caller does not have * ManagementPermission("control"). * diff -r ac24594d2c8c -r c29e49148be7 src/jdk.management/share/classes/com/sun/management/internal/HotSpotThreadImpl.java --- a/src/jdk.management/share/classes/com/sun/management/internal/HotSpotThreadImpl.java Wed Sep 25 14:33:44 2019 -0700 +++ b/src/jdk.management/share/classes/com/sun/management/internal/HotSpotThreadImpl.java Wed Sep 25 15:22:33 2019 -0700 @@ -58,6 +58,11 @@ } @Override + public long getCurrentThreadAllocatedBytes() { + return super.getCurrentThreadAllocatedBytes(); + } + + @Override public long getThreadAllocatedBytes(long id) { return super.getThreadAllocatedBytes(id); } diff -r ac24594d2c8c -r c29e49148be7 test/hotspot/jtreg/vmTestbase/nsk/monitoring/ThreadMXBean/GetThreadAllocatedBytes/BaseBehaviorTest.README --- a/test/hotspot/jtreg/vmTestbase/nsk/monitoring/ThreadMXBean/GetThreadAllocatedBytes/BaseBehaviorTest.README Wed Sep 25 14:33:44 2019 -0700 +++ b/test/hotspot/jtreg/vmTestbase/nsk/monitoring/ThreadMXBean/GetThreadAllocatedBytes/BaseBehaviorTest.README Wed Sep 25 15:22:33 2019 -0700 @@ -1,4 +1,4 @@ -Copyright (c) 2011, 2018, Oracle and/or its affiliates. All rights reserved. +Copyright (c) 2011, 2019, 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 @@ -21,7 +21,8 @@ DESCRIPTION - Tests getThreadAllocatedBytes(long id) and getThreadAllocatedBytes(long[] ids), + Tests getCurrentThreadAllocatedBytes(), getThreadAllocatedBytes(long id), + and getThreadAllocatedBytes(long[] ids), functions of com.sun.management.ThreadMXBean All methods should diff -r ac24594d2c8c -r c29e49148be7 test/hotspot/jtreg/vmTestbase/nsk/monitoring/ThreadMXBean/GetThreadAllocatedBytes/BaseBehaviorTest.java --- a/test/hotspot/jtreg/vmTestbase/nsk/monitoring/ThreadMXBean/GetThreadAllocatedBytes/BaseBehaviorTest.java Wed Sep 25 14:33:44 2019 -0700 +++ b/test/hotspot/jtreg/vmTestbase/nsk/monitoring/ThreadMXBean/GetThreadAllocatedBytes/BaseBehaviorTest.java Wed Sep 25 15:22:33 2019 -0700 @@ -1,5 +1,5 @@ /* - * Copyright (c) 2011, 2018, Oracle and/or its affiliates. All rights reserved. + * Copyright (c) 2011, 2019, 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 @@ -27,7 +27,8 @@ import nsk.monitoring.ThreadMXBean.*; /** - * Tests getThreadAllocatedBytes(long id) and getThreadAllocatedBytes(long[] ids), + * Tests getCurrentThreadAllocatedBytes(), getThreadAllocatedBytes(long id). + * and getThreadAllocatedBytes(long[] ids), * functions of com.sun.management.ThreadMXBean *

          * All methods should @@ -49,11 +50,31 @@ public void run() { if (threadMXBean == null) return; + + // Expect -1 if thread allocated memory is disabled + threadMXBean.setThreadAllocatedMemoryEnabled(false); + long result = threadMXBean.getCurrentThreadAllocatedBytes(); + if (result != -1) + throw new TestFailure("Failure! getCurrentThreadAllocatedBytes() should " + + "return -1 if ThreadAllocatedMemoryEnabled is set to false. " + + "Received : " + result); + threadMXBean.setThreadAllocatedMemoryEnabled(true); + // Expect >= 0 value for current thread + result = threadMXBean.getCurrentThreadAllocatedBytes(); + if (result < 0) + throw new TestFailure("Failure! getCurrentThreadAllocatedBytes() should " + + "return >= 0 value for current thread. Received : " + result); + // Expect >= 0 value for current thread from getThreadAllocatedBytes(id) + result = threadMXBean.getThreadAllocatedBytes(Thread.currentThread().getId()); + if (result < 0) + throw new TestFailure("Failure! getThreadAllocatedBytes(id) should " + + "return >= 0 value for current thread. Received : " + result); + MXBeanTestThread thread = new MXBeanTestThread(); long id = thread.getId(); long[] idArr = new long[] { id }; - long result; long[] resultArr; + // Expect -1 for not started threads result = threadMXBean.getThreadAllocatedBytes(id); if (result != -1) @@ -80,7 +101,7 @@ + "Recieved : " + resultArr[0]); threadMXBean.setThreadAllocatedMemoryEnabled(true); - // Expect > 0 value for running threads + // Expect >= 0 value for running threads result = threadMXBean.getThreadAllocatedBytes(id); if (result < 0) throw new TestFailure("Failure! getThreadAllocatedBytes(long id) should " diff -r ac24594d2c8c -r c29e49148be7 test/hotspot/jtreg/vmTestbase/nsk/monitoring/share/server/ServerThreadMXBeanNew.java --- a/test/hotspot/jtreg/vmTestbase/nsk/monitoring/share/server/ServerThreadMXBeanNew.java Wed Sep 25 14:33:44 2019 -0700 +++ b/test/hotspot/jtreg/vmTestbase/nsk/monitoring/share/server/ServerThreadMXBeanNew.java Wed Sep 25 15:22:33 2019 -0700 @@ -1,5 +1,5 @@ /* - * Copyright (c) 2011, 2018, Oracle and/or its affiliates. All rights reserved. + * Copyright (c) 2011, 2019, 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 @@ -59,6 +59,10 @@ new String[] { long.class.getName() }); } + public long getCurrentThreadAllocatedBytes() { + return getLongAttribute("CurrentThreadAllocatedBytes"); + } + public void setThreadAllocatedMemoryEnabled(boolean enabled) { setBooleanAttribute("ThreadAllocatedMemoryEnabled", enabled); } diff -r ac24594d2c8c -r c29e49148be7 test/jdk/com/sun/management/ThreadMXBean/ThreadAllocatedMemory.java --- a/test/jdk/com/sun/management/ThreadMXBean/ThreadAllocatedMemory.java Wed Sep 25 14:33:44 2019 -0700 +++ b/test/jdk/com/sun/management/ThreadMXBean/ThreadAllocatedMemory.java Wed Sep 25 15:22:33 2019 -0700 @@ -1,5 +1,5 @@ /* - * Copyright (c) 2011, 2015, Oracle and/or its affiliates. All rights reserved. + * Copyright (c) 2011, 2019, 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 @@ -23,7 +23,7 @@ /* * @test - * @bug 6173675 + * @bug 6173675 8231209 * @summary Basic test of ThreadMXBean.getThreadAllocatedBytes * @author Paul Hohensee */ @@ -33,9 +33,8 @@ public class ThreadAllocatedMemory { private static com.sun.management.ThreadMXBean mbean = (com.sun.management.ThreadMXBean)ManagementFactory.getThreadMXBean(); - private static boolean testFailed = false; - private static boolean done = false; - private static boolean done1 = false; + private static volatile boolean done = false; + private static volatile boolean done1 = false; private static Object obj = new Object(); private static final int NUM_THREADS = 10; private static Thread[] threads = new Thread[NUM_THREADS]; @@ -44,6 +43,22 @@ public static void main(String[] argv) throws Exception { + testSupportEnableDisable(); + + // Test current thread two ways + testGetCurrentThreadAllocatedBytes(); + testCurrentThreadGetThreadAllocatedBytes(); + + // Test a single thread that is not this one + testGetThreadAllocatedBytes(); + + // Test many threads that are not this one + testGetThreadsAllocatedBytes(); + + System.out.println("Test passed"); + } + + private static void testSupportEnableDisable() { if (!mbean.isThreadAllocatedMemorySupported()) { return; } @@ -58,10 +73,7 @@ "ThreadAllocatedMemory is expected to be disabled"); } - Thread curThread = Thread.currentThread(); - long id = curThread.getId(); - - long s = mbean.getThreadAllocatedBytes(id); + long s = mbean.getCurrentThreadAllocatedBytes(); if (s != -1) { throw new RuntimeException( "Invalid ThreadAllocatedBytes returned = " + @@ -77,63 +89,106 @@ throw new RuntimeException( "ThreadAllocatedMemory is expected to be enabled"); } + } + + private static void testGetCurrentThreadAllocatedBytes() { + long size = mbean.getCurrentThreadAllocatedBytes(); + ensureValidSize(size); + + // do some more allocation + doit(); + + checkResult(Thread.currentThread(), size, + mbean.getCurrentThreadAllocatedBytes()); + } + + private static void testCurrentThreadGetThreadAllocatedBytes() { + Thread curThread = Thread.currentThread(); + long id = curThread.getId(); long size = mbean.getThreadAllocatedBytes(id); - // implementation could have started measurement when - // measurement was enabled, in which case size can be 0 - if (size < 0) { - throw new RuntimeException( - "Invalid allocated bytes returned = " + size); - } + ensureValidSize(size); + // do some more allocation doit(); - // Expected to be size1 >= size - long size1 = mbean.getThreadAllocatedBytes(id); - if (size1 < size) { - throw new RuntimeException("Allocated bytes " + size1 + - " expected >= " + size); + checkResult(curThread, size, mbean.getThreadAllocatedBytes(id)); + } + + private static void testGetThreadAllocatedBytes() + throws Exception { + + // start a thread + done = false; done1 = false; + Thread curThread = new MyThread("MyThread"); + curThread.start(); + long id = curThread.getId(); + + // wait for thread to block after doing some allocation + waitUntilThreadBlocked(curThread); + + long size = mbean.getThreadAllocatedBytes(id); + ensureValidSize(size); + + // let thread go to do some more allocation + synchronized (obj) { + done = true; + obj.notifyAll(); } - System.out.println(curThread.getName() + - " Current thread allocated bytes = " + size + - " allocated bytes = " + size1); + + // wait for thread to get going again. we don't care if we + // catch it in mid-execution or if it hasn't + // restarted after we're done sleeping. + goSleep(400); + + checkResult(curThread, size, mbean.getThreadAllocatedBytes(id)); + + // let thread exit + synchronized (obj) { + done1 = true; + obj.notifyAll(); + } + try { + curThread.join(); + } catch (InterruptedException e) { + System.out.println("Unexpected exception is thrown."); + e.printStackTrace(System.out); + } + } - // start threads, wait for them to block + private static void testGetThreadsAllocatedBytes() + throws Exception { + + // start threads + done = false; done1 = false; for (int i = 0; i < NUM_THREADS; i++) { threads[i] = new MyThread("MyThread-" + i); threads[i].start(); } - // threads block after doing some allocation - waitUntilThreadBlocked(); + // wait for threads to block after doing some allocation + waitUntilThreadsBlocked(); for (int i = 0; i < NUM_THREADS; i++) { sizes[i] = mbean.getThreadAllocatedBytes(threads[i].getId()); + ensureValidSize(sizes[i]); } - // let threads go and do some more allocation + // let threads go to do some more allocation synchronized (obj) { done = true; obj.notifyAll(); } - // wait for threads to get going again. we don't care if we + // wait for threads to get going again. we don't care if we // catch them in mid-execution or if some of them haven't // restarted after we're done sleeping. goSleep(400); for (int i = 0; i < NUM_THREADS; i++) { - long newSize = mbean.getThreadAllocatedBytes(threads[i].getId()); - if (sizes[i] > newSize) { - throw new RuntimeException("TEST FAILED: " + - threads[i].getName() + - " previous allocated bytes = " + sizes[i] + - " > current allocated bytes = " + newSize); - } - System.out.println(threads[i].getName() + - " Previous allocated bytes = " + sizes[i] + - " Current allocated bytes = " + newSize); + checkResult(threads[i], sizes[i], + mbean.getThreadAllocatedBytes(threads[i].getId())); } // let threads exit @@ -148,17 +203,30 @@ } catch (InterruptedException e) { System.out.println("Unexpected exception is thrown."); e.printStackTrace(System.out); - testFailed = true; break; } } - if (testFailed) { - throw new RuntimeException("TEST FAILED"); + } + + private static void ensureValidSize(long size) { + // implementation could have started measurement when + // measurement was enabled, in which case size can be 0 + if (size < 0) { + throw new RuntimeException( + "Invalid allocated bytes returned = " + size); } - - System.out.println("Test passed"); } + private static void checkResult(Thread curThread, + long prev_size, long curr_size) { + if (curr_size < prev_size) { + throw new RuntimeException("Allocated bytes " + curr_size + + " expected >= " + prev_size); + } + System.out.println(curThread.getName() + + " Previous allocated bytes = " + prev_size + + " Current allocated bytes = " + curr_size); + } private static void goSleep(long ms) throws Exception { try { @@ -169,7 +237,18 @@ } } - private static void waitUntilThreadBlocked() + private static void waitUntilThreadBlocked(Thread thread) + throws Exception { + while (true) { + goSleep(100); + ThreadInfo info = mbean.getThreadInfo(thread.getId()); + if (info.getThreadState() == Thread.State.WAITING) { + break; + } + } + } + + private static void waitUntilThreadsBlocked() throws Exception { int count = 0; while (count != NUM_THREADS) { @@ -210,7 +289,6 @@ } catch (InterruptedException e) { System.out.println("Unexpected exception is thrown."); e.printStackTrace(System.out); - testFailed = true; break; } } @@ -225,7 +303,7 @@ " ThreadAllocatedBytes = " + size2); if (size1 > size2) { - throw new RuntimeException("TEST FAILED: " + getName() + + throw new RuntimeException(getName() + " ThreadAllocatedBytes = " + size1 + " > ThreadAllocatedBytes = " + size2); } @@ -237,7 +315,6 @@ } catch (InterruptedException e) { System.out.println("Unexpected exception is thrown."); e.printStackTrace(System.out); - testFailed = true; break; } }