Menu
▾
▴
ruby-gnome2-devel-ja
|
From: Masahiro S. ()
<sa...@to...> - 2006-05-27 04:01:18
|
酒井です。
rbgobj_closure.c について質問です。
> #define CALLBACK_KEY rb_str_new2("callback")
> #define EXTRA_ARGS_KEY rb_str_new2("extra_args")
としていますが、これだと CALLBACK_KEY や EXTRA_ARGS_KEY が使われるごとに
rb_str_new2することになって効率が悪いのではないでしょうか?
> 2006-04-15 Kouhei Sutou <ko...@co...>
>
> * src/rbgobj_closure.c (rclosure_marshal, rclosure_marshal_body):
> Changed an argument type to VALUE from struct marshal_arg *.
この変更は何のためなのでしょうか?
以前の struct marshal_arg をスタックに確保するコードに比べて、
ヒープに struct marshal_arg を確保する分遅くなります。
--
酒井 政裕 / Masahiro Sakai
|
|
From: Kouhei S. <ko...@co...> - 2006-05-27 04:15:57
|
須藤です．
私はあまり効率にこだわらないタイプです．ごめんなさい．
In <200...@to...>
"[ruby-gnome2-devel-ja] rbgobj_closure.c について質問" on Sat, 27 May 2006 13:01:00 +0900 (JST),
Masahiro Sakai (酒井政裕) <sa...@to...> wrote:
> > #define CALLBACK_KEY rb_str_new2("callback")
> > #define EXTRA_ARGS_KEY rb_str_new2("extra_args")
>
> としていますが、これだと CALLBACK_KEY や EXTRA_ARGS_KEY が使われるごとに
> rb_str_new2することになって効率が悪いのではないでしょうか?
常にメモリ上にもっておくことが嫌だったのでこうしました．
こだわっていないのでグローバルに持つようにしてもらってかまい
ません．
> > 2006-04-15 Kouhei Sutou <ko...@co...>
> >
> > * src/rbgobj_closure.c (rclosure_marshal, rclosure_marshal_body):
> > Changed an argument type to VALUE from struct marshal_arg *.
>
> この変更は何のためなのでしょうか?
VALUEにキャストしたりして強引（でもないですが）に呼び出すの
が気持ち悪かったのでこうしました．
> 以前の struct marshal_arg をスタックに確保するコードに比べて、
> ヒープに struct marshal_arg を確保する分遅くなります。
そういうことは全然考えていませんでした．revertしてかまいませ
ん．
|
|
From: Masao M. <mu...@hi...> - 2006-05-27 05:33:28
|
むとうです。 On Sat, 27 May 2006 13:34:54 +0900 (JST) Masahiro Sakai (酒井政裕) <sa...@to...> wrote: > 効率はともかく、それ以外でもまずいことになっているのに気づいたので、 > revertさせてください。 > > Sjoerd Simons の言うように別スレッドでrclosure_marshalが呼ばれること > があるとすると、Data_Make_StructでRubyのオブジェクトを割り当てるべき > ではないです。Data_Make_StructでGCが発生した場合にまずいし、 > またData_Make_Structしてからメインスレッドにそのオブジェクトが渡るまでの > 間はGCから保護されなくてまずいので。 おっと、これはちょっと待ってください。 今、Sjoerdのパッチが取り込まれるところです（SjoerdにCVSへのアクセス権を 与えました）。 なので、それとの関係性はどうなるのがベストでしょうか？ 最終形態がSjoerdのものを取り込んだ状態、あるいは同様のもの、 と言う形にはしたいのですが。 なので、revertというよりは、むしろ、SjoerdがあてるであろうCVS版への パッチ、と言う形が望ましいのではないかと思うのですが、いかがでしょうか。 -- .:% Masao Mutoh<mu...@hi...> |
|
From: Masahiro S. ()
<sa...@to...> - 2006-05-27 05:51:04
|
酒井です。 From: Masao Mutoh <mu...@hi...> Date: Sat, 27 May 2006 14:33:17 +0900 > むとうです。 > > On Sat, 27 May 2006 13:34:54 +0900 (JST) > Masahiro Sakai (酒井政裕) <sa...@to...> wrote: > > > 効率はともかく、それ以外でもまずいことになっているのに気づいたので、 > > revertさせてください。 > > > > Sjoerd Simons の言うように別スレッドでrclosure_marshalが呼ばれること > > があるとすると、Data_Make_StructでRubyのオブジェクトを割り当てるべき > > ではないです。Data_Make_StructでGCが発生した場合にまずいし、 > > またData_Make_Structしてからメインスレッドにそのオブジェクトが渡るまでの > > 間はGCから保護されなくてまずいので。 > > おっと、これはちょっと待ってください。 > > 今、Sjoerdのパッチが取り込まれるところです（SjoerdにCVSへのアクセス権を > 与えました）。 > > なので、それとの関係性はどうなるのがベストでしょうか？ > 最終形態がSjoerdのものを取り込んだ状態、あるいは同様のもの、 > と言う形にはしたいのですが。 > > なので、revertというよりは、むしろ、SjoerdがあてるであろうCVS版への > パッチ、と言う形が望ましいのではないかと思うのですが、いかがでしょうか。 うげっ。 Sjoerdのパッチは既にコミットされていたので、それに対して修正しちゃいました。 ひょっとして、Sjoerdの作業はまだ終わってなかったですか? # devel-enをあまりまじめに読んでませんでした…… -- 酒井 政裕 / Masahiro Sakai |
|
From: Masao M. <mu...@hi...> - 2006-05-27 05:59:13
|
むとうです。 On Sat, 27 May 2006 14:50:15 +0900 (JST) Masahiro Sakai (酒井政裕) <sa...@to...> wrote: > 酒井です。 > > From: Masao Mutoh <mu...@hi...> > Date: Sat, 27 May 2006 14:33:17 +0900 > > > むとうです。 > > > > On Sat, 27 May 2006 13:34:54 +0900 (JST) > > Masahiro Sakai (酒井政裕) <sa...@to...> wrote: > > > > > 効率はともかく、それ以外でもまずいことになっているのに気づいたので、 > > > revertさせてください。 > > > > > > Sjoerd Simons の言うように別スレッドでrclosure_marshalが呼ばれること > > > があるとすると、Data_Make_StructでRubyのオブジェクトを割り当てるべき > > > ではないです。Data_Make_StructでGCが発生した場合にまずいし、 > > > またData_Make_Structしてからメインスレッドにそのオブジェクトが渡るまでの > > > 間はGCから保護されなくてまずいので。 > > > > おっと、これはちょっと待ってください。 > > > > 今、Sjoerdのパッチが取り込まれるところです（SjoerdにCVSへのアクセス権を > > 与えました）。 > > > > なので、それとの関係性はどうなるのがベストでしょうか？ > > 最終形態がSjoerdのものを取り込んだ状態、あるいは同様のもの、 > > と言う形にはしたいのですが。 > > > > なので、revertというよりは、むしろ、SjoerdがあてるであろうCVS版への > > パッチ、と言う形が望ましいのではないかと思うのですが、いかがでしょうか。 > > うげっ。 > Sjoerdのパッチは既にコミットされていたので、それに対して修正しちゃいました。 > ひょっとして、Sjoerdの作業はまだ終わってなかったですか? > # devel-enをあまりまじめに読んでませんでした…… あ、いや、ごめんなさい。私の勘違いだったみたいです。 確かにコミットメールにSjoerdのがありました。 ＃昨日の夜にあげたばっかりだったのでまだ作業してないかと思いこんでました・・・。 終わりよければすべてよし、ということで。 ではでは。 -- .:% Masao Mutoh<mu...@hi...> |
|
From: Masahiro S. ()
<sa...@to...> - 2006-05-27 04:35:17
|
酒井です。
From: Kouhei Sutou <ko...@co...>
Date: Sat, 27 May 2006 13:15:51 +0900 (JST)
> 須藤です．
>
> 私はあまり効率にこだわらないタイプです．ごめんなさい．
>
> In <200...@to...>
> "[ruby-gnome2-devel-ja] rbgobj_closure.c について質問" on Sat, 27 May 2006 13:01:00 +0900 (JST),
> Masahiro Sakai (酒井政裕) <sa...@to...> wrote:
>
> > > #define CALLBACK_KEY rb_str_new2("callback")
> > > #define EXTRA_ARGS_KEY rb_str_new2("extra_args")
> >
> > としていますが、これだと CALLBACK_KEY や EXTRA_ARGS_KEY が使われるごとに
> > rb_str_new2することになって効率が悪いのではないでしょうか?
>
> 常にメモリ上にもっておくことが嫌だったのでこうしました．
> こだわっていないのでグローバルに持つようにしてもらってかまい
> ません．
この場合の正解は、callbackとextra_argsをGRClosureHolderのフィールドに
持つようにして、markすることだと思います。
> > > 2006-04-15 Kouhei Sutou <ko...@co...>
> > >
> > > * src/rbgobj_closure.c (rclosure_marshal, rclosure_marshal_body):
> > > Changed an argument type to VALUE from struct marshal_arg *.
> >
> > この変更は何のためなのでしょうか?
>
> VALUEにキャストしたりして強引（でもないですが）に呼び出すの
> が気持ち悪かったのでこうしました．
なるほど。
> > 以前の struct marshal_arg をスタックに確保するコードに比べて、
> > ヒープに struct marshal_arg を確保する分遅くなります。
>
> そういうことは全然考えていませんでした．revertしてかまいませ
> ん．
効率はともかく、それ以外でもまずいことになっているのに気づいたので、
revertさせてください。
Sjoerd Simons の言うように別スレッドでrclosure_marshalが呼ばれること
があるとすると、Data_Make_StructでRubyのオブジェクトを割り当てるべき
ではないです。Data_Make_StructでGCが発生した場合にまずいし、
またData_Make_Structしてからメインスレッドにそのオブジェクトが渡るまでの
間はGCから保護されなくてまずいので。
--
酒井 政裕 / Masahiro Sakai
|
|
From: Masahiro S. <sa...@to...> - 2007-06-11 16:00:44
|
酒井です。
1年前のメールですが、去年のRubyKaigi以来忘れていたのを、
先日のRubyKaigiで思い出したので……
From: Masahiro Sakai (酒井政裕) <sa...@to...>
Date: Sat, 27 May 2006 13:34:54 +0900 (JST)
> 酒井です。
>
> From: Kouhei Sutou <ko...@co...>
> Date: Sat, 27 May 2006 13:15:51 +0900 (JST)
>
> > 須藤です．
> >
> > 私はあまり効率にこだわらないタイプです．ごめんなさい．
> >
> > In <200...@to...>
> > "[ruby-gnome2-devel-ja] rbgobj_closure.c について質問" on Sat, 27 May 2006 13:01:00 +0900 (JST),
> > Masahiro Sakai (酒井政裕) <sa...@to...> wrote:
> >
> > > > #define CALLBACK_KEY rb_str_new2("callback")
> > > > #define EXTRA_ARGS_KEY rb_str_new2("extra_args")
> > >
> > > としていますが、これだと CALLBACK_KEY や EXTRA_ARGS_KEY が使われるごとに
> > > rb_str_new2することになって効率が悪いのではないでしょうか?
> >
> > 常にメモリ上にもっておくことが嫌だったのでこうしました．
> > こだわっていないのでグローバルに持つようにしてもらってかまい
> > ません．
>
> この場合の正解は、callbackとextra_argsをGRClosureHolderのフィールドに
> 持つようにして、markすることだと思います。
これは以下のような感じのコードにしてはどうか、ということです。
効率面だけでなく、GRClosureHolderというデータ構造が不要になる分、
コードも若干シンプルになると思います。
--
酒井 政裕 / Masahiro Sakai
Index: src/rbgobj_closure.c
===================================================================
RCS file: /cvsroot/ruby-gnome2/ruby-gnome2/glib/src/rbgobj_closure.c,v
retrieving revision 1.40
diff -u -p -r1.40 rbgobj_closure.c
--- src/rbgobj_closure.c 16 Dec 2006 05:01:30 -0000 1.40
+++ src/rbgobj_closure.c 11 Jun 2007 15:31:16 -0000
@@ -25,27 +25,14 @@
#endif
static ID id_call;
-static ID id_holder;
static gboolean rclosure_initialized = FALSE;
-#define CALLBACK_KEY rb_str_new2("callback")
-#define EXTRA_ARGS_KEY rb_str_new2("extra_args")
-
-#define HOLDER(closure) \
- (rb_ivar_get((closure)->rb_holder, id_holder))
-#define SETUP_HOLDER(closure) \
- (rb_ivar_set((closure)->rb_holder, id_holder, rb_hash_new()))
-
typedef struct _GRClosure GRClosure;
-
-typedef struct _GRClosureHolder
-{
- GRClosure *closure;
-} GRClosureHolder;
-
struct _GRClosure
{
GClosure closure;
+ VALUE callback;
+ VALUE extra_args;
VALUE rb_holder;
gint count;
GList *objects;
@@ -112,8 +99,8 @@ rclosure_marshal_do(VALUE arg_)
if (rclosure_alive_p(rclosure)) {
VALUE callback, extra_args;
- callback = rb_hash_aref(HOLDER(rclosure), CALLBACK_KEY);
- extra_args = rb_hash_aref(HOLDER(rclosure), EXTRA_ARGS_KEY);
+ callback = rclosure->callback;
+ extra_args = rclosure->extra_args;
if (!NIL_P(extra_args)) {
args = rb_ary_concat(args, extra_args);
@@ -249,13 +236,7 @@ rclosure_unref(GRClosure *rclosure)
}
g_list_free(rclosure->objects);
rclosure->objects = NULL;
-
- if (!NIL_P(rclosure->rb_holder)) {
- GRClosureHolder *holder;
- Data_Get_Struct(rclosure->rb_holder, GRClosureHolder, holder);
- holder->closure = NULL;
- rclosure->rb_holder = Qnil;
- }
+ rclosure->rb_holder = Qnil;
}
}
@@ -271,36 +252,35 @@ rclosure_invalidate(gpointer data, GClos
}
static void
-gr_closure_holder_free(GRClosureHolder *holder)
+gr_closure_holder_mark(GRClosure *rclosure)
{
- if (holder) {
- if (holder->closure) {
- holder->closure->rb_holder = Qnil;
- rclosure_invalidate(NULL, (GClosure *)(holder->closure));
- }
- free(holder);
- }
+ rb_gc_mark(rclosure->callback);
+ rb_gc_mark(rclosure->extra_args);
+}
+
+static void
+gr_closure_holder_free(GRClosure *rclosure)
+{
+ rclosure->rb_holder = Qnil;
+ rclosure_invalidate(NULL, (GClosure*)rclosure);
}
GClosure*
g_rclosure_new(VALUE callback_proc, VALUE extra_args, GValToRValSignalFunc g2r_func)
{
GRClosure* closure;
- GRClosureHolder *holder;
closure = (GRClosure*)g_closure_new_simple(sizeof(GRClosure), NULL);
closure->count = 1;
closure->g2r_func = g2r_func;
- closure->rb_holder = Data_Make_Struct(rb_cData, GRClosureHolder, 0,
- gr_closure_holder_free, holder);
closure->objects = NULL;
-
- holder->closure = closure;
- SETUP_HOLDER(closure);
-
- rb_hash_aset(HOLDER(closure), CALLBACK_KEY, callback_proc);
- rb_hash_aset(HOLDER(closure), EXTRA_ARGS_KEY, extra_args);
+ closure->callback = callback_proc;
+ closure->extra_args = extra_args;
+ closure->rb_holder = Data_Wrap_Struct(rb_cData,
+ gr_closure_holder_mark,
+ gr_closure_holder_free,
+ closure);
g_closure_set_marshal((GClosure*)closure, &rclosure_marshal);
g_closure_add_invalidate_notifier((GClosure*)closure, NULL,
@@ -350,7 +330,6 @@ static void
Init_rclosure()
{
id_call = rb_intern("call");
- id_holder = rb_intern("holder");
#ifdef HAVE_NATIVETHREAD
/* startup the ruby thread to pull callbacks from other threads */
|
|
From: Masao M. <mu...@hi...> - 2007-06-12 06:54:04
|
むとうです。 On Tue, 12 Jun 2007 00:59:53 +0900 (JST) Masahiro Sakai <sa...@to...> wrote: > 酒井です。 > > 1年前のメールですが、去年のRubyKaigi以来忘れていたのを、 > 先日のRubyKaigiで思い出したので…… 先日はどうもありがとうございました。 それにしても、またよく覚えてらっしゃいました（笑）。 > > この場合の正解は、callbackとextra_argsをGRClosureHolderのフィールドに > > 持つようにして、markすることだと思います。 > > これは以下のような感じのコードにしてはどうか、ということです。 > 効率面だけでなく、GRClosureHolderというデータ構造が不要になる分、 > コードも若干シンプルになると思います。 なるほど。確かにシンプルですね。良いと思います。 須藤さんはどう思われますか？ |
|
From: Masao M. <mu...@hi...> - 2007-06-13 12:40:12
|
むとうです。 On Tue, 12 Jun 2007 16:05:27 +0900 "Kouhei Sutou" <ko...@co...> wrote: > 須藤です。 > > 07/06/12 に Masao Mutoh<mu...@hi...> さんは書きました: > > > > > この場合の正解は、callbackとextra_argsをGRClosureHolderのフィールドに > > > > 持つようにして、markすることだと思います。 > > > > > > これは以下のような感じのコードにしてはどうか、ということです。 > > > 効率面だけでなく、GRClosureHolderというデータ構造が不要になる分、 > > > コードも若干シンプルになると思います。 > > > > なるほど。確かにシンプルですね。良いと思います。 > > 須藤さんはどう思われますか？ > > 近いうちにリリースする予定はないですよね？ > であれば、一度コミットしてもらってしばらく手元で動かしてみる > というのはどうでしょう。 > 手元で使っていて何か問題がでたらrevertすればよいだけですし。 そうですね。さかいさん、久しぶりにチェックインしていただいても いいでしょうか？ |
|
From: Masahiro S. <sa...@to...> - 2007-06-13 14:34:31
|
酒井です。 From: Masao Mutoh <mu...@hi...> Date: Tue, 12 Jun 2007 15:51:32 +0900 > むとうです。 > > On Tue, 12 Jun 2007 00:59:53 +0900 (JST) > Masahiro Sakai <sa...@to...> wrote: > > > 酒井です。 > > > > 1年前のメールですが、去年のRubyKaigi以来忘れていたのを、 > > 先日のRubyKaigiで思い出したので…… > > 先日はどうもありがとうございました。 こちらこそ先日はどうもありがとうございました。 むとうさんの発表良かったですよ。 > それにしても、またよく覚えてらっしゃいました（笑）。 何か忘れてることがあったなと思って手元のソースを見たら、 このコードがあったので、詳しいことはこれを見て思い出したのでした。 From: Masao Mutoh <mu...@hi...> Date: Wed, 13 Jun 2007 21:40:02 +0900 > むとうです。 > > On Tue, 12 Jun 2007 16:05:27 +0900 > "Kouhei Sutou" <ko...@co...> wrote: > > > 須藤です。 > > > > 07/06/12 に Masao Mutoh<mu...@hi...> さんは書きました: > > > > > > > この場合の正解は、callbackとextra_argsをGRClosureHolderのフィールドに > > > > > 持つようにして、markすることだと思います。 > > > > > > > > これは以下のような感じのコードにしてはどうか、ということです。 > > > > 効率面だけでなく、GRClosureHolderというデータ構造が不要になる分、 > > > > コードも若干シンプルになると思います。 > > > > > > なるほど。確かにシンプルですね。良いと思います。 > > > 須藤さんはどう思われますか？ > > > > 近いうちにリリースする予定はないですよね？ > > であれば、一度コミットしてもらってしばらく手元で動かしてみる > > というのはどうでしょう。 > > 手元で使っていて何か問題がでたらrevertすればよいだけですし。 > > そうですね。さかいさん、久しぶりにチェックインしていただいても > いいでしょうか？ というわけで、チェックインしました。 ChangeLogに怪しい英語を書くのも久しぶりです(^^; -- 酒井 政裕 / Masahiro Sakai |
|
From: Masao M. <mu...@hi...> - 2007-06-13 15:53:45
|
むとうです。 On Wed, 13 Jun 2007 23:33:55 +0900 (JST) Masahiro Sakai <sa...@to...> wrote: > 酒井です。 > > 先日はどうもありがとうございました。 > > こちらこそ先日はどうもありがとうございました。 > むとうさんの発表良かったですよ。 いやぁ、緊張しまくりでしたよ（苦笑）。 > というわけで、チェックインしました。 > ChangeLogに怪しい英語を書くのも久しぶりです(^^; いやー、ほんと久し振りですね！ 私も徐々に復活します！ ではでは。 |
|
From: Kouhei S. <ko...@co...> - 2007-06-12 07:05:29
|
須藤です。 07/06/12 に Masao Mutoh<mu...@hi...> さんは書きました: > > > この場合の正解は、callbackとextra_argsをGRClosureHolderのフィールドに > > > 持つようにして、markすることだと思います。 > > > > これは以下のような感じのコードにしてはどうか、ということです。 > > 効率面だけでなく、GRClosureHolderというデータ構造が不要になる分、 > > コードも若干シンプルになると思います。 > > なるほど。確かにシンプルですね。良いと思います。 > 須藤さんはどう思われますか？ 近いうちにリリースする予定はないですよね？ であれば、一度コミットしてもらってしばらく手元で動かしてみる というのはどうでしょう。 手元で使っていて何か問題がでたらrevertすればよいだけですし。 |
×
Want the latest updates on software, tech news, and AI?
Get latest updates about software, tech news, and AI from SourceForge directly in your inbox once a month.
Thanks for helping keep SourceForge clean.
X