Page MenuHomeFreeBSD
Differential D59401

sched: fix kern.sched.{4bsd,ule}.slice
AcceptedPublic
Actions

Authored by mchoo on Fri, Sep 4, 5:37 PM.
Tags
None
Referenced Files
F172171147: D59401.id186038.diff
Wed, Sep 16, 2:35 PM
Unknown Object (File)
Tue, Sep 15, 1:03 PM
Unknown Object (File)
Tue, Sep 15, 6:53 AM
Unknown Object (File)
Mon, Sep 14, 4:50 PM
Unknown Object (File)
Sun, Sep 13, 5:14 PM
Unknown Object (File)
Sat, Sep 12, 1:47 PM
Unknown Object (File)
Sat, Sep 12, 11:14 AM
Unknown Object (File)
Sat, Sep 12, 5:52 AM
View All 43 Files
Subscribers
imp
olce

Details

Reviewers
jhb
olce
Group Reviewers
scheduler
Summary

Fix three problems with kern.sched.{4bsd,ule}.slice:

  • Guarantee minimum slice is 1.
  • Recalculate hogticks on sysctl write.
  • For ULE, recalculate sched_slice_min on sysctl write.

MFC after: 2 weeks
MFC to: stable/14, stable/15
Sponsored by: FreeBSD Foundation

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Skipped
Unit
No Test Coverage
Build Status
Buildable 76617
Build 73500: arc lint + arc unit

Event Timeline

mchoo created this revision.Fri, Sep 4, 5:37 PM
Herald added subscribers: olce, imp. · View Herald TranscriptFri, Sep 4, 5:37 PM
mchoo requested review of this revision.Fri, Sep 4, 5:37 PM
Harbormaster completed remote builds in B76509: Diff 185837.Fri, Sep 4, 5:37 PM
mchoo added a parent revision: D59400: sched_ule: fix typo goup to group.Fri, Sep 4, 5:37 PM
mchoo added a child revision: D59402: sched_4bsd: fix comment in maybe_preempt().
olce requested changes to this revision.Sun, Sep 6, 4:38 PM
olce added inline comments.
sys/kern/sched_4bsd.c
214

It's an internal function, no need to have 4bsd_ in the name.

Additionally, I'm planning to rename back all implementations of the current interface (to the rest of the kernel) to their prior names, that is, with ule_ or 4bsd_ removed, relatively soon (but there is EuroBSDCon next week).

224

Not necessary given the previous test. Replace by an assertion?

225–226

Could you please factor out the hogticks computation and setting in a separate function, and use it throughout the file (preferably as a separate commit)?

sys/kern/sched_ule.c
3490–3502

Same comments as for 4BSD.

This revision now requires changes to proceed.Sun, Sep 6, 4:38 PM
mchoo updated this revision to Diff 186038.Sun, Sep 6, 9:02 PM

Remove unnecessary imax()

Harbormaster completed remote builds in B76611: Diff 186038.Sun, Sep 6, 9:02 PM
mchoo added inline comments.Sun, Sep 6, 9:02 PM
sys/kern/sched_4bsd.c
214

Since sysctl_kern_4bsd_quantum already has the namespace, I'll keep it for now. It can be renamed when the cleanup is done across 4BSD and ULE.

224

I think assertion is unnecessary here.

sys/kern/sched_ule.c
3499

D59408 adds imax() check for quantum sysctl.

olce added inline comments.Sun, Sep 6, 9:37 PM
sys/kern/sched_4bsd.c
214

Let's go right now in the intended direction. Additionally, having 4bsd_ here and no ule_ below is inconsistent. So please remove it.

mchoo updated this revision to Diff 186083.Sun, Sep 6, 10:42 PM

Remove namespacing

Harbormaster completed remote builds in B76617: Diff 186083.Sun, Sep 6, 10:42 PM
mchoo marked 4 inline comments as done.Sun, Sep 6, 10:42 PM
mchoo marked an inline comment as done.Sun, Sep 6, 10:52 PM
mchoo added inline comments.
sys/kern/sched_4bsd.c
225–226

Done in D59471

olce accepted this revision.Tue, Sep 8, 7:49 AM
This revision is now accepted and ready to land.Tue, Sep 8, 7:49 AM

Revision Contents
Changeset List

PathSize
sys/
kern/
21 lines
22 lines

Diff 186083

View Options

sys/kern/sched_4bsd.c

Loading...
View Options

sys/kern/sched_ule.c

Loading...