From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-qv1-f46.google.com (mail-qv1-f46.google.com [209.85.219.46]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 39A4C19DF55 for ; Mon, 3 Aug 2026 01:42:35 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.219.46 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1785721358; cv=none; b=ns0cLR0BLzTBon0QpTHiAYAriex3onpSr+wNA9ohNKWeK5PGEDPl8gxnSUIKIMh/UwT6xBxLkCV8aXLlb1vCov1/rFRdrjROpemS9fmUXrrGhNse0Ixha7ChliOBkvxzWpdGaEtRSiz3UjxrYjezP3PWsrCevHd/dRdpaALYV1Q= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1785721358; c=relaxed/simple; bh=S01THjx3GU4adJn0CGfQrT4Wr8H1w2uxdmBziA3XZE4=; h=From:To:Cc:Subject:Date:Message-ID:MIME-Version; b=PeRyg2PKZZqokqAOwYZxW4i9HZKbuiqwGQ0EV7OKi755PuvaOBloRDh+CDMG6yYLGOrtwobhS1nd7Q5fStXJK5ntea5yKNOKQr5CmNud8aQANl2jB3CIuXBo+8RMk5+UZMBWci8Jz11gIPnZLJADBUgSKfgOgU9atGETUKpCyQw= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com; spf=pass smtp.mailfrom=gmail.com; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b=ZpGt1sOc; arc=none smtp.client-ip=209.85.219.46 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=gmail.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="ZpGt1sOc" Received: by mail-qv1-f46.google.com with SMTP id 6a1803df08f44-8efef6533aaso19302906d6.0 for ; Sun, 02 Aug 2026 18:42:34 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20251104; t=1785721354; x=1786326154; darn=vger.kernel.org; h=content-transfer-encoding:mime-version:message-id:date:subject:cc :to:from:from:to:cc:subject:date:message-id:reply-to:content-type; bh=DawTbNdk1LZcBXCoupSLgXbzUs4SU4vadYzb1030Krs=; b=ZpGt1sOcEloIGzua3GT9grqvpPKpdOaiI9wB6VlmiyCOIgjn04HhnM9toTVaYn6Qg/ LKqKLkhsqff2ublpF7+Ibspwky3OlQPjlrNjcDt8Pn3lW+cwRPQoTmFxD4x+HdtMJ3gg 1+BphU7jJKH5om67dvSl8ErJ58tlMulXWeHstfnDtetDKGR/lPcqdy5NYfOBPDdolUDg grBjtA6TqhGTGStUaWHBhULBFh0mhti7edWSbYzJ82dSas6masm++hsbbu6Ngc2NBYdZ eBYDHYDcxv1x5YVOcUJHqfXE/A9/YqOH6s83vVJ2A2B7euW+V1mMVD1EMRwyz4/Z6SWP wsOQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1785721354; x=1786326154; h=content-transfer-encoding:mime-version:message-id:date:subject:cc :to:from:x-gm-gg:x-gm-message-state:from:to:cc:subject:date :message-id:reply-to:content-type; bh=DawTbNdk1LZcBXCoupSLgXbzUs4SU4vadYzb1030Krs=; b=mwXK8oFxM6NhlS0kIKnVCFNUzgjj136yB6vTczlDw+e/5PWH1/SnusidNELW+HWSsG M8DCPU5oLuNaWClK6GM3VXtwCKFvBSLY2jlfPzis1UK41PdWWMdCQmxafzFG9lUw9waB L1/yBmRtKB44F8oaxgpLVV/H7bN8kUPGW9nx8r+SghQMBto1IQoHatGxolmu0Z90+kq/ MnvOKXbu0RXpWAk+wD4Pwxi7W3I2lS8Prqn+/4O3KFpxCwB6amxTyVrIHKrNEqvwvIgn Yo9ANApR+bKaZyMa8Z9CPXUWw5MNtDn0TEPO1FDU5Hyr8Ep+mvij5wNWogI8hYxWBEhH ZiaA== X-Forwarded-Encrypted: i=1; AHgh+RotHC4CmBRCZZMfmy1HzabKAeZoq0mes/sKD4bWmrkH3FdplnyHPksCLJFFU2xmh1/b9Wif3p2S3HDM8BU=@vger.kernel.org X-Gm-Message-State: AOJu0YxApZFXS97xzmCs304iP8uOA3EICNz4mQDdgyhaRh0csY/hytuH cxJENINKlH26vezU0tA1KZbIaOGjL/EMd9C/5DaeOig7tZ2XmlV3jD77 X-Gm-Gg: AR+sD13By3y45prq6EPJPfnjJl/4dBYjURWmk/NFisxZcrJvEkzBbw9OqN9M6ysawhQ 0K6WKPBt1GkuHnAq1hDJv+nBLxbh1059FwU5K5jgS0S0GBeS3LoSMKA8BUKyXjYVJ6yskzWQkg2 rCaJOCVg3qZwKtp9AlfDwB0t7PtPn4B3J+qyWBvmdfv0fmG8Ai5VEIJHcx7ZBFCtGSUjYOPAVSR zCmqH3JYrVKBmRo88nAtNr/zRPYEh7TLODu6sHOsoiz4NbEM0XKdUxcDi4YcVGnrf2hQNvw1SNt c5KPpZUGH3Biid/3LoHTXMSn++9LRx0ZGZnaaTSaTcKbQ7eacoiUwg539h+5oCq0vs50gRspdwN JcoHB8a/c5g+fF/bJTbEtDDTJObQzfYamsMDw1fTLy3R2sp9GdXaan4y0If6Tl4D168Tmo+6Ymm rnma9lqziZVHkX9Zoyr3tMqRg7RMQyO5FRZRlDYK796K25EIdPT+/XDQYajhuKu6rZ8FhZQCH7y EuSyfFbigDxQpLjNjM1JennOgqCwqn66+TfiOXrHXPZaacvI+Jpzyhx/feiG+E= X-Received: by 2002:a05:6214:449f:b0:8f3:e97:59b8 with SMTP id 6a1803df08f44-908496af620mr181485536d6.29.1785721353981; Sun, 02 Aug 2026 18:42:33 -0700 (PDT) Received: from i4-gl-tmk5904.ad.psu.edu ([130.203.156.186]) by smtp.gmail.com with ESMTPSA id 6a1803df08f44-908435eab41sm64167096d6.34.2026.08.02.18.42.31 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Sun, 02 Aug 2026 18:42:32 -0700 (PDT) From: Yuho Choi To: ketan.mukadam@broadcom.com, James.Bottomley@HansenPartnership.com, martin.petersen@oracle.com Cc: linux-scsi@vger.kernel.org, linux-kernel@vger.kernel.org, Yuho Choi Subject: [PATCH v1] scsi: be2iscsi: Fix MSI-X IRQ cleanup Date: Sun, 2 Aug 2026 21:42:24 -0400 Message-ID: <20260803014224.614347-1-dbgh9129@gmail.com> X-Mailer: git-send-email 2.43.0 Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: 8bit beiscsi_init_irqs() uses j to walk previously registered MSI-X vectors when unwinding an allocation or request_irq() failure, but passes i to pci_irq_vector(). Consequently, each free_irq() targets the failing vector instead of the vector associated with be_eq[j]. Use j to release the same vector that was registered. This ensures all successfully requested IRQs are freed before initialization failure tears down the HBA. Fixes: 831488669a33 ("scsi: be2iscsi: switch to pci_alloc_irq_vectors") Signed-off-by: Yuho Choi --- drivers/scsi/be2iscsi/be_main.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/scsi/be2iscsi/be_main.c b/drivers/scsi/be2iscsi/be_main.c index fd18d4d3d219..9833ab06b299 100644 --- a/drivers/scsi/be2iscsi/be_main.c +++ b/drivers/scsi/be2iscsi/be_main.c @@ -878,7 +878,7 @@ static int beiscsi_init_irqs(struct beiscsi_hba *phba) return 0; free_msix_irqs: for (j = i - 1; j >= 0; j--) { - free_irq(pci_irq_vector(pcidev, i), &phwi_context->be_eq[j]); + free_irq(pci_irq_vector(pcidev, j), &phwi_context->be_eq[j]); kfree(phba->msi_name[j]); } return ret; -- 2.43.0