Bug #1310: HEAP_SIZE で不具合？

• Status: Closed
• Assignee: authorNari (Narihiro Nakamura)
• Target version: -
• ruby -v: ruby 1.9.2dev (2009-03-22) [i686-linux]

[hikari (Hikari AIKAWA)]

=begin
はじめまして．
東京大学の相川と申します．

gc.cのHEAP_SIZEの値を変化させてみたところ，

#define HEAP_SIZE 0x20000

とし，

$ make miniruby

とすると，ビルドは通るのですが，minirubyを実行するとSEGVが出ました．
とりあえず報告させていただきます．

$ ./miniruby -v
[BUG] Segmentation fault
ruby 1.9.2dev (2009-03-22) [i686-linux]

-- control frame ----------
c:0001 p:---- s:0002 b:0002 l:000001 d:000001 TOP

-- C level backtrace information -------------------------------------------
0x819a455 ./miniruby(rb_vm_bugreport+0xb5) [0x819a455]
0x808d9bb ./miniruby [0x808d9bb]
0x808da48 ./miniruby(rb_bug+0x28) [0x808da48]
0x8131eb5 ./miniruby [0x8131eb5]
0xb80a0410 [0xb80a0410]
0x81403cd ./miniruby [0x81403cd]
0x81406a7 ./miniruby(rb_enc_str_new+0x17) [0x81406a7]
0x80dc2ed ./miniruby(rb_intern3+0x16d) [0x80dc2ed]
0x80df74e ./miniruby(rb_intern+0x2e) [0x80df74e]
0x807a0ba ./miniruby(rb_define_module_function+0x4a) [0x807a0ba]
0x80a146c ./miniruby(Init_GC+0x27c) [0x80a146c]
0x80acc15 ./miniruby(rb_call_inits+0x115) [0x80acc15]
0x8092515 ./miniruby(ruby_init+0xa5) [0x8092515]
0x805c6a7 ./miniruby(main+0x47) [0x805c6a7]
0xb7eab685 /lib/tls/i686/cmov/libc.so.6(__libc_start_main+0xe5) [0xb7eab685]
0x805c5c1 ./miniruby [0x805c5c1]

[NOTE]
You may have encountered a bug in the Ruby interpreter or extension libraries.
Bug reports are welcome.
For details: http://www.ruby-lang.org/bugreport.html
Aborted

--
東京大学大学院情報理工学系研究科
創造情報学専攻　修士一年
相川　光
=end

[authorNari (Narihiro Nakamura)]

=begin
nariです．

どうやら HEAP_SIZE が大きすぎると，追加する heap_slot 数の計算がおかしくなるようです．

以下の様に修正すると直りました．

宜しければコミットしたいのですがいかがでしょうか．

diff --git a/gc.c b/gc.c
index 835ef5d..8280ec1 100644
--- a/gc.c
+++ b/gc.c
@@ -922,6 +922,9 @@ init_heap(rb_objspace_t *objspace)

  add = HEAP_MIN_SLOTS / HEAP_OBJ_LIMIT;

• if (add)

•    add = 1;
  

• if ((heaps_used + add) > heaps_length) {
  allocate_heaps(objspace, heaps_used + add);
  }
  @@ -938,6 +941,10 @@ static void
  set_heaps_increment(rb_objspace_t *objspace)
  {
  size_t next_heaps_length = (size_t)(heaps_used * 1.8);

• if (next_heaps_length == heaps_used)

•    next_heaps_length++;
  

• heaps_inc = next_heaps_length - heaps_used;

  if (next_heaps_length > heaps_length) {

---

Narihiro Nakmaura

=end

[nobu (Nobuyoshi Nakada)]

=begin
なかだです。

At Sun, 22 Mar 2009 22:48:24 +0900,
Narihiro Nakamura wrote in [ruby-dev:38185]:

@@ -922,6 +922,9 @@ init_heap(rb_objspace_t *objspace)

 add = HEAP_MIN_SLOTS / HEAP_OBJ_LIMIT;

• if (add)

•    add = 1;
  

if (!add) ではなく?

--
--- 僕の前にBugはない。
--- 僕の後ろにBugはできる。
中田 伸悦

=end

[authorNari (Narihiro Nakamura)]

=begin
nari です．

うぅ，失礼しました．．
ご指摘ありがとうございます．

diff --git a/gc.c b/gc.c
index 835ef5d..829fa7f 100644
--- a/gc.c
+++ b/gc.c
@@ -922,6 +922,9 @@ init_heap(rb_objspace_t *objspace)

  add = HEAP_MIN_SLOTS / HEAP_OBJ_LIMIT;

• if (!add)

•    add = 1;
  

• if ((heaps_used + add) > heaps_length) {
  allocate_heaps(objspace, heaps_used + add);
  }
  @@ -938,6 +941,10 @@ static void
  set_heaps_increment(rb_objspace_t *objspace)
  {
  size_t next_heaps_length = (size_t)(heaps_used * 1.8);

• if (next_heaps_length == heaps_used)

•    next_heaps_length++;
  

• heaps_inc = next_heaps_length - heaps_used;

  if (next_heaps_length > heaps_length) {

2009/03/23 6:52 Nobuyoshi Nakada [e-mail]:

なかだです。

At Sun, 22 Mar 2009 22:48:24 +0900,
Narihiro Nakamura wrote in [ruby-dev:38185]:

@@ -922,6 +922,9 @@ init_heap(rb_objspace_t *objspace)

 add = HEAP_MIN_SLOTS / HEAP_OBJ_LIMIT;

• if (add)

•    add = 1;
  

if (!add) ではなく?

--
--- 僕の前にBugはない。
--- 僕の後ろにBugはできる。
中田 伸悦

--
Narihiro Nakamura

=end

[matz (Yukihiro Matsumoto)]

=begin
まつもと ゆきひろです

In message "Re: [ruby-dev:38187] Re: [Bug #1310] HEAP_SIZE で不具合？"
on Mon, 23 Mar 2009 07:15:37 +0900, Narihiro Nakamura [e-mail] writes:

|ご指摘ありがとうございます．
|
|diff --git a/gc.c b/gc.c
|index 835ef5d..829fa7f 100644

ちゃんと動かして問題が出ないようならコミットしてください。

=end

[authorNari (Narihiro Nakamura)]

• Status changed from Open to Closed
• Assignee set to authorNari (Narihiro Nakamura)

=begin

=end