Page MenuHomeFreeBSD
Differential D53624

kern_thread: thread_suspend_check(1) must never suspend
ClosedPublic
Actions

Authored by kib on Nov 6 2025, 7:28 PM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Dec 30, 5:18 PM
Unknown Object (File)
Sun, Dec 28, 12:52 PM
Unknown Object (File)
Fri, Dec 26, 5:46 AM
Unknown Object (File)
Mon, Dec 22, 8:01 PM
Unknown Object (File)
Wed, Dec 10, 7:27 PM
Unknown Object (File)
Nov 27 2025, 11:28 PM
Unknown Object (File)
Nov 22 2025, 7:06 AM
Unknown Object (File)
Nov 20 2025, 1:11 AM
View All 29 Files
Subscribers
bdrewery
emaste
imp
olce
pho

Details

Reviewers
markj
bdrewery
Commits
rGe1c6f4cb9bd2: kern_thread: thread_suspend_check(1) must never suspend
Summary
Reported by:    bdrewery
PR:     290843

Diff Detail

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

Event Timeline

kib created this revision.Nov 6 2025, 7:28 PM
Herald added subscribers: olce, imp. · View Herald TranscriptNov 6 2025, 7:28 PM
kib requested review of this revision.Nov 6 2025, 7:28 PM
bdrewery added a comment.Nov 6 2025, 7:34 PM

Testing this now.

bdrewery added a comment.Nov 6 2025, 9:10 PM

Thanks for the quick fix! It passes my repro.

sys/kern/kern_thread.c
1491–1494

This branch is now always true.

kib updated this revision to Diff 165983.Nov 6 2025, 9:15 PM

Assert always true condition, instead of checking for it.

markj accepted this revision.Nov 7 2025, 2:25 PM
This revision is now accepted and ready to land.Nov 7 2025, 2:25 PM
emaste added a subscriber: emaste.Nov 8 2025, 6:07 PM
emaste added inline comments.
sys/kern/kern_thread.c
1450
kib marked an inline comment as done.Nov 8 2025, 7:58 PM
kib marked an inline comment as done.
bdrewery accepted this revision.Nov 10 2025, 4:51 PM

I have not reviewed each case but have been running it without problems and it looks reasonable.

Closed by commit rGe1c6f4cb9bd2: kern_thread: thread_suspend_check(1) must never suspend (authored by kib). · Explain WhyNov 11 2025, 8:54 AM
This revision was automatically updated to reflect the committed changes.
kib added a commit: rGe1c6f4cb9bd2: kern_thread: thread_suspend_check(1) must never suspend.

Revision Contents
Changeset List

PathSize
sys/
kern/
15 lines

Diff 166198

View Options

sys/kern/kern_thread.c

Loading...