Page MenuHomeFreeBSD

Round TF_SIZE up to the stack alignment (16-bytes).
ClosedPublic

Authored by brooks on Sep 4 2020, 5:58 PM.

Details

Summary

The kernel adjusts the stack by TF_SIZE and the RISC-V ABI requires
that it remain 16-byte aligned.

Test Plan

works on CheriBSD

Diff Detail

Lint
Lint OK
Unit
No Unit Test Coverage
Build Status
Buildable 33394
Build 30693: arc lint + arc unit

Event Timeline

brooks requested review of this revision.Sep 4 2020, 5:58 PM
brooks created this revision.
mhorne added inline comments.
sys/riscv/riscv/genassym.c
90

It seems to me that STACKALIGNBYTES should really just be 16.

This revision is now accepted and ready to land.Sep 4 2020, 6:43 PM
sys/riscv/riscv/genassym.c
90

I like this documents what we're doing. I didn't use STACKALIGNBYTES in the D26327 because in CheriBSD our roundup2 isn't assembly friendly (we use a type-preserving builtin).

This revision was automatically updated to reflect the committed changes.