lpfc: Refactor lpfc nvme headers
authorJames Smart <jsmart2021@gmail.com>
Tue, 31 Mar 2020 16:50:02 +0000 (09:50 -0700)
committerJens Axboe <axboe@kernel.dk>
Sat, 9 May 2020 22:18:34 +0000 (16:18 -0600)
A lot of files in lpfc include nvme headers, building up relationships that
require a file to change for its headers when there is no other change
necessary. It would be better to localize the nvme headers.

There is also no need for separate nvme (initiator) and nvmet (tgt)
header files.

Refactor the inclusion of nvme headers so that all nvme items are
included by lpfc_nvme.h

Merge lpfc_nvmet.h into lpfc_nvme.h so that there is a single header used
by both the nvme and nvmet sides. This prepares for structure sharing
between the two roles. Prep to add shared function prototypes for upcoming
shared routines.

Signed-off-by: Paul Ely <paul.ely@broadcom.com>
Signed-off-by: James Smart <jsmart2021@gmail.com>
Reviewed-by: Hannes Reinecke <hare@suse.de>
Signed-off-by: Christoph Hellwig <hch@lst.de>
Signed-off-by: Jens Axboe <axboe@kernel.dk>
12 files changed:
drivers/scsi/lpfc/lpfc_attr.c
drivers/scsi/lpfc/lpfc_ct.c
drivers/scsi/lpfc/lpfc_debugfs.c
drivers/scsi/lpfc/lpfc_hbadisc.c
drivers/scsi/lpfc/lpfc_init.c
drivers/scsi/lpfc/lpfc_mem.c
drivers/scsi/lpfc/lpfc_nportdisc.c
drivers/scsi/lpfc/lpfc_nvme.c
drivers/scsi/lpfc/lpfc_nvme.h
drivers/scsi/lpfc/lpfc_nvmet.c
drivers/scsi/lpfc/lpfc_nvmet.h [deleted file]
drivers/scsi/lpfc/lpfc_sli.c

index 1354c14..f089867 100644 (file)
@@ -37,8 +37,6 @@
 #include <scsi/scsi_transport_fc.h>
 #include <scsi/fc/fc_fs.h>
 
-#include <linux/nvme-fc-driver.h>
-
 #include "lpfc_hw4.h"
 #include "lpfc_hw.h"
 #include "lpfc_sli.h"
@@ -48,7 +46,6 @@
 #include "lpfc.h"
 #include "lpfc_scsi.h"
 #include "lpfc_nvme.h"
-#include "lpfc_nvmet.h"
 #include "lpfc_logmsg.h"
 #include "lpfc_version.h"
 #include "lpfc_compat.h"
index 2aa578d..196f6ae 100644 (file)
@@ -44,7 +44,6 @@
 #include "lpfc_disc.h"
 #include "lpfc.h"
 #include "lpfc_scsi.h"
-#include "lpfc_nvme.h"
 #include "lpfc_logmsg.h"
 #include "lpfc_crtn.h"
 #include "lpfc_version.h"
index 8a6e02a..8e78e49 100644 (file)
@@ -39,8 +39,6 @@
 #include <scsi/scsi_transport_fc.h>
 #include <scsi/fc/fc_fs.h>
 
-#include <linux/nvme-fc-driver.h>
-
 #include "lpfc_hw4.h"
 #include "lpfc_hw.h"
 #include "lpfc_sli.h"
@@ -50,7 +48,6 @@
 #include "lpfc.h"
 #include "lpfc_scsi.h"
 #include "lpfc_nvme.h"
-#include "lpfc_nvmet.h"
 #include "lpfc_logmsg.h"
 #include "lpfc_crtn.h"
 #include "lpfc_vport.h"
index 789eecb..8dec7b7 100644 (file)
@@ -36,8 +36,6 @@
 #include <scsi/scsi_transport_fc.h>
 #include <scsi/fc/fc_fs.h>
 
-#include <linux/nvme-fc-driver.h>
-
 #include "lpfc_hw4.h"
 #include "lpfc_hw.h"
 #include "lpfc_nl.h"
index 4104bdc..a295660 100644 (file)
@@ -50,8 +50,6 @@
 #include <scsi/scsi_tcq.h>
 #include <scsi/fc/fc_fs.h>
 
-#include <linux/nvme-fc-driver.h>
-
 #include "lpfc_hw4.h"
 #include "lpfc_hw.h"
 #include "lpfc_sli.h"
@@ -61,7 +59,6 @@
 #include "lpfc.h"
 #include "lpfc_scsi.h"
 #include "lpfc_nvme.h"
-#include "lpfc_nvmet.h"
 #include "lpfc_logmsg.h"
 #include "lpfc_crtn.h"
 #include "lpfc_vport.h"
index 7082279..726f661 100644 (file)
@@ -31,8 +31,6 @@
 #include <scsi/scsi_transport_fc.h>
 #include <scsi/fc/fc_fs.h>
 
-#include <linux/nvme-fc-driver.h>
-
 #include "lpfc_hw4.h"
 #include "lpfc_hw.h"
 #include "lpfc_sli.h"
@@ -41,8 +39,6 @@
 #include "lpfc_disc.h"
 #include "lpfc.h"
 #include "lpfc_scsi.h"
-#include "lpfc_nvme.h"
-#include "lpfc_nvmet.h"
 #include "lpfc_crtn.h"
 #include "lpfc_logmsg.h"
 
index a024e5a..81f4ba1 100644 (file)
@@ -32,8 +32,6 @@
 #include <scsi/scsi_transport_fc.h>
 #include <scsi/fc/fc_fs.h>
 
-#include <linux/nvme-fc-driver.h>
-
 #include "lpfc_hw4.h"
 #include "lpfc_hw.h"
 #include "lpfc_sli.h"
index a45936e..6045000 100644 (file)
@@ -36,9 +36,6 @@
 #include <scsi/scsi_transport_fc.h>
 #include <scsi/fc/fc_fs.h>
 
-#include <linux/nvme.h>
-#include <linux/nvme-fc-driver.h>
-#include <linux/nvme-fc.h>
 #include "lpfc_version.h"
 #include "lpfc_hw4.h"
 #include "lpfc_hw.h"
index 593c48f..e59dc1f 100644 (file)
  * included with this package.                                     *
  ********************************************************************/
 
+#include <linux/nvme.h>
+#include <linux/nvme-fc-driver.h>
+#include <linux/nvme-fc.h>
+
 #define LPFC_NVME_DEFAULT_SEGS         (64 + 1)        /* 256K IOs */
 
 #define LPFC_NVME_ERSP_LEN             0x20
@@ -74,3 +78,145 @@ struct lpfc_nvme_rport {
 struct lpfc_nvme_fcpreq_priv {
        struct lpfc_io_buf *nvme_buf;
 };
+
+
+#define LPFC_NVMET_DEFAULT_SEGS                (64 + 1)        /* 256K IOs */
+#define LPFC_NVMET_RQE_MIN_POST                128
+#define LPFC_NVMET_RQE_DEF_POST                512
+#define LPFC_NVMET_RQE_DEF_COUNT       2048
+#define LPFC_NVMET_SUCCESS_LEN         12
+
+#define LPFC_NVMET_MRQ_AUTO            0
+#define LPFC_NVMET_MRQ_MAX             16
+
+#define LPFC_NVMET_WAIT_TMO            (5 * MSEC_PER_SEC)
+
+/* Used for NVME Target */
+struct lpfc_nvmet_tgtport {
+       struct lpfc_hba *phba;
+       struct completion *tport_unreg_cmp;
+
+       /* Stats counters - lpfc_nvmet_unsol_ls_buffer */
+       atomic_t rcv_ls_req_in;
+       atomic_t rcv_ls_req_out;
+       atomic_t rcv_ls_req_drop;
+       atomic_t xmt_ls_abort;
+       atomic_t xmt_ls_abort_cmpl;
+
+       /* Stats counters - lpfc_nvmet_xmt_ls_rsp */
+       atomic_t xmt_ls_rsp;
+       atomic_t xmt_ls_drop;
+
+       /* Stats counters - lpfc_nvmet_xmt_ls_rsp_cmp */
+       atomic_t xmt_ls_rsp_error;
+       atomic_t xmt_ls_rsp_aborted;
+       atomic_t xmt_ls_rsp_xb_set;
+       atomic_t xmt_ls_rsp_cmpl;
+
+       /* Stats counters - lpfc_nvmet_unsol_fcp_buffer */
+       atomic_t rcv_fcp_cmd_in;
+       atomic_t rcv_fcp_cmd_out;
+       atomic_t rcv_fcp_cmd_drop;
+       atomic_t rcv_fcp_cmd_defer;
+       atomic_t xmt_fcp_release;
+
+       /* Stats counters - lpfc_nvmet_xmt_fcp_op */
+       atomic_t xmt_fcp_drop;
+       atomic_t xmt_fcp_read_rsp;
+       atomic_t xmt_fcp_read;
+       atomic_t xmt_fcp_write;
+       atomic_t xmt_fcp_rsp;
+
+       /* Stats counters - lpfc_nvmet_xmt_fcp_op_cmp */
+       atomic_t xmt_fcp_rsp_xb_set;
+       atomic_t xmt_fcp_rsp_cmpl;
+       atomic_t xmt_fcp_rsp_error;
+       atomic_t xmt_fcp_rsp_aborted;
+       atomic_t xmt_fcp_rsp_drop;
+
+       /* Stats counters - lpfc_nvmet_xmt_fcp_abort */
+       atomic_t xmt_fcp_xri_abort_cqe;
+       atomic_t xmt_fcp_abort;
+       atomic_t xmt_fcp_abort_cmpl;
+       atomic_t xmt_abort_sol;
+       atomic_t xmt_abort_unsol;
+       atomic_t xmt_abort_rsp;
+       atomic_t xmt_abort_rsp_error;
+
+       /* Stats counters - defer IO */
+       atomic_t defer_ctx;
+       atomic_t defer_fod;
+       atomic_t defer_wqfull;
+};
+
+struct lpfc_nvmet_ctx_info {
+       struct list_head nvmet_ctx_list;
+       spinlock_t      nvmet_ctx_list_lock; /* lock per CPU */
+       struct lpfc_nvmet_ctx_info *nvmet_ctx_next_cpu;
+       struct lpfc_nvmet_ctx_info *nvmet_ctx_start_cpu;
+       uint16_t        nvmet_ctx_list_cnt;
+       char pad[16];  /* pad to a cache-line */
+};
+
+/* This retrieves the context info associated with the specified cpu / mrq */
+#define lpfc_get_ctx_list(phba, cpu, mrq)  \
+       (phba->sli4_hba.nvmet_ctx_info + ((cpu * phba->cfg_nvmet_mrq) + mrq))
+
+struct lpfc_nvmet_rcv_ctx {
+       union {
+               struct nvmefc_ls_rsp ls_rsp;
+               struct nvmefc_tgt_fcp_req fcp_req;
+       } ctx;
+       struct list_head list;
+       struct lpfc_hba *phba;
+       struct lpfc_iocbq *wqeq;
+       struct lpfc_iocbq *abort_wqeq;
+       spinlock_t ctxlock; /* protect flag access */
+       uint32_t sid;
+       uint32_t offset;
+       uint16_t oxid;
+       uint16_t size;
+       uint16_t entry_cnt;
+       uint16_t cpu;
+       uint16_t idx;
+       uint16_t state;
+       /* States */
+#define LPFC_NVMET_STE_LS_RCV          1
+#define LPFC_NVMET_STE_LS_ABORT                2
+#define LPFC_NVMET_STE_LS_RSP          3
+#define LPFC_NVMET_STE_RCV             4
+#define LPFC_NVMET_STE_DATA            5
+#define LPFC_NVMET_STE_ABORT           6
+#define LPFC_NVMET_STE_DONE            7
+#define LPFC_NVMET_STE_FREE            0xff
+       uint16_t flag;
+#define LPFC_NVMET_IO_INP              0x1  /* IO is in progress on exchange */
+#define LPFC_NVMET_ABORT_OP            0x2  /* Abort WQE issued on exchange */
+#define LPFC_NVMET_XBUSY               0x4  /* XB bit set on IO cmpl */
+#define LPFC_NVMET_CTX_RLS             0x8  /* ctx free requested */
+#define LPFC_NVMET_ABTS_RCV            0x10  /* ABTS received on exchange */
+#define LPFC_NVMET_CTX_REUSE_WQ                0x20  /* ctx reused via WQ */
+#define LPFC_NVMET_DEFER_WQFULL                0x40  /* Waiting on a free WQE */
+#define LPFC_NVMET_TNOTIFY             0x80  /* notify transport of abts */
+       struct rqb_dmabuf *rqb_buffer;
+       struct lpfc_nvmet_ctxbuf *ctxbuf;
+       struct lpfc_sli4_hdw_queue *hdwq;
+
+#ifdef CONFIG_SCSI_LPFC_DEBUG_FS
+       uint64_t ts_isr_cmd;
+       uint64_t ts_cmd_nvme;
+       uint64_t ts_nvme_data;
+       uint64_t ts_data_wqput;
+       uint64_t ts_isr_data;
+       uint64_t ts_data_nvme;
+       uint64_t ts_nvme_status;
+       uint64_t ts_status_wqput;
+       uint64_t ts_isr_status;
+       uint64_t ts_status_nvme;
+#endif
+};
+
+
+/* routines found in lpfc_nvme.c */
+
+/* routines found in lpfc_nvmet.c */
index 3b25bcb..9576bc3 100644 (file)
 #include <scsi/scsi_transport_fc.h>
 #include <scsi/fc/fc_fs.h>
 
-#include <linux/nvme.h>
-#include <linux/nvme-fc-driver.h>
-#include <linux/nvme-fc.h>
-
 #include "lpfc_version.h"
 #include "lpfc_hw4.h"
 #include "lpfc_hw.h"
@@ -50,7 +46,6 @@
 #include "lpfc.h"
 #include "lpfc_scsi.h"
 #include "lpfc_nvme.h"
-#include "lpfc_nvmet.h"
 #include "lpfc_logmsg.h"
 #include "lpfc_crtn.h"
 #include "lpfc_vport.h"
diff --git a/drivers/scsi/lpfc/lpfc_nvmet.h b/drivers/scsi/lpfc/lpfc_nvmet.h
deleted file mode 100644 (file)
index f0196f3..0000000
+++ /dev/null
@@ -1,158 +0,0 @@
-/*******************************************************************
- * This file is part of the Emulex Linux Device Driver for         *
- * Fibre Channel Host Bus Adapters.                                *
- * Copyright (C) 2017-2019 Broadcom. All Rights Reserved. The term *
- * “Broadcom” refers to Broadcom Inc. and/or its subsidiaries.     *
- * Copyright (C) 2004-2016 Emulex.  All rights reserved.           *
- * EMULEX and SLI are trademarks of Emulex.                        *
- * www.broadcom.com                                                *
- * Portions Copyright (C) 2004-2005 Christoph Hellwig              *
- *                                                                 *
- * This program is free software; you can redistribute it and/or   *
- * modify it under the terms of version 2 of the GNU General       *
- * Public License as published by the Free Software Foundation.    *
- * This program is distributed in the hope that it will be useful. *
- * ALL EXPRESS OR IMPLIED CONDITIONS, REPRESENTATIONS AND          *
- * WARRANTIES, INCLUDING ANY IMPLIED WARRANTY OF MERCHANTABILITY,  *
- * FITNESS FOR A PARTICULAR PURPOSE, OR NON-INFRINGEMENT, ARE      *
- * DISCLAIMED, EXCEPT TO THE EXTENT THAT SUCH DISCLAIMERS ARE HELD *
- * TO BE LEGALLY INVALID.  See the GNU General Public License for  *
- * more details, a copy of which can be found in the file COPYING  *
- * included with this package.                                     *
- ********************************************************************/
-
-#define LPFC_NVMET_DEFAULT_SEGS                (64 + 1)        /* 256K IOs */
-#define LPFC_NVMET_RQE_MIN_POST                128
-#define LPFC_NVMET_RQE_DEF_POST                512
-#define LPFC_NVMET_RQE_DEF_COUNT       2048
-#define LPFC_NVMET_SUCCESS_LEN         12
-
-#define LPFC_NVMET_MRQ_AUTO            0
-#define LPFC_NVMET_MRQ_MAX             16
-
-#define LPFC_NVMET_WAIT_TMO            (5 * MSEC_PER_SEC)
-
-/* Used for NVME Target */
-struct lpfc_nvmet_tgtport {
-       struct lpfc_hba *phba;
-       struct completion *tport_unreg_cmp;
-
-       /* Stats counters - lpfc_nvmet_unsol_ls_buffer */
-       atomic_t rcv_ls_req_in;
-       atomic_t rcv_ls_req_out;
-       atomic_t rcv_ls_req_drop;
-       atomic_t xmt_ls_abort;
-       atomic_t xmt_ls_abort_cmpl;
-
-       /* Stats counters - lpfc_nvmet_xmt_ls_rsp */
-       atomic_t xmt_ls_rsp;
-       atomic_t xmt_ls_drop;
-
-       /* Stats counters - lpfc_nvmet_xmt_ls_rsp_cmp */
-       atomic_t xmt_ls_rsp_error;
-       atomic_t xmt_ls_rsp_aborted;
-       atomic_t xmt_ls_rsp_xb_set;
-       atomic_t xmt_ls_rsp_cmpl;
-
-       /* Stats counters - lpfc_nvmet_unsol_fcp_buffer */
-       atomic_t rcv_fcp_cmd_in;
-       atomic_t rcv_fcp_cmd_out;
-       atomic_t rcv_fcp_cmd_drop;
-       atomic_t rcv_fcp_cmd_defer;
-       atomic_t xmt_fcp_release;
-
-       /* Stats counters - lpfc_nvmet_xmt_fcp_op */
-       atomic_t xmt_fcp_drop;
-       atomic_t xmt_fcp_read_rsp;
-       atomic_t xmt_fcp_read;
-       atomic_t xmt_fcp_write;
-       atomic_t xmt_fcp_rsp;
-
-       /* Stats counters - lpfc_nvmet_xmt_fcp_op_cmp */
-       atomic_t xmt_fcp_rsp_xb_set;
-       atomic_t xmt_fcp_rsp_cmpl;
-       atomic_t xmt_fcp_rsp_error;
-       atomic_t xmt_fcp_rsp_aborted;
-       atomic_t xmt_fcp_rsp_drop;
-
-       /* Stats counters - lpfc_nvmet_xmt_fcp_abort */
-       atomic_t xmt_fcp_xri_abort_cqe;
-       atomic_t xmt_fcp_abort;
-       atomic_t xmt_fcp_abort_cmpl;
-       atomic_t xmt_abort_sol;
-       atomic_t xmt_abort_unsol;
-       atomic_t xmt_abort_rsp;
-       atomic_t xmt_abort_rsp_error;
-
-       /* Stats counters - defer IO */
-       atomic_t defer_ctx;
-       atomic_t defer_fod;
-       atomic_t defer_wqfull;
-};
-
-struct lpfc_nvmet_ctx_info {
-       struct list_head nvmet_ctx_list;
-       spinlock_t      nvmet_ctx_list_lock; /* lock per CPU */
-       struct lpfc_nvmet_ctx_info *nvmet_ctx_next_cpu;
-       struct lpfc_nvmet_ctx_info *nvmet_ctx_start_cpu;
-       uint16_t        nvmet_ctx_list_cnt;
-       char pad[16];  /* pad to a cache-line */
-};
-
-/* This retrieves the context info associated with the specified cpu / mrq */
-#define lpfc_get_ctx_list(phba, cpu, mrq)  \
-       (phba->sli4_hba.nvmet_ctx_info + ((cpu * phba->cfg_nvmet_mrq) + mrq))
-
-struct lpfc_nvmet_rcv_ctx {
-       union {
-               struct nvmefc_ls_rsp ls_rsp;
-               struct nvmefc_tgt_fcp_req fcp_req;
-       } ctx;
-       struct list_head list;
-       struct lpfc_hba *phba;
-       struct lpfc_iocbq *wqeq;
-       struct lpfc_iocbq *abort_wqeq;
-       spinlock_t ctxlock; /* protect flag access */
-       uint32_t sid;
-       uint32_t offset;
-       uint16_t oxid;
-       uint16_t size;
-       uint16_t entry_cnt;
-       uint16_t cpu;
-       uint16_t idx;
-       uint16_t state;
-       /* States */
-#define LPFC_NVMET_STE_LS_RCV          1
-#define LPFC_NVMET_STE_LS_ABORT                2
-#define LPFC_NVMET_STE_LS_RSP          3
-#define LPFC_NVMET_STE_RCV             4
-#define LPFC_NVMET_STE_DATA            5
-#define LPFC_NVMET_STE_ABORT           6
-#define LPFC_NVMET_STE_DONE            7
-#define LPFC_NVMET_STE_FREE            0xff
-       uint16_t flag;
-#define LPFC_NVMET_IO_INP              0x1  /* IO is in progress on exchange */
-#define LPFC_NVMET_ABORT_OP            0x2  /* Abort WQE issued on exchange */
-#define LPFC_NVMET_XBUSY               0x4  /* XB bit set on IO cmpl */
-#define LPFC_NVMET_CTX_RLS             0x8  /* ctx free requested */
-#define LPFC_NVMET_ABTS_RCV            0x10  /* ABTS received on exchange */
-#define LPFC_NVMET_CTX_REUSE_WQ                0x20  /* ctx reused via WQ */
-#define LPFC_NVMET_DEFER_WQFULL                0x40  /* Waiting on a free WQE */
-#define LPFC_NVMET_TNOTIFY             0x80  /* notify transport of abts */
-       struct rqb_dmabuf *rqb_buffer;
-       struct lpfc_nvmet_ctxbuf *ctxbuf;
-       struct lpfc_sli4_hdw_queue *hdwq;
-
-#ifdef CONFIG_SCSI_LPFC_DEBUG_FS
-       uint64_t ts_isr_cmd;
-       uint64_t ts_cmd_nvme;
-       uint64_t ts_nvme_data;
-       uint64_t ts_data_wqput;
-       uint64_t ts_isr_data;
-       uint64_t ts_data_nvme;
-       uint64_t ts_nvme_status;
-       uint64_t ts_status_wqput;
-       uint64_t ts_isr_status;
-       uint64_t ts_status_nvme;
-#endif
-};
index b6fb665..d57918d 100644 (file)
@@ -39,8 +39,6 @@
 #include <asm/set_memory.h>
 #endif
 
-#include <linux/nvme-fc-driver.h>
-
 #include "lpfc_hw4.h"
 #include "lpfc_hw.h"
 #include "lpfc_sli.h"
@@ -50,7 +48,6 @@
 #include "lpfc.h"
 #include "lpfc_scsi.h"
 #include "lpfc_nvme.h"
-#include "lpfc_nvmet.h"
 #include "lpfc_crtn.h"
 #include "lpfc_logmsg.h"
 #include "lpfc_compat.h"