Re: Detection of hadware feature => please do not use signal
Heikki Linnakangas <hlinnaka@iki.fi>
From: Heikki Linnakangas <hlinnaka@iki.fi>
To: Bastien Roucariès <rouca@debian.org>
Cc: pgsql-bugs@lists.postgresql.org, Tom Lane <tgl@sss.pgh.pa.us>
Date: 2024-10-31T18:01:41Z
Lists: pgsql-bugs
On 31/10/2024 19:48, Tom Lane wrote: > Heikki Linnakangas <hlinnaka@iki.fi> writes: >> On 31/10/2024 17:41, Bastien Roucariès wrote: >>> Could you use the canonical way under linux > >> We used to do that, but it was changed in commit 1c72ec6f. Looking at >> the discussion that led to it, it's not clear to me why we switched [1]. > > The complaint was precisely that it would not work under non-linux. > > Maybe we could do it one way on linux and the other way everywhere > else, but I do not find that attractive; it just makes it harder > to get full test coverage. Bastien, would you happen to know how other portable libraries that need to run on other operating system do this? We added the SIGILL probe back in 2018, there might be better options available now. -- Heikki Linnakangas Neon (https://neon.tech)
Commits
-
Update configure probes for CFLAGS needed for ARM CRC instructions.
- e266a0ed67d2 17.3 landed
- 851c6ff18fd3 15.11 landed
- 5980f1884fc9 18.0 landed
- 32057a0f791e 13.19 landed
- 2fc0199a5015 14.16 landed
- 1f4aadec41b8 16.7 landed
-
Support runtime CRC feature probing on NetBSD/ARM using sysctl().
- 4570b22666dd 18.0 landed
-
Use auxv to check for CRC32 instructions on ARM.
- aac831cafa6f 18.0 landed
-
Improve our method for probing the availability of ARM CRC instructions.
- 1c72ec6f4989 11.0 cited