tree c6ec1b959262c5d914ee11398aafae2461f4a788
parent 8a4650c779b9b1f729b5f338c6d88adf6cd98a19
author Tom Stellard <tstellar@redhat.com> 1559855057 +0000
committer Tom Stellard <tstellar@redhat.com> 1559855057 +0000

Merging r357986:
------------------------------------------------------------------------
r357986 | jyknight | 2019-04-09 02:21:45 -0700 (Tue, 09 Apr 2019) | 22 lines

Fix siod's gc implementation when used with GLIBC.

Patch from LuoYuanke --

In the siod gc, it depends on setjmp(...) to get the value of callee
saved register value, and traverse those register to get the possible
local object pointer. Also it traverse current stack to get the
possible local object pointer. For setjmp(...) on glibc X86-64, the
rbp register (callee saved register) is NOT saved in the setjmp
buffer, so object that pointed by rbp is NOT considered as local
object variable and its memory is collected as garbage. This patch is
to use getcontext(...) to get more register value of current context
and traverse those register to protect object from garbage collection.

This bug is not easy to expose, because usually rbp has been saved in
stack when do garbage collection, so the object pointer can be scanned
from stack. However when compiler do some optimization on register
allocation or licm, the rbp live in gc_mark_and_sweep(...) and rbp is
pointing to an object. In such situation, siod failed to run test.scm
because object is collected as garbage unexpectedly.

Differential Revision: D60039
------------------------------------------------------------------------

llvm-svn: 362747