Maintain a stack alignment of 16-bytes.
ClosedPublic
Actions

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

Tags

None

Referenced Files

	F167096962: D26327.id76654.diff
	Wed, Aug 19, 1:47 AM

	F167050904: D26327.diff
	Tue, Aug 18, 8:00 PM

	F167037214: D26327.diff
	Tue, Aug 18, 5:03 PM

	Unknown Object (File)
	Tue, Aug 11, 10:12 AM

	Unknown Object (File)
	Tue, Aug 11, 8:54 AM

	Unknown Object (File)
	Sat, Jul 25, 5:59 AM

	Unknown Object (File)
	Tue, Jul 21, 4:46 PM

	Unknown Object (File)
	Jul 19 2026, 2:11 PM

View All Files

Subscribers

bdragon

imp

Details

Reviewers

mhorne
br
jrtc27

Commits

rS365342: Maintain a stack alignment of 16-bytes.

Summary

This violated the RISC-V 64-bit ABI. On CheriBSD this manifested as
a trap when attempting to store 16-byte aligned types while zeroing
the stack.

Test Plan

works in CheriBSD

Diff Detail

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 33393
Build 30692: arc lint + arc unit

Event Timeline

brooks created this revision.Sep 4 2020, 5:55 PM

Herald added a subscriber: bdragon. · View Herald TranscriptSep 4 2020, 5:55 PM

brooks requested review of this revision.Sep 4 2020, 5:55 PM

Harbormaster completed remote builds in B33393: Diff 76654.Sep 4 2020, 5:55 PM

I'm not very familiar with rtld, but this seems correct.

This revision is now accepted and ready to land.Sep 4 2020, 6:47 PM

br accepted this revision.Sep 4 2020, 8:20 PM

Closed by commit rS365342: Maintain a stack alignment of 16-bytes. (authored by brooks). · Explain WhySep 4 2020, 9:46 PM

This revision was automatically updated to reflect the committed changes.

brooks added a commit: rS365342: Maintain a stack alignment of 16-bytes..

Herald added a subscriber: imp. · View Herald TranscriptSep 4 2020, 9:46 PM

brooks mentioned this in D26328: Round TF_SIZE up to the stack alignment (16-bytes)..Sep 4 2020, 9:55 PM

Revision Contents
Changeset List

Path

Size

libexec/

rtld-elf/

riscv/

rtld_start.S

5 lines

Diff 76654

View Options

libexec/rtld-elf/riscv/rtld_start.S

Maintain a stack alignment of 16-bytes.ClosedPublicActions