net/sfc: free MAE lock once switch domain is assigned

[dpdk.git] / drivers / net / bnxt / bnxt_cpr.c

diff --git a/drivers/net/bnxt/bnxt_cpr.c b/drivers/net/bnxt/bnxt_cpr.c
index 02006f6..63ff02a 100644 (file)
--- a/drivers/net/bnxt/bnxt_cpr.c
+++ b/drivers/net/bnxt/bnxt_cpr.c
@@ -91,6 +91,22 @@ bnxt_process_default_vnic_change(struct bnxt *bp,
        bnxt_rep_dev_start_op(eth_dev);
 }
 
+static void bnxt_handle_event_error_report(struct bnxt *bp,
+                                          uint32_t data1,
+                                          uint32_t data2)
+{
+       switch (BNXT_EVENT_ERROR_REPORT_TYPE(data1)) {
+       case HWRM_ASYNC_EVENT_CMPL_ERROR_REPORT_BASE_EVENT_DATA1_ERROR_TYPE_PAUSE_STORM:
+               PMD_DRV_LOG(WARNING, "Port:%d Pause Storm detected!\n",
+                           bp->eth_dev->data->port_id);
+               break;
+       default:
+               PMD_DRV_LOG(INFO, "FW reported unknown error type data1 %d"
+                           " data2: %d\n", data1, data2);
+               break;
+       }
+}
+
 /*
  * Async event handling
  */
@@ -104,6 +120,7 @@ void bnxt_handle_async_event(struct bnxt *bp,
        struct bnxt_error_recovery_info *info;
        uint32_t event_data;
        uint32_t data1, data2;
+       uint32_t status;
 
        data1 = rte_le_to_cpu_32(async_cmp->event_data1);
        data2 = rte_le_to_cpu_32(async_cmp->event_data2);
@@ -172,24 +189,26 @@ void bnxt_handle_async_event(struct bnxt *bp,
                if (!info)
                        return;
 
-               PMD_DRV_LOG(INFO, "Port %u: Error recovery async event received\n",
-                           port_id);
-
                event_data = data1 & EVENT_DATA1_FLAGS_MASK;
 
-               if (event_data & EVENT_DATA1_FLAGS_MASTER_FUNC)
-                       info->flags |= BNXT_FLAG_MASTER_FUNC;
-               else
-                       info->flags &= ~BNXT_FLAG_MASTER_FUNC;
-
-               if (event_data & EVENT_DATA1_FLAGS_RECOVERY_ENABLED)
+               if (event_data & EVENT_DATA1_FLAGS_RECOVERY_ENABLED) {
                        info->flags |= BNXT_FLAG_RECOVERY_ENABLED;
-               else
+               } else {
                        info->flags &= ~BNXT_FLAG_RECOVERY_ENABLED;
+                       PMD_DRV_LOG(INFO, "Driver recovery watchdog is disabled\n");
+                       return;
+               }
 
-               PMD_DRV_LOG(INFO, "Port %u: recovery enabled(%d), master function(%d)\n",
-                           port_id, bnxt_is_recovery_enabled(bp),
-                           bnxt_is_master_func(bp));
+               if (event_data & EVENT_DATA1_FLAGS_MASTER_FUNC)
+                       info->flags |= BNXT_FLAG_PRIMARY_FUNC;
+               else
+                       info->flags &= ~BNXT_FLAG_PRIMARY_FUNC;
+
+               status = bnxt_read_fw_status_reg(bp, BNXT_FW_STATUS_REG);
+               PMD_DRV_LOG(INFO,
+                           "Port: %u Driver recovery watchdog, role: %s, FW status: 0x%x (%s)\n",
+                           port_id, bnxt_is_primary_func(bp) ? "primary" : "backup", status,
+                           (status == BNXT_FW_STATUS_HEALTHY) ? "healthy" : "unhealthy");
 
                if (bp->flags & BNXT_FLAG_FW_HEALTH_CHECK_SCHEDULED)
                        return;
@@ -215,6 +234,9 @@ void bnxt_handle_async_event(struct bnxt *bp,
                if (bp->recovery_info)
                        bnxt_hwrm_fw_echo_reply(bp, data1, data2);
                break;
+       case HWRM_ASYNC_EVENT_CMPL_EVENT_ID_ERROR_REPORT:
+               bnxt_handle_event_error_report(bp, data1, data2);
+               break;
        default:
                PMD_DRV_LOG(DEBUG, "handle_async_event id = 0x%x\n", event_id);
                break;
@@ -342,9 +364,9 @@ int bnxt_event_hwrm_resp_handler(struct bnxt *bp, struct cmpl_base *cmp)
        return evt;
 }
 
-bool bnxt_is_master_func(struct bnxt *bp)
+bool bnxt_is_primary_func(struct bnxt *bp)
 {
-       if (bp->recovery_info->flags & BNXT_FLAG_MASTER_FUNC)
+       if (bp->recovery_info->flags & BNXT_FLAG_PRIMARY_FUNC)
                return true;
 
        return false;