Backport #8380: backport r40208 (refactor overflow check of integer multiplication)

• Status: Closed
• Assignee: nagachika (Tomoyuki Chikanaga)

[nagachika (Tomoyuki Chikanaga)]

rubyci.org にて OS X Mountain Lion(m64-o0) の時に TestThread#test_stack_size がタイムアウトするという現象が起きており、
r40600 で待ち時間を 10sec -> 50sec に伸ばしたのですが、それでもタイムアウトしていました。

http://a.mrkn.jp/~mrkn/chkbuild/mountain_lion/ruby-2.0.0-m64-o0/log/20130507T133423Z.log.html.gz

手元の類似の環境(x86_64-darwin12.3.0)で同じ configure オプションで追試したものの再現には至りませんでした。

rubyci のログを調べてみたところ trunk でも同じようなエラーは起きていて、
http://a.mrkn.jp/~mrkn/chkbuild/mountain_lion/ruby-trunk-m64-o0/log/20130409T131941Z.diff.html.gz
で直ったようです(znz さんに調査して頂きました。ありがとうございます)。

この間 r40204 から r40208 で、関係しそうなのは r40208 くらいで、これもどう関係するのかはっきりしませんが、バックポートして様子をみてみたいと思います。

[nagachika (Tomoyuki Chikanaga)]

• Status changed from Assigned to Closed
• % Done changed from 0 to 100

This issue was solved with changeset r40602.
Tomoyuki, thank you for reporting this issue.
Your contribution to Ruby is greatly appreciated.
May Ruby be with you.

---

merge revision(s) 40208: [Backport #8380]

* internal.h (MUL_OVERFLOW_SIGNED_INTEGER_P): New macro.
  (MUL_OVERFLOW_FIXNUM_P): Ditto.
  (MUL_OVERFLOW_LONG_P): Ditto.

* array.c (rb_ary_product): Don't overflow on signed integer
  multiplication.

* numeric.c (fix_mul): Ditto.
  (int_pow): Ditto.

* rational.c (f_imul): Ditto.

* insns.def (opt_mult): Ditto.

* thread.c (sleep_timeval): Don't overflow on signed integer addition.

* bignum.c (rb_int2big): Don't overflow on signed integer negation.
  (rb_big2ulong): Ditto.
  (rb_big2long): Ditto.
  (rb_big2ull): Ditto.
  (rb_big2ll): Ditto.

[znz (Kazuhiro NISHIYAMA)]

http://a.mrkn.jp/~mrkn/chkbuild/mountain_lion/ruby-2.0.0-m64-o0/log/20130507T213629Z.diff.html.gz
で TestThread#test_stack_size は直って、別のところで SEGV が起きていて、次の
http://a.mrkn.jp/~mrkn/chkbuild/mountain_lion/ruby-2.0.0-m64-o0/log/20130508T053526Z.diff.html.gz
も同じ revision で実行されていて、 SEGV もなく success になっているようです。

[nagachika (Tomoyuki Chikanaga)]

フォローありがとうございます。

理由はよくわかりませんが、タイムアウトはおきなくなったのでこのまま入れておこうと思います。