]> git.ipfire.org Git - thirdparty/kernel/stable.git/commitdiff
bnxt_en: Query FW parameters when the CAPS_CHANGE bit is set
authorshantiprasad shettar <shantiprasad.shettar@broadcom.com>
Mon, 10 Mar 2025 18:31:26 +0000 (11:31 -0700)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Thu, 29 May 2025 09:12:38 +0000 (11:12 +0200)
[ Upstream commit a6c81e32aeacbfd530d576fa401edd506ec966ef ]

Newer FW can set the CAPS_CHANGE flag during ifup if some capabilities
or configurations have changed.  For example, the CoS queue
configurations may have changed.  Support this new flag by treating it
almost like FW reset.  The driver will essentially rediscover all
features and capabilities, reconfigure all backing store context memory,
reset everything to default, and reserve all resources.

Reviewed-by: Somnath Kotur <somnath.kotur@broadcom.com>
Reviewed-by: Pavan Chebbi <pavan.chebbi@broadcom.com>
Signed-off-by: shantiprasad shettar <shantiprasad.shettar@broadcom.com>
Signed-off-by: Michael Chan <michael.chan@broadcom.com>
Link: https://patch.msgid.link/20250310183129.3154117-5-michael.chan@broadcom.com
Reviewed-by: Jacob Keller <jacob.e.keller@intel.com>
Signed-off-by: Paolo Abeni <pabeni@redhat.com>
Signed-off-by: Sasha Levin <sashal@kernel.org>
drivers/net/ethernet/broadcom/bnxt/bnxt.c

index bd8b9cb05ae9887e75712d5ed8760087b70c7b3b..1a19c922009d1a564599e8624a21d80f71c4ce20 100644 (file)
@@ -12104,6 +12104,7 @@ static int bnxt_hwrm_if_change(struct bnxt *bp, bool up)
        struct hwrm_func_drv_if_change_input *req;
        bool fw_reset = !bp->irq_tbl;
        bool resc_reinit = false;
+       bool caps_change = false;
        int rc, retry = 0;
        u32 flags = 0;
 
@@ -12159,8 +12160,11 @@ static int bnxt_hwrm_if_change(struct bnxt *bp, bool up)
                set_bit(BNXT_STATE_ABORT_ERR, &bp->state);
                return -ENODEV;
        }
-       if (resc_reinit || fw_reset) {
-               if (fw_reset) {
+       if (flags & FUNC_DRV_IF_CHANGE_RESP_FLAGS_CAPS_CHANGE)
+               caps_change = true;
+
+       if (resc_reinit || fw_reset || caps_change) {
+               if (fw_reset || caps_change) {
                        set_bit(BNXT_STATE_FW_RESET_DET, &bp->state);
                        if (!test_bit(BNXT_STATE_IN_FW_RESET, &bp->state))
                                bnxt_ulp_irq_stop(bp);