diff options
author | Koichi Sasada <ko1@atdot.net> | 2021-01-21 03:33:59 +0900 |
---|---|---|
committer | Koichi Sasada <ko1@atdot.net> | 2021-01-29 16:22:12 +0900 |
commit | 1ecda213668644d656eb0d60654737482447dd92 (patch) | |
tree | 2231cf25d2215ba1358c21c82e823fcae5203e34 /vm_callinfo.h | |
parent | 9241211538189a58b477bd55b539357617fd42ed (diff) | |
download | ruby-1ecda213668644d656eb0d60654737482447dd92.tar.gz |
global call-cache cache table for rb_funcall*
rb_funcall* (rb_funcall(), rb_funcallv(), ...) functions invokes
Ruby's method with given receiver. Ruby 2.7 introduced inline method
cache with static memory area. However, Ruby 3.0 reimplemented the
method cache data structures and the inline cache was removed.
Without inline cache, rb_funcall* searched methods everytime.
Most of cases per-Class Method Cache (pCMC) will be helped but
pCMC requires VM-wide locking and it hurts performance on
multi-Ractor execution, especially all Ractors calls methods
with rb_funcall*.
This patch introduced Global Call-Cache Cache Table (gccct) for
rb_funcall*. Call-Cache was introduced from Ruby 3.0 to manage
method cache entry atomically and gccct enables method-caching
without VM-wide locking. This table solves the performance issue
on multi-ractor execution.
[Bug #17497]
Ruby-level method invocation does not use gccct because it has
inline-method-cache and the table size is limited. Basically
rb_funcall* is not used frequently, so 1023 entries can be enough.
We will revisit the table size if it is not enough.
Diffstat (limited to 'vm_callinfo.h')
-rw-r--r-- | vm_callinfo.h | 12 |
1 files changed, 0 insertions, 12 deletions
diff --git a/vm_callinfo.h b/vm_callinfo.h index 7fc0a93806..c0716d6aa4 100644 --- a/vm_callinfo.h +++ b/vm_callinfo.h @@ -387,18 +387,6 @@ extern const struct rb_callcache *rb_vm_empty_cc(void); /* callcache: mutate */ static inline void -vm_cc_cme_set(const struct rb_callcache *cc, const struct rb_callable_method_entry_struct *cme) -{ - VM_ASSERT(IMEMO_TYPE_P(cc, imemo_callcache)); - VM_ASSERT(cc != vm_cc_empty()); - VM_ASSERT(vm_cc_cme(cc) != NULL); - VM_ASSERT(vm_cc_cme(cc)->called_id == cme->called_id); - VM_ASSERT(!vm_cc_markable(cc)); // only used for vm_eval.c - - *((const struct rb_callable_method_entry_struct **)&cc->cme_) = cme; -} - -static inline void vm_cc_call_set(const struct rb_callcache *cc, vm_call_handler call) { VM_ASSERT(IMEMO_TYPE_P(cc, imemo_callcache)); |