Module Name:    src
Committed By:   riastradh
Date:           Fri Apr 21 18:44:18 UTC 2023

Modified Files:
        src/sys/dev/dkwedge: dk.c

Log Message:
dk(4): Take a read-lock on dkwedges_lock if we're only reading.

- dkwedge_find_by_name
- dkwedge_find_by_parent
- dkwedge_print_wnames


To generate a diff of this commit:
cvs rdiff -u -r1.144 -r1.145 src/sys/dev/dkwedge/dk.c

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

Modified files:

Index: src/sys/dev/dkwedge/dk.c
diff -u src/sys/dev/dkwedge/dk.c:1.144 src/sys/dev/dkwedge/dk.c:1.145
--- src/sys/dev/dkwedge/dk.c:1.144	Fri Apr 21 18:31:00 2023
+++ src/sys/dev/dkwedge/dk.c	Fri Apr 21 18:44:18 2023
@@ -1,4 +1,4 @@
-/*	$NetBSD: dk.c,v 1.144 2023/04/21 18:31:00 riastradh Exp $	*/
+/*	$NetBSD: dk.c,v 1.145 2023/04/21 18:44:18 riastradh Exp $	*/
 
 /*-
  * Copyright (c) 2004, 2005, 2006, 2007 The NetBSD Foundation, Inc.
@@ -30,7 +30,7 @@
  */
 
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: dk.c,v 1.144 2023/04/21 18:31:00 riastradh Exp $");
+__KERNEL_RCSID(0, "$NetBSD: dk.c,v 1.145 2023/04/21 18:44:18 riastradh Exp $");
 
 #ifdef _KERNEL_OPT
 #include "opt_dkwedge.h"
@@ -877,7 +877,7 @@ dkwedge_find_by_wname(const char *wname)
 	struct dkwedge_softc *sc;
 	int i;
 
-	rw_enter(&dkwedges_lock, RW_WRITER);
+	rw_enter(&dkwedges_lock, RW_READER);
 	for (i = 0; i < ndkwedges; i++) {
 		if ((sc = dkwedges[i]) == NULL)
 			continue;
@@ -900,7 +900,7 @@ device_t
 dkwedge_find_by_parent(const char *name, size_t *i)
 {
 
-	rw_enter(&dkwedges_lock, RW_WRITER);
+	rw_enter(&dkwedges_lock, RW_READER);
 	for (; *i < (size_t)ndkwedges; (*i)++) {
 		struct dkwedge_softc *sc;
 		if ((sc = dkwedges[*i]) == NULL)
@@ -920,7 +920,7 @@ dkwedge_print_wnames(void)
 	struct dkwedge_softc *sc;
 	int i;
 
-	rw_enter(&dkwedges_lock, RW_WRITER);
+	rw_enter(&dkwedges_lock, RW_READER);
 	for (i = 0; i < ndkwedges; i++) {
 		if ((sc = dkwedges[i]) == NULL)
 			continue;

Reply via email to