Skip to content

This patch fix #29815 that built on big-endian ppc64 failed.#29827

Closed
dannytsen wants to merge 1 commit intoopenssl:masterfrom
dannytsen:fix_29815
Closed

This patch fix #29815 that built on big-endian ppc64 failed.#29827
dannytsen wants to merge 1 commit intoopenssl:masterfrom
dannytsen:fix_29815

Conversation

@dannytsen
Copy link
Copy Markdown
Contributor

@dannytsen dannytsen commented Jan 28, 2026

This patch fix #29815 that built on big-endian ppc64 failed.
Removed .localentry directive to prevent mixing ELFv1 ABI and ELFv2 ABI directives.

Signed-off-by: Danny Tsen [email protected]

Checklist
  • documentation is added or updated
  • tests are added or updated

@dannytsen
This patch fix openssl#29815 that built on big-endian ppc64 failed.
afaee5c 
Removed \.localentry directive to prevent mixing ELFv1 ABI and ELFv2 ABI directives.

Signed-off-by: Danny Tsen <[email protected]>
@github-actions github-actions Bot added the severity: fips change The pull request changes FIPS provider sources label Jan 28, 2026
@paulidale paulidale added branch: master Applies to master branch approval: review pending This pull request needs review by a committer labels Jan 29, 2026
@beldmit
Copy link
Copy Markdown
Member

beldmit commented Jan 29, 2026

Which branches should it go to?

@esyr esyr added branch: 3.3 Applies to openssl-3.3 (EOL) branch: 3.4 Applies to openssl-3.4 branch: 3.5 Applies to openssl-3.5 branch: 3.6 Applies to openssl-3.6 labels Jan 29, 2026
@esyr
Copy link
Copy Markdown
Member

esyr commented Jan 29, 2026

Which branches should it go to?

Since it is a fix for a change introduced in [1], openssl-3.3..openssl-3.6, I believe. I've added the tags.

[1] #28990

@esyr
Copy link
Copy Markdown
Member

esyr commented Jan 29, 2026

master: Fixes: ebd690b1eb0d "ppc64le: Restore VSX registers in AES/GCM ppc64le encrypt/decrypt"
openssl-3.6: Fixes: 8f24a148e6f1 "ppc64le: Restore VSX registers in AES/GCM ppc64le encrypt/decrypt"
openssl-3.5: Fixes: 67d68449982d "ppc64le: Restore VSX registers in AES/GCM ppc64le encrypt/decrypt"
openssl-3.4: Fixes: 31a439deb08b "ppc64le: Restore VSX registers in AES/GCM ppc64le encrypt/decrypt"
openssl-3.3: Fixes: 0ac1b1a2b2a1 "ppc64le: Restore VSX registers in AES/GCM ppc64le encrypt/decrypt"

@slontis slontis added approval: done This pull request has the required number of approvals and removed approval: review pending This pull request needs review by a committer labels Feb 2, 2026
@openssl-machine openssl-machine added approval: ready to merge The 24 hour grace period has passed, ready to merge and removed approval: done This pull request has the required number of approvals labels Feb 3, 2026
@openssl-machine
Copy link
Copy Markdown
Collaborator

openssl-machine commented Feb 3, 2026

This pull request is ready to merge

@t8m t8m added triaged: bug The issue/pr is/fixes a bug tests: exempted The PR is exempt from requirements for testing labels Feb 3, 2026
openssl-machine pushed a commit that referenced this pull request Feb 3, 2026
@dannytsen @t8m
aes-gcm-ppc.pl: Removed .localentry directive
7936b4c 
Otherwise there is mixing of  ELFv1 ABI and ELFv2 ABI directives
and PPC64 big endian builds fail.

Fixes #29815

Signed-off-by: Danny Tsen <[email protected]>

Reviewed-by: Paul Dale <[email protected]>
Reviewed-by: Shane Lontis <[email protected]>
Reviewed-by: Tomas Mraz <[email protected]>
MergeDate: Tue Feb  3 08:39:50 2026
(Merged from #29827)

(cherry picked from commit 5aaa7e5)
openssl-machine pushed a commit that referenced this pull request Feb 3, 2026
@dannytsen @t8m
aes-gcm-ppc.pl: Removed .localentry directive
5aaa7e5 
Otherwise there is mixing of  ELFv1 ABI and ELFv2 ABI directives
and PPC64 big endian builds fail.

Fixes #29815

Signed-off-by: Danny Tsen <[email protected]>

Reviewed-by: Paul Dale <[email protected]>
Reviewed-by: Shane Lontis <[email protected]>
Reviewed-by: Tomas Mraz <[email protected]>
MergeDate: Tue Feb  3 08:39:50 2026
(Merged from #29827)
openssl-machine pushed a commit that referenced this pull request Feb 3, 2026
@dannytsen @t8m
aes-gcm-ppc.pl: Removed .localentry directive
40735fe 
Otherwise there is mixing of  ELFv1 ABI and ELFv2 ABI directives
and PPC64 big endian builds fail.

Fixes #29815

Signed-off-by: Danny Tsen <[email protected]>

Reviewed-by: Paul Dale <[email protected]>
Reviewed-by: Shane Lontis <[email protected]>
Reviewed-by: Tomas Mraz <[email protected]>
MergeDate: Tue Feb  3 08:39:50 2026
(Merged from #29827)

(cherry picked from commit 5aaa7e5)
@t8m
Copy link
Copy Markdown
Member

t8m commented Feb 3, 2026

Merged to the master, 3.6, 3.5, 3.4 and 3.3 branches with reworded commit message. Thank you for your contribution.

@t8m t8m closed this Feb 3, 2026
openssl-machine pushed a commit that referenced this pull request Feb 3, 2026
@dannytsen @t8m
aes-gcm-ppc.pl: Removed .localentry directive
be0f0f1 
Otherwise there is mixing of  ELFv1 ABI and ELFv2 ABI directives
and PPC64 big endian builds fail.

Fixes #29815

Signed-off-by: Danny Tsen <[email protected]>

Reviewed-by: Paul Dale <[email protected]>
Reviewed-by: Shane Lontis <[email protected]>
Reviewed-by: Tomas Mraz <[email protected]>
MergeDate: Tue Feb  3 08:39:50 2026
(Merged from #29827)

(cherry picked from commit 5aaa7e5)
openssl-machine pushed a commit that referenced this pull request Feb 3, 2026
@dannytsen @t8m
aes-gcm-ppc.pl: Removed .localentry directive
8594fda 
Otherwise there is mixing of  ELFv1 ABI and ELFv2 ABI directives
and PPC64 big endian builds fail.

Fixes #29815

Signed-off-by: Danny Tsen <[email protected]>

Reviewed-by: Paul Dale <[email protected]>
Reviewed-by: Shane Lontis <[email protected]>
Reviewed-by: Tomas Mraz <[email protected]>
MergeDate: Tue Feb  3 08:39:50 2026
(Merged from #29827)

(cherry picked from commit 5aaa7e5)
weihanglo added a commit to weihanglo/cargo that referenced this pull request Feb 8, 2026
@weihanglo
chore: pin openssl-src to 300.5.4
616bd13 
OpenSSL 3.5.5 has an ELFv1/ELFv2 ABI mismatch
that breaks powerpc64-unknown-linux-gnu builds.

The fix is merged in openssl/openssl#29827 but not yet released.

See

* openssl/openssl#29815
* rust-lang/rust#152240 (comment)
@weihanglo weihanglo mentioned this pull request Feb 8, 2026
Merged
github-merge-queue Bot pushed a commit to rust-lang/cargo that referenced this pull request Feb 8, 2026
@ehuss
chore: pin openssl-src to 300.5.4 (#16601)
85a5ded 
OpenSSL 3.5.5 has an ELFv1/ELFv2 ABI mismatch
that breaks powerpc64-unknown-linux-gnu builds.

The fix is merged in openssl/openssl#29827 but not yet released.

See

* openssl/openssl#29815
* rust-lang/rust#152240 (comment)
bob-beck pushed a commit to bob-beck/openssl that referenced this pull request Feb 24, 2026
@dannytsen @bob-beck
aes-gcm-ppc.pl: Removed .localentry directive
a906b07 
Otherwise there is mixing of  ELFv1 ABI and ELFv2 ABI directives
and PPC64 big endian builds fail.

Fixes openssl#29815

Signed-off-by: Danny Tsen <[email protected]>

Reviewed-by: Paul Dale <[email protected]>
Reviewed-by: Shane Lontis <[email protected]>
Reviewed-by: Tomas Mraz <[email protected]>
MergeDate: Tue Feb  3 08:39:50 2026
(Merged from openssl#29827)
raspbian-autopush pushed a commit to raspbian-packages/openssl that referenced this pull request Mar 27, 2026
@dannytsen @sebastianas
aes-gcm-ppc.pl: Removed .localentry directive
1cb8778 
Otherwise there is mixing of  ELFv1 ABI and ELFv2 ABI directives
and PPC64 big endian builds fail.

Fixes #29815

Signed-off-by: Danny Tsen <[email protected]>

Reviewed-by: Paul Dale <[email protected]>
Reviewed-by: Shane Lontis <[email protected]>
Reviewed-by: Tomas Mraz <[email protected]>
MergeDate: Tue Feb  3 08:39:50 2026
(Merged from openssl/openssl#29827)

Gbp-Pq: Name aes-gcm-ppc.pl-Removed-.localentry-directive.patch
@philiptaron philiptaron mentioned this pull request Apr 12, 2026
Merged
13 tasks
ehuss added a commit to rust-lang/cargo that referenced this pull request May 1, 2026
@ehuss
Unpin opeensl-src
081aa5a 
300.6.0 contains OpenSSL 3.6.2 which contains the fix
openssl/openssl#29827 that should resolve the
ppc64 issue.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@t8m t8m t8m approved these changes

@paulidale paulidale paulidale approved these changes

@slontis slontis slontis approved these changes

Assignees

No one assigned

Labels

approval: ready to merge The 24 hour grace period has passed, ready to merge branch: master Applies to master branch branch: 3.3 Applies to openssl-3.3 (EOL) branch: 3.4 Applies to openssl-3.4 branch: 3.5 Applies to openssl-3.5 branch: 3.6 Applies to openssl-3.6 severity: fips change The pull request changes FIPS provider sources tests: exempted The PR is exempt from requirements for testing triaged: bug The issue/pr is/fixes a bug

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Fails to build on big-endian ppc64 by trying to link ELFv1 to ELFv2 code

7 participants

@dannytsen @beldmit @esyr @openssl-machine @t8m @paulidale @slontis