Fix zxdh driver packet sending and receiving errors.

Signed-off-by: Bingbin Chen <chen.bingbin@zte.com.cn>
---
 drivers/net/zxdh/zxdh_common.h     |  99 ++++++++
 drivers/net/zxdh/zxdh_ethdev.c     |  19 +-
 drivers/net/zxdh/zxdh_ethdev_ops.c | 109 +++++----
 drivers/net/zxdh/zxdh_ethdev_ops.h |  35 +++
 drivers/net/zxdh/zxdh_msg.c        | 381 +++++++++++++++++------------
 drivers/net/zxdh/zxdh_msg.h        |  99 +++++---
 drivers/net/zxdh/zxdh_mtr.c        |  50 ++--
 drivers/net/zxdh/zxdh_mtr.h        |   5 +
 drivers/net/zxdh/zxdh_np.c         |  70 +++++-
 drivers/net/zxdh/zxdh_np.h         |   8 +-
 drivers/net/zxdh/zxdh_pci.c        |  24 +-
 drivers/net/zxdh/zxdh_pci.h        |   2 +-
 drivers/net/zxdh/zxdh_rxtx.c       |  12 +-
 drivers/net/zxdh/zxdh_tables.c     |  42 ++--
 drivers/net/zxdh/zxdh_tables.h     |  68 +++--
 15 files changed, 708 insertions(+), 315 deletions(-)

diff --git a/drivers/net/zxdh/zxdh_common.h b/drivers/net/zxdh/zxdh_common.h
index d78a822ebf..b34dcddd4d 100644
--- a/drivers/net/zxdh/zxdh_common.h
+++ b/drivers/net/zxdh/zxdh_common.h
@@ -20,6 +20,105 @@ struct zxdh_res_para {
     uint16_t src_type; /* refer to BAR_DRIVER_TYPE */
 };
 
+static inline size_t
+zxdh_get_value(uint32_t fld_sz, uint8_t *addr) {
+    size_t result = 0;
+    switch (fld_sz) {
+    case 1:
+        result = *((uint8_t *)addr);
+        break;
+    case 2:
+        result = *((uint16_t *)addr);
+        break;
+    case 4:
+        result = *((uint32_t *)addr);
+        break;
+    case 8:
+        result = *((uint64_t *)addr);
+        break;
+    default:
+        printf("Error: unreachable field size %u\n", fld_sz);
+        break;
+    }
+    return result;
+}
+
+static inline void
+zxdh_set_value(uint32_t fld_sz, uint8_t *addr, size_t value) {
+    switch (fld_sz) {
+    case 1:
+        *(uint8_t *)addr = (uint8_t)value;
+        break;
+    case 2:
+        *(uint16_t *)addr = (uint16_t)value;
+        break;
+    case 4:
+        *(uint32_t *)addr = (uint32_t)value;
+        break;
+    case 8:
+        *(uint64_t *)addr = (uint64_t)value;
+        break;
+    default:
+        printf("Error: unreachable field size %u\n", fld_sz);
+        break;
+    }
+}
+
+#define __zxdh_nullp(typ) ((struct zxdh_ifc_##typ##_bits *)0)
+#define __zxdh_bit_sz(typ, fld) sizeof(__zxdh_nullp(typ)->fld)
+#define __zxdh_bit_off(typ, fld) ((unsigned int)(uintptr_t) \
+                  (&(__zxdh_nullp(typ)->fld)))
+#define __zxdh_dw_bit_off(typ, fld) (32 - __zxdh_bit_sz(typ, fld) - \
+                    (__zxdh_bit_off(typ, fld) & 0x1f))
+#define __zxdh_dw_off(typ, fld) (__zxdh_bit_off(typ, fld) / 32)
+#define __zxdh_64_off(typ, fld) (__zxdh_bit_off(typ, fld) / 64)
+#define __zxdh_dw_mask(typ, fld) (__zxdh_mask(typ, fld) << \
+                  __zxdh_dw_bit_off(typ, fld))
+#define __zxdh_mask(typ, fld) ((uint32_t)((1ull << __zxdh_bit_sz(typ, fld)) - 1))
+#define __zxdh_16_off(typ, fld) (__zxdh_bit_off(typ, fld) / 16)
+#define __zxdh_16_bit_off(typ, fld) (16 - __zxdh_bit_sz(typ, fld) - \
+                    (__zxdh_bit_off(typ, fld) & 0xf))
+#define __zxdh_mask16(typ, fld) ((uint16_t)((1ull << __zxdh_bit_sz(typ, fld)) - 1))
+#define __zxdh_16_mask(typ, fld) (__zxdh_mask16(typ, fld) << \
+                  __zxdh_16_bit_off(typ, fld))
+#define ZXDH_ST_SZ_BYTES(typ) (sizeof(struct zxdh_ifc_##typ##_bits) / 8)
+#define ZXDH_ST_SZ_DW(typ) (sizeof(struct zxdh_ifc_##typ##_bits) / 32)
+#define ZXDH_BYTE_OFF(typ, fld) (__zxdh_bit_off(typ, fld) / 8)
+#define ZXDH_ADDR_OF(typ, p, fld) ((uint8_t *)(p) + ZXDH_BYTE_OFF(typ, fld))
+
+#define BUILD_BUG_ON(condition) do { \
+    if (condition) \
+        __builtin_unreachable(); \
+    } while (0)
+
+#define ZXDH_SET(typ, p, fld, v) do { \
+    BUILD_BUG_ON(__zxdh_bit_sz(typ, fld) % 8); \
+    uint32_t fld_sz = __zxdh_bit_sz(typ, fld) / 8; \
+    uint8_t *addr = ZXDH_ADDR_OF(typ, p, fld); \
+    zxdh_set_value(fld_sz, addr, v); \
+} while (0)
+
+#define ZXDH_GET(typ, p, fld) ({ \
+    BUILD_BUG_ON(__zxdh_bit_sz(typ, fld) % 8); \
+    uint32_t fld_sz = __zxdh_bit_sz(typ, fld) / 8; \
+    uint8_t *addr = ZXDH_ADDR_OF(typ, p, fld); \
+    zxdh_get_value(fld_sz, addr); \
+})
+
+#define ZXDH_SET_ARRAY(typ, p, fld, index, value, type) \
+    do { \
+        type *addr = (type *)((uint8_t *)ZXDH_ADDR_OF(typ, p, fld) + \
+        (index) * sizeof(type)); \
+        *addr = (type)(value); \
+    } while (0)
+
+#define ZXDH_GET_ARRAY(typ, p, fld, index, type) \
+    ({ \
+        type *addr = (type *)((uint8_t *)ZXDH_ADDR_OF(typ, p, fld) + \
+        (index) * sizeof(type)); \
+        *addr; \
+    })
+
 int32_t zxdh_phyport_get(struct rte_eth_dev *dev, uint8_t *phyport);
 int32_t zxdh_panelid_get(struct rte_eth_dev *dev, uint8_t *pannelid);
 int32_t zxdh_hashidx_get(struct rte_eth_dev *dev, uint8_t *hash_idx);
diff --git a/drivers/net/zxdh/zxdh_ethdev.c b/drivers/net/zxdh/zxdh_ethdev.c
index ea8b18e5e1..ba7ea52d20 100644
--- a/drivers/net/zxdh/zxdh_ethdev.c
+++ b/drivers/net/zxdh/zxdh_ethdev.c
@@ -1250,10 +1250,6 @@ zxdh_set_rxtx_funcs(struct rte_eth_dev *eth_dev)
 {
     struct zxdh_hw *hw = eth_dev->data->dev_private;
 
-    if (!zxdh_pci_packed_queue(hw)) {
-        PMD_DRV_LOG(ERR, "port %u not support packed queue", eth_dev->data->port_id);
-        return -1;
-    }
     if (!zxdh_pci_with_feature(hw, ZXDH_NET_F_MRG_RXBUF)) {
         PMD_DRV_LOG(ERR, "port %u not support rx mergeable", eth_dev->data->port_id);
         return -1;
@@ -1498,6 +1494,8 @@ zxdh_dtb_dump_res_init(struct zxdh_hw *hw, ZXDH_DEV_INIT_CTRL_T *dpp_ctrl)
 
     struct zxdh_dtb_bulk_dump_info dtb_dump_baseres[] = {
         {"sdt_vport_att_table", 4 * 1024 * 1024, ZXDH_SDT_VPORT_ATT_TABLE, NULL},
+        {"sdt_vlan_att_table", 4 * 1024 * 1024, ZXDH_SDT_VLAN_ATT_TABLE, NULL},
+        {"sdt_rss_table", 4 * 1024 * 1024, ZXDH_SDT_RSS_ATT_TABLE, NULL},
         {"sdt_l2_entry_table0", 5 * 1024 * 1024, ZXDH_SDT_L2_ENTRY_TABLE0, NULL},
         {"sdt_l2_entry_table1", 5 * 1024 * 1024, ZXDH_SDT_L2_ENTRY_TABLE1, NULL},
         {"sdt_l2_entry_table2", 5 * 1024 * 1024, ZXDH_SDT_L2_ENTRY_TABLE2, NULL},
@@ -1514,7 +1512,8 @@ zxdh_dtb_dump_res_init(struct zxdh_hw *hw, ZXDH_DEV_INIT_CTRL_T *dpp_ctrl)
     for (i = 0; i < (int)RTE_DIM(dtb_dump_baseres); i++) {
         struct zxdh_dtb_bulk_dump_info *p = dtb_dump_baseres + i;
         char buf[ZXDH_MAX_NAME_LEN] = {0};
-
+        memset(buf, '\0', sizeof(buf));
+        sprintf(buf, "%s_%x", p->mz_name, hw->dev_id);
         p->mz_name = buf;
 
         const struct rte_memzone *generic_dump_mz =
@@ -1544,6 +1543,7 @@ zxdh_np_dtb_res_init(struct rte_eth_dev *dev)
     struct zxdh_hw *hw = dev->data->dev_private;
     struct zxdh_bar_offset_params param = {0};
     struct zxdh_bar_offset_res res = {0};
+    char buf[ZXDH_MAX_NAME_LEN] = {0};
     struct zxdh_dtb_shared_data *dtb_data = &hw->dev_sd->dtb_sd;
     int ret = 0;
 
@@ -1569,7 +1569,7 @@ zxdh_np_dtb_res_init(struct rte_eth_dev *dev)
     dpp_ctrl->vport = hw->vport.vport;
     dpp_ctrl->vector = ZXDH_MSIX_INTR_DTB_VEC;
     strlcpy(dpp_ctrl->port_name, dev->device->name, sizeof(dpp_ctrl->port_name));
-    dpp_ctrl->pcie_vir_addr = (uint32_t)hw->bar_addr[0];
+    dpp_ctrl->pcie_vir_addr = (uint64_t)hw->bar_addr[0];
 
     param.pcie_id = hw->pcie_id;
     param.virt_addr = hw->bar_addr[0] + ZXDH_CTRLCH_OFFSET;
@@ -1584,7 +1584,8 @@ zxdh_np_dtb_res_init(struct rte_eth_dev *dev)
     dpp_ctrl->np_bar_offset = res.bar_offset;
 
     if (!dtb_data->dtb_table_conf_mz) {
-        const struct rte_memzone *conf_mz = rte_memzone_reserve_aligned("zxdh_dtb_table_conf_mz",
+        sprintf(buf, "%s_%x", "zxdh_dtb_table_conf_mz", hw->dev_id);
+        const struct rte_memzone *conf_mz = rte_memzone_reserve_aligned(buf,
                 ZXDH_DTB_TABLE_CONF_SIZE, SOCKET_ID_ANY, 0, RTE_CACHE_LINE_SIZE);
 
         if (conf_mz == NULL) {
@@ -1600,7 +1601,9 @@ zxdh_np_dtb_res_init(struct rte_eth_dev *dev)
     }
 
     if (!dtb_data->dtb_table_dump_mz) {
-        const struct rte_memzone *dump_mz = rte_memzone_reserve_aligned("zxdh_dtb_table_dump_mz",
+        memset(buf, '\0', sizeof(buf));
+        sprintf(buf, "%s_%x", "zxdh_dtb_table_dump_mz", hw->dev_id);
+        const struct rte_memzone *dump_mz = rte_memzone_reserve_aligned(buf,
                 ZXDH_DTB_TABLE_DUMP_SIZE, SOCKET_ID_ANY, 0, RTE_CACHE_LINE_SIZE);
 
         if (dump_mz == NULL) {
diff --git a/drivers/net/zxdh/zxdh_ethdev_ops.c b/drivers/net/zxdh/zxdh_ethdev_ops.c
index 2b02734c62..f8e8d26c50 100644
--- a/drivers/net/zxdh/zxdh_ethdev_ops.c
+++ b/drivers/net/zxdh/zxdh_ethdev_ops.c
@@ -15,6 +15,7 @@
 #include "zxdh_np.h"
 #include "zxdh_queue.h"
 #include "zxdh_mtr.h"
+#include "zxdh_common.h"
 
 #define ZXDH_VLAN_FILTER_GROUPS       64
 #define ZXDH_INVALID_LOGIC_QID        0xFFFFU
@@ -278,9 +279,11 @@ zxdh_link_info_get(struct rte_eth_dev *dev, struct rte_eth_link *link)
 {
     struct zxdh_hw *hw = dev->data->dev_private;
     struct zxdh_msg_info msg_info = {0};
-    struct zxdh_msg_reply_info reply_info = {0};
+    uint8_t zxdh_msg_reply_info[ZXDH_ST_SZ_BYTES(msg_reply_info)] = {0};
     uint16_t status = 0;
     int32_t ret = 0;
+    void *reply_body_addr = ZXDH_ADDR_OF(msg_reply_info, zxdh_msg_reply_info, reply_body);
+    void *link_msg_addr = ZXDH_ADDR_OF(msg_reply_body, reply_body_addr, link_msg);
 
     if (zxdh_pci_with_feature(hw, ZXDH_NET_F_STATUS))
         zxdh_pci_read_dev_config(hw, offsetof(struct zxdh_net_config, status),
@@ -295,17 +298,18 @@ zxdh_link_info_get(struct rte_eth_dev *dev, struct rte_eth_link *link)
         zxdh_agent_msg_build(hw, ZXDH_MAC_LINK_GET, &msg_info);
 
         ret = zxdh_send_msg_to_riscv(dev, &msg_info, sizeof(struct zxdh_msg_info),
-                &reply_info, sizeof(struct zxdh_msg_reply_info),
+                zxdh_msg_reply_info, ZXDH_ST_SZ_BYTES(msg_reply_info),
                 ZXDH_BAR_MODULE_MAC);
         if (ret) {
             PMD_DRV_LOG(ERR, "Failed to send msg: port 0x%x msg type %d",
                     hw->vport.vport, ZXDH_MAC_LINK_GET);
             return -1;
         }
-        link->link_speed = reply_info.reply_body.link_msg.speed;
-        link->link_autoneg = reply_info.reply_body.link_msg.autoneg;
-        hw->speed_mode = reply_info.reply_body.link_msg.speed_modes;
-        if ((reply_info.reply_body.link_msg.duplex & RTE_ETH_LINK_FULL_DUPLEX) ==
+
+        link->link_speed = ZXDH_GET(link_info_msg, link_msg_addr, speed);
+        link->link_autoneg = ZXDH_GET(link_info_msg, link_msg_addr, autoneg);
+        hw->speed_mode = ZXDH_GET(link_info_msg, link_msg_addr, speed_modes);
+        if ((ZXDH_GET(link_info_msg, link_msg_addr, duplex) & RTE_ETH_LINK_FULL_DUPLEX) ==
                 RTE_ETH_LINK_FULL_DUPLEX)
             link->link_duplex = RTE_ETH_LINK_FULL_DUPLEX;
         else
@@ -433,7 +437,7 @@ zxdh_dev_mac_addr_set(struct rte_eth_dev *dev, struct rte_ether_addr *addr)
         ret = zxdh_del_mac_table(hw, hw->vport.vport, old_addr,
             hw->hash_search_index, 0, 0);
         if (ret) {
-            PMD_DRV_LOG(ERR, "mac_addr_add failed, code:%d", ret);
+            PMD_DRV_LOG(ERR, "mac_addr_del failed, code:%d", ret);
             return ret;
         }
         hw->uc_num--;
@@ -467,6 +471,8 @@ zxdh_dev_mac_addr_set(struct rte_eth_dev *dev, struct rte_ether_addr *addr)
         hw->uc_num--;
     }
     rte_ether_addr_copy(addr, (struct rte_ether_addr *)hw->mac_addr);
+    zxdh_pci_write_dev_config(hw, offsetof(struct zxdh_net_config, mac),
+                                &hw->mac_addr, RTE_ETHER_ADDR_LEN);
     return ret;
 }
 
@@ -566,7 +572,7 @@ zxdh_dev_mac_addr_add(struct rte_eth_dev *dev, struct rte_ether_addr *mac_addr,
 
 void zxdh_dev_mac_addr_remove(struct rte_eth_dev *dev, uint32_t index)
 {
-    struct zxdh_hw *hw    = dev->data->dev_private;
+    struct zxdh_hw *hw = dev->data->dev_private;
     struct zxdh_msg_info msg_info = {0};
     struct rte_ether_addr *mac_addr = &dev->data->mac_addrs[index];
     uint16_t ret = 0;
@@ -1072,7 +1078,9 @@ zxdh_dev_rss_reta_query(struct rte_eth_dev *dev,
 {
     struct zxdh_hw *hw = (struct zxdh_hw *)dev->data->dev_private;
     struct zxdh_msg_info msg = {0};
-    struct zxdh_msg_reply_info reply_msg = {0};
+    uint8_t zxdh_msg_reply_info[ZXDH_ST_SZ_BYTES(msg_reply_info)] = {0};
+    void *reply_body_addr = ZXDH_ADDR_OF(msg_reply_info, zxdh_msg_reply_info, reply_body);
+    void *rss_reta_msg_addr = ZXDH_ADDR_OF(msg_reply_body, reply_body_addr, rss_reta_msg);
     uint16_t idx;
     uint16_t i;
     int ret = 0;
@@ -1094,21 +1102,21 @@ zxdh_dev_rss_reta_query(struct rte_eth_dev *dev,
     zxdh_msg_head_build(hw, ZXDH_RSS_RETA_GET, &msg);
 
     if (hw->is_pf) {
-        ret = zxdh_rss_table_get(hw, hw->vport.vport, &reply_msg.reply_body.rss_reta);
+        ret = zxdh_rss_table_get(hw, hw->vport.vport, rss_reta_msg_addr);
         if (ret) {
             PMD_DRV_LOG(ERR, "rss reta table set failed");
             return -EINVAL;
         }
     } else {
         ret = zxdh_vf_send_msg_to_pf(dev, &msg, sizeof(struct zxdh_msg_info),
-                    &reply_msg, sizeof(struct zxdh_msg_reply_info));
+                    zxdh_msg_reply_info, ZXDH_ST_SZ_BYTES(msg_reply_info));
         if (ret) {
             PMD_DRV_LOG(ERR, "vf rss reta table get failed");
             return -EINVAL;
         }
     }
 
-    struct zxdh_rss_reta *reta_table = &reply_msg.reply_body.rss_reta;
+    struct zxdh_rss_reta *reta_table = rss_reta_msg_addr;
 
     for (idx = 0, i = 0; i < reta_size; ++i) {
         idx = i / RTE_ETH_RETA_GROUP_SIZE;
@@ -1232,10 +1240,13 @@ zxdh_rss_hash_conf_get(struct rte_eth_dev *dev, struct rte_eth_rss_conf *rss_con
     struct zxdh_hw *hw = (struct zxdh_hw *)dev->data->dev_private;
     struct rte_eth_rss_conf *old_rss_conf = &dev->data->dev_conf.rx_adv_conf.rss_conf;
     struct zxdh_msg_info msg = {0};
-    struct zxdh_msg_reply_info reply_msg = {0};
+    uint8_t zxdh_msg_reply_info[ZXDH_ST_SZ_BYTES(msg_reply_info)] = {0};
+    void *reply_body_addr = ZXDH_ADDR_OF(msg_reply_info, zxdh_msg_reply_info, reply_body);
+    void *rss_hf_msg_addr = ZXDH_ADDR_OF(msg_reply_body, reply_body_addr, rss_hf_msg);
     struct zxdh_port_attr_table port_attr = {0};
-    int ret;
+    uint32_t rss_hf;
     uint32_t hw_hf;
+    int ret;
 
     if (rss_conf == NULL) {
         PMD_DRV_LOG(ERR, "rss conf is NULL");
@@ -1252,16 +1263,17 @@ zxdh_rss_hash_conf_get(struct rte_eth_dev *dev, struct rte_eth_rss_conf *rss_con
             PMD_DRV_LOG(ERR, "rss hash factor set failed");
             return -EINVAL;
         }
-        reply_msg.reply_body.rss_hf.rss_hf = port_attr.rss_hash_factor;
+        ZXDH_SET(rss_hf, rss_hf_msg_addr, rss_hf, port_attr.rss_hash_factor);
     } else {
         ret = zxdh_vf_send_msg_to_pf(dev, &msg, sizeof(struct zxdh_msg_info),
-                &reply_msg, sizeof(struct zxdh_msg_reply_info));
+                zxdh_msg_reply_info, ZXDH_ST_SZ_BYTES(msg_reply_info));
         if (ret) {
             PMD_DRV_LOG(ERR, "rss hash factor set failed");
             return -EINVAL;
         }
     }
-    rss_conf->rss_hf = zxdh_rss_hf_to_eth(reply_msg.reply_body.rss_hf.rss_hf);
+    rss_hf = ZXDH_GET(rss_hf, rss_hf_msg_addr, rss_hf);
+    rss_conf->rss_hf = zxdh_rss_hf_to_eth(rss_hf);
 
     return 0;
 }
@@ -1382,7 +1394,8 @@ zxdh_hw_vqm_stats_get(struct rte_eth_dev *dev, enum zxdh_agent_msg_type opcode,
 {
     struct zxdh_hw *hw = dev->data->dev_private;
     struct zxdh_msg_info msg_info = {0};
-    struct zxdh_msg_reply_info reply_info = {0};
+    uint8_t zxdh_msg_reply_info[ZXDH_ST_SZ_BYTES(msg_reply_info)] = {0};
+    void *reply_body_addr = ZXDH_ADDR_OF(msg_reply_info, zxdh_msg_reply_info, reply_body);
     enum ZXDH_BAR_MODULE_ID module_id;
     int ret = 0;
 
@@ -1404,14 +1417,15 @@ zxdh_hw_vqm_stats_get(struct rte_eth_dev *dev, enum zxdh_agent_msg_type opcode,
     zxdh_agent_msg_build(hw, opcode, &msg_info);
 
     ret = zxdh_send_msg_to_riscv(dev, &msg_info, sizeof(struct zxdh_msg_info),
-                &reply_info, sizeof(struct zxdh_msg_reply_info), module_id);
+                zxdh_msg_reply_info, ZXDH_ST_SZ_BYTES(msg_reply_info), module_id);
     if (ret) {
         PMD_DRV_LOG(ERR, "Failed to get hw stats");
         return -1;
     }
-    struct zxdh_msg_reply_body *reply_body = &reply_info.reply_body;
 
-    memcpy(hw_stats, &reply_body->vqm_stats, sizeof(struct zxdh_hw_vqm_stats));
+    void *hw_vqm_stats = ZXDH_ADDR_OF(msg_reply_body, reply_body_addr, vqm_stats);
+    memcpy(hw_stats, hw_vqm_stats, sizeof(struct zxdh_hw_vqm_stats));
+
     return 0;
 }
 
@@ -1578,7 +1592,9 @@ zxdh_hw_np_stats_get(struct rte_eth_dev *dev, struct zxdh_hw_np_stats *np_stats)
 {
     struct zxdh_hw *hw = dev->data->dev_private;
     struct zxdh_msg_info msg_info = {0};
-    struct zxdh_msg_reply_info reply_info = {0};
+    uint8_t zxdh_msg_reply_info[ZXDH_ST_SZ_BYTES(msg_reply_info)] = {0};
+    void *reply_body_addr = ZXDH_ADDR_OF(msg_reply_info, zxdh_msg_reply_info, reply_body);
+    void *hw_stas_addr = ZXDH_ADDR_OF(msg_reply_body, reply_body_addr, hw_stats);
     int ret = 0;
 
     if (hw->is_pf) {
@@ -1590,13 +1606,13 @@ zxdh_hw_np_stats_get(struct rte_eth_dev *dev, struct zxdh_hw_np_stats *np_stats)
     } else {
         zxdh_msg_head_build(hw, ZXDH_GET_NP_STATS, &msg_info);
         ret = zxdh_vf_send_msg_to_pf(dev, &msg_info, sizeof(struct zxdh_msg_info),
-                    &reply_info, sizeof(struct zxdh_msg_reply_info));
+                    zxdh_msg_reply_info, ZXDH_ST_SZ_BYTES(msg_reply_info));
         if (ret) {
             PMD_DRV_LOG(ERR,
                 "Failed to send msg: port 0x%x msg type", hw->vport.vport);
             return -1;
         }
-        memcpy(np_stats, &reply_info.reply_body.np_stats, sizeof(struct zxdh_hw_np_stats));
+        memcpy(np_stats, hw_stas_addr, sizeof(struct zxdh_hw_np_stats));
     }
     return ret;
 }
@@ -1666,7 +1682,7 @@ zxdh_hw_stats_reset(struct rte_eth_dev *dev, enum zxdh_agent_msg_type opcode)
 {
     struct zxdh_hw *hw = dev->data->dev_private;
     struct zxdh_msg_info msg_info = {0};
-    struct zxdh_msg_reply_info reply_info = {0};
+    uint8_t zxdh_msg_reply_info[ZXDH_ST_SZ_BYTES(msg_reply_info)] = {0};
     enum ZXDH_BAR_MODULE_ID module_id;
     int ret = 0;
 
@@ -1685,7 +1701,7 @@ zxdh_hw_stats_reset(struct rte_eth_dev *dev, enum zxdh_agent_msg_type opcode)
     zxdh_agent_msg_build(hw, opcode, &msg_info);
 
     ret = zxdh_send_msg_to_riscv(dev, &msg_info, sizeof(struct zxdh_msg_info),
-                &reply_info, sizeof(struct zxdh_msg_reply_info), module_id);
+                zxdh_msg_reply_info, ZXDH_ST_SZ_BYTES(msg_reply_info), module_id);
     if (ret) {
         PMD_DRV_LOG(ERR, "Failed to reset hw stats");
         return -1;
@@ -1767,13 +1783,13 @@ zxdh_hw_np_stats_vf_reset(struct rte_eth_dev *dev)
 {
     struct zxdh_hw *hw = dev->data->dev_private;
     struct zxdh_msg_info msg_info = {0};
-    struct zxdh_msg_reply_info reply_info = {0};
+    uint8_t zxdh_msg_reply_info[ZXDH_ST_SZ_BYTES(msg_reply_info)] = {0};
     int ret = 0;
 
     msg_info.data.np_stats_query.clear_mode = 1;
     zxdh_msg_head_build(hw, ZXDH_GET_NP_STATS, &msg_info);
     ret = zxdh_vf_send_msg_to_pf(dev, &msg_info, sizeof(struct zxdh_msg_info),
-            &reply_info, sizeof(reply_info));
+            zxdh_msg_reply_info, ZXDH_ST_SZ_BYTES(msg_reply_info));
     if (ret)
         PMD_DRV_LOG(ERR, "Failed to send ZXDH_PORT_METER_STAT_GET msg. code:%d", ret);
 
@@ -2034,29 +2050,24 @@ zxdh_dev_fw_version_get(struct rte_eth_dev *dev,
 {
     struct zxdh_hw *hw = dev->data->dev_private;
     struct zxdh_msg_info msg_info = {0};
-    struct zxdh_msg_reply_info reply_info = {0};
+    uint8_t zxdh_msg_reply_info[ZXDH_ST_SZ_BYTES(msg_reply_info)] = {0};
+    void *reply_body_addr = ZXDH_ADDR_OF(msg_reply_info, zxdh_msg_reply_info, reply_body);
+    void *flash_msg_addr = ZXDH_ADDR_OF(msg_reply_body, reply_body_addr, flash_msg);
     char fw_ver[ZXDH_FWVERS_LEN] = {0};
     uint32_t ret = 0;
 
     zxdh_agent_msg_build(hw, ZXDH_FLASH_FIR_VERSION_GET, &msg_info);
 
-    struct zxdh_msg_recviver_mem rsp_data = {
-            .recv_buffer = (void *)&reply_info,
-            .buffer_len = sizeof(struct zxdh_msg_reply_info),
-    };
-
     ret = zxdh_send_msg_to_riscv(dev, &msg_info, sizeof(struct zxdh_msg_info),
-                &reply_info, sizeof(struct zxdh_msg_reply_info),
+                zxdh_msg_reply_info, ZXDH_ST_SZ_BYTES(msg_reply_info),
                 ZXDH_MODULE_FLASH);
     if (ret) {
         PMD_DRV_LOG(ERR, "Failed to send msg: port 0x%x msg type %d",
                 hw->vport.vport, ZXDH_FLASH_FIR_VERSION_GET);
         return -1;
     }
-    struct zxdh_msg_reply_body *ack_msg =
-             &(((struct zxdh_msg_reply_info *)rsp_data.recv_buffer)->reply_body);
 
-    memcpy(fw_ver, ack_msg->flash_msg.firmware_version, ZXDH_FWVERS_LEN);
+    memcpy(fw_ver, flash_msg_addr, ZXDH_FWVERS_LEN);
     snprintf(fw_version, ZXDH_FWVERS_LEN - 1, "%s", fw_ver);
 
     return 0;
@@ -2068,7 +2079,7 @@ zxdh_en_module_eeprom_read(struct rte_eth_dev *dev,
 {
     struct zxdh_hw *hw = dev->data->dev_private;
     struct zxdh_msg_info msg_info = {0};
-    struct zxdh_msg_reply_info reply_info = {0};
+    uint8_t zxdh_msg_reply_info[ZXDH_ST_SZ_BYTES(msg_reply_info)] = {0};
     uint8_t ret = 0;
 
     zxdh_agent_msg_build(hw, ZXDH_MAC_MODULE_EEPROM_READ, &msg_info);
@@ -2079,26 +2090,24 @@ zxdh_en_module_eeprom_read(struct rte_eth_dev *dev,
     msg_info.data.module_eeprom_msg.offset = query->offset;
     msg_info.data.module_eeprom_msg.length = query->length;
 
-    struct zxdh_msg_recviver_mem rsp_data = {
-            .recv_buffer = (void *)&reply_info,
-            .buffer_len = sizeof(struct zxdh_msg_reply_info),
-    };
-
     ret = zxdh_send_msg_to_riscv(dev, &msg_info, sizeof(struct zxdh_msg_info),
-                &reply_info, sizeof(struct zxdh_msg_reply_info),
+                zxdh_msg_reply_info, ZXDH_ST_SZ_BYTES(msg_reply_info),
                 ZXDH_BAR_MODULE_MAC);
     if (ret) {
         PMD_DRV_LOG(ERR, "Failed to send msg: port 0x%x msg type %d",
                 hw->vport.vport, ZXDH_MAC_MODULE_EEPROM_READ);
         return -1;
     }
-    struct zxdh_msg_reply_body *ack_msg =
-             &(((struct zxdh_msg_reply_info *)rsp_data.recv_buffer)->reply_body);
-
+    void *reply_body_addr = ZXDH_ADDR_OF(msg_reply_info, zxdh_msg_reply_info, reply_body);
+    void *module_eeprom_msg_addr =
+            ZXDH_ADDR_OF(msg_reply_body, reply_body_addr, module_eeprom_msg);
+    void *agent_mac_module_eeprom_msg_data_addr =
+            ZXDH_ADDR_OF(agent_mac_module_eeprom_msg, module_eeprom_msg_addr, data);
+    uint8_t length = ZXDH_GET(agent_mac_module_eeprom_msg, module_eeprom_msg_addr, length);
     if (data)
-        memcpy(data, ack_msg->module_eeprom_msg.data, ack_msg->module_eeprom_msg.length);
+        memcpy(data, agent_mac_module_eeprom_msg_data_addr, length);
 
-    return ack_msg->module_eeprom_msg.length;
+    return length;
 }
 
 int
diff --git a/drivers/net/zxdh/zxdh_ethdev_ops.h b/drivers/net/zxdh/zxdh_ethdev_ops.h
index a5162a6d6b..97a1eb4532 100644
--- a/drivers/net/zxdh/zxdh_ethdev_ops.h
+++ b/drivers/net/zxdh/zxdh_ethdev_ops.h
@@ -62,6 +62,30 @@ struct zxdh_hw_np_stats {
     uint64_t tx_ssvpc_pkts;
 };
 
+struct zxdh_ifc_hw_np_stats_bits {
+    uint8_t rx_unicast_pkts[0x40];
+    uint8_t tx_unicast_pkts[0x40];
+    uint8_t rx_unicast_bytes[0x40];
+    uint8_t tx_unicast_bytes[0x40];
+    uint8_t rx_multicast_pkts[0x40];
+    uint8_t tx_multicast_pkts[0x40];
+    uint8_t rx_multicast_bytes[0x40];
+    uint8_t tx_multicast_bytes[0x40];
+    uint8_t rx_broadcast_pkts[0x40];
+    uint8_t tx_broadcast_pkts[0x40];
+    uint8_t rx_broadcast_bytes[0x40];
+    uint8_t tx_broadcast_bytes[0x40];
+    uint8_t rx_mtu_drop_pkts[0x40];
+    uint8_t tx_mtu_drop_pkts[0x40];
+    uint8_t rx_mtu_drop_bytes[0x40];
+    uint8_t tx_mtu_drop_bytes[0x40];
+    uint8_t rx_mtr_drop_pkts[0x40];
+    uint8_t tx_mtr_drop_pkts[0x40];
+    uint8_t rx_mtr_drop_bytes[0x40];
+    uint8_t tx_mtr_drop_bytes[0x40];
+    uint8_t tx_ssvpc_pkts[0x40];
+};
+
 struct zxdh_hw_vqm_stats {
     uint64_t rx_total;
     uint64_t tx_total;
@@ -72,6 +96,17 @@ struct zxdh_hw_vqm_stats {
     uint64_t rx_drop;
 };
 
+struct zxdh_ifc_hw_vqm_stats_bits {
+    uint8_t rx_total[0x40];
+    uint8_t tx_total[0x40];
+    uint8_t rx_bytes[0x40];
+    uint8_t tx_bytes[0x40];
+    uint8_t rx_error[0x40];
+    uint8_t tx_error[0x40];
+    uint8_t rx_drop[0x40];
+};
+
+
 int zxdh_dev_set_link_up(struct rte_eth_dev *dev);
 int zxdh_dev_set_link_down(struct rte_eth_dev *dev);
 int32_t zxdh_dev_link_update(struct rte_eth_dev *dev, int32_t wait_to_complete);
diff --git a/drivers/net/zxdh/zxdh_msg.c b/drivers/net/zxdh/zxdh_msg.c
index 96ad638e83..02ecd93b12 100644
--- a/drivers/net/zxdh/zxdh_msg.c
+++ b/drivers/net/zxdh/zxdh_msg.c
@@ -18,6 +18,7 @@
 #include "zxdh_pci.h"
 #include "zxdh_tables.h"
 #include "zxdh_np.h"
+#include "zxdh_common.h"
 
 #define ZXDH_REPS_INFO_FLAG_USABLE  0x00
 #define ZXDH_BAR_SEQID_NUM_MAX      256
@@ -695,7 +696,7 @@ static uint16_t
 zxdh_bar_chan_sync_msg_reps_get(uint64_t subchan_addr,
         uint64_t recv_buffer, uint16_t buffer_len)
 {
-    struct zxdh_bar_msg_header msg_header = {0};
+    struct zxdh_bar_msg_header msg_header;
     uint16_t msg_id = 0;
     uint16_t msg_len = 0;
 
@@ -987,7 +988,7 @@ zxdh_bar_chan_msg_header_check(struct zxdh_bar_msg_header *msg_header)
 int
 zxdh_bar_irq_recv(uint8_t src, uint8_t dst, uint64_t virt_addr, void *dev)
 {
-    struct zxdh_bar_msg_header msg_header = {0};
+    struct zxdh_bar_msg_header msg_header;
     uint64_t recv_addr = 0;
     uint64_t reps_addr = 0;
     uint16_t ret = 0;
@@ -1083,22 +1084,20 @@ zxdh_vf_send_msg_to_pf(struct rte_eth_dev *dev,  void *msg_req,
 {
     struct zxdh_hw *hw  = dev->data->dev_private;
     struct zxdh_msg_recviver_mem result = {0};
-    struct zxdh_msg_reply_info reply_info = {0};
+    uint8_t zxdh_msg_reply_info[ZXDH_ST_SZ_BYTES(msg_reply_info)] = {0};
     int ret = 0;
 
     if (reply) {
-        RTE_ASSERT(reply_len < sizeof(struct zxdh_msg_reply_info));
+        RTE_ASSERT(reply_len < ZXDH_ST_SZ_BYTES(msg_reply_info));
         result.recv_buffer  = reply;
         result.buffer_len = reply_len;
     } else {
-        result.recv_buffer = &reply_info;
-        result.buffer_len = sizeof(reply_info);
+        result.recv_buffer = zxdh_msg_reply_info;
+        result.buffer_len = ZXDH_ST_SZ_BYTES(msg_reply_info);
     }
 
-    struct zxdh_msg_reply_head *reply_head =
-                &(((struct zxdh_msg_reply_info *)result.recv_buffer)->reply_head);
-    struct zxdh_msg_reply_body *reply_body =
-                &(((struct zxdh_msg_reply_info *)result.recv_buffer)->reply_body);
+    void *reply_head_addr = ZXDH_ADDR_OF(msg_reply_info, result.recv_buffer, reply_head);
+    void *reply_body_addr = ZXDH_ADDR_OF(msg_reply_info, result.recv_buffer, reply_body);
 
     struct zxdh_pci_bar_msg in = {
         .virt_addr = (uint64_t)(hw->bar_addr[ZXDH_BAR0_INDEX] +
@@ -1118,12 +1117,16 @@ zxdh_vf_send_msg_to_pf(struct rte_eth_dev *dev,  void *msg_req,
             "vf[%d] send bar msg to pf failed.ret %d", hw->vport.vfid, ret);
         return -1;
     }
-    if (reply_head->flag != ZXDH_MSG_REPS_OK) {
+
+    uint8_t flag = ZXDH_GET(msg_reply_head, reply_head_addr, flag);
+    uint16_t reps_len = ZXDH_GET(msg_reply_head, reply_head_addr, reps_len);
+    if (flag != ZXDH_MSG_REPS_OK) {
         PMD_MSG_LOG(ERR, "vf[%d] get pf reply failed: reply_head flag : 0x%x(0xff is OK).replylen %d",
-                hw->vport.vfid, reply_head->flag, reply_head->reps_len);
+                hw->vport.vfid, flag, reps_len);
         return -1;
     }
-    if (reply_body->flag != ZXDH_REPS_SUCC) {
+    uint8_t reply_body_flag = ZXDH_GET(msg_reply_body, reply_body_addr, flag);
+    if (reply_body_flag != ZXDH_REPS_SUCC) {
         PMD_MSG_LOG(ERR, "vf[%d] msg processing failed", hw->vfid);
         return -1;
     }
@@ -1137,23 +1140,19 @@ zxdh_send_msg_to_riscv(struct rte_eth_dev *dev, void *msg_req,
 {
     struct zxdh_hw *hw = dev->data->dev_private;
     struct zxdh_msg_recviver_mem result = {0};
-    struct zxdh_msg_reply_info reply_info = {0};
+    uint8_t zxdh_msg_reply_info[ZXDH_ST_SZ_BYTES(msg_reply_info)] = {0};
 
     if (reply) {
-        RTE_ASSERT(reply_len < sizeof(struct zxdh_msg_reply_info));
+        RTE_ASSERT(reply_len < ZXDH_ST_SZ_BYTES(msg_reply_info));
         result.recv_buffer  = reply;
         result.buffer_len = reply_len;
     } else {
-        result.recv_buffer = &reply_info;
-        result.buffer_len = sizeof(reply_info);
+        result.recv_buffer = zxdh_msg_reply_info;
+        result.buffer_len = ZXDH_ST_SZ_BYTES(msg_reply_info);
     }
-    struct zxdh_msg_reply_head *reply_head =
-                &(((struct zxdh_msg_reply_info *)result.recv_buffer)->reply_head);
-    struct zxdh_msg_reply_body *reply_body =
-                &(((struct zxdh_msg_reply_info *)result.recv_buffer)->reply_body);
 
     struct zxdh_pci_bar_msg in = {
-        .payload_addr = &msg_req,
+        .payload_addr = msg_req,
         .payload_len = msg_req_len,
         .virt_addr = (uint64_t)(hw->bar_addr[ZXDH_BAR0_INDEX] + ZXDH_CTRLCH_OFFSET),
         .src = hw->is_pf ? ZXDH_MSG_CHAN_END_PF : ZXDH_MSG_CHAN_END_VF,
@@ -1166,15 +1165,6 @@ zxdh_send_msg_to_riscv(struct rte_eth_dev *dev, void *msg_req,
         PMD_MSG_LOG(ERR, "Failed to send sync messages or receive response");
         return -1;
     }
-    if (reply_head->flag != ZXDH_MSG_REPS_OK) {
-        PMD_MSG_LOG(ERR, "vf[%d] get pf reply failed: reply_head flag : 0x%x(0xff is OK).replylen %d",
-                hw->vport.vfid, reply_head->flag, reply_head->reps_len);
-        return -1;
-    }
-    if (reply_body->flag != ZXDH_REPS_SUCC) {
-        PMD_MSG_LOG(ERR, "vf[%d] msg processing failed", hw->vfid);
-        return -1;
-    }
 
     return 0;
 }
@@ -1245,7 +1235,7 @@ zxdh_vf_promisc_uninit(struct zxdh_hw *hw, union zxdh_virport_num vport)
 
 static int
 zxdh_vf_port_init(struct zxdh_hw *pf_hw, uint16_t vport, void *cfg_data,
-        struct zxdh_msg_reply_body *res_info, uint16_t *res_len)
+        void *res_info, uint16_t *res_len)
 {
     struct zxdh_port_attr_table port_attr = {0};
     union zxdh_virport_num port = {.vport = vport};
@@ -1275,14 +1265,13 @@ zxdh_vf_port_init(struct zxdh_hw *pf_hw, uint16_t vport, void *cfg_data,
         PMD_DRV_LOG(ERR, "vf_promisc_table_init failed, code:%d", ret);
         goto proc_end;
     }
-
-    res_info->flag = ZXDH_REPS_SUCC;
-    *res_len = sizeof(res_info->flag);
+    ZXDH_SET(msg_reply_body, res_info, flag, ZXDH_REPS_SUCC);
+    *res_len = sizeof(uint8_t);
 
     return ret;
 proc_end:
-    *res_len = sizeof(res_info->flag);
-    res_info->flag = ZXDH_REPS_FAIL;
+    *res_len = sizeof(uint8_t);
+    ZXDH_SET(msg_reply_body, res_info, flag, ZXDH_REPS_FAIL);
     return ret;
 }
 
@@ -1311,11 +1300,12 @@ zxdh_mac_clear(struct zxdh_hw *hw, union zxdh_virport_num vport)
 static int
 zxdh_vf_port_uninit(struct zxdh_hw *pf_hw,
         uint16_t vport, void *cfg_data __rte_unused,
-        struct zxdh_msg_reply_body *res_info, uint16_t *res_len)
+        void *res_info, uint16_t *res_len)
 {
     char str[ZXDH_MSG_REPLY_BODY_MAX_LEN] = "uninit";
     struct zxdh_port_attr_table port_attr = {0};
     union zxdh_virport_num vport_num = {.vport = vport};
+    void *reply_data_addr = ZXDH_ADDR_OF(msg_reply_body, res_info, reply_data);
     int ret = 0;
 
     *res_len =  ZXDH_MSG_REPLYBODY_HEAD;
@@ -1340,20 +1330,20 @@ zxdh_vf_port_uninit(struct zxdh_hw *pf_hw,
     }
 
     *res_len += strlen(str);
-    rte_memcpy(&res_info->reply_data, str, strlen(str) + 1);
-    res_info->flag = ZXDH_REPS_SUCC;
+    rte_memcpy(reply_data_addr, str, strlen(str) + 1);
+    ZXDH_SET(msg_reply_body, res_info, flag, ZXDH_REPS_SUCC);
     return ret;
 
 proc_end:
     *res_len += strlen(str);
-    rte_memcpy(&res_info->reply_data, str, strlen(str) + 1);
-    res_info->flag = ZXDH_REPS_FAIL;
+    rte_memcpy(reply_data_addr, str, strlen(str) + 1);
+    ZXDH_SET(msg_reply_body, res_info, flag, ZXDH_REPS_FAIL);
     return ret;
 }
 
 static int
 zxdh_add_vf_mac_table(struct zxdh_hw *hw, uint16_t vport, void *cfg_data,
-        struct zxdh_msg_reply_body *reply_body, uint16_t *reply_len)
+        void *reply_body, uint16_t *reply_len)
 {
     char str[ZXDH_MSG_REPLY_BODY_MAX_LEN] = "add mac";
     union zxdh_virport_num port = {0};
@@ -1362,6 +1352,8 @@ zxdh_add_vf_mac_table(struct zxdh_hw *hw, uint16_t vport, void *cfg_data,
     int i = 0, ret = 0;
     uint16_t vf_id = port.vfid;
     port.vport = vport;
+    void *reply_data_addr = ZXDH_ADDR_OF(msg_reply_body, reply_body, reply_data);
+    void *mac_reply_msg_addr = ZXDH_ADDR_OF(msg_reply_body, reply_body, mac_reply_msg);
 
     for (i = 0; i < ZXDH_MAX_MAC_ADDRS; i++)
         if (rte_is_same_ether_addr(&hw->vfinfo[vf_id].vf_mac[i], addr))
@@ -1369,7 +1361,7 @@ zxdh_add_vf_mac_table(struct zxdh_hw *hw, uint16_t vport, void *cfg_data,
 
     ret = zxdh_add_mac_table(hw, vport, addr, hw->hash_search_index, 0, 0);
     if (ret == -EADDRINUSE) {
-        reply_body->mac_reply_msg.mac_flag = ZXDH_EEXIST_MAC_FLAG;
+        ZXDH_SET(mac_reply_msg, mac_reply_msg_addr, mac_flag, ZXDH_EEXIST_MAC_FLAG);
         PMD_DRV_LOG(ERR, "vf vport 0x%x set mac ret 0x%x failed. mac is in used.",
                 port.vport, ret);
         goto failure;
@@ -1389,26 +1381,27 @@ zxdh_add_vf_mac_table(struct zxdh_hw *hw, uint16_t vport, void *cfg_data,
 success:
     sprintf(str, " vport 0x%x set mac ret 0x%x\n", port.vport, ret);
     *reply_len =  strlen(str) + ZXDH_MSG_REPLYBODY_HEAD;
-    rte_memcpy(&reply_body->reply_data, str, strlen(str) + 1);
-    reply_body->flag = ZXDH_REPS_SUCC;
+    rte_memcpy(reply_data_addr, str, strlen(str) + 1);
+    ZXDH_SET(msg_reply_body, reply_body, flag, ZXDH_REPS_SUCC);
     PMD_DRV_LOG(DEBUG, " reply len %d", *reply_len);
     return ret;
 
 failure:
     *reply_len = strlen(str) + ZXDH_MSG_REPLYBODY_HEAD;
-    reply_body->flag = ZXDH_REPS_FAIL;
+    ZXDH_SET(msg_reply_body, reply_body, flag, ZXDH_REPS_FAIL);
     return ret;
 }
 
 static int
 zxdh_del_vf_mac_table(struct zxdh_hw *hw, uint16_t vport, void *cfg_data,
-    struct zxdh_msg_reply_body *res_info, uint16_t *res_len)
+    void *res_info, uint16_t *res_len)
 {
     int ret, i = 0;
     struct zxdh_mac_filter *mac_filter = (struct zxdh_mac_filter *)cfg_data;
     union zxdh_virport_num  port = (union zxdh_virport_num)vport;
     char str[ZXDH_MSG_REPLY_BODY_MAX_LEN] = "del mac";
     uint16_t  vf_id = port.vfid;
+    void *reply_data_addr = ZXDH_ADDR_OF(msg_reply_body, res_info, reply_data);
 
     PMD_DRV_LOG(DEBUG, "[PF GET MSG FROM VF]--vf mac to del.");
     ret = zxdh_del_mac_table(hw, vport, &mac_filter->mac, hw->hash_search_index, 0, 0);
@@ -1428,19 +1421,19 @@ zxdh_del_vf_mac_table(struct zxdh_hw *hw, uint16_t vport, void *cfg_data,
 
     sprintf(str, "vport 0x%x del mac ret 0x%x\n", port.vport, ret);
     *res_len =  strlen(str) + ZXDH_MSG_REPLYBODY_HEAD;
-    rte_memcpy(&res_info->reply_data, str, strlen(str) + 1);
-    res_info->flag = ZXDH_REPS_SUCC;
+    rte_memcpy(reply_data_addr, str, strlen(str) + 1);
+    ZXDH_SET(msg_reply_body, res_info, flag, ZXDH_REPS_SUCC);
     return ret;
 
 proc_end:
     *res_len = strlen(str) + ZXDH_MSG_REPLYBODY_HEAD;
-    res_info->flag = ZXDH_REPS_FAIL;
+    ZXDH_SET(msg_reply_body, res_info, flag, ZXDH_REPS_FAIL);
     return ret;
 }
 
 static int
 zxdh_vf_promisc_set(struct zxdh_hw *hw, uint16_t vport, void *cfg_data,
-        struct zxdh_msg_reply_body *reply, uint16_t *res_len)
+        void *reply, uint16_t *res_len)
 {
     struct zxdh_port_promisc_msg *promisc_msg = (struct zxdh_port_promisc_msg *)cfg_data;
     int ret = 0;
@@ -1458,24 +1451,25 @@ zxdh_vf_promisc_set(struct zxdh_hw *hw, uint16_t vport, void *cfg_data,
         goto proc_end;
     }
 
-    *res_len = sizeof(struct zxdh_port_attr_set_msg) + sizeof(enum zxdh_reps_flag);
-    reply->flag = ZXDH_REPS_SUCC;
+    *res_len = sizeof(struct zxdh_port_attr_set_msg) + sizeof(uint8_t);
+    ZXDH_SET(msg_reply_body, reply, flag, ZXDH_REPS_SUCC);
 
     return ret;
 
 proc_end:
-    *res_len = sizeof(struct zxdh_port_attr_set_msg) + sizeof(enum zxdh_reps_flag);
-    reply->flag = ZXDH_REPS_FAIL;
+    *res_len = sizeof(struct zxdh_port_attr_set_msg) + sizeof(uint8_t);
+    ZXDH_SET(msg_reply_body, reply, flag, ZXDH_REPS_FAIL);
     return ret;
 }
 
 static int
 zxdh_vf_vlan_filter_table_process(struct zxdh_hw *hw, uint16_t vport, void *cfg_data,
-        struct zxdh_msg_reply_body *res_info, uint16_t *res_len, uint8_t enable)
+        void *res_info, uint16_t *res_len, uint8_t enable)
 {
     struct zxdh_vlan_filter *vlan_filter = cfg_data;
     uint16_t vlan_id =  vlan_filter->vlan_id;
     char str[ZXDH_MSG_REPLY_BODY_MAX_LEN] = "vlan filter table";
+    void *reply_data_addr = ZXDH_ADDR_OF(msg_reply_body, res_info, reply_data);
     int ret = 0;
 
     ret = zxdh_vlan_filter_table_set(hw, vport, vlan_id, enable);
@@ -1483,33 +1477,38 @@ zxdh_vf_vlan_filter_table_process(struct zxdh_hw *hw, uint16_t vport, void *cfg_
         sprintf(str, "vlan filter op-code[%d] vlan id:%d failed, code:%d\n",
             enable, vlan_id, ret);
 
-    *res_len = strlen(str) + sizeof(enum zxdh_reps_flag);
-    memcpy(&res_info->reply_data, str, strlen(str) + 1);
-    res_info->flag = (ret == 0) ? ZXDH_REPS_SUCC : ZXDH_REPS_FAIL;
+    *res_len = strlen(str) + sizeof(uint8_t);
+
+    memcpy(reply_data_addr, str, strlen(str) + 1);
+    if (ret == 0)
+        ZXDH_SET(msg_reply_body, res_info, flag, ZXDH_REPS_SUCC);
+    else
+        ZXDH_SET(msg_reply_body, res_info, flag, ZXDH_REPS_FAIL);
     return ret;
 }
 
 static int
 zxdh_vf_vlan_filter_table_add(struct zxdh_hw *hw, uint16_t vport, void *cfg_data,
-        struct zxdh_msg_reply_body *res_info, uint16_t *res_len)
+        void *res_info, uint16_t *res_len)
 {
     return zxdh_vf_vlan_filter_table_process(hw, vport, cfg_data, res_info, res_len, 1);
 }
 
 static int
 zxdh_vf_vlan_filter_table_del(struct zxdh_hw *hw, uint16_t vport, void *cfg_data,
-        struct zxdh_msg_reply_body *res_info, uint16_t *res_len)
+        void *res_info, uint16_t *res_len)
 {
     return zxdh_vf_vlan_filter_table_process(hw, vport, cfg_data, res_info, res_len, 0);
 }
 
 static int
 zxdh_vf_set_vlan_filter(struct zxdh_hw *hw, uint16_t vport, void *cfg_data,
-        struct zxdh_msg_reply_body *reply, uint16_t *res_len)
+        void *reply, uint16_t *res_len)
 {
     struct zxdh_vlan_filter_set *vlan_filter = cfg_data;
     union zxdh_virport_num port = (union zxdh_virport_num)vport;
     char str[ZXDH_MSG_REPLY_BODY_MAX_LEN] = "vlan filter";
+    void *reply_data_addr = ZXDH_ADDR_OF(msg_reply_body, reply, reply_data);
     int ret = 0;
     uint16_t vfid = port.vfid;
 
@@ -1517,19 +1516,23 @@ zxdh_vf_set_vlan_filter(struct zxdh_hw *hw, uint16_t vport, void *cfg_data,
     if (ret)
         sprintf(str, "[vfid:%d] vlan filter. set failed, ret:%d\n", vfid, ret);
 
-    *res_len = strlen(str) + sizeof(enum zxdh_reps_flag);
-    reply->flag = (ret == 0) ? ZXDH_REPS_SUCC : ZXDH_REPS_FAIL;
-    memcpy(&reply->reply_data, str, strlen(str) + 1);
+    *res_len = strlen(str) + sizeof(uint8_t);
+    if (ret == 0)
+        ZXDH_SET(msg_reply_body, reply, flag, ZXDH_REPS_SUCC);
+    else
+        ZXDH_SET(msg_reply_body, reply, flag, ZXDH_REPS_FAIL);
+    memcpy(reply_data_addr, str, strlen(str) + 1);
     return ret;
 }
 
 static int
 zxdh_vf_set_vlan_offload(struct zxdh_hw *hw, uint16_t vport, void *cfg_data,
-        struct zxdh_msg_reply_body *reply, uint16_t *res_len)
+        void *reply, uint16_t *res_len)
 {
     struct zxdh_vlan_offload *vlan_offload = cfg_data;
     union zxdh_virport_num port = (union zxdh_virport_num)vport;
     char str[ZXDH_MSG_REPLY_BODY_MAX_LEN] = "vlan offload";
+    void *reply_data_addr = ZXDH_ADDR_OF(msg_reply_body, reply, reply_data);
     int ret = 0;
     uint16_t vfid = port.vfid;
 
@@ -1540,18 +1543,23 @@ zxdh_vf_set_vlan_offload(struct zxdh_hw *hw, uint16_t vport, void *cfg_data,
     if (ret)
         sprintf(str, "[vfid:%d] vlan offload set failed, ret:%d\n", vfid, ret);
 
-    *res_len = strlen(str) + sizeof(enum zxdh_reps_flag);
-    reply->flag = (ret == 0) ? ZXDH_REPS_SUCC : ZXDH_REPS_FAIL;
-    memcpy(&reply->reply_data, str, strlen(str) + 1);
+    *res_len = strlen(str) + sizeof(uint8_t);
+    if (ret == 0)
+        ZXDH_SET(msg_reply_body, reply, flag, ZXDH_REPS_SUCC);
+    else
+        ZXDH_SET(msg_reply_body, reply, flag, ZXDH_REPS_FAIL);
+    memcpy(reply_data_addr, str, strlen(str) + 1);
     return ret;
 }
 
 static int
 zxdh_vf_rss_hf_get(struct zxdh_hw *hw, uint16_t vport, void *cfg_data __rte_unused,
-            struct zxdh_msg_reply_body *reply, uint16_t *res_len)
+            void *reply, uint16_t *res_len)
 {
     char str[ZXDH_MSG_REPLY_BODY_MAX_LEN] = "rss_hf";
     struct zxdh_port_attr_table vport_att = {0};
+    void *reply_data_addr = ZXDH_ADDR_OF(msg_reply_body, reply, reply_data);
+    void *rss_hf_msg_addr = ZXDH_ADDR_OF(msg_reply_body, reply, rss_hf_msg);
     int ret = 0;
 
     ret = zxdh_get_port_attr(hw, vport, &vport_att);
@@ -1561,22 +1569,26 @@ zxdh_vf_rss_hf_get(struct zxdh_hw *hw, uint16_t vport, void *cfg_data __rte_unus
         goto proc_end;
     }
 
-    reply->rss_hf.rss_hf = vport_att.rss_hash_factor;
+    ZXDH_SET(rss_hf, rss_hf_msg_addr, rss_hf, vport_att.rss_hash_factor);
 
 proc_end:
-    *res_len = strlen(str) + sizeof(enum zxdh_reps_flag);
-    reply->flag = (ret == 0) ? ZXDH_REPS_SUCC : ZXDH_REPS_FAIL;
-    memcpy(&reply->reply_data, str, strlen(str) + 1);
+    *res_len = strlen(str) + sizeof(uint8_t);
+    if (ret == 0)
+        ZXDH_SET(msg_reply_body, reply, flag, ZXDH_REPS_SUCC);
+    else
+        ZXDH_SET(msg_reply_body, reply, flag, ZXDH_REPS_FAIL);
+    memcpy(reply_data_addr, str, strlen(str) + 1);
     return ret;
 }
 
 static int
 zxdh_vf_rss_hf_set(struct zxdh_hw *hw, uint16_t vport, void *cfg_data,
-            struct zxdh_msg_reply_body *reply, uint16_t *res_len)
+            void *reply, uint16_t *res_len)
 {
     char str[ZXDH_MSG_REPLY_BODY_MAX_LEN] = "rss_hf";
     struct zxdh_rss_hf *rss_hf = cfg_data;
     struct zxdh_port_attr_table vport_att = {0};
+    void *reply_data_addr = ZXDH_ADDR_OF(msg_reply_body, reply, reply_data);
     int ret = 0;
 
     ret = zxdh_get_port_attr(hw, vport, &vport_att);
@@ -1596,19 +1608,23 @@ zxdh_vf_rss_hf_set(struct zxdh_hw *hw, uint16_t vport, void *cfg_data,
     }
 
 proc_end:
-    *res_len = strlen(str) + sizeof(enum zxdh_reps_flag);
-    reply->flag = (ret == 0) ? ZXDH_REPS_SUCC : ZXDH_REPS_FAIL;
-    memcpy(&reply->reply_data, str, strlen(str) + 1);
+    *res_len = strlen(str) + sizeof(uint8_t);
+    if (ret == 0)
+        ZXDH_SET(msg_reply_body, reply, flag, ZXDH_REPS_SUCC);
+    else
+        ZXDH_SET(msg_reply_body, reply, flag, ZXDH_REPS_FAIL);
+    memcpy(reply_data_addr, str, strlen(str) + 1);
     return ret;
 }
 
 static int
 zxdh_vf_rss_enable(struct zxdh_hw *hw, uint16_t vport, void *cfg_data,
-            struct zxdh_msg_reply_body *reply, uint16_t *res_len)
+            void *reply, uint16_t *res_len)
 {
     char str[ZXDH_MSG_REPLY_BODY_MAX_LEN] = "rss_enable";
     struct zxdh_rss_enable *rss_enable = cfg_data;
     struct zxdh_port_attr_table vport_att = {0};
+    void *reply_data_addr = ZXDH_ADDR_OF(msg_reply_body, reply, reply_data);
     int ret = 0;
 
     ret = zxdh_get_port_attr(hw, vport, &vport_att);
@@ -1628,51 +1644,63 @@ zxdh_vf_rss_enable(struct zxdh_hw *hw, uint16_t vport, void *cfg_data,
     }
 
 proc_end:
-    *res_len = strlen(str) + sizeof(enum zxdh_reps_flag);
-    reply->flag = (ret == 0) ? ZXDH_REPS_SUCC : ZXDH_REPS_FAIL;
-    memcpy(&reply->reply_data, str, strlen(str) + 1);
+    *res_len = strlen(str) + sizeof(uint8_t);
+    if (ret == 0)
+        ZXDH_SET(msg_reply_body, reply, flag, ZXDH_REPS_SUCC);
+    else
+        ZXDH_SET(msg_reply_body, reply, flag, ZXDH_REPS_FAIL);
+    memcpy(reply_data_addr, str, strlen(str) + 1);
     return ret;
 }
 
 static int
 zxdh_vf_rss_table_set(struct zxdh_hw *hw, uint16_t vport, void *cfg_data,
-        struct zxdh_msg_reply_body *reply, uint16_t *res_len)
+        void *reply, uint16_t *res_len)
 {
     char str[ZXDH_MSG_REPLY_BODY_MAX_LEN] = "rss_table";
     struct zxdh_rss_reta *rss_reta = cfg_data;
+    void *reply_data_addr = ZXDH_ADDR_OF(msg_reply_body, reply, reply_data);
     int32_t ret = 0;
 
     ret = zxdh_rss_table_set(hw, vport, rss_reta);
     if (ret)
         sprintf(str, "set rss reta tbl failed, code:%d", ret);
 
-    *res_len = strlen(str) + sizeof(enum zxdh_reps_flag);
-    reply->flag = (ret == 0) ? ZXDH_REPS_SUCC : ZXDH_REPS_FAIL;
-    memcpy(&reply->reply_data, str, strlen(str) + 1);
+    *res_len = strlen(str) + sizeof(uint8_t);
+    if (ret == 0)
+        ZXDH_SET(msg_reply_body, reply, flag, ZXDH_REPS_SUCC);
+    else
+        ZXDH_SET(msg_reply_body, reply, flag, ZXDH_REPS_FAIL);
+    memcpy(reply_data_addr, str, strlen(str) + 1);
     return ret;
 }
 
 static int
 zxdh_vf_rss_table_get(struct zxdh_hw *hw, uint16_t vport, void *cfg_data __rte_unused,
-        struct zxdh_msg_reply_body *reply, uint16_t *res_len)
+        void *reply, uint16_t *res_len)
 {
     char str[ZXDH_MSG_REPLY_BODY_MAX_LEN] = "rss_table";
-    struct zxdh_rss_reta *rss_reta = &reply->rss_reta;
+    void *rss_reta_msg_addr = ZXDH_ADDR_OF(msg_reply_body, reply, rss_reta_msg);
+    struct zxdh_rss_reta *rss_reta = (struct zxdh_rss_reta *)rss_reta_msg_addr;
+    void *reply_data_addr = ZXDH_ADDR_OF(msg_reply_body, reply, reply_data);
     int ret = 0;
 
     ret = zxdh_rss_table_get(hw, vport, rss_reta);
     if (ret)
         sprintf(str, "set rss reta tbl failed, code:%d", ret);
 
-    *res_len = strlen(str) + sizeof(enum zxdh_reps_flag);
-    reply->flag = (ret == 0) ? ZXDH_REPS_SUCC : ZXDH_REPS_FAIL;
-    memcpy(&reply->reply_data, str, strlen(str) + 1);
+    *res_len = strlen(str) + sizeof(uint8_t);
+    if (ret == 0)
+        ZXDH_SET(msg_reply_body, reply, flag, ZXDH_REPS_SUCC);
+    else
+        ZXDH_SET(msg_reply_body, reply, flag, ZXDH_REPS_FAIL);
+    memcpy(reply_data_addr, str, strlen(str) + 1);
     return ret;
 }
 
 static int
 zxdh_vf_port_attr_set(struct zxdh_hw *pf_hw, uint16_t vport, void *cfg_data,
-    struct zxdh_msg_reply_body *res_info, uint16_t *res_len)
+    void *res_info, uint16_t *res_len)
 {
     RTE_ASSERT(!cfg_data || !pf_hw);
     if (res_info)
@@ -1734,7 +1762,7 @@ zxdh_vf_port_attr_set(struct zxdh_hw *pf_hw, uint16_t vport, void *cfg_data,
 
 static int
 zxdh_vf_np_stats_update(struct zxdh_hw *pf_hw, uint16_t vport,
-        void *cfg_data, struct zxdh_msg_reply_body *res_info,
+        void *cfg_data, void *res_info,
         uint16_t *res_len)
 {
     struct zxdh_np_stats_updata_msg *np_stats_query =
@@ -1745,6 +1773,47 @@ zxdh_vf_np_stats_update(struct zxdh_hw *pf_hw, uint16_t vport,
     uint32_t idx = 0;
     int ret = 0;
 
+    void *hw_stats_addr = ZXDH_ADDR_OF(msg_reply_body, res_info, hw_stats);
+    void *tx_unicast_pkts_addr =
+        ZXDH_ADDR_OF(hw_np_stats, hw_stats_addr, tx_unicast_pkts);
+    void *rx_unicast_pkts_addr =
+        ZXDH_ADDR_OF(hw_np_stats, hw_stats_addr, rx_unicast_pkts);
+    void *tx_unicast_bytes_addr =
+        ZXDH_ADDR_OF(hw_np_stats, hw_stats_addr, tx_unicast_bytes);
+    void *rx_unicast_bytes_addr =
+        ZXDH_ADDR_OF(hw_np_stats, hw_stats_addr, rx_unicast_bytes);
+    void *tx_multicast_pkts_addr =
+        ZXDH_ADDR_OF(hw_np_stats, hw_stats_addr, tx_multicast_pkts);
+    void *rx_multicast_pkts_addr =
+        ZXDH_ADDR_OF(hw_np_stats, hw_stats_addr, rx_multicast_pkts);
+    void *tx_multicast_bytes_addr =
+        ZXDH_ADDR_OF(hw_np_stats, hw_stats_addr, tx_multicast_bytes);
+    void *rx_multicast_bytes_addr =
+        ZXDH_ADDR_OF(hw_np_stats, hw_stats_addr, rx_multicast_bytes);
+    void *tx_broadcast_pkts_addr =
+        ZXDH_ADDR_OF(hw_np_stats, hw_stats_addr, tx_broadcast_pkts);
+    void *tx_broadcast_bytes_addr =
+        ZXDH_ADDR_OF(hw_np_stats, hw_stats_addr, tx_broadcast_bytes);
+    void *rx_broadcast_pkts_addr =
+        ZXDH_ADDR_OF(hw_np_stats, hw_stats_addr, rx_broadcast_pkts);
+    void *rx_broadcast_bytes_addr =
+        ZXDH_ADDR_OF(hw_np_stats, hw_stats_addr, rx_broadcast_bytes);
+    void *tx_mtu_drop_pkts_addr =
+        ZXDH_ADDR_OF(hw_np_stats, hw_stats_addr, tx_mtu_drop_pkts);
+    void *tx_mtu_drop_bytes_addr =
+        ZXDH_ADDR_OF(hw_np_stats, hw_stats_addr, tx_mtu_drop_bytes);
+    void *rx_mtu_drop_pkts_addr =
+        ZXDH_ADDR_OF(hw_np_stats, hw_stats_addr, rx_mtu_drop_pkts);
+    void *rx_mtu_drop_bytes_addr =
+        ZXDH_ADDR_OF(hw_np_stats, hw_stats_addr, rx_mtu_drop_bytes);
+    void *tx_mtr_drop_pkts_addr =
+        ZXDH_ADDR_OF(hw_np_stats, hw_stats_addr, tx_mtr_drop_pkts);
+    void *tx_mtr_drop_bytes_addr =
+        ZXDH_ADDR_OF(hw_np_stats, hw_stats_addr, tx_mtr_drop_bytes);
+    void *rx_mtr_drop_pkts_addr =
+        ZXDH_ADDR_OF(hw_np_stats, hw_stats_addr, rx_mtr_drop_pkts);
+    void *rx_mtr_drop_bytes_addr =
+        ZXDH_ADDR_OF(hw_np_stats, hw_stats_addr, rx_mtr_drop_bytes);
     if (!res_len || !res_info) {
         PMD_DRV_LOG(ERR, "get stat invalid inparams");
         return -1;
@@ -1760,8 +1829,8 @@ zxdh_vf_np_stats_update(struct zxdh_hw *pf_hw, uint16_t vport,
         PMD_DRV_LOG(ERR, "get stats failed. code:%d", ret);
         return ret;
     }
-    zxdh_data_hi_to_lo(&res_info->np_stats.tx_unicast_pkts);
-    zxdh_data_hi_to_lo(&res_info->np_stats.tx_unicast_bytes);
+    zxdh_data_hi_to_lo(tx_unicast_pkts_addr);
+    zxdh_data_hi_to_lo(tx_unicast_bytes_addr);
 
     idx = zxdh_vport_to_vfid(vport_num) + ZXDH_UNICAST_STATS_INGRESS_BASE;
     memset(&stats_data, 0, sizeof(stats_data));
@@ -1771,8 +1840,8 @@ zxdh_vf_np_stats_update(struct zxdh_hw *pf_hw, uint16_t vport,
         PMD_DRV_LOG(ERR, "get stats failed. code:%d", ret);
         return ret;
     }
-    zxdh_data_hi_to_lo(&res_info->np_stats.rx_unicast_pkts);
-    zxdh_data_hi_to_lo(&res_info->np_stats.rx_unicast_bytes);
+    zxdh_data_hi_to_lo(rx_unicast_pkts_addr);
+    zxdh_data_hi_to_lo(rx_unicast_bytes_addr);
 
     idx = zxdh_vport_to_vfid(vport_num) + ZXDH_MULTICAST_STATS_EGRESS_BASE;
     ret = zxdh_np_dtb_stats_get(pf_hw->dev_id, pf_hw->dev_sd->dtb_sd.queueid,
@@ -1781,8 +1850,8 @@ zxdh_vf_np_stats_update(struct zxdh_hw *pf_hw, uint16_t vport,
         PMD_DRV_LOG(ERR, "get stats failed. code:%d", ret);
         return ret;
     }
-    zxdh_data_hi_to_lo(&res_info->np_stats.tx_multicast_pkts);
-    zxdh_data_hi_to_lo(&res_info->np_stats.tx_multicast_bytes);
+    zxdh_data_hi_to_lo(tx_multicast_pkts_addr);
+    zxdh_data_hi_to_lo(tx_multicast_bytes_addr);
 
     idx = zxdh_vport_to_vfid(vport_num) + ZXDH_MULTICAST_STATS_INGRESS_BASE;
     memset(&stats_data, 0, sizeof(stats_data));
@@ -1792,8 +1861,8 @@ zxdh_vf_np_stats_update(struct zxdh_hw *pf_hw, uint16_t vport,
         PMD_DRV_LOG(ERR, "get stats failed. code:%d", ret);
         return ret;
     }
-    zxdh_data_hi_to_lo(&res_info->np_stats.rx_multicast_pkts);
-    zxdh_data_hi_to_lo(&res_info->np_stats.rx_multicast_bytes);
+    zxdh_data_hi_to_lo(rx_multicast_pkts_addr);
+    zxdh_data_hi_to_lo(rx_multicast_bytes_addr);
 
     idx = zxdh_vport_to_vfid(vport_num) + ZXDH_BROAD_STATS_EGRESS_BASE;
     ret = zxdh_np_dtb_stats_get(pf_hw->dev_id, pf_hw->dev_sd->dtb_sd.queueid,
@@ -1802,8 +1871,8 @@ zxdh_vf_np_stats_update(struct zxdh_hw *pf_hw, uint16_t vport,
         PMD_DRV_LOG(ERR, "get stats failed. code:%d", ret);
         return ret;
     }
-    zxdh_data_hi_to_lo(&res_info->np_stats.tx_broadcast_pkts);
-    zxdh_data_hi_to_lo(&res_info->np_stats.tx_broadcast_bytes);
+    zxdh_data_hi_to_lo(tx_broadcast_pkts_addr);
+    zxdh_data_hi_to_lo(tx_broadcast_bytes_addr);
 
     idx = zxdh_vport_to_vfid(vport_num) + ZXDH_BROAD_STATS_INGRESS_BASE;
     memset(&stats_data, 0, sizeof(stats_data));
@@ -1813,8 +1882,8 @@ zxdh_vf_np_stats_update(struct zxdh_hw *pf_hw, uint16_t vport,
         PMD_DRV_LOG(ERR, "get stats failed. code:%d", ret);
         return ret;
     }
-    zxdh_data_hi_to_lo(&res_info->np_stats.rx_broadcast_pkts);
-    zxdh_data_hi_to_lo(&res_info->np_stats.rx_broadcast_bytes);
+    zxdh_data_hi_to_lo(rx_broadcast_pkts_addr);
+    zxdh_data_hi_to_lo(rx_broadcast_bytes_addr);
 
     idx = zxdh_vport_to_vfid(vport_num) + ZXDH_MTU_STATS_EGRESS_BASE;
     memset(&stats_data, 0, sizeof(stats_data));
@@ -1824,10 +1893,10 @@ zxdh_vf_np_stats_update(struct zxdh_hw *pf_hw, uint16_t vport,
         PMD_DRV_LOG(ERR, "get stats failed. code:%d", ret);
         return ret;
     }
-    res_info->np_stats.tx_mtu_drop_pkts = stats_data.n_pkts_dropped;
-    res_info->np_stats.tx_mtu_drop_bytes = stats_data.n_bytes_dropped;
-    zxdh_data_hi_to_lo(&res_info->np_stats.tx_mtu_drop_pkts);
-    zxdh_data_hi_to_lo(&res_info->np_stats.tx_mtu_drop_bytes);
+    ZXDH_SET(hw_np_stats, hw_stats_addr, tx_mtu_drop_pkts, stats_data.n_pkts_dropped);
+    ZXDH_SET(hw_np_stats, hw_stats_addr, tx_mtu_drop_bytes, stats_data.n_bytes_dropped);
+    zxdh_data_hi_to_lo(tx_mtu_drop_pkts_addr);
+    zxdh_data_hi_to_lo(tx_mtu_drop_bytes_addr);
 
     idx = zxdh_vport_to_vfid(vport_num) + ZXDH_MTU_STATS_INGRESS_BASE;
     memset(&stats_data, 0, sizeof(stats_data));
@@ -1837,10 +1906,10 @@ zxdh_vf_np_stats_update(struct zxdh_hw *pf_hw, uint16_t vport,
         PMD_DRV_LOG(ERR, "get stats failed. code:%d", ret);
         return ret;
     }
-    res_info->np_stats.rx_mtu_drop_pkts = stats_data.n_pkts_dropped;
-    res_info->np_stats.rx_mtu_drop_bytes = stats_data.n_bytes_dropped;
-    zxdh_data_hi_to_lo(&res_info->np_stats.rx_mtu_drop_pkts);
-    zxdh_data_hi_to_lo(&res_info->np_stats.rx_mtu_drop_bytes);
+    ZXDH_SET(hw_np_stats, hw_stats_addr, rx_mtu_drop_pkts, stats_data.n_pkts_dropped);
+    ZXDH_SET(hw_np_stats, hw_stats_addr, rx_mtu_drop_bytes, stats_data.n_bytes_dropped);
+    zxdh_data_hi_to_lo(rx_mtu_drop_pkts_addr);
+    zxdh_data_hi_to_lo(rx_mtu_drop_bytes_addr);
 
     idx = zxdh_vport_to_vfid(vport_num) + ZXDH_MTR_STATS_EGRESS_BASE;
     memset(&stats_data, 0, sizeof(stats_data));
@@ -1850,10 +1919,11 @@ zxdh_vf_np_stats_update(struct zxdh_hw *pf_hw, uint16_t vport,
         PMD_DRV_LOG(ERR, "get stats failed. code:%d", ret);
         return ret;
     }
-    res_info->np_stats.tx_mtr_drop_pkts = stats_data.n_pkts_dropped;
-    res_info->np_stats.tx_mtr_drop_bytes = stats_data.n_bytes_dropped;
-    zxdh_data_hi_to_lo(&res_info->np_stats.tx_mtr_drop_pkts);
-    zxdh_data_hi_to_lo(&res_info->np_stats.tx_mtr_drop_bytes);
+    ZXDH_SET(hw_np_stats, hw_stats_addr, tx_mtr_drop_pkts, stats_data.n_pkts_dropped);
+    ZXDH_SET(hw_np_stats, hw_stats_addr, tx_mtr_drop_bytes, stats_data.n_bytes_dropped);
+
+    zxdh_data_hi_to_lo(tx_mtr_drop_pkts_addr);
+    zxdh_data_hi_to_lo(tx_mtr_drop_bytes_addr);
 
     idx = zxdh_vport_to_vfid(vport_num) + ZXDH_MTR_STATS_INGRESS_BASE;
     memset(&stats_data, 0, sizeof(stats_data));
@@ -1863,10 +1933,11 @@ zxdh_vf_np_stats_update(struct zxdh_hw *pf_hw, uint16_t vport,
         PMD_DRV_LOG(ERR, "get stats failed. code:%d", ret);
         return ret;
     }
-    res_info->np_stats.rx_mtr_drop_pkts = stats_data.n_pkts_dropped;
-    res_info->np_stats.rx_mtr_drop_bytes = stats_data.n_bytes_dropped;
-    zxdh_data_hi_to_lo(&res_info->np_stats.rx_mtr_drop_pkts);
-    zxdh_data_hi_to_lo(&res_info->np_stats.rx_mtr_drop_bytes);
+    ZXDH_SET(hw_np_stats, hw_stats_addr, rx_mtr_drop_pkts, stats_data.n_pkts_dropped);
+    ZXDH_SET(hw_np_stats, hw_stats_addr, rx_mtr_drop_bytes, stats_data.n_bytes_dropped);
+
+    zxdh_data_hi_to_lo(rx_mtr_drop_pkts_addr);
+    zxdh_data_hi_to_lo(rx_mtr_drop_bytes_addr);
     *res_len = sizeof(struct zxdh_hw_np_stats);
 
     return 0;
@@ -1875,12 +1946,13 @@ zxdh_vf_np_stats_update(struct zxdh_hw *pf_hw, uint16_t vport,
 static int
 zxdh_vf_mtr_hw_stats_get(struct zxdh_hw *pf_hw,
     uint16_t vport, void *cfg_data,
-    struct zxdh_msg_reply_body *res_info,
+    void *res_info,
     uint16_t *res_len)
 {
     struct zxdh_mtr_stats_query  *zxdh_mtr_stats_query =
             (struct zxdh_mtr_stats_query  *)cfg_data;
     union zxdh_virport_num v_port = {.vport = vport};
+    uint8_t *hw_mtr_stats_addr = ZXDH_ADDR_OF(msg_reply_body, res_info, hw_mtr_stats);
     int ret = 0;
 
     uint32_t stat_baseaddr = zxdh_mtr_stats_query->direction ==
@@ -1892,14 +1964,14 @@ zxdh_vf_mtr_hw_stats_get(struct zxdh_hw *pf_hw,
         PMD_DRV_LOG(ERR, "get stat invalid in params");
         return -1;
     }
-    res_info->flag = ZXDH_REPS_FAIL;
+    ZXDH_SET(msg_reply_body, res_info, flag, ZXDH_REPS_FAIL);
     ret = zxdh_np_dtb_stats_get(pf_hw->dev_id, pf_hw->dev_sd->dtb_sd.queueid,
-                1, idx, (uint32_t *)&res_info->hw_mtr_stats);
+                1, idx, (uint32_t *)hw_mtr_stats_addr);
     if (ret) {
         PMD_DRV_LOG(ERR, "get dir %d stats  failed", zxdh_mtr_stats_query->direction);
         return ret;
     }
-    res_info->flag = ZXDH_REPS_SUCC;
+    ZXDH_SET(msg_reply_body, res_info, flag, ZXDH_REPS_SUCC);
     *res_len = sizeof(struct zxdh_hw_mtr_stats);
     return 0;
 }
@@ -1908,7 +1980,7 @@ static int
 zxdh_vf_mtr_hw_profile_add(struct zxdh_hw *pf_hw,
     uint16_t vport,
     void *cfg_data,
-    struct zxdh_msg_reply_body *res_info,
+    void *res_info,
     uint16_t *res_len)
 {
     if (!cfg_data || !res_len || !res_info) {
@@ -1917,16 +1989,18 @@ zxdh_vf_mtr_hw_profile_add(struct zxdh_hw *pf_hw,
     }
     struct rte_mtr_error error = {0};
     int ret = 0;
-    uint64_t profile_id = HW_PROFILE_MAX;
+    uint64_t hw_profile_id = HW_PROFILE_MAX;
+    void *mtr_profile_info_addr = ZXDH_ADDR_OF(msg_reply_body, res_info, mtr_profile_info);
 
     struct zxdh_plcr_profile_add  *zxdh_plcr_profile_add =
         (struct zxdh_plcr_profile_add *)cfg_data;
 
-    res_info->flag = ZXDH_REPS_FAIL;
+    ZXDH_SET(msg_reply_body, res_info, flag, ZXDH_REPS_FAIL);
+
     *res_len = sizeof(struct zxdh_mtr_profile_info);
     ret = zxdh_hw_profile_alloc_direct(pf_hw->eth_dev,
         zxdh_plcr_profile_add->car_type,
-        &profile_id, &error);
+        &hw_profile_id, &error);
 
     if (ret) {
         PMD_DRV_LOG(ERR, "pf 0x%x for vf 0x%x alloc hw profile failed",
@@ -1935,9 +2009,9 @@ zxdh_vf_mtr_hw_profile_add(struct zxdh_hw *pf_hw,
         );
         return -1;
     }
-    zxdh_hw_profile_ref(profile_id);
-    res_info->mtr_profile_info.profile_id = profile_id;
-    res_info->flag = ZXDH_REPS_SUCC;
+    zxdh_hw_profile_ref(hw_profile_id);
+    ZXDH_SET(mtr_profile_info, mtr_profile_info_addr, profile_id, hw_profile_id);
+    ZXDH_SET(msg_reply_body, res_info, flag, ZXDH_REPS_SUCC);
 
     return 0;
 }
@@ -1946,7 +2020,7 @@ static int
 zxdh_vf_mtr_hw_profile_del(struct zxdh_hw *pf_hw,
     uint16_t vport,
     void *cfg_data,
-    struct zxdh_msg_reply_body *res_info,
+    void *res_info,
     uint16_t *res_len)
 {
     if (!cfg_data || !res_len || !res_info) {
@@ -1954,7 +2028,7 @@ zxdh_vf_mtr_hw_profile_del(struct zxdh_hw *pf_hw,
         return -1;
     }
 
-    res_info->flag = ZXDH_REPS_FAIL;
+    ZXDH_SET(msg_reply_body, res_info, flag, ZXDH_REPS_FAIL);
     *res_len = 0;
     struct zxdh_plcr_profile_free *mtr_profile_free = (struct zxdh_plcr_profile_free *)cfg_data;
     uint64_t profile_id = mtr_profile_free->profile_id;
@@ -1980,7 +2054,7 @@ zxdh_vf_mtr_hw_profile_del(struct zxdh_hw *pf_hw,
                 RTE_MTR_ERROR_TYPE_METER_PROFILE_ID, NULL,
                 "Meter offload del profile failed ");
     }
-    res_info->flag = ZXDH_REPS_SUCC;
+    ZXDH_SET(msg_reply_body, res_info, flag, ZXDH_REPS_SUCC);
     return 0;
 }
 
@@ -1988,7 +2062,7 @@ static int
 zxdh_vf_mtr_hw_plcrflow_cfg(struct zxdh_hw *pf_hw,
     uint16_t vport,
     void *cfg_data,
-    struct zxdh_msg_reply_body *res_info,
+    void *res_info,
     uint16_t *res_len)
 {
     int ret = 0;
@@ -2000,7 +2074,7 @@ zxdh_vf_mtr_hw_plcrflow_cfg(struct zxdh_hw *pf_hw,
     struct rte_mtr_error error = {0};
     struct zxdh_plcr_flow_cfg *zxdh_plcr_flow_cfg = (struct zxdh_plcr_flow_cfg *)cfg_data;
 
-    res_info->flag = ZXDH_REPS_FAIL;
+    ZXDH_SET(msg_reply_body, res_info, flag, ZXDH_REPS_FAIL);
     *res_len = 0;
     ret = zxdh_np_stat_car_queue_cfg_set(pf_hw->dev_id,
         zxdh_plcr_flow_cfg->car_type,
@@ -2019,7 +2093,7 @@ zxdh_vf_mtr_hw_plcrflow_cfg(struct zxdh_hw *pf_hw,
                 RTE_MTR_ERROR_TYPE_MTR_PARAMS,
                 NULL, "Failed to bind plcr flow.");
     }
-    res_info->flag = ZXDH_REPS_SUCC;
+    ZXDH_SET(msg_reply_body, res_info, flag, ZXDH_REPS_SUCC);
     return 0;
 }
 
@@ -2027,7 +2101,7 @@ static int
 zxdh_vf_mtr_hw_profile_cfg(struct zxdh_hw *pf_hw __rte_unused,
     uint16_t vport,
     void *cfg_data,
-    struct zxdh_msg_reply_body *res_info,
+    void *res_info,
     uint16_t *res_len)
 {
     int ret = 0;
@@ -2036,14 +2110,15 @@ zxdh_vf_mtr_hw_profile_cfg(struct zxdh_hw *pf_hw __rte_unused,
         PMD_DRV_LOG(ERR, " cfg profile invalid inparams");
         return -1;
     }
-    res_info->flag = ZXDH_REPS_FAIL;
+    ZXDH_SET(msg_reply_body, res_info, flag, ZXDH_REPS_FAIL);
     *res_len = 0;
     struct rte_mtr_error error = {0};
     struct zxdh_plcr_profile_cfg *zxdh_plcr_profile_cfg =
         (struct zxdh_plcr_profile_cfg *)cfg_data;
     union zxdh_offload_profile_cfg *plcr_param = &zxdh_plcr_profile_cfg->plcr_param;
 
-    ret = zxdh_np_car_profile_cfg_set(vport,
+    ret = zxdh_np_car_profile_cfg_set(pf_hw->dev_id,
+        vport,
         zxdh_plcr_profile_cfg->car_type,
         zxdh_plcr_profile_cfg->packet_mode,
         zxdh_plcr_profile_cfg->hw_profile_id,
@@ -2052,7 +2127,7 @@ zxdh_vf_mtr_hw_profile_cfg(struct zxdh_hw *pf_hw __rte_unused,
         PMD_DRV_LOG(ERR, "(vport %d)config hw profilefailed", vport);
         return -rte_mtr_error_set(&error, ENOTSUP, RTE_MTR_ERROR_TYPE_METER_PROFILE, NULL, "Meter offload cfg profile failed");
     }
-    res_info->flag = ZXDH_REPS_SUCC;
+    ZXDH_SET(msg_reply_body, res_info, flag, ZXDH_REPS_SUCC);
     return 0;
 }
 
@@ -2083,7 +2158,7 @@ static const zxdh_msg_process_callback zxdh_proc_cb[] = {
 
 static inline int
 zxdh_config_process_callback(struct zxdh_hw *hw, struct zxdh_msg_info *msg_info,
-    struct zxdh_msg_reply_body *res, uint16_t *res_len)
+    void *res, uint16_t *res_len)
 {
     struct zxdh_msg_head *msghead = &msg_info->msg_head;
     int ret = -1;
@@ -2096,13 +2171,13 @@ zxdh_config_process_callback(struct zxdh_hw *hw, struct zxdh_msg_info *msg_info,
         ret = zxdh_proc_cb[msghead->msg_type](hw, msghead->vport,
                     (void *)&msg_info->data, res, res_len);
         if (!ret)
-            res->flag = ZXDH_REPS_SUCC;
+            ZXDH_SET(msg_reply_body, res, flag, ZXDH_REPS_SUCC);
         else
-            res->flag = ZXDH_REPS_FAIL;
+            ZXDH_SET(msg_reply_body, res, flag, ZXDH_REPS_FAIL);
     } else {
-        res->flag = ZXDH_REPS_INVALID;
+        ZXDH_SET(msg_reply_body, res, flag, ZXDH_REPS_INVALID);
     }
-    *res_len += sizeof(res->flag);
+    *res_len += sizeof(uint8_t);
     return ret;
 }
 
@@ -2111,7 +2186,7 @@ pf_recv_bar_msg(void *pay_load, uint16_t len, void *reps_buffer,
     uint16_t *reps_len, void *eth_dev)
 {
     struct zxdh_msg_info *msg_info = (struct zxdh_msg_info *)pay_load;
-    struct zxdh_msg_reply_body *reply_body = reps_buffer;
+    void *reply_data_addr = ZXDH_ADDR_OF(msg_reply_body, reps_buffer, reply_data);
     struct rte_eth_dev *dev = (struct rte_eth_dev *)eth_dev;
     int32_t ret = 0;
     struct zxdh_hw *hw;
@@ -2131,14 +2206,14 @@ pf_recv_bar_msg(void *pay_load, uint16_t len, void *reps_buffer,
         goto msg_proc_end;
     }
 
-    ret = zxdh_config_process_callback(hw, msg_info, reply_body, &reply_len);
-    *reps_len = reply_len + sizeof(struct zxdh_msg_reply_head);
+    ret = zxdh_config_process_callback(hw, msg_info, reps_buffer, &reply_len);
+    *reps_len = reply_len + ZXDH_ST_SZ_BYTES(msg_reply_head);
     return ret;
 
 msg_proc_end:
-    memcpy(reply_body->reply_data, &ret, sizeof(ret));
+    memcpy(reply_data_addr, &ret, sizeof(ret));
     reply_len = sizeof(ret);
-    *reps_len = sizeof(struct zxdh_msg_reply_head) + reply_len;
+    *reps_len = ZXDH_ST_SZ_BYTES(msg_reply_head) + reply_len;
     return ret;
 }
 
diff --git a/drivers/net/zxdh/zxdh_msg.h b/drivers/net/zxdh/zxdh_msg.h
index 58836bb4b7..7dad6f7335 100644
--- a/drivers/net/zxdh/zxdh_msg.h
+++ b/drivers/net/zxdh/zxdh_msg.h
@@ -11,6 +11,7 @@
 
 #include "zxdh_ethdev_ops.h"
 #include "zxdh_mtr.h"
+#include "zxdh_common.h"
 
 #define ZXDH_BAR0_INDEX                 0
 #define ZXDH_CTRLCH_OFFSET              (0x2000)
@@ -46,8 +47,10 @@
 
 #define ZXDH_MSG_REPLYBODY_HEAD    sizeof(enum zxdh_reps_flag)
 #define ZXDH_MSG_HEADER_SIZE       4
-#define ZXDH_MSG_REPLY_BODY_MAX_LEN  \
-        (ZXDH_MSG_PAYLOAD_MAX_LEN - sizeof(struct zxdh_msg_reply_head))
+#define ZXDH_MSG_REPLY_BODY_MAX_LEN \
+        (ZXDH_MSG_PAYLOAD_MAX_LEN - ZXDH_MSG_HEADER_SIZE)
+#define ZXDH_MSG_REPLY_DATA \
+        (ZXDH_MSG_REPLY_BODY_MAX_LEN - ZXDH_MSG_REPLYBODY_HEAD)
 
 #define ZXDH_MSG_HEAD_LEN            8
 #define ZXDH_MSG_REQ_BODY_MAX_LEN  \
@@ -329,12 +332,6 @@ struct zxdh_offset_get_msg {
     uint16_t type;
 };
 
-struct zxdh_msg_reply_head {
-    uint8_t flag;
-    uint16_t reps_len;
-    uint8_t resvd;
-};
-
 enum zxdh_reps_flag {
     ZXDH_REPS_FAIL,
     ZXDH_REPS_SUCC = 0xaa,
@@ -354,18 +351,39 @@ struct zxdh_link_info_msg {
     uint32_t speed;
 };
 
+struct zxdh_ifc_link_info_msg_bits {
+    uint8_t autoneg[0x8];
+    uint8_t link_state[0x8];
+    uint8_t blink_enable[0x8];
+    uint8_t duplex[0x8];
+    uint8_t speed_modes[0x20];
+    uint8_t speed[0x20];
+};
+
 struct zxdh_rss_reta {
     uint32_t reta[RTE_ETH_RSS_RETA_SIZE_256];
 };
 
+struct zxdh_ifc_rss_reta_bits {
+    uint32_t reta[RTE_ETH_RSS_RETA_SIZE_256 * 8];
+};
+
 struct zxdh_rss_hf {
     uint32_t rss_hf;
 };
 
+struct zxdh_ifc_rss_hf_bits {
+    uint8_t rss_hf[0x20];
+};
+
 struct zxdh_mac_reply_msg {
     uint8_t mac_flag;
 };
 
+struct zxdh_ifc_mac_reply_msg_bits {
+    uint8_t mac_flag[0x8];
+};
+
 struct zxdh_mac_module_eeprom_msg {
     uint8_t i2c_addr;
     uint8_t bank;
@@ -375,34 +393,57 @@ struct zxdh_mac_module_eeprom_msg {
     uint8_t data[ZXDH_MODULE_EEPROM_DATA_LEN];
 };
 
+struct zxdh_ifc_agent_mac_module_eeprom_msg_bits {
+    uint8_t i2c_addr[0x8];
+    uint8_t bank[0x8];
+    uint8_t page[0x8];
+    uint8_t offset[0x8];
+    uint8_t length[0x8];
+    uint8_t data[ZXDH_MODULE_EEPROM_DATA_LEN * 8];
+};
+
 struct zxdh_flash_msg {
     uint8_t firmware_version[ZXDH_FWVERS_LEN];
 };
 
+struct zxdh_ifc_agent_flash_msg_bits {
+    uint8_t firmware_version[0x100];
+};
+
 struct zxdh_mtr_profile_info {
     uint64_t profile_id;
 };
 
-struct zxdh_msg_reply_body {
-    enum zxdh_reps_flag flag;
+struct zxdh_ifc_mtr_profile_info_bits {
+    uint8_t profile_id[0x40];
+};
+
+struct zxdh_ifc_msg_reply_body_bits {
+    uint8_t flag[0x8];
     union {
-        uint8_t reply_data[ZXDH_MSG_REPLY_BODY_MAX_LEN - sizeof(enum zxdh_reps_flag)];
-        struct zxdh_hw_np_stats np_stats;
-        struct zxdh_link_info_msg link_msg;
-        struct zxdh_rss_reta rss_reta;
-        struct zxdh_rss_hf rss_hf;
-        struct zxdh_hw_vqm_stats vqm_stats;
-        struct zxdh_mac_reply_msg mac_reply_msg;
-        struct zxdh_flash_msg flash_msg;
-        struct zxdh_mac_module_eeprom_msg module_eeprom_msg;
-        struct zxdh_mtr_profile_info mtr_profile_info;
-        struct zxdh_mtr_stats hw_mtr_stats;
+        uint8_t reply_data[ZXDH_MSG_REPLY_DATA * 8];
+        struct zxdh_ifc_hw_np_stats_bits hw_stats;
+        struct zxdh_ifc_link_info_msg_bits link_msg;
+        struct zxdh_ifc_rss_reta_bits rss_reta_msg;
+        struct zxdh_ifc_rss_hf_bits rss_hf_msg;
+        struct zxdh_ifc_hw_vqm_stats_bits vqm_stats;
+        struct zxdh_ifc_mac_reply_msg_bits mac_reply_msg;
+        struct zxdh_ifc_agent_flash_msg_bits flash_msg;
+        struct zxdh_ifc_agent_mac_module_eeprom_msg_bits module_eeprom_msg;
+        struct zxdh_ifc_mtr_profile_info_bits  mtr_profile_info;
+        struct zxdh_ifc_mtr_stats_bits hw_mtr_stats;
     };
 };
 
-struct zxdh_msg_reply_info {
-    struct zxdh_msg_reply_head reply_head;
-    struct zxdh_msg_reply_body reply_body;
+struct zxdh_ifc_msg_reply_head_bits {
+    uint8_t flag[0x8];
+    uint8_t reps_len[0x10];
+    uint8_t resvd[0x8];
+};
+
+struct zxdh_ifc_msg_reply_info_bits {
+    struct zxdh_ifc_msg_reply_head_bits reply_head;
+    struct zxdh_ifc_msg_reply_body_bits reply_body;
 };
 
 struct zxdh_vf_init_msg {
@@ -412,12 +453,12 @@ struct zxdh_vf_init_msg {
     uint8_t rss_enable;
 };
 
-struct zxdh_msg_head {
-    enum zxdh_msg_type msg_type;
+struct __rte_packed_begin zxdh_msg_head {
+    uint8_t msg_type;
     uint16_t  vport;
     uint16_t  vf_id;
     uint16_t pcieid;
-};
+} __rte_packed_end;
 
 struct zxdh_port_attr_set_msg {
     uint32_t mode;
@@ -455,7 +496,7 @@ struct zxdh_rss_enable {
 };
 
 struct zxdh_agent_msg_head {
-    enum zxdh_agent_msg_type msg_type;
+    uint8_t msg_type;
     uint8_t panel_id;
     uint8_t phyport;
     uint8_t rsv;
@@ -526,7 +567,7 @@ struct zxdh_msg_info {
 typedef int (*zxdh_bar_chan_msg_recv_callback)(void *pay_load, uint16_t len,
         void *reps_buffer, uint16_t *reps_len, void *dev);
 typedef int (*zxdh_msg_process_callback)(struct zxdh_hw *hw, uint16_t vport, void *cfg_data,
-    struct zxdh_msg_reply_body *res_info, uint16_t *res_len);
+    void *res_info, uint16_t *res_len);
 
 typedef int (*zxdh_bar_chan_msg_recv_callback)(void *pay_load, uint16_t len,
             void *reps_buffer, uint16_t *reps_len, void *dev);
diff --git a/drivers/net/zxdh/zxdh_mtr.c b/drivers/net/zxdh/zxdh_mtr.c
index 3797a5b29b..b23f6535a8 100644
--- a/drivers/net/zxdh/zxdh_mtr.c
+++ b/drivers/net/zxdh/zxdh_mtr.c
@@ -281,7 +281,7 @@ zxdh_hw_profile_free_direct(struct rte_eth_dev *dev, ZXDH_PROFILE_TYPE car_type,
 {
     struct zxdh_hw *hw = dev->data->dev_private;
     uint16_t vport = hw->vport.vport;
-    int ret = zxdh_np_car_profile_id_delete(vport, car_type,
+    int ret = zxdh_np_car_profile_id_delete(hw->dev_id, vport, car_type,
             (uint64_t)hw_profile_id);
     if (ret) {
         PMD_DRV_LOG(ERR, "port %u free hw profile %u failed", vport, hw_profile_id);
@@ -299,7 +299,7 @@ zxdh_hw_profile_alloc_direct(struct rte_eth_dev *dev, ZXDH_PROFILE_TYPE car_type
     uint64_t profile_id = HW_PROFILE_MAX;
     struct zxdh_hw *hw = dev->data->dev_private;
     uint16_t vport = hw->vport.vport;
-    int ret = zxdh_np_car_profile_id_add(vport, car_type, &profile_id);
+    int ret = zxdh_np_car_profile_id_add(hw->dev_id, vport, car_type, &profile_id);
 
     if (ret) {
         PMD_DRV_LOG(ERR, "port %u alloc hw profile failed", vport);
@@ -326,7 +326,7 @@ zxdh_hw_profile_free(struct rte_eth_dev *dev, uint8_t car_type,
         ret = zxdh_hw_profile_free_direct(dev, car_type, (uint64_t)hw_profile_id, error);
     } else {
         struct zxdh_msg_info msg_info = {0};
-        struct zxdh_msg_reply_info reply_info = {0};
+        uint8_t zxdh_msg_reply_info[ZXDH_ST_SZ_BYTES(msg_reply_info)] = {0};
         struct zxdh_plcr_profile_free *zxdh_plcr_profile_free =
             &msg_info.data.zxdh_plcr_profile_free;
 
@@ -335,7 +335,7 @@ zxdh_hw_profile_free(struct rte_eth_dev *dev, uint8_t car_type,
         zxdh_msg_head_build(hw, ZXDH_PLCR_CAR_PROFILE_ID_DELETE, &msg_info);
         ret = zxdh_vf_send_msg_to_pf(dev, &msg_info,
             ZXDH_MSG_HEAD_LEN + sizeof(struct zxdh_plcr_profile_free),
-            &reply_info, sizeof(struct zxdh_msg_reply_info));
+            zxdh_msg_reply_info, ZXDH_ST_SZ_BYTES(msg_reply_info));
 
         if (ret)
             return -rte_mtr_error_set(error, ENOTSUP,
@@ -357,15 +357,19 @@ zxdh_hw_profile_alloc(struct rte_eth_dev *dev, uint64_t *hw_profile_id,
         ret = zxdh_hw_profile_alloc_direct(dev, CAR_A, hw_profile_id, error);
     } else {
         struct zxdh_msg_info msg_info = {0};
-        struct zxdh_msg_reply_info reply_info = {0};
+        uint8_t zxdh_msg_reply_info[ZXDH_ST_SZ_BYTES(msg_reply_info)] = {0};
         struct zxdh_plcr_profile_add  *zxdh_plcr_profile_add =
             &msg_info.data.zxdh_plcr_profile_add;
+        void *reply_body_addr =
+            ZXDH_ADDR_OF(msg_reply_info, zxdh_msg_reply_info, reply_body);
+        void *mtr_profile_info_addr =
+            ZXDH_ADDR_OF(msg_reply_body, reply_body_addr, mtr_profile_info);
 
         zxdh_plcr_profile_add->car_type = CAR_A;
         zxdh_msg_head_build(hw, ZXDH_PLCR_CAR_PROFILE_ID_ADD, &msg_info);
         ret = zxdh_vf_send_msg_to_pf(dev, &msg_info,
             ZXDH_MSG_HEAD_LEN + sizeof(struct zxdh_plcr_profile_add),
-            &reply_info, sizeof(struct zxdh_msg_reply_info));
+            zxdh_msg_reply_info, ZXDH_ST_SZ_BYTES(msg_reply_info));
 
         if (ret) {
             PMD_DRV_LOG(ERR,
@@ -376,7 +380,7 @@ zxdh_hw_profile_alloc(struct rte_eth_dev *dev, uint64_t *hw_profile_id,
                     RTE_MTR_ERROR_TYPE_METER_PROFILE_ID, NULL,
                     "Meter offload alloc profile  id msg failed ");
         }
-        *hw_profile_id = reply_info.reply_body.mtr_profile_info.profile_id;
+        *hw_profile_id = ZXDH_GET(mtr_profile_info, mtr_profile_info_addr, profile_id);
         if (*hw_profile_id == ZXDH_HW_PROFILE_MAX) {
             return -rte_mtr_error_set(error, ENOTSUP,
                     RTE_MTR_ERROR_TYPE_METER_PROFILE_ID, NULL,
@@ -432,7 +436,7 @@ zxdh_mtr_hw_counter_query(struct rte_eth_dev *dev,
         }
     } else { /* send msg to pf */
         struct zxdh_msg_info msg_info = {0};
-        struct zxdh_msg_reply_info reply_info = {0};
+        uint8_t zxdh_msg_reply_info[ZXDH_ST_SZ_BYTES(msg_reply_info)] = {0};
         struct zxdh_mtr_stats_query *zxdh_mtr_stats_query =
                 &msg_info.data.zxdh_mtr_stats_query;
 
@@ -442,8 +446,8 @@ zxdh_mtr_hw_counter_query(struct rte_eth_dev *dev,
         ret = zxdh_vf_send_msg_to_pf(dev,
             &msg_info,
             sizeof(msg_info),
-            &reply_info,
-            sizeof(struct zxdh_msg_reply_info));
+            zxdh_msg_reply_info,
+            ZXDH_ST_SZ_BYTES(msg_reply_info));
 
         if (ret) {
             PMD_DRV_LOG(ERR,
@@ -451,7 +455,11 @@ zxdh_mtr_hw_counter_query(struct rte_eth_dev *dev,
                 hw->vport.vport);
             return -rte_mtr_error_set(error, ENOTSUP, RTE_MTR_ERROR_TYPE_STATS, NULL, "Meter offload alloc profile failed");
         }
-        struct zxdh_mtr_stats *hw_mtr_stats = &reply_info.reply_body.hw_mtr_stats;
+        void *reply_body_addr =
+            ZXDH_ADDR_OF(msg_reply_info, zxdh_msg_reply_info, reply_body);
+        void *hw_mtr_stats_addr =
+            ZXDH_ADDR_OF(msg_reply_body, reply_body_addr, hw_mtr_stats);
+        struct zxdh_mtr_stats *hw_mtr_stats = (struct zxdh_mtr_stats *)hw_mtr_stats_addr;
 
         mtr_stats->n_bytes_dropped = hw_mtr_stats->n_bytes_dropped;
         mtr_stats->n_pkts_dropped = hw_mtr_stats->n_pkts_dropped;
@@ -551,7 +559,9 @@ zxdh_hw_profile_config_direct(struct rte_eth_dev *dev __rte_unused,
     struct zxdh_meter_profile *mp,
     struct rte_mtr_error *error)
 {
-    int ret = zxdh_np_car_profile_cfg_set(mp->hw_profile_owner_vport,
+    struct zxdh_hw *hw = dev->data->dev_private;
+    int ret = zxdh_np_car_profile_cfg_set(hw->dev_id,
+        mp->hw_profile_owner_vport,
         car_type, mp->profile.packet_mode,
         (uint32_t)hw_profile_id, &mp->plcr_param);
     if (ret) {
@@ -573,7 +583,7 @@ static int zxdh_hw_profile_config(struct rte_eth_dev *dev, uint16_t hw_profile_i
         ret = zxdh_hw_profile_config_direct(dev, CAR_A, hw_profile_id, mp, error);
     } else {
         struct zxdh_msg_info msg_info = {0};
-        struct zxdh_msg_reply_info reply_info = {0};
+        uint8_t zxdh_msg_reply_info[ZXDH_ST_SZ_BYTES(msg_reply_info)] = {0};
         struct zxdh_plcr_profile_cfg *zxdh_plcr_profile_cfg =
             &msg_info.data.zxdh_plcr_profile_cfg;
 
@@ -588,9 +598,9 @@ static int zxdh_hw_profile_config(struct rte_eth_dev *dev, uint16_t hw_profile_i
         ret = zxdh_vf_send_msg_to_pf(dev,
             &msg_info,
             ZXDH_MSG_HEAD_LEN + sizeof(struct zxdh_plcr_profile_cfg),
-            &reply_info,
-            sizeof(struct zxdh_msg_reply_info));
-
+            zxdh_msg_reply_info,
+            ZXDH_ST_SZ_BYTES(msg_reply_info)
+        );
         if (ret) {
             PMD_DRV_LOG(ERR,
                 "Failed msg: port 0x%x msg type ZXDH_PLCR_CAR_PROFILE_CFG_SET ",
@@ -874,7 +884,7 @@ zxdh_set_mtr_enable(struct rte_eth_dev *dev, uint8_t dir, bool enable, struct rt
 
     if (priv->is_pf) {
         ret = zxdh_get_port_attr(priv, priv->vport.vport, &port_attr);
-        port_attr.ingress_meter_enable = enable;
+        port_attr.egress_meter_enable = enable;
         ret = zxdh_set_port_attr(priv, priv->vport.vport, &port_attr);
         if (ret) {
             PMD_DRV_LOG(ERR, "%s set port attr failed", __func__);
@@ -936,7 +946,7 @@ zxdh_hw_plcrflow_config(struct rte_eth_dev *dev, uint16_t hw_flow_id,
         }
     } else {
         struct zxdh_msg_info msg_info = {0};
-        struct zxdh_msg_reply_info reply_info = {0};
+        uint8_t zxdh_msg_reply_info[ZXDH_ST_SZ_BYTES(msg_reply_info)] = {0};
         struct zxdh_plcr_flow_cfg *zxdh_plcr_flow_cfg = &msg_info.data.zxdh_plcr_flow_cfg;
 
         zxdh_plcr_flow_cfg->car_type = CAR_A;
@@ -947,8 +957,8 @@ zxdh_hw_plcrflow_config(struct rte_eth_dev *dev, uint16_t hw_flow_id,
         zxdh_msg_head_build(hw, ZXDH_PLCR_CAR_QUEUE_CFG_SET, &msg_info);
         ret = zxdh_vf_send_msg_to_pf(dev, &msg_info,
             ZXDH_MSG_HEAD_LEN + sizeof(struct zxdh_plcr_flow_cfg),
-            &reply_info,
-            sizeof(struct zxdh_msg_reply_info));
+            zxdh_msg_reply_info,
+            ZXDH_ST_SZ_BYTES(msg_reply_info));
         if (ret) {
             PMD_DRV_LOG(ERR,
                 "Failed msg: port 0x%x msg type ZXDH_PLCR_CAR_QUEUE_CFG_SET ",
diff --git a/drivers/net/zxdh/zxdh_mtr.h b/drivers/net/zxdh/zxdh_mtr.h
index 749ceb7479..3efcb6b591 100644
--- a/drivers/net/zxdh/zxdh_mtr.h
+++ b/drivers/net/zxdh/zxdh_mtr.h
@@ -93,6 +93,11 @@ struct zxdh_mtr_stats {
     uint64_t n_bytes_dropped;
 };
 
+struct zxdh_ifc_mtr_stats_bits {
+    uint8_t n_pkts_dropped[0x40];
+    uint8_t n_bytes_dropped[0x40];
+};
+
 struct zxdh_hw_mtr_stats {
     uint32_t n_pkts_dropped_hi;
     uint32_t n_pkts_dropped_lo;
diff --git a/drivers/net/zxdh/zxdh_np.c b/drivers/net/zxdh/zxdh_np.c
index ab8b3ae688..66902e7e92 100644
--- a/drivers/net/zxdh/zxdh_np.c
+++ b/drivers/net/zxdh/zxdh_np.c
@@ -2350,6 +2350,8 @@ zxdh_np_dev_add(uint32_t  dev_id, ZXDH_DEV_TYPE_E dev_type,
 
     rte_spinlock_init(&p_dev_info->dtb_spinlock.spinlock);
 
+    rte_spinlock_init(&p_dev_info->smmu0_spinlock.spinlock);
+
     for (i = 0; i < ZXDH_DTB_QUEUE_NUM_MAX; i++)
         rte_spinlock_init(&p_dev_info->dtb_queue_spinlock[i].spinlock);
 
@@ -3391,6 +3393,32 @@ zxdh_np_reg_read(uint32_t dev_id, uint32_t reg_no,
     return rc;
 }
 
+static uint32_t
+zxdh_np_reg_read32(uint32_t dev_id, uint32_t reg_no,
+    uint32_t m_offset, uint32_t n_offset, uint32_t *p_data)
+{
+    uint32_t rc = 0;
+    uint32_t addr = 0;
+    ZXDH_REG_T *p_reg_info = &g_dpp_reg_info[reg_no];
+    uint32_t p_buff[ZXDH_REG_DATA_MAX] = {0};
+    uint32_t reg_real_no = p_reg_info->reg_no;
+    uint32_t reg_type = p_reg_info->flags;
+    uint32_t reg_module = p_reg_info->module_no;
+
+    addr = zxdh_np_reg_get_reg_addr(reg_no, m_offset, n_offset);
+
+    if (reg_module == DTB4K) {
+        rc = p_reg_info->p_read_fun(dev_id, addr, p_data);
+        ZXDH_COMM_CHECK_DEV_RC(dev_id, rc, "p_reg_info->p_read_fun");
+    } else {
+        rc = zxdh_np_agent_channel_reg_read(dev_id, reg_type, reg_real_no, 4, addr, p_buff);
+        ZXDH_COMM_CHECK_DEV_RC(dev_id, rc, "zxdh_np_agent_channel_reg_read");
+        *p_data = p_buff[0];
+    }
+
+    return rc;
+}
+
 static uint32_t
 zxdh_np_dtb_queue_vm_info_get(uint32_t dev_id,
         uint32_t queue_id,
@@ -10542,9 +10570,9 @@ zxdh_np_se_done_status_check(uint32_t dev_id, uint32_t reg_no, uint32_t pos)
     uint32_t done_flag = 0;
 
     while (!done_flag) {
-        rc = zxdh_np_reg_read(dev_id, reg_no, 0, 0, &data);
+        rc = zxdh_np_reg_read32(dev_id, reg_no, 0, 0, &data);
         if (rc != 0) {
-            PMD_DRV_LOG(ERR, "reg_read fail!");
+            PMD_DRV_LOG(ERR, "reg_read32 fail!");
             return rc;
         }
 
@@ -10577,10 +10605,17 @@ zxdh_np_se_smmu0_ind_read(uint32_t dev_id,
     uint32_t temp_data[4] = {0};
     uint32_t *p_temp_data = NULL;
     ZXDH_SMMU0_SMMU0_CPU_IND_CMD_T cpu_ind_cmd = {0};
+    ZXDH_SPINLOCK_T *p_ind_spinlock = NULL;
+
+    rc = zxdh_np_dev_opr_spinlock_get(dev_id, ZXDH_DEV_SPINLOCK_T_SMMU0, &p_ind_spinlock);
+    ZXDH_COMM_CHECK_DEV_RC(dev_id, rc, "zxdh_np_dev_opr_spinlock_get");
+
+    rte_spinlock_lock(&p_ind_spinlock->spinlock);
 
     rc = zxdh_np_se_done_status_check(dev_id, ZXDH_SMMU0_SMMU0_WR_ARB_CPU_RDYR, 0);
     if (rc != ZXDH_OK) {
         PMD_DRV_LOG(ERR, "se done status check failed, rc=0x%x.", rc);
+        rte_spinlock_unlock(&p_ind_spinlock->spinlock);
         return ZXDH_ERR;
     }
 
@@ -10592,11 +10627,13 @@ zxdh_np_se_smmu0_ind_read(uint32_t dev_id,
         switch (rd_mode) {
         case ZXDH_ERAM128_OPR_128b:
             if ((0xFFFFFFFF - (base_addr)) < (index)) {
+                rte_spinlock_unlock(&p_ind_spinlock->spinlock);
                 PMD_DRV_LOG(ERR, "index 0x%x is invalid!", index);
                 return ZXDH_PAR_CHK_INVALID_INDEX;
             }
             if (base_addr + index > ZXDH_SE_SMMU0_ERAM_ADDR_NUM_TOTAL - 1) {
                 PMD_DRV_LOG(ERR, "index out of range!");
+                rte_spinlock_unlock(&p_ind_spinlock->spinlock);
                 return ZXDH_ERR;
             }
             row_index = (index << 7) & ZXDH_ERAM128_BADDR_MASK;
@@ -10604,6 +10641,7 @@ zxdh_np_se_smmu0_ind_read(uint32_t dev_id,
         case ZXDH_ERAM128_OPR_64b:
             if ((base_addr + (index >> 1)) > ZXDH_SE_SMMU0_ERAM_ADDR_NUM_TOTAL - 1) {
                 PMD_DRV_LOG(ERR, "index out of range!");
+                rte_spinlock_unlock(&p_ind_spinlock->spinlock);
                 return ZXDH_ERR;
             }
             row_index = (index << 6) & ZXDH_ERAM128_BADDR_MASK;
@@ -10612,6 +10650,7 @@ zxdh_np_se_smmu0_ind_read(uint32_t dev_id,
         case ZXDH_ERAM128_OPR_32b:
             if ((base_addr + (index >> 2)) > ZXDH_SE_SMMU0_ERAM_ADDR_NUM_TOTAL - 1) {
                 PMD_DRV_LOG(ERR, "index out of range!");
+                rte_spinlock_unlock(&p_ind_spinlock->spinlock);
                 return ZXDH_ERR;
             }
             row_index = (index << 5) & ZXDH_ERAM128_BADDR_MASK;
@@ -10620,6 +10659,7 @@ zxdh_np_se_smmu0_ind_read(uint32_t dev_id,
         case ZXDH_ERAM128_OPR_1b:
             if ((base_addr + (index >> 7)) > ZXDH_SE_SMMU0_ERAM_ADDR_NUM_TOTAL - 1) {
                 PMD_DRV_LOG(ERR, "index out of range!");
+                rte_spinlock_unlock(&p_ind_spinlock->spinlock);
                 return ZXDH_ERR;
             }
             row_index = index & ZXDH_ERAM128_BADDR_MASK;
@@ -10638,10 +10678,12 @@ zxdh_np_se_smmu0_ind_read(uint32_t dev_id,
         case ZXDH_ERAM128_OPR_128b:
             if ((0xFFFFFFFF - (base_addr)) < (index)) {
                 PMD_DRV_LOG(ERR, "index 0x%x is invalid!", index);
+                rte_spinlock_unlock(&p_ind_spinlock->spinlock);
                 return ZXDH_PAR_CHK_INVALID_INDEX;
             }
             if (base_addr + index > ZXDH_SE_SMMU0_ERAM_ADDR_NUM_TOTAL - 1) {
                 PMD_DRV_LOG(ERR, "index out of range!");
+                rte_spinlock_unlock(&p_ind_spinlock->spinlock);
                 return ZXDH_ERR;
             }
             row_index = (index << 7);
@@ -10650,6 +10692,7 @@ zxdh_np_se_smmu0_ind_read(uint32_t dev_id,
         case ZXDH_ERAM128_OPR_64b:
             if ((base_addr + (index >> 1)) > ZXDH_SE_SMMU0_ERAM_ADDR_NUM_TOTAL - 1) {
                 PMD_DRV_LOG(ERR, "index out of range!");
+                rte_spinlock_unlock(&p_ind_spinlock->spinlock);
                 return ZXDH_ERR;
             }
             row_index = (index << 6);
@@ -10658,6 +10701,7 @@ zxdh_np_se_smmu0_ind_read(uint32_t dev_id,
         case ZXDH_ERAM128_OPR_32b:
             if ((base_addr + (index >> 2)) > ZXDH_SE_SMMU0_ERAM_ADDR_NUM_TOTAL - 1) {
                 PMD_DRV_LOG(ERR, "index out of range!");
+                rte_spinlock_unlock(&p_ind_spinlock->spinlock);
                 return ZXDH_ERR;
             }
             row_index = (index << 5);
@@ -10665,7 +10709,8 @@ zxdh_np_se_smmu0_ind_read(uint32_t dev_id,
             break;
         case ZXDH_ERAM128_OPR_1b:
             PMD_DRV_LOG(ERR, "rd_clr_mode[%u] or rd_mode[%u] error!",
-            rd_clr_mode, rd_mode);
+                rd_clr_mode, rd_mode);
+            rte_spinlock_unlock(&p_ind_spinlock->spinlock);
             return ZXDH_ERR;
         default:
             break;
@@ -10680,12 +10725,14 @@ zxdh_np_se_smmu0_ind_read(uint32_t dev_id,
             &cpu_ind_cmd);
     if (rc != ZXDH_OK) {
         PMD_DRV_LOG(ERR, "zxdh_np_reg_write failed, rc=0x%x.", rc);
+        rte_spinlock_unlock(&p_ind_spinlock->spinlock);
         return ZXDH_ERR;
     }
 
     rc = zxdh_np_se_done_status_check(dev_id, ZXDH_SMMU0_SMMU0_CPU_IND_RD_DONER, 0);
     if (rc != ZXDH_OK) {
         PMD_DRV_LOG(ERR, "se done status check failed, rc=0x%x.", rc);
+        rte_spinlock_unlock(&p_ind_spinlock->spinlock);
         return ZXDH_ERR;
     }
 
@@ -10698,6 +10745,7 @@ zxdh_np_se_smmu0_ind_read(uint32_t dev_id,
             p_temp_data + 3 - i);
         if (rc != ZXDH_OK) {
             PMD_DRV_LOG(ERR, "zxdh_np_reg_write failed, rc=0x%x.", rc);
+            rte_spinlock_unlock(&p_ind_spinlock->spinlock);
             return ZXDH_ERR;
         }
     }
@@ -10736,6 +10784,8 @@ zxdh_np_se_smmu0_ind_read(uint32_t dev_id,
         }
     }
 
+    rte_spinlock_unlock(&p_ind_spinlock->spinlock);
+
     return rc;
 }
 
@@ -10789,7 +10839,7 @@ zxdh_np_agent_channel_plcr_sync_send(uint32_t dev_id, ZXDH_AGENT_CHANNEL_PLCR_MS
 {
     uint32_t ret = 0;
     ZXDH_AGENT_CHANNEL_MSG_T agent_msg = {
-        .msg = (void *)&p_msg,
+        .msg = (void *)p_msg,
         .msg_len = sizeof(ZXDH_AGENT_CHANNEL_PLCR_MSG_T),
     };
 
@@ -11012,7 +11062,8 @@ zxdh_np_stat_carc_queue_cfg_set(uint32_t dev_id,
 }
 
 uint32_t
-zxdh_np_car_profile_id_add(uint32_t vport_id,
+zxdh_np_car_profile_id_add(uint32_t dev_id,
+        uint32_t vport_id,
         uint32_t flags,
         uint64_t *p_profile_id)
 {
@@ -11027,7 +11078,7 @@ zxdh_np_car_profile_id_add(uint32_t vport_id,
         PMD_DRV_LOG(ERR, "profile_id point null!");
         return ZXDH_PAR_CHK_POINT_NULL;
     }
-    ret = zxdh_np_agent_channel_plcr_profileid_request(0, vport_id, flags, profile_id);
+    ret = zxdh_np_agent_channel_plcr_profileid_request(dev_id, vport_id, flags, profile_id);
 
     profile_id_h = *(profile_id + 1);
     profile_id_l = *profile_id;
@@ -11045,14 +11096,14 @@ zxdh_np_car_profile_id_add(uint32_t vport_id,
 }
 
 uint32_t
-zxdh_np_car_profile_cfg_set(uint32_t vport_id __rte_unused,
+zxdh_np_car_profile_cfg_set(uint32_t dev_id,
+        uint32_t vport_id __rte_unused,
         uint32_t car_type,
         uint32_t pkt_sign,
         uint32_t profile_id,
         void *p_car_profile_cfg)
 {
     uint32_t ret = 0;
-    uint32_t dev_id = 0;
 
     ret = zxdh_np_agent_channel_plcr_car_rate(dev_id, car_type,
         pkt_sign, profile_id, p_car_profile_cfg);
@@ -11065,11 +11116,10 @@ zxdh_np_car_profile_cfg_set(uint32_t vport_id __rte_unused,
 }
 
 uint32_t
-zxdh_np_car_profile_id_delete(uint32_t vport_id,
+zxdh_np_car_profile_id_delete(uint32_t dev_id, uint32_t vport_id,
     uint32_t flags, uint64_t profile_id)
 {
     uint32_t ret = 0;
-    uint32_t dev_id = 0;
     uint32_t profileid = profile_id & 0xFFFF;
 
     ret = zxdh_np_agent_channel_plcr_profileid_release(dev_id, vport_id, flags, profileid);
diff --git a/drivers/net/zxdh/zxdh_np.h b/drivers/net/zxdh/zxdh_np.h
index b1d8b1aef8..1b8f17474d 100644
--- a/drivers/net/zxdh/zxdh_np.h
+++ b/drivers/net/zxdh/zxdh_np.h
@@ -1934,15 +1934,17 @@ uint32_t zxdh_np_stat_ppu_cnt_get_ex(uint32_t dev_id,
             uint32_t clr_mode,
             uint32_t *p_data);
 uint32_t
-zxdh_np_car_profile_id_add(uint32_t vport_id,
+zxdh_np_car_profile_id_add(uint32_t dev_id,
+            uint32_t vport_id,
             uint32_t flags,
             uint64_t *p_profile_id);
-uint32_t zxdh_np_car_profile_cfg_set(uint32_t vport_id,
+uint32_t zxdh_np_car_profile_cfg_set(uint32_t dev_id,
+            uint32_t vport_id,
             uint32_t car_type,
             uint32_t pkt_sign,
             uint32_t profile_id,
             void *p_car_profile_cfg);
-uint32_t zxdh_np_car_profile_id_delete(uint32_t vport_id,
+uint32_t zxdh_np_car_profile_id_delete(uint32_t dev_id, uint32_t vport_id,
             uint32_t flags, uint64_t profile_id);
 uint32_t zxdh_np_stat_car_queue_cfg_set(uint32_t dev_id,
             uint32_t car_type,
diff --git a/drivers/net/zxdh/zxdh_pci.c b/drivers/net/zxdh/zxdh_pci.c
index 3d1a3ff0dd..4ff0f065df 100644
--- a/drivers/net/zxdh/zxdh_pci.c
+++ b/drivers/net/zxdh/zxdh_pci.c
@@ -27,6 +27,23 @@
         1ULL << ZXDH_F_NOTIFICATION_DATA | \
         1ULL << ZXDH_NET_F_MAC)
 
+#define ZXDH_PMD_DEFAULT_HOST_FEATURES   \
+    (1ULL << ZXDH_NET_F_MRG_RXBUF | \
+     1ULL << ZXDH_NET_F_STATUS    | \
+     1ULL << ZXDH_NET_F_MQ        | \
+     1ULL << ZXDH_F_ANY_LAYOUT    | \
+     1ULL << ZXDH_F_VERSION_1   | \
+     1ULL << ZXDH_F_RING_PACKED | \
+     1ULL << ZXDH_F_IN_ORDER    | \
+     1ULL << ZXDH_F_NOTIFICATION_DATA |\
+     1ULL << ZXDH_NET_F_MAC | \
+     1ULL << ZXDH_NET_F_CSUM |\
+     1ULL << ZXDH_NET_F_GUEST_CSUM |\
+     1ULL << ZXDH_NET_F_GUEST_TSO4 |\
+     1ULL << ZXDH_NET_F_GUEST_TSO6 |\
+     1ULL << ZXDH_NET_F_HOST_TSO4 |\
+     1ULL << ZXDH_NET_F_HOST_TSO6)
+
 static void
 zxdh_read_dev_config(struct zxdh_hw *hw, size_t offset,
         void *dst, int32_t length)
@@ -391,13 +408,18 @@ zxdh_pci_read_dev_config(struct zxdh_hw *hw, size_t offset, void *dst, int32_t l
     ZXDH_VTPCI_OPS(hw)->read_dev_cfg(hw, offset, dst, length);
 }
 
+void zxdh_pci_write_dev_config(struct zxdh_hw *hw, size_t offset, const void *src, int32_t length)
+{
+    ZXDH_VTPCI_OPS(hw)->write_dev_cfg(hw, offset, src, length);
+}
+
 void
 zxdh_get_pci_dev_config(struct zxdh_hw *hw)
 {
     uint64_t guest_features = 0;
     uint64_t nego_features = 0;
 
-    hw->host_features = zxdh_pci_get_features(hw);
+    hw->host_features = ZXDH_PMD_DEFAULT_HOST_FEATURES;
 
     guest_features = (uint64_t)ZXDH_PMD_DEFAULT_GUEST_FEATURES;
     nego_features = guest_features & hw->host_features;
diff --git a/drivers/net/zxdh/zxdh_pci.h b/drivers/net/zxdh/zxdh_pci.h
index 9b8bef6c09..a1834f6615 100644
--- a/drivers/net/zxdh/zxdh_pci.h
+++ b/drivers/net/zxdh/zxdh_pci.h
@@ -162,7 +162,7 @@ void zxdh_pci_read_dev_config(struct zxdh_hw *hw, size_t offset,
 
 int32_t zxdh_read_pci_caps(struct rte_pci_device *dev, struct zxdh_hw *hw);
 void zxdh_get_pci_dev_config(struct zxdh_hw *hw);
-
+void zxdh_pci_write_dev_config(struct zxdh_hw *hw, size_t offset, const void *src, int32_t length);
 uint16_t zxdh_pci_get_features(struct zxdh_hw *hw);
 enum zxdh_msix_status zxdh_pci_msix_detect(struct rte_pci_device *dev);
 uint8_t zxdh_pci_isr(struct zxdh_hw *hw);
diff --git a/drivers/net/zxdh/zxdh_rxtx.c b/drivers/net/zxdh/zxdh_rxtx.c
index 1921a23f25..9bb57d4f84 100644
--- a/drivers/net/zxdh/zxdh_rxtx.c
+++ b/drivers/net/zxdh/zxdh_rxtx.c
@@ -760,7 +760,6 @@ zxdh_rx_update_mbuf(struct rte_mbuf *m, struct zxdh_net_hdr_ul *hdr)
         idx = (pkt_type_inner >> 4)  & 0xF;
         m->packet_type |= zxdh_inner_l4_type[idx];
     }
-
 }
 
 static void zxdh_discard_rxbuf(struct zxdh_virtqueue *vq, struct rte_mbuf *m)
@@ -818,9 +817,14 @@ zxdh_recv_pkts_packed(void *rx_queue, struct rte_mbuf **rx_pkts,
         seg_num  = header->type_hdr.num_buffers;
 
         /* Private queue only handle type hdr */
-        hdr_size = ZXDH_TYPE_HDR_SIZE;
-        rxm->pkt_len = ((header->type_hdr.port & 0x7f) << 8) +
-                            header->type_hdr.pd_len;
+        hdr_size = header->type_hdr.pd_len << 1;
+        if (unlikely(hdr_size > lens[i] || hdr_size < ZXDH_TYPE_HDR_SIZE)) {
+            PMD_RX_LOG(ERR, "hdr_size:%u is invalid", hdr_size);
+            rte_pktmbuf_free(rxm);
+            rxvq->stats.errors++;
+            rxvq->stats.invalid_hdr_len_err++;
+            continue;
+        }
         rxm->data_off += hdr_size;
         rxm->nb_segs = seg_num;
         rxm->ol_flags = 0;
diff --git a/drivers/net/zxdh/zxdh_tables.c b/drivers/net/zxdh/zxdh_tables.c
index 253d9ce438..ab91d51948 100644
--- a/drivers/net/zxdh/zxdh_tables.c
+++ b/drivers/net/zxdh/zxdh_tables.c
@@ -8,14 +8,7 @@
 #include "zxdh_tables.h"
 #include "zxdh_logs.h"
 
-#define ZXDH_SDT_VPORT_ATT_TABLE          1
-#define ZXDH_SDT_PANEL_ATT_TABLE          2
-#define ZXDH_SDT_RSS_ATT_TABLE            3
-#define ZXDH_SDT_VLAN_ATT_TABLE           4
-#define ZXDH_SDT_BROCAST_ATT_TABLE        6
-#define ZXDH_SDT_UNICAST_ATT_TABLE        10
-#define ZXDH_SDT_MULTICAST_ATT_TABLE      11
-#define ZXDH_SDT_PORT_VLAN_ATT_TABLE      16
+
 
 #define ZXDH_MAC_HASH_INDEX_BASE          64
 #define ZXDH_MAC_HASH_INDEX(index)        (ZXDH_MAC_HASH_INDEX_BASE + (index))
@@ -40,15 +33,16 @@ zxdh_set_port_attr(struct zxdh_hw *hw, uint16_t vport, struct zxdh_port_attr_tab
 {
     struct zxdh_dtb_shared_data *dtb_data = &hw->dev_sd->dtb_sd;
     union zxdh_virport_num vport_num = (union zxdh_virport_num)vport;
+    uint16_t vfid = zxdh_vport_to_vfid(vport_num);
     int ret = 0;
 
-    ZXDH_DTB_ERAM_ENTRY_INFO_T entry = {vport_num.vfid, (uint32_t *)port_attr};
+    ZXDH_DTB_ERAM_ENTRY_INFO_T entry = {vfid, (uint32_t *)port_attr};
     ZXDH_DTB_USER_ENTRY_T user_entry_write = {ZXDH_SDT_VPORT_ATT_TABLE, (void *)&entry};
 
     ret = zxdh_np_dtb_table_entry_write(hw->slot_id,
                 dtb_data->queueid, 1, &user_entry_write);
     if (ret != 0)
-        PMD_DRV_LOG(ERR, "write vport_att failed vfid:%d failed", vport_num.vfid);
+        PMD_DRV_LOG(ERR, "write vport_att failed vfid:%d failed", vfid);
 
     return ret;
 }
@@ -72,6 +66,7 @@ zxdh_port_attr_init(struct rte_eth_dev *dev)
         port_attr.mtu = dev->data->mtu;
         port_attr.mtu_enable = 1;
         port_attr.is_up = 0;
+        port_attr.hash_search_index = hw->hash_search_index;
         if (!port_attr.rss_enable)
             port_attr.port_base_qid = 0;
 
@@ -144,6 +139,7 @@ int zxdh_panel_table_init(struct rte_eth_dev *dev)
     panel.pf_vfid = zxdh_vport_to_vfid(hw->vport);
     panel.mtu_enable = 1;
     panel.mtu = dev->data->mtu;
+    panel.port_vfid_1588 = panel.pf_vfid;
 
     ZXDH_DTB_ERAM_ENTRY_INFO_T panel_entry = {
         .index = hw->phyport,
@@ -212,13 +208,14 @@ zxdh_get_port_attr(struct zxdh_hw *hw, uint16_t vport, struct zxdh_port_attr_tab
 {
     struct zxdh_dtb_shared_data *dtb_data = &hw->dev_sd->dtb_sd;
     union zxdh_virport_num vport_num = (union zxdh_virport_num)vport;
-    ZXDH_DTB_ERAM_ENTRY_INFO_T entry = {vport_num.vfid, (uint32_t *)port_attr};
+    uint16_t vfid = zxdh_vport_to_vfid(vport_num);
+    ZXDH_DTB_ERAM_ENTRY_INFO_T entry = {vfid, (uint32_t *)port_attr};
     ZXDH_DTB_USER_ENTRY_T user_entry_get = {ZXDH_SDT_VPORT_ATT_TABLE, &entry};
     int ret;
 
     ret = zxdh_np_dtb_table_entry_get(hw->slot_id, dtb_data->queueid, &user_entry_get, 1);
     if (ret != 0)
-        PMD_DRV_LOG(ERR, "get port_attr vfid:%d failed, ret:%d", vport_num.vfid, ret);
+        PMD_DRV_LOG(ERR, "get port_attr vfid:%d failed, ret:%d", vfid, ret);
 
     return ret;
 }
@@ -229,7 +226,8 @@ zxdh_delete_port_attr(struct zxdh_hw *hw, uint16_t vport,
 {
     struct zxdh_dtb_shared_data *dtb_data = &hw->dev_sd->dtb_sd;
     union zxdh_virport_num vport_num = (union zxdh_virport_num)vport;
-    ZXDH_DTB_ERAM_ENTRY_INFO_T entry = {vport_num.vfid, (uint32_t *)port_attr};
+    uint16_t vfid = zxdh_vport_to_vfid(vport_num);
+    ZXDH_DTB_ERAM_ENTRY_INFO_T entry = {vfid, (uint32_t *)port_attr};
     ZXDH_DTB_USER_ENTRY_T user_entry = {
         .sdt_no = ZXDH_SDT_VPORT_ATT_TABLE,
         .p_entry_data = (void *)&entry
@@ -247,9 +245,9 @@ zxdh_add_mac_table(struct zxdh_hw *hw, uint16_t vport, struct rte_ether_addr *ad
     struct zxdh_mac_unicast_table unicast_table = {0};
     struct zxdh_mac_multicast_table multicast_table = {0};
     union zxdh_virport_num vport_num = (union zxdh_virport_num)vport;
+    uint16_t vfid = zxdh_vport_to_vfid(vport_num);
     uint32_t ret;
     uint16_t group_id = 0;
-    uint16_t vfid = vport_num.vfid;
 
     if (rte_is_unicast_ether_addr(addr)) {
         rte_memcpy(unicast_table.key.dmac_addr, addr, sizeof(struct rte_ether_addr));
@@ -351,15 +349,17 @@ zxdh_del_mac_table(struct zxdh_hw *hw, uint16_t vport, struct rte_ether_addr *ad
     struct zxdh_mac_unicast_table unicast_table = {0};
     struct zxdh_mac_multicast_table multicast_table = {0};
     union zxdh_virport_num vport_num = (union zxdh_virport_num)vport;
-    uint32_t ret, del_flag = 0;
-    uint16_t group_id = 0;
     union zxdh_virport_num port = (union zxdh_virport_num)vport;
     uint16_t vfid = zxdh_vport_to_vfid(port);
+    uint32_t ret, del_flag = 0;
+    uint16_t group_id = 0;
 
     if (rte_is_unicast_ether_addr(addr)) {
         rte_memcpy(unicast_table.key.dmac_addr, addr, sizeof(struct rte_ether_addr));
         unicast_table.key.sriov_vlan_id = srv_vlanid;
         unicast_table.key.sriov_vlan_tpid = srv_tpid;
+        unicast_table.entry.hit_flag = 0;
+        unicast_table.entry.vfid = rte_cpu_to_be_16(vfid & 0x7ff);
 
         ZXDH_DTB_HASH_ENTRY_INFO_T dtb_hash_entry = {
             .p_actu_key = (uint8_t *)&unicast_table.key,
@@ -800,6 +800,7 @@ zxdh_rss_table_set(struct zxdh_hw *hw, uint16_t vport, struct zxdh_rss_reta *rss
     struct zxdh_dtb_shared_data *dtb_data = &hw->dev_sd->dtb_sd;
     struct zxdh_rss_to_vqid_table rss_vqid = {0};
     union zxdh_virport_num vport_num = (union zxdh_virport_num)vport;
+    uint16_t vfid = zxdh_vport_to_vfid(vport_num);
     int ret = 0;
 
     for (uint16_t i = 0; i < RTE_ETH_RSS_RETA_SIZE_256 / 8; i++) {
@@ -820,7 +821,7 @@ zxdh_rss_table_set(struct zxdh_hw *hw, uint16_t vport, struct zxdh_rss_reta *rss
             rss_vqid.vqm_qid[0] |= 0x8000;
 #endif
         ZXDH_DTB_ERAM_ENTRY_INFO_T entry = {
-            .index = vport_num.vfid * 32 + i,
+            .index = vfid * 32 + i,
             .p_data = (uint32_t *)&rss_vqid
         };
         ZXDH_DTB_USER_ENTRY_T user_entry_write = {
@@ -830,7 +831,7 @@ zxdh_rss_table_set(struct zxdh_hw *hw, uint16_t vport, struct zxdh_rss_reta *rss
         ret = zxdh_np_dtb_table_entry_write(hw->slot_id,
                     dtb_data->queueid, 1, &user_entry_write);
         if (ret != 0) {
-            PMD_DRV_LOG(ERR, "write rss base qid failed vfid:%d", vport_num.vfid);
+            PMD_DRV_LOG(ERR, "write rss base qid failed vfid:%d", vfid);
             return ret;
         }
     }
@@ -843,16 +844,17 @@ zxdh_rss_table_get(struct zxdh_hw *hw, uint16_t vport, struct zxdh_rss_reta *rss
     struct zxdh_dtb_shared_data *dtb_data = &hw->dev_sd->dtb_sd;
     struct zxdh_rss_to_vqid_table rss_vqid = {0};
     union zxdh_virport_num vport_num = (union zxdh_virport_num)vport;
+    uint16_t vfid = zxdh_vport_to_vfid(vport_num);
     int ret = 0;
 
     for (uint16_t i = 0; i < RTE_ETH_RSS_RETA_SIZE_256 / 8; i++) {
-        ZXDH_DTB_ERAM_ENTRY_INFO_T entry = {vport_num.vfid * 32 + i, (uint32_t *)&rss_vqid};
+        ZXDH_DTB_ERAM_ENTRY_INFO_T entry = {vfid * 32 + i, (uint32_t *)&rss_vqid};
         ZXDH_DTB_USER_ENTRY_T user_entry = {ZXDH_SDT_RSS_ATT_TABLE, &entry};
 
         ret = zxdh_np_dtb_table_entry_get(hw->slot_id,
                     dtb_data->queueid, &user_entry, 1);
         if (ret != 0) {
-            PMD_DRV_LOG(ERR, "get rss tbl failed, vfid:%d", vport_num.vfid);
+            PMD_DRV_LOG(ERR, "get rss tbl failed, vfid:%d", vfid);
             return -1;
         }
 
diff --git a/drivers/net/zxdh/zxdh_tables.h b/drivers/net/zxdh/zxdh_tables.h
index 2f2ada3a9f..cb34e38be8 100644
--- a/drivers/net/zxdh/zxdh_tables.h
+++ b/drivers/net/zxdh/zxdh_tables.h
@@ -9,7 +9,13 @@
 
 /* eram */
 #define ZXDH_SDT_VPORT_ATT_TABLE          1
-
+#define ZXDH_SDT_PANEL_ATT_TABLE          2
+#define ZXDH_SDT_RSS_ATT_TABLE            3
+#define ZXDH_SDT_VLAN_ATT_TABLE           4
+#define ZXDH_SDT_BROCAST_ATT_TABLE        6
+#define ZXDH_SDT_UNICAST_ATT_TABLE        10
+#define ZXDH_SDT_MULTICAST_ATT_TABLE      11
+#define ZXDH_SDT_PORT_VLAN_ATT_TABLE      16
 /* hash */
 #define ZXDH_SDT_L2_ENTRY_TABLE0          64
 #define ZXDH_SDT_L2_ENTRY_TABLE1          65
@@ -80,8 +86,6 @@
 #define ZXDH_MTR_STATS_EGRESS_BASE           0x7481
 #define ZXDH_MTR_STATS_INGRESS_BASE          0x7C81
 
-extern struct zxdh_dtb_shared_data g_dtb_data;
-
 struct zxdh_port_vlan_table {
 #if RTE_BYTE_ORDER == RTE_LITTLE_ENDIAN
     uint16_t business_vlan_tpid:16;
@@ -233,19 +237,51 @@ struct zxdh_port_attr_table {
 };
 
 struct zxdh_panel_table {
-    uint16_t port_vfid_1588 : 11,
-             rsv2           : 5;
-    uint16_t pf_vfid        : 11,
-             rsv1           : 1,
-             enable_1588_tc : 2,
-             trust_mode     : 1,
-             hit_flag       : 1;
-    uint32_t mtu            : 16,
-             mtu_enable     : 1,
-             rsv            : 3,
-             tm_base_queue  : 12;
-    uint32_t rsv_1;
-    uint32_t rsv_2;
+#if RTE_BYTE_ORDER == RTE_LITTLE_ENDIAN
+    uint16_t port_vfid_1588   : 11,
+                rsv2             : 5;
+    uint16_t rsv1             : 11,
+             tm_shape_enable  : 1,
+             enable_1588_tc   : 2,
+             trust_mode       : 1,
+             hit_flag         : 1;
+    uint16_t mtu              : 16;
+    uint16_t mtu_enable       : 1,
+             rsv              : 3,
+             tm_base_queue    : 12;
+    uint16_t lacp_pf_qid      : 12,
+                rsv5             : 4;
+    uint16_t lacp_pf_vfid     : 11,
+                rsv6             : 2,
+                member_port_up   : 1,
+                bond_link_up     : 1,
+                hw_bond_enable   : 1;
+    uint16_t rsv3             : 16;
+    uint16_t pf_vfid          : 11,
+                rsv4             : 5;
+#else
+    uint16_t rsv1             : 11,
+                tm_shape_enable  : 1,
+                enable_1588_tc   : 2,
+                trust_mode       : 1,
+                hit_flag         : 1;
+    uint16_t port_vfid_1588   : 11,
+                rsv2             : 5;
+    uint16_t mtu_enable       : 1,
+                rsv              : 3,
+                tm_base_queue    : 12;
+    uint16_t mtu              : 16;
+    uint16_t lacp_pf_vfid     : 11,
+                rsv6             : 2,
+                member_port_up   : 1,
+                bond_link_up     : 1,
+                hw_bond_enable   : 1;
+    uint16_t lacp_pf_qid      : 12,
+                rsv5             : 4;
+    uint16_t pf_vfid          : 11,
+                rsv4             : 5;
+    uint16_t rsv3             : 16;
+#endif
 }; /* 16B */
 
 struct zxdh_mac_unicast_key {
-- 
2.27.0