Re: Segfault in jit tuple deforming on arm64 due to LLVM issue

From: Thomas Munro <thomas(dot)munro(at)gmail(dot)com>
To: Anthonin Bonnefoy <anthonin(dot)bonnefoy(at)datadoghq(dot)com>
Cc: PostgreSQL Hackers <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: Segfault in jit tuple deforming on arm64 due to LLVM issue
Date: 2024-10-31 05:48:54
Message-ID: CA+hUKGJJuhfPGq_ZxtsgDJ6NXa5pB4XAP-ZhQGTNfmG9Ghj64Q@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Thu, Oct 17, 2024 at 10:41 PM Thomas Munro <thomas(dot)munro(at)gmail(dot)com> wrote:
> Thanks! I'm going to go ahead and commit this.

(Sorry for the delay, I got distracted by pgconf.eu.)

Today I set out to commit this patch, and wrote a proper commit
message to explain the code provenance, circumstances that led to it,
and the future conditions that will allow us to delete it in a few
years. Please see attached. In the process I struck a potential
snag:

https://llvm.org/LICENSE.txt
https://en.wikipedia.org/wiki/Apache_License

There are a couple of cases of dual-licensed code in our tree where we
explicitly used the Boost alternative instead of Apache 2. I plead
complete ignorance of this topic and defer to those who know about
such things: can we actually do this? I guess at a minimum a copy of
the licence would need to appear somewhere -- perhaps under
src/backend/jit/llvm? 4d says that if you modified the code you have
to say so prominently, but I did that at the top (and the changes are
completely trivial, just some #ifdef swizzling to massage some
function prototypes to suit older LLVMs). Otherwise I understand it
to be generally "BSD-like" (sans advert clause) but there is also some
stuff about patents, which surely aren't relevant to this in
practice... but I know that some projects object to it on principle
and because it smells like contract law, or something.... not an area
I am well informed about. Who should I be asking? (Naively, I
wondered: could there be some kind of fair use concept for
back-patching fixes to broken libraries that you're merely a user of
where you can be excused from the burdens of a distributor? Yeah
wishful thinking I'm sure.)

Attachment Content-Type Size
v7-0001-Monkey-patch-LLVM-code-to-fix-ARM-relocation-bug.patch text/x-patch 33.7 KB

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Bertrand Drouvot 2024-10-31 05:52:49 Re: define pg_structiszero(addr, s, r)
Previous Message Yushi Ogiwara 2024-10-31 05:18:09 Fix for Extra Parenthesis in pgbench progress message