Source-Changes-HG archive
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]
[src/trunk]: src/sys Split some common stuff into scsi_subr module. This ena...
details: https://anonhg.NetBSD.org/src/rev/c25625bf1b53
branches: trunk
changeset: 365216:c25625bf1b53
user: pgoyette <pgoyette%NetBSD.org@localhost>
date: Thu Apr 14 16:50:26 2022 +0000
description:
Split some common stuff into scsi_subr module. This enables loading
of the iscsi module whether or not there are any scsi things built
into the kernel.
Addresses the iscsi portion of kern/56772
diffstat:
sys/dev/iscsi/iscsi_main.c | 4 +-
sys/dev/scsipi/files.scsipi | 3 +-
sys/dev/scsipi/scsi_subr.c | 134 ++++++++++++++++++++++++++++++++++++
sys/dev/scsipi/scsiconf.c | 61 +---------------
sys/modules/Makefile | 5 +-
sys/modules/scsi_subr/Makefile | 10 ++
sys/rump/dev/lib/libscsipi/Makefile | 4 +-
7 files changed, 155 insertions(+), 66 deletions(-)
diffs (truncated from 315 to 300 lines):
diff -r d9677144f3b0 -r c25625bf1b53 sys/dev/iscsi/iscsi_main.c
--- a/sys/dev/iscsi/iscsi_main.c Thu Apr 14 15:51:29 2022 +0000
+++ b/sys/dev/iscsi/iscsi_main.c Thu Apr 14 16:50:26 2022 +0000
@@ -1,4 +1,4 @@
-/* $NetBSD: iscsi_main.c,v 1.39 2022/03/31 19:30:16 pgoyette Exp $ */
+/* $NetBSD: iscsi_main.c,v 1.40 2022/04/14 16:50:26 pgoyette Exp $ */
/*-
* Copyright (c) 2004,2005,2006,2011 The NetBSD Foundation, Inc.
@@ -656,7 +656,7 @@
#include <sys/module.h>
-MODULE(MODULE_CLASS_DRIVER, iscsi, NULL); /* Possibly a builtin module */
+MODULE(MODULE_CLASS_DRIVER, iscsi, "scsi_subr"); /* Possibly a builtin module */
#ifdef _MODULE
static const struct cfiattrdata ibescsi_info = { "scsi", 1,
diff -r d9677144f3b0 -r c25625bf1b53 sys/dev/scsipi/files.scsipi
--- a/sys/dev/scsipi/files.scsipi Thu Apr 14 15:51:29 2022 +0000
+++ b/sys/dev/scsipi/files.scsipi Thu Apr 14 16:50:26 2022 +0000
@@ -1,4 +1,4 @@
-# $NetBSD: files.scsipi,v 1.42 2019/02/12 13:49:13 kardel Exp $
+# $NetBSD: files.scsipi,v 1.43 2022/04/14 16:50:26 pgoyette Exp $
#
# Config file and device description for machine-independent SCSI code.
# Included by ports that need it. Ports that use it must provide
@@ -26,6 +26,7 @@
device scsibus {[target = -1], [lun = -1]}: scsi_core
attach scsibus at scsi
file dev/scsipi/scsiconf.c scsibus needs-flag
+file dev/scsipi/scsi_subr.c scsibus
device atapibus {[drive = -1]}
attach atapibus at atapi
diff -r d9677144f3b0 -r c25625bf1b53 sys/dev/scsipi/scsi_subr.c
--- /dev/null Thu Jan 01 00:00:00 1970 +0000
+++ b/sys/dev/scsipi/scsi_subr.c Thu Apr 14 16:50:26 2022 +0000
@@ -0,0 +1,134 @@
+/* $NetBSD: scsi_subr.c,v 1.1 2022/04/14 16:50:26 pgoyette Exp $ */
+
+/*-
+ * Copyright (c) 1998, 1999, 2004 The NetBSD Foundation, Inc.
+ * All rights reserved.
+ *
+ * This code is derived from software contributed to The NetBSD Foundation
+ * by Charles M. Hannum; Jason R. Thorpe of the Numerical Aerospace
+ * Simulation Facility, NASA Ames Research Center.
+ *
+ * Redistribution and use in source and binary forms, with or without
+ * modification, are permitted provided that the following conditions
+ * are met:
+ * 1. Redistributions of source code must retain the above copyright
+ * notice, this list of conditions and the following disclaimer.
+ * 2. Redistributions in binary form must reproduce the above copyright
+ * notice, this list of conditions and the following disclaimer in the
+ * documentation and/or other materials provided with the distribution.
+ *
+ * THIS SOFTWARE IS PROVIDED BY THE NETBSD FOUNDATION, INC. AND CONTRIBUTORS
+ * ``AS IS'' AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED
+ * TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR
+ * PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE FOUNDATION OR CONTRIBUTORS
+ * BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR
+ * CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF
+ * SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS
+ * INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN
+ * CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE)
+ * ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE
+ * POSSIBILITY OF SUCH DAMAGE.
+ */
+
+/*
+ * Originally written by Julian Elischer (julian%tfs.com@localhost)
+ * for TRW Financial Systems for use under the MACH(2.5) operating system.
+ *
+ * TRW Financial Systems, in accordance with their agreement with Carnegie
+ * Mellon University, makes this software available to CMU to distribute
+ * or use in any manner that they see fit as long as this message is kept with
+ * the software. For this reason TFS also grants any other persons or
+ * organisations permission to use or modify this software.
+ *
+ * TFS supplies this software to be publicly redistributed
+ * on the understanding that TFS is not responsible for the correct
+ * functioning of this software in any circumstances.
+ *
+ * Ported to run under 386BSD by Julian Elischer (julian%tfs.com@localhost) Sept 1992
+ */
+
+#include <sys/cdefs.h>
+__KERNEL_RCSID(0, "$NetBSD: scsi_subr.c,v 1.1 2022/04/14 16:50:26 pgoyette Exp $");
+
+#include <sys/param.h>
+#include <sys/systm.h>
+
+#include <sys/device.h>
+#include <sys/kernel.h>
+#include <sys/proc.h>
+#include <sys/module.h>
+
+#include <dev/scsipi/scsi_all.h>
+#include <dev/scsipi/scsipi_all.h>
+#include <dev/scsipi/scsiconf.h>
+
+const struct scsipi_bustype scsi_bustype = {
+ .bustype_type = SCSIPI_BUSTYPE_BUSTYPE(SCSIPI_BUSTYPE_SCSI,
+ SCSIPI_BUSTYPE_SCSI_PSCSI),
+ .bustype_cmd = scsi_scsipi_cmd,
+ .bustype_interpret_sense = scsipi_interpret_sense,
+ .bustype_printaddr = scsi_print_addr,
+ .bustype_kill_pending = scsi_kill_pending,
+ .bustype_async_event_xfer_mode = scsi_async_event_xfer_mode,
+};
+
+const struct scsipi_bustype scsi_fc_bustype = {
+ .bustype_type = SCSIPI_BUSTYPE_BUSTYPE(SCSIPI_BUSTYPE_SCSI,
+ SCSIPI_BUSTYPE_SCSI_FC),
+ .bustype_cmd = scsi_scsipi_cmd,
+ .bustype_interpret_sense = scsipi_interpret_sense,
+ .bustype_printaddr = scsi_print_addr,
+ .bustype_kill_pending = scsi_kill_pending,
+ .bustype_async_event_xfer_mode = scsi_fc_sas_async_event_xfer_mode,
+};
+
+const struct scsipi_bustype scsi_sas_bustype = {
+ .bustype_type = SCSIPI_BUSTYPE_BUSTYPE(SCSIPI_BUSTYPE_SCSI,
+ SCSIPI_BUSTYPE_SCSI_SAS),
+ .bustype_cmd = scsi_scsipi_cmd,
+ .bustype_interpret_sense = scsipi_interpret_sense,
+ .bustype_printaddr = scsi_print_addr,
+ .bustype_kill_pending = scsi_kill_pending,
+ .bustype_async_event_xfer_mode = scsi_fc_sas_async_event_xfer_mode,
+};
+
+const struct scsipi_bustype scsi_usb_bustype = {
+ .bustype_type = SCSIPI_BUSTYPE_BUSTYPE(SCSIPI_BUSTYPE_SCSI,
+ SCSIPI_BUSTYPE_SCSI_USB),
+ .bustype_cmd = scsi_scsipi_cmd,
+ .bustype_interpret_sense = scsipi_interpret_sense,
+ .bustype_printaddr = scsi_print_addr,
+ .bustype_kill_pending = scsi_kill_pending,
+ .bustype_async_event_xfer_mode = NULL,
+};
+
+int
+scsiprint(void *aux, const char *pnp)
+{
+ struct scsipi_channel *chan = aux;
+ struct scsipi_adapter *adapt = chan->chan_adapter;
+
+ /* only "scsibus"es can attach to "scsi"s; easy. */
+ if (pnp)
+ aprint_normal("scsibus at %s", pnp);
+
+ /* don't print channel if the controller says there can be only one. */
+ if (adapt->adapt_nchannels != 1)
+ aprint_normal(" channel %d", chan->chan_channel);
+
+ return (UNCONF);
+}
+
+MODULE(MODULE_CLASS_EXEC, scsi_subr, NULL);
+
+static int scsi_subr_modcmd(modcmd_t cmd, void *opaque)
+{
+
+ switch(cmd) {
+ case MODULE_CMD_INIT:
+ case MODULE_CMD_FINI:
+ return 0;
+ default:
+ return ENOTTY;
+ }
+}
diff -r d9677144f3b0 -r c25625bf1b53 sys/dev/scsipi/scsiconf.c
--- a/sys/dev/scsipi/scsiconf.c Thu Apr 14 15:51:29 2022 +0000
+++ b/sys/dev/scsipi/scsiconf.c Thu Apr 14 16:50:26 2022 +0000
@@ -1,4 +1,4 @@
-/* $NetBSD: scsiconf.c,v 1.301 2022/04/09 23:38:32 riastradh Exp $ */
+/* $NetBSD: scsiconf.c,v 1.302 2022/04/14 16:50:26 pgoyette Exp $ */
/*-
* Copyright (c) 1998, 1999, 2004 The NetBSD Foundation, Inc.
@@ -48,7 +48,7 @@
*/
#include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: scsiconf.c,v 1.301 2022/04/09 23:38:32 riastradh Exp $");
+__KERNEL_RCSID(0, "$NetBSD: scsiconf.c,v 1.302 2022/04/14 16:50:26 pgoyette Exp $");
#include <sys/param.h>
#include <sys/systm.h>
@@ -126,46 +126,6 @@
static void scsibus_discover_thread(void *);
static void scsibus_config(struct scsibus_softc *);
-const struct scsipi_bustype scsi_bustype = {
- .bustype_type = SCSIPI_BUSTYPE_BUSTYPE(SCSIPI_BUSTYPE_SCSI,
- SCSIPI_BUSTYPE_SCSI_PSCSI),
- .bustype_cmd = scsi_scsipi_cmd,
- .bustype_interpret_sense = scsipi_interpret_sense,
- .bustype_printaddr = scsi_print_addr,
- .bustype_kill_pending = scsi_kill_pending,
- .bustype_async_event_xfer_mode = scsi_async_event_xfer_mode,
-};
-
-const struct scsipi_bustype scsi_fc_bustype = {
- .bustype_type = SCSIPI_BUSTYPE_BUSTYPE(SCSIPI_BUSTYPE_SCSI,
- SCSIPI_BUSTYPE_SCSI_FC),
- .bustype_cmd = scsi_scsipi_cmd,
- .bustype_interpret_sense = scsipi_interpret_sense,
- .bustype_printaddr = scsi_print_addr,
- .bustype_kill_pending = scsi_kill_pending,
- .bustype_async_event_xfer_mode = scsi_fc_sas_async_event_xfer_mode,
-};
-
-const struct scsipi_bustype scsi_sas_bustype = {
- .bustype_type = SCSIPI_BUSTYPE_BUSTYPE(SCSIPI_BUSTYPE_SCSI,
- SCSIPI_BUSTYPE_SCSI_SAS),
- .bustype_cmd = scsi_scsipi_cmd,
- .bustype_interpret_sense = scsipi_interpret_sense,
- .bustype_printaddr = scsi_print_addr,
- .bustype_kill_pending = scsi_kill_pending,
- .bustype_async_event_xfer_mode = scsi_fc_sas_async_event_xfer_mode,
-};
-
-const struct scsipi_bustype scsi_usb_bustype = {
- .bustype_type = SCSIPI_BUSTYPE_BUSTYPE(SCSIPI_BUSTYPE_SCSI,
- SCSIPI_BUSTYPE_SCSI_USB),
- .bustype_cmd = scsi_scsipi_cmd,
- .bustype_interpret_sense = scsipi_interpret_sense,
- .bustype_printaddr = scsi_print_addr,
- .bustype_kill_pending = scsi_kill_pending,
- .bustype_async_event_xfer_mode = NULL,
-};
-
static int
scsibus_init(void)
{
@@ -176,23 +136,6 @@
return 0;
}
-int
-scsiprint(void *aux, const char *pnp)
-{
- struct scsipi_channel *chan = aux;
- struct scsipi_adapter *adapt = chan->chan_adapter;
-
- /* only "scsibus"es can attach to "scsi"s; easy. */
- if (pnp)
- aprint_normal("scsibus at %s", pnp);
-
- /* don't print channel if the controller says there can be only one. */
- if (adapt->adapt_nchannels != 1)
- aprint_normal(" channel %d", chan->chan_channel);
-
- return (UNCONF);
-}
-
static int
scsibusmatch(device_t parent, cfdata_t cf, void *aux)
{
diff -r d9677144f3b0 -r c25625bf1b53 sys/modules/Makefile
--- a/sys/modules/Makefile Thu Apr 14 15:51:29 2022 +0000
+++ b/sys/modules/Makefile Thu Apr 14 16:50:26 2022 +0000
@@ -1,4 +1,4 @@
-# $NetBSD: Makefile,v 1.264 2022/04/01 21:09:24 pgoyette Exp $
+# $NetBSD: Makefile,v 1.265 2022/04/14 16:50:26 pgoyette Exp $
.include <bsd.own.mk>
@@ -146,9 +146,10 @@
SUBDIR+= puffs
SUBDIR+= putter
SUBDIR+= raid
-SUBDIR+= scsiverbose
SUBDIR+= scmd
SUBDIR+= scmdi2c
+SUBDIR+= scsi_subr
+SUBDIR+= scsiverbose
SUBDIR+= sdtemp
SUBDIR+= secmodel_bsd44
SUBDIR+= secmodel_extensions
diff -r d9677144f3b0 -r c25625bf1b53 sys/modules/scsi_subr/Makefile
--- /dev/null Thu Jan 01 00:00:00 1970 +0000
+++ b/sys/modules/scsi_subr/Makefile Thu Apr 14 16:50:26 2022 +0000
@@ -0,0 +1,10 @@
+# $NetBSD: Makefile,v 1.1 2022/04/14 16:50:26 pgoyette Exp $
+
+.include "../Makefile.inc"
+
+.PATH: ${S}/dev/scsipi/
+
+KMOD= scsi_subr
+SRCS= scsi_subr.c scsi_base.c
+
+.include <bsd.kmodule.mk>
diff -r d9677144f3b0 -r c25625bf1b53 sys/rump/dev/lib/libscsipi/Makefile
--- a/sys/rump/dev/lib/libscsipi/Makefile Thu Apr 14 15:51:29 2022 +0000
+++ b/sys/rump/dev/lib/libscsipi/Makefile Thu Apr 14 16:50:26 2022 +0000
Home |
Main Index |
Thread Index |
Old Index