Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752881AbbKVXOv (ORCPT ); Sun, 22 Nov 2015 18:14:51 -0500 Received: from mout.gmx.net ([212.227.15.15]:53779 "EHLO mout.gmx.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752193AbbKVXOt (ORCPT ); Sun, 22 Nov 2015 18:14:49 -0500 Date: Mon, 23 Nov 2015 00:14:37 +0100 From: Sebastian Herbszt To: Saurabh Sengar Cc: james.smart@avagotech.com, dick.kennedy@avagotech.com, JBottomley@odin.com, linux-scsi@vger.kernel.org, linux-kernel@vger.kernel.org, Sebastian Herbszt Subject: Re: [PATCH v2] lpfc: replaced kmalloc + memset with kzalloc Message-ID: <20151123001437.000018a1@localhost> In-Reply-To: <1447918452-15299-1-git-send-email-saurabh.truth@gmail.com> References: <1446015219-16131-1-git-send-email-saurabh.truth@gmail.com> <1447918452-15299-1-git-send-email-saurabh.truth@gmail.com> X-Mailer: Claws Mail 3.9.2-55-g74b05b (GTK+ 2.16.6; i586-pc-mingw32msvc) MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit X-Provags-ID: V03:K0:xvKTAAf/9KaR09sKBR2NDnbgVs3449gkRSU6C4v+oJeeRfAiHa7 3r14QtdJ+gXt0PewlE26bPRH1WzgzaqbzDqSznd5MCYH/GwrX1HN/xpDylNu2u9o47MpvAq dFSkUvBnN55RkOzAT6Stu1CvnRMrS+BCcpjcFrcXFE4+qX1rrdBUeD5J2W8OYmsNZ+N157s AxQcKJfWCQTixF20qpedQ== X-UI-Out-Filterresults: notjunk:1;V01:K0:fRUwy3DrIrI=:mfK8k0vCcpEroUO7uIUqoW 93lnzdZtmvhjjV/oaen5L7scvW/eBEC4OjEyF8SyTslYQJBd98dQlwNVsHB+cmLVdlSTbEtpz /vUU3kZ0wqam2uNZqJPQTdCoDzPRI6rgB7tYFXA1LyoRk3c6eX3dHv33ThgWhXh7Mht96Td29 1pBpI1HVkdCGcAKnDiq3ElFgg2eZFQbpYOUDd2fy10gg3ALB+wO3Itj/c2P/A8IHwRI1lW4/r 7cmT2OIiDLnghbQ60tYo4wRUHM+7VYlj5bbll7a5Ey79d7su2drFfcf+ZIF67vSI+Glmb0/vY Rwnu4/+8kAgV6ZSvuKuEzJg6qKxU/xKYyyYaPz1sq5au/hmGKm5Cw6QCI3qRTiMIaN7teGw0M 6V3SV7BzU+0x8Vg5r4cMdKXl7AdJYFKJVBlLBfuKMh5fcvvMaHao0INKU6stgh0ynzDVTd1Qf 2T576mXLbF26oC6oZssvVxkrQRYxCvLnM54bhDKrl5GMnjWBJzurSHBw7j6tkhi2ddRmBLswj PfjESrLXXJXO6t6pyD5ienYahsvM5f6RTBkVMvIP+oSJfwEHXVV2o5sqQfJyDAD66CsQRSvM7 gh2wpLGw3E+CVgrFu6c/Hdabwk36ACTEmw4v0QC73tu/nIaaDWXmjhZ+ADO7SWbcWV2pnKdUE +kB+ZnfnbBFQ+RzBQxH61iO6IgjHPVcstfAL8BKX9wcoLi/51hGUfS03IAvcysg7o+XZkpBUT osQHZNz9P7BfqQ15zAqhh8tasHjO9WhuacwenSVx2/XmYJJwX0pj98Uxt19d14FTh/p+7rbMa zms/l1b Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1460 Lines: 39 Saurabh Sengar wrote: > replacing kmalloc and memset by a single call of kzalloc > > Signed-off-by: Saurabh Sengar > --- > v2 : I didn't got any response for my initial patch, > I am sending it again on top of latest kernel(today's) > > drivers/scsi/lpfc/lpfc_els.c | 3 +-- > 1 file changed, 1 insertion(+), 2 deletions(-) > > diff --git a/drivers/scsi/lpfc/lpfc_els.c b/drivers/scsi/lpfc/lpfc_els.c > index b6fa257..92dd204 100644 > --- a/drivers/scsi/lpfc/lpfc_els.c > +++ b/drivers/scsi/lpfc/lpfc_els.c > @@ -4956,13 +4956,12 @@ lpfc_els_rcv_rdp(struct lpfc_vport *vport, struct lpfc_iocbq *cmdiocb, > if (RDP_NPORT_ID_SIZE != > be32_to_cpu(rdp_req->nport_id_desc.length)) > goto rjt_logerr; > - rdp_context = kmalloc(sizeof(struct lpfc_rdp_context), GFP_KERNEL); > + rdp_context = kzalloc(sizeof(struct lpfc_rdp_context), GFP_KERNEL); > if (!rdp_context) { > rjt_err = LSRJT_UNABLE_TPC; > goto error; > } > > - memset(rdp_context, 0, sizeof(struct lpfc_rdp_context)); > cmd = &cmdiocb->iocb; > rdp_context->ndlp = lpfc_nlp_get(ndlp); > rdp_context->ox_id = cmd->unsli3.rcvsli3.ox_id; Same patch was already sent by Punit Vara on 2015-10-24. Sebastian -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/