HomeFreeBSD

Fix an extremely subtle concurrency bug triggered by running on 32-thread

Description

Fix an extremely subtle concurrency bug triggered by running on 32-thread
POWER8 systems. During thread switch, there was a very small window when
the stack pointer was set to the stack pointer of the outgoing thread, but
after the lock on that thread had already been released.

If, during that window, the outgoing thread were rescheduled on another CPU
and begin execution and an exception were taken on the original CPU, the
trap handler and the outgoing thread would simultaneously execute on the same
stack, causing memory corruption. Fix this by making sure to release the
old thread only after cpu_switch() is done with its stack.

MFC after: 2 weeks
Sponsored by: FreeBSD Foundation

Details

Provenance
nwhitehornAuthored on
Parents
rS278428: Technically speaking, using one virtal processor area for all CPUs is a
Branches
Unknown
Tags
Unknown

Event Timeline