Page MenuHomeFreeBSD
Differential D56085

kern_mib: remove kern.fallback_elf_brand compat sysctl
AcceptedPublic
Actions

Authored by chris.longros_gmail.com on Wed, Mar 25, 2:06 PM.
Tags
None
Referenced Files
F151966471: D56085.id.diff
Sat, Apr 11, 8:04 PM
F151937145: D56085.diff
Sat, Apr 11, 3:31 PM
Unknown Object (File)
Thu, Apr 9, 2:24 AM
Unknown Object (File)
Wed, Apr 8, 9:59 AM
Unknown Object (File)
Mon, Apr 6, 12:45 PM
Unknown Object (File)
Tue, Mar 31, 6:10 PM
Unknown Object (File)
Mon, Mar 30, 8:55 PM
Unknown Object (File)
Mon, Mar 30, 3:57 PM
View All 10 Files
Subscribers
imp
olce

Details

Reviewers
kib
Summary

This compatibility alias for kern.elf{32,64}.fallback_brand has been
marked for removal since FreeBSD 6.0.

Signed-off-by: Christos Longros <chris.longros@gmail.com>

Diff Detail

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

Event Timeline

chris.longros_gmail.com created this revision.Wed, Mar 25, 2:06 PM
Herald added subscribers: olce, imp. · View Herald TranscriptWed, Mar 25, 2:06 PM
chris.longros_gmail.com requested review of this revision.Wed, Mar 25, 2:06 PM
Harbormaster completed remote builds in B71716: Diff 174293.Wed, Mar 25, 2:06 PM
chris.longros_gmail.com added a reviewer: kib.Wed, Mar 25, 2:09 PM
kib added a comment.Wed, Mar 25, 10:12 PM

Does the extern declaration for __elfN(fallback_brand) in sys/sys/imgact_elf.h is needed after the patch? Could the __elfN(fallback_brand) var made static in kern_elf.c?

chris.longros_gmail.com added a comment.EditedThu, Mar 26, 6:46 PM

Good point. Updated the made __elfN(fallback_brand) from extern to static in imgact_elf.c.

chris.longros_gmail.com updated this revision to Diff 174344.EditedThu, Mar 26, 6:47 PM

make fallback_brand static, remove extern

Harbormaster completed remote builds in B71725: Diff 174344.Thu, Mar 26, 6:47 PM
kib accepted this revision.Fri, Mar 27, 2:04 AM
This revision is now accepted and ready to land.Fri, Mar 27, 2:04 AM

Revision Contents
Changeset List

PathSize
sys/
kern/
6 lines
tools/
tools/
sysdoc/
3 lines

Diff 174293

View Options

sys/kern/kern_mib.c

Loading...
View Options

tools/tools/sysdoc/tunables.mdoc

Loading...