Page MenuHomeFreeBSD

riscv: Add static asssert for context size
ClosedPublic

Authored by imp on Feb 8 2022, 10:05 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Nov 8, 2:36 AM
Unknown Object (File)
Fri, Nov 1, 5:22 AM
Unknown Object (File)
Fri, Oct 25, 8:10 AM
Unknown Object (File)
Oct 3 2024, 2:49 PM
Unknown Object (File)
Sep 22 2024, 10:44 PM
Unknown Object (File)
Sep 22 2024, 7:08 PM
Unknown Object (File)
Sep 22 2024, 4:13 PM
Unknown Object (File)
Sep 21 2024, 6:10 AM
Subscribers
None

Details

Summary

Add a static assert for the mcontext_t and ucontext_t sizes. These are
de-factor ABI options and cannot change size ever.

Sponsored by: Netflix

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 44350
Build 41238: arc lint + arc unit

Event Timeline

imp requested review of this revision.Feb 8 2022, 10:05 PM
imp created this revision.

update, per review (for real this time)

IMO "has incorrect size" would be more clear than "bogusly changed size"

IMO "has incorrect size" would be more clear than "bogusly changed size"

I'll make this change globally across all the reviews, but I won't upload them again unless other changes are needed.
Reviewers should assume that change is made.

I'll make this change globally across all the reviews, but I won't upload them again unless other changes are needed.

👍

sys/riscv/riscv/exec_machdep.c
81

Shouldn't this message be ucontext_t ...?

This revision was not accepted when it landed; it landed in state Needs Review.Feb 10 2022, 9:35 PM
This revision was automatically updated to reflect the committed changes.