Hi Minh, Did you get time to look at this patch, please consider reviewing it with priority
Thanks, Ravi -----Original Message----- From: ravi-sekhar [mailto:[email protected]] Sent: Thursday, March 29, 2018 11:30 AM To: [email protected]; [email protected]; [email protected] Cc: [email protected]; ravi-sekhar <[email protected]> Subject: [PATCH 1/1] amfnd: unlock before releasing the monitoring thread to avoid deadlock [#2818] --- src/amf/amfnd/mon.cc | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/amf/amfnd/mon.cc b/src/amf/amfnd/mon.cc index 9cdfc37..4932d50 100644 --- a/src/amf/amfnd/mon.cc +++ b/src/amf/amfnd/mon.cc @@ -161,6 +161,8 @@ uint32_t avnd_mon_req_del(AVND_CB *cb, SaUint64T pid) { mon_rec = (AVND_MON_REQ *)m_NCS_DBLIST_FIND_FIRST(pid_mon_list); + m_NCS_UNLOCK(&cb->mon_lock, NCS_LOCK_WRITE); + /* No more PIDs exists in the pid_mon_list for monitoring */ if (!mon_rec) { /* destroy the task */ @@ -173,8 +175,6 @@ uint32_t avnd_mon_req_del(AVND_CB *cb, SaUint64T pid) { } } - m_NCS_UNLOCK(&cb->mon_lock, NCS_LOCK_WRITE); - return rc; } -- 1.9.1 ------------------------------------------------------------------------------ Check out the vibrant tech community on one of the world's most engaging tech sites, Slashdot.org! http://sdm.link/slashdot _______________________________________________ Opensaf-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/opensaf-devel
