https://bugs.ruby-lang.org/
https://bugs.ruby-lang.org/favicon.ico?1711330511
2013-03-20T22:37:34Z
Ruby Issue Tracking System
Backport200 - Backport #8064: r39687 - io.c (rb_update_max_fd): use ATOMIC_CAS のバックポート
https://bugs.ruby-lang.org/issues/8064?journal_id=37762
2013-03-20T22:37:34Z
nagachika (Tomoyuki Chikanaga)
nagachika00@gmail.com
<ul></ul><p>r39687 と一緒に変数の型を変更している r39776 もバックポートしようと思います。</p>
Backport200 - Backport #8064: r39687 - io.c (rb_update_max_fd): use ATOMIC_CAS のバックポート
https://bugs.ruby-lang.org/issues/8064?journal_id=37904
2013-03-25T05:21:55Z
kosaki (Motohiro KOSAKI)
kosaki.motohiro@gmail.com
<ul></ul><p>あ、すいません。これちょっと待ってください。<br>
r39687書いたときはちょっと考慮できていなかったのですが、型を変更してしまうとどこが壊れるかわからないので、<br>
素直にmutex導入して、変更最小限修正に出来ないか一度考えたい。</p>
Backport200 - Backport #8064: r39687 - io.c (rb_update_max_fd): use ATOMIC_CAS のバックポート
https://bugs.ruby-lang.org/issues/8064?journal_id=38760
2013-04-20T00:37:37Z
nagachika (Tomoyuki Chikanaga)
nagachika00@gmail.com
<ul></ul><p>こちらどうでしょうか。急がなくてもいいと思いますが、r39687 だけ入れていてもあまり意味はないでしょうか?</p>
Backport200 - Backport #8064: r39687 - io.c (rb_update_max_fd): use ATOMIC_CAS のバックポート
https://bugs.ruby-lang.org/issues/8064?journal_id=38856
2013-04-24T14:18:16Z
kosaki (Motohiro KOSAKI)
kosaki.motohiro@gmail.com
<ul></ul><p>見なおしたけど r40438 入れれば、触ってる場所一箇所になるから僕的に許せる感じ。</p>
Backport200 - Backport #8064: r39687 - io.c (rb_update_max_fd): use ATOMIC_CAS のバックポート
https://bugs.ruby-lang.org/issues/8064?journal_id=38935
2013-04-27T02:09:53Z
nagachika (Tomoyuki Chikanaga)
nagachika00@gmail.com
<ul></ul><p>ありがとうございます。<br>
一旦 r39687, r39776, r40438 をマージして閉じようと思います。<br>
追加で変更があればまた backport チケットを作ってください。</p>
Backport200 - Backport #8064: r39687 - io.c (rb_update_max_fd): use ATOMIC_CAS のバックポート
https://bugs.ruby-lang.org/issues/8064?journal_id=38937
2013-04-27T02:31:48Z
nagachika (Tomoyuki Chikanaga)
nagachika00@gmail.com
<ul><li><strong>Status</strong> changed from <i>Assigned</i> to <i>Closed</i></li><li><strong>% Done</strong> changed from <i>0</i> to <i>100</i></li></ul><p>This issue was solved with changeset r40490.<br>
Motohiro, thank you for reporting this issue.<br>
Your contribution to Ruby is greatly appreciated.<br>
May Ruby be with you.</p>
<hr>
<p>merge revision(s) 39687,39776,40438: [Backport <a class="issue tracker-4 status-5 priority-4 priority-default closed" title="Backport: r39687 - io.c (rb_update_max_fd): use ATOMIC_CAS のバックポート (Closed)" href="https://bugs.ruby-lang.org/issues/8064">#8064</a>]</p>
<pre><code>* io.c (rb_update_max_fd): use ATOMIC_CAS because this function
is used from timer thread too.
* io.c (rb_fd_fix_cloexec): use rb_update_max_fd().
</code></pre>