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
F174454047: D60167.id.diff
Sat, Oct 3, 8:49 AM
F174450344: D60167.id.diff
Sat, Oct 3, 7:59 AM
F174421471: D60167.id188297.diff
Sat, Oct 3, 2:02 AM
F174393058: D60167.id188166.diff
Fri, Oct 2, 9:23 PM
Unknown Object (File)
Fri, Oct 2, 1:10 AM
Unknown Object (File)
Fri, Oct 2, 1:01 AM
Unknown Object (File)
Fri, Oct 2, 12:52 AM
Unknown Object (File)
Thu, Oct 1, 9:22 PM
View All 13 Files
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 Skipped
Unit
Tests Skipped
Build Status
Buildable 77519
Build 74402: arc lint + arc unit

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/
16 lines
arm64/
include/
14 lines
powerpc/
include/
14 lines
riscv/
include/
17 lines
x86/
include/
16 lines

Diff 188166

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...