Hi Ravi,
I start reviewing today
Thanks
Minh
On 04/04/18 17:19, Ravi Sekhar Reddy Konda wrote:
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
