[issue46968] Insufficient sigaltstack size used by CPython prevents extensions from using new ISA

STINNER Victor added the comment: Thanks for the fix Pablo. -- priority: release blocker -> ___ Python tracker ___ ___

[issue46968] Insufficient sigaltstack size used by CPython prevents extensions from using new ISA

Change by Pablo Galindo Salgado : -- resolution: -> fixed stage: patch review -> resolved status: open -> closed ___ Python tracker ___

[issue46968] Insufficient sigaltstack size used by CPython prevents extensions from using new ISA

Pablo Galindo Salgado added the comment: New changeset 6fd9737373f2bed03f409440b4fd50b9f8f121cb by Pablo Galindo Salgado in branch '3.10': [3.10] bpo-46968: Check for 'sys/auxv.h' in the configure script (GH-31961). (GH-31974)

[issue46968] Insufficient sigaltstack size used by CPython prevents extensions from using new ISA

Pablo Galindo Salgado added the comment: New changeset a12ef81231d65da5efbef4fa1434716270a19af6 by Pablo Galindo Salgado in branch '3.9': [3.9] bpo-46968: Check for 'sys/auxv.h' in the configure script (GH-31961). (GH-31975)

[issue46968] Insufficient sigaltstack size used by CPython prevents extensions from using new ISA

Change by Pablo Galindo Salgado : -- pull_requests: +30065 pull_request: https://github.com/python/cpython/pull/31975 ___ Python tracker ___

[issue46968] Insufficient sigaltstack size used by CPython prevents extensions from using new ISA

Change by Pablo Galindo Salgado : -- pull_requests: +30064 pull_request: https://github.com/python/cpython/pull/31974 ___ Python tracker ___

[issue46968] Insufficient sigaltstack size used by CPython prevents extensions from using new ISA

Change by László Kiss Kollár : -- nosy: +lkollar ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue46968] Insufficient sigaltstack size used by CPython prevents extensions from using new ISA

miss-islington added the comment: New changeset 8e3fde728f547f1d32bde8adf62b4c50bb877b9d by Pablo Galindo Salgado in branch 'main': bpo-46968: Check for 'sys/auxv.h' in the configure script (GH-31961) https://github.com/python/cpython/commit/8e3fde728f547f1d32bde8adf62b4c50bb877b9d

[issue46968] Insufficient sigaltstack size used by CPython prevents extensions from using new ISA

STINNER Victor added the comment: > Commit 393e2bf6bc6effbfe821f051a230978f0edd70df has broken CPython in RedHat 6 Too bad that we want to support RHEL 6 but have no buildbot for that. -- ___ Python tracker

[issue46968] Insufficient sigaltstack size used by CPython prevents extensions from using new ISA

Change by Pablo Galindo Salgado : -- pull_requests: +30049 stage: resolved -> patch review pull_request: https://github.com/python/cpython/pull/31961 ___ Python tracker ___

[issue46968] Insufficient sigaltstack size used by CPython prevents extensions from using new ISA

Pablo Galindo Salgado added the comment: This may be quite bad, because this means that 3.10 and 3.9 doesn't build in CentOS 6, which is used for manylinux2010 wheels -- ___ Python tracker

[issue46968] Insufficient sigaltstack size used by CPython prevents extensions from using new ISA

Change by Pablo Galindo Salgado : -- priority: normal -> release blocker ___ Python tracker ___ ___ Python-bugs-list mailing list

[issue46968] Insufficient sigaltstack size used by CPython prevents extensions from using new ISA

Pablo Galindo Salgado added the comment: The configure file is checking for "linux/auxvec.h" checking linux/auxvec.h usability... yes but the code is including "sys/auxvec.h" -- ___ Python tracker

[issue46968] Insufficient sigaltstack size used by CPython prevents extensions from using new ISA

Pablo Galindo Salgado added the comment: The code is assuming that if linux/auxvec.h then sys/auxv.h will be available, which is wrong. -- ___ Python tracker ___

[issue46968] Insufficient sigaltstack size used by CPython prevents extensions from using new ISA

Pablo Galindo Salgado added the comment: We may need to revert these commits and do another release sigh :( -- ___ Python tracker ___

[issue46968] Insufficient sigaltstack size used by CPython prevents extensions from using new ISA

Pablo Galindo Salgado added the comment: This is problematic because this has been backported to stable releases. -- nosy: +lukasz.langa ___ Python tracker ___

[issue46968] Insufficient sigaltstack size used by CPython prevents extensions from using new ISA

Pablo Galindo Salgado added the comment: Commit 393e2bf6bc6effbfe821f051a230978f0edd70df has broken CPython in RedHat 6: [2022-03-16T18:49:20.608Z] 2022/03/16 14:48:55 ERROR /tmp/python3.10-3.10.3-0/Modules/faulthandler.c:28:12: fatal error: sys/auxv.h: No such file or directory # include

[issue46968] Insufficient sigaltstack size used by CPython prevents extensions from using new ISA

STINNER Victor added the comment: > So where getauxval(AT_MINSIGSTKSZ) < SIGSTKSZ the merged changes actually > resulted in decrease of the allocated signal deliver stack. faulthandler stack is only used in the least likely case: on a fatal error. It should reduce its memory footprint, so

[issue46968] Insufficient sigaltstack size used by CPython prevents extensions from using new ISA

Oleksandr Pavlyk added the comment: So where getauxval(AT_MINSIGSTKSZ) < SIGSTKSZ the merged changes actually resulted in decrease of the allocated signal deliver stack. On Sapphire Rapids due to tile registers size we have getauxval(AT_MINSIGSTKSZ) > SIGSTKSZ. This is why the initial

[issue46968] Insufficient sigaltstack size used by CPython prevents extensions from using new ISA

STINNER Victor added the comment: On my x86-64 Fedora 35 with the CPU "Intel(R) Core(TM) i7-6820HQ CPU @ 2.70GHz", I get: * SIGSTKSZ = 8192 * getauxval(AT_MINSIGSTKSZ) = 2032 * faulthandler stack.ss_size = 10224 * os.sysconf('SC_MINSIGSTKSZ') = 2032 In C, sysconf(_SC_MINSIGSTKSZ) is similar

[issue46968] Insufficient sigaltstack size used by CPython prevents extensions from using new ISA

STINNER Victor added the comment: Thanks Oleksandr Pavlyk for your nice enhancement! I merged your change to 3.9, 3.10 and main branches! I also added os.sysconf('SC_MINSIGSTKSZ') to Python 3.11. -- resolution: -> fixed stage: patch review -> resolved status: open -> closed

[issue46968] Insufficient sigaltstack size used by CPython prevents extensions from using new ISA

STINNER Victor added the comment: New changeset ba2b7956fa3932769a5c0aa2575de5c8d7e7ba4b by Victor Stinner in branch '3.9': bpo-46968: Fix faulthandler for Sapphire Rapids Xeon (GH-31789) (GH-31831) https://github.com/python/cpython/commit/ba2b7956fa3932769a5c0aa2575de5c8d7e7ba4b

[issue46968] Insufficient sigaltstack size used by CPython prevents extensions from using new ISA

Change by STINNER Victor : -- pull_requests: +29928 pull_request: https://github.com/python/cpython/pull/31831 ___ Python tracker ___

[issue46968] Insufficient sigaltstack size used by CPython prevents extensions from using new ISA

STINNER Victor added the comment: New changeset 393e2bf6bc6effbfe821f051a230978f0edd70df by Victor Stinner in branch '3.10': bpo-46968: Fix faulthandler for Sapphire Rapids Xeon (GH-31789) (GH-31830) https://github.com/python/cpython/commit/393e2bf6bc6effbfe821f051a230978f0edd70df

[issue46968] Insufficient sigaltstack size used by CPython prevents extensions from using new ISA

STINNER Victor added the comment: The latest faulthandler bug on sigaltstack() was bpo-21131 about FPU state stored by "XSAVE" and a Linux kernel change to fix a security vulnerability: https://bugs.python.org/issue21131#msg349688 -- ___ Python

[issue46968] Insufficient sigaltstack size used by CPython prevents extensions from using new ISA

Change by STINNER Victor : -- pull_requests: +29927 pull_request: https://github.com/python/cpython/pull/31830 ___ Python tracker ___

[issue46968] Insufficient sigaltstack size used by CPython prevents extensions from using new ISA

STINNER Victor added the comment: New changeset 3b128c054885fe881c3b57a5978de3ea89c81a9c by Oleksandr Pavlyk in branch 'main': bpo-46968: Fix faulthandler for Sapphire Rapids Xeon (GH-31789) https://github.com/python/cpython/commit/3b128c054885fe881c3b57a5978de3ea89c81a9c --

[issue46968] Insufficient sigaltstack size used by CPython prevents extensions from using new ISA

STINNER Victor added the comment: New changeset dc374ac7b0fabaed49461a2044c220765f48d229 by Victor Stinner in branch 'main': bpo-46968: Add os.sysconf_names['SC_MINSIGSTKSZ'] (GH-31824) https://github.com/python/cpython/commit/dc374ac7b0fabaed49461a2044c220765f48d229 --

[issue46968] Insufficient sigaltstack size used by CPython prevents extensions from using new ISA

Change by STINNER Victor : -- pull_requests: +29921 pull_request: https://github.com/python/cpython/pull/31824 ___ Python tracker ___

[issue46968] Insufficient sigaltstack size used by CPython prevents extensions from using new ISA

Change by Kumar Aditya : -- nosy: +vstinner ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue46968] Insufficient sigaltstack size used by CPython prevents extensions from using new ISA

Change by Oleksandr Pavlyk : -- keywords: +patch pull_requests: +29893 stage: -> patch review pull_request: https://github.com/python/cpython/pull/31789 ___ Python tracker

[issue46968] Insufficient sigaltstack size used by CPython prevents extensions from using new ISA

New submission from Oleksandr Pavlyk : The following snippet illustrates request by an extension to use AMX registers: ``` // no increase sigaltstack size fix will not wark till we fix python void enable_amx_no_fix() { unsigned long bitmask; long rc; rc = syscall(SYS_arch_prctl,