changeset 175:14c704c52409

Bug 3167: Fix incorrect message caused by class unloading in CMS. Reviewed-by: ykubota https://github.com/HeapStats/heapstats/pull/32
author Yasumasa Suenaga <yasuenag@gmail.com>
date Wed, 14 Sep 2016 23:14:54 +0900
parents 47922c6927e1
children cd34afdf3d2c
files ChangeLog agent/src/heapstats-engines/classContainer.cpp agent/src/heapstats-engines/overrideFunc.S
diffstat 3 files changed, 6 insertions(+), 2 deletions(-) [+]
line wrap: on
line diff
--- a/ChangeLog	Sun Jul 31 22:58:05 2016 +0900
+++ b/ChangeLog	Wed Sep 14 23:14:54 2016 +0900
@@ -1,3 +1,7 @@
+2016-09-14  Yasumasa Suenaga <yasuenag@gmail.com>
+
+	* Bug 3167: Fix incorrect message caused by class unloading in CMS.
+
 2016-07-31  Yasumasa Suenaga <yasuenag@gmail.com>
 
 	* Bug 3104: SEGV in TSnapShotContainer::mergeChildren with Oracle JDK8u92
--- a/agent/src/heapstats-engines/classContainer.cpp	Sun Jul 31 22:58:05 2016 +0900
+++ b/agent/src/heapstats-engines/classContainer.cpp	Wed Sep 14 23:14:54 2016 +0900
@@ -194,7 +194,7 @@
   /* If failure getting class name. */
   if (unlikely(cur->className == NULL)) {
     /* Adding empty to list is deny. */
-    logger->printWarnMsg("Couldn't allocate counter memory!");
+    logger->printWarnMsg("Couldn't get class name!");
     free(cur);
     return NULL;
   }
--- a/agent/src/heapstats-engines/overrideFunc.S	Sun Jul 31 22:58:05 2016 +0900
+++ b/agent/src/heapstats-engines/overrideFunc.S	Wed Sep 14 23:14:54 2016 +0900
@@ -99,7 +99,7 @@
  */
 
 /* SweepClosure::do_blk_careful(HeapWord*) */
-OVERRIDE_DEFINE_WITHOUT_PERMCHECK(cms_sweep, 0, 2)
+OVERRIDE_DEFINE(cms_sweep, 0, 2)
 
 /* instanceKlass::oop_oop_iterate_nv(oopDesc*, Par_MarkRefsIntoAndScanClosure*) */
 OVERRIDE_DEFINE(cms_new, 0, 2)