Feature #14487: [PATCH] simplify altstack and enable reuse with thread cache

• Status: Closed
• Assignee: -
• Target version: -

[normalperson (Eric Wong)]

simplify altstack and enable reuse with thread cache

Instead of allocating and registering the altstack in different
places, do it together to reduce code and improve readability.
When thread cache is enabled, storing altstack in rb_thread_t
is wasteful and we may reuse altstack in the same pthread.

This also lets us clearly allow use of xmalloc to allow GC to
recover from ENOMEM.

This seems straighforward and will commit soonish if no objections.

(Still working on improving + benchmarking thread cache; hardware sucks :<)

[normalperson (Eric Wong)]

• Status changed from Open to Closed

Applied in changeset trunk|r63213.

---

simplify altstack and enable reuse with thread cache

Instead of allocating and registering the altstack in different
places, do it together to reduce code and improve readability.
When thread cache is enabled, storing altstack in rb_thread_t
is wasteful and we may reuse altstack in the same pthread.

This also lets us clearly allow use of xmalloc to allow GC to
recover from ENOMEM.

[ruby-core:85621] [Feature #14487]