Page MenuHomeFreeBSD
Differential D44071

kernel: Move the debug stuff into a common script
ClosedPublic
Actions

Authored by imp on Feb 24 2024, 8:45 PM.
Tags
None
Referenced Files
F135806594: D44071.id141622.diff
Thu, Nov 13, 1:49 AM
Unknown Object (File)
Wed, Nov 12, 3:15 AM
Unknown Object (File)
Tue, Nov 11, 9:28 AM
Unknown Object (File)
Tue, Nov 4, 2:01 PM
Unknown Object (File)
Tue, Nov 4, 2:01 PM
Unknown Object (File)
Tue, Nov 4, 10:21 AM
Unknown Object (File)
Tue, Nov 4, 6:39 AM
Unknown Object (File)
Tue, Nov 4, 6:39 AM
View All Files
Subscribers
emaste

Details

Reviewers
markj
kib
jrtc27
kevans
andrew
emaste
Commits
rG37d6d682af59: kernel: Move the debug stuff into a common script
Summary

Sponsored by: Netflix

Diff Detail

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

Event Timeline

imp created this revision.Feb 24 2024, 8:45 PM
Herald added a reviewer: andrew. · View Herald TranscriptFeb 24 2024, 8:45 PM
Herald added a reviewer: andrew. · View Herald Transcript
Herald added a subscriber: emaste. · View Herald Transcript
imp requested review of this revision.Feb 24 2024, 8:45 PM
Harbormaster completed remote builds in B56226: Diff 134956.Feb 24 2024, 8:45 PM
imp added a parent revision: D44070: kernel: Bring DWARF3 symbols to all kernels.Feb 24 2024, 8:45 PM
imp added a child revision: D44072: kernel: Add new dwarf symbols.
imp added a reviewer: emaste.Feb 24 2024, 8:47 PM
imp updated this revision to Diff 134958.Feb 24 2024, 9:05 PM

Rebase after dropping the addition of the dwarf 3 symbols. Just note it here
since it has the same effect in one fewer changes.

Harbormaster completed remote builds in B56228: Diff 134958.Feb 24 2024, 9:05 PM
imp edited parent revisions, added: D44069: kernel: Sync ldscript files wrt gnu.attributes and .note.GNU-stack; removed: D44070: kernel: Bring DWARF3 symbols to all kernels.Feb 24 2024, 9:08 PM
kib added a comment.Feb 24 2024, 9:16 PM

I would call the script debuginfo.ldscript instead of debug.

sys/conf/debug.ldscript
9

Is this still relevant?

imp added inline comments.Feb 24 2024, 9:18 PM
sys/conf/debug.ldscript
9

Likely not.

imp added inline comments.Feb 24 2024, 9:22 PM
sys/conf/debug.ldscript
9

Likely not.

Since we have them in the vdso_*.ldscript files, maybe we should remove them from there too?

kib added inline comments.Feb 24 2024, 9:29 PM
sys/conf/debug.ldscript
9

Could you point me specifically at the place where in vdso scripts we include stabs?

imp added inline comments.Feb 24 2024, 9:32 PM
sys/conf/debug.ldscript
9

No. We have .comment there

imp updated this revision to Diff 134964.Feb 25 2024, 12:53 AM

Drop .comment...
Maybe it should be discard?

Harbormaster completed remote builds in B56232: Diff 134964.Feb 25 2024, 12:53 AM
imp added inline comments.Feb 25 2024, 2:01 AM
sys/conf/debug.ldscript
9

Ah, we explicitly discard it there... my mistake.

imp updated this revision to Diff 135227.Feb 29 2024, 4:28 PM

Update script

Harbormaster completed remote builds in B56350: Diff 135227.Feb 29 2024, 4:28 PM
kib added a comment.Mar 1 2024, 7:31 AM

I still think that the script should be renamed to 'debuginfo'.

imp added a comment.Mar 1 2024, 3:25 PM
In D44071#1007601, @kib wrote:

I still think that the script should be renamed to 'debuginfo'.

Ah yes. That detail slipped through the cracks. Will correct and respin

imp updated this revision to Diff 141614.Jul 31 2024, 9:53 PM

rename, per kib

Harbormaster completed remote builds in B58906: Diff 141614.Jul 31 2024, 9:53 PM
This revision was not accepted when it landed; it landed in state Needs Review.Aug 1 2024, 4:24 AM
Closed by commit rG37d6d682af59: kernel: Move the debug stuff into a common script (authored by imp). · Explain Why
This revision was automatically updated to reflect the committed changes.
imp added a commit: rG37d6d682af59: kernel: Move the debug stuff into a common script.

Revision Contents
Changeset List

Diff 134956

View Options

sys/conf/debug.ldscript

Loading...
View Options

sys/conf/kern.pre.mk

Loading...
View Options

sys/conf/ldscript.amd64

Loading...
View Options

sys/conf/ldscript.arm

Loading...
View Options

sys/conf/ldscript.arm64

Loading...
View Options

sys/conf/ldscript.i386

Loading...
View Options

sys/conf/ldscript.powerpc

Loading...
View Options

sys/conf/ldscript.powerpc64

Loading...
View Options

sys/conf/ldscript.powerpc64le

Loading...
View Options

sys/conf/ldscript.powerpcspe

Loading...
View Options

sys/conf/ldscript.riscv

Loading...