Bug #7474

I am getting undefined symbol: rb_enable_interrupt when running event machine

Added by Sam Saffron over 2 years ago. Updated about 2 years ago.

[ruby-core:50352]
Status:Rejected
Priority:Normal
Assignee:Motohiro KOSAKI
ruby -v:2.0 Backport:

Description

I would like to cross report

https://github.com/eventmachine/eventmachine/issues/389

Clearly this breakage could be "on purpose" not sure who should be fixing it.

History

#1 Updated by Nobuyoshi Nakada about 2 years ago

  • Category set to core
  • Status changed from Open to Assigned
  • Assignee set to Motohiro KOSAKI

It's been static since r32510.
I'm not sure if it is a part of public API.

#2 Updated by Motohiro KOSAKI about 2 years ago

  • Status changed from Assigned to Feedback

It's been static since r32510.
I'm not sure if it is a part of public API.

vm_core.h is definitely internal api place.
Moreover rb_disable_interrupt() is wrong api and we may be removed later.
There are two problems:

1)

rb_disable_interrupt()
..
rb_enable_interrupt()

is expand to

sigfillset(&mask);
pthread_sigmask(SIG_SETMASK, &mask, NULL);

...

sigemptyset(&mask);
pthread_sigmask(SIG_SETMASK, &mask, NULL);

See? rb_enable_interrupt() ignore old sigmask and set signal mask to empty.

2) thread unsafe

pthread_sigmask() only change current thread sigmask. then other thread still be able to
be interrupted by signal. then, this api only work single thread context. MRI only use this
api when process creation time.

So, I have a question. Why do event machine need rb_disable_interrupt()? If I understand correctly,
event machine don't have single thread guarantee.

#3 Updated by Motohiro KOSAKI about 2 years ago

  • Status changed from Feedback to Rejected

No response. I'll close this then.

Also available in: Atom PDF