Add compatibility check between (np)network processor software and firmware. Signed-off-by: Bingbin Chen <chen.bingbin@zte.com.cn> --- drivers/net/zxdh/zxdh_np.c | 93 ++++++++++++++++++++++++++++++++++++++ drivers/net/zxdh/zxdh_np.h | 12 +++++ 2 files changed, 105 insertions(+) diff --git a/drivers/net/zxdh/zxdh_np.c b/drivers/net/zxdh/zxdh_np.c index cbab2a3aaa..538e3829aa 100644 --- a/drivers/net/zxdh/zxdh_np.c +++ b/drivers/net/zxdh/zxdh_np.c @@ -23,6 +23,9 @@ ZXDH_DTB_MGR_T *p_dpp_dtb_mgr[ZXDH_DEV_CHANNEL_MAX]; ZXDH_RISCV_DTB_MGR *p_riscv_dtb_queue_mgr[ZXDH_DEV_CHANNEL_MAX]; ZXDH_SDT_TBL_DATA_T g_sdt_info[ZXDH_DEV_CHANNEL_MAX][ZXDH_DEV_SDT_ID_MAX]; ZXDH_PPU_STAT_CFG_T g_ppu_stat_cfg; +static uint64_t g_np_fw_compat_addr[ZXDH_DEV_CHANNEL_MAX]; +static ZXDH_VERSION_COMPATIBLE_REG_T g_np_sdk_version = { + ZXDH_NPSDK_COMPAT_ITEM_ID, 1, 0, 0, 0, {0} }; ZXDH_FIELD_T g_smmu0_smmu0_cpu_ind_cmd_reg[] = { {"cpu_ind_rw", ZXDH_FIELD_FLAG_RW, 31, 1, 0x0, 0x0}, @@ -965,6 +968,91 @@ zxdh_np_addr_calc(uint64_t pcie_vir_baddr, uint32_t bar_offset) return np_addr; } +static uint64_t +zxdh_np_fw_compatible_addr_calc(uint64_t pcie_vir_baddr, uint64_t compatible_offset) +{ + return (pcie_vir_baddr + compatible_offset); +} + +static void +zxdh_np_pf_fw_compatible_addr_set(uint32_t dev_id, uint64_t pcie_vir_baddr) +{ + uint64_t compatible_offset = ZXDH_DPU_NO_DEBUG_PF_COMPAT_REG_OFFSET; + uint64_t compatible_addr = 0; + + compatible_addr = zxdh_np_fw_compatible_addr_calc(pcie_vir_baddr, compatible_offset); + + g_np_fw_compat_addr[dev_id] = compatible_addr; +} + +static void +zxdh_np_fw_compatible_addr_get(uint32_t dev_id, uint64_t *p_compatible_addr) +{ + *p_compatible_addr = g_np_fw_compat_addr[dev_id]; +} + +static void +zxdh_np_fw_version_data_read(uint64_t compatible_base_addr, + ZXDH_VERSION_COMPATIBLE_REG_T *p_fw_version_data, uint32_t module_id) +{ + void *fw_addr = NULL; + uint64_t module_compatible_addr = 0; + + module_compatible_addr = compatible_base_addr + + sizeof(ZXDH_VERSION_COMPATIBLE_REG_T) * (module_id - 1); + + fw_addr = (void *)module_compatible_addr; + + rte_memcpy(p_fw_version_data, fw_addr, sizeof(ZXDH_VERSION_COMPATIBLE_REG_T)); +} + +static void +zxdh_np_fw_version_compatible_data_get(uint32_t dev_id, + ZXDH_VERSION_COMPATIBLE_REG_T *p_version_compatible_value, + uint32_t module_id) +{ + uint64_t compatible_addr = 0; + + zxdh_np_fw_compatible_addr_get(dev_id, &compatible_addr); + + zxdh_np_fw_version_data_read(compatible_addr, p_version_compatible_value, module_id); +} + +static uint32_t +zxdh_np_np_sdk_version_compatible_check(uint32_t dev_id) +{ + ZXDH_VERSION_COMPATIBLE_REG_T fw_version = {0}; + + zxdh_np_fw_version_compatible_data_get(dev_id, &fw_version, ZXDH_NPSDK_COMPAT_ITEM_ID); + + if (fw_version.version_compatible_item != ZXDH_NPSDK_COMPAT_ITEM_ID) { + PMD_DRV_LOG(ERR, "version_compatible_item is not DH_NPSDK."); + return ZXDH_ERR; + } + + if (g_np_sdk_version.major != fw_version.major) { + PMD_DRV_LOG(ERR, "dh_npsdk major:%hhu: is not match fw:%hhu!", + g_np_sdk_version.major, fw_version.major); + return ZXDH_ERR; + } + + if (g_np_sdk_version.fw_minor > fw_version.fw_minor) { + PMD_DRV_LOG(ERR, "dh_npsdk fw_minor:%hhu is higher than fw:%hhu!", + g_np_sdk_version.fw_minor, fw_version.fw_minor); + return ZXDH_ERR; + } + + if (g_np_sdk_version.drv_minor < fw_version.drv_minor) { + PMD_DRV_LOG(ERR, "dh_npsdk drv_minor:%hhu is lower than fw:%hhu!", + g_np_sdk_version.drv_minor, fw_version.drv_minor); + return ZXDH_ERR; + } + + PMD_DRV_LOG(INFO, "dh_npsdk compatible check success!"); + + return ZXDH_OK; +} + static ZXDH_RISCV_DTB_MGR * zxdh_np_riscv_dtb_queue_mgr_get(uint32_t dev_id) { @@ -2626,5 +2714,10 @@ zxdh_np_host_init(uint32_t dev_id, agent_addr = ZXDH_PCIE_AGENT_ADDR_OFFSET + p_dev_init_ctrl->pcie_vir_addr; zxdh_np_dev_agent_addr_set(dev_id, agent_addr); + zxdh_np_pf_fw_compatible_addr_set(dev_id, p_dev_init_ctrl->pcie_vir_addr); + + rc = zxdh_np_np_sdk_version_compatible_check(dev_id); + ZXDH_COMM_CHECK_DEV_RC(dev_id, rc, "zxdh_np_np_sdk_version_compatible_check"); + return 0; } diff --git a/drivers/net/zxdh/zxdh_np.h b/drivers/net/zxdh/zxdh_np.h index 35130bdd1b..1df85bd382 100644 --- a/drivers/net/zxdh/zxdh_np.h +++ b/drivers/net/zxdh/zxdh_np.h @@ -112,6 +112,9 @@ #define ZXDH_SE_OPR_RD (1) +#define ZXDH_NPSDK_COMPAT_ITEM_ID (10) +#define ZXDH_DPU_NO_DEBUG_PF_COMPAT_REG_OFFSET (0x5400) + /**errco code */ #define ZXDH_RC_BASE (0x1000U) #define ZXDH_PARAMETER_CHK_BASE (ZXDH_RC_BASE | 0x200) @@ -628,6 +631,15 @@ typedef enum zxdh_stat_cnt_mode_e { ZXDH_STAT_MAX_MODE, } ZXDH_STAT_CNT_MODE_E; +typedef struct __rte_aligned(2) zxdh_version_compatible_reg_t { + uint8_t version_compatible_item; + uint8_t major; + uint8_t fw_minor; + uint8_t drv_minor; + uint16_t patch; + uint8_t rsv[2]; +} ZXDH_VERSION_COMPATIBLE_REG_T; + int zxdh_np_host_init(uint32_t dev_id, ZXDH_DEV_INIT_CTRL_T *p_dev_init_ctrl); int zxdh_np_online_uninit(uint32_t dev_id, char *port_name, uint32_t queue_id); int zxdh_np_dtb_table_entry_write(uint32_t dev_id, uint32_t queue_id, -- 2.27.0