Skip site navigation (1)Skip section navigation (2)
Date:      Thu, 20 Oct 2022 15:02:37 GMT
From:      "Kenneth D. Merry" <ken@FreeBSD.org>
To:        src-committers@FreeBSD.org, dev-commits-src-all@FreeBSD.org, dev-commits-src-branches@FreeBSD.org
Subject:   git: 2edc70060d1e - stable/13 - Fix mpr(4) panic during a firmware update.
Message-ID:  <202210201502.29KF2bN3090315@gitrepo.freebsd.org>

next in thread | raw e-mail | index | archive | help
The branch stable/13 has been updated by ken:

URL: https://cgit.FreeBSD.org/src/commit/?id=2edc70060d1e0f75eb975b4318345343600852d8

commit 2edc70060d1e0f75eb975b4318345343600852d8
Author:     Kenneth D. Merry <ken@FreeBSD.org>
AuthorDate: 2022-10-17 16:48:34 +0000
Commit:     Kenneth D. Merry <ken@FreeBSD.org>
CommitDate: 2022-10-20 15:01:03 +0000

    Fix mpr(4) panic during a firmware update.
    
    Issue Description:
    The RequestCredits field of IOCFacts got changed between the Phase23
    firmware to Phase24 firmware. So as part of firmware update operation,
    driver has to free the resources & pools which are created with the Phase23
    Firmware's IOCFacts data (i.e. during driver load time) and has to
    reallocate the resources and pools using Phase24's IOCFacts data. Here
    driver has freed the interrupts but missed to reallocate the interrupts and
    hence config page read operation is getting timed out and controller is
    going for recursive reinit (controller reset) operations and leading to
    kernel panic.
    
    Fix:
    Reallocate the interrupts if the interrupts are disabled as part of
    firmware update/downgrade operation.
    
    Submitted by:   Sreekanth Ready <sreekanth.reddy@broadcom.com>
    Tested by:      ken
    MFC after:      3 days
    
    (cherry picked from commit 11778fca4a83f5e3b597c75785aa5c0ee0dc518e)
---
 sys/dev/mpr/mpr.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/sys/dev/mpr/mpr.c b/sys/dev/mpr/mpr.c
index 0f4c5c906216..6f54b1f57950 100644
--- a/sys/dev/mpr/mpr.c
+++ b/sys/dev/mpr/mpr.c
@@ -740,7 +740,7 @@ mpr_iocfacts_allocate(struct mpr_softc *sc, uint8_t attaching)
 	 * XXX If the number of MSI-X vectors changes during re-init, this
 	 * won't see it and adjust.
 	 */
-	if (attaching && (error = mpr_pci_setup_interrupts(sc)) != 0) {
+	if ((attaching || reallocating) && (error = mpr_pci_setup_interrupts(sc)) != 0) {
 		mpr_dprint(sc, MPR_INIT|MPR_ERROR,
 		    "Failed to setup interrupts\n");
 		mpr_free(sc);



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?202210201502.29KF2bN3090315>