Hi Martin, Any comment on this patch? On Thu, Dec 1, 2016 at 11:34 AM, Souptick Joarder <jrdr.linux@xxxxxxxxx> wrote: > On Mon, Nov 28, 2016 at 4:56 PM, Souptick Joarder <jrdr.linux@xxxxxxxxx> wrote: >> Inside mvs_task_prep(), pci_pool_alloc() followed by memset will be >> replaced by pci_pool_zalloc() >> >> Signed-off-by: Souptick joarder <jrdr.linux@xxxxxxxxx> >> --- >> drivers/scsi/mvsas/mv_sas.c | 3 +-- >> 1 file changed, 1 insertion(+), 2 deletions(-) >> >> diff --git a/drivers/scsi/mvsas/mv_sas.c b/drivers/scsi/mvsas/mv_sas.c >> index 86eb199..681e5f7 100644 >> --- a/drivers/scsi/mvsas/mv_sas.c >> +++ b/drivers/scsi/mvsas/mv_sas.c >> @@ -790,10 +790,9 @@ static int mvs_task_prep(struct sas_task *task, struct mvs_info *mvi, int is_tmf >> slot->n_elem = n_elem; >> slot->slot_tag = tag; >> >> - slot->buf = pci_pool_alloc(mvi->dma_pool, GFP_ATOMIC, &slot->buf_dma); >> + slot->buf = pci_pool_zalloc(mvi->dma_pool, GFP_ATOMIC, &slot->buf_dma); >> if (!slot->buf) >> goto err_out_tag; >> - memset(slot->buf, 0, MVS_SLOT_BUF_SZ); >> >> tei.task = task; >> tei.hdr = &mvi->slot[tag]; >> -- >> 1.9.1 > > Any Comment on this patch? >> Regards Souptick -- To unsubscribe from this list: send the line "unsubscribe linux-scsi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
