isci: make sgl explicit/aligned request object member

Towards unifying request objects we need all members to be defined in the
object and not carved out of anonymous buffer space.

Signed-off-by: Dan Williams <dan.j.williams@intel.com>

diff --git a/drivers/scsi/isci/core/scic_sds_request.c b/drivers/scsi/isci/core/scic_sds_request.c
index fd7bd33..189a799 100644 (file)
--- a/drivers/scsi/isci/core/scic_sds_request.c
+++ b/drivers/scsi/isci/core/scic_sds_request.c
@@ -212,8 +212,7 @@ static u32 scic_sds_ssp_request_get_object_size(void)
        return sizeof(struct scic_sds_request)
               + scic_ssp_io_request_get_object_size()
               + sizeof(struct scu_task_context)
-              + SMP_CACHE_BYTES
-              + sizeof(struct scu_sgl_element_pair) * SCU_MAX_SGL_ELEMENT_PAIRS;
+              + SMP_CACHE_BYTES;
 }
 
 /**
@@ -239,7 +238,7 @@ static struct scu_sgl_element_pair *scic_sds_request_get_sgl_element_pair(
                return &task_context->sgl_pair_cd;
        }
 
-       return &sci_req->sgl_element_pair_buffer[sgl_pair_index - 2];
+       return &sci_req->sg_table[sgl_pair_index - 2];
 }
 
 /**
@@ -328,11 +327,6 @@ static void scic_sds_ssp_io_request_assign_buffers(
                scic_sds_ssp_request_get_command_buffer(sci_req);
        sci_req->response_buffer =
                scic_sds_ssp_request_get_response_buffer(sci_req);
-       sci_req->sgl_element_pair_buffer =
-               scic_sds_ssp_request_get_sgl_element_buffer(sci_req);
-       sci_req->sgl_element_pair_buffer =
-               PTR_ALIGN(sci_req->sgl_element_pair_buffer,
-                         sizeof(struct scu_sgl_element_pair));
 
        if (sci_req->was_tag_assigned_by_user == false) {
                sci_req->task_context_buffer =
@@ -535,7 +529,6 @@ static void scic_sds_ssp_task_request_assign_buffers(
                scic_sds_ssp_task_request_get_command_buffer(sci_req);
        sci_req->response_buffer =
                scic_sds_ssp_task_request_get_response_buffer(sci_req);
-       sci_req->sgl_element_pair_buffer = NULL;
 
        if (sci_req->was_tag_assigned_by_user == false) {
                sci_req->task_context_buffer =

diff --git a/drivers/scsi/isci/core/scic_sds_request.h b/drivers/scsi/isci/core/scic_sds_request.h
index c93f3ed..83d737a 100644 (file)
--- a/drivers/scsi/isci/core/scic_sds_request.h
+++ b/drivers/scsi/isci/core/scic_sds_request.h
@@ -60,6 +60,7 @@
 #include "sci_base_state_machine.h"
 #include "scu_task_context.h"
 #include "scic_sds_stp_request.h"
+#include "scu_constants.h"
 
 struct scic_sds_controller;
 struct scic_sds_remote_device;
@@ -183,7 +184,10 @@ struct scic_sds_request {
        void *command_buffer;
        void *response_buffer;
        struct scu_task_context *task_context_buffer;
-       struct scu_sgl_element_pair *sgl_element_pair_buffer;
+
+       /* could be larger with sg chaining */
+       #define SCU_SGL_SIZE ((SCU_IO_REQUEST_SGE_COUNT + 1) / 2)
+       struct scu_sgl_element_pair sg_table[SCU_SGL_SIZE] __attribute__ ((aligned(32)));
 
        /**
         * This field indicates if this request is a task management request or
@@ -327,14 +331,6 @@ struct scic_sds_io_request_state_handler {
 
 extern const struct sci_base_state scic_sds_io_request_started_task_mgmt_substate_table[];
 
-/**
- *
- *
- * This macro returns the maximum number of SGL element paris that we will
- * support in a single IO request.
- */
-#define SCU_MAX_SGL_ELEMENT_PAIRS ((SCU_IO_REQUEST_SGE_COUNT + 1) / 2)
-
 /**
  * scic_sds_request_get_controller() -
  *

diff --git a/drivers/scsi/isci/core/scic_sds_smp_request.c b/drivers/scsi/isci/core/scic_sds_smp_request.c
index 1f8773d..cb1adef 100644 (file)
--- a/drivers/scsi/isci/core/scic_sds_smp_request.c
+++ b/drivers/scsi/isci/core/scic_sds_smp_request.c
@@ -131,7 +131,6 @@ void scic_sds_smp_request_assign_buffers(
                scic_sds_smp_request_get_command_buffer(sci_req);
        sci_req->response_buffer =
                scic_sds_smp_request_get_response_buffer(sci_req);
-       sci_req->sgl_element_pair_buffer = NULL;
 
        if (sci_req->was_tag_assigned_by_user == false) {
                sci_req->task_context_buffer =

diff --git a/drivers/scsi/isci/core/scic_sds_stp_request.c b/drivers/scsi/isci/core/scic_sds_stp_request.c
index 7dba40f..013af11 100644 (file)
--- a/drivers/scsi/isci/core/scic_sds_stp_request.c
+++ b/drivers/scsi/isci/core/scic_sds_stp_request.c
@@ -105,18 +105,6 @@
                 + SSP_RESP_IU_MAX_SIZE \
                 ))
 
-/**
- * scic_sds_stp_request_get_sgl_element_buffer() -
- *
- * This macro returns the address of the sgl elment pairs in the io request
- * memory buffer
- */
-#define scic_sds_stp_request_get_sgl_element_buffer(memory) \
-       ((struct scu_sgl_element_pair *)(\
-                ((char *)(scic_sds_stp_request_get_task_context_buffer(memory))) \
-                + sizeof(struct scu_task_context) \
-                ))
-
 /**
  *
  *
@@ -128,8 +116,7 @@ u32 scic_sds_stp_request_get_object_size(void)
               + sizeof(struct host_to_dev_fis)
               + sizeof(struct dev_to_host_fis)
               + sizeof(struct scu_task_context)
-              + SMP_CACHE_BYTES
-              + sizeof(struct scu_sgl_element_pair) * SCU_MAX_SGL_ELEMENT_PAIRS;
+              + SMP_CACHE_BYTES;
 }
 
 void scic_sds_stp_request_assign_buffers(struct scic_sds_request *sci_req)
@@ -138,9 +125,6 @@ void scic_sds_stp_request_assign_buffers(struct scic_sds_request *sci_req)
 
        sci_req->command_buffer = scic_sds_stp_request_get_h2d_reg_buffer(stp_req);
        sci_req->response_buffer = scic_sds_stp_request_get_response_buffer(stp_req);
-       sci_req->sgl_element_pair_buffer = scic_sds_stp_request_get_sgl_element_buffer(stp_req);
-       sci_req->sgl_element_pair_buffer = PTR_ALIGN(sci_req->sgl_element_pair_buffer,
-                                                    sizeof(struct scu_sgl_element_pair));
 
        if (sci_req->was_tag_assigned_by_user == false) {
                sci_req->task_context_buffer =

diff --cc drivers/scsi/isci/request.c
Simple merge

diff --cc drivers/scsi/isci/request.h
Simple merge

diff --cc drivers/scsi/isci/task.c
Simple merge