Page MenuHomeFreeBSD
Differential D52243

arm64: 32-bit compat is little endian, read the correct word
Needs ReviewPublic
Actions

Authored by jmg on Aug 29 2025, 5:24 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Nov 2, 6:17 AM
Unknown Object (File)
Sat, Nov 1, 1:10 PM
Unknown Object (File)
Sat, Nov 1, 3:27 AM
Unknown Object (File)
Tue, Oct 21, 2:06 AM
Unknown Object (File)
Tue, Oct 21, 2:06 AM
Unknown Object (File)
Mon, Oct 20, 4:26 PM
Unknown Object (File)
Sun, Oct 19, 4:25 PM
Unknown Object (File)
Fri, Oct 17, 7:38 AM
View All 23 Files
Subscribers
emaste
imp

Details

Reviewers
andrew
manu
Summary

Obtained from: Juniper Networks, Inc.

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Skipped
Unit
Tests Skipped
Build Status
Buildable 66643
Build 63526: arc lint + arc unit

Event Timeline

jmg created this revision.Aug 29 2025, 5:24 PM
Herald added a reviewer: andrew. · View Herald TranscriptAug 29 2025, 5:24 PM
Herald added a reviewer: andrew. · View Herald Transcript
Herald added a reviewer: manu. · View Herald Transcript
Herald added subscribers: emaste, imp. · View Herald Transcript
jmg requested review of this revision.Aug 29 2025, 5:24 PM
Harbormaster completed remote builds in B66643: Diff 161201.Aug 29 2025, 5:24 PM
jmg added a comment.EditedAug 29 2025, 5:29 PM

The attached program can be used to test/verify the problem/fix. Compile w/ -m32.

I did try to see if I could run a big endian 32-bit binary, but I couldn't figure out how to compile one, plus, we don't ship big endian libraries anyway, so definitely the old code is wrong.

syscalltest.c1 KBDownload

jmg attached a referenced file: F127258092: syscalltest.c. (Show Details)Aug 29 2025, 5:30 PM
andrew added a comment.Thu, Nov 13, 1:57 PM

It looks like this was fixed in 1ca09538d94273601dac08204c1d0b3ca9115864

Revision Contents
Changeset List

PathSize
sys/
arm64/
arm64/
2 lines

Diff 161201

View Options

sys/arm64/arm64/elf32_machdep.c

Loading...