> -----Original Message----- > From: Miguel Bernal Marin [mailto:miguel.bernal.marin@xxxxxxxxxxxxxxx] > Sent: Wednesday, March 15, 2017 11:58 PM > To: KY Srinivasan <kys@xxxxxxxxxxxxx>; Martin K. Petersen > <martin.petersen@xxxxxxxxxx> > Cc: linux-kernel@xxxxxxxxxxxxxxx; linux-scsi@xxxxxxxxxxxxxxx; > devel@xxxxxxxxxxxxxxxxxxxxxx; Haiyang Zhang <haiyangz@xxxxxxxxxxxxx>; > Stephen Hemminger <sthemmin@xxxxxxxxxxxxx>; James E.J. Bottomley > <jejb@xxxxxxxxxxxxxxxxxx> > Subject: [PATCH] scsi: storvsc: Prefer kcalloc over kzalloc with multiply > > Use kcalloc for allocating an array instead of kzalloc with multiply, > kcalloc is the preferred API. > > Found with checkpatch. > > Signed-off-by: Miguel Bernal Marin <miguel.bernal.marin@xxxxxxxxxxxxxxx> Reviewed-by: K. Y. Srinivasan <kys@xxxxxxxxxxxxx> > --- > drivers/scsi/storvsc_drv.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/scsi/storvsc_drv.c b/drivers/scsi/storvsc_drv.c > index 638e5f427c90..3d70d1cf49a3 100644 > --- a/drivers/scsi/storvsc_drv.c > +++ b/drivers/scsi/storvsc_drv.c > @@ -866,7 +866,7 @@ static int storvsc_channel_init(struct hv_device > *device, bool is_fc) > * We will however populate all the slots to evenly distribute > * the load. > */ > - stor_device->stor_chns = kzalloc(sizeof(void *) * > num_possible_cpus(), > + stor_device->stor_chns = kcalloc(num_possible_cpus(), sizeof(void > *), > GFP_KERNEL); > if (stor_device->stor_chns == NULL) > return -ENOMEM; > -- > 2.12.0
