RE: [EXT] [PATCH 1/2] scsi: qedi: Fix error codes in qedi_alloc_global_queues()

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

 



> -----Original Message-----
> From: Dan Carpenter <dan.carpenter@xxxxxxxxxx>
> Sent: Tuesday, August 10, 2021 2:18 PM
> To: Nilesh Javali <njavali@xxxxxxxxxxx>; Manish Rangankar
> <manish.rangankar@xxxxxxxxxx>
> Cc: GR-QLogic-Storage-Upstream <GR-QLogic-Storage-
> Upstream@xxxxxxxxxxx>; James E.J. Bottomley <jejb@xxxxxxxxxxxxx>; Martin
> K. Petersen <martin.petersen@xxxxxxxxxx>; Hannes Reinecke <hare@xxxxxxx>;
> Arun Easi <arun.easi@xxxxxxxxxx>; Adheer Chandravanshi
> <adheer.chandravanshi@xxxxxxxxxx>; Johannes Thumshirn
> <jthumshirn@xxxxxxx>; linux-scsi@xxxxxxxxxxxxxxx; kernel-
> janitors@xxxxxxxxxxxxxxx
> Subject: [EXT] [PATCH 1/2] scsi: qedi: Fix error codes in
> qedi_alloc_global_queues()
> 
> External Email
> 
> ----------------------------------------------------------------------
> This function had some left over code that returned 1 on error instead negative
> error codes.  Convert everything to use negative error codes.
> The caller treats all non-zero returns the same so this does not affect run time.
> 
> A couple places set "rc" instead of "status" so those error paths ended up
> returning success by mistake.  Get rid of the "rc" variable and use "status"
> everywhere.
> 
> Remove the bogus "status = 0" initialization, as a future proofing measure so the
> compiler will warn about uninitialized error codes.
> 
> Fixes: ace7f46ba5fd ("scsi: qedi: Add QLogic FastLinQ offload iSCSI driver
> framework.")
> Signed-off-by: Dan Carpenter <dan.carpenter@xxxxxxxxxx>
> ---
>  drivers/scsi/qedi/qedi_main.c | 14 +++++++-------
>  1 file changed, 7 insertions(+), 7 deletions(-)
> 
> diff --git a/drivers/scsi/qedi/qedi_main.c b/drivers/scsi/qedi/qedi_main.c index
> 0b0acb827071..e6dc0b495a82 100644
> --- a/drivers/scsi/qedi/qedi_main.c
> +++ b/drivers/scsi/qedi/qedi_main.c
> @@ -1621,7 +1621,7 @@ static int qedi_alloc_global_queues(struct qedi_ctx
> *qedi)  {
>  	u32 *list;
>  	int i;
> -	int status = 0, rc;
> +	int status;
>  	u32 *pbl;
>  	dma_addr_t page;
>  	int num_pages;
> @@ -1632,14 +1632,14 @@ static int qedi_alloc_global_queues(struct qedi_ctx
> *qedi)
>  	 */
>  	if (!qedi->num_queues) {
>  		QEDI_ERR(&qedi->dbg_ctx, "No MSI-X vectors available!\n");
> -		return 1;
> +		return -ENOMEM;
>  	}
> 
>  	/* Make sure we allocated the PBL that will contain the physical
>  	 * addresses of our queues
>  	 */
>  	if (!qedi->p_cpuq) {
> -		status = 1;
> +		status = -EINVAL;
>  		goto mem_alloc_failure;
>  	}
> 
> @@ -1654,13 +1654,13 @@ static int qedi_alloc_global_queues(struct qedi_ctx
> *qedi)
>  		  "qedi->global_queues=%p.\n", qedi->global_queues);
> 
>  	/* Allocate DMA coherent buffers for BDQ */
> -	rc = qedi_alloc_bdq(qedi);
> -	if (rc)
> +	status = qedi_alloc_bdq(qedi);
> +	if (status)
>  		goto mem_alloc_failure;
> 
>  	/* Allocate DMA coherent buffers for NVM_ISCSI_CFG */
> -	rc = qedi_alloc_nvm_iscsi_cfg(qedi);
> -	if (rc)
> +	status = qedi_alloc_nvm_iscsi_cfg(qedi);
> +	if (status)
>  		goto mem_alloc_failure;
> 
>  	/* Allocate a CQ and an associated PBL for each MSI-X
> --
> 2.20.1

Thanks,
Acked-by: Manish Rangankar <mrangankar@xxxxxxxxxxx>




[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