提交 ba929992 编写于 作者: B Bart Van Assche 提交者: James Bottomley

target: Minimize SCSI header #include directives

Only include SCSI initiator header files in target code that needs
these header files, namely the SCSI pass-through code and the tcm_loop
driver. Change SCSI_SENSE_BUFFERSIZE into TRANSPORT_SENSE_BUFFER in
target code because the former is intended for initiator code and the
latter for target code. With this patch the only initiator include
directives in target code that remain are as follows:

$ git grep -nHE 'include .scsi/(scsi.h|scsi_host.h|scsi_device.h|scsi_cmnd.h)' drivers/target drivers/infiniband/ulp/{isert,srpt} drivers/usb/gadget/legacy/tcm_*.[ch] drivers/{vhost,xen} include/{target,trace/events/target.h}
drivers/target/loopback/tcm_loop.c:29:#include <scsi/scsi.h>
drivers/target/loopback/tcm_loop.c:31:#include <scsi/scsi_host.h>
drivers/target/loopback/tcm_loop.c:32:#include <scsi/scsi_device.h>
drivers/target/loopback/tcm_loop.c:33:#include <scsi/scsi_cmnd.h>
drivers/target/target_core_pscsi.c:39:#include <scsi/scsi_device.h>
drivers/target/target_core_pscsi.c:40:#include <scsi/scsi_host.h>
drivers/xen/xen-scsiback.c:52:#include <scsi/scsi_host.h> /* SG_ALL */
Signed-off-by: NBart Van Assche <bart.vanassche@sandisk.com>
Reviewed-by: NChristoph Hellwig <hch@lst.de>
Signed-off-by: NJames Bottomley <JBottomley@Odin.com>
上级 8d3a8248
...@@ -503,11 +503,8 @@ def tcm_mod_dump_fabric_ops(proto_ident, fabric_mod_dir_var, fabric_mod_name): ...@@ -503,11 +503,8 @@ def tcm_mod_dump_fabric_ops(proto_ident, fabric_mod_dir_var, fabric_mod_name):
buf += "#include <linux/string.h>\n" buf += "#include <linux/string.h>\n"
buf += "#include <linux/ctype.h>\n" buf += "#include <linux/ctype.h>\n"
buf += "#include <asm/unaligned.h>\n" buf += "#include <asm/unaligned.h>\n"
buf += "#include <scsi/scsi.h>\n" buf += "#include <scsi/scsi_common.h>\n"
buf += "#include <scsi/scsi_host.h>\n" buf += "#include <scsi/scsi_proto.h>\n"
buf += "#include <scsi/scsi_device.h>\n"
buf += "#include <scsi/scsi_cmnd.h>\n"
buf += "#include <scsi/libfc.h>\n\n"
buf += "#include <target/target_core_base.h>\n" buf += "#include <target/target_core_base.h>\n"
buf += "#include <target/target_core_fabric.h>\n" buf += "#include <target/target_core_fabric.h>\n"
buf += "#include <target/target_core_configfs.h>\n\n" buf += "#include <target/target_core_configfs.h>\n\n"
......
...@@ -41,6 +41,7 @@ ...@@ -41,6 +41,7 @@
#include <linux/string.h> #include <linux/string.h>
#include <linux/delay.h> #include <linux/delay.h>
#include <linux/atomic.h> #include <linux/atomic.h>
#include <scsi/scsi_proto.h>
#include <scsi/scsi_tcq.h> #include <scsi/scsi_tcq.h>
#include <target/configfs_macros.h> #include <target/configfs_macros.h>
#include <target/target_core_base.h> #include <target/target_core_base.h>
......
...@@ -245,7 +245,7 @@ struct srpt_send_ioctx { ...@@ -245,7 +245,7 @@ struct srpt_send_ioctx {
u8 n_rdma; u8 n_rdma;
u8 n_rbuf; u8 n_rbuf;
bool queue_status_only; bool queue_status_only;
u8 sense_data[SCSI_SENSE_BUFFERSIZE]; u8 sense_data[TRANSPORT_SENSE_BUFFER];
}; };
/** /**
......
...@@ -23,7 +23,7 @@ ...@@ -23,7 +23,7 @@
#include <linux/module.h> #include <linux/module.h>
#include <linux/idr.h> #include <linux/idr.h>
#include <asm/unaligned.h> #include <asm/unaligned.h>
#include <scsi/scsi_device.h> #include <scsi/scsi_proto.h>
#include <scsi/iscsi_proto.h> #include <scsi/iscsi_proto.h>
#include <scsi/scsi_tcq.h> #include <scsi/scsi_tcq.h>
#include <target/target_core_base.h> #include <target/target_core_base.h>
......
...@@ -17,7 +17,6 @@ ...@@ -17,7 +17,6 @@
* GNU General Public License for more details. * GNU General Public License for more details.
******************************************************************************/ ******************************************************************************/
#include <scsi/scsi_device.h>
#include <target/target_core_base.h> #include <target/target_core_base.h>
#include <target/target_core_fabric.h> #include <target/target_core_fabric.h>
......
...@@ -17,7 +17,7 @@ ...@@ -17,7 +17,7 @@
******************************************************************************/ ******************************************************************************/
#include <asm/unaligned.h> #include <asm/unaligned.h>
#include <scsi/scsi_device.h> #include <scsi/scsi_proto.h>
#include <scsi/iscsi_proto.h> #include <scsi/iscsi_proto.h>
#include <target/target_core_base.h> #include <target/target_core_base.h>
#include <target/target_core_fabric.h> #include <target/target_core_fabric.h>
......
...@@ -30,7 +30,7 @@ ...@@ -30,7 +30,7 @@
#include <linux/ctype.h> #include <linux/ctype.h>
#include <linux/firewire.h> #include <linux/firewire.h>
#include <linux/firewire-constants.h> #include <linux/firewire-constants.h>
#include <scsi/scsi.h> #include <scsi/scsi_proto.h>
#include <scsi/scsi_tcq.h> #include <scsi/scsi_tcq.h>
#include <target/target_core_base.h> #include <target/target_core_base.h>
#include <target/target_core_backend.h> #include <target/target_core_backend.h>
......
...@@ -28,8 +28,7 @@ ...@@ -28,8 +28,7 @@
#include <linux/configfs.h> #include <linux/configfs.h>
#include <linux/export.h> #include <linux/export.h>
#include <linux/file.h> #include <linux/file.h>
#include <scsi/scsi.h> #include <scsi/scsi_proto.h>
#include <scsi/scsi_cmnd.h>
#include <asm/unaligned.h> #include <asm/unaligned.h>
#include <target/target_core_base.h> #include <target/target_core_base.h>
......
...@@ -35,8 +35,8 @@ ...@@ -35,8 +35,8 @@
#include <linux/export.h> #include <linux/export.h>
#include <net/sock.h> #include <net/sock.h>
#include <net/tcp.h> #include <net/tcp.h>
#include <scsi/scsi.h> #include <scsi/scsi_common.h>
#include <scsi/scsi_device.h> #include <scsi/scsi_proto.h>
#include <target/target_core_base.h> #include <target/target_core_base.h>
#include <target/target_core_backend.h> #include <target/target_core_backend.h>
......
...@@ -29,8 +29,6 @@ ...@@ -29,8 +29,6 @@
#include <linux/ctype.h> #include <linux/ctype.h>
#include <linux/spinlock.h> #include <linux/spinlock.h>
#include <linux/export.h> #include <linux/export.h>
#include <scsi/scsi.h>
#include <scsi/scsi_cmnd.h>
#include <target/target_core_base.h> #include <target/target_core_base.h>
#include <target/target_core_fabric.h> #include <target/target_core_fabric.h>
......
...@@ -31,8 +31,7 @@ ...@@ -31,8 +31,7 @@
#include <linux/spinlock.h> #include <linux/spinlock.h>
#include <linux/module.h> #include <linux/module.h>
#include <linux/falloc.h> #include <linux/falloc.h>
#include <scsi/scsi.h> #include <scsi/scsi_proto.h>
#include <scsi/scsi_host.h>
#include <asm/unaligned.h> #include <asm/unaligned.h>
#include <target/target_core_base.h> #include <target/target_core_base.h>
......
...@@ -35,8 +35,7 @@ ...@@ -35,8 +35,7 @@
#include <linux/genhd.h> #include <linux/genhd.h>
#include <linux/file.h> #include <linux/file.h>
#include <linux/module.h> #include <linux/module.h>
#include <scsi/scsi.h> #include <scsi/scsi_proto.h>
#include <scsi/scsi_host.h>
#include <asm/unaligned.h> #include <asm/unaligned.h>
#include <target/target_core_base.h> #include <target/target_core_base.h>
......
...@@ -28,8 +28,7 @@ ...@@ -28,8 +28,7 @@
#include <linux/spinlock.h> #include <linux/spinlock.h>
#include <linux/list.h> #include <linux/list.h>
#include <linux/file.h> #include <linux/file.h>
#include <scsi/scsi.h> #include <scsi/scsi_proto.h>
#include <scsi/scsi_cmnd.h>
#include <asm/unaligned.h> #include <asm/unaligned.h>
#include <target/target_core_base.h> #include <target/target_core_base.h>
......
...@@ -36,9 +36,7 @@ ...@@ -36,9 +36,7 @@
#include <linux/module.h> #include <linux/module.h>
#include <asm/unaligned.h> #include <asm/unaligned.h>
#include <scsi/scsi.h>
#include <scsi/scsi_device.h> #include <scsi/scsi_device.h>
#include <scsi/scsi_cmnd.h>
#include <scsi/scsi_host.h> #include <scsi/scsi_host.h>
#include <scsi/scsi_tcq.h> #include <scsi/scsi_tcq.h>
......
...@@ -16,13 +16,13 @@ ...@@ -16,13 +16,13 @@
#define PS_TIMEOUT_OTHER (500*HZ) #define PS_TIMEOUT_OTHER (500*HZ)
#include <linux/device.h> #include <linux/device.h>
#include <scsi/scsi_driver.h>
#include <scsi/scsi_device.h>
#include <linux/kref.h> #include <linux/kref.h>
#include <linux/kobject.h> #include <linux/kobject.h>
struct scsi_device;
struct pscsi_plugin_task { struct pscsi_plugin_task {
unsigned char pscsi_sense[SCSI_SENSE_BUFFERSIZE]; unsigned char pscsi_sense[TRANSPORT_SENSE_BUFFER];
int pscsi_direction; int pscsi_direction;
int pscsi_result; int pscsi_result;
u32 pscsi_resid; u32 pscsi_resid;
......
...@@ -29,8 +29,7 @@ ...@@ -29,8 +29,7 @@
#include <linux/timer.h> #include <linux/timer.h>
#include <linux/slab.h> #include <linux/slab.h>
#include <linux/spinlock.h> #include <linux/spinlock.h>
#include <scsi/scsi.h> #include <scsi/scsi_proto.h>
#include <scsi/scsi_host.h>
#include <target/target_core_base.h> #include <target/target_core_base.h>
#include <target/target_core_backend.h> #include <target/target_core_backend.h>
......
...@@ -25,7 +25,7 @@ ...@@ -25,7 +25,7 @@
#include <linux/ratelimit.h> #include <linux/ratelimit.h>
#include <linux/crc-t10dif.h> #include <linux/crc-t10dif.h>
#include <asm/unaligned.h> #include <asm/unaligned.h>
#include <scsi/scsi.h> #include <scsi/scsi_proto.h>
#include <scsi/scsi_tcq.h> #include <scsi/scsi_tcq.h>
#include <target/target_core_base.h> #include <target/target_core_base.h>
......
...@@ -24,7 +24,8 @@ ...@@ -24,7 +24,8 @@
#include <linux/module.h> #include <linux/module.h>
#include <asm/unaligned.h> #include <asm/unaligned.h>
#include <scsi/scsi.h> #include <scsi/scsi_proto.h>
#include <scsi/scsi_common.h>
#include <scsi/scsi_tcq.h> #include <scsi/scsi_tcq.h>
#include <target/target_core_base.h> #include <target/target_core_base.h>
......
...@@ -33,9 +33,6 @@ ...@@ -33,9 +33,6 @@
#include <linux/proc_fs.h> #include <linux/proc_fs.h>
#include <linux/seq_file.h> #include <linux/seq_file.h>
#include <linux/configfs.h> #include <linux/configfs.h>
#include <scsi/scsi.h>
#include <scsi/scsi_device.h>
#include <scsi/scsi_host.h>
#include <target/target_core_base.h> #include <target/target_core_base.h>
#include <target/target_core_backend.h> #include <target/target_core_backend.h>
......
...@@ -27,8 +27,6 @@ ...@@ -27,8 +27,6 @@
#include <linux/spinlock.h> #include <linux/spinlock.h>
#include <linux/list.h> #include <linux/list.h>
#include <linux/export.h> #include <linux/export.h>
#include <scsi/scsi.h>
#include <scsi/scsi_cmnd.h>
#include <target/target_core_base.h> #include <target/target_core_base.h>
#include <target/target_core_backend.h> #include <target/target_core_backend.h>
......
...@@ -32,8 +32,7 @@ ...@@ -32,8 +32,7 @@
#include <linux/export.h> #include <linux/export.h>
#include <net/sock.h> #include <net/sock.h>
#include <net/tcp.h> #include <net/tcp.h>
#include <scsi/scsi.h> #include <scsi/scsi_proto.h>
#include <scsi/scsi_cmnd.h>
#include <target/target_core_base.h> #include <target/target_core_base.h>
#include <target/target_core_backend.h> #include <target/target_core_backend.h>
......
...@@ -37,9 +37,7 @@ ...@@ -37,9 +37,7 @@
#include <asm/unaligned.h> #include <asm/unaligned.h>
#include <net/sock.h> #include <net/sock.h>
#include <net/tcp.h> #include <net/tcp.h>
#include <scsi/scsi.h> #include <scsi/scsi_proto.h>
#include <scsi/scsi_cmnd.h>
#include <scsi/scsi_tcq.h>
#include <target/target_core_base.h> #include <target/target_core_base.h>
#include <target/target_core_backend.h> #include <target/target_core_backend.h>
......
...@@ -25,8 +25,7 @@ ...@@ -25,8 +25,7 @@
#include <linux/slab.h> #include <linux/slab.h>
#include <linux/spinlock.h> #include <linux/spinlock.h>
#include <scsi/scsi.h> #include <scsi/scsi_proto.h>
#include <scsi/scsi_cmnd.h>
#include <target/target_core_base.h> #include <target/target_core_base.h>
#include <target/target_core_fabric.h> #include <target/target_core_fabric.h>
......
...@@ -19,12 +19,13 @@ ...@@ -19,12 +19,13 @@
#include <linux/spinlock.h> #include <linux/spinlock.h>
#include <linux/module.h> #include <linux/module.h>
#include <linux/idr.h> #include <linux/idr.h>
#include <linux/kernel.h>
#include <linux/timer.h> #include <linux/timer.h>
#include <linux/parser.h> #include <linux/parser.h>
#include <scsi/scsi.h>
#include <scsi/scsi_host.h>
#include <linux/uio_driver.h> #include <linux/uio_driver.h>
#include <net/genetlink.h> #include <net/genetlink.h>
#include <scsi/scsi_common.h>
#include <scsi/scsi_proto.h>
#include <target/target_core_base.h> #include <target/target_core_base.h>
#include <target/target_core_fabric.h> #include <target/target_core_fabric.h>
#include <target/target_core_backend.h> #include <target/target_core_backend.h>
......
...@@ -25,8 +25,7 @@ ...@@ -25,8 +25,7 @@
#include <linux/spinlock.h> #include <linux/spinlock.h>
#include <linux/list.h> #include <linux/list.h>
#include <linux/configfs.h> #include <linux/configfs.h>
#include <scsi/scsi.h> #include <scsi/scsi_proto.h>
#include <scsi/scsi_cmnd.h>
#include <asm/unaligned.h> #include <asm/unaligned.h>
#include <target/target_core_base.h> #include <target/target_core_base.h>
......
...@@ -30,10 +30,6 @@ ...@@ -30,10 +30,6 @@
#include <linux/hash.h> #include <linux/hash.h>
#include <linux/percpu_ida.h> #include <linux/percpu_ida.h>
#include <asm/unaligned.h> #include <asm/unaligned.h>
#include <scsi/scsi.h>
#include <scsi/scsi_host.h>
#include <scsi/scsi_device.h>
#include <scsi/scsi_cmnd.h>
#include <scsi/scsi_tcq.h> #include <scsi/scsi_tcq.h>
#include <scsi/libfc.h> #include <scsi/libfc.h>
#include <scsi/fc_encode.h> #include <scsi/fc_encode.h>
......
...@@ -34,10 +34,6 @@ ...@@ -34,10 +34,6 @@
#include <linux/kernel.h> #include <linux/kernel.h>
#include <linux/ctype.h> #include <linux/ctype.h>
#include <asm/unaligned.h> #include <asm/unaligned.h>
#include <scsi/scsi.h>
#include <scsi/scsi_host.h>
#include <scsi/scsi_device.h>
#include <scsi/scsi_cmnd.h>
#include <scsi/libfc.h> #include <scsi/libfc.h>
#include <target/target_core_base.h> #include <target/target_core_base.h>
......
...@@ -39,10 +39,6 @@ ...@@ -39,10 +39,6 @@
#include <linux/hash.h> #include <linux/hash.h>
#include <linux/ratelimit.h> #include <linux/ratelimit.h>
#include <asm/unaligned.h> #include <asm/unaligned.h>
#include <scsi/scsi.h>
#include <scsi/scsi_host.h>
#include <scsi/scsi_device.h>
#include <scsi/scsi_cmnd.h>
#include <scsi/libfc.h> #include <scsi/libfc.h>
#include <scsi/fc_encode.h> #include <scsi/fc_encode.h>
......
...@@ -32,10 +32,6 @@ ...@@ -32,10 +32,6 @@
#include <linux/rculist.h> #include <linux/rculist.h>
#include <linux/kref.h> #include <linux/kref.h>
#include <asm/unaligned.h> #include <asm/unaligned.h>
#include <scsi/scsi.h>
#include <scsi/scsi_host.h>
#include <scsi/scsi_device.h>
#include <scsi/scsi_cmnd.h>
#include <scsi/libfc.h> #include <scsi/libfc.h>
#include <target/target_core_base.h> #include <target/target_core_base.h>
......
...@@ -16,7 +16,6 @@ ...@@ -16,7 +16,6 @@
#include <linux/usb/composite.h> #include <linux/usb/composite.h>
#include <linux/usb/gadget.h> #include <linux/usb/gadget.h>
#include <linux/usb/storage.h> #include <linux/usb/storage.h>
#include <scsi/scsi.h>
#include <scsi/scsi_tcq.h> #include <scsi/scsi_tcq.h>
#include <target/target_core_base.h> #include <target/target_core_base.h>
#include <target/target_core_fabric.h> #include <target/target_core_fabric.h>
......
...@@ -6,7 +6,6 @@ ...@@ -6,7 +6,6 @@
#include <linux/usb/composite.h> #include <linux/usb/composite.h>
#include <linux/usb/uas.h> #include <linux/usb/uas.h>
#include <linux/usb/storage.h> #include <linux/usb/storage.h>
#include <scsi/scsi.h>
#include <target/target_core_base.h> #include <target/target_core_base.h>
#include <target/target_core_fabric.h> #include <target/target_core_fabric.h>
......
...@@ -37,7 +37,8 @@ ...@@ -37,7 +37,8 @@
#include <linux/fs.h> #include <linux/fs.h>
#include <linux/miscdevice.h> #include <linux/miscdevice.h>
#include <asm/unaligned.h> #include <asm/unaligned.h>
#include <scsi/scsi.h> #include <scsi/scsi_common.h>
#include <scsi/scsi_proto.h>
#include <target/target_core_base.h> #include <target/target_core_base.h>
#include <target/target_core_fabric.h> #include <target/target_core_fabric.h>
#include <target/target_core_fabric_configfs.h> #include <target/target_core_fabric_configfs.h>
......
...@@ -49,10 +49,7 @@ ...@@ -49,10 +49,7 @@
#include <generated/utsrelease.h> #include <generated/utsrelease.h>
#include <scsi/scsi.h> #include <scsi/scsi_host.h> /* SG_ALL */
#include <scsi/scsi_dbg.h>
#include <scsi/scsi_eh.h>
#include <scsi/scsi_tcq.h>
#include <target/target_core_base.h> #include <target/target_core_base.h>
#include <target/target_core_fabric.h> #include <target/target_core_fabric.h>
......
...@@ -5,7 +5,6 @@ ...@@ -5,7 +5,6 @@
#include <linux/configfs.h> #include <linux/configfs.h>
#include <net/sock.h> #include <net/sock.h>
#include <net/tcp.h> #include <net/tcp.h>
#include <scsi/scsi_cmnd.h>
#include <scsi/iscsi_proto.h> #include <scsi/iscsi_proto.h>
#include <target/target_core_base.h> #include <target/target_core_base.h>
......
...@@ -6,7 +6,6 @@ ...@@ -6,7 +6,6 @@
#include <linux/dma-mapping.h> #include <linux/dma-mapping.h>
#include <linux/blkdev.h> #include <linux/blkdev.h>
#include <linux/percpu_ida.h> #include <linux/percpu_ida.h>
#include <scsi/scsi_cmnd.h>
#include <net/sock.h> #include <net/sock.h>
#include <net/tcp.h> #include <net/tcp.h>
...@@ -25,7 +24,7 @@ ...@@ -25,7 +24,7 @@
* From include/scsi/scsi_cmnd.h:SCSI_SENSE_BUFFERSIZE, currently * From include/scsi/scsi_cmnd.h:SCSI_SENSE_BUFFERSIZE, currently
* defined 96, but the real limit is 252 (or 260 including the header) * defined 96, but the real limit is 252 (or 260 including the header)
*/ */
#define TRANSPORT_SENSE_BUFFER SCSI_SENSE_BUFFERSIZE #define TRANSPORT_SENSE_BUFFER 96
/* Used by transport_send_check_condition_and_sense() */ /* Used by transport_send_check_condition_and_sense() */
#define SPC_SENSE_KEY_OFFSET 2 #define SPC_SENSE_KEY_OFFSET 2
#define SPC_ADD_SENSE_LEN_OFFSET 7 #define SPC_ADD_SENSE_LEN_OFFSET 7
......
...@@ -6,7 +6,7 @@ ...@@ -6,7 +6,7 @@
#include <linux/tracepoint.h> #include <linux/tracepoint.h>
#include <linux/trace_seq.h> #include <linux/trace_seq.h>
#include <scsi/scsi.h> #include <scsi/scsi_proto.h>
#include <scsi/scsi_tcq.h> #include <scsi/scsi_tcq.h>
#include <target/target_core_base.h> #include <target/target_core_base.h>
......
Markdown is supported
0% .
You are about to add 0 people to the discussion. Proceed with caution.
先完成此消息的编辑!
想要评论请 注册