Paths

Table of Contentst

Differential D54830

sched_ule: mark scheduler interface functions as weak symbols
Needs ReviewPublic
Actions

Authored by minsoochoo0122_proton.me on Thu, Jan 22, 5:38 PM.

Details

Reviewers

olce

Group Reviewers

Contributor Reviews (src)

Summary

When a kernel module scheduler is loaded, its sched_* implementation
will override GENERIC's weak sched interface.

As part of this effort, two more changes are included in this review:

sched_ule: add dummy sched_pctcpu_delta
Rename __inline to inline and reorder some static inline functions in sched.h

Test Plan

GENERIC build and boot succeeded.

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 70066
Build 66949: arc lint + arc unit

Event Timeline

minsoochoo0122_proton.me created this revision.Thu, Jan 22, 5:38 PM

Herald added subscribers: olce, imp. · View Herald TranscriptThu, Jan 22, 5:38 PM

minsoochoo0122_proton.me requested review of this revision.Thu, Jan 22, 5:38 PM

Harbormaster completed remote builds in B70066: Diff 170248.Thu, Jan 22, 5:38 PM

minsoochoo0122_proton.me edited the test plan for this revision. (Show Details)Thu, Jan 22, 5:39 PM

minsoochoo0122_proton.me added reviewers: Contributor Reviews (src), olce.

minsoochoo0122_proton.me added a project: Contributor Reviews (src).

minsoochoo0122_proton.me edited the test plan for this revision. (Show Details)Thu, Jan 22, 5:42 PM

minsoochoo0122_proton.me added inline comments.Thu, Jan 22, 5:48 PM

sys/kern/sched_ule.c
3019	ULE doesn't need this, and personally I don't think any modern scheduler would require this. Should we provide this as `<sys/sched_4bsd.h>`?

minsoochoo0122_proton.me mentioned this in D54831: Make ULE and 4BSD coexists.Thu, Jan 22, 8:24 PM

Revision Contents
Changeset List

Path

Size

sys/

kern/

sched_ule.c

96 lines

sys/

sched.h

43 lines

Diff 170248

View Options

sys/kern/sched_ule.c