diff --git a/drivers/scsi/isci/core/scic_sds_request.c b/drivers/scsi/isci/core/scic_sds_request.c index 189a7999726cd6666258f454f1299c19a9f32d3c..baf69edab699cc86cdcb8ca3af2fc6177a2b26ce 100644 --- a/drivers/scsi/isci/core/scic_sds_request.c +++ b/drivers/scsi/isci/core/scic_sds_request.c @@ -124,31 +124,6 @@ + sizeof(struct ssp_cmd_iu) \ )) -/** - * scic_sds_ssp_request_get_task_context_buffer() - - * - * This macro returns the address of the task context buffer in the io request - * memory - */ -#define scic_sds_ssp_request_get_task_context_buffer(memory) \ - ((struct scu_task_context *)(\ - ((char *)(scic_sds_ssp_request_get_response_buffer(memory))) \ - + SSP_RESP_IU_MAX_SIZE \ - )) - -/** - * scic_sds_ssp_request_get_sgl_element_buffer() - - * - * This macro returns the address of the sgl elment pairs in the io request - * memory buffer - */ -#define scic_sds_ssp_request_get_sgl_element_buffer(memory) \ - ((struct scu_sgl_element_pair *)(\ - ((char *)(scic_sds_ssp_request_get_task_context_buffer(memory))) \ - + sizeof(struct scu_task_context) \ - )) - - /** * scic_ssp_task_request_get_object_size() - * @@ -184,24 +159,6 @@ + sizeof(struct ssp_task_iu) \ )) -/** - * scic_sds_ssp_task_request_get_task_context_buffer() - - * - * This macro returs the task context buffer for the SSP task request. - */ -#define scic_sds_ssp_task_request_get_task_context_buffer(memory) \ - ((struct scu_task_context *)(\ - ((char *)(scic_sds_ssp_task_request_get_response_buffer(memory))) \ - + SSP_RESP_IU_MAX_SIZE \ - )) - - - -/* - * **************************************************************************** - * * SCIC SDS IO REQUEST PRIVATE METHODS - * **************************************************************************** */ - /** * * @@ -210,9 +167,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; + + scic_ssp_io_request_get_object_size(); } /** @@ -328,13 +283,8 @@ static void scic_sds_ssp_io_request_assign_buffers( sci_req->response_buffer = scic_sds_ssp_request_get_response_buffer(sci_req); - if (sci_req->was_tag_assigned_by_user == false) { - sci_req->task_context_buffer = - scic_sds_ssp_request_get_task_context_buffer(sci_req); - sci_req->task_context_buffer = - PTR_ALIGN(sci_req->task_context_buffer, - SMP_CACHE_BYTES); - } + if (sci_req->was_tag_assigned_by_user == false) + sci_req->task_context_buffer = &sci_req->tc; } static void scic_sds_io_request_build_ssp_command_iu(struct scic_sds_request *sci_req) @@ -530,12 +480,8 @@ static void scic_sds_ssp_task_request_assign_buffers( sci_req->response_buffer = scic_sds_ssp_task_request_get_response_buffer(sci_req); - if (sci_req->was_tag_assigned_by_user == false) { - sci_req->task_context_buffer = - scic_sds_ssp_task_request_get_task_context_buffer(sci_req); - sci_req->task_context_buffer = - PTR_ALIGN(sci_req->task_context_buffer, SMP_CACHE_BYTES); - } + if (sci_req->was_tag_assigned_by_user == false) + sci_req->task_context_buffer = &sci_req->tc; } /** diff --git a/drivers/scsi/isci/core/scic_sds_request.h b/drivers/scsi/isci/core/scic_sds_request.h index 83d737adbc4894511fe595c7dd8a3adaf0de0f1c..8f65814e79c8ad9e2871c3a04830bd0ca26ba80e 100644 --- a/drivers/scsi/isci/core/scic_sds_request.h +++ b/drivers/scsi/isci/core/scic_sds_request.h @@ -184,6 +184,7 @@ struct scic_sds_request { void *command_buffer; void *response_buffer; struct scu_task_context *task_context_buffer; + struct scu_task_context tc ____cacheline_aligned; /* could be larger with sg chaining */ #define SCU_SGL_SIZE ((SCU_IO_REQUEST_SGE_COUNT + 1) / 2) diff --git a/drivers/scsi/isci/core/scic_sds_smp_request.c b/drivers/scsi/isci/core/scic_sds_smp_request.c index cb1adef1d559f6c11af7c6ef0278731c59ddf852..53b1260f44bcbee98fb154d9a370104913decd59 100644 --- a/drivers/scsi/isci/core/scic_sds_smp_request.c +++ b/drivers/scsi/isci/core/scic_sds_smp_request.c @@ -78,9 +78,7 @@ u32 scic_sds_smp_request_get_object_size(void) { return sizeof(struct scic_sds_request) + sizeof(struct smp_req) - + sizeof(struct smp_resp) - + sizeof(struct scu_task_context) - + SMP_CACHE_BYTES; + + sizeof(struct smp_resp); } /** @@ -102,29 +100,7 @@ u32 scic_sds_smp_request_get_object_size(void) (((char *)(scic_sds_smp_request_get_command_buffer(memory))) \ + sizeof(struct smp_req)) -/** - * scic_sds_smp_request_get_task_context_buffer() - - * - * This macro returs the task context buffer for the SMP request. - */ -#define scic_sds_smp_request_get_task_context_buffer(memory) \ - ((struct scu_task_context *)(\ - ((char *)(scic_sds_smp_request_get_response_buffer(memory))) \ - + sizeof(struct smp_resp) \ - )) - - - -/** - * This method build the remainder of the IO request object. - * @sci_req: This parameter specifies the request object being constructed. - * - * The scic_sds_general_request_construct() must be called before this call is - * valid. none - */ - -void scic_sds_smp_request_assign_buffers( - struct scic_sds_request *sci_req) +void scic_sds_smp_request_assign_buffers(struct scic_sds_request *sci_req) { /* Assign all of the buffer pointers */ sci_req->command_buffer = @@ -132,13 +108,8 @@ void scic_sds_smp_request_assign_buffers( sci_req->response_buffer = scic_sds_smp_request_get_response_buffer(sci_req); - if (sci_req->was_tag_assigned_by_user == false) { - sci_req->task_context_buffer = - scic_sds_smp_request_get_task_context_buffer(sci_req); - sci_req->task_context_buffer = - PTR_ALIGN(sci_req->task_context_buffer, SMP_CACHE_BYTES); - } - + if (sci_req->was_tag_assigned_by_user == false) + sci_req->task_context_buffer = &sci_req->tc; } /* diff --git a/drivers/scsi/isci/core/scic_sds_stp_request.c b/drivers/scsi/isci/core/scic_sds_stp_request.c index 013af11df62bc8b387509dbe8a8a17c02a3576f4..f21354571ff85c65ca3176d8c96896969610c290 100644 --- a/drivers/scsi/isci/core/scic_sds_stp_request.c +++ b/drivers/scsi/isci/core/scic_sds_stp_request.c @@ -93,18 +93,6 @@ + sizeof(struct host_to_dev_fis) \ )) -/** - * scic_sds_stp_request_get_task_context_buffer() - - * - * This macro returns the address of the task context buffer in the io request - * memory - */ -#define scic_sds_stp_request_get_task_context_buffer(memory) \ - ((struct scu_task_context *)(\ - ((char *)(scic_sds_stp_request_get_response_buffer(memory))) \ - + SSP_RESP_IU_MAX_SIZE \ - )) - /** * * @@ -114,9 +102,7 @@ u32 scic_sds_stp_request_get_object_size(void) { return sizeof(struct scic_sds_stp_request) + sizeof(struct host_to_dev_fis) - + sizeof(struct dev_to_host_fis) - + sizeof(struct scu_task_context) - + SMP_CACHE_BYTES; + + sizeof(struct dev_to_host_fis); } void scic_sds_stp_request_assign_buffers(struct scic_sds_request *sci_req) @@ -126,12 +112,8 @@ 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); - if (sci_req->was_tag_assigned_by_user == false) { - sci_req->task_context_buffer = - scic_sds_stp_request_get_task_context_buffer(stp_req); - sci_req->task_context_buffer = PTR_ALIGN(sci_req->task_context_buffer, - SMP_CACHE_BYTES); - } + if (sci_req->was_tag_assigned_by_user == false) + sci_req->task_context_buffer = &sci_req->tc; } /**