dev.dpdk.org archive mirror
 help / color / mirror / Atom feed
From: "Wei Hu (Xavier)" <xavier.huwei@huawei.com>
To: "Gavin Hu (Arm Technology China)" <Gavin.Hu@arm.com>,
	"dev@dpdk.org" <dev@dpdk.org>
Cc: "linuxarm@huawei.com" <linuxarm@huawei.com>,
	"xavier_huwei@163.com" <xavier_huwei@163.com>,
	"liudongdong3@huawei.com" <liudongdong3@huawei.com>,
	"forest.zhouchang@huawei.com" <forest.zhouchang@huawei.com>
Subject: Re: [dpdk-dev] [PATCH 02/22] net/hns3: add some definitions for data structure and macro
Date: Thu, 5 Sep 2019 14:01:17 +0800	[thread overview]
Message-ID: <18059d08-a9d2-3eca-82c7-642fc4a49869@huawei.com> (raw)
In-Reply-To: <VI1PR08MB53765807E8E2C65F5E335D9C8FBD0@VI1PR08MB5376.eurprd08.prod.outlook.com>

Hi, Gavin Hu


On 2019/8/30 16:25, Gavin Hu (Arm Technology China) wrote:
> Hi Xavier,
>
>> -----Original Message-----
>> From: dev <dev-bounces@dpdk.org> On Behalf Of Wei Hu (Xavier)
>> Sent: Friday, August 23, 2019 9:47 PM
>> To: dev@dpdk.org
>> Cc: linuxarm@huawei.com; xavier_huwei@163.com;
>> liudongdong3@huawei.com; forest.zhouchang@huawei.com
>> Subject: [dpdk-dev] [PATCH 02/22] net/hns3: add some definitions for data
>> structure and macro
>>
>> This patch adds some data structure definitions, macro definitions and
>> inline functions for hns3 PMD drivers.
>>
>> Signed-off-by: Wei Hu (Xavier) <xavier.huwei@huawei.com>
>> Signed-off-by: Chunsong Feng <fengchunsong@huawei.com>
>> Signed-off-by: Min Hu (Connor) <humin29@huawei.com>
>> Signed-off-by: Hao Chen <chenhao164@huawei.com>
>> Signed-off-by: Huisong Li <lihuisong@huawei.com>
>> ---
>>  drivers/net/hns3/hns3_ethdev.h | 609
>> +++++++++++++++++++++++++++++++++++++++++
>>  1 file changed, 609 insertions(+)
>>  create mode 100644 drivers/net/hns3/hns3_ethdev.h
>>
>> diff --git a/drivers/net/hns3/hns3_ethdev.h
>> b/drivers/net/hns3/hns3_ethdev.h
>> new file mode 100644
>> index 0000000..bfb54f2
>> --- /dev/null
>> +++ b/drivers/net/hns3/hns3_ethdev.h
>> @@ -0,0 +1,609 @@
>> +/* SPDX-License-Identifier: BSD-3-Clause
>> + * Copyright(c) 2018-2019 Hisilicon Limited.
>> + */
>> +
>> +#ifndef _HNS3_ETHDEV_H_
>> +#define _HNS3_ETHDEV_H_
>> +
>> +#include <sys/time.h>
>> +#include <rte_alarm.h>
>> +
>> +/* Vendor ID */
>> +#define PCI_VENDOR_ID_HUAWEI                 0x19e5
>> +
>> +/* Device IDs */
>> +#define HNS3_DEV_ID_GE                               0xA220
>> +#define HNS3_DEV_ID_25GE                     0xA221
>> +#define HNS3_DEV_ID_25GE_RDMA                        0xA222
>> +#define HNS3_DEV_ID_50GE_RDMA                        0xA224
>> +#define HNS3_DEV_ID_100G_RDMA_MACSEC         0xA226
>> +#define HNS3_DEV_ID_100G_VF                  0xA22E
>> +#define HNS3_DEV_ID_100G_RDMA_PFC_VF         0xA22F
>> +
>> +#define HNS3_UC_MACADDR_NUM          96
>> +#define HNS3_MC_MACADDR_NUM          128
>> +
>> +#define HNS3_MAX_BD_SIZE             65535
>> +#define HNS3_MAX_TX_BD_PER_PKT               8
>> +#define HNS3_MAX_FRAME_LEN           9728
>> +#define HNS3_MIN_FRAME_LEN           64
>> +#define HNS3_VLAN_TAG_SIZE           4
>> +#define HNS3_DEFAULT_RX_BUF_LEN              2048
>> +
>> +#define HNS3_ETH_OVERHEAD \
>> +     (RTE_ETHER_HDR_LEN + RTE_ETHER_CRC_LEN +
>> HNS3_VLAN_TAG_SIZE * 2)
>> +#define HNS3_PKTLEN_TO_MTU(pktlen)   ((pktlen) -
>> HNS3_ETH_OVERHEAD)
>> +#define HNS3_MAX_MTU (HNS3_MAX_FRAME_LEN -
>> HNS3_ETH_OVERHEAD)
>> +#define HNS3_DEFAULT_MTU             1500UL
>> +#define HNS3_DEFAULT_FRAME_LEN               (HNS3_DEFAULT_MTU +
>> HNS3_ETH_OVERHEAD)
>> +
>> +#define HNS3_4_TCS                   4
>> +#define HNS3_8_TCS                   8
>> +#define HNS3_MAX_TC_NUM                      8
>> +
>> +#define HNS3_MAX_PF_NUM                      8
>> +#define HNS3_UMV_TBL_SIZE            3072
>> +#define HNS3_DEFAULT_UMV_SPACE_PER_PF \
>> +     (HNS3_UMV_TBL_SIZE / HNS3_MAX_PF_NUM)
>> +
>> +#define HNS3_PF_CFG_BLOCK_SIZE               32
>> +#define HNS3_PF_CFG_DESC_NUM \
>> +     (HNS3_PF_CFG_BLOCK_SIZE / HNS3_CFG_RD_LEN_BYTES)
>> +
>> +#define HNS3_DEFAULT_ENABLE_PFC_NUM  0
>> +
>> +#define HNS3_INTR_UNREG_FAIL_RETRY_CNT       5
>> +#define HNS3_INTR_UNREG_FAIL_DELAY_MS        500
>> +
>> +#define HNS3_QUIT_RESET_CNT          10
>> +#define HNS3_QUIT_RESET_DELAY_MS     100
>> +
>> +#define HNS3_POLL_RESPONE_MS         1
>> +
>> +#define HNS3_MAX_USER_PRIO           8
>> +#define HNS3_PG_NUM                  4
>> +enum hns3_fc_mode {
>> +     HNS3_FC_NONE,
>> +     HNS3_FC_RX_PAUSE,
>> +     HNS3_FC_TX_PAUSE,
>> +     HNS3_FC_FULL,
>> +     HNS3_FC_DEFAULT
>> +};
>> +
>> +#define HNS3_SCH_MODE_SP     0
>> +#define HNS3_SCH_MODE_DWRR   1
>> +struct hns3_pg_info {
>> +     uint8_t pg_id;
>> +     uint8_t pg_sch_mode;  /* 0: sp; 1: dwrr */
>> +     uint8_t tc_bit_map;
>> +     uint32_t bw_limit;
>> +     uint8_t tc_dwrr[HNS3_MAX_TC_NUM];
>> +};
>> +
>> +struct hns3_tc_info {
>> +     uint8_t tc_id;
>> +     uint8_t tc_sch_mode;  /* 0: sp; 1: dwrr */
>> +     uint8_t pgid;
>> +     uint32_t bw_limit;
>> +     uint8_t up_to_tc_map; /* user priority maping on the TC */
>> +};
>> +
>> +struct hns3_dcb_info {
>> +     uint8_t num_tc;
>> +     uint8_t num_pg;     /* It must be 1 if vNET-Base schd */
>> +     uint8_t pg_dwrr[HNS3_PG_NUM];
>> +     uint8_t prio_tc[HNS3_MAX_USER_PRIO];
>> +     struct hns3_pg_info pg_info[HNS3_PG_NUM];
>> +     struct hns3_tc_info tc_info[HNS3_MAX_TC_NUM];
>> +     uint8_t hw_pfc_map; /* Allow for packet drop or not on this TC */
>> +     uint8_t pfc_en; /* Pfc enabled or not for user priority */
>> +};
>> +
>> +enum hns3_fc_status {
>> +     HNS3_FC_STATUS_NONE,
>> +     HNS3_FC_STATUS_MAC_PAUSE,
>> +     HNS3_FC_STATUS_PFC,
>> +};
>> +
>> +struct hns3_tc_queue_info {
>> +     uint8_t tqp_offset;     /* TQP offset from base TQP */
>> +     uint8_t tqp_count;      /* Total TQPs */
>> +     uint8_t tc;             /* TC index */
>> +     bool enable;            /* If this TC is enable or not */
>> +};
>> +
>> +struct hns3_cfg {
>> +     uint8_t vmdq_vport_num;
>> +     uint8_t tc_num;
>> +     uint16_t tqp_desc_num;
>> +     uint16_t rx_buf_len;
>> +     uint16_t rss_size_max;
>> +     uint8_t phy_addr;
>> +     uint8_t media_type;
>> +     uint8_t mac_addr[RTE_ETHER_ADDR_LEN];
>> +     uint8_t default_speed;
>> +     uint32_t numa_node_map;
>> +     uint8_t speed_ability;
>> +     uint16_t umv_space;
>> +};
>> +
>> +/* mac media type */
>> +enum hns3_media_type {
>> +     HNS3_MEDIA_TYPE_UNKNOWN,
>> +     HNS3_MEDIA_TYPE_FIBER,
>> +     HNS3_MEDIA_TYPE_COPPER,
>> +     HNS3_MEDIA_TYPE_BACKPLANE,
>> +     HNS3_MEDIA_TYPE_NONE,
>> +};
>> +
>> +struct hns3_mac {
>> +     uint8_t mac_addr[RTE_ETHER_ADDR_LEN];
>> +     bool default_addr_setted; /* whether default addr(mac_addr) is
>> setted */
>> +     uint8_t media_type;
>> +     uint8_t phy_addr;
>> +     uint8_t link_duplex  : 1; /* ETH_LINK_[HALF/FULL]_DUPLEX */
>> +     uint8_t link_autoneg : 1; /* ETH_LINK_[AUTONEG/FIXED] */
>> +     uint8_t link_status  : 1; /* ETH_LINK_[DOWN/UP] */
>> +     uint32_t link_speed;      /* ETH_SPEED_NUM_ */
>> +};
>> +
>> +
>> +/* Primary process maintains driver state in main thread.
>> + *
>> + * +---------------+
>> + * | UNINITIALIZED |<-----------+
>> + * +---------------+         |
>> + *   |.eth_dev_init          |.eth_dev_uninit
>> + *   V                       |
>> + * +---------------+------------+
>> + * |  INITIALIZED  |
>> + * +---------------+<-----------<---------------+
>> + *   |.dev_configure         |               |
>> + *   V                       |failed         |
>> + * +---------------+------------+            |
>> + * |  CONFIGURING  |                         |
>> + * +---------------+----+                    |
>> + *   |success        |                       |
>> + *   |               |               +---------------+
>> + *   |               |               |    CLOSING    |
>> + *   |               |               +---------------+
>> + *   |               |                       ^
>> + *   V               |.dev_configure         |
>> + * +---------------+----+                    |.dev_close
>> + * |  CONFIGURED   |----------------------------+
>> + * +---------------+<-----------+
>> + *   |.dev_start             |
>> + *   V                       |
>> + * +---------------+         |
>> + * |   STARTING    |------------^
>> + * +---------------+ failed  |
>> + *   |success                |
>> + *   |               +---------------+
>> + *   |               |   STOPPING    |
>> + *   |               +---------------+
>> + *   |                       ^
>> + *   V                       |.dev_stop
>> + * +---------------+------------+
>> + * |    STARTED    |
>> + * +---------------+
>> + */
>> +enum hns3_adapter_state {
>> +     HNS3_NIC_UNINITIALIZED = 0,
>> +     HNS3_NIC_INITIALIZED,
>> +     HNS3_NIC_CONFIGURING,
>> +     HNS3_NIC_CONFIGURED,
>> +     HNS3_NIC_STARTING,
>> +     HNS3_NIC_STARTED,
>> +     HNS3_NIC_STOPPING,
>> +     HNS3_NIC_CLOSING,
>> +     HNS3_NIC_CLOSED,
>> +     HNS3_NIC_REMOVED,
>> +     HNS3_NIC_NSTATES
>> +};
>> +
>> +/* Reset various stages, execute in order */
>> +enum hns3_reset_stage {
>> +     /* Stop query services, stop transceiver, disable MAC */
>> +     RESET_STAGE_DOWN,
>> +     /* Clear reset completion flags, disable send command */
>> +     RESET_STAGE_PREWAIT,
>> +     /* Inform IMP to start resetting */
>> +     RESET_STAGE_REQ_HW_RESET,
>> +     /* Waiting for hardware reset to complete */
>> +     RESET_STAGE_WAIT,
>> +     /* Reinitialize hardware */
>> +     RESET_STAGE_DEV_INIT,
>> +     /* Restore user settings and enable MAC */
>> +     RESET_STAGE_RESTORE,
>> +     /* Restart query services, start transceiver */
>> +     RESET_STAGE_DONE,
>> +     /* Not in reset state */
>> +     RESET_STAGE_NONE,
>> +};
>> +
>> +enum hns3_reset_level {
>> +     HNS3_NONE_RESET,
>> +     HNS3_VF_FUNC_RESET, /* A VF function reset */
>> +     /*
>> +      * All VFs under a PF perform function reset.
>> +      * Kernel PF driver use mailbox to inform DPDK VF to do reset, the
>> value
>> +      * of the reset level and the one defined in kernel driver should be
>> +      * same.
>> +      */
>> +     HNS3_VF_PF_FUNC_RESET = 2,
>> +     /*
>> +      * All VFs under a PF perform FLR reset.
>> +      * Kernel PF driver use mailbox to inform DPDK VF to do reset, the
>> value
>> +      * of the reset level and the one defined in kernel driver should be
>> +      * same.
>> +      */
>> +     HNS3_VF_FULL_RESET = 3,
>> +     HNS3_FLR_RESET,     /* A VF perform FLR reset */
>> +     /* All VFs under the rootport perform a global or IMP reset */
>> +     HNS3_VF_RESET,
>> +     HNS3_FUNC_RESET,    /* A PF function reset */
>> +     /* All PFs under the rootport perform a global reset */
>> +     HNS3_GLOBAL_RESET,
>> +     HNS3_IMP_RESET,     /* All PFs under the rootport perform a IMP
>> reset */
>> +     HNS3_MAX_RESET
>> +};
>> +
>> +enum hns3_wait_result {
>> +     HNS3_WAIT_UNKNOWN,
>> +     HNS3_WAIT_REQUEST,
>> +     HNS3_WAIT_SUCCESS,
>> +     HNS3_WAIT_TIMEOUT
>> +};
>> +
>> +#define HNS3_RESET_SYNC_US 100000
>> +
>> +struct hns3_reset_stats {
>> +     uint64_t request_cnt; /* Total request reset times */
>> +     uint64_t global_cnt;  /* Total GLOBAL reset times */
>> +     uint64_t imp_cnt;     /* Total IMP reset times */
>> +     uint64_t exec_cnt;    /* Total reset executive times */
>> +     uint64_t success_cnt; /* Total reset successful times */
>> +     uint64_t fail_cnt;    /* Total reset failed times */
>> +     uint64_t merge_cnt;   /* Total merged in high reset times */
>> +};
>> +
>> +typedef bool (*check_completion_func)(struct hns3_hw *hw);
>> +
>> +struct hns3_wait_data {
>> +     void *hns;
>> +     uint64_t end_ms;
>> +     uint64_t interval;
>> +     int16_t count;
>> +     enum hns3_wait_result result;
>> +     check_completion_func check_completion;
>> +};
>> +
>> +struct hns3_reset_ops {
>> +     void (*reset_service)(void *arg);
>> +     int (*stop_service)(struct hns3_adapter *hns);
>> +     int (*prepare_reset)(struct hns3_adapter *hns);
>> +     int (*wait_hardware_ready)(struct hns3_adapter *hns);
>> +     int (*reinit_dev)(struct hns3_adapter *hns);
>> +     int (*restore_conf)(struct hns3_adapter *hns);
>> +     int (*start_service)(struct hns3_adapter *hns);
>> +};
>> +
>> +enum hns3_schedule {
>> +     SCHEDULE_NONE,
>> +     SCHEDULE_PENDING,
>> +     SCHEDULE_REQUESTED,
>> +     SCHEDULE_DEFERRED,
>> +};
>> +
>> +struct hns3_reset_data {
>> +     enum hns3_reset_stage stage;
>> +     rte_atomic16_t schedule;
>> +     /* Reset flag, covering the entire reset process */
>> +     rte_atomic16_t resetting;
>> +     /* Used to disable sending cmds during reset */
>> +     rte_atomic16_t disable_cmd;
>> +     /* The reset level being processed */
>> +     enum hns3_reset_level level;
>> +     /* Reset level set, each bit represents a reset level */
>> +     uint64_t pending;
>> +     /* Request reset level set, from interrupt or mailbox */
>> +     uint64_t request;
>> +     int attempts; /* Reset failure retry */
>> +     int retries;  /* Timeout failure retry in reset_post */
>> +     /*
>> +      * At the time of global or IMP reset, the command cannot be sent
>> to
>> +      * stop the tx/rx queues. Tx/Rx queues may be access mbuf during
>> the
>> +      * reset process, so the mbuf is required to be released after the
>> reset
>> +      * is completed.The mbuf_deferred_free is used to mark whether
>> mbuf
>> +      * needs to be released.
>> +      */
>> +     bool mbuf_deferred_free;
>> +     struct timeval start_time;
>> +     struct hns3_reset_stats stats;
>> +     const struct hns3_reset_ops *ops;
>> +     struct hns3_wait_data *wait_data;
>> +};
>> +
>> +struct hns3_hw {
>> +     struct rte_eth_dev_data *data;
>> +     void *io_base;
>> +     struct hns3_mac mac;
>> +     unsigned int secondary_cnt; /* Number of secondary processes
>> init'd. */
>> +     uint32_t fw_version;
>> +
>> +     uint16_t num_msi;
>> +     uint16_t total_tqps_num;    /* total task queue pairs of this PF */
>> +     uint16_t tqps_num;          /* num task queue pairs of this function */
>> +     uint16_t rss_size_max;      /* HW defined max RSS task queue */
>> +     uint16_t rx_buf_len;
>> +     uint16_t num_tx_desc;       /* desc num of per tx queue */
>> +     uint16_t num_rx_desc;       /* desc num of per rx queue */
>> +
>> +     struct rte_ether_addr mc_addrs[HNS3_MC_MACADDR_NUM];
>> +     int mc_addrs_num; /* Multicast mac addresses number */
>> +
>> +     uint8_t num_tc;             /* Total number of enabled TCs */
>> +     uint8_t hw_tc_map;
>> +     enum hns3_fc_mode current_mode;
>> +     enum hns3_fc_mode requested_mode;
>> +     struct hns3_dcb_info dcb_info;
>> +     enum hns3_fc_status current_fc_status; /* current flow control
>> status */
>> +     struct hns3_tc_queue_info tc_queue[HNS3_MAX_TC_NUM];
>> +     uint16_t alloc_tqps;
>> +     uint16_t alloc_rss_size;    /* Queue number per TC */
>> +
>> +     uint32_t flag;
>> +     /*
>> +      * PMD setup and configuration is not thread safe. Since it is not
>> +      * performance sensitive, it is better to guarantee thread-safety
>> +      * and add device level lock. Adapter control operations which
>> +      * change its state should acquire the lock.
>> +      */
>> +     rte_spinlock_t lock;
>> +     enum hns3_adapter_state adapter_state;
>> +     struct hns3_reset_data reset;
>> +};
>> +
>> +#define HNS3_FLAG_TC_BASE_SCH_MODE           1
>> +#define HNS3_FLAG_VNET_BASE_SCH_MODE         2
>> +
>> +struct hns3_err_msix_intr_stats {
>> +     uint64_t mac_afifo_tnl_intr_cnt;
>> +     uint64_t ppu_mpf_abnormal_intr_st2_cnt;
>> +     uint64_t ssu_port_based_pf_intr_cnt;
>> +     uint64_t ppp_pf_abnormal_intr_cnt;
>> +     uint64_t ppu_pf_abnormal_intr_cnt;
>> +};
>> +
>> +/* vlan entry information. */
>> +struct hns3_user_vlan_table {
>> +     LIST_ENTRY(hns3_user_vlan_table) next;
>> +     bool hd_tbl_status;
>> +     uint16_t vlan_id;
>> +};
>> +
>> +struct hns3_port_base_vlan_config {
>> +     uint16_t state;
>> +     uint16_t pvid;
>> +};
>> +
>> +/* Vlan tag configuration for RX direction */
>> +struct hns3_rx_vtag_cfg {
>> +     uint8_t rx_vlan_offload_en; /* Whether enable rx vlan offload */
>> +     uint8_t strip_tag1_en;      /* Whether strip inner vlan tag */
>> +     uint8_t strip_tag2_en;      /* Whether strip outer vlan tag */
>> +     uint8_t vlan1_vlan_prionly; /* Inner VLAN Tag up to descriptor
>> Enable */
>> +     uint8_t vlan2_vlan_prionly; /* Outer VLAN Tag up to descriptor
>> Enable */
>> +};
>> +
>> +/* Vlan tag configuration for TX direction */
>> +struct hns3_tx_vtag_cfg {
>> +     bool accept_tag1;           /* Whether accept tag1 packet from host
>> */
>> +     bool accept_untag1;         /* Whether accept untag1 packet from
>> host */
>> +     bool accept_tag2;
>> +     bool accept_untag2;
>> +     bool insert_tag1_en;        /* Whether insert inner vlan tag */
>> +     bool insert_tag2_en;        /* Whether insert outer vlan tag */
>> +     uint16_t default_tag1;      /* The default inner vlan tag to insert */
>> +     uint16_t default_tag2;      /* The default outer vlan tag to insert */
>> +};
>> +
>> +struct hns3_vtag_cfg {
>> +     struct hns3_rx_vtag_cfg rx_vcfg;
>> +     struct hns3_tx_vtag_cfg tx_vcfg;
>> +};
>> +
>> +/* Request types for IPC. */
>> +enum hns3_mp_req_type {
>> +     HNS3_MP_REQ_START_RXTX = 1,
>> +     HNS3_MP_REQ_STOP_RXTX,
>> +     HNS3_MP_REQ_MAX
>> +};
>> +
>> +/* Pameters for IPC. */
>> +struct hns3_mp_param {
>> +     enum hns3_mp_req_type type;
>> +     int port_id;
>> +     int result;
>> +};
>> +
>> +/* Request timeout for IPC. */
>> +#define HNS3_MP_REQ_TIMEOUT_SEC 5
>> +
>> +/* Key string for IPC. */
>> +#define HNS3_MP_NAME "net_hns3_mp"
>> +
>> +struct hns3_pf {
>> +     struct hns3_adapter *adapter;
>> +     bool is_main_pf;
>> +
>> +     uint32_t pkt_buf_size; /* Total pf buf size for tx/rx */
>> +     uint32_t tx_buf_size; /* Tx buffer size for each TC */
>> +     uint32_t dv_buf_size; /* Dv buffer size for each TC */
>> +
>> +     uint16_t mps; /* Max packet size */
>> +
>> +     uint8_t tx_sch_mode;
>> +     uint8_t tc_max; /* max number of tc driver supported */
>> +     uint8_t local_max_tc; /* max number of local tc */
>> +     uint8_t pfc_max;
>> +     uint8_t prio_tc[HNS3_MAX_USER_PRIO]; /* TC indexed by prio */
>> +     uint16_t pause_time;
>> +     bool support_fc_autoneg;       /* support FC autonegotiate */
>> +
>> +     uint16_t wanted_umv_size;
>> +     uint16_t max_umv_size;
>> +     uint16_t used_umv_size;
>> +
>> +     /* Statistics information for abnormal interrupt */
>> +     struct hns3_err_msix_intr_stats abn_int_stats;
>> +
>> +     bool support_sfp_query;
>> +
>> +     struct hns3_vtag_cfg vtag_config;
>> +     struct hns3_port_base_vlan_config port_base_vlan_cfg;
>> +     LIST_HEAD(vlan_tbl, hns3_user_vlan_table) vlan_list;
>> +};
>> +
>> +struct hns3_vf {
>> +     struct hns3_adapter *adapter;
>> +};
>> +
>> +struct hns3_adapter {
>> +     struct hns3_hw hw;
>> +
>> +     /* Specific for PF or VF */
>> +     bool is_vf; /* false - PF, true - VF */
>> +     union {
>> +             struct hns3_pf pf;
>> +             struct hns3_vf vf;
>> +     };
>> +};
>> +
>> +#define HNS3_DEV_SUPPORT_DCB_B                       0x0
>> +
>> +#define hns3_dev_dcb_supported(hw) \
>> +     hns3_get_bit((hw)->flag, HNS3_DEV_SUPPORT_DCB_B)
>> +
>> +#define HNS3_DEV_PRIVATE_TO_HW(adapter) \
>> +     (&((struct hns3_adapter *)adapter)->hw)
>> +#define HNS3_DEV_PRIVATE_TO_ADAPTER(adapter) \
>> +     ((struct hns3_adapter *)adapter)
>> +#define HNS3_DEV_PRIVATE_TO_PF(adapter) \
>> +     (&((struct hns3_adapter *)adapter)->pf)
>> +#define HNS3VF_DEV_PRIVATE_TO_VF(adapter) \
>> +     (&((struct hns3_adapter *)adapter)->vf)
>> +#define HNS3_DEV_HW_TO_ADAPTER(hw) \
>> +     container_of(hw, struct hns3_adapter, hw)
>> +
>> +#define hns3_set_field(origin, mask, shift, val) \
>> +     do { \
>> +             (origin) &= (~(mask)); \
>> +             (origin) |= ((val) << (shift)) & (mask); \
>> +     } while (0)
>> +#define hns3_get_field(origin, mask, shift) \
>> +     (((origin) & (mask)) >> (shift))
>> +#define hns3_set_bit(origin, shift, val) \
>> +     hns3_set_field((origin), (0x1UL << (shift)), (shift), (val))
>> +#define hns3_get_bit(origin, shift) \
>> +     hns3_get_field((origin), (0x1UL << (shift)), (shift))
>> +
>> +/*
>> + * upper_32_bits - return bits 32-63 of a number
>> + * A basic shift-right of a 64- or 32-bit quantity. Use this to suppress
>> + * the "right shift count >= width of type" warning when that quantity is
>> + * 32-bits.
>> + */
>> +#define upper_32_bits(n) ((uint32_t)(((n) >> 16) >> 16))
>> +
>> +/* lower_32_bits - return bits 0-31 of a number */
>> +#define lower_32_bits(n) ((uint32_t)(n))
>> +
>> +#define BIT(nr) (1UL << (nr))
>> +
>> +#define BITS_PER_LONG        (__SIZEOF_LONG__ * 8)
>> +#define GENMASK(h, l) \
>> +     (((~0UL) << (l)) & (~0UL >> (BITS_PER_LONG - 1 - (h))))
>> +
>> +#define roundup(x, y) ((((x) + ((y) - 1)) / (y)) * (y))
>> +#define rounddown(x, y) ((x) - ((x) % (y)))
>> +
>> +#define DIV_ROUND_UP(n, d) (((n) + (d) - 1) / (d))
>> +
>> +#define max_t(type, x, y) ({                    \
>> +     type __max1 = (x);                      \
>> +     type __max2 = (y);                      \
>> +     __max1 > __max2 ? __max1 : __max2; })
>> +
>> +static inline void hns3_write_reg(void *base, uint32_t reg, uint32_t value)
>> +{
>> +     rte_write32(value, (volatile void *)((char *)base + reg));
>> +}
>> +
>> +static inline uint32_t hns3_read_reg(void *base, uint32_t reg)
>> +{
>> +     return rte_read32((volatile void *)((char *)base + reg));
>> +}
>> +
>> +#define hns3_write_dev(a, reg, value) \
>> +     hns3_write_reg((a)->io_base, (reg), (value))
>> +
>> +#define hns3_read_dev(a, reg) \
>> +     hns3_read_reg((a)->io_base, (reg))
>> +
>> +#define ARRAY_SIZE(x) (sizeof(x) / sizeof((x)[0]))
>> +
>> +#define NEXT_ITEM_OF_ACTION(act, actions, index)                        \
>> +     do {                                                            \
>> +             act = (actions) + (index);                              \
>> +             while (act->type == RTE_FLOW_ACTION_TYPE_VOID) {        \
>> +                     (index)++;                                      \
>> +                     act = actions + index;                          \
>> +             }                                                       \
>> +     } while (0)
>> +
>> +#define MSEC_PER_SEC              1000L
>> +#define USEC_PER_MSEC             1000L
>> +
>> +static inline uint64_t
>> +get_timeofday_ms(void)
>> +{
>> +     struct timeval tv;
>> +
>> +     (void)gettimeofday(&tv, NULL);
>> +
>> +     return (uint64_t)tv.tv_sec * MSEC_PER_SEC + tv.tv_usec /
>> USEC_PER_MSEC;
>> +}
>> +
>> +static inline uint64_t
>> +hns3_atomic_test_bit(unsigned int nr, volatile uint64_t *addr)
>> +{
>> +     uint64_t res;
>> +
>> +     rte_mb();
> Is *addr CIO memory or a MMIO register?
> Looking at the patch 20/22, it should be a MMIO register. Whether a barrier is required depends on preceding accesses, so advise move the barrier out.
>> +     res = ((*addr) & (1UL << nr)) != 0;
>> +     rte_mb();
> If *addr is CIO memory, rte_cio_rmb is enough, rte_mb is overkilling.
> If *addr is MMIO register, rte_rmb one-way barrier is also adequate.
> Whether this barrier is required is depending on the following accesses, also advise to move out.
> This API should not includes barriers inside as a common API.
addr is a shared variable address between threads, we will remove the
memory barrier, and use atomic read __atomic_load_n as below:

static inline uint64_t
hns3_atomic_test_bit(unsigned int nr, volatile uint64_t *addr)
{
    uint64_t res;

    res = (__atomic_load_n(addr, __ATOMIC_RELAXED) & (1UL << nr)) != 0;   

    return res;
}

Thanks for your suggestion.
>> +     return res;
>> +}
>> +
>> +static inline void
>> +hns3_atomic_set_bit(unsigned int nr, volatile uint64_t *addr)
>> +{
>> +     __sync_fetch_and_or(addr, (1UL << nr));
> Gcc/clang provides '__atomic' builtins to replace the legacy '__sync' builtins, new code should always use the '__atomic' builtins rather than the '__sync' builtins.
> This function can be replaced with __atomic_fetch_or(addr, (1UL << nr), __ATOMIC_RELAXED);
> https://gcc.gnu.org/onlinedocs/gcc-6.1.0/gcc/_005f_005fatomic-Builtins.html
OK, we will update and replace it with __atomic_xxx in patch V2.
Thanks for your suggestion.

    Regards
Xavier
>> +}
>> +
>> +static inline void
>> +hns3_atomic_clear_bit(unsigned int nr, volatile uint64_t *addr)
>> +{
>> +     __sync_fetch_and_and(addr, ~(1UL << nr));
>> +}
>> +
>> +static inline int64_t
>> +hns3_test_and_clear_bit(unsigned int nr, volatile uint64_t *addr)
>> +{
>> +     uint64_t mask = (1UL << nr);
>> +
>> +     return __sync_fetch_and_and(addr, ~mask) & mask;
>> +}
>> +
>> +#endif /* _HNS3_ETHDEV_H_ */
>> --
>> 2.7.4
> IMPORTANT NOTICE: The contents of this email and any attachments are confidential and may also be privileged. If you are not the intended recipient, please notify the sender immediately and do not disclose the contents to any other person, use it for any purpose, or store or copy the information in any medium. Thank you.
>
> .
>



  reply	other threads:[~2019-09-05  6:01 UTC|newest]

Thread overview: 75+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-08-23 13:46 [dpdk-dev] [PATCH 00/22] add hns3 ethernet PMD driver Wei Hu (Xavier)
2019-08-23 13:46 ` [dpdk-dev] [PATCH 01/22] net/hns3: add hardware registers definition Wei Hu (Xavier)
2019-08-23 13:46 ` [dpdk-dev] [PATCH 02/22] net/hns3: add some definitions for data structure and macro Wei Hu (Xavier)
2019-08-30  8:25   ` Gavin Hu (Arm Technology China)
2019-09-05  6:01     ` Wei Hu (Xavier) [this message]
2019-08-23 13:46 ` [dpdk-dev] [PATCH 03/22] net/hns3: register hns3 PMD driver Wei Hu (Xavier)
2019-08-30 15:01   ` Ferruh Yigit
2019-09-06  6:20     ` Wei Hu (Xavier)
2019-08-23 13:46 ` [dpdk-dev] [PATCH 04/22] net/hns3: add support for cmd of " Wei Hu (Xavier)
2019-08-30 15:02   ` Ferruh Yigit
2019-09-06  6:49     ` Wei Hu (Xavier)
2019-08-23 13:46 ` [dpdk-dev] [PATCH 05/22] net/hns3: add the initialization " Wei Hu (Xavier)
2019-08-23 13:46 ` [dpdk-dev] [PATCH 06/22] net/hns3: add support for MAC address related operations Wei Hu (Xavier)
2019-08-30 15:03   ` Ferruh Yigit
2019-09-05  5:40     ` Wei Hu (Xavier)
2019-08-23 13:46 ` [dpdk-dev] [PATCH 07/22] net/hns3: add support for some misc operations Wei Hu (Xavier)
2019-08-30 15:04   ` Ferruh Yigit
2019-08-23 13:46 ` [dpdk-dev] [PATCH 08/22] net/hns3: add support for link update operation Wei Hu (Xavier)
2019-08-30 15:04   ` Ferruh Yigit
2019-09-06  6:56     ` Wei Hu (Xavier)
2019-08-23 13:46 ` [dpdk-dev] [PATCH 09/22] net/hns3: add support for flow directory of hns3 PMD driver Wei Hu (Xavier)
2019-08-30 15:06   ` Ferruh Yigit
2019-09-06  8:23     ` Wei Hu (Xavier)
2019-09-06 11:08     ` Wei Hu (Xavier)
2019-08-23 13:46 ` [dpdk-dev] [PATCH 10/22] net/hns3: add support for RSS " Wei Hu (Xavier)
2019-08-30 15:07   ` Ferruh Yigit
2019-08-31  9:16     ` Wei Hu (Xavier)
2019-08-23 13:47 ` [dpdk-dev] [PATCH 11/22] net/hns3: add support for flow control " Wei Hu (Xavier)
2019-08-30 15:07   ` Ferruh Yigit
2019-08-31  8:04     ` Wei Hu (Xavier)
2019-08-23 13:47 ` [dpdk-dev] [PATCH 12/22] net/hns3: add support for VLAN " Wei Hu (Xavier)
2019-08-30 15:08   ` Ferruh Yigit
2019-08-31  9:04     ` Wei Hu (Xavier)
2019-08-23 13:47 ` [dpdk-dev] [PATCH 13/22] net/hns3: add support for mailbox " Wei Hu (Xavier)
2019-08-30 15:08   ` Ferruh Yigit
2019-09-06 11:25     ` Wei Hu (Xavier)
2019-08-23 13:47 ` [dpdk-dev] [PATCH 14/22] net/hns3: add support for hns3 VF " Wei Hu (Xavier)
2019-08-30 15:11   ` Ferruh Yigit
2019-08-31  9:03     ` Wei Hu (Xavier)
2019-09-06 11:27     ` Wei Hu (Xavier)
2019-08-23 13:47 ` [dpdk-dev] [PATCH 15/22] net/hns3: add package and queue related operation Wei Hu (Xavier)
2019-08-23 15:42   ` Aaron Conole
2019-08-30 15:13   ` Ferruh Yigit
2019-09-11 11:40     ` Wei Hu (Xavier)
2019-08-23 13:47 ` [dpdk-dev] [PATCH 16/22] net/hns3: add start stop configure promiscuous ops Wei Hu (Xavier)
2019-08-30 15:14   ` Ferruh Yigit
2019-09-06 11:51     ` Wei Hu (Xavier)
2019-08-23 13:47 ` [dpdk-dev] [PATCH 17/22] net/hns3: add dump register ops for hns3 PMD driver Wei Hu (Xavier)
2019-08-23 13:47 ` [dpdk-dev] [PATCH 18/22] net/hns3: add abnormal interrupt process " Wei Hu (Xavier)
2019-08-23 13:47 ` [dpdk-dev] [PATCH 19/22] net/hns3: add stats related ops " Wei Hu (Xavier)
2019-08-30 15:20   ` Ferruh Yigit
2019-08-31  8:49     ` Wei Hu (Xavier)
2019-08-23 13:47 ` [dpdk-dev] [PATCH 20/22] net/hns3: add reset related process " Wei Hu (Xavier)
2019-08-23 13:47 ` [dpdk-dev] [PATCH 21/22] net/hns3: add multiple process support " Wei Hu (Xavier)
2019-08-30 15:14   ` Ferruh Yigit
2019-09-02 13:41     ` Wei Hu (Xavier)
2019-08-23 13:47 ` [dpdk-dev] [PATCH 22/22] net/hns3: add hns3 build files Wei Hu (Xavier)
2019-08-23 14:08   ` Jerin Jacob Kollanukkaran
2019-08-30  3:22     ` Wei Hu (Xavier)
2019-08-31  2:10       ` Wei Hu (Xavier)
2019-08-30 14:57     ` Ferruh Yigit
2019-08-30  6:16   ` Stephen Hemminger
2019-08-31  8:46     ` Wei Hu (Xavier)
2019-08-30  6:17   ` Stephen Hemminger
2019-08-31  8:44     ` Wei Hu (Xavier)
2019-09-03 15:27     ` Ye Xiaolong
2019-09-11 11:36       ` Wei Hu (Xavier)
2019-08-30 14:58   ` Ferruh Yigit
2019-09-10 11:43     ` Wei Hu (Xavier)
2019-08-30 15:00   ` Ferruh Yigit
2019-08-31  8:07     ` Wei Hu (Xavier)
2019-08-30 15:12   ` Ferruh Yigit
2019-08-31  8:07     ` Wei Hu (Xavier)
2019-08-30 15:23 ` [dpdk-dev] [PATCH 00/22] add hns3 ethernet PMD driver Ferruh Yigit
2019-08-31  8:06   ` Wei Hu (Xavier)

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=18059d08-a9d2-3eca-82c7-642fc4a49869@huawei.com \
    --to=xavier.huwei@huawei.com \
    --cc=Gavin.Hu@arm.com \
    --cc=dev@dpdk.org \
    --cc=forest.zhouchang@huawei.com \
    --cc=linuxarm@huawei.com \
    --cc=liudongdong3@huawei.com \
    --cc=xavier_huwei@163.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).