]> CyberLeo.Net >> Repos - FreeBSD/FreeBSD.git/commit
kthread: Set *tdptr earlier in kproc_kthread_add()
authorMark Johnston <markj@FreeBSD.org>
Thu, 25 Apr 2024 13:35:38 +0000 (09:35 -0400)
committerMark Johnston <markj@FreeBSD.org>
Thu, 25 Apr 2024 13:35:38 +0000 (09:35 -0400)
commitd66399326cb4f89d1565fb62c1c07974886893c5
treeeccda3056fd59c69810b88904f6dfff4db2b689c
parent42255af64a5471a2668e3a0552d82639e1f70ee4
kthread: Set *tdptr earlier in kproc_kthread_add()

See commit ae77041e0714 ("kthread: Set *newtdp earlier in
kthread_add1()") for details.  That commit was incomplete since
g_init()'s first call to kproc_kthread_add() will cause
kproc_kthread_add() to take the `*procptr == NULL` branch, which avoids
kthread_create().

To ensure that the thread pointer is initialized before the thread
starts running, we have to start the kernel process with RFSTOPPED.
We could perhaps go further and use RFSTOPPED only when tdptr != NULL,
but it's probably better to have consistent behaviour.

Reviewed by: olce, kib
Reported by: syzbot+e91e798f3c088215ace6@syzkaller.appspotmail.com
MFC after: 1 week
Differential Revision: https://reviews.freebsd.org/D44927
sys/kern/kern_kthread.c