プロジェクト

全般

プロフィール

バグ #2327

RingBuffer関連の問題

匿名ユーザー が12年以上前に追加. ほぼ11年前に更新.

ステータス:
終了
優先度:
通常
担当者:
対象バージョン:
-
開始日:
2012/01/16
期日:
進捗率:

100%

予定工数:

説明

KEK仲吉さん安さんから指摘。

とりあえずの対処として read()の前に InPortに対し isEmpty()を
行っていました。この件を安さんが調査し次のような問題を見つけ
られましたので報告致します(以下、安さんからのメールの抜粋)。

| RingBuffer.hのwriteメソッドで、
| 従来は
|
| if (empty_)
| {
|    Guard eguard(m_empty.mutex);
|    m_empty.cond.signal();
| }
| advanceWptr(1);
|
| となっていました。
| これだと、readでバッファがなく待っていた状態で、writeが入り、
| cond.signalでシグナルを出しますが、 advanceWptr(1)をする前に、
| シグナルを出しているため、writeのバッファポインタがインクリ
| メントされる前に、readがwake upする可能性が出てきてしまいます。
| そこで、下記のように変更しました。
|
| if (empty_)
| {
|     Guard eguard(m_empty.mutex);
|     advanceWptr(1);
|     m_empty.cond.signal();
| } else
|     advanceWptr(1);
|
| これで、昨日の昼から走らせていますが、数秒で止まってしまう場合
| があったり数十分で止まってしまったありする場合があるのに比べて、
| いまだ走っていますので、つじつまがあう点を考えても、問題は解決
| したものと思います。

関連するチケット

関連している OpenRTM-aist (C++) - バグ #1426: 1.0.0のRingBuffer関連の問題終了2010-04-22

関係しているリビジョン

リビジョン 641 (差分)
fsi-katami が12年以上前に追加

RingBuffer\'s bug was fixed. refs #2327

リビジョン 684 (差分)
ga約11年前に追加

The bugs of RingBuffer class have been fixed. refs #2327

履歴

#1 匿名ユーザー が12年以上前に更新

  • ステータス新規 から 解決 に変更
  • 担当者匿名ユーザー にセット

#2 ga11年以上前に更新

  • 担当者匿名ユーザー から ga に変更
  • 進捗率0 から 80 に変更

#3 ga約11年前に更新

  • 進捗率80 から 100 に変更
1. write()メソッドの問題修正
  • m_rtimeoutをm_wtimeoutに修正
  • block modeにおいて、m_full.mutex.wait()からの復帰がnotify()によるものか、タイムアウトによるものなのかにかかわらず"return ReturnCode.TIMEOUT;"となっていたため、"full()"にて確認したうえで "return ReturnCode.TIMEOUT;"するように修正。
2. read()メソッドの問題修正
  • m_rtimeoutをm_wtimeoutに修正
  • block modeにおいて、m_full.mutex.wait()からの復帰がnotify()によるものか、タイムアウトによるものなのかにかかわらず"return ReturnCode.TIMEOUT;"となっていたため、"full()"にて確認したうえで "return ReturnCode.TIMEOUT;"するように修正。
3. 本チケットに関する実装の見直し
  • advanceRptr(1)の直前でfull()をコールするように変更。
  • advanceWptr(1)の直前でempty()をコールするように変更。

上記の変更は、 r684 にて対応しました。

#4 n-andoほぼ11年前に更新

  • ステータス解決 から 終了 に変更

他の形式にエクスポート: Atom PDF