From owner-svn-src-head@freebsd.org Thu Mar 30 04:56:30 2017 Return-Path: Delivered-To: svn-src-head@mailman.ysv.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:1900:2254:206a::19:1]) by mailman.ysv.freebsd.org (Postfix) with ESMTP id 1BD7BD25CA4; Thu, 30 Mar 2017 04:56:30 +0000 (UTC) (envelope-from ngie@FreeBSD.org) Received: from repo.freebsd.org (repo.freebsd.org [IPv6:2610:1c1:1:6068::e6a:0]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (Client did not present a certificate) by mx1.freebsd.org (Postfix) with ESMTPS id DB0B3EC6; Thu, 30 Mar 2017 04:56:29 +0000 (UTC) (envelope-from ngie@FreeBSD.org) Received: from repo.freebsd.org ([127.0.1.37]) by repo.freebsd.org (8.15.2/8.15.2) with ESMTP id v2U4uTsv072043; Thu, 30 Mar 2017 04:56:29 GMT (envelope-from ngie@FreeBSD.org) Received: (from ngie@localhost) by repo.freebsd.org (8.15.2/8.15.2/Submit) id v2U4uRfo072031; Thu, 30 Mar 2017 04:56:27 GMT (envelope-from ngie@FreeBSD.org) Message-Id: <201703300456.v2U4uRfo072031@repo.freebsd.org> X-Authentication-Warning: repo.freebsd.org: ngie set sender to ngie@FreeBSD.org using -f From: Ngie Cooper Date: Thu, 30 Mar 2017 04:56:27 +0000 (UTC) To: src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org Subject: svn commit: r316212 - in head: share/man/man4 sys/cam/ctl sys/conf sys/modules sys/modules/cfiscsi sys/modules/ctl usr.sbin/ctladm usr.sbin/ctld X-SVN-Group: head MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit X-BeenThere: svn-src-head@freebsd.org X-Mailman-Version: 2.1.23 Precedence: list List-Id: SVN commit messages for the src tree for head/-current List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 30 Mar 2017 04:56:30 -0000 Author: ngie Date: Thu Mar 30 04:56:27 2017 New Revision: 316212 URL: https://svnweb.freebsd.org/changeset/base/316212 Log: Split iscsi(4) ctl frontend off of ctl(4) as cfiscsi(4) The goal of this work is to remove the explicit dependency for ctl(4) on iscsi(4), so end-users without iscsi(4) support in the kernel can use ctl(4) for its other functions. This allows those without iscsi(4) support built into the kernel to use ctl(4) as a test mechanism. As a sidenote, this was possible around the 10.0-RELEASE period, but made impossible for end-users without iscsi(4) between 10.0-RELEASE and 11.0-RELEASE. Automatically load cfiscsi(4) from ctladm(8) and ctld(8) for backwards compatibility with previously releases. The automatic loading feature is compiled into the beforementioned tools if MK_ISCSI == yes when building world. Add a manpage for cfiscsi(4) and refer to it in ctl(4). Differential Revision: D10099 MFC after: 2 months Relnotes: yes Reviewed by: mav, trasz Sponsored by: Dell EMC Isilon Added: head/share/man/man4/cfiscsi.4 - copied, changed from r315671, head/share/man/man4/ctl.4 head/sys/modules/cfiscsi/ - copied from r315671, head/sys/modules/ctl/ Modified: head/share/man/man4/Makefile head/share/man/man4/ctl.4 head/sys/cam/ctl/ctl_frontend_iscsi.c head/sys/conf/files head/sys/modules/Makefile head/sys/modules/cfiscsi/Makefile head/sys/modules/ctl/Makefile head/usr.sbin/ctladm/Makefile head/usr.sbin/ctladm/ctladm.c head/usr.sbin/ctld/Makefile head/usr.sbin/ctld/kernel.c Modified: head/share/man/man4/Makefile ============================================================================== --- head/share/man/man4/Makefile Thu Mar 30 04:21:02 2017 (r316211) +++ head/share/man/man4/Makefile Thu Mar 30 04:56:27 2017 (r316212) @@ -884,6 +884,7 @@ _dtrace_udp.4= dtrace_udp.4 .endif .if ${MK_ISCSI} != "no" +MAN+= cfiscsi.4 MAN+= iscsi.4 MAN+= iscsi_initiator.4 MAN+= iser.4 Copied and modified: head/share/man/man4/cfiscsi.4 (from r315671, head/share/man/man4/ctl.4) ============================================================================== --- head/share/man/man4/ctl.4 Tue Mar 21 08:57:20 2017 (r315671, copy source) +++ head/share/man/man4/cfiscsi.4 Thu Mar 30 04:56:27 2017 (r316212) @@ -1,5 +1,6 @@ .\" Copyright (c) 2013 Edward Tomasz Napierala .\" Copyright (c) 2015-2017 Alexander Motin +.\" Copyright (c) 2017 Ngie Cooper .\" All rights reserved. .\" .\" Redistribution and use in source and binary forms, with or without @@ -24,110 +25,35 @@ .\" SUCH DAMAGE. .\" .\" $FreeBSD$ -.Dd March 19, 2017 -.Dt CTL 4 +.Dd March 29, 2017 +.Dt CFISCSI 4 .Os .Sh NAME -.Nm ctl -.Nd CAM Target Layer / iSCSI target subsystem +.Nm cfiscsi +.Nd CAM Target Layer iSCSI target frontend .Sh SYNOPSIS To compile this driver into the kernel, -place the following line in your +place the following lines in your kernel configuration file: .Bd -ragged -offset indent -.Cd "device iscsi" +.Cd "device cfiscsi" .Cd "device ctl" +.Cd "device iscsi" .Ed .Pp Alternatively, to load the driver as a module at boot time, place the following line in .Xr loader.conf 5 : .Bd -literal -offset indent -ctl_load="YES" +cfiscsi_load="YES" .Ed .Sh DESCRIPTION The .Nm -subsystem provides SCSI target devices emulation. -It supports features such as: -.Pp -.Bl -bullet -compact -.It -Disk, CD-ROM and processor device emulation -.It -Tagged queueing -.It -SCSI task attribute support (ordered, head of queue, simple tags) -.It -SCSI implicit command ordering support -.It -Full task management support (abort, query, reset, etc.) -.It -Support for multiple ports, initiators, targets and backing stores -.It -Support for VMWare VAAI and Microsoft ODX offload (COMPARE AND WRITE, -XCOPY, POPULATE TOKEN/WRITE USING TOKEN, WRITE SAME and UNMAP) -.It -Persistent reservation support -.It -Extensive VPD/mode/log pages support -.It -Featured error reporting, error injection and basic SMART support -.It -High Availability clustering support with ALUA -.It -All I/O handled in-kernel, no userland context switch overhead -.El -.Pp -The -.Nm -subsystem includes multiple frontends to provide access using different -transport protocols and implementations: -.Bl -tag -width cfumass -.It camsim -Provides access for local system via virtual initiator mode -.Xr CAM 4 -SIM. -.It camtgt -Provides access for remote systems via target mode -.Xr CAM 4 -SIMs, such as Fibre Channel -.Xr isp 4 +subsystem provides iSCSI target device emulation via +.Xr ctl 4 and -.Xr mpt 4 . -.It cfumass -Provides access for remote systems via USB Mass Storage Class -Bulk Only (BBB) Transport. -.It ha -Internal frontend used to receive requests from other node ports in -High Availability cluster. -.It ioctl -Provides access for local user-level applications via -.Xr ioctl 2 -based API. -.It iscsi -Combined with -.Xr iscsi 4 -and -.Xr ctld 8 , -provides access for remote systems via iSCSI protocol. -.It tpc -Internal frontend used to receive requests from Third Party Copy engine, -implementing copy offload operations. -.El -.Pp -The -.Nm -subsystem includes two backends to create logical units using different -kinds of backing stores: -.Bl -tag -width ramdisk -.It block -Stores data in ZFS ZVOLs, files or raw block devices. -.It ramdisk -Stores data in RAM, that makes it mostly useful for performance testing. -Depending on configured capacity can work as black hole, thin or thick -provisioned disk. -.El +.Xr iscsi 4 . .Sh SYSCTL VARIABLES The following variables are available as both .Xr sysctl 8 @@ -135,65 +61,6 @@ variables and .Xr loader 8 tunables: .Bl -tag -width indent -.It Va kern.cam.ctl.debug -Bit mask of enabled CTL log levels: -.Bl -tag -offset indent -compact -.It 1 -log commands with errors; -.It 2 -log all commands; -.It 4 -log data for commands other then READ/WRITE. -.El -Defaults to 0. -.It Va kern.cam.ctl.ha_id -Specifies unique position of this node within High Availability cluster. -Default is 0 -- no HA, 1 and 2 -- HA enabled at specified position. -.It Va kern.cam.ctl.ha_mode -Specifies High Availability cluster operation mode: -.Bl -tag -offset indent -compact -.It 0 -Active/Standby -- primary node has backend access and processes requests, -while secondary can only do basic LUN discovery and reservation; -.It 1 -Active/Active -- both nodes have backend access and process requests, -while secondary node synchronizes processing with primary one; -.It 2 -Active/Active -- primary node has backend access and processes requests, -while secondary node forwards all requests and data to primary one; -.El -All above modes require established connection between HA cluster nodes. -If connection is not configured, secondary node will report Unavailable -state; if configured but not established -- Transitioning state. -Defaults to 0. -.It Va kern.cam.ctl.ha_peer -String value, specifying method to establish connection to peer HA node. -Can be "listen IP:port", "connect IP:port" or empty. -.It Va kern.cam.ctl.ha_link -Reports present state of connection between HA cluster nodes: -.Bl -tag -offset indent -compact -.It 0 -not configured; -.It 1 -configured but not established; -.It 2 -established. -.El -.It Va kern.cam.ctl.ha_role -Specifies default role of this node: -.Bl -tag -offset indent -compact -.It 0 -primary; -.It 1 -secondary. -.El -This role can be overridden on per-LUN basis using "ha_role" LUN option, -so that for one LUN one node is primary, while for another -- another. -Role change from primary to secondary for HA modes 0 and 2 closes backends, -the opposite change -- opens. -If there is no primary node (both nodes are secondary, or secondary node has -no connection to primary one), secondary node(s) report Transitioning state. -State with two primary nodes is illegal (split brain condition). .It Va kern.cam.ctl.iscsi.debug Verbosity level for log messages from the kernel part of iSCSI target. Set to 0 to disable logging or 1 to warn about potential problems. @@ -212,19 +79,26 @@ Set to 0 to disable sending NOP-In PDUs. Defaults to 5. .El .Sh SEE ALSO -.Xr cfumass 4 , -.Xr ctladm 8 , -.Xr ctld 8 , -.Xr ctlstat 8 +.Xr ctl 4 , +.Xr iscsi 4 .Sh HISTORY The .Nm subsystem first appeared in -.Fx 9.1 . +.Fx 10.0 +as part of the +.Xr ctl 4 +driver. +It was split off of +.Xr ctl 4 +in +.Fx 12.0 . .Sh AUTHORS +.An -nosplit The .Nm -subsystem was originally written by -.An Kenneth Merry Aq Mt ken@FreeBSD.org . -Later work was done by -.An Alexander Motin Aq Mt mav@FreeBSD.org . +subsystem was developed by +.An Edward Tomasz Napierala Aq Mt trasz@FreeBSD.org +under sponsorship from the FreeBSD Foundation. +This manual page was written by +.An Ngie Cooper Aq Mt ngie@FreeBSD.org . Modified: head/share/man/man4/ctl.4 ============================================================================== --- head/share/man/man4/ctl.4 Thu Mar 30 04:21:02 2017 (r316211) +++ head/share/man/man4/ctl.4 Thu Mar 30 04:56:27 2017 (r316212) @@ -24,18 +24,17 @@ .\" SUCH DAMAGE. .\" .\" $FreeBSD$ -.Dd March 19, 2017 +.Dd March 29, 2017 .Dt CTL 4 .Os .Sh NAME .Nm ctl -.Nd CAM Target Layer / iSCSI target subsystem +.Nd CAM Target Layer .Sh SYNOPSIS To compile this driver into the kernel, place the following line in your kernel configuration file: .Bd -ragged -offset indent -.Cd "device iscsi" .Cd "device ctl" .Ed .Pp @@ -106,11 +105,8 @@ Provides access for local user-level app .Xr ioctl 2 based API. .It iscsi -Combined with -.Xr iscsi 4 -and -.Xr ctld 8 , -provides access for remote systems via iSCSI protocol. +Provides access for remote systems via the iSCSI protocol using +.Xr cfiscsi 4 . .It tpc Internal frontend used to receive requests from Third Party Copy engine, implementing copy offload operations. @@ -194,24 +190,9 @@ the opposite change -- opens. If there is no primary node (both nodes are secondary, or secondary node has no connection to primary one), secondary node(s) report Transitioning state. State with two primary nodes is illegal (split brain condition). -.It Va kern.cam.ctl.iscsi.debug -Verbosity level for log messages from the kernel part of iSCSI target. -Set to 0 to disable logging or 1 to warn about potential problems. -Larger values enable debugging output. -Defaults to 1. -.It Va kern.cam.ctl.iscsi.maxtags -The number of outstanding commands to advertise to each iSCSI initiator. -Current implementation is not very accurate, so do not set this below 2. -Defaults to 256. -.It Va kern.cam.ctl.iscsi.ping_timeout -The number of seconds to wait for the iSCSI initiator to respond to a NOP-In -PDU. -In the event that there is no response within that time the session gets -forcibly terminated. -Set to 0 to disable sending NOP-In PDUs. -Defaults to 5. .El .Sh SEE ALSO +.Xr cfiscsi 4 , .Xr cfumass 4 , .Xr ctladm 8 , .Xr ctld 8 , Modified: head/sys/cam/ctl/ctl_frontend_iscsi.c ============================================================================== --- head/sys/cam/ctl/ctl_frontend_iscsi.c Thu Mar 30 04:21:02 2017 (r316211) +++ head/sys/cam/ctl/ctl_frontend_iscsi.c Thu Mar 30 04:56:27 2017 (r316212) @@ -184,8 +184,8 @@ static struct ctl_frontend cfiscsi_front .ioctl = cfiscsi_ioctl, .shutdown = cfiscsi_shutdown, }; -CTL_FRONTEND_DECLARE(ctlcfiscsi, cfiscsi_frontend); -MODULE_DEPEND(ctlcfiscsi, icl, 1, 1, 1); +CTL_FRONTEND_DECLARE(cfiscsi, cfiscsi_frontend); +MODULE_DEPEND(cfiscsi, icl, 1, 1, 1); static struct icl_pdu * cfiscsi_pdu_new_response(struct icl_pdu *request, int flags) Modified: head/sys/conf/files ============================================================================== --- head/sys/conf/files Thu Mar 30 04:21:02 2017 (r316211) +++ head/sys/conf/files Thu Mar 30 04:56:27 2017 (r316212) @@ -102,7 +102,7 @@ cam/ctl/ctl_cmd_table.c optional ctl cam/ctl/ctl_frontend.c optional ctl cam/ctl/ctl_frontend_cam_sim.c optional ctl cam/ctl/ctl_frontend_ioctl.c optional ctl -cam/ctl/ctl_frontend_iscsi.c optional ctl +cam/ctl/ctl_frontend_iscsi.c optional ctl cfiscsi cam/ctl/ctl_ha.c optional ctl cam/ctl/ctl_scsi_all.c optional ctl cam/ctl/ctl_tpc.c optional ctl @@ -1783,10 +1783,10 @@ ipw_monitor.fw optional ipwmonitorfw | compile-with "${NORMAL_FW}" \ no-obj no-implicit-rule \ clean "ipw_monitor.fw" -dev/iscsi/icl.c optional iscsi | ctl -dev/iscsi/icl_conn_if.m optional iscsi | ctl -dev/iscsi/icl_soft.c optional iscsi | ctl -dev/iscsi/icl_soft_proxy.c optional iscsi | ctl +dev/iscsi/icl.c optional iscsi +dev/iscsi/icl_conn_if.m optional cfiscsi | iscsi +dev/iscsi/icl_soft.c optional iscsi +dev/iscsi/icl_soft_proxy.c optional iscsi dev/iscsi/iscsi.c optional iscsi scbus dev/iscsi_initiator/iscsi.c optional iscsi_initiator scbus dev/iscsi_initiator/iscsi_subr.c optional iscsi_initiator scbus Modified: head/sys/modules/Makefile ============================================================================== --- head/sys/modules/Makefile Thu Mar 30 04:21:02 2017 (r316211) +++ head/sys/modules/Makefile Thu Mar 30 04:56:27 2017 (r316212) @@ -471,6 +471,7 @@ _ipfilter= ipfilter .endif .if ${MK_ISCSI} != "no" || defined(ALL_MODULES) +SUBDIR+= cfiscsi SUBDIR+= iscsi SUBDIR+= iscsi_initiator .endif Modified: head/sys/modules/cfiscsi/Makefile ============================================================================== --- head/sys/modules/ctl/Makefile Tue Mar 21 08:57:20 2017 (r315671) +++ head/sys/modules/cfiscsi/Makefile Thu Mar 30 04:56:27 2017 (r316212) @@ -2,32 +2,11 @@ .PATH: ${SRCTOP}/sys/cam/ctl -KMOD= ctl - -SRCS= ctl.c -SRCS+= ctl_backend.c -SRCS+= ctl_backend_block.c -SRCS+= ctl_backend_ramdisk.c -SRCS+= ctl_cmd_table.c -SRCS+= ctl_frontend.c -SRCS+= ctl_frontend_cam_sim.c -SRCS+= ctl_frontend_ioctl.c -SRCS+= ctl_frontend_iscsi.c -SRCS+= ctl_ha.c -SRCS+= ctl_scsi_all.c -SRCS+= ctl_tpc.c -SRCS+= ctl_tpc_local.c -SRCS+= ctl_error.c -SRCS+= ctl_util.c -SRCS+= scsi_ctl.c -SRCS+= bus_if.h -SRCS+= device_if.h -SRCS+= vnode_if.h -SRCS+= icl_conn_if.h -SRCS+= opt_cam.h +KMOD= cfiscsi #CFLAGS+=-DICL_KERNEL_PROXY -MFILES= kern/bus_if.m kern/device_if.m dev/iscsi/icl_conn_if.m +MFILES= dev/iscsi/icl_conn_if.m +SRCS= ctl_frontend_iscsi.c icl_conn_if.h .include Modified: head/sys/modules/ctl/Makefile ============================================================================== --- head/sys/modules/ctl/Makefile Thu Mar 30 04:21:02 2017 (r316211) +++ head/sys/modules/ctl/Makefile Thu Mar 30 04:56:27 2017 (r316212) @@ -12,7 +12,6 @@ SRCS+= ctl_cmd_table.c SRCS+= ctl_frontend.c SRCS+= ctl_frontend_cam_sim.c SRCS+= ctl_frontend_ioctl.c -SRCS+= ctl_frontend_iscsi.c SRCS+= ctl_ha.c SRCS+= ctl_scsi_all.c SRCS+= ctl_tpc.c @@ -23,11 +22,10 @@ SRCS+= scsi_ctl.c SRCS+= bus_if.h SRCS+= device_if.h SRCS+= vnode_if.h -SRCS+= icl_conn_if.h SRCS+= opt_cam.h #CFLAGS+=-DICL_KERNEL_PROXY -MFILES= kern/bus_if.m kern/device_if.m dev/iscsi/icl_conn_if.m +MFILES= kern/bus_if.m kern/device_if.m .include Modified: head/usr.sbin/ctladm/Makefile ============================================================================== --- head/usr.sbin/ctladm/Makefile Thu Mar 30 04:21:02 2017 (r316211) +++ head/usr.sbin/ctladm/Makefile Thu Mar 30 04:56:27 2017 (r316212) @@ -1,5 +1,7 @@ # $FreeBSD$ +.include + PROG= ctladm SRCS= ctladm.c util.c ctl_util.c ctl_scsi_all.c .PATH: ${SRCTOP}/sys/cam/ctl @@ -17,4 +19,8 @@ WARNS?= 3 LIBADD= cam sbuf bsdxml util MAN= ctladm.8 +.if ${MK_ISCSI} != "no" +CFLAGS+= -DWANT_ISCSI +.endif + .include Modified: head/usr.sbin/ctladm/ctladm.c ============================================================================== --- head/usr.sbin/ctladm/ctladm.c Thu Mar 30 04:21:02 2017 (r316211) +++ head/usr.sbin/ctladm/ctladm.c Thu Mar 30 04:56:27 2017 (r316212) @@ -47,6 +47,7 @@ __FBSDID("$FreeBSD$"); #include #include #include +#include #include #include #include @@ -4152,6 +4153,13 @@ main(int argc, char **argv) retval = 1; goto bailout; } +#ifdef WANT_ISCSI + else { + if (modfind("cfiscsi") == -1 && + kldload("cfiscsi") == -1) + warn("couldn't load cfiscsi"); + } +#endif } else if ((command != CTLADM_CMD_HELP) && ((cmdargs & CTLADM_ARG_DEVICE) == 0)) { fprintf(stderr, "%s: you must specify a device with the " Modified: head/usr.sbin/ctld/Makefile ============================================================================== --- head/usr.sbin/ctld/Makefile Thu Mar 30 04:21:02 2017 (r316211) +++ head/usr.sbin/ctld/Makefile Thu Mar 30 04:56:27 2017 (r316212) @@ -1,5 +1,7 @@ # $FreeBSD$ +.include + CFLAGS+=-I${SRCTOP}/contrib/libucl/include .PATH: ${SRCTOP}/contrib/libucl/include @@ -21,4 +23,8 @@ CLEANFILES= y.tab.c y.tab.h y.output WARNS?= 6 NO_WMISSING_VARIABLE_DECLARATIONS= +.if ${MK_ISCSI} != "no" +CFLAGS+= -DWANT_ISCSI +.endif + .include Modified: head/usr.sbin/ctld/kernel.c ============================================================================== --- head/usr.sbin/ctld/kernel.c Thu Mar 30 04:21:02 2017 (r316211) +++ head/usr.sbin/ctld/kernel.c Thu Mar 30 04:56:27 2017 (r316212) @@ -42,6 +42,7 @@ __FBSDID("$FreeBSD$"); #include #include #include +#include #include #include #include @@ -90,6 +91,14 @@ kernel_init(void) } if (ctl_fd < 0) log_err(1, "failed to open %s", CTL_DEFAULT_DEV); +#ifdef WANT_ISCSI + else { + saved_errno = errno; + if (modfind("cfiscsi") == -1 && kldload("cfiscsi") == -1) + log_warn("couldn't load cfiscsi"); + errno = saved_errno; + } +#endif } /*