[openrtm-users 00038] RtcRingBuffer::get_new_list()のデッドロック

1 post / 0 new
root
Offline
Last seen: 11 hours 51 min ago
Joined: 2009-06-23 14:31
[openrtm-users 00038] RtcRingBuffer::get_new_list()のデッドロック

稲村@IHIです。

バグ報告です。

InPortで、getNewList()メソッドを使うと、デッドロックします。
getNewList()メソッドは、RtcRingBufferのget_new_list()メソッド
を呼び出していますが、ここで、同じミューテックスを2重にロックしていました。

inline std::vector get_new_list()
{
ACE_Guard guard(m_Mutex); // <= ここでロック
std::vector data;
int len(new_data_len()); // <= この中でもロック
data.resize(len);
 …
}

そこで、ロックを行なわないプライベートメソッドを用意し、
get_new_list()、new_data_len()から呼び出すように修正しました。

inline std::vector get_new_list()
{
ACE_Guard guard(m_Mutex); // <= ここでロック

std::vector data;
int len(new_data_len_no_guard()); // <= ロックしない
data.resize(len);
 …
}

inline int new_data_len()
{
ACE_Guard guard(m_Mutex); // <= ここでロック
return new_data_len_no_guard(); // <= ロックしない
}

private:
inline int new_data_len_no_guard()
{
int cnt(0);
for (int i(m_Newest);
m_Buffer[i].is_new() && cnt < m_Length;
++cnt, --i < 0 ? (i = m_Length - 1) : 0)
{;}
return cnt;
}

Undefined

Download

latest Releases : 2.0.0-RELESE

2.0.0-RELESE Download page

Number of Projects

Choreonoid

Motion editor/Dynamics simulator

OpenHRP3

Dynamics simulator

OpenRTP

Integrated Development Platform

AIST RTC collection

RT-Components collection by AIST

TORK

Tokyo Opensource Robotics Association

DAQ-Middleware

Middleware for DAQ (Data Aquisition) by KEK