git: 09e161a29ee5 - stable/13 - mrsas: if controller reset is in progress, refrain from firing DCMDs to firmware in shutdown

From: Warner Losh <imp_at_FreeBSD.org>
Date: Wed, 06 Apr 2022 16:27:28 UTC
The branch stable/13 has been updated by imp:

URL: https://cgit.FreeBSD.org/src/commit/?id=09e161a29ee5b0e52494c861f451de35b286e327

commit 09e161a29ee5b0e52494c861f451de35b286e327
Author:     Chandrakanth Patil <chandrakanth.patil@broadcom.com>
AuthorDate: 2022-01-21 09:24:37 +0000
Commit:     Warner Losh <imp@FreeBSD.org>
CommitDate: 2022-04-06 16:27:25 +0000

    mrsas: if controller reset is in progress, refrain from firing DCMDs to
    firmware in shutdown
    
    If controller reset is in progress, at same time if system shutdown is
    issued then corresponding shutdown function in driver will be invoked
    where driver is waiting 15 seconds to complete the controller reset.
    
    If the reset is not complteted within that time frame driver will go
    ahead and fire cache flush and shutdown DCMDs which will end up
    accessing the the queues which are not initialized due to undergoing
    reset leads to FMU error in firmware.
    
    Fix:
    In shutdown function, if controller reset is not finished within 15
    seconds than driver will return to the OS without firing any DCMDs.
    
    Reviewed by: imp
    PR: 261375
    
    (cherry picked from commit 79c4c4be9618470711480f46ef2cd3a15c00cdd2)
---
 sys/dev/mrsas/mrsas.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/sys/dev/mrsas/mrsas.c b/sys/dev/mrsas/mrsas.c
index 8814642fccd0..0055ddda8ce9 100644
--- a/sys/dev/mrsas/mrsas.c
+++ b/sys/dev/mrsas/mrsas.c
@@ -1198,6 +1198,7 @@ mrsas_shutdown(device_t dev)
 		if (sc->reset_in_progress) {
 			mrsas_dprint(sc, MRSAS_INFO,
 			    "gave up waiting for OCR to be finished\n");
+			return (0);
 		}
 	}