lkml.org 
[lkml]   [2013]   [Mar]   [12]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
SubjectRe: [PATCH] aic7xxx: use kzalloc instead of kmalloc + memset
From
On Sun, Mar 10, 2013 at 2:26 PM, Silviu-Mihai Popescu
<silviupopescu1990@gmail.com> wrote:
> This replaces a call to kmalloc() followed by memset() with just one
> call to kzalloc(). This was found using make coccicheck.
>
> Signed-off-by: Silviu-Mihai Popescu <silviupopescu1990@gmail.com>
> ---
> drivers/scsi/aic7xxx/aic7xxx_core.c | 9 +++------
> 1 file changed, 3 insertions(+), 6 deletions(-)
>
> diff --git a/drivers/scsi/aic7xxx/aic7xxx_core.c b/drivers/scsi/aic7xxx/aic7xxx_core.c
> index 10172a3..7f34d25 100644
> --- a/drivers/scsi/aic7xxx/aic7xxx_core.c
> +++ b/drivers/scsi/aic7xxx/aic7xxx_core.c
> @@ -4464,10 +4464,9 @@ ahc_softc_init(struct ahc_softc *ahc)
> ahc->pause = ahc->unpause | PAUSE;
> /* XXX The shared scb data stuff should be deprecated */
> if (ahc->scb_data == NULL) {
> - ahc->scb_data = kmalloc(sizeof(*ahc->scb_data), GFP_ATOMIC);
> + ahc->scb_data = kzalloc(sizeof(*ahc->scb_data), GFP_ATOMIC);
> if (ahc->scb_data == NULL)
> return (ENOMEM);
> - memset(ahc->scb_data, 0, sizeof(*ahc->scb_data));
> }
>
> return (0);
> @@ -4780,10 +4779,9 @@ ahc_init_scbdata(struct ahc_softc *ahc)
> SLIST_INIT(&scb_data->sg_maps);
>
> /* Allocate SCB resources */
> - scb_data->scbarray = kmalloc(sizeof(struct scb) * AHC_SCB_MAX_ALLOC, GFP_ATOMIC);
> + scb_data->scbarray = kzalloc(sizeof(struct scb) * AHC_SCB_MAX_ALLOC, GFP_ATOMIC);
> if (scb_data->scbarray == NULL)
> return (ENOMEM);
> - memset(scb_data->scbarray, 0, sizeof(struct scb) * AHC_SCB_MAX_ALLOC);
>
> /* Determine the number of hardware SCBs and initialize them */
>
> @@ -7558,14 +7556,13 @@ ahc_handle_en_lun(struct ahc_softc *ahc, struct cam_sim *sim, union ccb *ccb)
> return;
> }
> }
> - lstate = kmalloc(sizeof(*lstate), GFP_ATOMIC);
> + lstate = kzalloc(sizeof(*lstate), GFP_ATOMIC);
> if (lstate == NULL) {
> xpt_print_path(ccb->ccb_h.path);
> printk("Couldn't allocate lstate\n");
> ccb->ccb_h.status = CAM_RESRC_UNAVAIL;
> return;
> }
> - memset(lstate, 0, sizeof(*lstate));
> status = xpt_create_path(&lstate->path, /*periph*/NULL,
> xpt_path_path_id(ccb->ccb_h.path),
> xpt_path_target_id(ccb->ccb_h.path),
> --
> 1.7.9.5
>

Hello,

Would anyone be so kind as to review this?

Thanks,
Silviu Popescu


\
 
 \ /
  Last update: 2013-03-12 08:21    [W:0.186 / U:0.020 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site