Page MenuHomeFreeBSD
Differential D20281

Remove resolver_qual from DEFINE_IFUNC/DEFINE_UIFUNC macros.
ClosedPublic
Actions

Authored by kib on May 16 2019, 7:47 PM.
Tags
None
Referenced Files
F169489996: D20281.id57463.diff
Tue, Sep 1, 4:49 PM
F169258710: D20281.diff
Tue, Sep 1, 2:08 AM
F169247877: D20281.id.diff
Tue, Sep 1, 1:31 AM
Unknown Object (File)
Sun, Aug 30, 2:47 PM
Unknown Object (File)
Sun, Aug 30, 1:47 AM
Unknown Object (File)
Sun, Aug 30, 1:25 AM
Unknown Object (File)
Wed, Aug 26, 8:54 AM
Unknown Object (File)
Mon, Aug 10, 2:51 PM
View All Files
Subscribers
andrew
imp
rpokala

Details

Reviewers
emaste
gordon
gnn
jmg
manu
Group Reviewers
secteam
O3: Kernel Random Numbers Generator
Commits
rS347895: Remove resolver_qual from DEFINE_IFUNC/DEFINE_UIFUNC macros.
Summary

In all practical situations, the resolver visibility is static.

Requested by: markj
Sponsored by: The FreeBSD Foundation
MFC after: 1 week

[The reason for review is that I touch dev/random, and commit hook triggers]

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

kib created this revision.May 16 2019, 7:47 PM
Herald added a reviewer: gnn. · View Herald TranscriptMay 16 2019, 7:47 PM
Herald added a reviewer: jmg. · View Herald Transcript
Herald added a reviewer: manu. · View Herald Transcript
Herald added subscribers: rpokala, andrew, imp. · View Herald Transcript
Harbormaster completed remote builds in B24266: Diff 57460.May 16 2019, 7:47 PM
emaste added a comment.May 16 2019, 8:48 PM

One ifunc.h has static built-in now and one does not?

sys/amd64/conf/X
89 ↗(On Diff #57460)

?

kib updated this revision to Diff 57463.May 16 2019, 9:33 PM

Fix patch for arm64.

Harbormaster completed remote builds in B24268: Diff 57463.May 16 2019, 9:33 PM
emaste accepted this revision as: emaste.May 16 2019, 10:11 PM

LGTM other than presumably unintentional sys/amd64/conf/X

This revision was not accepted when it landed; it landed in state Needs Review.May 16 2019, 10:21 PM
Closed by commit rS347895: Remove resolver_qual from DEFINE_IFUNC/DEFINE_UIFUNC macros. (authored by kib). · Explain Why
This revision was automatically updated to reflect the committed changes.
Owners added a reviewer: O3: Kernel Random Numbers Generator.May 16 2019, 10:21 PM

Revision Contents
Changeset List

PathSizePackages
head/
lib/
libc/
amd64/
sys/
2 lines
2 lines
2 lines
2 lines
x86/
gen/
4 lines
sys/
2 lines
4 lines
sys/
amd64/
amd64/
32 lines
4 lines
11 lines
26 lines
linux/
10 lines
linux32/
10 lines
arm64/
include/
12 lines
cddl/
dev/
dtrace/
amd64/
14 lines
dev/
random/
2 linesO3: Kernel Random Numbers Generator
i386/
i386/
2 lines
3 lines
x86/
include/
14 lines

Diff 57465

View Options

head/lib/libc/amd64/sys/amd64_get_fsbase.c

Loading...
View Options

head/lib/libc/amd64/sys/amd64_get_gsbase.c

Loading...
View Options

head/lib/libc/amd64/sys/amd64_set_fsbase.c

Loading...
View Options

head/lib/libc/amd64/sys/amd64_set_gsbase.c

Loading...
View Options

head/lib/libc/x86/gen/getcontextx.c

Loading...
View Options

head/lib/libc/x86/sys/__vdso_gettc.c

Loading...
View Options

head/lib/libc/x86/sys/pkru.c

Loading...
View Options

head/sys/amd64/amd64/copyout.c

Loading...
View Options

head/sys/amd64/amd64/fpu.c

Loading...
View Options

head/sys/amd64/amd64/machdep.c

Loading...
View Options

head/sys/amd64/amd64/pmap.c

Loading...
View Options

head/sys/amd64/linux/linux_machdep.c

Loading...
View Options

head/sys/amd64/linux32/linux32_machdep.c

Loading...
View Options

head/sys/arm64/include/ifunc.h

Loading...
View Options

head/sys/cddl/dev/dtrace/amd64/dtrace_isa.c

Loading...
View Options

head/sys/dev/random/ivy.c

Loading...
View Options

head/sys/i386/i386/npx.c

Loading...
View Options

head/sys/i386/i386/pmap_base.c

Loading...
View Options

head/sys/x86/include/ifunc.h

Loading...