> -----Original Message----- > From: Miguel Bernal Marin [mailto:miguel.bernal.ma...@linux.intel.com] > Sent: Wednesday, March 15, 2017 11:58 PM > To: KY Srinivasan <k...@microsoft.com>; Martin K. Petersen > <martin.peter...@oracle.com> > Cc: linux-kernel@vger.kernel.org; linux-s...@vger.kernel.org; > de...@linuxdriverproject.org; Haiyang Zhang <haiya...@microsoft.com>; > Stephen Hemminger <sthem...@microsoft.com>; James E.J. Bottomley > <j...@linux.vnet.ibm.com> > 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.ma...@linux.intel.com> Reviewed-by: K. Y. Srinivasan <k...@microsoft.com>
> --- > 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
