Page MenuHomeFreeBSD
Differential D34213

powerpc: Add static asssert for context size
ClosedPublic
Actions

Authored by imp on Feb 8 2022, 10:04 PM.
Tags
None
Referenced Files
F105326599: D34213.diff
Sat, Dec 14, 11:09 PM
Unknown Object (File)
Wed, Dec 11, 10:58 AM
Unknown Object (File)
Sun, Dec 1, 12:15 PM
Unknown Object (File)
Sun, Dec 1, 10:10 AM
Unknown Object (File)
Fri, Nov 15, 1:05 AM
Unknown Object (File)
Nov 14 2024, 8:47 PM
Unknown Object (File)
Nov 7 2024, 4:29 PM
Unknown Object (File)
Oct 17 2024, 7:58 AM
View All 60 Files
Subscribers
jhibbits

Details

Reviewers
jhb
andrew
kib
manu
mhorne
emaste
Commits
rG6e48e160ae93: powerpc: Add static asssert for context size
Summary

Add a static assert for the mcontext_t and ucontext_t sizes. These are
de-facto ABI options and cannot change size ever. For powerpc64, also
add asserts for {u,m}mcontext32_t.

Sponsored by: Netflix

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

imp created this revision.Feb 8 2022, 10:04 PM
Herald added a subscriber: jhibbits. · View Herald TranscriptFeb 8 2022, 10:04 PM
imp requested review of this revision.Feb 8 2022, 10:04 PM
imp added a parent revision: D34212: amd64: Add static asssert for context size.
imp added a child revision: D34214: riscv: Add static asssert for context size.
Harbormaster completed remote builds in B44349: Diff 102522.Feb 8 2022, 10:05 PM
imp updated this revision to Diff 102535.Feb 9 2022, 12:00 AM

update per review

Harbormaster completed remote builds in B44360: Diff 102535.Feb 9 2022, 12:00 AM
andrew accepted this revision.Feb 9 2022, 2:38 PM
andrew added inline comments.
sys/powerpc/powerpc/exec_machdep.c
135

Shouldn't this message be ucontext_t ...?

144

And here

This revision is now accepted and ready to land.Feb 9 2022, 2:38 PM
imp updated this revision to Diff 102624.Feb 10 2022, 7:31 PM

update per review

This revision now requires review to proceed.Feb 10 2022, 7:31 PM
Harbormaster completed remote builds in B44397: Diff 102624.Feb 10 2022, 7:31 PM
This revision was not accepted when it landed; it landed in state Needs Review.Feb 10 2022, 9:34 PM
Closed by commit rG6e48e160ae93: powerpc: Add static asssert for context size (authored by imp). · Explain Why
This revision was automatically updated to reflect the committed changes.
imp added a commit: rG6e48e160ae93: powerpc: Add static asssert for context size.

Revision Contents
Changeset List

PathSize
sys/
powerpc/
powerpc/
15 lines

Diff 102642

View Options

sys/powerpc/powerpc/exec_machdep.c

Loading...