diff --git a/ChangeLog b/ChangeLog index d85115a7b7..d5149435b3 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,66 @@ +Tue Oct 29 20:01:58 2013 Koichi Sasada + + * add RUBY_TYPED_FREE_IMMEDIATELY to data types which only use + safe functions during garbage collection such as xfree(). + + On default, T_DATA objects are freed at same points as fianlizers. + This approach protects issues such as reported by [ruby-dev:35578]. + However, freeing T_DATA objects immediately helps heap usage. + + Most of T_DATA (in other words, most of dfree functions) are safe. + However, we turned off RUBY_TYPED_FREE_IMMEDIATELY by default + for safety. + + * cont.c: ditto. + + * dir.c: ditto. + + * encoding.c: ditto. + + * enumerator.c: ditto. + + * error.c: ditto. + + * file.c: ditto. + + * gc.c: ditto. + + * io.c: ditto. + + * iseq.c: ditto. + + * marshal.c: ditto. + + * parse.y: ditto. + + * proc.c: ditto. + + * process.c: ditto. + + * random.c: ditto. + + * thread.c: ditto. + + * time.c: ditto. + + * transcode.c: ditto. + + * variable.c: ditto. + + * vm.c: ditto. + + * vm_backtrace.c: ditto. + + * vm_trace.c: ditto. + + * ext/bigdecimal/bigdecimal.c: ditto. + + * ext/objspace/objspace.c: ditto. + + * ext/stringio/stringio.c: ditto. + + * ext/strscan/strscan.c: ditto. + Tue Oct 29 19:48:33 2013 Koichi Sasada * include/ruby/ruby.h: fix typo (FL_WB_PROTECT -> FL_WB_PROTECTED). diff --git a/cont.c b/cont.c index 5c1f10e571..b976056955 100644 --- a/cont.c +++ b/cont.c @@ -417,6 +417,7 @@ cont_save_machine_stack(rb_thread_t *th, rb_context_t *cont) static const rb_data_type_t cont_data_type = { "continuation", {cont_mark, cont_free, cont_memsize,}, + NULL, NULL, RUBY_TYPED_FREE_IMMEDIATELY }; static void @@ -1032,6 +1033,7 @@ rb_cont_call(int argc, VALUE *argv, VALUE contval) static const rb_data_type_t fiber_data_type = { "fiber", {fiber_mark, fiber_free, fiber_memsize,}, + NULL, NULL, RUBY_TYPED_FREE_IMMEDIATELY }; static VALUE diff --git a/dir.c b/dir.c index d16819011f..5e36270e64 100644 --- a/dir.c +++ b/dir.c @@ -381,6 +381,7 @@ dir_memsize(const void *ptr) static const rb_data_type_t dir_data_type = { "dir", {dir_mark, dir_free, dir_memsize,}, + NULL, NULL, RUBY_TYPED_FREE_IMMEDIATELY }; static VALUE dir_close(VALUE); diff --git a/encoding.c b/encoding.c index b073311c8d..fa6e5afaa7 100644 --- a/encoding.c +++ b/encoding.c @@ -71,6 +71,7 @@ enc_memsize(const void *p) static const rb_data_type_t encoding_data_type = { "encoding", {0, 0, enc_memsize,}, + NULL, NULL, RUBY_TYPED_FREE_IMMEDIATELY }; #define is_data_encoding(obj) (RTYPEDDATA_P(obj) && RTYPEDDATA_TYPE(obj) == &encoding_data_type) diff --git a/enumerator.c b/enumerator.c index 012a5b036a..1c4d5072e0 100644 --- a/enumerator.c +++ b/enumerator.c @@ -168,6 +168,7 @@ static const rb_data_type_t enumerator_data_type = { enumerator_free, enumerator_memsize, }, + NULL, NULL, RUBY_TYPED_FREE_IMMEDIATELY }; static struct enumerator * @@ -1076,6 +1077,7 @@ static const rb_data_type_t yielder_data_type = { yielder_free, yielder_memsize, }, + NULL, NULL, RUBY_TYPED_FREE_IMMEDIATELY }; static struct yielder * @@ -1181,6 +1183,7 @@ static const rb_data_type_t generator_data_type = { generator_free, generator_memsize, }, + NULL, NULL, RUBY_TYPED_FREE_IMMEDIATELY }; static struct generator * diff --git a/error.c b/error.c index ba9c720821..424999f0b1 100644 --- a/error.c +++ b/error.c @@ -1033,6 +1033,7 @@ static const rb_data_type_t name_err_mesg_data_type = { name_err_mesg_free, name_err_mesg_memsize, }, + NULL, NULL, RUBY_TYPED_FREE_IMMEDIATELY }; /* :nodoc: */ diff --git a/ext/bigdecimal/bigdecimal.c b/ext/bigdecimal/bigdecimal.c index 31e9be984f..e82ac38189 100644 --- a/ext/bigdecimal/bigdecimal.c +++ b/ext/bigdecimal/bigdecimal.c @@ -146,6 +146,7 @@ BigDecimal_memsize(const void *ptr) static const rb_data_type_t BigDecimal_data_type = { "BigDecimal", { 0, BigDecimal_delete, BigDecimal_memsize, }, + NULL, NULL, RUBY_TYPED_FREE_IMMEDIATELY }; static inline int diff --git a/ext/objspace/objspace.c b/ext/objspace/objspace.c index 2e7bd07029..b0df466ee3 100644 --- a/ext/objspace/objspace.c +++ b/ext/objspace/objspace.c @@ -643,6 +643,7 @@ iow_size(const void *ptr) static const rb_data_type_t iow_data_type = { "ObjectSpace::InternalObjectWrapper", {iow_mark, 0, iow_size,}, + NULL, NULL, RUBY_TYPED_FREE_IMMEDIATELY }; static VALUE rb_mInternalObjectWrapper; diff --git a/ext/stringio/stringio.c b/ext/stringio/stringio.c index ebf678bac8..3fef619de6 100644 --- a/ext/stringio/stringio.c +++ b/ext/stringio/stringio.c @@ -78,6 +78,7 @@ static const rb_data_type_t strio_data_type = { strio_free, strio_memsize, }, + NULL, NULL, RUBY_TYPED_FREE_IMMEDIATELY }; #define check_strio(self) ((struct StringIO*)rb_check_typeddata((self), &strio_data_type)) diff --git a/ext/strscan/strscan.c b/ext/strscan/strscan.c index 65b5ca7627..f020ba780d 100644 --- a/ext/strscan/strscan.c +++ b/ext/strscan/strscan.c @@ -190,7 +190,8 @@ strscan_memsize(const void *ptr) static const rb_data_type_t strscanner_type = { "StringScanner", - {strscan_mark, strscan_free, strscan_memsize} + {strscan_mark, strscan_free, strscan_memsize}, + NULL, NULL, RUBY_TYPED_FREE_IMMEDIATELY }; static VALUE diff --git a/file.c b/file.c index 3b0cd67ea6..8b74369d21 100644 --- a/file.c +++ b/file.c @@ -362,6 +362,7 @@ stat_memsize(const void *p) static const rb_data_type_t stat_data_type = { "stat", {NULL, RUBY_TYPED_DEFAULT_FREE, stat_memsize,}, + NULL, NULL, RUBY_TYPED_FREE_IMMEDIATELY }; static VALUE diff --git a/gc.c b/gc.c index e01a75ba4e..93b15882d1 100644 --- a/gc.c +++ b/gc.c @@ -1378,6 +1378,8 @@ obj_free(rb_objspace_t *objspace, VALUE obj) if (RTYPEDDATA_P(obj)) { free_immediately = RANY(obj)->as.typeddata.type->flags & RUBY_TYPED_FREE_IMMEDIATELY; RDATA(obj)->dfree = RANY(obj)->as.typeddata.type->function.dfree; + if (0 && free_immediately == 0) /* to expose non-free-immediate T_DATA */ + fprintf(stderr, "not immediate -> %s\n", RANY(obj)->as.typeddata.type->wrap_struct_name); } if (RANY(obj)->as.data.dfree == RUBY_DEFAULT_FREE) { xfree(DATA_PTR(obj)); @@ -5369,7 +5371,8 @@ static const rb_data_type_t weakmap_type = { wmap_mark, wmap_free, wmap_memsize, - } + }, + NULL, NULL, RUBY_TYPED_FREE_IMMEDIATELY }; static VALUE diff --git a/io.c b/io.c index 9ad7bff972..d136d88192 100644 --- a/io.c +++ b/io.c @@ -7583,6 +7583,7 @@ argf_memsize(const void *ptr) static const rb_data_type_t argf_type = { "ARGF", {argf_mark, argf_free, argf_memsize}, + NULL, NULL, RUBY_TYPED_FREE_IMMEDIATELY }; static inline void diff --git a/iseq.c b/iseq.c index 8545e6c881..504e96be1e 100644 --- a/iseq.c +++ b/iseq.c @@ -168,9 +168,8 @@ static const rb_data_type_t iseq_data_type = { iseq_free, iseq_memsize, }, /* functions */ - 0, /* parent */ - 0, /* data */ - FL_WB_PROTECTED /* flags */ + NULL, NULL, + RUBY_TYPED_FREE_IMMEDIATELY | RUBY_TYPED_WB_PROTECTED }; static VALUE diff --git a/marshal.c b/marshal.c index 4cba05d4e1..c1b6a79752 100644 --- a/marshal.c +++ b/marshal.c @@ -186,6 +186,7 @@ memsize_dump_arg(const void *ptr) static const rb_data_type_t dump_arg_data = { "dump_arg", {mark_dump_arg, free_dump_arg, memsize_dump_arg,}, + NULL, NULL, RUBY_TYPED_FREE_IMMEDIATELY }; static const char * @@ -1021,6 +1022,7 @@ memsize_load_arg(const void *ptr) static const rb_data_type_t load_arg_data = { "load_arg", {mark_load_arg, free_load_arg, memsize_load_arg,}, + NULL, NULL, RUBY_TYPED_FREE_IMMEDIATELY }; #define r_entry(v, arg) r_entry0((v), (arg)->data->num_entries, (arg)) diff --git a/parse.y b/parse.y index a2174ca4dd..d04286f8d9 100644 --- a/parse.y +++ b/parse.y @@ -11018,6 +11018,7 @@ rb_data_type_t parser_data_type = { parser_free, parser_memsize, }, + NULL, NULL, RUBY_TYPED_FREE_IMMEDIATELY }; #ifndef RIPPER diff --git a/proc.c b/proc.c index 01af8de774..f3e38e9e02 100644 --- a/proc.c +++ b/proc.c @@ -78,6 +78,7 @@ static const rb_data_type_t proc_data_type = { proc_free, proc_memsize, }, + NULL, NULL, RUBY_TYPED_FREE_IMMEDIATELY }; VALUE @@ -276,6 +277,7 @@ const rb_data_type_t ruby_binding_data_type = { binding_free, binding_memsize, }, + NULL, NULL, RUBY_TYPED_FREE_IMMEDIATELY }; static VALUE @@ -1104,6 +1106,7 @@ static const rb_data_type_t method_data_type = { bm_free, bm_memsize, }, + NULL, NULL, RUBY_TYPED_FREE_IMMEDIATELY }; VALUE diff --git a/process.c b/process.c index 4c645762ea..c12182abfe 100644 --- a/process.c +++ b/process.c @@ -1366,6 +1366,7 @@ memsize_exec_arg(const void *ptr) static const rb_data_type_t exec_arg_data_type = { "exec_arg", {mark_exec_arg, free_exec_arg, memsize_exec_arg}, + NULL, NULL, RUBY_TYPED_FREE_IMMEDIATELY }; #ifdef _WIN32 diff --git a/random.c b/random.c index 4026614044..41c20f3dee 100644 --- a/random.c +++ b/random.c @@ -335,6 +335,7 @@ static const rb_data_type_t random_data_type = { random_free, random_memsize, }, + NULL, NULL, RUBY_TYPED_FREE_IMMEDIATELY }; static rb_random_t * diff --git a/thread.c b/thread.c index e501f78675..ef869c6c28 100644 --- a/thread.c +++ b/thread.c @@ -3957,6 +3957,7 @@ thgroup_memsize(const void *ptr) static const rb_data_type_t thgroup_data_type = { "thgroup", {NULL, RUBY_TYPED_DEFAULT_FREE, thgroup_memsize,}, + NULL, NULL, RUBY_TYPED_FREE_IMMEDIATELY }; /* @@ -4195,6 +4196,7 @@ mutex_memsize(const void *ptr) static const rb_data_type_t mutex_data_type = { "mutex", {mutex_mark, mutex_free, mutex_memsize,}, + NULL, NULL, RUBY_TYPED_FREE_IMMEDIATELY }; VALUE @@ -4644,6 +4646,7 @@ thread_shield_mark(void *ptr) static const rb_data_type_t thread_shield_data_type = { "thread_shield", {thread_shield_mark, 0, 0,}, + NULL, NULL, RUBY_TYPED_FREE_IMMEDIATELY }; static VALUE diff --git a/time.c b/time.c index 2c83aa76fc..c8f15bfc97 100644 --- a/time.c +++ b/time.c @@ -1796,6 +1796,7 @@ time_memsize(const void *tobj) static const rb_data_type_t time_data_type = { "time", {time_mark, time_free, time_memsize,}, + NULL, NULL, RUBY_TYPED_FREE_IMMEDIATELY }; static VALUE diff --git a/transcode.c b/transcode.c index 95ef566b45..4a86f84d1b 100644 --- a/transcode.c +++ b/transcode.c @@ -2921,6 +2921,7 @@ econv_memsize(const void *ptr) static const rb_data_type_t econv_data_type = { "econv", {NULL, econv_free, econv_memsize,}, + NULL, NULL, RUBY_TYPED_FREE_IMMEDIATELY }; static VALUE diff --git a/variable.c b/variable.c index d07f9ff9dd..1c7bfb030f 100644 --- a/variable.c +++ b/variable.c @@ -1535,6 +1535,7 @@ autoload_memsize(const void *ptr) static const rb_data_type_t autoload_data_type = { "autoload", {autoload_mark, autoload_free, autoload_memsize,}, + NULL, NULL, RUBY_TYPED_FREE_IMMEDIATELY }; #define check_autoload_table(av) \ @@ -1585,6 +1586,7 @@ autoload_i_memsize(const void *ptr) static const rb_data_type_t autoload_data_i_type = { "autoload_i", {autoload_i_mark, autoload_i_free, autoload_i_memsize,}, + NULL, NULL, RUBY_TYPED_FREE_IMMEDIATELY }; #define check_autoload_data(av) \ diff --git a/vm.c b/vm.c index af2966589d..830e66cf34 100644 --- a/vm.c +++ b/vm.c @@ -305,6 +305,7 @@ env_memsize(const void *ptr) static const rb_data_type_t env_data_type = { "VM/env", {env_mark, env_free, env_memsize,}, + NULL, NULL, RUBY_TYPED_FREE_IMMEDIATELY }; static VALUE @@ -1686,6 +1687,7 @@ vm_memsize(const void *ptr) static const rb_data_type_t vm_data_type = { "VM", {rb_vm_mark, vm_free, vm_memsize,}, + NULL, NULL, RUBY_TYPED_FREE_IMMEDIATELY }; @@ -1963,6 +1965,7 @@ const rb_data_type_t ruby_threadptr_data_type = { thread_free, thread_memsize, }, + NULL, NULL, RUBY_TYPED_FREE_IMMEDIATELY }; VALUE diff --git a/vm_backtrace.c b/vm_backtrace.c index 541eda35cd..5b05b04050 100644 --- a/vm_backtrace.c +++ b/vm_backtrace.c @@ -112,6 +112,7 @@ location_memsize(const void *ptr) static const rb_data_type_t location_data_type = { "frame_info", {location_mark, location_free, location_memsize,}, + NULL, NULL, RUBY_TYPED_FREE_IMMEDIATELY }; static inline rb_backtrace_location_t * @@ -407,6 +408,7 @@ backtrace_memsize(const void *ptr) static const rb_data_type_t backtrace_data_type = { "backtrace", {backtrace_mark, backtrace_free, backtrace_memsize,}, + NULL, NULL, RUBY_TYPED_FREE_IMMEDIATELY }; int diff --git a/vm_trace.c b/vm_trace.c index 77af93b7bf..ce078ca162 100644 --- a/vm_trace.c +++ b/vm_trace.c @@ -630,6 +630,7 @@ tp_memsize(const void *ptr) static const rb_data_type_t tp_data_type = { "tracepoint", {tp_mark, RUBY_TYPED_NEVER_FREE, tp_memsize,}, + NULL, NULL, RUBY_TYPED_FREE_IMMEDIATELY }; static VALUE