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)
Sat, Jul 4, 7:17 AM
Unknown Object (File)
Wed, Jul 1, 8:50 PM
Unknown Object (File)
Wed, Jul 1, 1:22 PM
Unknown Object (File)
Tue, Jun 30, 9:31 PM
Unknown Object (File)
Mon, Jun 29, 10:20 PM
Unknown Object (File)
Sat, Jun 27, 4:14 AM
Unknown Object (File)
Wed, Jun 24, 1:32 AM
Unknown Object (File)
Mon, Jun 22, 11:18 PM
View All 94 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...