Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1736389

[PATCH 3/6] scsi: lpfc: Cocci spatch "pool_zalloc-simple"

From Thomas Meyer <thomas@m3y3r.de>
Newsgroups linux.kernel
Subject [PATCH 3/6] scsi: lpfc: Cocci spatch "pool_zalloc-simple"
Date 2017-09-21 08:30 +0200
Message-ID <us39M-5A5-17@gated-at.bofh.it> (permalink)
References <us39M-5A5-3@gated-at.bofh.it>
Organization linux.* mail to news gateway

Show all headers | View raw


Use *_pool_zalloc rather than *_pool_alloc followed by memset with 0.
Found by coccinelle spatch "api/alloc/pool_zalloc-simple.cocci"

Signed-off-by: Thomas Meyer <thomas@m3y3r.de>
---

diff -u -p a/drivers/scsi/lpfc/lpfc_nvme.c b/drivers/scsi/lpfc/lpfc_nvme.c
--- a/drivers/scsi/lpfc/lpfc_nvme.c
+++ b/drivers/scsi/lpfc/lpfc_nvme.c
@@ -1938,14 +1938,13 @@ lpfc_new_nvme_buf(struct lpfc_vport *vpo
 		 * pci bus space for an I/O. The DMA buffer includes the
 		 * number of SGE's necessary to support the sg_tablesize.
 		 */
-		lpfc_ncmd->data = dma_pool_alloc(phba->lpfc_sg_dma_buf_pool,
-						 GFP_KERNEL,
-						 &lpfc_ncmd->dma_handle);
+		lpfc_ncmd->data = dma_pool_zalloc(phba->lpfc_sg_dma_buf_pool,
+						  GFP_KERNEL,
+						  &lpfc_ncmd->dma_handle);
 		if (!lpfc_ncmd->data) {
 			kfree(lpfc_ncmd);
 			break;
 		}
-		memset(lpfc_ncmd->data, 0, phba->cfg_sg_dma_buf_size);
 
 		lxri = lpfc_sli4_next_xritag(phba);
 		if (lxri == NO_XRI) {

Back to linux.kernel | Previous | NextNext in thread | Find similar | Unroll thread


Thread

[PATCH 3/6] scsi: lpfc: Cocci spatch "pool_zalloc-simple" Thomas Meyer <thomas@m3y3r.de> - 2017-09-21 08:30 +0200
  Re: [PATCH 3/6] scsi: lpfc: Cocci spatch "pool_zalloc-simple" James Smart <james.smart@broadcom.com> - 2017-09-22 17:30 +0200
  Re: [PATCH 3/6] scsi: lpfc: Cocci spatch "pool_zalloc-simple" "Martin K. Petersen" <martin.petersen@oracle.com> - 2017-09-26 01:10 +0200

csiph-web