Skip site navigation (1)Skip section navigation (2)
Date:      Mon, 31 May 2021 15:10:45 GMT
From:      Konstantin Belousov <kib@FreeBSD.org>
To:        src-committers@FreeBSD.org, dev-commits-src-all@FreeBSD.org, dev-commits-src-main@FreeBSD.org
Subject:   git: f62c7e54e9cc - main - Add thread_reap_barrier()
Message-ID:  <202105311510.14VFAjv3006425@gitrepo.freebsd.org>

next in thread | raw e-mail | index | archive | help
The branch main has been updated by kib:

URL: https://cgit.FreeBSD.org/src/commit/?id=f62c7e54e9cc692603081328597ba0ba0d1f21cf

commit f62c7e54e9cc692603081328597ba0ba0d1f21cf
Author:     Konstantin Belousov <kib@FreeBSD.org>
AuthorDate: 2021-05-25 18:51:00 +0000
Commit:     Konstantin Belousov <kib@FreeBSD.org>
CommitDate: 2021-05-31 15:09:22 +0000

    Add thread_reap_barrier()
    
    Reviewed by:    hselasky,markj
    Sponsored by:   Mellanox Technologies/NVidia Networking
    MFC after:      1 week
    Differential revision:  https://reviews.freebsd.org/D30468
---
 sys/kern/kern_thread.c | 32 ++++++++++++++++++++++++++++++++
 sys/sys/proc.h         |  1 +
 2 files changed, 33 insertions(+)

diff --git a/sys/kern/kern_thread.c b/sys/kern/kern_thread.c
index 935d7071e452..b5983eb4beb7 100644
--- a/sys/kern/kern_thread.c
+++ b/sys/kern/kern_thread.c
@@ -709,6 +709,38 @@ thread_reap_callout_cb(void *arg __unused)
 	    thread_reap_callout_cb, NULL);
 }
 
+/*
+ * Calling this function guarantees that any thread that exited before
+ * the call is reaped when the function returns.  By 'exited' we mean
+ * a thread removed from the process linkage with thread_unlink().
+ * Practically this means that caller must lock/unlock corresponding
+ * process lock before the call, to synchronize with thread_exit().
+ */
+void
+thread_reap_barrier(void)
+{
+	struct thread *td;
+	struct task *t;
+
+	td = curthread;
+
+	/*
+	 * First do context switches to each CPU to ensure that all
+	 * PCPU pc_deadthreads are moved to zombie list.
+	 */
+	quiesce_all_cpus("", PDROP);
+
+	/*
+	 * Second, fire the task in the same thread as normal
+	 * thread_reap() is done, to serialize reaping.
+	 */
+	t = malloc(sizeof(*t), M_TEMP, M_WAITOK);
+	TASK_INIT(t, 0, thread_reap_task_cb, t);
+	taskqueue_enqueue(taskqueue_thread, t);
+	taskqueue_drain(taskqueue_thread, t);
+	free(t, M_TEMP);
+}
+
 /*
  * Allocate a thread.
  */
diff --git a/sys/sys/proc.h b/sys/sys/proc.h
index 2630c2b5a1d0..afa0be05d33a 100644
--- a/sys/sys/proc.h
+++ b/sys/sys/proc.h
@@ -1189,6 +1189,7 @@ int	thread_create(struct thread *td, struct rtprio *rtp,
 void	thread_exit(void) __dead2;
 void	thread_free(struct thread *td);
 void	thread_link(struct thread *td, struct proc *p);
+void	thread_reap_barrier(void);
 int	thread_single(struct proc *p, int how);
 void	thread_single_end(struct proc *p, int how);
 void	thread_stash(struct thread *td);



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?202105311510.14VFAjv3006425>