Skip to content

Force patchelf to use 64KB page size on loongarch64#1107

Merged
jjhelmus merged 1 commit intoastral-sh:mainfrom
loong64:patchelf_alignment
May 1, 2026
Merged

Force patchelf to use 64KB page size on loongarch64#1107
jjhelmus merged 1 commit intoastral-sh:mainfrom
loong64:patchelf_alignment

Conversation

@wojiushixiaobai
Copy link
Copy Markdown
Contributor

This patch addresses a segmentation fault issue on LoongArch64 platforms caused by incorrect ELF segment alignment during the build process.

@wojiushixiaobai wojiushixiaobai force-pushed the patchelf_alignment branch 2 times, most recently from 4aca216 to b3eb6af Compare May 1, 2026 17:48
@jjhelmus
Copy link
Copy Markdown
Contributor

jjhelmus commented May 1, 2026

Is updating to patchelf 0.14.5 which includes this (and other) patches acceptable? It seems to be a better solution that patching the old version. #1115 does this update

@jjhelmus
Copy link
Copy Markdown
Contributor

jjhelmus commented May 1, 2026

Updating to patchelf 0.14.5 requires a compiler with C++17 support which the cross-compilers do not all support. Pulling in the one patch seems the best option at this time.

Upgrading to a newer patchelf should be done in the future after compiler which support C++17 are available from cross-compiling.

@jjhelmus jjhelmus added arch:all Select all architectures build:all platform:linux Specific to the Linux platform python:3.13 labels May 1, 2026
Comment thread cpython-unix/build-patchelf.sh Outdated
Signed-off-by: 吴小白 <296015668@qq.com>
@jjhelmus jjhelmus merged commit b8a0627 into astral-sh:main May 1, 2026
108 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

arch:all Select all architectures build:all platform:linux Specific to the Linux platform python:3.13

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Inconsistent page-size on loongarch64

2 participants