diff mbox

scsi: don't reinitialize adapter's req_table

Message ID 1471747164-31441-1-git-send-email-shawn.lin@rock-chips.com (mailing list archive)
State Accepted, archived
Headers show

Commit Message

Shawn Lin Aug. 21, 2016, 2:39 a.m. UTC
req_table is allocate by kzalloc, so we don't need to
zero it again anyway.

Signed-off-by: Shawn Lin <shawn.lin@rock-chips.com>
---

 drivers/scsi/esas2r/esas2r_init.c | 4 ----
 1 file changed, 4 deletions(-)

Comments

Grove, Bradley Aug. 25, 2016, 6:45 p.m. UTC | #1
Acked-by: Bradley Grove <bgrove@attotech.com>

On 08/20/2016 10:39 PM, Shawn Lin wrote:
> req_table is allocate by kzalloc, so we don't need to
> zero it again anyway.
>
> Signed-off-by: Shawn Lin <shawn.lin@rock-chips.com>
> ---
>
>  drivers/scsi/esas2r/esas2r_init.c | 4 ----
>  1 file changed, 4 deletions(-)
>
> diff --git a/drivers/scsi/esas2r/esas2r_init.c b/drivers/scsi/esas2r/esas2r_init.c
> index 78ce4d61..d6e53ae 100644
> --- a/drivers/scsi/esas2r/esas2r_init.c
> +++ b/drivers/scsi/esas2r/esas2r_init.c
> @@ -963,10 +963,6 @@ bool esas2r_init_adapter_struct(struct esas2r_adapter *a,
>
>  	/* initialize the allocated memory */
>  	if (test_bit(AF_FIRST_INIT, &a->flags)) {
> -		memset(a->req_table, 0,
> -		       (num_requests + num_ae_requests +
> -			1) * sizeof(struct esas2r_request *));
> -
>  		esas2r_targ_db_initialize(a);
>
>  		/* prime parts of the inbound list */
>




This electronic transmission and any attachments hereto are intended only for the use of the individual or entity to which it is addressed and may contain confidential information belonging to ATTO Technology, Inc. If you have reason to believe that you are not the intended recipient, you are hereby notified that any disclosure, copying, distribution or the taking of any action in reliance on the contents of this electronic transmission is strictly prohibited. If you have reason to believe that you have received this transmission in error, please notify ATTO immediately by return e-mail and delete and destroy this communication.   
--
To unsubscribe from this list: send the line "unsubscribe linux-scsi" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Martin K. Petersen Aug. 26, 2016, 2:29 a.m. UTC | #2
>>>>> "Shawn" == Shawn Lin <shawn.lin@rock-chips.com> writes:

Shawn> req_table is allocate by kzalloc, so we don't need to zero it
Shawn> again anyway.

Applied to 4.9/scsi-queue.
diff mbox

Patch

diff --git a/drivers/scsi/esas2r/esas2r_init.c b/drivers/scsi/esas2r/esas2r_init.c
index 78ce4d61..d6e53ae 100644
--- a/drivers/scsi/esas2r/esas2r_init.c
+++ b/drivers/scsi/esas2r/esas2r_init.c
@@ -963,10 +963,6 @@  bool esas2r_init_adapter_struct(struct esas2r_adapter *a,
 
 	/* initialize the allocated memory */
 	if (test_bit(AF_FIRST_INIT, &a->flags)) {
-		memset(a->req_table, 0,
-		       (num_requests + num_ae_requests +
-			1) * sizeof(struct esas2r_request *));
-
 		esas2r_targ_db_initialize(a);
 
 		/* prime parts of the inbound list */