From patchwork Thu Dec 28 17:22:01 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Markus Elfring X-Patchwork-Id: 183793 Return-Path: Delivered-To: ouuuleilei@gmail.com Received: by 2002:a05:7301:6f82:b0:100:9c79:88ff with SMTP id tb2csp2127295dyb; Thu, 28 Dec 2023 09:22:38 -0800 (PST) X-Google-Smtp-Source: AGHT+IGg1GFT7GA/DzejUJeNk++/STVVf25Mggfu1l6ekkA5Hc35y4ocl3aVWf3t4DT8DP2JhtXu X-Received: by 2002:a05:6830:4414:b0:6d9:d36a:8cc0 with SMTP id q20-20020a056830441400b006d9d36a8cc0mr11016167otv.28.1703784158157; Thu, 28 Dec 2023 09:22:38 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1703784158; cv=none; d=google.com; s=arc-20160816; b=U+tCQCgrv1yaIGVrHb0sXKZZQLkakMkFoAhJzbC0Ih4NT13TfXXs4FX3BaGydXEofF IF0BnNBqp8TeDuRNScU+DUTbm5skW7z4Ki+44FQFZpADW8fgwpEADsgAvTAlkY6SDyAq TI48M33OKLy5EklO6XmmTGdP1Fnlf0nsrXy+DH5V8DrnTxTLZGOrdUNibsDdVakPXGAE 84Qc2gldVEzZBOpRGfzBa04+BJ7xrl/7vKjI6TAd6p6GvcibeYo7LdvEjRDb482gsHPb sPODdmkcoRN1tuO7zBu5M1QxmHMmD0+DnhFLRkZEVsd8NcTB4zMYSRgv46u6c7lMle0B UGqw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=ui-outboundreport:content-transfer-encoding:in-reply-to:references :cc:to:from:content-language:subject:user-agent:mime-version :list-unsubscribe:list-subscribe:list-id:precedence:date:message-id :dkim-signature; bh=RWtJZMdrLq9uoy0WoBVJuixX5u79svqqm/Keg8om1kI=; fh=UF3DLcBgV+qVMrcMwsk5goUAvcQPQpxbgR5Yw1H6KJk=; b=bp1rBFqjdr8JKwfSEucH6bDKrwG38iRuABy8zniQ6d+9z80It6kKaKlKTk6iFhZ6zg W4WFN1Tiqx1USknzPavBROZsn0qKSzfz2sFSHNYgQ92blfGFOHjKJEGwxGhp63L71wg1 1lwyJHZb2X4lpPWu/j4jlx0qrNISOWFkznviGjmcNXS8mXfmFo4InJVkA8engR7kBq9+ vDmpEu1ofWT/7Z7gcqxGV65A1IF7vS1ExS6Cj+7wiP62ZynDOvj0aRBXZ96KlTMd80ai fdrKjsghtE/TV6uIly0DttXZwh+ir67cwY8BOY1zePR/RK6wTnbWxR6KP+vG+kpIZM2g L2Mg== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@web.de header.s=s29768273 header.b=qNO5iXGk; spf=pass (google.com: domain of linux-kernel+bounces-12810-ouuuleilei=gmail.com@vger.kernel.org designates 2604:1380:45d1:ec00::1 as permitted sender) smtp.mailfrom="linux-kernel+bounces-12810-ouuuleilei=gmail.com@vger.kernel.org"; dmarc=pass (p=QUARANTINE sp=QUARANTINE dis=NONE) header.from=web.de Received: from ny.mirrors.kernel.org (ny.mirrors.kernel.org. [2604:1380:45d1:ec00::1]) by mx.google.com with ESMTPS id i14-20020ac85c0e000000b0042800c9f9b4si354263qti.236.2023.12.28.09.22.38 for (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 28 Dec 2023 09:22:38 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel+bounces-12810-ouuuleilei=gmail.com@vger.kernel.org designates 2604:1380:45d1:ec00::1 as permitted sender) client-ip=2604:1380:45d1:ec00::1; Authentication-Results: mx.google.com; dkim=pass header.i=@web.de header.s=s29768273 header.b=qNO5iXGk; spf=pass (google.com: domain of linux-kernel+bounces-12810-ouuuleilei=gmail.com@vger.kernel.org designates 2604:1380:45d1:ec00::1 as permitted sender) smtp.mailfrom="linux-kernel+bounces-12810-ouuuleilei=gmail.com@vger.kernel.org"; dmarc=pass (p=QUARANTINE sp=QUARANTINE dis=NONE) header.from=web.de Received: from smtp.subspace.kernel.org (wormhole.subspace.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ny.mirrors.kernel.org (Postfix) with ESMTPS id D70951C20D52 for ; Thu, 28 Dec 2023 17:22:37 +0000 (UTC) Received: from localhost.localdomain (localhost.localdomain [127.0.0.1]) by smtp.subspace.kernel.org (Postfix) with ESMTP id A833F101E8; Thu, 28 Dec 2023 17:22:17 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=web.de header.i=markus.elfring@web.de header.b="qNO5iXGk" X-Original-To: linux-kernel@vger.kernel.org Received: from mout.web.de (mout.web.de [217.72.192.78]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id D3D78F9C7; Thu, 28 Dec 2023 17:22:12 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=web.de Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=web.de DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=web.de; s=s29768273; t=1703784122; x=1704388922; i=markus.elfring@web.de; bh=X/gKRB8Ifpq3Kc0ZFe/BKNuVoOjfjzMQxZThCkDRvjc=; h=X-UI-Sender-Class:Date:Subject:From:To:Cc:References: In-Reply-To; b=qNO5iXGkYXAaJJk/swBDMVii4SHZm7mEsIRQSq+9a9WzXT7r2XBGPbQSYzvZ2POu AowrRyal6e8iaBKxqohIc3OjBRcgHaSbOU/7eSCYf28X4EhgC7KeBJatiq3uZ7Zqs lX0t4YV5ZpM41hB7A6UTPC2gOEyWFPC65fi/CE8jPBk077iHrdiZEm/7/8CDSGDYx ZBcZ+V0v9RB20lwwFwREjviWJEqhk85DadTAxS4wbAcdFjAzi7rTgd/VCcB9QMLMK wGLQxZpK53IImxal0yCpY/HrkTxzN0mU5wUFsaK33KFDH/2EZPbVcEm5yOFJ+pRDF txmNlJJwhrzXERyhWA== X-UI-Sender-Class: 814a7b36-bfc1-4dae-8640-3722d8ec6cd6 Received: from [192.168.178.21] ([94.31.85.95]) by smtp.web.de (mrweb105 [213.165.67.124]) with ESMTPSA (Nemesis) id 1MZjIb-1rmvHR1T8R-00Wt5l; Thu, 28 Dec 2023 18:22:02 +0100 Message-ID: <562d25d9-9550-4dc2-a7d9-40af3109e3db@web.de> Date: Thu, 28 Dec 2023 18:22:01 +0100 Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: [PATCH 1/4] scsi: lpfc: Improve exception handling in lpfc_rcv_plogi() Content-Language: en-GB From: Markus Elfring To: linux-scsi@vger.kernel.org, kernel-janitors@vger.kernel.org, Dick Kennedy , James Smart , "James E. J. Bottomley" , "Martin K. Petersen" Cc: LKML References: <48a127e8-ca4f-4fc4-81b7-226080fba720@web.de> In-Reply-To: <48a127e8-ca4f-4fc4-81b7-226080fba720@web.de> X-Provags-ID: V03:K1:PfPgGX9wqxCqCOCcwdoypN85NPCwwIWH1fFjD6sPXjtEvgFZjiy CFtls3X4+2+fyyKPS6F/wvPlaZU6YGMXuT2vNcHqfH8EHI1+MxzublFag9fOQdACFyYKoHR Vf/ooIuhH+dB0v0IZ3njZ5FLtH34rwkHAfQWeINzH8mFHO1zHvdjYVUZCuliAcUMgpwMxv+ TxY8NtMqAsFmvXn7mpPww== X-Spam-Flag: NO UI-OutboundReport: notjunk:1;M01:P0:PGu0RpL5zzo=;7LyogwglXRvDOIz4ELTClkKFIC/ 2xHXh1dzuYt8S0eY8JKF6InUpgW8dDJuCGyO3TrM6kwfbW3PC8l3ufmpHbkiUMCGd27CxtWQM TWJTaWeGYNOi5gmkdva+/+NNl22kzNS6AjWFJSILzmcvR3Vz2Tez86ytfBHOkkwA6L9CaVNq1 6WfvGOHyz0I6XlnfSq6lazjgRdBHtekK0a7vR9Z4jBwW44adLLABHJhWDe5IhnVcFtoxA7yc4 US+krwNqKcW5hflyNTauCbDJEbR0BIC2HDRQfB3Su/ulqgNDwediuytz4Oou6Edf/rUxrnoW+ 5MhF2M6AKJ3nuYeDOJpDr99aMl3aapmGNp8NOgq5fDVNcbH4Lj2dluVNNDDDOSX46+/LvJMrE 42nY/ykz2nMGigXTZbR9Ak2k4/FsEa7drHsqwXZmDaEoCjg1b+iuS0GywvcW7DB9OTC6ac4ab Lf7jx8Lmc4iWIl4g3NTIQsPiSkJClZ3mW8Mq0oK1IRRnz9GZy5KCI28lvmp5ln2vyZCPvOPYn gAiR3JcHkgJgI3h6pDtQDJe+tDHeJoBucFvK/LPslV7Yk/VQP8BlDyalsRpFuKj8r8ZYTvES2 xdTugIPhQcuxrqPHoHR08dSSZZI5Se89lQ20ePKACV8xbp3hjGSGMoP8N3bRDxeJ85KN5LNZ5 Mnpf0Qc+b+H/wLTOmClA1a+mXmwGNJfMOJHVJ7z2V73ssWAnRmbOf1jCbCp7DML9NRITWQjQT cuIa6ySqhq7IdMBNwEZJ4SKxVuE7DOUfve9h2v4tSE/Jm+npsNIV5gb1cVvKxtCDBbTUoPRD0 tw2+A1OPKD/s8ynzLQBqmIEFtLGvdi62CLP6Jo6O9DBNXPhc0F9OyiBOHB0ka0x0KtXw48p17 wvZMUh0beuhx9kojioKc+6hQFBFuzmsnCjZdPvXpGOBvDoFE19UJfj5UbIFgwQsB8GR8Svnxr NSOyHGPhMphz0p2NUqAufDfw50A= X-getmail-retrieved-from-mailbox: INBOX X-GMAIL-THRID: 1786547177402834694 X-GMAIL-MSGID: 1786547177402834694 From: Markus Elfring Date: Thu, 28 Dec 2023 17:00:07 +0100 The kfree() function was called in one case by the lpfc_rcv_plogi() function during error handling even if the passed variable contained a null pointer. This issue was detected by using the Coccinelle software. * Thus adjust jump targets. * Delete three variable assignments and a repeated pointer check which became unnecessary with this refactoring. Signed-off-by: Markus Elfring --- drivers/scsi/lpfc/lpfc_nportdisc.c | 18 +++++++----------- 1 file changed, 7 insertions(+), 11 deletions(-) -- 2.43.0 diff --git a/drivers/scsi/lpfc/lpfc_nportdisc.c b/drivers/scsi/lpfc/lpfc_nportdisc.c index d9074929fbab..2559f2ac7277 100644 --- a/drivers/scsi/lpfc/lpfc_nportdisc.c +++ b/drivers/scsi/lpfc/lpfc_nportdisc.c @@ -459,10 +459,6 @@ lpfc_rcv_plogi(struct lpfc_vport *vport, struct lpfc_nodelist *ndlp, ndlp->nlp_nvme_info &= ~NLP_NVME_NSLER; ndlp->nlp_flag &= ~NLP_FIRSTBURST; - login_mbox = NULL; - link_mbox = NULL; - save_iocb = NULL; - /* Check for Nport to NPort pt2pt protocol */ if ((vport->fc_flag & FC_PT2PT) && !(vport->fc_flag & FC_PT2PT_PLOGI)) { @@ -509,7 +505,7 @@ lpfc_rcv_plogi(struct lpfc_vport *vport, struct lpfc_nodelist *ndlp, link_mbox = mempool_alloc(phba->mbox_mem_pool, GFP_KERNEL); if (!link_mbox) - goto out; + goto reject_response; lpfc_config_link(phba, link_mbox); link_mbox->mbox_cmpl = lpfc_sli_def_mbox_cmpl; link_mbox->vport = vport; @@ -522,7 +518,7 @@ lpfc_rcv_plogi(struct lpfc_vport *vport, struct lpfc_nodelist *ndlp, rc = lpfc_sli_issue_mbox(phba, link_mbox, MBX_NOWAIT); if (rc == MBX_NOT_FINISHED) { mempool_free(link_mbox, phba->mbox_mem_pool); - goto out; + goto reject_response; } } @@ -540,11 +536,11 @@ lpfc_rcv_plogi(struct lpfc_vport *vport, struct lpfc_nodelist *ndlp, login_mbox = mempool_alloc(phba->mbox_mem_pool, GFP_KERNEL); if (!login_mbox) - goto out; + goto reject_response; save_iocb = kzalloc(sizeof(*save_iocb), GFP_KERNEL); if (!save_iocb) - goto out; + goto free_mempool; /* Save info from cmd IOCB to be used in rsp after all mbox completes */ memcpy((uint8_t *)save_iocb, (uint8_t *)cmdiocb, @@ -657,9 +653,9 @@ lpfc_rcv_plogi(struct lpfc_vport *vport, struct lpfc_nodelist *ndlp, return 1; out: kfree(save_iocb); - if (login_mbox) - mempool_free(login_mbox, phba->mbox_mem_pool); - +free_mempool: + mempool_free(login_mbox, phba->mbox_mem_pool); +reject_response: stat.un.b.lsRjtRsnCode = LSRJT_UNABLE_TPC; stat.un.b.lsRjtRsnCodeExp = LSEXP_OUT_OF_RESOURCE; lpfc_els_rsp_reject(vport, stat.un.lsRjtError, cmdiocb, ndlp, NULL); From patchwork Thu Dec 28 17:26:06 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: Markus Elfring X-Patchwork-Id: 183794 Return-Path: Delivered-To: ouuuleilei@gmail.com Received: by 2002:a05:7301:6f82:b0:100:9c79:88ff with SMTP id tb2csp2129476dyb; Thu, 28 Dec 2023 09:26:58 -0800 (PST) X-Google-Smtp-Source: AGHT+IFCmNgE0ALrFP4Hg7q6oD/UBAcdT/Ri+Z6gJ2b+jLClCBA3X4rm3JFzqZylKK0f4NVq4iQa X-Received: by 2002:a05:6e02:190f:b0:360:f9:e54b with SMTP id w15-20020a056e02190f00b0036000f9e54bmr8641929ilu.7.1703784418677; Thu, 28 Dec 2023 09:26:58 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1703784418; cv=none; d=google.com; s=arc-20160816; b=aQ+f6zhX0ncqqyTwqgS2Ozkb6q0FrAdiYvpahn0lCaBNEb/L1NJ6e6dNB1WUtvI9v4 8XqulSu3coEpbvV5repyQZLef9T4wiJ59H8XmX/wiveivFiwuc+WFeoXrHzMpXUIOWeb 8W2NAcIgEcKvsxsewwLXQ5cF4W3CRpssFWpEmktlk97654jzCCoM/MEAFuazt4gdKvZi iKNBR3n+1XNireV1yvbTPeIgquFVHFzY5qReGGV6wTmTJTayqIYM6uPDTyR/pX+zdzhn W064K9RgxoTTPgljCFMv+8ohaiRs5n3O6o2bxhSvNe2Yoy0x2cXn5HP+hGgHUXifmpV4 oINQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=ui-outboundreport:content-transfer-encoding:in-reply-to:references :cc:to:from:content-language:subject:user-agent:mime-version :list-unsubscribe:list-subscribe:list-id:precedence:date:message-id :dkim-signature; bh=rwPH75KwxOZsKTn94BdN8S4VSMlQH80PgjnpnIfWu88=; fh=UF3DLcBgV+qVMrcMwsk5goUAvcQPQpxbgR5Yw1H6KJk=; b=EJv0ZpHjI033cyMlHwvBG1d+MBSuATdziCK1vy2t5Vv9aqn83vULjAfyyrkr3yTlq4 0+MgwoiUcIsjN0dHE7noqadsD+gdeGeziW3eUilcvvOdNBpPLDO6iCSMcTdepl2qy3B7 GM22UwUnH9yskqyLh/wiqjL2pe6HpZn13CfCvgOoyHWxZZ9rcllbuGdjzVjKLugcLXKe bTsop0NYq/b0o+hg1/Hq4ybcOoMP5yl6gO91QGrQNQV9/pwBmyvgiIm3cONIZIjsd2Py tBXar410YYQ7IKpJRYo1xN6yQS1s4hzGcK+GhB+2qzMD5wgH64lOpIlvNwm/GL/MKdPb Tb2A== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@web.de header.s=s29768273 header.b=Vi3h62jX; spf=pass (google.com: domain of linux-kernel+bounces-12812-ouuuleilei=gmail.com@vger.kernel.org designates 2604:1380:45e3:2400::1 as permitted sender) smtp.mailfrom="linux-kernel+bounces-12812-ouuuleilei=gmail.com@vger.kernel.org"; dmarc=pass (p=QUARANTINE sp=QUARANTINE dis=NONE) header.from=web.de Received: from sv.mirrors.kernel.org (sv.mirrors.kernel.org. [2604:1380:45e3:2400::1]) by mx.google.com with ESMTPS id l195-20020a633ecc000000b005ce018f428bsi9708100pga.681.2023.12.28.09.26.58 for (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 28 Dec 2023 09:26:58 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel+bounces-12812-ouuuleilei=gmail.com@vger.kernel.org designates 2604:1380:45e3:2400::1 as permitted sender) client-ip=2604:1380:45e3:2400::1; Authentication-Results: mx.google.com; dkim=pass header.i=@web.de header.s=s29768273 header.b=Vi3h62jX; spf=pass (google.com: domain of linux-kernel+bounces-12812-ouuuleilei=gmail.com@vger.kernel.org designates 2604:1380:45e3:2400::1 as permitted sender) smtp.mailfrom="linux-kernel+bounces-12812-ouuuleilei=gmail.com@vger.kernel.org"; dmarc=pass (p=QUARANTINE sp=QUARANTINE dis=NONE) header.from=web.de Received: from smtp.subspace.kernel.org (wormhole.subspace.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by sv.mirrors.kernel.org (Postfix) with ESMTPS id 6E5D728481F for ; Thu, 28 Dec 2023 17:26:58 +0000 (UTC) Received: from localhost.localdomain (localhost.localdomain [127.0.0.1]) by smtp.subspace.kernel.org (Postfix) with ESMTP id 0E341101E3; Thu, 28 Dec 2023 17:26:37 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=web.de header.i=markus.elfring@web.de header.b="Vi3h62jX" X-Original-To: linux-kernel@vger.kernel.org Received: from mout.web.de (mout.web.de [212.227.17.12]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 53BC3F51F; Thu, 28 Dec 2023 17:26:30 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=web.de Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=web.de DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=web.de; s=s29768273; t=1703784367; x=1704389167; i=markus.elfring@web.de; bh=nENeSFE2OfbA9iTcUIGri8Kr4fNiIopTSZGwGt/7Yhg=; h=X-UI-Sender-Class:Date:Subject:From:To:Cc:References: In-Reply-To; b=Vi3h62jXDDODEK0LqbLIQ49NCjBR5R3NC2GdjFiEliruaVRytDLr6CSWT2Bz+haB U0zyGF5Im2U1ccAlr/GoNxyTlx3iMGp04pGErrlF936iMDvUwmtLH09PYqKkx6+T4 Q0KwfaoKQ5KDif+2L3zi00iRTquJT7QKGP+m84Aoe8PS7gFOFHQg18N/4RXep5DpI 5xDmWdkYKt8phqwRO6F0u+WH5KAKOA+mGih2m9iB82NFUlFM3cIYzrA40u6PlAnJX vJ9F5UD0ggqXDP0DNhNcHwWTEYiSxcALCV5v7FQY2qSzBPRBVt4J/d40ull90jQ5Y MbXYUxaU8oRVqlPr+A== X-UI-Sender-Class: 814a7b36-bfc1-4dae-8640-3722d8ec6cd6 Received: from [192.168.178.21] ([94.31.85.95]) by smtp.web.de (mrweb106 [213.165.67.124]) with ESMTPSA (Nemesis) id 1MP3CA-1rbg152Vzg-00Pkmx; Thu, 28 Dec 2023 18:26:07 +0100 Message-ID: <8007263d-fe17-454e-92f3-cde218b7ed4f@web.de> Date: Thu, 28 Dec 2023 18:26:06 +0100 Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: [PATCH 3/4] scsi: lpfc: Delete an unnecessary return statement in lpfc_sli_read_link_ste() Content-Language: en-GB From: Markus Elfring To: linux-scsi@vger.kernel.org, kernel-janitors@vger.kernel.org, Dick Kennedy , James Smart , "James E. J. Bottomley" , "Martin K. Petersen" Cc: LKML References: <48a127e8-ca4f-4fc4-81b7-226080fba720@web.de> In-Reply-To: <48a127e8-ca4f-4fc4-81b7-226080fba720@web.de> X-Provags-ID: V03:K1:LDGrC+vqHWIocEGPv0lD4hzzP1mq5REoUWT0LydrdmXSQuolF8h cqwB1fOHHm19gObqS/eG0H5PdmvJm+/zyZTzrg+haOR9R39wBU+0qY1a5byXl3nHxUsjXB+ /lC+QedabMdUv0S/EaZ2PjeFSXGJDBSOyfqg/fTE2CHprAl3xzSClwnoJtnhTaKr2eq57XN eHhB9L/t6Es1aqCwtLDkg== X-Spam-Flag: NO UI-OutboundReport: notjunk:1;M01:P0:slsbpT8E4KY=;5AKxhEkof4r2xFb7LF3LQMHrzyO 5kjWrd6r+vEvsq3MZV/P14SyUprcw5EX5jvnil9JKPd0c5LD+YWT7kwd/Q/CqgQtUZsjAzYOD i1WEfRR7gsdeOvcvrDKJ81tVeEV//lDHKB2n8G+/mGcsssl/Yp+Myj/68GQQT1WgBvMWWmkeL Ntp01nArKiD19atcLdHh5zjwVi8BIatGafsI5Rj8Hax9Ht9BaLW5FNJCC52AXsodMQZqhW8FA oN1OHl93PlYJRLLyYkaF5kaXJaPSYeu8NcNU8ZSDGAKmwl8U+K1gYxBgKztK6Z6FBzuk8rUQp OC7OoCaLnmIcAme5JzU6gvBJl1voAKztWmJX/AKp4VAEeNdlLiLWLvOyVwBLWPFt6t6ABtLxi AKegdAsw0r3x/0kQEd1WkdwyDYnt0znmbHR5w2gK/uVwuwK7zFB9yETVhEfDsT2UyrQ8sc5xV 1E/2wrJejoil5omf3+pEVdW7kAKM4uXE3kC7bj2GNS8md1sCEOhTY4GqRL4U4yH57ZwtzUFec J3SWxj600iQi76+uqN/fOfr+PchTOGtPkR9wa6ex3EI+PshnGuwDITBZlS8NlE0faeRn6ABZk scmoD41IA8+LyCzkYFGq48ey4xXBWdPeTUrRdd5DvnPTnBP9ZdiR/77U8Idk3LT3DNJJGlgvJ Ifx5zuskHKK6QEINPnGuVUIOQLfpbuN+7YjmPBzFhbumy+7kL/k9oi468ZP3GZ6Ji3Qr0h28J NTCcI26W51ExMbaJOPYB3TNAVstGIpeFGpO4yldWooqRaRkxIfQYFX8dc3IJyB5GCstk0WsIi BtPIMfNNMHz6aLFtr5VC33oSMah+nG90VXWstMsqr6uvTjKWD4hVZtWWE3VrhZXAry3q96Uxz TpPfv64RFPLgec6JXuknFOf/ALzCkpwy3xyIqKUDtpAnYmOEDqNA0St06ER1ATwbgRSW1g7rD TUPLDg== X-getmail-retrieved-from-mailbox: INBOX X-GMAIL-THRID: 1786547450495550573 X-GMAIL-MSGID: 1786547450495550573 From: Markus Elfring Date: Thu, 28 Dec 2023 17:45:29 +0100 The script “checkpatch.pl” pointed information out like the following. WARNING: void function return statements are not generally useful Thus remove such a statement in the affected function. Signed-off-by: Markus Elfring --- drivers/scsi/lpfc/lpfc_sli.c | 1 - 1 file changed, 1 deletion(-) -- 2.43.0 diff --git a/drivers/scsi/lpfc/lpfc_sli.c b/drivers/scsi/lpfc/lpfc_sli.c index 93339425ce3c..8de598c9e03b 100644 --- a/drivers/scsi/lpfc/lpfc_sli.c +++ b/drivers/scsi/lpfc/lpfc_sli.c @@ -20778,7 +20778,6 @@ lpfc_sli_read_link_ste(struct lpfc_hba *phba) out: kfree(rgn23_data); - return; } /**