Page MenuHomeFreeBSD
Differential D55189

frame.h: mark struct trapframe to be preserved
ClosedPublic
Actions

Authored by minsoochoo0122_proton.me on Feb 8 2026, 11:07 PM.
Tags
None
Referenced Files
F154767937: D55189.id171477.diff
Wed, Apr 29, 10:02 AM
F154767228: D55189.id171477.diff
Wed, Apr 29, 10:00 AM
F154722806: D55189.id171481.diff
Wed, Apr 29, 7:37 AM
F154720583: D55189.id171481.diff
Wed, Apr 29, 7:29 AM
F154703242: D55189.id171480.diff
Wed, Apr 29, 6:32 AM
F154700866: D55189.id171480.diff
Wed, Apr 29, 6:26 AM
Unknown Object (File)
Tue, Apr 28, 10:11 PM
Unknown Object (File)
Tue, Apr 28, 10:06 PM
View All 52 Files
Subscribers
emaste
imp
jhibbits
kib
riscv

Details

Reviewers
andrew
manu
Commits
rG2088e742013b: x86: Note that trapframe is used by kernel debuggers
Summary

There are programs that depend on this structure (e.g. kernel debuggers)
that breaks when the ABI changes.

Signed-off-by: Minsoo Choo <minsoochoo0122@proton.me>

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 70536
Build 67419: arc lint + arc unit

Event Timeline

minsoochoo0122_proton.me created this revision.Feb 8 2026, 11:07 PM
Herald added a reviewer: andrew. · View Herald TranscriptFeb 8 2026, 11:07 PM
Herald added a reviewer: andrew. · View Herald Transcript
Herald added a reviewer: manu. · View Herald Transcript
Herald added subscribers: riscv, jhibbits, emaste, imp. · View Herald Transcript
minsoochoo0122_proton.me requested review of this revision.Feb 8 2026, 11:07 PM
Harbormaster completed remote builds in B70531: Diff 171471.Feb 8 2026, 11:07 PM
minsoochoo0122_proton.me added a subscriber: kib.Feb 8 2026, 11:08 PM
kib added inline comments.Feb 9 2026, 12:08 AM
sys/x86/include/frame.h
50

It is enough to have one instance of this in x86/frame.h. Move one comment out of #ifdef, and remove another.

Then, the wording must be completely different, at least for x86, because the layout is defined by hardware, and its changes in somewhat compatible way.

So I think it would be fine to have sentence like 'struct trapframe is known to kernel debuggers.' But not more.

minsoochoo0122_proton.me updated this revision to Diff 171477.Feb 9 2026, 1:06 AM

cleanup x86

minsoochoo0122_proton.me marked an inline comment as done.Feb 9 2026, 1:06 AM
Harbormaster completed remote builds in B70534: Diff 171477.Feb 9 2026, 1:06 AM
kib added inline comments.Feb 9 2026, 1:09 AM
sys/x86/include/frame.h
41

Remove this blank line.

Then I suggest you to commit the x86 part.

minsoochoo0122_proton.me updated this revision to Diff 171478.Feb 9 2026, 1:21 AM

remove blank line and separate commit for x86

Harbormaster completed remote builds in B70535: Diff 171478.Feb 9 2026, 1:21 AM
minsoochoo0122_proton.me marked an inline comment as done.Feb 9 2026, 1:21 AM
kib added inline comments.Feb 9 2026, 1:31 AM
sys/x86/include/frame.h
47

No, move this line to the top-level comment, applicable to both i386 and amd64.

minsoochoo0122_proton.me updated this revision to Diff 171480.Feb 9 2026, 1:37 AM

update misplaced comment

Harbormaster completed remote builds in B70536: Diff 171480.Feb 9 2026, 1:37 AM
minsoochoo0122_proton.me marked an inline comment as done.Feb 9 2026, 1:37 AM
kib added a comment.Feb 9 2026, 1:41 AM

Please commit the x86 change.

minsoochoo0122_proton.me added a comment.Feb 9 2026, 1:45 AM
In D55189#1261892, @kib wrote:

Please commit the x86 change.

I did in 728adff455a3. Or do you mean submit only x86 change and discard others?

kib added a comment.Feb 9 2026, 1:51 AM
In D55189#1261893, @minsoochoo0122_proton.me wrote:
In D55189#1261892, @kib wrote:

Please commit the x86 change.

I did in 728adff455a3. Or do you mean submit only x86 change and discard others?

I do not see this revision in the FreeBSD git repo. I mean literally commit the sys/x86/include part, leaving other arches for their maintainers to review.

minsoochoo0122_proton.me added a comment.Feb 9 2026, 1:55 AM
In D55189#1261894, @kib wrote:
In D55189#1261893, @minsoochoo0122_proton.me wrote:
In D55189#1261892, @kib wrote:

Please commit the x86 change.

I did in 728adff455a3. Or do you mean submit only x86 change and discard others?

I do not see this revision in the FreeBSD git repo. I mean literally commit the sys/x86/include part, leaving other arches for their maintainers to review.

For 728adff455a3, I meant under commits tab in this revision. I don't understand what you mean by commit because I don't have commit bit. Sorry if I'm causing miscommunication here, but I'm confused.

kib added a comment.Feb 9 2026, 1:58 AM
In D55189#1261895, @minsoochoo0122_proton.me wrote:
In D55189#1261894, @kib wrote:
In D55189#1261893, @minsoochoo0122_proton.me wrote:
In D55189#1261892, @kib wrote:

Please commit the x86 change.

I did in 728adff455a3. Or do you mean submit only x86 change and discard others?

I do not see this revision in the FreeBSD git repo. I mean literally commit the sys/x86/include part, leaving other arches for their maintainers to review.

For 728adff455a3, I meant under commits tab in this revision. I don't understand what you mean by commit because I don't have commit bit. Sorry if I'm causing miscommunication here, but I'm confused.

Ok, send me the git-format patch for the x86 part, with the right author, commit message, and all the metadata. I will push it for you.

This revision was not accepted when it landed; it landed in state Needs Review.Feb 9 2026, 2:59 AM
Closed by commit rG2088e742013b: x86: Note that trapframe is used by kernel debuggers (authored by minsoochoo0122_proton.me, committed by kib). · Explain Why
This revision was automatically updated to reflect the committed changes.
kib added a commit: rG2088e742013b: x86: Note that trapframe is used by kernel debuggers.

Revision Contents
Changeset List

PathSize
sys/
arm/
include/
5 lines
arm64/
include/
4 lines
powerpc/
include/
6 lines
riscv/
include/
4 lines
x86/
include/
1 line

Diff 171480

View Options

sys/arm/include/frame.h

Loading...
View Options

sys/arm64/include/frame.h

Loading...
View Options

sys/powerpc/include/frame.h

Loading...
View Options

sys/riscv/include/frame.h

Loading...
View Options

sys/x86/include/frame.h

Loading...