Attachment 219740 Details for Bug 251192 – Patch

[patch] Patch

postgresql13-server.patch (text/plain), 1.59 KB, created by Dmitry Marakasov on 2020-11-16 19:18:42 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Dmitry Marakasov

Created: 2020-11-16 19:18:42 UTC

Size: 1.59 KB

patch

obsolete

>Index: Makefile
>===================================================================
>--- Makefile	(revision 553928)
>+++ Makefile	(working copy)
>@@ -5,7 +5,7 @@
> DISTVERSION?=	13.0
> # PORTREVISION must be ?= otherwise, all slave ports get this PORTREVISION and
> # not their own.  Probably best to keep it at ?=0 when reset here too.
>-PORTREVISION?=	4
>+PORTREVISION?=	5
> CATEGORIES?=	databases
> MASTER_SITES=	PGSQL/source/v${DISTVERSION}
> PKGNAMESUFFIX?= ${PORTVERSION:R}${COMPONENT}
>Index: files/patch-disable-llvm-jit-inlining-with-tls
>===================================================================
>--- files/patch-disable-llvm-jit-inlining-with-tls	(nonexistent)
>+++ files/patch-disable-llvm-jit-inlining-with-tls	(working copy)
>@@ -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.

Actions: View | Diff

Attachments on bug 251192: 219740