Re: ubsan fails on 32bit builds - Mailing list pgsql-hackers

From Tom Lane
Subject Re: ubsan fails on 32bit builds
Date
Msg-id 1955346.1668716104@sss.pgh.pa.us
Whole thread Raw
In response to Re: ubsan fails on 32bit builds  (Andres Freund <andres@anarazel.de>)
List pgsql-hackers
Andres Freund <andres@anarazel.de> writes:
> On 2022-11-17 14:20:47 -0500, Robert Haas wrote:
>> Not that I object to a targeted fix

> Should we backpatch this fix? Likely this doesn't cause active breakage
> outside of 32bit builds under ubsan, but that's not an unreasonable thing to
> want to do in the backbranches.

+1 for backpatching what you showed.

>> but it's been 10 years since
>> slist and dlist were committed, and we really ought to eliminate
>> SHM_QUEUE entirely in favor of using those.

> Agreed. I had started on a set of patches for some of the SHM_QUEUE uses, but
> somehow we ended up a bit stuck on the naming of dlist_delete variant that
> afterwards zeroes next/prev so we can replace SHMQueueIsDetached() uses.

Also +1, but of course for HEAD only.

            regards, tom lane



pgsql-hackers by date:

Previous
From: Andres Freund
Date:
Subject: Re: ubsan fails on 32bit builds
Next
From: Tomas Vondra
Date:
Subject: Re: SUBTRANS: Minimizing calls to SubTransSetParent()