pg.ddx.io  pgsql-bugs@postgresql.org mailing list archive  

help / color / mirror / Atom feed
From: Dmitry Marakasov <amdmi3@amdmi3.ru>
To: Thomas Munro <thomas.munro@gmail.com>
Cc: Andres Freund <andres@anarazel.de>
Cc: Palle <girgen@pingpong.net>
Cc: pgsql-bugs@postgresql.org
Subject: Re: BUG #16696: Backend crash in llvmjit
Date: Thu, 22 Jul 2021 17:06:34 +0300
Message-ID: <YPl6htNahh/n5Mvj@hades.panopticon> (raw)
In-Reply-To: <CA+hUKGK6J+Zsa1j-pjKAJmqRq6=E55Tr2EX_WoZ2j0RNYgXo-g@mail.gmail.com>
References: <16696-29d944a33801fbfe@postgresql.org>
	<20201102214509.GV30304@hades.panopticon>
	<7463da2f-80f2-43c4-81d4-c3bab1f040b4@www.fastmail.com>
	<20201104212015.GA30304@hades.panopticon>
	<20201110054802.bjmjkfv4gzj3nkbe@alap3.anarazel.de>
	<20201116004529.GC5149@hades.panopticon>
	<1621435742019-0.post@n3.nabble.com>
	<CA+hUKG+LaEwA0KwDgAjsuKy2yT+q2coNVh2hFfdQxsgAA-C+zA@mail.gmail.com>
	<20210721165927.3x6cazq3hsni6m7r@alap3.anarazel.de>
	<CA+hUKGK6J+Zsa1j-pjKAJmqRq6=E55Tr2EX_WoZ2j0RNYgXo-g@mail.gmail.com>

* Thomas Munro (thomas.munro@gmail.com) wrote:

> On Thu, Jul 22, 2021 at 4:59 AM Andres Freund <andres@anarazel.de> wrote:
> > On 2021-07-18 17:13:50 +1200, Thomas Munro wrote:
> > > On Thu, May 20, 2021 at 2:49 AM Palle <girgen@pingpong.net> wrote:
> > > > This seems to still not be fixed. Is it planned for 14.0 release?
> > >
> > > I don't know enough about LLVM and TLS to have an opinion on the
> > > details, but I tested this and it works, and I think it should
> > > probably be back-patched based on the discussion so far (and hopefully
> > > in the future some way can be figured out to make it actually work?).
> 
> Done.
> 
> For the record, the original repro now causes PostgreSQL on FreeBSD
> (with INLINE_DEBUG defined) to say:
> 
> 2021-07-22 15:20:51.738 NZST [82982] DEBUG:  checking inlinability of boolin
> 2021-07-22 15:20:51.738 NZST [82982] DEBUG:  cannot inline boolin due
> to thread-local variable _ThreadRuneLocale
> 2021-07-22 15:20:51.738 NZST [82982] DEBUG:  had to skip inlining boolin
> 
> > FWIW, there's ongoing work in llvm to make the jit stuff handle TLS
> > correctly.
> 
> Cool.
> 
> Dmitry: Thanks very much for the report, reproducer, analysis and patch!

Superb, thanks a lot!

-- 
Dmitry Marakasov   .   55B5 0596 FF1E 8D84 5F56  9510 D35A 80DD F9D2 F77D
amdmi3@amdmi3.ru  ..:     https://github.com/AMDmi3  https://amdmi3.ru






view thread (14+ messages)

Message-ID: <YPl6htNahh/n5Mvj@hades.panopticon>
Permalink:  ../YPl6htNahh%2Fn5Mvj@hades.panopticon/
Also on:    postgresql.org/message-id/YPl6htNahh/n5Mvj@hades.panopticon

 · 

reply
Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Reply to all the recipients using the --to and --cc options:
  reply via email

  To: pgsql-bugs@postgresql.org
  Cc: amdmi3@amdmi3.ru, thomas.munro@gmail.com, andres@anarazel.de, girgen@pingpong.net
  Subject: Re: BUG #16696: Backend crash in llvmjit
  In-Reply-To: <YPl6htNahh/n5Mvj@hades.panopticon>

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

This inbox is served by DDX for PostgreSQL; see mirroring instructions
for how to clone and mirror all data and code used for this inbox