Module Name:    src
Committed By:   riastradh
Date:           Fri Nov 18 06:41:52 UTC 2016

Modified Files:
        src/sys/sys: pslist.h

Log Message:
Explain why we don't touch the next pointer of the removed entry.


To generate a diff of this commit:
cvs rdiff -u -r1.3 -r1.4 src/sys/sys/pslist.h

Please note that diffs are not public domain; they are subject to the
copyright notices on the relevant files.

Modified files:

Index: src/sys/sys/pslist.h
diff -u src/sys/sys/pslist.h:1.3 src/sys/sys/pslist.h:1.4
--- src/sys/sys/pslist.h:1.3	Thu Jul  7 06:56:25 2016
+++ src/sys/sys/pslist.h	Fri Nov 18 06:41:52 2016
@@ -1,4 +1,4 @@
-/*	$NetBSD: pslist.h,v 1.3 2016/07/07 06:56:25 ozaki-r Exp $	*/
+/*	$NetBSD: pslist.h,v 1.4 2016/11/18 06:41:52 riastradh Exp $	*/
 
 /*-
  * Copyright (c) 2016 The NetBSD Foundation, Inc.
@@ -175,6 +175,13 @@ pslist_writer_remove(struct pslist_entry
 		entry->ple_next->ple_prevp = entry->ple_prevp;
 	*entry->ple_prevp = entry->ple_next;
 	entry->ple_prevp = NULL;
+
+	/*
+	 * Leave entry->ple_next intact so that any extant readers can
+	 * continue iterating through the list.  The caller must then
+	 * wait for readers to drain, e.g. with pserialize_perform,
+	 * before destroying and reusing the entry.
+	 */
 }
 
 static inline struct pslist_entry *

Reply via email to