[-next] scsi: lpfc: Use memset_startat() helper

Message ID 20221111074310.132125-1-xiujianfeng@huawei.com
State New
Headers
Series [-next] scsi: lpfc: Use memset_startat() helper |

Commit Message

Xiu Jianfeng Nov. 11, 2022, 7:43 a.m. UTC
  User memset_startat() helper to simplify the code, no functional
changes in this patch.

Signed-off-by: Xiu Jianfeng <xiujianfeng@huawei.com>
---
 drivers/scsi/lpfc/lpfc_sli.c | 6 ++----
 1 file changed, 2 insertions(+), 4 deletions(-)
  

Comments

James Smart Nov. 12, 2022, 4:50 p.m. UTC | #1
On 11/10/2022 11:43 PM, Xiu Jianfeng wrote:
> User memset_startat() helper to simplify the code, no functional
> changes in this patch.
> 
> Signed-off-by: Xiu Jianfeng <xiujianfeng@huawei.com>

Looks good

Reviewed-by: James Smart <jsmart2021@gmail.com>

-- james
  
Martin K. Petersen Nov. 17, 2022, 6:13 p.m. UTC | #2
Xiu,

> User memset_startat() helper to simplify the code, no functional
> changes in this patch.

Applied to 6.2/scsi-staging, thanks!
  
Martin K. Petersen Nov. 26, 2022, 3:27 a.m. UTC | #3
On Fri, 11 Nov 2022 15:43:10 +0800, Xiu Jianfeng wrote:

> User memset_startat() helper to simplify the code, no functional
> changes in this patch.
> 
> 

Applied to 6.2/scsi-queue, thanks!

[1/1] scsi: lpfc: Use memset_startat() helper
      https://git.kernel.org/mkp/scsi/c/b29e91385ce2
  

Patch

diff --git a/drivers/scsi/lpfc/lpfc_sli.c b/drivers/scsi/lpfc/lpfc_sli.c
index d25afc9dde14..35c9404d5e49 100644
--- a/drivers/scsi/lpfc/lpfc_sli.c
+++ b/drivers/scsi/lpfc/lpfc_sli.c
@@ -1373,7 +1373,6 @@  static void
 __lpfc_sli_release_iocbq_s4(struct lpfc_hba *phba, struct lpfc_iocbq *iocbq)
 {
 	struct lpfc_sglq *sglq;
-	size_t start_clean = offsetof(struct lpfc_iocbq, wqe);
 	unsigned long iflag = 0;
 	struct lpfc_sli_ring *pring;
 
@@ -1430,7 +1429,7 @@  __lpfc_sli_release_iocbq_s4(struct lpfc_hba *phba, struct lpfc_iocbq *iocbq)
 	/*
 	 * Clean all volatile data fields, preserve iotag and node struct.
 	 */
-	memset((char *)iocbq + start_clean, 0, sizeof(*iocbq) - start_clean);
+	memset_startat(iocbq, 0, wqe);
 	iocbq->sli4_lxritag = NO_XRI;
 	iocbq->sli4_xritag = NO_XRI;
 	iocbq->cmd_flag &= ~(LPFC_IO_NVME | LPFC_IO_NVMET | LPFC_IO_CMF |
@@ -1453,12 +1452,11 @@  __lpfc_sli_release_iocbq_s4(struct lpfc_hba *phba, struct lpfc_iocbq *iocbq)
 static void
 __lpfc_sli_release_iocbq_s3(struct lpfc_hba *phba, struct lpfc_iocbq *iocbq)
 {
-	size_t start_clean = offsetof(struct lpfc_iocbq, iocb);
 
 	/*
 	 * Clean all volatile data fields, preserve iotag and node struct.
 	 */
-	memset((char*)iocbq + start_clean, 0, sizeof(*iocbq) - start_clean);
+	memset_startat(iocbq, 0, iocb);
 	iocbq->sli4_xritag = NO_XRI;
 	list_add_tail(&iocbq->list, &phba->lpfc_iocb_list);
 }