Page MenuHomeFreeBSD
Differential D28156

Add missing no-ctfconvert to C generation rules
ClosedPublic
Actions

Authored by andrew on Jan 14 2021, 3:42 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Apr 27, 1:27 AM
Unknown Object (File)
Tue, Apr 23, 1:41 PM
Unknown Object (File)
Mar 29 2024, 5:30 AM
Unknown Object (File)
Mar 7 2024, 4:07 PM
Unknown Object (File)
Mar 2 2024, 2:14 AM
Unknown Object (File)
Jan 11 2024, 12:15 AM
Unknown Object (File)
Dec 20 2023, 6:10 PM
Unknown Object (File)
Dec 20 2023, 8:29 AM
View All 29 Files
Subscribers
emaste
imp
mmel

Details

Reviewers
emaste
mmel
Commits
rG94611a838866: Add missing no-ctfconvert to C generation rules
Summary

Stop running ctfconvert over generated C files in the kernel by marking
them with no-ctfconvert.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 36199
Build 33088: arc lint + arc unit

Event Timeline

andrew created this revision.Jan 14 2021, 3:42 PM
Herald added subscribers: emaste, imp. · View Herald TranscriptJan 14 2021, 3:42 PM
andrew requested review of this revision.Jan 14 2021, 3:42 PM
Harbormaster completed remote builds in B36199: Diff 82266.Jan 14 2021, 3:42 PM
emaste accepted this revision.Jan 15 2021, 3:27 AM

Looks fine to me. Is there anything specific that made you notice this?

This revision is now accepted and ready to land.Jan 15 2021, 3:27 AM
mmel accepted this revision.Jan 15 2021, 9:09 AM
mmel added a subscriber: mmel.

LGTM. thanks.
It is a bit questionable whether ctfconvert should generate an error in this case (I don't think so), but calling it is clearly unnecessary in this case.

andrew added a comment.Jan 15 2021, 9:44 AM

It fixes warning similar to the following. This is because it's running ctfconvert on the .c file rather than a .o file.

ctfconvert: rtwn-rtl8821aufw.c: Couldn't read ehdr: Invalid argument

Closed by commit rG94611a838866: Add missing no-ctfconvert to C generation rules (authored by andrew). · Explain WhyJan 15 2021, 9:48 AM
This revision was automatically updated to reflect the committed changes.
andrew added a commit: rG94611a838866: Add missing no-ctfconvert to C generation rules.

Revision Contents
Changeset List

PathSize
sys/
conf/
98 lines
2 lines
4 lines

Diff 82266

View Options

sys/conf/files

Loading...
View Options

sys/conf/files.amd64

Loading...
View Options

sys/conf/files.arm64

Loading...