Page MenuHomeFreeBSD
Differential D60167

ifuncs: Have DEFINE_*IFUNC() macros expand the passed name
ClosedPublic
Actions

Authored by olce on Wed, Sep 30, 9:26 AM.
Tags
None
Referenced Files
F174158586: D60167.id188166.diff
Thu, Oct 1, 12:32 AM
F174154300: D60167.diff
Wed, Sep 30, 11:44 PM
F174131073: D60167.diff
Wed, Sep 30, 7:55 PM
Unknown Object (File)
Wed, Sep 30, 10:04 AM
Unknown Object (File)
Wed, Sep 30, 10:02 AM
Subscribers
imp
jhibbits
riscv

Details

Reviewers
emaste
kib
andrew
manu
Group Reviewers
scheduler
Commits
rGab9fcc5a553a: ifuncs: Have DEFINE_*IFUNC() macros expand the passed name

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

olce created this revision.Wed, Sep 30, 9:26 AM
Herald added a reviewer: andrew. · View Herald TranscriptWed, Sep 30, 9:26 AM
Herald added a reviewer: andrew. · View Herald Transcript
Herald added a reviewer: manu. · View Herald Transcript
Herald added subscribers: riscv, jhibbits, imp. · View Herald Transcript
olce requested review of this revision.Wed, Sep 30, 9:26 AM
Harbormaster completed remote builds in B77519: Diff 188166.Wed, Sep 30, 9:26 AM
olce added a child revision: D59988: sched: New scheduler interface definition scheme.Wed, Sep 30, 9:26 AM
olce added a comment.Wed, Sep 30, 11:10 AM

I'm wondering whether to put #include <sys/cdefs.h> in all headers, and add the missing includes to make all these headers standalone.

kib accepted this revision.Thu, Oct 1, 5:26 AM
In D60167#1380633, @olce wrote:

I'm wondering whether to put #include <sys/cdefs.h> in all headers, and add the missing includes to make all these headers standalone.

It would be sys/types.h then.

This revision is now accepted and ready to land.Thu, Oct 1, 5:26 AM
olce added a comment.Thu, Oct 1, 7:58 AM
In D60167#1381261, @kib wrote:
In D60167#1380633, @olce wrote:

I'm wondering whether to put #include <sys/cdefs.h> in all headers, and add the missing includes to make all these headers standalone.

It would be sys/types.h then.

Right, that's simpler.

Closed by commit rGab9fcc5a553a: ifuncs: Have DEFINE_*IFUNC() macros expand the passed name (authored by olce). · Explain WhyThu, Oct 1, 8:26 AM
This revision was automatically updated to reflect the committed changes.
olce added a commit: rGab9fcc5a553a: ifuncs: Have DEFINE_*IFUNC() macros expand the passed name.

Revision Contents
Changeset List

PathSize
sys/
arm/
include/
18 lines
arm64/
include/
16 lines
powerpc/
include/
14 lines
riscv/
include/
19 lines
x86/
include/
18 lines

Diff 188297

View Options

sys/arm/include/ifunc.h

Loading...
View Options

sys/arm64/include/ifunc.h

Loading...
View Options

sys/powerpc/include/ifunc.h

Loading...
View Options

sys/riscv/include/ifunc.h

Loading...
View Options

sys/x86/include/ifunc.h

Loading...