RE: [PATCH v3 06/24] mpi3mr: add support of event handling part-1

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



> > @@ -1924,6 +1931,12 @@ static int mpi3mr_alloc_reply_sense_bufs(struct
> mpi3mr_ioc *mrioc)
> >  	if (!mrioc->init_cmds.reply)
> >  		goto out_failed;
> >
> > +	for (i = 0; i < MPI3MR_NUM_DEVRMCMD; i++) {
> > +		mrioc->dev_rmhs_cmds[i].reply = kzalloc(mrioc-
> >facts.reply_sz,
> > +		    GFP_KERNEL);
> > +		if (!mrioc->dev_rmhs_cmds[i].reply)
> > +			goto out_failed;
> > +	}
>
> As I wrote in V2, the above allocation chunk ends up in twice in
> mpi3mr_alloc_reply_sense_bufs. Please fix it (probably in 13/24).

I missed this.  Both allocation and free part of "dev_rmhs_cmds " end up in
twice. I will fix it in 13/24.

Kashyap

Attachment: smime.p7s
Description: S/MIME Cryptographic Signature


[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Index of Archives]     [SCSI Target Devel]     [Linux SCSI Target Infrastructure]     [Kernel Newbies]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Linux IIO]     [Samba]     [Device Mapper]

  Powered by Linux