From: krw <krw@openbsd.org>
Date: Sun, 28 Dec 2014 18:32:12 +0000 (+0000)
Subject: Replace last workq in tree (reading disklabels from newly attached
X-Git-Url: http://artulab.com/gitweb/?a=commitdiff_plain;h=40a0de4a8f33d9abff12beeb17ffb8ec24c00808;p=openbsd

Replace last workq in tree (reading disklabels from newly attached
disks) with taskq. Diff originally from blambert@.

ok kettenis@
---

diff --git a/sys/kern/subr_disk.c b/sys/kern/subr_disk.c
index ae37a4fa816..75a4cde4e9e 100644
--- a/sys/kern/subr_disk.c
+++ b/sys/kern/subr_disk.c
@@ -1,4 +1,4 @@
-/*	$OpenBSD: subr_disk.c,v 1.177 2014/12/20 15:54:48 krw Exp $	*/
+/*	$OpenBSD: subr_disk.c,v 1.178 2014/12/28 18:32:12 krw Exp $	*/
 /*	$NetBSD: subr_disk.c,v 1.17 1996/03/16 23:17:08 christos Exp $	*/
 
 /*
@@ -55,7 +55,7 @@
 #include <sys/reboot.h>
 #include <sys/dkio.h>
 #include <sys/vnode.h>
-#include <sys/workq.h>
+#include <sys/task.h>
 
 #include <sys/socket.h>
 #include <sys/socketvar.h>
@@ -89,6 +89,8 @@ int	disk_change;		/* set if a disk has been attached/detached
 u_char	bootduid[8];		/* DUID of boot disk. */
 u_char	rootduid[8];		/* DUID of root disk. */
 
+struct task disktask;
+
 /* softraid callback, do not use! */
 void (*softraid_disk_attach)(struct disk *, int);
 
@@ -1034,6 +1036,7 @@ disk_init(void)
 
 	TAILQ_INIT(&disklist);
 	disk_count = disk_change = 0;
+	task_set(&disktask, disk_attach_callback, NULL, NULL);
 }
 
 int
@@ -1092,8 +1095,7 @@ disk_attach(struct device *dv, struct disk *diskp)
 			    MAKEDISKDEV(majdev, dv->dv_unit, RAW_PART);
 	}
 	if (diskp->dk_devno != NODEV)
-		workq_add_task(NULL, 0, disk_attach_callback,
-		    (void *)(long)(diskp->dk_devno), NULL);
+		task_add(systq, &disktask);
 
 	if (softraid_disk_attach)
 		softraid_disk_attach(diskp, 1);
@@ -1105,32 +1107,32 @@ disk_attach_callback(void *arg1, void *arg2)
 	char errbuf[100];
 	struct disklabel dl;
 	struct disk *dk;
-	dev_t dev = (dev_t)(long)arg1;
+	dev_t dev;
 
-	/* Locate disk associated with device no. */
+	/* Read disklabel(s) for newly-attached disk(s) */
 	TAILQ_FOREACH(dk, &disklist, dk_link) {
-		if (dk->dk_devno == dev)
-			break;
-	}
-	if (dk == NULL)
-		return;
 
-	/* XXX: Assumes dk is part of the device softc. */
-	device_ref(dk->dk_device);
+		if (dk->dk_devno == NODEV)
+			continue;
+
+		/* XXX: Assumes dk is part of the device softc. */
+		device_ref(dk->dk_device);
 
-	if (dk->dk_flags & (DKF_OPENED | DKF_NOLABELREAD))
-		goto done;
+		if (dk->dk_flags & (DKF_OPENED | DKF_NOLABELREAD))
+			goto done;
 
-	/* Read disklabel. */
-	if (disk_readlabel(&dl, dev, errbuf, sizeof(errbuf)) == NULL) {
-		add_timer_randomness(dl.d_checksum);
-		dk->dk_flags |= DKF_LABELVALID;
-	}
+		/* Read disklabel. */
+		dev = dk->dk_devno;
+		if (disk_readlabel(&dl, dev, errbuf, sizeof(errbuf)) == NULL) {
+			add_timer_randomness(dl.d_checksum);
+			dk->dk_flags |= DKF_LABELVALID;
+		}
 
 done:
-	dk->dk_flags |= DKF_OPENED;
-	device_unref(dk->dk_device);
-	wakeup(dk);
+		dk->dk_flags |= DKF_OPENED;
+		device_unref(dk->dk_device);
+		wakeup(dk);
+	}
 }
 
 /*