Skip to content
Permalink

Comparing changes

Choose two branches to see what’s changed or to start a new pull request. If you need to, you can also or learn more about diff comparisons.

Open a pull request

Create a new pull request by comparing changes across two branches. If you need to, you can also . Learn more about diff comparisons here.
base repository: rubinius/rubinius
base: aa7814191b3c
Choose a base ref
...
head repository: rubinius/rubinius
compare: 9088c10dd6f6
Choose a head ref
  • 2 commits
  • 5 files changed
  • 1 contributor

Commits on Jul 17, 2015

  1. Synchronize ThreadGroup#add. Closes #3467. 

    Ruby generally makes no guarantees about thread safety, however, many
accidental "guarantees" are assumed by Ruby programmers because of MRI's
global interpreter lock.

Rubinius makes internal operaions thread safe but generally does not make any
thread safety guarantees for common Ruby objects like Array or Hash instances.
It's perfectly possible to "corrupt" an Array by concurrently modifying it
without synchronization.

ThreadGroup presents an interesting case, however, because it is arguably a
system structure. The API of ThreadGroup is minimal, consisting of these
methods: add, enclose, enclosed?, list.

Of these, only add presents a thread safety issue from the perspective of a
potentially "corrupted" structure (ie the list of Threads in the group). It's
possible for one Thread to enclose the group and another to get 'false' when
calling #enclosed? "after" the other Thread called #enclose. That's not a case
we are concerned with. The case of calling #add (which will implicitly also
remove the Thread from another group if it is a member) should probably be
thread safe.

For this reason, we add synchronization around adding (and removing) a Thread
from a ThreadGroup.
    @brixen
    brixen committed Jul 17, 2015
    Copy the full SHA
    cddadf0 View commit details
    Browse the repository at this point in the history

  2. Switched JIT log warnings to info.

    @brixen
    brixen committed Jul 17, 2015
    Copy the full SHA
    9088c10 View commit details
    Browse the repository at this point in the history