# HG changeset patch # User Yasumasa Suenaga # Date 1522634216 -32400 # Node ID a4e61cec5f0e181a00bf8a2a5eb2060396af438a # Parent c8b1c5b3c6839f925a400ebeb9df36961ad2be49 Bug 3527: Remove unused code for deadlock detector Reviewed-by: ykubota https://github.com/HeapStats/heapstats/pull/132 diff -r c8b1c5b3c683 -r a4e61cec5f0e ChangeLog --- a/ChangeLog Thu Mar 29 13:43:09 2018 +0900 +++ b/ChangeLog Mon Apr 02 10:56:56 2018 +0900 @@ -1,3 +1,7 @@ +2018-04-02 Yasumasa Suenaga + + * Bug 3527: Remove unused code for deadlock detector + 2018-03-29 KUBOTA Yuji * Bump to 2.1 diff -r c8b1c5b3c683 -r a4e61cec5f0e agent/src/heapstats-engines/vmFunctions.cpp --- a/agent/src/heapstats-engines/vmFunctions.cpp Thu Mar 29 13:43:09 2018 +0900 +++ b/agent/src/heapstats-engines/vmFunctions.cpp Mon Apr 02 10:56:56 2018 +0900 @@ -1,7 +1,7 @@ /*! * \file vmFunctions.cpp * \brief This file includes functions in HotSpot VM.
- * Copyright (C) 2014-2017 Yasumasa Suenaga + * Copyright (C) 2014-2018 Yasumasa Suenaga * * This program is free software; you can redistribute it and/or * modify it under the terms of the GNU General Public License @@ -198,14 +198,6 @@ } } - /* Search "ObjectSynchronizer::get_lock_owner()" function symbol. */ - getLockOwner = (TGetLockOwner) this->symFinder->findSymbol( - GETLOCKOWNER_SYMBOL); - if (unlikely(getLockOwner == NULL)) { - logger->printWarnMsg("ObjectSynchronizer::get_lock_owner() not found."); - return false; - } - /* Search "ThreadSafepointState::create()" function symbol. */ threadSafepointStateCreate = (TVMThreadFunction) this->symFinder->findSymbol( THREADSAFEPOINTSTATE_CREATE_SYMBOL); diff -r c8b1c5b3c683 -r a4e61cec5f0e agent/src/heapstats-engines/vmFunctions.hpp --- a/agent/src/heapstats-engines/vmFunctions.hpp Thu Mar 29 13:43:09 2018 +0900 +++ b/agent/src/heapstats-engines/vmFunctions.hpp Mon Apr 02 10:56:56 2018 +0900 @@ -1,7 +1,7 @@ /*! * \file vmFunctions.hpp * \brief This file includes functions in HotSpot VM. - * Copyright (C) 2014-2017 Yasumasa Suenaga + * Copyright (C) 2014-2018 Yasumasa Suenaga * * This program is free software; you can redistribute it and/or * modify it under the terms of the GNU General Public License @@ -136,11 +136,6 @@ /*! - * \brief Symbol of ObjectSynchronizer::get_lock_owner(). - */ -#define GETLOCKOWNER_SYMBOL "_ZN18ObjectSynchronizer14get_lock_ownerE6Handleb" - -/*! * \brief Symbol of ThreadSafepointState::create(). */ #define THREADSAFEPOINTSTATE_CREATE_SYMBOL \ @@ -271,16 +266,6 @@ typedef void (*TSR_Handler)(int sig, siginfo_t *siginfo, ucontext_t *context); /*! - * \brief function type of - * "JavaThread* ObjectSynchronizer::get_lock_owner(Handle h_obj, bool doLock)". - * \param monitor_oop [in] Target monitor oop. - * \param doLock [in] Enable oop lock. - * \return Monitor owner thread oop.
- * Value is NULL, if owner is none. - */ -typedef void *(*TGetLockOwner)(void *monitor_oop, bool doLock); - -/*! * \brief function type of common thread operation. * \param thread [in] Target thread object is inner JVM class instance. */ @@ -364,11 +349,6 @@ TSR_Handler sr_handler; /*! - * \brief Function pointer for "ObjectSynchronizer::get_lock_owner()". - */ - TGetLockOwner getLockOwner; - - /*! * \brief Function pointer for "ThreadSafepointState::create()". */ TVMThreadFunction threadSafepointStateCreate; @@ -476,10 +456,6 @@ inline void *GetSRHandlerPointer(void) { return (void *)sr_handler; } - inline void *GetLockOwner(void *monitor_oop, bool doLock) { - return getLockOwner(monitor_oop, doLock); - } - inline void ThreadSafepointStateCreate(void *thread) { threadSafepointStateCreate(thread); }