Source-Changes-HG archive
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]
[src/trunk]: src/sys/sys Rip out pserialize(9) logic now that the RCU patent ...
details: https://anonhg.NetBSD.org/src/rev/f5ef9be35bf3
branches: trunk
changeset: 967232:f5ef9be35bf3
user: riastradh <riastradh%NetBSD.org@localhost>
date: Tue Dec 03 05:07:48 2019 +0000
description:
Rip out pserialize(9) logic now that the RCU patent has expired.
pserialize_perform() is now basically just xc_barrier(XC_HIGHPRI).
No more tentacles throughout the scheduler. Simplify the psz read
count for diagnostic assertions by putting it unconditionally into
cpu_info.
>From rmind@, tidied up by me.
diffstat:
sys/arch/mips/rmi/rmixl_cpu.c | 10 +-
sys/kern/kern_lwp.c | 8 +-
sys/kern/kern_softint.c | 8 +-
sys/kern/kern_synch.c | 8 +-
sys/kern/subr_pserialize.c | 241 +++++---------------------------------
sys/rump/librump/rumpkern/rump.c | 9 +-
sys/sys/cpu_data.h | 4 +-
sys/sys/pserialize.h | 3 +-
8 files changed, 49 insertions(+), 242 deletions(-)
diffs (truncated from 598 to 300 lines):
diff -r db6f345abb7c -r f5ef9be35bf3 sys/arch/mips/rmi/rmixl_cpu.c
--- a/sys/arch/mips/rmi/rmixl_cpu.c Tue Dec 03 05:01:58 2019 +0000
+++ b/sys/arch/mips/rmi/rmixl_cpu.c Tue Dec 03 05:07:48 2019 +0000
@@ -1,4 +1,4 @@
-/* $NetBSD: rmixl_cpu.c,v 1.10 2019/12/01 15:34:45 ad Exp $ */
+/* $NetBSD: rmixl_cpu.c,v 1.11 2019/12/03 05:07:48 riastradh Exp $ */
/*
* Copyright 2002 Wasabi Systems, Inc.
@@ -38,7 +38,7 @@
#include "locators.h"
#include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: rmixl_cpu.c,v 1.10 2019/12/01 15:34:45 ad Exp $");
+__KERNEL_RCSID(0, "$NetBSD: rmixl_cpu.c,v 1.11 2019/12/03 05:07:48 riastradh Exp $");
#include "opt_multiprocessor.h"
#include "opt_ddb.h"
@@ -414,8 +414,6 @@
{
printf("cpu_biglock_wanted %p\n", dp->cpu_biglock_wanted);
printf("cpu_callout %p\n", dp->cpu_callout);
- printf("cpu_unused1 %p\n", dp->cpu_unused1);
- printf("cpu_unused2 %d\n", dp->cpu_unused2);
printf("&cpu_schedstate %p\n", &dp->cpu_schedstate); /* TBD */
printf("&cpu_xcall %p\n", &dp->cpu_xcall); /* TBD */
printf("cpu_xcall_pending %d\n", dp->cpu_xcall_pending);
@@ -423,9 +421,7 @@
printf("cpu_lockstat %p\n", dp->cpu_lockstat);
printf("cpu_index %d\n", dp->cpu_index);
printf("cpu_biglock_count %d\n", dp->cpu_biglock_count);
- printf("cpu_spin_locks %d\n", dp->cpu_spin_locks);
- printf("cpu_simple_locks %d\n", dp->cpu_simple_locks);
- printf("cpu_spin_locks2 %d\n", dp->cpu_spin_locks2);
+ printf("cpu_psz_read_depth %d\n", dp->cpu_psz_read_depth);
printf("cpu_lkdebug_recurse %d\n", dp->cpu_lkdebug_recurse);
printf("cpu_softints %d\n", dp->cpu_softints);
printf("cpu_nsyscall %"PRIu64"\n", dp->cpu_nsyscall);
diff -r db6f345abb7c -r f5ef9be35bf3 sys/kern/kern_lwp.c
--- a/sys/kern/kern_lwp.c Tue Dec 03 05:01:58 2019 +0000
+++ b/sys/kern/kern_lwp.c Tue Dec 03 05:07:48 2019 +0000
@@ -1,4 +1,4 @@
-/* $NetBSD: kern_lwp.c,v 1.215 2019/12/01 15:27:58 ad Exp $ */
+/* $NetBSD: kern_lwp.c,v 1.216 2019/12/03 05:07:48 riastradh Exp $ */
/*-
* Copyright (c) 2001, 2006, 2007, 2008, 2009, 2019 The NetBSD Foundation, Inc.
@@ -209,7 +209,7 @@
*/
#include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: kern_lwp.c,v 1.215 2019/12/01 15:27:58 ad Exp $");
+__KERNEL_RCSID(0, "$NetBSD: kern_lwp.c,v 1.216 2019/12/03 05:07:48 riastradh Exp $");
#include "opt_ddb.h"
#include "opt_lockdebug.h"
@@ -225,7 +225,6 @@
#include <sys/syscallargs.h>
#include <sys/syscall_stats.h>
#include <sys/kauth.h>
-#include <sys/pserialize.h>
#include <sys/sleepq.h>
#include <sys/lockdebug.h>
#include <sys/kmem.h>
@@ -1036,9 +1035,6 @@
pmap_activate(new_lwp);
spl0();
- /* Note trip through cpu_switchto(). */
- pserialize_switchpoint();
-
LOCKDEBUG_BARRIER(NULL, 0);
KPREEMPT_ENABLE(new_lwp);
if ((new_lwp->l_pflag & LP_MPSAFE) == 0) {
diff -r db6f345abb7c -r f5ef9be35bf3 sys/kern/kern_softint.c
--- a/sys/kern/kern_softint.c Tue Dec 03 05:01:58 2019 +0000
+++ b/sys/kern/kern_softint.c Tue Dec 03 05:07:48 2019 +0000
@@ -1,4 +1,4 @@
-/* $NetBSD: kern_softint.c,v 1.52 2019/12/01 15:34:46 ad Exp $ */
+/* $NetBSD: kern_softint.c,v 1.53 2019/12/03 05:07:48 riastradh Exp $ */
/*-
* Copyright (c) 2007, 2008, 2019 The NetBSD Foundation, Inc.
@@ -170,7 +170,7 @@
*/
#include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: kern_softint.c,v 1.52 2019/12/01 15:34:46 ad Exp $");
+__KERNEL_RCSID(0, "$NetBSD: kern_softint.c,v 1.53 2019/12/03 05:07:48 riastradh Exp $");
#include <sys/param.h>
#include <sys/proc.h>
@@ -182,7 +182,6 @@
#include <sys/evcnt.h>
#include <sys/cpu.h>
#include <sys/xcall.h>
-#include <sys/pserialize.h>
#include <net/netisr.h>
@@ -884,9 +883,6 @@
l->l_pflag &= ~LP_TIMEINTR;
}
- /* Indicate a soft-interrupt switch. */
- pserialize_switchpoint();
-
/*
* If we blocked while handling the interrupt, the pinned LWP is
* gone so switch to the idle LWP. It will select a new LWP to
diff -r db6f345abb7c -r f5ef9be35bf3 sys/kern/kern_synch.c
--- a/sys/kern/kern_synch.c Tue Dec 03 05:01:58 2019 +0000
+++ b/sys/kern/kern_synch.c Tue Dec 03 05:07:48 2019 +0000
@@ -1,4 +1,4 @@
-/* $NetBSD: kern_synch.c,v 1.327 2019/12/01 15:34:46 ad Exp $ */
+/* $NetBSD: kern_synch.c,v 1.328 2019/12/03 05:07:48 riastradh Exp $ */
/*-
* Copyright (c) 1999, 2000, 2004, 2006, 2007, 2008, 2009, 2019
@@ -69,7 +69,7 @@
*/
#include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: kern_synch.c,v 1.327 2019/12/01 15:34:46 ad Exp $");
+__KERNEL_RCSID(0, "$NetBSD: kern_synch.c,v 1.328 2019/12/03 05:07:48 riastradh Exp $");
#include "opt_kstack.h"
#include "opt_dtrace.h"
@@ -743,9 +743,6 @@
l->l_lwpctl->lc_pctr++;
}
- /* Note trip through cpu_switchto(). */
- pserialize_switchpoint();
-
KASSERT(l->l_cpu == ci);
splx(oldspl);
/*
@@ -755,7 +752,6 @@
retval = 1;
} else {
/* Nothing to do - just unlock and return. */
- pserialize_switchpoint();
mutex_spin_exit(spc->spc_mutex);
l->l_pflag &= ~LP_PREEMPTING;
lwp_unlock(l);
diff -r db6f345abb7c -r f5ef9be35bf3 sys/kern/subr_pserialize.c
--- a/sys/kern/subr_pserialize.c Tue Dec 03 05:01:58 2019 +0000
+++ b/sys/kern/subr_pserialize.c Tue Dec 03 05:07:48 2019 +0000
@@ -1,4 +1,4 @@
-/* $NetBSD: subr_pserialize.c,v 1.13 2019/10/06 15:11:17 uwe Exp $ */
+/* $NetBSD: subr_pserialize.c,v 1.14 2019/12/03 05:07:49 riastradh Exp $ */
/*-
* Copyright (c) 2010, 2011 The NetBSD Foundation, Inc.
@@ -28,58 +28,26 @@
/*
* Passive serialization.
- *
- * Implementation accurately matches the lapsed US patent 4809168, therefore
- * code is patent-free in the United States. Your use of this code is at
- * your own risk.
- *
- * Note for NetBSD developers: all changes to this source file must be
- * approved by the <core>.
*/
#include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: subr_pserialize.c,v 1.13 2019/10/06 15:11:17 uwe Exp $");
+__KERNEL_RCSID(0, "$NetBSD: subr_pserialize.c,v 1.14 2019/12/03 05:07:49 riastradh Exp $");
#include <sys/param.h>
-
-#include <sys/condvar.h>
+#include <sys/atomic.h>
#include <sys/cpu.h>
#include <sys/evcnt.h>
#include <sys/kmem.h>
-#include <sys/mutex.h>
#include <sys/pserialize.h>
-#include <sys/proc.h>
-#include <sys/queue.h>
#include <sys/xcall.h>
struct pserialize {
- TAILQ_ENTRY(pserialize) psz_chain;
lwp_t * psz_owner;
- kcpuset_t * psz_target;
- kcpuset_t * psz_pass;
};
-static u_int psz_work_todo __cacheline_aligned;
-static kmutex_t psz_lock __cacheline_aligned;
static struct evcnt psz_ev_excl __cacheline_aligned;
/*
- * As defined in "Method 1":
- * q0: "0 MP checkpoints have occured".
- * q1: "1 MP checkpoint has occured".
- * q2: "2 MP checkpoints have occured".
- */
-static TAILQ_HEAD(, pserialize) psz_queue0 __cacheline_aligned;
-static TAILQ_HEAD(, pserialize) psz_queue1 __cacheline_aligned;
-static TAILQ_HEAD(, pserialize) psz_queue2 __cacheline_aligned;
-
-#ifdef LOCKDEBUG
-#include <sys/percpu.h>
-
-static percpu_t *psz_debug_nreads __cacheline_aligned;
-#endif
-
-/*
* pserialize_init:
*
* Initialize passive serialization structures.
@@ -88,16 +56,8 @@
pserialize_init(void)
{
- psz_work_todo = 0;
- TAILQ_INIT(&psz_queue0);
- TAILQ_INIT(&psz_queue1);
- TAILQ_INIT(&psz_queue2);
- mutex_init(&psz_lock, MUTEX_DEFAULT, IPL_SCHED);
evcnt_attach_dynamic(&psz_ev_excl, EVCNT_TYPE_MISC, NULL,
"pserialize", "exclusive access");
-#ifdef LOCKDEBUG
- psz_debug_nreads = percpu_alloc(sizeof(uint32_t));
-#endif
}
/*
@@ -110,11 +70,7 @@
{
pserialize_t psz;
- psz = kmem_zalloc(sizeof(struct pserialize), KM_SLEEP);
- kcpuset_create(&psz->psz_target, true);
- kcpuset_create(&psz->psz_pass, true);
- psz->psz_owner = NULL;
-
+ psz = kmem_zalloc(sizeof(*psz), KM_SLEEP);
return psz;
}
@@ -128,25 +84,19 @@
{
KASSERT(psz->psz_owner == NULL);
-
- kcpuset_destroy(psz->psz_target);
- kcpuset_destroy(psz->psz_pass);
- kmem_free(psz, sizeof(struct pserialize));
+ kmem_free(psz, sizeof(*psz));
}
/*
* pserialize_perform:
*
- * Perform the write side of passive serialization. The calling
- * thread holds an exclusive lock on the data object(s) being updated.
- * We wait until every processor in the system has made at least two
- * passes through cpu_switchto(). The wait is made with the caller's
- * update lock held, but is short term.
+ * Perform the write side of passive serialization. This operation
+ * MUST be serialized at a caller level (e.g. with a mutex or by a
+ * single-threaded use).
*/
void
pserialize_perform(pserialize_t psz)
{
- int n;
KASSERT(!cpu_intr_p());
KASSERT(!cpu_softintr_p());
@@ -155,46 +105,23 @@
return;
}
KASSERT(psz->psz_owner == NULL);
- KASSERT(ncpu > 0);
if (__predict_false(mp_online == false)) {
psz_ev_excl.ev_count++;
return;
}
- /*
- * Set up the object and put it onto the queue. The lock
- * activity here provides the necessary memory barrier to
- * make the caller's data update completely visible to
- * other processors.
- */
psz->psz_owner = curlwp;
- kcpuset_copy(psz->psz_target, kcpuset_running);
- kcpuset_zero(psz->psz_pass);
-
- mutex_spin_enter(&psz_lock);
- TAILQ_INSERT_TAIL(&psz_queue0, psz, psz_chain);
- psz_work_todo++;
Home |
Main Index |
Thread Index |
Old Index