Use uintptr_t instead of uint64_t for pointers in stack frames.
ClosedPublic
Actions

Authored by jhb on Nov 24 2020, 9:49 PM.

Details

Reviewers

manu
andrew
gnn

Commits

rS368245: Use uintptr_t instead of uint64_t for pointers in stack frames.

Diff Detail

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 35014
Build 31999: arc lint + arc unit

Event Timeline

jhb created this revision.Nov 24 2020, 9:49 PM

Herald added a reviewer: manu. · View Herald TranscriptNov 24 2020, 9:49 PM

Herald added subscribers: emaste, andrew. · View Herald Transcript

jhb requested review of this revision.Nov 24 2020, 9:49 PM

Harbormaster completed remote builds in B35014: Diff 79957.Nov 24 2020, 9:49 PM

jhb added a child revision: D27362: Check that the frame pointer is within the current stack..Nov 24 2020, 9:52 PM

This will be useful for Morello (CHERI aarch64) support where these will be capabilities instead of uint64_t for a pure-cap kernel.

andrew accepted this revision.Nov 25 2020, 4:10 PM

This revision is now accepted and ready to land.Nov 25 2020, 4:10 PM

Closed by commit rS368245: Use uintptr_t instead of uint64_t for pointers in stack frames. (authored by jhb). · Explain WhyDec 1 2020, 6:23 PM

This revision was automatically updated to reflect the committed changes.

jhb added a commit: rS368245: Use uintptr_t instead of uint64_t for pointers in stack frames..

Herald added a reviewer: gnn. · View Herald TranscriptDec 1 2020, 6:23 PM

Herald added a subscriber: imp. · View Herald Transcript

Revision Contents
Changeset List

Path

Size

sys/

arm64/

db_trace.c

14 lines

stack_machdep.c

6 lines

unwind.c

8 lines

include/

csan.h

6 lines

stack.h

6 lines

cddl/

dev/

dtrace/

aarch64/

dtrace_isa.c

8 lines

Diff 79957

View Options

sys/arm64/arm64/db_trace.c

View Options

sys/arm64/arm64/stack_machdep.c

View Options

sys/arm64/arm64/unwind.c

View Options

sys/arm64/include/csan.h

View Options

sys/arm64/include/stack.h

View Options

sys/cddl/dev/dtrace/aarch64/dtrace_isa.c

Use uintptr_t instead of uint64_t for pointers in stack frames.ClosedPublicActions

Details

Diff Detail

Event Timeline

Revision ContentsChangeset List

Diff 79957

sys/arm64/arm64/db_trace.c

sys/arm64/arm64/stack_machdep.c

sys/arm64/arm64/unwind.c

sys/arm64/include/csan.h

sys/arm64/include/stack.h

sys/cddl/dev/dtrace/aarch64/dtrace_isa.c

Use uintptr_t instead of uint64_t for pointers in stack frames.
ClosedPublic
Actions

Revision Contents
Changeset List