summaryrefslogtreecommitdiffstats
path: root/drivers
diff options
context:
space:
mode:
authorDave Jiang <dave.jiang@intel.com>2011-05-05 02:44:54 +0200
committerDan Williams <dan.j.williams@intel.com>2011-07-03 13:04:46 +0200
commit0cfa890e5a8a9e3b01b75c17a7856cf96e026e27 (patch)
treee9d6edb440c8a6e4ffcc349b97e8133e6f96f02e /drivers
parentisci: renaming sas_capabilities to scic_phy_cap (diff)
downloadlinux-0cfa890e5a8a9e3b01b75c17a7856cf96e026e27.tar.xz
linux-0cfa890e5a8a9e3b01b75c17a7856cf96e026e27.zip
isci: Fixup SSP command IU and task IU
Fixup of SSP command IU and SSP task IU to something that looks like Linux Signed-off-by: Dave Jiang <dave.jiang@intel.com> Signed-off-by: Dan Williams <dan.j.williams@intel.com>
Diffstat (limited to 'drivers')
-rw-r--r--drivers/scsi/isci/core/intel_sas.h47
-rw-r--r--drivers/scsi/isci/core/scic_sds_request.c108
-rw-r--r--drivers/scsi/isci/request.c121
-rw-r--r--drivers/scsi/isci/sas.h34
-rw-r--r--drivers/scsi/isci/task.c49
5 files changed, 74 insertions, 285 deletions
diff --git a/drivers/scsi/isci/core/intel_sas.h b/drivers/scsi/isci/core/intel_sas.h
index 91c30cfee21d..f7301aa14f02 100644
--- a/drivers/scsi/isci/core/intel_sas.h
+++ b/drivers/scsi/isci/core/intel_sas.h
@@ -160,53 +160,6 @@ enum sci_sas_frame_type {
SCI_SAS_TASK_FRAME = 0x16
};
-/**
- * struct sci_ssp_command_iu - This structure depicts the contents of the SSP
- * COMMAND INFORMATION UNIT. For specific information on each of these
- * individual fields please reference the SAS specification SSP transport
- * layer section.
- *
- *
- */
-struct sci_ssp_command_iu {
- u32 lun_upper;
- u32 lun_lower;
-
- u32 additional_cdb_length:6;
- u32 reserved0:2;
- u32 reserved1:8;
- u32 enable_first_burst:1;
- u32 task_priority:4;
- u32 task_attribute:3;
- u32 reserved2:8;
-
- u32 cdb[4];
-
-};
-
-/**
- * struct sci_ssp_task_iu - This structure depicts the contents of the SSP TASK
- * INFORMATION UNIT. For specific information on each of these individual
- * fields please reference the SAS specification SSP transport layer section.
- *
- *
- */
-struct sci_ssp_task_iu {
- u32 lun_upper;
- u32 lun_lower;
-
- u32 reserved0:8;
- u32 task_function:8;
- u32 reserved1:8;
- u32 reserved2:8;
-
- u32 reserved3:16;
- u32 task_tag:16;
-
- u32 reserved4[3];
-
-};
-
#define SSP_RESPONSE_IU_MAX_DATA 64
#define SCI_SSP_RESPONSE_IU_DATA_PRESENT_MASK (0x03)
diff --git a/drivers/scsi/isci/core/scic_sds_request.c b/drivers/scsi/isci/core/scic_sds_request.c
index 31df5267c096..b6d1333e01fd 100644
--- a/drivers/scsi/isci/core/scic_sds_request.c
+++ b/drivers/scsi/isci/core/scic_sds_request.c
@@ -54,6 +54,7 @@
*/
#include <scsi/sas.h>
+#include "sas.h"
#include "intel_sas.h"
#include "scic_controller.h"
#include "scic_io_request.h"
@@ -98,7 +99,7 @@
*/
#define scic_ssp_io_request_get_object_size() \
(\
- sizeof(struct sci_ssp_command_iu) \
+ sizeof(struct ssp_cmd_iu) \
+ sizeof(struct sci_ssp_response_iu) \
)
@@ -109,7 +110,7 @@
* memory
*/
#define scic_sds_ssp_request_get_command_buffer(memory) \
- ((struct sci_ssp_command_iu *)(\
+ ((struct ssp_cmd_iu *)(\
((char *)(memory)) + sizeof(struct scic_sds_request) \
))
@@ -122,7 +123,7 @@
#define scic_sds_ssp_request_get_response_buffer(memory) \
((struct sci_ssp_response_iu *)(\
((char *)(scic_sds_ssp_request_get_command_buffer(memory))) \
- + sizeof(struct sci_ssp_command_iu) \
+ + sizeof(struct ssp_cmd_iu) \
))
/**
@@ -158,7 +159,7 @@
*/
#define scic_ssp_task_request_get_object_size() \
(\
- sizeof(struct sci_ssp_task_iu) \
+ sizeof(struct ssp_task_iu) \
+ sizeof(struct sci_ssp_response_iu) \
)
@@ -169,7 +170,7 @@
* memory. Yes its the same as the above macro except for the name.
*/
#define scic_sds_ssp_task_request_get_command_buffer(memory) \
- ((struct sci_ssp_task_iu *)(\
+ ((struct ssp_task_iu *)(\
((char *)(memory)) + sizeof(struct scic_sds_request) \
))
@@ -182,7 +183,7 @@
#define scic_sds_ssp_task_request_get_response_buffer(memory) \
((struct sci_ssp_response_iu *)(\
((char *)(scic_sds_ssp_task_request_get_command_buffer(memory))) \
- + sizeof(struct sci_ssp_task_iu) \
+ + sizeof(struct ssp_task_iu) \
))
/**
@@ -344,80 +345,49 @@ static void scic_sds_ssp_io_request_assign_buffers(
}
}
-/**
- * This method constructs the SSP Command IU data for this io request object.
- * @sci_req: This parameter specifies the request object for which the SSP
- * command information unit is being built.
- *
- */
-static void scic_sds_io_request_build_ssp_command_iu(
- struct scic_sds_request *sds_request)
+static void scic_sds_io_request_build_ssp_command_iu(struct scic_sds_request *sci_req)
{
- struct sci_ssp_command_iu *command_frame;
- u32 cdb_length;
- u32 *cdb_buffer;
- struct isci_request *isci_request = sds_request->ireq;
-
- command_frame =
- (struct sci_ssp_command_iu *)sds_request->command_buffer;
-
- command_frame->lun_upper = 0;
- command_frame->lun_lower =
- isci_request_ssp_io_request_get_lun(isci_request);
-
- ((u32 *)command_frame)[2] = 0;
+ struct ssp_cmd_iu *cmd_iu;
+ struct isci_request *ireq = sci_req->ireq;
+ struct sas_task *task = isci_request_access_task(ireq);
- cdb_length = isci_request_ssp_io_request_get_cdb_length(isci_request);
- cdb_buffer = (u32 *)isci_request_ssp_io_request_get_cdb_address(
- isci_request);
+ cmd_iu = sci_req->command_buffer;
- if (cdb_length > 16) {
- command_frame->additional_cdb_length = cdb_length - 16;
- }
+ memcpy(cmd_iu->LUN, task->ssp_task.LUN, 8);
+ cmd_iu->add_cdb_len = 0;
+ cmd_iu->_r_a = 0;
+ cmd_iu->_r_b = 0;
+ cmd_iu->en_fburst = 0; /* unsupported */
+ cmd_iu->task_prio = task->ssp_task.task_prio;
+ cmd_iu->task_attr = task->ssp_task.task_attr;
+ cmd_iu->_r_c = 0;
- /* / @todo Is it ok to leave junk at the end of the cdb buffer? */
scic_word_copy_with_swap(
- (u32 *)(&command_frame->cdb),
- (u32 *)(cdb_buffer),
- (cdb_length + 3) / sizeof(u32)
- );
-
- command_frame->enable_first_burst = 0;
- command_frame->task_priority =
- isci_request_ssp_io_request_get_command_priority(isci_request);
- command_frame->task_attribute =
- isci_request_ssp_io_request_get_task_attribute(isci_request);
+ (u32 *)(&cmd_iu->cdb),
+ (u32 *)task->ssp_task.cdb,
+ sizeof(task->ssp_task.cdb) / sizeof(u32));
}
-
-/**
- * This method constructs the SSP Task IU data for this io request object.
- * @sci_req:
- *
- */
-static void scic_sds_task_request_build_ssp_task_iu(
- struct scic_sds_request *sds_request)
+static void scic_sds_task_request_build_ssp_task_iu(struct scic_sds_request *sci_req)
{
- struct sci_ssp_task_iu *command_frame;
- struct isci_request *isci_request = sds_request->ireq;
+ struct ssp_task_iu *task_iu;
+ struct isci_request *ireq = sci_req->ireq;
+ struct sas_task *task = isci_request_access_task(ireq);
+ struct isci_tmf *isci_tmf = isci_request_access_tmf(ireq);
- command_frame =
- (struct sci_ssp_task_iu *)sds_request->command_buffer;
+ task_iu = sci_req->command_buffer;
- command_frame->lun_upper = 0;
- command_frame->lun_lower = isci_request_ssp_io_request_get_lun(
- isci_request);
+ memset(task_iu, 0, sizeof(struct ssp_task_iu));
- ((u32 *)command_frame)[2] = 0;
+ memcpy(task_iu->LUN, task->ssp_task.LUN, 8);
- command_frame->task_function =
- isci_task_ssp_request_get_function(isci_request);
- command_frame->task_tag =
- isci_task_ssp_request_get_io_tag_to_manage(
- isci_request);
+ task_iu->task_func = isci_tmf->tmf_code;
+ task_iu->task_tag =
+ (ireq->ttype == tmf_task) ?
+ isci_tmf->io_tag :
+ SCI_CONTROLLER_INVALID_IO_TAG;
}
-
/**
* This method is will fill in the SCU Task Context for any type of SSP request.
* @sci_req:
@@ -533,7 +503,8 @@ static void scu_ssp_io_request_construct_task_context(
scu_ssp_reqeust_construct_task_context(sci_req, task_context);
- task_context->ssp_command_iu_length = sizeof(struct sci_ssp_command_iu) / sizeof(u32);
+ task_context->ssp_command_iu_length =
+ sizeof(struct ssp_cmd_iu) / sizeof(u32);
task_context->type.ssp.frame_type = SCI_SAS_COMMAND_FRAME;
switch (dir) {
@@ -605,7 +576,8 @@ static void scu_ssp_task_request_construct_task_context(
task_context->task_type = SCU_TASK_TYPE_RAW_FRAME;
task_context->transfer_length_bytes = 0;
task_context->type.ssp.frame_type = SCI_SAS_TASK_FRAME;
- task_context->ssp_command_iu_length = sizeof(struct sci_ssp_task_iu) / sizeof(u32);
+ task_context->ssp_command_iu_length =
+ sizeof(struct ssp_task_iu) / sizeof(u32);
}
diff --git a/drivers/scsi/isci/request.c b/drivers/scsi/isci/request.c
index c45e78e41f27..501df3ca4280 100644
--- a/drivers/scsi/isci/request.c
+++ b/drivers/scsi/isci/request.c
@@ -1241,124 +1241,3 @@ enum dma_data_direction isci_request_io_request_get_data_direction(
return task->data_dir;
}
-
-/**
- * isci_request_sge_get_address_field() - This function is called by the sci
- * core to retrieve the address field contents for a given sge.
- * @request: This parameter is the isci_request object.
- * @sge_address: This parameter is the sge.
- *
- * physical address in the specified sge.
- */
-
-
-/**
- * isci_request_sge_get_length_field() - This function is called by the sci
- * core to retrieve the length field contents for a given sge.
- * @request: This parameter is the isci_request object.
- * @sge_address: This parameter is the sge.
- *
- * length field value in the specified sge.
- */
-
-
-/**
- * isci_request_ssp_io_request_get_cdb_address() - This function is called by
- * the sci core to retrieve the cdb address for a given request.
- * @request: This parameter is the isci_request object.
- *
- * cdb address for specified request.
- */
-void *isci_request_ssp_io_request_get_cdb_address(
- struct isci_request *request)
-{
- struct sas_task *task = isci_request_access_task(request);
-
- dev_dbg(&request->isci_host->pdev->dev,
- "%s: request->task->ssp_task.cdb = %p\n",
- __func__,
- task->ssp_task.cdb);
- return task->ssp_task.cdb;
-}
-
-
-/**
- * isci_request_ssp_io_request_get_cdb_length() - This function is called by
- * the sci core to retrieve the cdb length for a given request.
- * @request: This parameter is the isci_request object.
- *
- * cdb length for specified request.
- */
-u32 isci_request_ssp_io_request_get_cdb_length(
- struct isci_request *request)
-{
- return 16;
-}
-
-
-/**
- * isci_request_ssp_io_request_get_lun() - This function is called by the sci
- * core to retrieve the lun for a given request.
- * @request: This parameter is the isci_request object.
- *
- * lun for specified request.
- */
-u32 isci_request_ssp_io_request_get_lun(
- struct isci_request *request)
-{
- struct sas_task *task = isci_request_access_task(request);
-
-#ifdef DEBUG
- int i;
-
- for (i = 0; i < 8; i++)
- dev_dbg(&request->isci_host->pdev->dev,
- "%s: task->ssp_task.LUN[%d] = %x\n",
- __func__, i, task->ssp_task.LUN[i]);
-
-#endif
-
- return task->ssp_task.LUN[0];
-}
-
-
-/**
- * isci_request_ssp_io_request_get_task_attribute() - This function is called
- * by the sci core to retrieve the task attribute for a given request.
- * @request: This parameter is the isci_request object.
- *
- * task attribute for specified request.
- */
-u32 isci_request_ssp_io_request_get_task_attribute(
- struct isci_request *request)
-{
- struct sas_task *task = isci_request_access_task(request);
-
- dev_dbg(&request->isci_host->pdev->dev,
- "%s: request->task->ssp_task.task_attr = %x\n",
- __func__,
- task->ssp_task.task_attr);
-
- return task->ssp_task.task_attr;
-}
-
-
-/**
- * isci_request_ssp_io_request_get_command_priority() - This function is called
- * by the sci core to retrieve the command priority for a given request.
- * @request: This parameter is the isci_request object.
- *
- * command priority for specified request.
- */
-u32 isci_request_ssp_io_request_get_command_priority(
- struct isci_request *request)
-{
- struct sas_task *task = isci_request_access_task(request);
-
- dev_dbg(&request->isci_host->pdev->dev,
- "%s: request->task->ssp_task.task_prio = %x\n",
- __func__,
- task->ssp_task.task_prio);
-
- return task->ssp_task.task_prio;
-}
diff --git a/drivers/scsi/isci/sas.h b/drivers/scsi/isci/sas.h
index 05af8bd61453..21ddd63924eb 100644
--- a/drivers/scsi/isci/sas.h
+++ b/drivers/scsi/isci/sas.h
@@ -69,4 +69,38 @@
#define FIS_PIO_SETUP 0x5F
#define FIS_DATA 0x46
+/*
+ * contents of the SSP COMMAND INFORMATION UNIT.
+ * For specific information on each of these individual fields please
+ * reference the SAS specification SSP transport layer section.
+ * XXX: This needs to go into <scsi/sas.h>
+ */
+struct ssp_cmd_iu {
+ u8 LUN[8];
+ u8 add_cdb_len:6;
+ u8 _r_a:2;
+ u8 _r_b;
+ u8 en_fburst:1;
+ u8 task_prio:4;
+ u8 task_attr:3;
+ u8 _r_c;
+
+ u8 cdb[16];
+} __packed;
+
+/*
+ * contents of the SSP TASK INFORMATION UNIT.
+ * For specific information on each of these individual fields please
+ * reference the SAS specification SSP transport layer section.
+ * XXX: This needs to go into <scsi/sas.h>
+ */
+struct ssp_task_iu {
+ u8 LUN[8];
+ u8 _r_a;
+ u8 task_func;
+ u8 _r_b[4];
+ u16 task_tag;
+ u8 _r_c[12];
+} __packed;
+
#endif
diff --git a/drivers/scsi/isci/task.c b/drivers/scsi/isci/task.c
index f9a1c41a5ce8..c4db95933d41 100644
--- a/drivers/scsi/isci/task.c
+++ b/drivers/scsi/isci/task.c
@@ -1481,55 +1481,6 @@ void isci_task_request_complete(
complete(tmf_complete);
}
-
-/**
- * isci_task_ssp_request_get_lun() - This function is called by the sci core to
- * retrieve the lun for a given task request.
- * @request: This parameter is the isci_request object.
- *
- * lun for specified task request.
- */
-
-/**
- * isci_task_ssp_request_get_function() - This function is called by the sci
- * core to retrieve the function for a given task request.
- * @request: This parameter is the isci_request object.
- *
- * function code for specified task request.
- */
-u8 isci_task_ssp_request_get_function(struct isci_request *request)
-{
- struct isci_tmf *isci_tmf = isci_request_access_tmf(request);
-
- dev_dbg(&request->isci_host->pdev->dev,
- "%s: func = %d\n", __func__, isci_tmf->tmf_code);
-
- return isci_tmf->tmf_code;
-}
-
-/**
- * isci_task_ssp_request_get_io_tag_to_manage() - This function is called by
- * the sci core to retrieve the io tag for a given task request.
- * @request: This parameter is the isci_request object.
- *
- * io tag for specified task request.
- */
-u16 isci_task_ssp_request_get_io_tag_to_manage(struct isci_request *request)
-{
- u16 io_tag = SCI_CONTROLLER_INVALID_IO_TAG;
-
- if (tmf_task == request->ttype) {
- struct isci_tmf *tmf = isci_request_access_tmf(request);
- io_tag = tmf->io_tag;
- }
-
- dev_dbg(&request->isci_host->pdev->dev,
- "%s: request = %p, io_tag = %d\n",
- __func__, request, io_tag);
-
- return io_tag;
-}
-
/**
* isci_task_ssp_request_get_response_data_address() - This function is called
* by the sci core to retrieve the response data address for a given task