Feature #1597
closedObjectSpace.count_nodes
Description
=begin
ささだです.
ObjectSpace.count_objects という,オブジェクトの種類を数えるメソッドが
ありますが,同じように ObjectSpace.count_nodes というものを加えるのはど
うでしょうか.
いや,使うのは多分 Ruby 開発者だけのような気もするので,正直どうかなぁ
とも思うんですが.
slot を全部巡る, st_each みたいなインターフェースがあれば,¶
拡張ライブラリとして実装することも不可能じゃなくなる,¶
けど,コールバックベースだと遅いかもなあ.¶
利用例:
例えば benchmark/bm_pentomino.rb を実行すると,なんか node が沢山出来
ているのが分かるの (*1) で,なぜだろうかと解析することが出来ます (*2).
*1:
http://www.atdot.net/fp_store/f.89p1lk/file.g.png
X軸が時間,左Y軸がオブジェクト数,右Y軸がGC回数
1秒ごとのサンプリング.
node の数が GC 回数に影響していることがわかる
*2:
http://www.atdot.net/fp_store/f.eio1lk/file.g.png
X軸が時間,Y軸が消費量・GC回数
1秒ごとのサンプリング.
ほぼ,NODE_LITの生成回数にGC回数が依存していることがわかる.
NODE_LIT は,実は break 時にテンポラリオブジェクトとして生成される
んだけど,pentomino では break を沢山するので,こういうことになる,
ということがわかった.
余談:¶
じゃぁ,break の実装を変えれば性能が向上するかというと,¶
GC の実行時間は1秒中0.1秒くらいなので,1割の性能向上が見込める,¶
かもしれない.¶
+static VALUE
+count_nodes(int argc, VALUE *argv, VALUE os)
+{
- rb_objspace_t *objspace = &rb_objspace;
- size_t nodes[NODE_LAST];
- size_t i;
- VALUE hash;
- if (rb_scan_args(argc, argv, "01", &hash) == 1) {
- }
- for (i = 0; i <= NODE_LAST; i++) {
- nodes[i] = 0;
- }
- for (i = 0; i < heaps_used; i++) {
- }
- }
- if (hash == Qnil) {
- }
- else if (!RHASH_EMPTY_P(hash)) {
- }
- for (i=0; i<NODE_LAST; i++) {
- if (nodes[i] != 0) {
+#define COUNT_NODE(n) case n: node = ID2SYM(rb_intern(#n)); break;
+#undef COUNT_NODE
- }
- }
- return hash;
+}
--
// SASADA Koichi at atdot dot net
=end
Updated by matz (Yukihiro Matsumoto) about 17 years ago
=begin
まつもと ゆきひろです
In message "Re: [ruby-dev:38584] [Feature: trunk] ObjectSpace.count_nodes"
on Thu, 11 Jun 2009 08:05:56 +0900, SASADA Koichi ko1@atdot.net writes:
| ObjectSpace.count_objects という,オブジェクトの種類を数えるメソッドが
|ありますが,同じように ObjectSpace.count_nodes というものを加えるのはど
|うでしょうか.
|
| いや,使うのは多分 Ruby 開発者だけのような気もするので,正直どうかなぁ
|とも思うんですが.
「nodeが存在する」というのは特定の実装に依存する話ですので、
メソッドとして追加するのはどうかと思います。関数を用意して
おいて、拡張ライブラリでメソッドとして登録するならありなんじゃ
ないかなあ。
=end
Updated by yugui (Yuki Sonoda) almost 17 years ago
- Assignee set to ko1 (Koichi Sasada)
- Target version set to 2.0.0
=begin
=end
Updated by naruse (Yui NARUSE) over 16 years ago
- Status changed from Open to Rejected
=begin
=end