net/bnxt: improve error recovery information messages
[dpdk.git] / drivers / net / bnxt / bnxt_cpr.c
index 40e5350..63ff02a 100644 (file)
@@ -1,5 +1,5 @@
 /* SPDX-License-Identifier: BSD-3-Clause
- * Copyright(c) 2014-2018 Broadcom
+ * Copyright(c) 2014-2021 Broadcom
  * All rights reserved.
  */
 
@@ -46,6 +46,67 @@ void bnxt_wait_for_device_shutdown(struct bnxt *bp)
        } while (timeout);
 }
 
+static void
+bnxt_process_default_vnic_change(struct bnxt *bp,
+                                struct hwrm_async_event_cmpl *async_cmp)
+{
+       uint16_t vnic_state, vf_fid, vf_id;
+       struct bnxt_representor *vf_rep_bp;
+       struct rte_eth_dev *eth_dev;
+       bool vfr_found = false;
+       uint32_t event_data;
+
+       if (!BNXT_TRUFLOW_EN(bp))
+               return;
+
+       PMD_DRV_LOG(INFO, "Default vnic change async event received\n");
+       event_data = rte_le_to_cpu_32(async_cmp->event_data1);
+
+       vnic_state = (event_data & BNXT_DEFAULT_VNIC_STATE_MASK) >>
+                       BNXT_DEFAULT_VNIC_STATE_SFT;
+       if (vnic_state != BNXT_DEFAULT_VNIC_ALLOC)
+               return;
+
+       if (!bp->rep_info)
+               return;
+
+       vf_fid = (event_data & BNXT_DEFAULT_VNIC_CHANGE_VF_ID_MASK) >>
+                       BNXT_DEFAULT_VNIC_CHANGE_VF_ID_SFT;
+       PMD_DRV_LOG(INFO, "async event received vf_id 0x%x\n", vf_fid);
+
+       for (vf_id = 0; vf_id < BNXT_MAX_VF_REPS; vf_id++) {
+               eth_dev = bp->rep_info[vf_id].vfr_eth_dev;
+               if (!eth_dev)
+                       continue;
+               vf_rep_bp = eth_dev->data->dev_private;
+               if (vf_rep_bp &&
+                   vf_rep_bp->fw_fid == vf_fid) {
+                       vfr_found = true;
+                       break;
+               }
+       }
+       if (!vfr_found)
+               return;
+
+       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
  */
@@ -55,15 +116,23 @@ void bnxt_handle_async_event(struct bnxt *bp,
        struct hwrm_async_event_cmpl *async_cmp =
                                (struct hwrm_async_event_cmpl *)cmp;
        uint16_t event_id = rte_le_to_cpu_16(async_cmp->event_id);
+       uint16_t port_id = bp->eth_dev->data->port_id;
        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);
 
        switch (event_id) {
        case HWRM_ASYNC_EVENT_CMPL_EVENT_ID_LINK_STATUS_CHANGE:
        case HWRM_ASYNC_EVENT_CMPL_EVENT_ID_LINK_SPEED_CHANGE:
        case HWRM_ASYNC_EVENT_CMPL_EVENT_ID_LINK_SPEED_CFG_CHANGE:
                /* FALLTHROUGH */
-               bnxt_link_update(bp->eth_dev, 0, ETH_LINK_UP);
+               bnxt_link_update_op(bp->eth_dev, 0);
+               rte_eth_dev_callback_process(bp->eth_dev,
+                       RTE_ETH_EVENT_INTR_LSC, NULL);
                break;
        case HWRM_ASYNC_EVENT_CMPL_EVENT_ID_PF_DRVR_UNLOAD:
                PMD_DRV_LOG(INFO, "Async event: PF driver unloaded\n");
@@ -76,13 +145,20 @@ void bnxt_handle_async_event(struct bnxt *bp,
                PMD_DRV_LOG(INFO, "Port conn async event\n");
                break;
        case HWRM_ASYNC_EVENT_CMPL_EVENT_ID_RESET_NOTIFY:
+               /*
+                * Avoid any rx/tx packet processing during firmware reset
+                * operation.
+                */
+               bnxt_stop_rxtx(bp);
+
                /* Ignore reset notify async events when stopping the port */
                if (!bp->eth_dev->data->dev_started) {
                        bp->flags |= BNXT_FLAG_FATAL_ERROR;
                        return;
                }
 
-               event_data = rte_le_to_cpu_32(async_cmp->event_data1);
+               pthread_mutex_lock(&bp->err_recovery_lock);
+               event_data = data1;
                /* timestamp_lo/hi values are in units of 100ms */
                bp->fw_reset_max_msecs = async_cmp->timestamp_hi ?
                        rte_le_to_cpu_16(async_cmp->timestamp_hi) * 100 :
@@ -93,14 +169,17 @@ void bnxt_handle_async_event(struct bnxt *bp,
                if ((event_data & EVENT_DATA1_REASON_CODE_MASK) ==
                    EVENT_DATA1_REASON_CODE_FW_EXCEPTION_FATAL) {
                        PMD_DRV_LOG(INFO,
-                                   "Firmware fatal reset event received\n");
+                                   "Port %u: Firmware fatal reset event received\n",
+                                   port_id);
                        bp->flags |= BNXT_FLAG_FATAL_ERROR;
                } else {
                        PMD_DRV_LOG(INFO,
-                                   "Firmware non-fatal reset event received\n");
+                                   "Port %u: Firmware non-fatal reset event received\n",
+                                   port_id);
                }
 
                bp->flags |= BNXT_FLAG_FW_RESET;
+               pthread_mutex_unlock(&bp->err_recovery_lock);
                rte_eal_alarm_set(US_PER_MS, bnxt_dev_reset_and_resume,
                                  (void *)bp);
                break;
@@ -110,24 +189,26 @@ void bnxt_handle_async_event(struct bnxt *bp,
                if (!info)
                        return;
 
-               PMD_DRV_LOG(INFO, "Error recovery async event received\n");
+               event_data = data1 & EVENT_DATA1_FLAGS_MASK;
 
-               event_data = rte_le_to_cpu_32(async_cmp->event_data1) &
-                               EVENT_DATA1_FLAGS_MASK;
+               if (event_data & EVENT_DATA1_FLAGS_RECOVERY_ENABLED) {
+                       info->flags |= BNXT_FLAG_RECOVERY_ENABLED;
+               } else {
+                       info->flags &= ~BNXT_FLAG_RECOVERY_ENABLED;
+                       PMD_DRV_LOG(INFO, "Driver recovery watchdog is disabled\n");
+                       return;
+               }
 
                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)
-                       info->flags |= BNXT_FLAG_RECOVERY_ENABLED;
+                       info->flags |= BNXT_FLAG_PRIMARY_FUNC;
                else
-                       info->flags &= ~BNXT_FLAG_RECOVERY_ENABLED;
+                       info->flags &= ~BNXT_FLAG_PRIMARY_FUNC;
 
-               PMD_DRV_LOG(INFO, "recovery enabled(%d), master function(%d)\n",
-                           bnxt_is_recovery_enabled(bp),
-                           bnxt_is_master_func(bp));
+               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;
@@ -140,9 +221,21 @@ void bnxt_handle_async_event(struct bnxt *bp,
                bnxt_schedule_fw_health_check(bp);
                break;
        case HWRM_ASYNC_EVENT_CMPL_EVENT_ID_DEBUG_NOTIFICATION:
-               PMD_DRV_LOG(INFO, "DNC event: evt_data1 %#x evt_data2 %#x\n",
-                           rte_le_to_cpu_32(async_cmp->event_data1),
-                           rte_le_to_cpu_32(async_cmp->event_data2));
+               PMD_DRV_LOG(INFO, "Port: %u DNC event: data1 %#x data2 %#x\n",
+                           port_id, data1, data2);
+               break;
+       case HWRM_ASYNC_EVENT_CMPL_EVENT_ID_DEFAULT_VNIC_CHANGE:
+               bnxt_process_default_vnic_change(bp, async_cmp);
+               break;
+       case HWRM_ASYNC_EVENT_CMPL_EVENT_ID_ECHO_REQUEST:
+               PMD_DRV_LOG(INFO,
+                           "Port %u: Received fw echo request: data1 %#x data2 %#x\n",
+                           port_id, data1, data2);
+               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);
@@ -188,7 +281,7 @@ void bnxt_handle_fwd_req(struct bnxt *bp, struct cmpl_base *cmpl)
                goto reject;
        }
 
-       if (bnxt_rcv_msg_from_vf(bp, vf_id, fwd_cmd) == true) {
+       if (bnxt_rcv_msg_from_vf(bp, vf_id, fwd_cmd)) {
                /*
                 * In older firmware versions, the MAC had to be all zeros for
                 * the VF to set it's MAC via hwrm_func_vf_cfg. Set to all
@@ -203,6 +296,7 @@ void bnxt_handle_fwd_req(struct bnxt *bp, struct cmpl_base *cmpl)
                                (const uint8_t *)"\x00\x00\x00\x00\x00");
                        }
                }
+
                if (fwd_cmd->req_type == HWRM_CFA_L2_SET_RX_MASK) {
                        struct hwrm_cfa_l2_set_rx_mask_input *srm =
                                                        (void *)fwd_cmd;
@@ -214,6 +308,7 @@ void bnxt_handle_fwd_req(struct bnxt *bp, struct cmpl_base *cmpl)
                            HWRM_CFA_L2_SET_RX_MASK_INPUT_MASK_VLAN_NONVLAN |
                            HWRM_CFA_L2_SET_RX_MASK_INPUT_MASK_ANYVLAN_NONVLAN);
                }
+
                /* Forward */
                rc = bnxt_hwrm_exec_fwd_resp(bp, fw_vf_id, fwd_cmd, req_len);
                if (rc) {
@@ -255,7 +350,7 @@ int bnxt_event_hwrm_resp_handler(struct bnxt *bp, struct cmpl_base *cmp)
                bnxt_handle_async_event(bp, cmp);
                evt = 1;
                break;
-       case CMPL_BASE_TYPE_HWRM_FWD_RESP:
+       case CMPL_BASE_TYPE_HWRM_FWD_REQ:
                /* Handle HWRM forwarded responses */
                bnxt_handle_fwd_req(bp, cmp);
                evt = 1;
@@ -269,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;
@@ -287,3 +382,9 @@ bool bnxt_is_recovery_enabled(struct bnxt *bp)
 
        return false;
 }
+
+void bnxt_stop_rxtx(struct bnxt *bp)
+{
+       bp->eth_dev->rx_pkt_burst = &bnxt_dummy_recv_pkts;
+       bp->eth_dev->tx_pkt_burst = &bnxt_dummy_xmit_pkts;
+}