change /* * This queue contains only priorities between MIN and MAX * realtime. Use the whole queue to represent these values. */ to /* * This queue contains only priorities between MIN and MAX * timeshare. Use the whole queue to represent these values. */ The queue the comment refers is tdq_timeshare.
It's actually >= PRI_MIN_BATCH and <= PRI_MAX_BATCH, and PRI_MIN_BATCH is PRI_MIN_TIMESHARE + PRI_INTERACT_RANGE, so it's not MIN and MAX timeshare either. I think both the comment and code warrant some more consideration.
A commit in branch main references this bug: URL: https://cgit.FreeBSD.org/src/commit/?id=ba71333f6049ee76b0023b3b9e8444d5ccb1a7ff commit ba71333f6049ee76b0023b3b9e8444d5ccb1a7ff Author: Mark Johnston <markj@FreeBSD.org> AuthorDate: 2022-07-11 19:33:20 +0000 Commit: Mark Johnston <markj@FreeBSD.org> CommitDate: 2022-07-11 19:58:43 +0000 sched_ule: Fix a typo in a comment PR: 226107 MFC after: 1 week sys/kern/sched_ule.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
A commit in branch stable/13 references this bug: URL: https://cgit.FreeBSD.org/src/commit/?id=8342f4b1ef973f44813ee9720d290dc257ddfc48 commit 8342f4b1ef973f44813ee9720d290dc257ddfc48 Author: Mark Johnston <markj@FreeBSD.org> AuthorDate: 2022-07-11 19:33:20 +0000 Commit: Mark Johnston <markj@FreeBSD.org> CommitDate: 2022-07-18 13:08:16 +0000 sched_ule: Fix a typo in a comment PR: 226107 (cherry picked from commit ba71333f6049ee76b0023b3b9e8444d5ccb1a7ff) sys/kern/sched_ule.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
Thanks for the report.
MARKED AS SPAM