From 3b1693bb3e7d3d00817c345e77d9999a84f6b17b Mon Sep 17 00:00:00 2001 From: Palle Girgensohn Date: Tue, 15 Dec 2020 20:50:10 +0000 Subject: Patch to mitigate a crash with LLVM JIT PR: 251192 (Submitted by Dmitry Marakasov) --- .../files/patch-disable-llvm-jit-inlining-with-tls | 24 ++++++++++++++++++++++ 1 file changed, 24 insertions(+) create mode 100644 databases/postgresql13-server/files/patch-disable-llvm-jit-inlining-with-tls (limited to 'databases/postgresql13-server/files/patch-disable-llvm-jit-inlining-with-tls') diff --git a/databases/postgresql13-server/files/patch-disable-llvm-jit-inlining-with-tls b/databases/postgresql13-server/files/patch-disable-llvm-jit-inlining-with-tls new file mode 100644 index 000000000000..02686061ce99 --- /dev/null +++ b/databases/postgresql13-server/files/patch-disable-llvm-jit-inlining-with-tls @@ -0,0 +1,24 @@ +Do not inline functions which access TLS in LLVM JIT, as +this leads to crashes with unsupported relocation error + +diff --git src/backend/jit/llvm/llvmjit_inline.cpp src/backend/jit/llvm/llvmjit_inline.cpp +index 2617a46..a063edb 100644 +--- src/backend/jit/llvm/llvmjit_inline.cpp ++++ src/backend/jit/llvm/llvmjit_inline.cpp +@@ -608,6 +608,16 @@ function_inlinable(llvm::Function &F, + if (rv->materialize()) + elog(FATAL, "failed to materialize metadata"); + ++ /* ++ * Don't inline functions with thread-local variables until ++ * related crashes are investigated (see BUG #16696) ++ */ ++ if (rv->isThreadLocal()) { ++ ilog(DEBUG1, "cannot inline %s due to thread-local variable %s", ++ F.getName().data(), rv->getName().data()); ++ return false; ++ } ++ + /* + * Never want to inline externally visible vars, cheap enough to + * reference. -- cgit v1.2.3