Page MenuHomeFreeBSD
Differential D59649

trap_cfi(9): add assembly interface for amd64
Needs ReviewPublic
Actions

Authored by mchoo on Sun, Sep 13, 6:19 PM.
Tags
None
Referenced Files
F171855118: D59649.id186619.diff
Mon, Sep 14, 2:40 AM
F171854919: D59649.diff
Mon, Sep 14, 2:38 AM
F171836647: D59649.id186619.diff
Sun, Sep 13, 10:44 PM
Subscribers
imp

Details

Reviewers
jhb
kib
Summary

MFC after: 2 weeks
MFC to: stable/14, stable/15
Sponsored by: FreeBSD Foundation

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Skipped
Unit
Tests Skipped
Build Status
Buildable 76856
Build 73739: arc lint + arc unit

Event Timeline

mchoo created this revision.Sun, Sep 13, 6:19 PM
Herald added a subscriber: imp. · View Herald TranscriptSun, Sep 13, 6:19 PM
mchoo requested review of this revision.Sun, Sep 13, 6:19 PM
Harbormaster completed remote builds in B76856: Diff 186619.Sun, Sep 13, 6:19 PM
mchoo added a parent revision: D59648: trap_cfi(9): add assembly interface for arm64.Sun, Sep 13, 6:19 PM
mchoo added a child revision: D59650: trap_cfi(9): add assembly interface for arm.
mchoo added a reviewer: kib.Sun, Sep 13, 6:33 PM
mchoo edited the summary of this revision. (Show Details)Sun, Sep 13, 9:07 PM
kib added a comment.Mon, Sep 14, 2:50 AM

I do not quite understand what could be a subject for the review. It is standalone macro set, which I have no idea what supposed to do, and how are used.

sys/x86/include/frame.h
119

This comment is perhaps misplaced. Should it go somewhere where the 'trap frame CFI information' is applied?

Revision Contents
Changeset List

PathSize
sys/
amd64/
include/
81 lines
x86/
include/
2 lines

Diff 186619

View Options

sys/amd64/include/trap_cfi.h

Loading...
View Options

sys/x86/include/frame.h

Loading...