kotovalexarian-likes-github/mini_racer
1
0
Fork 0
mirror of https://github.com/rubyjs/mini_racer synced 2023-03-27 23:21:28 -04:00
Code Releases Activity

use C++11 atomics to check for RubyVM exit

Browse source 
There's no need to deal with pthread_rwlock for a single
variable since we already rely on C++11 atomics elsewhere in our
code.
This commit is contained in:
Sam Saffron 2022-02-11 09:28:55 +11:00
parent 4487f11202
commit b8638e8899
No known key found for this signature in database
GPG key ID: B9606168D2FFD9F5
1 changed files with 5 additions and 24 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

29
ext/mini_racer_extension/mini_racer_extension.cc
View file

@ -305,8 +305,7 @@ static std::unique_ptr<Platform> current_platform = NULL;
static std::mutex platform_lock;
static pthread_attr_t *thread_attr_p;
static pthread_rwlock_t exit_lock = PTHREAD_RWLOCK_INITIALIZER;
static bool ruby_exiting = false; // guarded by exit_lock
static std::atomic_int ruby_exiting(0);
static bool single_threaded = false;
static void mark_context(void *);
@ -1477,24 +1476,15 @@ static void free_context_raw(void *arg) {
}
static void *free_context_thr(void* arg) {
if (pthread_rwlock_tryrdlock(&exit_lock) != 0) {
return NULL;
if (ruby_exiting.load() == 0) {
free_context_raw(arg);
xfree(arg);
}
if (ruby_exiting) {
return NULL;
}
free_context_raw(arg);
xfree(arg);
pthread_rwlock_unlock(&exit_lock);
return NULL;
}
// destroys everything except freeing the ContextInfo struct (see deallocate())
static void free_context(ContextInfo* context_info) {
IsolateInfo* isolate_info = context_info->isolate_info;
if (isolate_info && isolate_info->refs() > 1) {
@ -1870,12 +1860,7 @@ static VALUE rb_context_create_isolate_value(VALUE self) {
static void set_ruby_exiting(VALUE value) {
(void)value;
int res = pthread_rwlock_wrlock(&exit_lock);
ruby_exiting = true;
if (res == 0) {
pthread_rwlock_unlock(&exit_lock);
}
ruby_exiting.store(1);
}
extern "C" {
@ -1941,9 +1926,5 @@ extern "C" {
thread_attr_p = &attr;
}
}
auto on_fork_for_child = []() {
exit_lock = PTHREAD_RWLOCK_INITIALIZER;
};
pthread_atfork(nullptr, nullptr, on_fork_for_child);
}
}