Hi Sunil,
My comments are inline.
>-----Original Message-----
>From: Sunil Kumar Kori <sk...@marvell.com>
>Sent: Thursday, January 12, 2023 10:09 PM
>To: Ankur Dwivedi <adwiv...@marvell.com>; dev@dpdk.org
>Cc: tho...@monjalon.net; david.march...@redhat.com; m...@ashroe.eu;
>or...@nvidia.com; ferruh.yi...@amd.com; ch...@att.com;
>humi...@huawei.com; linvi...@tuxdriver.com; ciara.lof...@intel.com;
>qi.z.zh...@intel.com; m...@semihalf.com; m...@semihalf.com;
>shaib...@amazon.com; evge...@amazon.com; igo...@amazon.com;
>cha...@amd.com; Igor Russkikh <irussk...@marvell.com>;
>shepard.sie...@atomicrules.com; ed.cz...@atomicrules.com;
>john.mil...@atomicrules.com; ajit.khapa...@broadcom.com;
>somnath.ko...@broadcom.com; Jerin Jacob Kollanukkaran
><jer...@marvell.com>; Maciej Czekaj [C] <mcze...@marvell.com>; Shijith
>Thotton <sthot...@marvell.com>; Srisivasubramanian Srinivasan
><sriniva...@marvell.com>; Harman Kalra <hka...@marvell.com>;
>rahul.lakkire...@chelsio.com; johnd...@cisco.com; hyon...@cisco.com;
>liudongdo...@huawei.com; yisen.zhu...@huawei.com;
>xuanziya...@huawei.com; cloud.wangxiao...@huawei.com;
>zhouguoy...@huawei.com; simei...@intel.com; wenjun1...@intel.com;
>qiming.y...@intel.com; yuying.zh...@intel.com; beilei.x...@intel.com;
>xiao.w.w...@intel.com; jingjing...@intel.com; junfeng....@intel.com;
>rosen...@intel.com; Nithin Kumar Dabilpuram <ndabilpu...@marvell.com>;
>Kiran Kumar Kokkilagadda <kirankum...@marvell.com>; Satha Koteswara Rao
>Kottidi <skotesh...@marvell.com>; Liron Himi <lir...@marvell.com>;
>z...@semihalf.com; Radha Chintakuntla <rad...@marvell.com>;
>Veerasenareddy Burru <vbu...@marvell.com>; Sathesh B Edara
><sed...@marvell.com>; ma...@nvidia.com; viachesl...@nvidia.com;
>lon...@microsoft.com; spin...@cesnet.cz; chaoyong...@corigine.com;
>niklas.soderl...@corigine.com; hemant.agra...@nxp.com;
>sachin.sax...@oss.nxp.com; g.si...@nxp.com; apeksha.gu...@nxp.com;
>sachin.sax...@nxp.com; abo...@pensando.io; Rasesh Mody
><rm...@marvell.com>; Shahed Shaikh <shsha...@marvell.com>; Devendra
>Singh Rawat <dsinghra...@marvell.com>; andrew.rybche...@oktetlabs.ru;
>jiawe...@trustnetic.com; jianw...@trustnetic.com; jbehr...@vmware.com;
>maxime.coque...@redhat.com; chenbo....@intel.com;
>steven.webs...@windriver.com; matt.pet...@windriver.com;
>bruce.richard...@intel.com; mtetsu...@gmail.com; gr...@u256.net;
>jasvinder.si...@intel.com; cristian.dumitre...@intel.com; jgraj...@cisco.com;
>m...@smartsharesystems.com; Ankur Dwivedi <adwiv...@marvell.com>
>Subject: RE: [PATCH v5 3/6] ethdev: add trace points for remaining functions
>
>> -----Original Message-----
>> From: Ankur Dwivedi <adwiv...@marvell.com>
>> Sent: Thursday, January 12, 2023 4:52 PM
>> To: dev@dpdk.org
>> Cc: tho...@monjalon.net; david.march...@redhat.com; m...@ashroe.eu;
>> or...@nvidia.com; ferruh.yi...@amd.com; ch...@att.com;
>> humi...@huawei.com; linvi...@tuxdriver.com; ciara.lof...@intel.com;
>> qi.z.zh...@intel.com; m...@semihalf.com; m...@semihalf.com;
>> shaib...@amazon.com; evge...@amazon.com; igo...@amazon.com;
>> cha...@amd.com; Igor Russkikh <irussk...@marvell.com>;
>> shepard.sie...@atomicrules.com; ed.cz...@atomicrules.com;
>> john.mil...@atomicrules.com; ajit.khapa...@broadcom.com;
>> somnath.ko...@broadcom.com; Jerin Jacob Kollanukkaran
>> <jer...@marvell.com>; Maciej Czekaj [C] <mcze...@marvell.com>; Shijith
>> Thotton <sthot...@marvell.com>; Srisivasubramanian Srinivasan
>> <sriniva...@marvell.com>; Harman Kalra <hka...@marvell.com>;
>> rahul.lakkire...@chelsio.com; johnd...@cisco.com; hyon...@cisco.com;
>> liudongdo...@huawei.com; yisen.zhu...@huawei.com;
>> xuanziya...@huawei.com; cloud.wangxiao...@huawei.com;
>> zhouguoy...@huawei.com; simei...@intel.com; wenjun1...@intel.com;
>> qiming.y...@intel.com; yuying.zh...@intel.com; beilei.x...@intel.com;
>> xiao.w.w...@intel.com; jingjing...@intel.com; junfeng....@intel.com;
>> rosen...@intel.com; Nithin Kumar Dabilpuram
>> <ndabilpu...@marvell.com>; Kiran Kumar Kokkilagadda
>> <kirankum...@marvell.com>; Sunil Kumar Kori <sk...@marvell.com>; Satha
>> Koteswara Rao Kottidi <skotesh...@marvell.com>; Liron Himi
>> <lir...@marvell.com>; z...@semihalf.com; Radha Chintakuntla
>> <rad...@marvell.com>; Veerasenareddy Burru <vbu...@marvell.com>;
>> Sathesh B Edara <sed...@marvell.com>; ma...@nvidia.com;
>> viachesl...@nvidia.com; lon...@microsoft.com; spin...@cesnet.cz;
>> chaoyong...@corigine.com; niklas.soderl...@corigine.com;
>> hemant.agra...@nxp.com; sachin.sax...@oss.nxp.com; g.si...@nxp.com;
>> apeksha.gu...@nxp.com; sachin.sax...@nxp.com; abo...@pensando.io;
>> Rasesh Mody <rm...@marvell.com>; Shahed Shaikh
>> <shsha...@marvell.com>; Devendra Singh Rawat
>> <dsinghra...@marvell.com>; andrew.rybche...@oktetlabs.ru;
>> jiawe...@trustnetic.com; jianw...@trustnetic.com;
>> jbehr...@vmware.com; maxime.coque...@redhat.com;
>> chenbo....@intel.com; steven.webs...@windriver.com;
>> matt.pet...@windriver.com; bruce.richard...@intel.com;
>> mtetsu...@gmail.com; gr...@u256.net; jasvinder.si...@intel.com;
>> cristian.dumitre...@intel.com; jgraj...@cisco.com;
>> m...@smartsharesystems.com; Ankur Dwivedi <adwiv...@marvell.com>
>> Subject: [PATCH v5 3/6] ethdev: add trace points for remaining functions
>>
>> Adds trace points for remaining ethdev functions.
>>
>> Signed-off-by: Ankur Dwivedi <adwiv...@marvell.com>
>> ---
>> lib/ethdev/ethdev_trace_points.c | 252 +++++++++++++++
>> lib/ethdev/rte_ethdev.c | 476 ++++++++++++++++++++++-----
>> lib/ethdev/rte_ethdev_cman.c | 30 +-
>> lib/ethdev/rte_ethdev_trace.h | 529 +++++++++++++++++++++++++++++++
>> lib/ethdev/rte_ethdev_trace_fp.h | 342 ++++++++++++++++++++
>> lib/ethdev/version.map | 81 +++++
>> 6 files changed, 1622 insertions(+), 88 deletions(-)
>>
>
>Why can't this patch be part of [PATCH v5 2/6] ?
Combining this patch with PATCH v5 2/6 will make the combined patch large.
There was a review comment in earlier version (v3) to split the patch.
>
>> diff --git a/lib/ethdev/ethdev_trace_points.c
>> b/lib/ethdev/ethdev_trace_points.c
>> index 4fea76e0ff..102a18fcc1 100644
>> --- a/lib/ethdev/ethdev_trace_points.c
>> +++ b/lib/ethdev/ethdev_trace_points.c
>> @@ -222,3 +222,255 @@
>> RTE_TRACE_POINT_REGISTER(rte_ethdev_trace_set_rx_queue_stats_mappin
>> g,
>>
>> RTE_TRACE_POINT_REGISTER(rte_ethdev_trace_fw_version_get,
>> lib.ethdev.fw_version_get)
>> +
>> +RTE_TRACE_POINT_REGISTER(rte_ethdev_trace_info_get,
>> + lib.ethdev.info_get)
>> +
>> +RTE_TRACE_POINT_REGISTER(rte_ethdev_trace_conf_get,
>> + lib.ethdev.conf_get)
>> +
>> +RTE_TRACE_POINT_REGISTER(rte_ethdev_trace_get_supported_ptypes,
>> + lib.ethdev.get_supported_ptypes)
>> +
>> +RTE_TRACE_POINT_REGISTER(rte_ethdev_trace_set_ptypes,
>> + lib.ethdev.set_ptypes)
>> +
>> +RTE_TRACE_POINT_REGISTER(rte_eth_trace_macaddrs_get,
>> + lib.ethdev.macaddrs_get)
>> +
>> +RTE_TRACE_POINT_REGISTER(rte_eth_trace_macaddr_get,
>> + lib.ethdev.macaddr_get)
>> +
>> +RTE_TRACE_POINT_REGISTER(rte_ethdev_trace_get_mtu,
>> + lib.ethdev.get_mtu)
>> +
>> +RTE_TRACE_POINT_REGISTER(rte_ethdev_trace_set_mtu,
>> + lib.ethdev.set_mtu)
>> +
>> +RTE_TRACE_POINT_REGISTER(rte_ethdev_trace_vlan_filter,
>> + lib.ethdev.vlan_filter)
>> +
>> +RTE_TRACE_POINT_REGISTER(rte_ethdev_trace_set_vlan_strip_on_queue,
>> + lib.ethdev.set_vlan_strip_on_queue)
>> +
>> +RTE_TRACE_POINT_REGISTER(rte_ethdev_trace_set_vlan_ether_type,
>> + lib.ethdev.set_vlan_ether_type)
>> +
>> +RTE_TRACE_POINT_REGISTER(rte_ethdev_trace_set_vlan_offload,
>> + lib.ethdev.set_vlan_offload)
>> +
>> +RTE_TRACE_POINT_REGISTER(rte_ethdev_trace_get_vlan_offload,
>> + lib.ethdev.get_vlan_offload)
>> +
>> +RTE_TRACE_POINT_REGISTER(rte_ethdev_trace_set_vlan_pvid,
>> + lib.ethdev.set_vlan_pvid)
>> +
>> +RTE_TRACE_POINT_REGISTER(rte_ethdev_trace_flow_ctrl_get,
>> + lib.ethdev.flow_ctrl_get)
>> +
>> +RTE_TRACE_POINT_REGISTER(rte_ethdev_trace_flow_ctrl_set,
>> + lib.ethdev.flow_ctrl_set)
>> +
>> +RTE_TRACE_POINT_REGISTER(rte_ethdev_trace_priority_flow_ctrl_set,
>> + lib.ethdev.priority_flow_ctrl_set)
>> +
>> +RTE_TRACE_POINT_REGISTER(rte_ethdev_trace_priority_flow_ctrl_queue_i
>> nfo_get,
>> + lib.ethdev.priority_flow_ctrl_queue_info_get)
>> +
>> +RTE_TRACE_POINT_REGISTER(rte_ethdev_trace_priority_flow_ctrl_queue_c
>> onfigure,
>> + lib.ethdev.priority_flow_ctrl_queue_configure)
>> +
>> +RTE_TRACE_POINT_REGISTER(rte_ethdev_trace_rss_reta_update,
>> + lib.ethdev.rss_reta_update)
>> +
>> +RTE_TRACE_POINT_REGISTER(rte_ethdev_trace_rss_reta_query,
>> + lib.ethdev.rss_reta_query)
>> +
>> +RTE_TRACE_POINT_REGISTER(rte_ethdev_trace_rss_hash_update,
>> + lib.ethdev.rss_hash_update)
>> +
>> +RTE_TRACE_POINT_REGISTER(rte_ethdev_trace_rss_hash_conf_get,
>> + lib.ethdev.rss_hash_conf_get)
>> +
>> +RTE_TRACE_POINT_REGISTER(rte_ethdev_trace_udp_tunnel_port_add,
>> + lib.ethdev.udp_tunnel_port_add)
>> +
>> +RTE_TRACE_POINT_REGISTER(rte_ethdev_trace_udp_tunnel_port_delete,
>> + lib.ethdev.udp_tunnel_port_delete)
>> +
>> +RTE_TRACE_POINT_REGISTER(rte_eth_trace_led_on,
>> + lib.ethdev.led_on)
>> +
>> +RTE_TRACE_POINT_REGISTER(rte_eth_trace_led_off,
>> + lib.ethdev.led_off)
>> +
>> +RTE_TRACE_POINT_REGISTER(rte_eth_trace_fec_get_capability,
>> + lib.ethdev.fec_get_capability)
>> +
>> +RTE_TRACE_POINT_REGISTER(rte_eth_trace_fec_get,
>> + lib.ethdev.fec_get)
>> +
>> +RTE_TRACE_POINT_REGISTER(rte_eth_trace_fec_set,
>> + lib.ethdev.fec_set)
>> +
>> +RTE_TRACE_POINT_REGISTER(rte_ethdev_trace_mac_addr_add,
>> + lib.ethdev.mac_addr_add)
>> +
>> +RTE_TRACE_POINT_REGISTER(rte_ethdev_trace_mac_addr_remove,
>> + lib.ethdev.mac_addr_remove)
>> +
>> +RTE_TRACE_POINT_REGISTER(rte_ethdev_trace_default_mac_addr_set,
>> + lib.ethdev.default_mac_addr_set)
>> +
>> +RTE_TRACE_POINT_REGISTER(rte_ethdev_trace_uc_hash_table_set,
>> + lib.ethdev.uc_hash_table_set)
>> +
>> +RTE_TRACE_POINT_REGISTER(rte_ethdev_trace_uc_all_hash_table_set,
>> + lib.ethdev.uc_all_hash_table_set)
>> +
>> +RTE_TRACE_POINT_REGISTER(rte_eth_trace_set_queue_rate_limit,
>> + lib.ethdev.set_queue_rate_limit)
>> +
>> +RTE_TRACE_POINT_REGISTER(rte_eth_trace_rx_avail_thresh_set,
>> + lib.ethdev.rx_avail_thresh_set)
>> +
>> +RTE_TRACE_POINT_REGISTER(rte_eth_trace_rx_avail_thresh_query,
>> + lib.ethdev.rx_avail_thresh_query)
>> +
>> +RTE_TRACE_POINT_REGISTER(rte_ethdev_trace_callback_register,
>> + lib.ethdev.callback_register)
>> +
>> +RTE_TRACE_POINT_REGISTER(rte_ethdev_trace_callback_unregister,
>> + lib.ethdev.callback_unregister)
>> +
>> +RTE_TRACE_POINT_REGISTER(rte_ethdev_trace_rx_intr_ctl,
>> + lib.ethdev.rx_intr_ctl)
>> +
>> +RTE_TRACE_POINT_REGISTER(rte_ethdev_trace_rx_intr_ctl_q_get_fd,
>> + lib.ethdev.rx_intr_ctl_q_get_fd)
>> +
>> +RTE_TRACE_POINT_REGISTER(rte_ethdev_trace_rx_intr_ctl_q,
>> + lib.ethdev.rx_intr_ctl_q)
>> +
>> +RTE_TRACE_POINT_REGISTER(rte_ethdev_trace_rx_intr_enable,
>> + lib.ethdev.rx_intr_enable)
>> +
>> +RTE_TRACE_POINT_REGISTER(rte_ethdev_trace_rx_intr_disable,
>> + lib.ethdev.rx_intr_disable)
>> +
>> +RTE_TRACE_POINT_REGISTER(rte_eth_trace_add_rx_callback,
>> + lib.ethdev.add_rx_callback)
>> +
>> +RTE_TRACE_POINT_REGISTER(rte_eth_trace_add_first_rx_callback,
>> + lib.ethdev.add_first_rx_callback)
>> +
>> +RTE_TRACE_POINT_REGISTER(rte_eth_trace_add_tx_callback,
>> + lib.ethdev.add_tx_callback)
>> +
>> +RTE_TRACE_POINT_REGISTER(rte_eth_trace_remove_rx_callback,
>> + lib.ethdev.remove_rx_callback)
>> +
>> +RTE_TRACE_POINT_REGISTER(rte_eth_trace_remove_tx_callback,
>> + lib.ethdev.remove_tx_callback)
>> +
>> +RTE_TRACE_POINT_REGISTER(rte_eth_trace_rx_queue_info_get,
>> + lib.ethdev.rx_queue_info_get)
>> +
>> +RTE_TRACE_POINT_REGISTER(rte_eth_trace_tx_queue_info_get,
>> + lib.ethdev.tx_queue_info_get)
>> +
>> +RTE_TRACE_POINT_REGISTER(rte_eth_trace_rx_burst_mode_get,
>> + lib.ethdev.rx_burst_mode_get)
>> +
>> +RTE_TRACE_POINT_REGISTER(rte_eth_trace_tx_burst_mode_get,
>> + lib.ethdev.tx_burst_mode_get)
>> +
>> +RTE_TRACE_POINT_REGISTER(rte_eth_trace_get_monitor_addr,
>> + lib.ethdev.get_monitor_addr)
>> +
>> +RTE_TRACE_POINT_REGISTER(rte_ethdev_trace_set_mc_addr_list,
>> + lib.ethdev.set_mc_addr_list)
>> +
>> +RTE_TRACE_POINT_REGISTER(rte_eth_trace_timesync_enable,
>> + lib.ethdev.timesync_enable)
>> +
>> +RTE_TRACE_POINT_REGISTER(rte_eth_trace_timesync_disable,
>> + lib.ethdev.timesync_disable)
>> +
>> +RTE_TRACE_POINT_REGISTER(rte_eth_trace_timesync_read_rx_timestamp,
>> + lib.ethdev.timesync_read_rx_timestamp)
>> +
>> +RTE_TRACE_POINT_REGISTER(rte_eth_trace_timesync_read_tx_timestamp,
>> + lib.ethdev.timesync_read_tx_timestamp)
>> +
>> +RTE_TRACE_POINT_REGISTER(rte_eth_trace_timesync_adjust_time,
>> + lib.ethdev.timesync_adjust_time)
>> +
>> +RTE_TRACE_POINT_REGISTER(rte_eth_trace_timesync_read_time,
>> + lib.ethdev.timesync_read_time)
>> +
>> +RTE_TRACE_POINT_REGISTER(rte_eth_trace_timesync_write_time,
>> + lib.ethdev.timesync_write_time)
>> +
>> +RTE_TRACE_POINT_REGISTER(rte_eth_trace_read_clock,
>> + lib.ethdev.read_clock)
>> +
>> +RTE_TRACE_POINT_REGISTER(rte_ethdev_trace_get_reg_info,
>> + lib.ethdev.get_reg_info)
>> +
>> +RTE_TRACE_POINT_REGISTER(rte_ethdev_trace_get_eeprom_length,
>> + lib.ethdev.get_eeprom_length)
>> +
>> +RTE_TRACE_POINT_REGISTER(rte_ethdev_trace_get_eeprom,
>> + lib.ethdev.get_eeprom)
>> +
>> +RTE_TRACE_POINT_REGISTER(rte_ethdev_trace_set_eeprom,
>> + lib.ethdev.set_eeprom)
>> +
>> +RTE_TRACE_POINT_REGISTER(rte_ethdev_trace_get_module_info,
>> + lib.ethdev.get_module_info)
>> +
>> +RTE_TRACE_POINT_REGISTER(rte_ethdev_trace_get_module_eeprom,
>> + lib.ethdev.get_module_eeprom)
>> +
>> +RTE_TRACE_POINT_REGISTER(rte_ethdev_trace_get_dcb_info,
>> + lib.ethdev.get_dcb_info)
>> +
>> +RTE_TRACE_POINT_REGISTER(rte_ethdev_trace_adjust_nb_rx_tx_desc,
>> + lib.ethdev.adjust_nb_rx_tx_desc)
>> +
>> +RTE_TRACE_POINT_REGISTER(rte_ethdev_trace_hairpin_capability_get,
>> + lib.ethdev.hairpin_capability_get)
>> +
>> +RTE_TRACE_POINT_REGISTER(rte_ethdev_trace_pool_ops_supported,
>> + lib.ethdev.pool_ops_supported)
>> +
>> +RTE_TRACE_POINT_REGISTER(rte_eth_trace_representor_info_get,
>> + lib.ethdev.representor_info_get)
>> +
>> +RTE_TRACE_POINT_REGISTER(rte_eth_trace_rx_metadata_negotiate,
>> + lib.ethdev.rx_metadata_negotiate)
>> +
>> +RTE_TRACE_POINT_REGISTER(rte_eth_trace_ip_reassembly_capability_get,
>> + lib.ethdev.ip_reassembly_capability_get)
>> +
>> +RTE_TRACE_POINT_REGISTER(rte_eth_trace_ip_reassembly_conf_get,
>> + lib.ethdev.ip_reassembly_conf_get)
>> +
>> +RTE_TRACE_POINT_REGISTER(rte_eth_trace_ip_reassembly_conf_set,
>> + lib.ethdev.ip_reassembly_conf_set)
>> +
>> +RTE_TRACE_POINT_REGISTER(rte_eth_trace_buffer_split_get_supported_h
>> dr_ptypes,
>> + lib.ethdev.buffer_split_get_supported_hdr_ptypes)
>> +
>> +RTE_TRACE_POINT_REGISTER(rte_eth_trace_cman_info_get,
>> + lib.ethdev.cman_info_get)
>> +
>> +RTE_TRACE_POINT_REGISTER(rte_eth_trace_cman_config_init,
>> + lib.ethdev.cman_config_init)
>> +
>> +RTE_TRACE_POINT_REGISTER(rte_eth_trace_cman_config_set,
>> + lib.ethdev.cman_config_set)
>> +
>> +RTE_TRACE_POINT_REGISTER(rte_eth_trace_cman_config_get,
>> + lib.ethdev.cman_config_get)
>> diff --git a/lib/ethdev/rte_ethdev.c b/lib/ethdev/rte_ethdev.c
>> index 40897ad94d..60d2ab47ea 100644
>> --- a/lib/ethdev/rte_ethdev.c
>> +++ b/lib/ethdev/rte_ethdev.c
>> @@ -3539,6 +3539,8 @@ rte_eth_dev_info_get(uint16_t port_id, struct
>> rte_eth_dev_info *dev_info)
>>
>> dev_info->dev_flags = &dev->data->dev_flags;
>>
>> + rte_ethdev_trace_info_get(port_id, dev_info);
>> +
>> return 0;
>> }
>>
>> @@ -3559,6 +3561,8 @@ rte_eth_dev_conf_get(uint16_t port_id, struct
>> rte_eth_conf *dev_conf)
>>
>> memcpy(dev_conf, &dev->data->dev_conf, sizeof(struct
>> rte_eth_conf));
>>
>> + rte_ethdev_trace_conf_get(port_id, dev_conf);
>> +
>> return 0;
>> }
>>
>> @@ -3589,8 +3593,10 @@ rte_eth_dev_get_supported_ptypes(uint16_t
>> port_id, uint32_t ptype_mask,
>>
>> for (i = 0, j = 0; all_ptypes[i] != RTE_PTYPE_UNKNOWN; ++i)
>> if (all_ptypes[i] & ptype_mask) {
>> - if (j < num)
>> + if (j < num) {
>> ptypes[j] = all_ptypes[i];
>> +
>> rte_ethdev_trace_get_supported_ptypes(port_id, j, num, ptypes[j]);
>> + }
>> j++;
>> }
>>
>> @@ -3668,6 +3674,7 @@ rte_eth_dev_set_ptypes(uint16_t port_id, uint32_t
>> ptype_mask,
>> if (ptype_mask & all_ptypes[i]) {
>> if (j < num - 1) {
>> set_ptypes[j] = all_ptypes[i];
>> + rte_ethdev_trace_set_ptypes(port_id, j, num,
>> set_ptypes[j]);
>> j++;
>> continue;
>> }
>> @@ -3709,6 +3716,8 @@ rte_eth_macaddrs_get(uint16_t port_id, struct
>> rte_ether_addr *ma,
>> num = RTE_MIN(dev_info.max_mac_addrs, num);
>> memcpy(ma, dev->data->mac_addrs, num * sizeof(ma[0]));
>>
>> + rte_eth_trace_macaddrs_get(port_id, num);
>> +
>> return num;
>> }
>>
>> @@ -3729,6 +3738,8 @@ rte_eth_macaddr_get(uint16_t port_id, struct
>> rte_ether_addr *mac_addr)
>>
>> rte_ether_addr_copy(&dev->data->mac_addrs[0], mac_addr);
>>
>> + rte_eth_trace_macaddr_get(port_id, mac_addr);
>> +
>> return 0;
>> }
>>
>> @@ -3747,6 +3758,9 @@ rte_eth_dev_get_mtu(uint16_t port_id, uint16_t
>> *mtu)
>> }
>>
>> *mtu = dev->data->mtu;
>> +
>> + rte_ethdev_trace_get_mtu(port_id, *mtu);
>> +
>> return 0;
>> }
>>
>> @@ -3789,7 +3803,11 @@ rte_eth_dev_set_mtu(uint16_t port_id, uint16_t
>> mtu)
>> if (ret == 0)
>> dev->data->mtu = mtu;
>>
>> - return eth_err(port_id, ret);
>> + ret = eth_err(port_id, ret);
>> +
>> + rte_ethdev_trace_set_mtu(port_id, mtu, ret);
>> +
>> + return ret;
>> }
>>
>> int
>> @@ -3832,7 +3850,11 @@ rte_eth_dev_vlan_filter(uint16_t port_id,
>> uint16_t vlan_id, int on)
>> vfc->ids[vidx] &= ~RTE_BIT64(vbit);
>> }
>>
>> - return eth_err(port_id, ret);
>> + ret = eth_err(port_id, ret);
>> +
>> + rte_ethdev_trace_vlan_filter(port_id, vlan_id, on, ret);
>> +
>> + return ret;
>> }
>>
>> int
>> @@ -3851,8 +3873,11 @@ rte_eth_dev_set_vlan_strip_on_queue(uint16_t
>> port_id, uint16_t rx_queue_id,
>>
>> if (*dev->dev_ops->vlan_strip_queue_set == NULL)
>> return -ENOTSUP;
>> +
>> (*dev->dev_ops->vlan_strip_queue_set)(dev, rx_queue_id, on);
>>
>> + rte_ethdev_trace_set_vlan_strip_on_queue(port_id, rx_queue_id,
>> on);
>> +
>> return 0;
>> }
>>
>> @@ -3862,14 +3887,18 @@ rte_eth_dev_set_vlan_ether_type(uint16_t
>> port_id,
>> uint16_t tpid)
>> {
>> struct rte_eth_dev *dev;
>> + int ret;
>>
>> RTE_ETH_VALID_PORTID_OR_ERR_RET(port_id, -ENODEV);
>> dev = &rte_eth_devices[port_id];
>>
>> if (*dev->dev_ops->vlan_tpid_set == NULL)
>> return -ENOTSUP;
>> - return eth_err(port_id, (*dev->dev_ops->vlan_tpid_set)(dev,
>> vlan_type,
>> - tpid));
>> + ret = eth_err(port_id, (*dev->dev_ops->vlan_tpid_set)(dev,
>> vlan_type, tpid));
>> +
>> + rte_ethdev_trace_set_vlan_ether_type(port_id, vlan_type, tpid, ret);
>> +
>> + return ret;
>> }
>>
>> int
>> @@ -3961,7 +3990,11 @@ rte_eth_dev_set_vlan_offload(uint16_t port_id,
>> int offload_mask)
>> dev->data->dev_conf.rxmode.offloads = orig_offloads;
>> }
>>
>> - return eth_err(port_id, ret);
>> + ret = eth_err(port_id, ret);
>> +
>> + rte_ethdev_trace_set_vlan_offload(port_id, offload_mask, ret);
>> +
>> + return ret;
>> }
>>
>> int
>> @@ -3987,6 +4020,8 @@ rte_eth_dev_get_vlan_offload(uint16_t port_id)
>> if (*dev_offloads & RTE_ETH_RX_OFFLOAD_QINQ_STRIP)
>> ret |= RTE_ETH_QINQ_STRIP_OFFLOAD;
>>
>> + rte_ethdev_trace_get_vlan_offload(port_id, ret);
>> +
>> return ret;
>> }
>>
>> @@ -3994,19 +4029,25 @@ int
>> rte_eth_dev_set_vlan_pvid(uint16_t port_id, uint16_t pvid, int on)
>> {
>> struct rte_eth_dev *dev;
>> + int ret;
>>
>> RTE_ETH_VALID_PORTID_OR_ERR_RET(port_id, -ENODEV);
>> dev = &rte_eth_devices[port_id];
>>
>> if (*dev->dev_ops->vlan_pvid_set == NULL)
>> return -ENOTSUP;
>> - return eth_err(port_id, (*dev->dev_ops->vlan_pvid_set)(dev, pvid,
>> on));
>> + ret = eth_err(port_id, (*dev->dev_ops->vlan_pvid_set)(dev, pvid,
>> on));
>> +
>> + rte_ethdev_trace_set_vlan_pvid(port_id, pvid, on, ret);
>> +
>> + return ret;
>> }
>>
>> int
>> rte_eth_dev_flow_ctrl_get(uint16_t port_id, struct rte_eth_fc_conf *fc_conf)
>> {
>> struct rte_eth_dev *dev;
>> + int ret;
>>
>> RTE_ETH_VALID_PORTID_OR_ERR_RET(port_id, -ENODEV);
>> dev = &rte_eth_devices[port_id];
>> @@ -4021,13 +4062,18 @@ rte_eth_dev_flow_ctrl_get(uint16_t port_id,
>> struct rte_eth_fc_conf *fc_conf)
>> if (*dev->dev_ops->flow_ctrl_get == NULL)
>> return -ENOTSUP;
>> memset(fc_conf, 0, sizeof(*fc_conf));
>> - return eth_err(port_id, (*dev->dev_ops->flow_ctrl_get)(dev,
>> fc_conf));
>> + ret = eth_err(port_id, (*dev->dev_ops->flow_ctrl_get)(dev, fc_conf));
>> +
>> + rte_ethdev_trace_flow_ctrl_get(port_id, fc_conf, ret);
>> +
>> + return ret;
>> }
>>
>> int
>> rte_eth_dev_flow_ctrl_set(uint16_t port_id, struct rte_eth_fc_conf *fc_conf)
>> {
>> struct rte_eth_dev *dev;
>> + int ret;
>>
>> RTE_ETH_VALID_PORTID_OR_ERR_RET(port_id, -ENODEV);
>> dev = &rte_eth_devices[port_id];
>> @@ -4046,7 +4092,11 @@ rte_eth_dev_flow_ctrl_set(uint16_t port_id,
>> struct rte_eth_fc_conf *fc_conf)
>>
>> if (*dev->dev_ops->flow_ctrl_set == NULL)
>> return -ENOTSUP;
>> - return eth_err(port_id, (*dev->dev_ops->flow_ctrl_set)(dev,
>> fc_conf));
>> + ret = eth_err(port_id, (*dev->dev_ops->flow_ctrl_set)(dev, fc_conf));
>> +
>> + rte_ethdev_trace_flow_ctrl_set(port_id, fc_conf, ret);
>> +
>> + return ret;
>> }
>>
>> int
>> @@ -4054,6 +4104,7 @@ rte_eth_dev_priority_flow_ctrl_set(uint16_t
>> port_id,
>> struct rte_eth_pfc_conf *pfc_conf)
>> {
>> struct rte_eth_dev *dev;
>> + int ret;
>>
>> RTE_ETH_VALID_PORTID_OR_ERR_RET(port_id, -ENODEV);
>> dev = &rte_eth_devices[port_id];
>> @@ -4071,9 +4122,14 @@ rte_eth_dev_priority_flow_ctrl_set(uint16_t
>> port_id,
>> }
>>
>> /* High water, low water validation are device specific */
>> - if (*dev->dev_ops->priority_flow_ctrl_set)
>> - return eth_err(port_id, (*dev->dev_ops-
>> >priority_flow_ctrl_set)
>> - (dev, pfc_conf));
>> + if (*dev->dev_ops->priority_flow_ctrl_set) {
>> + ret = eth_err(port_id, (*dev->dev_ops-
>> >priority_flow_ctrl_set)
>> + (dev, pfc_conf));
>> + rte_ethdev_trace_priority_flow_ctrl_set(port_id, pfc_conf,
>> ret);
>> +
>> + return ret;
>> + }
>> +
>> return -ENOTSUP;
>> }
>>
>> @@ -4132,6 +4188,7 @@
>> rte_eth_dev_priority_flow_ctrl_queue_info_get(uint16_t port_id,
>> struct rte_eth_pfc_queue_info *pfc_queue_info)
>> {
>> struct rte_eth_dev *dev;
>> + int ret;
>>
>> RTE_ETH_VALID_PORTID_OR_ERR_RET(port_id, -ENODEV);
>> dev = &rte_eth_devices[port_id];
>> @@ -4142,9 +4199,13 @@
>> rte_eth_dev_priority_flow_ctrl_queue_info_get(uint16_t port_id,
>> return -EINVAL;
>> }
>>
>> - if (*dev->dev_ops->priority_flow_ctrl_queue_info_get)
>> - return eth_err(port_id, (*dev->dev_ops-
>> >priority_flow_ctrl_queue_info_get)
>> + if (*dev->dev_ops->priority_flow_ctrl_queue_info_get) {
>> + ret = eth_err(port_id, (*dev->dev_ops-
>> >priority_flow_ctrl_queue_info_get)
>> (dev, pfc_queue_info));
>> + rte_ethdev_trace_priority_flow_ctrl_queue_info_get(port_id,
>> pfc_queue_info,
>> + ret);
>> + return ret;
>> + }
>> return -ENOTSUP;
>> }
>>
>> @@ -4213,10 +4274,16 @@
>> rte_eth_dev_priority_flow_ctrl_queue_configure(uint16_t port_id,
>> return ret;
>> }
>>
>> - if (*dev->dev_ops->priority_flow_ctrl_queue_config)
>> - return eth_err(port_id,
>> - (*dev->dev_ops-
>> >priority_flow_ctrl_queue_config)(
>> - dev, pfc_queue_conf));
>> + if (*dev->dev_ops->priority_flow_ctrl_queue_config) {
>> + ret = eth_err(port_id,
>> + (*dev->dev_ops-
>> >priority_flow_ctrl_queue_config)(
>> + dev, pfc_queue_conf));
>> +
>> rte_ethdev_trace_priority_flow_ctrl_queue_configure(port_id,
>> +
>> pfc_queue_conf,
>> + ret);
>> + return ret;
>> + }
>> +
>> return -ENOTSUP;
>> }
>>
>> @@ -4308,8 +4375,12 @@ rte_eth_dev_rss_reta_update(uint16_t port_id,
>>
>> if (*dev->dev_ops->reta_update == NULL)
>> return -ENOTSUP;
>> - return eth_err(port_id, (*dev->dev_ops->reta_update)(dev,
>> reta_conf,
>> - reta_size));
>> + ret = eth_err(port_id, (*dev->dev_ops->reta_update)(dev, reta_conf,
>> + reta_size));
>> +
>> + rte_ethdev_trace_rss_reta_update(port_id, reta_conf, reta_size,
>> ret);
>> +
>> + return ret;
>> }
>>
>> int
>> @@ -4337,8 +4408,12 @@ rte_eth_dev_rss_reta_query(uint16_t port_id,
>>
>> if (*dev->dev_ops->reta_query == NULL)
>> return -ENOTSUP;
>> - return eth_err(port_id, (*dev->dev_ops->reta_query)(dev, reta_conf,
>> - reta_size));
>> + ret = eth_err(port_id, (*dev->dev_ops->reta_query)(dev, reta_conf,
>> + reta_size));
>> +
>> + rte_ethdev_trace_rss_reta_query(port_id, reta_conf, reta_size, ret);
>> +
>> + return ret;
>> }
>>
>> int
>> @@ -4382,8 +4457,11 @@ rte_eth_dev_rss_hash_update(uint16_t port_id,
>>
>> if (*dev->dev_ops->rss_hash_update == NULL)
>> return -ENOTSUP;
>> - return eth_err(port_id, (*dev->dev_ops->rss_hash_update)(dev,
>> - rss_conf));
>> + ret = eth_err(port_id, (*dev->dev_ops->rss_hash_update)(dev,
>> rss_conf));
>> +
>> + rte_ethdev_trace_rss_hash_update(port_id, rss_conf, ret);
>> +
>> + return ret;
>> }
>>
>> int
>> @@ -4391,6 +4469,7 @@ rte_eth_dev_rss_hash_conf_get(uint16_t port_id,
>> struct rte_eth_rss_conf *rss_conf)
>> {
>> struct rte_eth_dev *dev;
>> + int ret;
>>
>> RTE_ETH_VALID_PORTID_OR_ERR_RET(port_id, -ENODEV);
>> dev = &rte_eth_devices[port_id];
>> @@ -4404,8 +4483,12 @@ rte_eth_dev_rss_hash_conf_get(uint16_t port_id,
>>
>> if (*dev->dev_ops->rss_hash_conf_get == NULL)
>> return -ENOTSUP;
>> - return eth_err(port_id, (*dev->dev_ops->rss_hash_conf_get)(dev,
>> - rss_conf));
>> + ret = eth_err(port_id, (*dev->dev_ops->rss_hash_conf_get)(dev,
>> + rss_conf));
>> +
>> + rte_ethdev_trace_rss_hash_conf_get(port_id, rss_conf, ret);
>> +
>> + return ret;
>> }
>>
>> int
>> @@ -4413,6 +4496,7 @@ rte_eth_dev_udp_tunnel_port_add(uint16_t
>> port_id,
>> struct rte_eth_udp_tunnel *udp_tunnel)
>> {
>> struct rte_eth_dev *dev;
>> + int ret;
>>
>> RTE_ETH_VALID_PORTID_OR_ERR_RET(port_id, -ENODEV);
>> dev = &rte_eth_devices[port_id];
>> @@ -4431,8 +4515,11 @@ rte_eth_dev_udp_tunnel_port_add(uint16_t
>> port_id,
>>
>> if (*dev->dev_ops->udp_tunnel_port_add == NULL)
>> return -ENOTSUP;
>> - return eth_err(port_id, (*dev->dev_ops-
>> >udp_tunnel_port_add)(dev,
>> - udp_tunnel));
>> + ret = eth_err(port_id, (*dev->dev_ops->udp_tunnel_port_add)(dev,
>> udp_tunnel));
>> +
>> + rte_ethdev_trace_udp_tunnel_port_add(port_id, udp_tunnel, ret);
>> +
>> + return ret;
>> }
>>
>> int
>> @@ -4440,6 +4527,7 @@ rte_eth_dev_udp_tunnel_port_delete(uint16_t
>> port_id,
>> struct rte_eth_udp_tunnel *udp_tunnel)
>> {
>> struct rte_eth_dev *dev;
>> + int ret;
>>
>> RTE_ETH_VALID_PORTID_OR_ERR_RET(port_id, -ENODEV);
>> dev = &rte_eth_devices[port_id];
>> @@ -4458,34 +4546,47 @@ rte_eth_dev_udp_tunnel_port_delete(uint16_t
>> port_id,
>>
>> if (*dev->dev_ops->udp_tunnel_port_del == NULL)
>> return -ENOTSUP;
>> - return eth_err(port_id, (*dev->dev_ops->udp_tunnel_port_del)(dev,
>> - udp_tunnel));
>> + ret = eth_err(port_id, (*dev->dev_ops->udp_tunnel_port_del)(dev,
>> udp_tunnel));
>> +
>> + rte_ethdev_trace_udp_tunnel_port_delete(port_id, udp_tunnel, ret);
>> +
>> + return ret;
>> }
>>
>> int
>> rte_eth_led_on(uint16_t port_id)
>> {
>> struct rte_eth_dev *dev;
>> + int ret;
>>
>> RTE_ETH_VALID_PORTID_OR_ERR_RET(port_id, -ENODEV);
>> dev = &rte_eth_devices[port_id];
>>
>> if (*dev->dev_ops->dev_led_on == NULL)
>> return -ENOTSUP;
>> - return eth_err(port_id, (*dev->dev_ops->dev_led_on)(dev));
>> + ret = eth_err(port_id, (*dev->dev_ops->dev_led_on)(dev));
>> +
>> + rte_eth_trace_led_on(port_id, ret);
>> +
>> + return ret;
>> }
>>
>> int
>> rte_eth_led_off(uint16_t port_id)
>> {
>> struct rte_eth_dev *dev;
>> + int ret;
>>
>> RTE_ETH_VALID_PORTID_OR_ERR_RET(port_id, -ENODEV);
>> dev = &rte_eth_devices[port_id];
>>
>> if (*dev->dev_ops->dev_led_off == NULL)
>> return -ENOTSUP;
>> - return eth_err(port_id, (*dev->dev_ops->dev_led_off)(dev));
>> + ret = eth_err(port_id, (*dev->dev_ops->dev_led_off)(dev));
>> +
>> + rte_eth_trace_led_off(port_id, ret);
>> +
>> + return ret;
>> }
>>
>> int
>> @@ -4510,6 +4611,8 @@ rte_eth_fec_get_capability(uint16_t port_id,
>> return -ENOTSUP;
>> ret = (*dev->dev_ops->fec_get_capability)(dev, speed_fec_capa,
>> num);
>>
>> + rte_eth_trace_fec_get_capability(port_id, speed_fec_capa, num,
>> ret);
>> +
>> return ret;
>> }
>>
>> @@ -4517,6 +4620,7 @@ int
>> rte_eth_fec_get(uint16_t port_id, uint32_t *fec_capa)
>> {
>> struct rte_eth_dev *dev;
>> + int ret;
>>
>> RTE_ETH_VALID_PORTID_OR_ERR_RET(port_id, -ENODEV);
>> dev = &rte_eth_devices[port_id];
>> @@ -4530,20 +4634,29 @@ rte_eth_fec_get(uint16_t port_id, uint32_t
>> *fec_capa)
>>
>> if (*dev->dev_ops->fec_get == NULL)
>> return -ENOTSUP;
>> - return eth_err(port_id, (*dev->dev_ops->fec_get)(dev, fec_capa));
>> + ret = eth_err(port_id, (*dev->dev_ops->fec_get)(dev, fec_capa));
>> +
>> + rte_eth_trace_fec_get(port_id, fec_capa, ret);
>> +
>> + return ret;
>> }
>>
>> int
>> rte_eth_fec_set(uint16_t port_id, uint32_t fec_capa)
>> {
>> struct rte_eth_dev *dev;
>> + int ret;
>>
>> RTE_ETH_VALID_PORTID_OR_ERR_RET(port_id, -ENODEV);
>> dev = &rte_eth_devices[port_id];
>>
>> if (*dev->dev_ops->fec_set == NULL)
>> return -ENOTSUP;
>> - return eth_err(port_id, (*dev->dev_ops->fec_set)(dev, fec_capa));
>> + ret = eth_err(port_id, (*dev->dev_ops->fec_set)(dev, fec_capa));
>> +
>> + rte_eth_trace_fec_set(port_id, fec_capa, ret);
>> +
>> + return ret;
>> }
>>
>> /*
>> @@ -4631,7 +4744,11 @@ rte_eth_dev_mac_addr_add(uint16_t port_id,
>> struct rte_ether_addr *addr,
>> dev->data->mac_pool_sel[index] |= RTE_BIT64(pool);
>> }
>>
>> - return eth_err(port_id, ret);
>> + ret = eth_err(port_id, ret);
>> +
>> + rte_ethdev_trace_mac_addr_add(port_id, addr, pool, ret);
>> +
>> + return ret;
>> }
>>
>> int
>> @@ -4671,6 +4788,8 @@ rte_eth_dev_mac_addr_remove(uint16_t port_id,
>> struct rte_ether_addr *addr)
>> /* reset pool bitmap */
>> dev->data->mac_pool_sel[index] = 0;
>>
>> + rte_ethdev_trace_mac_addr_remove(port_id, addr);
>> +
>> return 0;
>> }
>>
>> @@ -4703,6 +4822,8 @@ rte_eth_dev_default_mac_addr_set(uint16_t
>> port_id, struct rte_ether_addr *addr)
>> /* Update default address in NIC data structure */
>> rte_ether_addr_copy(addr, &dev->data->mac_addrs[0]);
>>
>> + rte_ethdev_trace_default_mac_addr_set(port_id, addr);
>> +
>> return 0;
>> }
>>
>> @@ -4793,21 +4914,29 @@ rte_eth_dev_uc_hash_table_set(uint16_t
>> port_id, struct rte_ether_addr *addr,
>> &dev->data-
>> >hash_mac_addrs[index]);
>> }
>>
>> - return eth_err(port_id, ret);
>> + ret = eth_err(port_id, ret);
>> +
>> + rte_ethdev_trace_uc_hash_table_set(port_id, on, ret);
>> +
>> + return ret;
>> }
>>
>> int
>> rte_eth_dev_uc_all_hash_table_set(uint16_t port_id, uint8_t on)
>> {
>> struct rte_eth_dev *dev;
>> + int ret;
>>
>> RTE_ETH_VALID_PORTID_OR_ERR_RET(port_id, -ENODEV);
>> dev = &rte_eth_devices[port_id];
>>
>> if (*dev->dev_ops->uc_all_hash_table_set == NULL)
>> return -ENOTSUP;
>> - return eth_err(port_id, (*dev->dev_ops-
>> >uc_all_hash_table_set)(dev,
>> - on));
>> + ret = eth_err(port_id, (*dev->dev_ops->uc_all_hash_table_set)(dev,
>> on));
>> +
>> + rte_ethdev_trace_uc_all_hash_table_set(port_id, on, ret);
>> +
>> + return ret;
>> }
>>
>> int rte_eth_set_queue_rate_limit(uint16_t port_id, uint16_t queue_idx,
>> @@ -4843,14 +4972,18 @@ int rte_eth_set_queue_rate_limit(uint16_t
>> port_id, uint16_t queue_idx,
>>
>> if (*dev->dev_ops->set_queue_rate_limit == NULL)
>> return -ENOTSUP;
>> - return eth_err(port_id, (*dev->dev_ops->set_queue_rate_limit)(dev,
>> - queue_idx, tx_rate));
>> + ret = eth_err(port_id, (*dev->dev_ops->set_queue_rate_limit)(dev,
>> queue_idx, tx_rate));
>> +
>> + rte_eth_trace_set_queue_rate_limit(port_id, queue_idx, tx_rate,
>> ret);
>> +
>> + return ret;
>> }
>>
>> int rte_eth_rx_avail_thresh_set(uint16_t port_id, uint16_t queue_id,
>> uint8_t avail_thresh)
>> {
>> struct rte_eth_dev *dev;
>> + int ret;
>>
>> RTE_ETH_VALID_PORTID_OR_ERR_RET(port_id, -ENODEV);
>> dev = &rte_eth_devices[port_id];
>> @@ -4870,14 +5003,19 @@ int rte_eth_rx_avail_thresh_set(uint16_t
>> port_id, uint16_t queue_id,
>> }
>> if (*dev->dev_ops->rx_queue_avail_thresh_set == NULL)
>> return -ENOTSUP;
>> - return eth_err(port_id, (*dev->dev_ops-
>> >rx_queue_avail_thresh_set)(dev,
>> - queue_id,
>> avail_thresh));
>> + ret = eth_err(port_id, (*dev->dev_ops-
>> >rx_queue_avail_thresh_set)(dev, queue_id,
>> +
>> avail_thresh));
>> +
>> + rte_eth_trace_rx_avail_thresh_set(port_id, queue_id, avail_thresh,
>> ret);
>> +
>> + return ret;
>> }
>>
>> int rte_eth_rx_avail_thresh_query(uint16_t port_id, uint16_t *queue_id,
>> uint8_t *avail_thresh)
>> {
>> struct rte_eth_dev *dev;
>> + int ret;
>>
>> RTE_ETH_VALID_PORTID_OR_ERR_RET(port_id, -ENODEV);
>> dev = &rte_eth_devices[port_id];
>> @@ -4889,8 +5027,12 @@ int rte_eth_rx_avail_thresh_query(uint16_t
>> port_id, uint16_t *queue_id,
>>
>> if (*dev->dev_ops->rx_queue_avail_thresh_query == NULL)
>> return -ENOTSUP;
>> - return eth_err(port_id, (*dev->dev_ops-
>> >rx_queue_avail_thresh_query)(dev,
>> - queue_id,
>> avail_thresh));
>> + ret = eth_err(port_id, (*dev->dev_ops-
>> >rx_queue_avail_thresh_query)(dev, queue_id,
>> +
>> avail_thresh));
>> +
>> + rte_eth_trace_rx_avail_thresh_query(port_id, *queue_id, ret);
>> +
>> + return ret;
>> }
>>
>> RTE_INIT(eth_dev_init_fp_ops)
>> @@ -4972,6 +5114,10 @@ rte_eth_dev_callback_register(uint16_t port_id,
>> } while (++next_port <= last_port);
>>
>> rte_spinlock_unlock(ð_dev_cb_lock);
>> +
>> + rte_ethdev_trace_callback_register(port_id, event, cb_fn, cb_arg,
>> next_port,
>> + last_port);
>> +
>> return 0;
>> }
>>
>> @@ -5033,6 +5179,9 @@ rte_eth_dev_callback_unregister(uint16_t port_id,
>> } while (++next_port <= last_port);
>>
>> rte_spinlock_unlock(ð_dev_cb_lock);
>> +
>> + rte_ethdev_trace_callback_unregister(port_id, event, cb_fn, cb_arg,
>> ret);
>> +
>> return ret;
>> }
>>
>> @@ -5062,6 +5211,9 @@ rte_eth_dev_rx_intr_ctl(uint16_t port_id, int epfd,
>> int op, void *data)
>> for (qid = 0; qid < dev->data->nb_rx_queues; qid++) {
>> vec = rte_intr_vec_list_index_get(intr_handle, qid);
>> rc = rte_intr_rx_ctl(intr_handle, epfd, op, vec, data);
>> +
>> + rte_ethdev_trace_rx_intr_ctl(port_id, qid, epfd, op, data, rc);
>> +
>> if (rc && rc != -EEXIST) {
>> RTE_ETHDEV_LOG(ERR,
>> "p %u q %u Rx ctl error op %d epfd %d vec
>> %u\n",
>> @@ -5105,6 +5257,8 @@ rte_eth_dev_rx_intr_ctl_q_get_fd(uint16_t
>> port_id, uint16_t queue_id)
>> (vec - RTE_INTR_VEC_RXTX_OFFSET) : vec;
>> fd = rte_intr_efds_index_get(intr_handle, efd_idx);
>>
>> + rte_ethdev_trace_rx_intr_ctl_q_get_fd(port_id, queue_id, fd);
>> +
>> return fd;
>> }
>>
>> @@ -5138,6 +5292,9 @@ rte_eth_dev_rx_intr_ctl_q(uint16_t port_id,
>> uint16_t queue_id,
>>
>> vec = rte_intr_vec_list_index_get(intr_handle, queue_id);
>> rc = rte_intr_rx_ctl(intr_handle, epfd, op, vec, data);
>> +
>> + rte_ethdev_trace_rx_intr_ctl_q(port_id, queue_id, epfd, op, data,
>> rc);
>> +
>> if (rc && rc != -EEXIST) {
>> RTE_ETHDEV_LOG(ERR,
>> "p %u q %u Rx ctl error op %d epfd %d vec %u\n",
>> @@ -5164,7 +5321,11 @@ rte_eth_dev_rx_intr_enable(uint16_t port_id,
>>
>> if (*dev->dev_ops->rx_queue_intr_enable == NULL)
>> return -ENOTSUP;
>> - return eth_err(port_id, (*dev->dev_ops-
>> >rx_queue_intr_enable)(dev, queue_id));
>> + ret = eth_err(port_id, (*dev->dev_ops->rx_queue_intr_enable)(dev,
>> queue_id));
>> +
>> + rte_ethdev_trace_rx_intr_enable(port_id, queue_id, ret);
>> +
>> + return ret;
>> }
>>
>> int
>> @@ -5183,7 +5344,11 @@ rte_eth_dev_rx_intr_disable(uint16_t port_id,
>>
>> if (*dev->dev_ops->rx_queue_intr_disable == NULL)
>> return -ENOTSUP;
>> - return eth_err(port_id, (*dev->dev_ops-
>> >rx_queue_intr_disable)(dev, queue_id));
>> + ret = eth_err(port_id, (*dev->dev_ops->rx_queue_intr_disable)(dev,
>> queue_id));
>> +
>> + rte_ethdev_trace_rx_intr_disable(port_id, queue_id, ret);
>> +
>> + return ret;
>> }
>>
>>
>> @@ -5241,6 +5406,8 @@ rte_eth_add_rx_callback(uint16_t port_id,
>> uint16_t queue_id,
>> }
>> rte_spinlock_unlock(ð_dev_rx_cb_lock);
>>
>> + rte_eth_trace_add_rx_callback(port_id, queue_id, fn, user_param,
>> cb);
>> +
>> return cb;
>> }
>>
>> @@ -5280,6 +5447,8 @@ rte_eth_add_first_rx_callback(uint16_t port_id,
>> uint16_t queue_id,
>> cb, __ATOMIC_RELEASE);
>> rte_spinlock_unlock(ð_dev_rx_cb_lock);
>>
>> + rte_eth_trace_add_first_rx_callback(port_id, queue_id, fn,
>> user_param, cb);
>> +
>> return cb;
>> }
>>
>> @@ -5339,6 +5508,8 @@ rte_eth_add_tx_callback(uint16_t port_id,
>> uint16_t queue_id,
>> }
>> rte_spinlock_unlock(ð_dev_tx_cb_lock);
>>
>> + rte_eth_trace_add_tx_callback(port_id, queue_id, fn, user_param,
>> cb);
>> +
>> return cb;
>> }
>>
>> @@ -5373,6 +5544,8 @@ rte_eth_remove_rx_callback(uint16_t port_id,
>> uint16_t queue_id,
>> }
>> rte_spinlock_unlock(ð_dev_rx_cb_lock);
>>
>> + rte_eth_trace_remove_rx_callback(port_id, queue_id, user_cb, ret);
>> +
>> return ret;
>> }
>>
>> @@ -5407,6 +5580,8 @@ rte_eth_remove_tx_callback(uint16_t port_id,
>> uint16_t queue_id,
>> }
>> rte_spinlock_unlock(ð_dev_tx_cb_lock);
>>
>> + rte_eth_trace_remove_tx_callback(port_id, queue_id, user_cb, ret);
>> +
>> return ret;
>> }
>>
>> @@ -5453,6 +5628,8 @@ rte_eth_rx_queue_info_get(uint16_t port_id,
>> uint16_t queue_id,
>> dev->dev_ops->rxq_info_get(dev, queue_id, qinfo);
>> qinfo->queue_state = dev->data->rx_queue_state[queue_id];
>>
>> + rte_eth_trace_rx_queue_info_get(port_id, queue_id, qinfo);
>> +
>> return 0;
>> }
>>
>> @@ -5499,6 +5676,8 @@ rte_eth_tx_queue_info_get(uint16_t port_id,
>> uint16_t queue_id,
>> dev->dev_ops->txq_info_get(dev, queue_id, qinfo);
>> qinfo->queue_state = dev->data->tx_queue_state[queue_id];
>>
>> + rte_eth_trace_tx_queue_info_get(port_id, queue_id, qinfo);
>> +
>> return 0;
>> }
>>
>> @@ -5507,6 +5686,7 @@ rte_eth_rx_burst_mode_get(uint16_t port_id,
>> uint16_t queue_id,
>> struct rte_eth_burst_mode *mode)
>> {
>> struct rte_eth_dev *dev;
>> + int ret;
>>
>> RTE_ETH_VALID_PORTID_OR_ERR_RET(port_id, -ENODEV);
>> dev = &rte_eth_devices[port_id];
>> @@ -5526,8 +5706,11 @@ rte_eth_rx_burst_mode_get(uint16_t port_id,
>> uint16_t queue_id,
>> if (*dev->dev_ops->rx_burst_mode_get == NULL)
>> return -ENOTSUP;
>> memset(mode, 0, sizeof(*mode));
>> - return eth_err(port_id,
>> - dev->dev_ops->rx_burst_mode_get(dev, queue_id,
>> mode));
>> + ret = eth_err(port_id, dev->dev_ops->rx_burst_mode_get(dev,
>> queue_id, mode));
>> +
>> + rte_eth_trace_rx_burst_mode_get(port_id, queue_id, mode, ret);
>> +
>> + return ret;
>> }
>>
>> int
>> @@ -5535,6 +5718,7 @@ rte_eth_tx_burst_mode_get(uint16_t port_id,
>> uint16_t queue_id,
>> struct rte_eth_burst_mode *mode)
>> {
>> struct rte_eth_dev *dev;
>> + int ret;
>>
>> RTE_ETH_VALID_PORTID_OR_ERR_RET(port_id, -ENODEV);
>> dev = &rte_eth_devices[port_id];
>> @@ -5554,8 +5738,11 @@ rte_eth_tx_burst_mode_get(uint16_t port_id,
>> uint16_t queue_id,
>> if (*dev->dev_ops->tx_burst_mode_get == NULL)
>> return -ENOTSUP;
>> memset(mode, 0, sizeof(*mode));
>> - return eth_err(port_id,
>> - dev->dev_ops->tx_burst_mode_get(dev, queue_id,
>> mode));
>> + ret = eth_err(port_id, dev->dev_ops->tx_burst_mode_get(dev,
>> queue_id, mode));
>> +
>> + rte_eth_trace_tx_burst_mode_get(port_id, queue_id, mode, ret);
>> +
>> + return ret;
>> }
>>
>> int
>> @@ -5563,6 +5750,7 @@ rte_eth_get_monitor_addr(uint16_t port_id,
>> uint16_t queue_id,
>> struct rte_power_monitor_cond *pmc)
>> {
>> struct rte_eth_dev *dev;
>> + int ret;
>>
>> RTE_ETH_VALID_PORTID_OR_ERR_RET(port_id, -ENODEV);
>> dev = &rte_eth_devices[port_id];
>> @@ -5581,8 +5769,11 @@ rte_eth_get_monitor_addr(uint16_t port_id,
>> uint16_t queue_id,
>>
>> if (*dev->dev_ops->get_monitor_addr == NULL)
>> return -ENOTSUP;
>> - return eth_err(port_id,
>> - dev->dev_ops->get_monitor_addr(dev->data-
>> >rx_queues[queue_id], pmc));
>> + ret = eth_err(port_id, dev->dev_ops->get_monitor_addr(dev->data-
>> >rx_queues[queue_id], pmc));
>> +
>> + rte_eth_trace_get_monitor_addr(port_id, queue_id, pmc, ret);
>> +
>> + return ret;
>> }
>>
>> int
>> @@ -5591,40 +5782,54 @@ rte_eth_dev_set_mc_addr_list(uint16_t port_id,
>> uint32_t nb_mc_addr)
>> {
>> struct rte_eth_dev *dev;
>> + int ret;
>>
>> RTE_ETH_VALID_PORTID_OR_ERR_RET(port_id, -ENODEV);
>> dev = &rte_eth_devices[port_id];
>>
>> if (*dev->dev_ops->set_mc_addr_list == NULL)
>> return -ENOTSUP;
>> - return eth_err(port_id, dev->dev_ops->set_mc_addr_list(dev,
>> - mc_addr_set, nb_mc_addr));
>> + ret = eth_err(port_id, dev->dev_ops->set_mc_addr_list(dev,
>> mc_addr_set, nb_mc_addr));
>> +
>> + rte_ethdev_trace_set_mc_addr_list(port_id, mc_addr_set,
>> nb_mc_addr, ret);
>> +
>> + return ret;
>> }
>>
>> int
>> rte_eth_timesync_enable(uint16_t port_id)
>> {
>> struct rte_eth_dev *dev;
>> + int ret;
>>
>> RTE_ETH_VALID_PORTID_OR_ERR_RET(port_id, -ENODEV);
>> dev = &rte_eth_devices[port_id];
>>
>> if (*dev->dev_ops->timesync_enable == NULL)
>> return -ENOTSUP;
>> - return eth_err(port_id, (*dev->dev_ops->timesync_enable)(dev));
>> + ret = eth_err(port_id, (*dev->dev_ops->timesync_enable)(dev));
>> +
>> + rte_eth_trace_timesync_enable(port_id, ret);
>> +
>> + return ret;
>> }
>>
>> int
>> rte_eth_timesync_disable(uint16_t port_id)
>> {
>> struct rte_eth_dev *dev;
>> + int ret;
>>
>> RTE_ETH_VALID_PORTID_OR_ERR_RET(port_id, -ENODEV);
>> dev = &rte_eth_devices[port_id];
>>
>> if (*dev->dev_ops->timesync_disable == NULL)
>> return -ENOTSUP;
>> - return eth_err(port_id, (*dev->dev_ops->timesync_disable)(dev));
>> + ret = eth_err(port_id, (*dev->dev_ops->timesync_disable)(dev));
>> +
>> + rte_eth_trace_timesync_disable(port_id, ret);
>> +
>> + return ret;
>> }
>>
>> int
>> @@ -5632,6 +5837,7 @@ rte_eth_timesync_read_rx_timestamp(uint16_t
>> port_id, struct timespec *timestamp,
>> uint32_t flags)
>> {
>> struct rte_eth_dev *dev;
>> + int ret;
>>
>> RTE_ETH_VALID_PORTID_OR_ERR_RET(port_id, -ENODEV);
>> dev = &rte_eth_devices[port_id];
>> @@ -5645,8 +5851,13 @@ rte_eth_timesync_read_rx_timestamp(uint16_t
>> port_id, struct timespec *timestamp,
>>
>> if (*dev->dev_ops->timesync_read_rx_timestamp == NULL)
>> return -ENOTSUP;
>> - return eth_err(port_id, (*dev->dev_ops-
>> >timesync_read_rx_timestamp)
>> - (dev, timestamp, flags));
>> +
>> + ret = eth_err(port_id, (*dev->dev_ops-
>> >timesync_read_rx_timestamp)
>> + (dev, timestamp, flags));
>> +
>> + rte_eth_trace_timesync_read_rx_timestamp(port_id, timestamp,
>> flags, ret);
>> +
>> + return ret;
>> }
>>
>> int
>> @@ -5654,6 +5865,7 @@ rte_eth_timesync_read_tx_timestamp(uint16_t
>> port_id,
>> struct timespec *timestamp)
>> {
>> struct rte_eth_dev *dev;
>> + int ret;
>>
>> RTE_ETH_VALID_PORTID_OR_ERR_RET(port_id, -ENODEV);
>> dev = &rte_eth_devices[port_id];
>> @@ -5667,27 +5879,39 @@ rte_eth_timesync_read_tx_timestamp(uint16_t
>> port_id,
>>
>> if (*dev->dev_ops->timesync_read_tx_timestamp == NULL)
>> return -ENOTSUP;
>> - return eth_err(port_id, (*dev->dev_ops-
>> >timesync_read_tx_timestamp)
>> - (dev, timestamp));
>> +
>> + ret = eth_err(port_id, (*dev->dev_ops-
>> >timesync_read_tx_timestamp)
>> + (dev, timestamp));
>> +
>> + rte_eth_trace_timesync_read_tx_timestamp(port_id, timestamp,
>> ret);
>> +
>> + return ret;
>> +
>> }
>>
>> int
>> rte_eth_timesync_adjust_time(uint16_t port_id, int64_t delta)
>> {
>> struct rte_eth_dev *dev;
>> + int ret;
>>
>> RTE_ETH_VALID_PORTID_OR_ERR_RET(port_id, -ENODEV);
>> dev = &rte_eth_devices[port_id];
>>
>> if (*dev->dev_ops->timesync_adjust_time == NULL)
>> return -ENOTSUP;
>> - return eth_err(port_id, (*dev->dev_ops->timesync_adjust_time)(dev,
>> delta));
>> + ret = eth_err(port_id, (*dev->dev_ops->timesync_adjust_time)(dev,
>> delta));
>> +
>> + rte_eth_trace_timesync_adjust_time(port_id, delta, ret);
>> +
>> + return ret;
>> }
>>
>> int
>> rte_eth_timesync_read_time(uint16_t port_id, struct timespec *timestamp)
>> {
>> struct rte_eth_dev *dev;
>> + int ret;
>>
>> RTE_ETH_VALID_PORTID_OR_ERR_RET(port_id, -ENODEV);
>> dev = &rte_eth_devices[port_id];
>> @@ -5701,14 +5925,19 @@ rte_eth_timesync_read_time(uint16_t port_id,
>> struct timespec *timestamp)
>>
>> if (*dev->dev_ops->timesync_read_time == NULL)
>> return -ENOTSUP;
>> - return eth_err(port_id, (*dev->dev_ops->timesync_read_time)(dev,
>> - timestamp));
>> +
>> + ret = eth_err(port_id, (*dev->dev_ops->timesync_read_time)(dev,
>> timestamp));
>> +
>> + rte_eth_trace_timesync_read_time(port_id, timestamp, ret);
>> +
>> + return ret;
>> }
>>
>> int
>> rte_eth_timesync_write_time(uint16_t port_id, const struct timespec
>> *timestamp)
>> {
>> struct rte_eth_dev *dev;
>> + int ret;
>>
>> RTE_ETH_VALID_PORTID_OR_ERR_RET(port_id, -ENODEV);
>> dev = &rte_eth_devices[port_id];
>> @@ -5722,14 +5951,18 @@ rte_eth_timesync_write_time(uint16_t port_id,
>> const struct timespec *timestamp)
>>
>> if (*dev->dev_ops->timesync_write_time == NULL)
>> return -ENOTSUP;
>> - return eth_err(port_id, (*dev->dev_ops->timesync_write_time)(dev,
>> - timestamp));
>> + ret = eth_err(port_id, (*dev->dev_ops->timesync_write_time)(dev,
>> timestamp));
>> +
>> + rte_eth_trace_timesync_write_time(port_id, timestamp, ret);
>> +
>> + return ret;
>> }
>>
>> int
>> rte_eth_read_clock(uint16_t port_id, uint64_t *clock)
>> {
>> struct rte_eth_dev *dev;
>> + int ret;
>>
>> RTE_ETH_VALID_PORTID_OR_ERR_RET(port_id, -ENODEV);
>> dev = &rte_eth_devices[port_id];
>> @@ -5742,13 +5975,18 @@ rte_eth_read_clock(uint16_t port_id, uint64_t
>> *clock)
>>
>> if (*dev->dev_ops->read_clock == NULL)
>> return -ENOTSUP;
>> - return eth_err(port_id, (*dev->dev_ops->read_clock)(dev, clock));
>> + ret = eth_err(port_id, (*dev->dev_ops->read_clock)(dev, clock));
>> +
>> + rte_eth_trace_read_clock(port_id, clock, ret);
>> +
>> + return ret;
>> }
>>
>> int
>> rte_eth_dev_get_reg_info(uint16_t port_id, struct rte_dev_reg_info *info)
>> {
>> struct rte_eth_dev *dev;
>> + int ret;
>>
>> RTE_ETH_VALID_PORTID_OR_ERR_RET(port_id, -ENODEV);
>> dev = &rte_eth_devices[port_id];
>> @@ -5762,26 +6000,36 @@ rte_eth_dev_get_reg_info(uint16_t port_id,
>> struct rte_dev_reg_info *info)
>>
>> if (*dev->dev_ops->get_reg == NULL)
>> return -ENOTSUP;
>> - return eth_err(port_id, (*dev->dev_ops->get_reg)(dev, info));
>> + ret = eth_err(port_id, (*dev->dev_ops->get_reg)(dev, info));
>> +
>> + rte_ethdev_trace_get_reg_info(port_id, info, ret);
>> +
>> + return ret;
>> }
>>
>> int
>> rte_eth_dev_get_eeprom_length(uint16_t port_id)
>> {
>> struct rte_eth_dev *dev;
>> + int ret;
>>
>> RTE_ETH_VALID_PORTID_OR_ERR_RET(port_id, -ENODEV);
>> dev = &rte_eth_devices[port_id];
>>
>> if (*dev->dev_ops->get_eeprom_length == NULL)
>> return -ENOTSUP;
>> - return eth_err(port_id, (*dev->dev_ops->get_eeprom_length)(dev));
>> + ret = eth_err(port_id, (*dev->dev_ops->get_eeprom_length)(dev));
>> +
>> + rte_ethdev_trace_get_eeprom_length(port_id, ret);
>> +
>> + return ret;
>> }
>>
>> int
>> rte_eth_dev_get_eeprom(uint16_t port_id, struct rte_dev_eeprom_info
>> *info)
>> {
>> struct rte_eth_dev *dev;
>> + int ret;
>>
>> RTE_ETH_VALID_PORTID_OR_ERR_RET(port_id, -ENODEV);
>> dev = &rte_eth_devices[port_id];
>> @@ -5795,13 +6043,18 @@ rte_eth_dev_get_eeprom(uint16_t port_id,
>> struct rte_dev_eeprom_info *info)
>>
>> if (*dev->dev_ops->get_eeprom == NULL)
>> return -ENOTSUP;
>> - return eth_err(port_id, (*dev->dev_ops->get_eeprom)(dev, info));
>> + ret = eth_err(port_id, (*dev->dev_ops->get_eeprom)(dev, info));
>> +
>> + rte_ethdev_trace_get_eeprom(port_id, info, ret);
>> +
>> + return ret;
>> }
>>
>> int
>> rte_eth_dev_set_eeprom(uint16_t port_id, struct rte_dev_eeprom_info
>> *info)
>> {
>> struct rte_eth_dev *dev;
>> + int ret;
>>
>> RTE_ETH_VALID_PORTID_OR_ERR_RET(port_id, -ENODEV);
>> dev = &rte_eth_devices[port_id];
>> @@ -5815,7 +6068,11 @@ rte_eth_dev_set_eeprom(uint16_t port_id, struct
>> rte_dev_eeprom_info *info)
>>
>> if (*dev->dev_ops->set_eeprom == NULL)
>> return -ENOTSUP;
>> - return eth_err(port_id, (*dev->dev_ops->set_eeprom)(dev, info));
>> + ret = eth_err(port_id, (*dev->dev_ops->set_eeprom)(dev, info));
>> +
>> + rte_ethdev_trace_set_eeprom(port_id, info, ret);
>> +
>> + return ret;
>> }
>>
>> int
>> @@ -5823,6 +6080,7 @@ rte_eth_dev_get_module_info(uint16_t port_id,
>> struct rte_eth_dev_module_info *modinfo)
>> {
>> struct rte_eth_dev *dev;
>> + int ret;
>>
>> RTE_ETH_VALID_PORTID_OR_ERR_RET(port_id, -ENODEV);
>> dev = &rte_eth_devices[port_id];
>> @@ -5836,7 +6094,11 @@ rte_eth_dev_get_module_info(uint16_t port_id,
>>
>> if (*dev->dev_ops->get_module_info == NULL)
>> return -ENOTSUP;
>> - return (*dev->dev_ops->get_module_info)(dev, modinfo);
>> + ret = (*dev->dev_ops->get_module_info)(dev, modinfo);
>> +
>> + rte_ethdev_trace_get_module_info(port_id, modinfo, ret);
>> +
>> + return ret;
>> }
>>
>> int
>> @@ -5844,6 +6106,7 @@ rte_eth_dev_get_module_eeprom(uint16_t
>> port_id,
>> struct rte_dev_eeprom_info *info)
>> {
>> struct rte_eth_dev *dev;
>> + int ret;
>>
>> RTE_ETH_VALID_PORTID_OR_ERR_RET(port_id, -ENODEV);
>> dev = &rte_eth_devices[port_id];
>> @@ -5871,7 +6134,11 @@ rte_eth_dev_get_module_eeprom(uint16_t
>> port_id,
>>
>> if (*dev->dev_ops->get_module_eeprom == NULL)
>> return -ENOTSUP;
>> - return (*dev->dev_ops->get_module_eeprom)(dev, info);
>> + ret = (*dev->dev_ops->get_module_eeprom)(dev, info);
>> +
>> + rte_ethdev_trace_get_module_eeprom(port_id, info, ret);
>> +
>> + return ret;
>> }
>>
>> int
>> @@ -5879,6 +6146,7 @@ rte_eth_dev_get_dcb_info(uint16_t port_id,
>> struct rte_eth_dcb_info *dcb_info)
>> {
>> struct rte_eth_dev *dev;
>> + int ret;
>>
>> RTE_ETH_VALID_PORTID_OR_ERR_RET(port_id, -ENODEV);
>> dev = &rte_eth_devices[port_id];
>> @@ -5894,7 +6162,11 @@ rte_eth_dev_get_dcb_info(uint16_t port_id,
>>
>> if (*dev->dev_ops->get_dcb_info == NULL)
>> return -ENOTSUP;
>> - return eth_err(port_id, (*dev->dev_ops->get_dcb_info)(dev,
>> dcb_info));
>> + ret = eth_err(port_id, (*dev->dev_ops->get_dcb_info)(dev,
>> dcb_info));
>> +
>> + rte_ethdev_trace_get_dcb_info(port_id, dcb_info, ret);
>> +
>> + return ret;
>> }
>>
>> static void
>> @@ -5930,6 +6202,8 @@ rte_eth_dev_adjust_nb_rx_tx_desc(uint16_t
>> port_id,
>> if (nb_tx_desc != NULL)
>> eth_dev_adjust_nb_desc(nb_tx_desc,
>> &dev_info.tx_desc_lim);
>>
>> + rte_ethdev_trace_adjust_nb_rx_tx_desc(port_id);
>> +
>> return 0;
>> }
>>
>> @@ -5938,6 +6212,7 @@ rte_eth_dev_hairpin_capability_get(uint16_t
>> port_id,
>> struct rte_eth_hairpin_cap *cap)
>> {
>> struct rte_eth_dev *dev;
>> + int ret;
>>
>> RTE_ETH_VALID_PORTID_OR_ERR_RET(port_id, -ENODEV);
>> dev = &rte_eth_devices[port_id];
>> @@ -5952,13 +6227,18 @@ rte_eth_dev_hairpin_capability_get(uint16_t
>> port_id,
>> if (*dev->dev_ops->hairpin_cap_get == NULL)
>> return -ENOTSUP;
>> memset(cap, 0, sizeof(*cap));
>> - return eth_err(port_id, (*dev->dev_ops->hairpin_cap_get)(dev,
>> cap));
>> + ret = eth_err(port_id, (*dev->dev_ops->hairpin_cap_get)(dev, cap));
>> +
>> + rte_ethdev_trace_hairpin_capability_get(port_id, cap, ret);
>> +
>> + return ret;
>> }
>>
>> int
>> rte_eth_dev_pool_ops_supported(uint16_t port_id, const char *pool)
>> {
>> struct rte_eth_dev *dev;
>> + int ret;
>>
>> RTE_ETH_VALID_PORTID_OR_ERR_RET(port_id, -ENODEV);
>> dev = &rte_eth_devices[port_id];
>> @@ -5973,7 +6253,11 @@ rte_eth_dev_pool_ops_supported(uint16_t
>> port_id, const char *pool)
>> if (*dev->dev_ops->pool_ops_supported == NULL)
>> return 1; /* all pools are supported */
>>
>> - return (*dev->dev_ops->pool_ops_supported)(dev, pool);
>> + ret = (*dev->dev_ops->pool_ops_supported)(dev, pool);
>> +
>> + rte_ethdev_trace_pool_ops_supported(port_id, pool, ret);
>> +
>> + return ret;
>> }
>>
>> static int
>> @@ -6263,19 +6547,25 @@ rte_eth_representor_info_get(uint16_t port_id,
>> struct rte_eth_representor_info *info)
>> {
>> struct rte_eth_dev *dev;
>> + int ret;
>>
>> RTE_ETH_VALID_PORTID_OR_ERR_RET(port_id, -ENODEV);
>> dev = &rte_eth_devices[port_id];
>>
>> if (*dev->dev_ops->representor_info_get == NULL)
>> return -ENOTSUP;
>> - return eth_err(port_id, (*dev->dev_ops->representor_info_get)(dev,
>> info));
>> + ret = eth_err(port_id, (*dev->dev_ops->representor_info_get)(dev,
>> info));
>> +
>> + rte_eth_trace_representor_info_get(port_id, info, ret);
>> +
>> + return ret;
>> }
>>
>> int
>> rte_eth_rx_metadata_negotiate(uint16_t port_id, uint64_t *features)
>> {
>> struct rte_eth_dev *dev;
>> + int ret;
>>
>> RTE_ETH_VALID_PORTID_OR_ERR_RET(port_id, -ENODEV);
>> dev = &rte_eth_devices[port_id];
>> @@ -6294,8 +6584,11 @@ rte_eth_rx_metadata_negotiate(uint16_t port_id,
>> uint64_t *features)
>>
>> if (*dev->dev_ops->rx_metadata_negotiate == NULL)
>> return -ENOTSUP;
>> - return eth_err(port_id,
>> - (*dev->dev_ops->rx_metadata_negotiate)(dev,
>> features));
>> + ret = eth_err(port_id, (*dev->dev_ops->rx_metadata_negotiate)(dev,
>> features));
>> +
>> + rte_eth_trace_rx_metadata_negotiate(port_id, features, *features,
>> ret);
>> +
>> + return ret;
>> }
>>
>> int
>> @@ -6303,6 +6596,7 @@ rte_eth_ip_reassembly_capability_get(uint16_t
>> port_id,
>> struct rte_eth_ip_reassembly_params *reassembly_capa)
>> {
>> struct rte_eth_dev *dev;
>> + int ret;
>>
>> RTE_ETH_VALID_PORTID_OR_ERR_RET(port_id, -ENODEV);
>> dev = &rte_eth_devices[port_id];
>> @@ -6324,8 +6618,12 @@ rte_eth_ip_reassembly_capability_get(uint16_t
>> port_id,
>> return -ENOTSUP;
>> memset(reassembly_capa, 0, sizeof(struct
>> rte_eth_ip_reassembly_params));
>>
>> - return eth_err(port_id, (*dev->dev_ops-
>> >ip_reassembly_capability_get)
>> + ret = eth_err(port_id, (*dev->dev_ops-
>> >ip_reassembly_capability_get)
>> (dev, reassembly_capa));
>> +
>> + rte_eth_trace_ip_reassembly_capability_get(port_id,
>> reassembly_capa, ret);
>> +
>> + return ret;
>> }
>>
>> int
>> @@ -6333,6 +6631,7 @@ rte_eth_ip_reassembly_conf_get(uint16_t port_id,
>> struct rte_eth_ip_reassembly_params *conf)
>> {
>> struct rte_eth_dev *dev;
>> + int ret;
>>
>> RTE_ETH_VALID_PORTID_OR_ERR_RET(port_id, -ENODEV);
>> dev = &rte_eth_devices[port_id];
>> @@ -6353,8 +6652,11 @@ rte_eth_ip_reassembly_conf_get(uint16_t
>> port_id,
>> if (*dev->dev_ops->ip_reassembly_conf_get == NULL)
>> return -ENOTSUP;
>> memset(conf, 0, sizeof(struct rte_eth_ip_reassembly_params));
>> - return eth_err(port_id,
>> - (*dev->dev_ops->ip_reassembly_conf_get)(dev, conf));
>> + ret = eth_err(port_id, (*dev->dev_ops-
>> >ip_reassembly_conf_get)(dev, conf));
>> +
>> + rte_eth_trace_ip_reassembly_conf_get(port_id, conf, ret);
>> +
>> + return ret;
>> }
>>
>> int
>> @@ -6362,6 +6664,7 @@ rte_eth_ip_reassembly_conf_set(uint16_t port_id,
>> const struct rte_eth_ip_reassembly_params *conf)
>> {
>> struct rte_eth_dev *dev;
>> + int ret;
>>
>> RTE_ETH_VALID_PORTID_OR_ERR_RET(port_id, -ENODEV);
>> dev = &rte_eth_devices[port_id];
>> @@ -6390,8 +6693,11 @@ rte_eth_ip_reassembly_conf_set(uint16_t
>> port_id,
>>
>> if (*dev->dev_ops->ip_reassembly_conf_set == NULL)
>> return -ENOTSUP;
>> - return eth_err(port_id,
>> - (*dev->dev_ops->ip_reassembly_conf_set)(dev, conf));
>> + ret = eth_err(port_id, (*dev->dev_ops-
>> >ip_reassembly_conf_set)(dev, conf));
>> +
>> + rte_eth_trace_ip_reassembly_conf_set(port_id, conf, ret);
>> +
>> + return ret;
>> }
>>
>> int
>> @@ -6489,8 +6795,10 @@
>> rte_eth_buffer_split_get_supported_hdr_ptypes(uint16_t port_id, uint32_t
>> *ptypes
>> return 0;
>>
>> for (i = 0, j = 0; all_types[i] != RTE_PTYPE_UNKNOWN; ++i) {
>> - if (j < num)
>> + if (j < num) {
>> ptypes[j] = all_types[i];
>> +
>> rte_eth_trace_buffer_split_get_supported_hdr_ptypes(port_id, j,
>> ptypes[j]);
>> + }
>> j++;
>> }
>>
>> diff --git a/lib/ethdev/rte_ethdev_cman.c b/lib/ethdev/rte_ethdev_cman.c
>> index 4a1bdd7bd0..c4d877d878 100644
>> --- a/lib/ethdev/rte_ethdev_cman.c
>> +++ b/lib/ethdev/rte_ethdev_cman.c
>> @@ -6,6 +6,8 @@
>>
>> #include <rte_errno.h>
>> #include "rte_ethdev.h"
>> +#include "rte_ethdev_trace.h"
>> +#include "rte_ethdev_trace_fp.h"
>> #include "ethdev_driver.h"
>> #include "ethdev_private.h"
>>
>> @@ -14,6 +16,7 @@ int
>> rte_eth_cman_info_get(uint16_t port_id, struct rte_eth_cman_info *info)
>> {
>> struct rte_eth_dev *dev;
>> + int ret;
>>
>> RTE_ETH_VALID_PORTID_OR_ERR_RET(port_id, -ENODEV);
>> dev = &rte_eth_devices[port_id];
>> @@ -29,7 +32,11 @@ rte_eth_cman_info_get(uint16_t port_id, struct
>> rte_eth_cman_info *info)
>> }
>>
>> memset(info, 0, sizeof(struct rte_eth_cman_info));
>> - return eth_err(port_id, (*dev->dev_ops->cman_info_get)(dev, info));
>> + ret = eth_err(port_id, (*dev->dev_ops->cman_info_get)(dev, info));
>> +
>> + rte_eth_trace_cman_info_get(port_id, info, ret);
>> +
>> + return ret;
>> }
>>
>> /* Initialize congestion management structure with default values */
>> @@ -37,6 +44,7 @@ int
>> rte_eth_cman_config_init(uint16_t port_id, struct rte_eth_cman_config
>> *config)
>> {
>> struct rte_eth_dev *dev;
>> + int ret;
>>
>> RTE_ETH_VALID_PORTID_OR_ERR_RET(port_id, -ENODEV);
>> dev = &rte_eth_devices[port_id];
>> @@ -52,7 +60,11 @@ rte_eth_cman_config_init(uint16_t port_id, struct
>> rte_eth_cman_config *config)
>> }
>>
>> memset(config, 0, sizeof(struct rte_eth_cman_config));
>> - return eth_err(port_id, (*dev->dev_ops->cman_config_init)(dev,
>> config));
>> + ret = eth_err(port_id, (*dev->dev_ops->cman_config_init)(dev,
>> config));
>> +
>> + rte_eth_trace_cman_config_init(port_id, config, ret);
>> +
>> + return ret;
>> }
>>
>> /* Configure congestion management on a port */
>> @@ -60,6 +72,7 @@ int
>> rte_eth_cman_config_set(uint16_t port_id, const struct
>> rte_eth_cman_config *config)
>> {
>> struct rte_eth_dev *dev;
>> + int ret;
>>
>> RTE_ETH_VALID_PORTID_OR_ERR_RET(port_id, -ENODEV);
>> dev = &rte_eth_devices[port_id];
>> @@ -74,7 +87,11 @@ rte_eth_cman_config_set(uint16_t port_id, const
>> struct rte_eth_cman_config *conf
>> return -ENOTSUP;
>> }
>>
>> - return eth_err(port_id, (*dev->dev_ops->cman_config_set)(dev,
>> config));
>> + ret = eth_err(port_id, (*dev->dev_ops->cman_config_set)(dev,
>> config));
>> +
>> + rte_eth_trace_cman_config_set(port_id, config, ret);
>> +
>> + return ret;
>> }
>>
>> /* Retrieve congestion management configuration of a port */
>> @@ -82,6 +99,7 @@ int
>> rte_eth_cman_config_get(uint16_t port_id, struct rte_eth_cman_config
>> *config)
>> {
>> struct rte_eth_dev *dev;
>> + int ret;
>>
>> RTE_ETH_VALID_PORTID_OR_ERR_RET(port_id, -ENODEV);
>> dev = &rte_eth_devices[port_id];
>> @@ -97,5 +115,9 @@ rte_eth_cman_config_get(uint16_t port_id, struct
>> rte_eth_cman_config *config)
>> }
>>
>> memset(config, 0, sizeof(struct rte_eth_cman_config));
>> - return eth_err(port_id, (*dev->dev_ops->cman_config_get)(dev,
>> config));
>> + ret = eth_err(port_id, (*dev->dev_ops->cman_config_get)(dev,
>> config));
>> +
>> + rte_eth_trace_cman_config_get(port_id, config, ret);
>> +
>> + return ret;
>> }
>> diff --git a/lib/ethdev/rte_ethdev_trace.h b/lib/ethdev/rte_ethdev_trace.h
>> index bc3b3d2a1b..4c66834290 100644
>> --- a/lib/ethdev/rte_ethdev_trace.h
>> +++ b/lib/ethdev/rte_ethdev_trace.h
>> @@ -373,6 +373,535 @@ RTE_TRACE_POINT(
>> rte_trace_point_emit_int(ret);
>> )
>>
>> +RTE_TRACE_POINT(
>> + rte_ethdev_trace_info_get,
>> + RTE_TRACE_POINT_ARGS(uint16_t port_id,
>> + struct rte_eth_dev_info *dev_info),
>> + rte_trace_point_emit_u16(port_id);
>> + rte_trace_point_emit_string(dev_info->driver_name);
>> + rte_trace_point_emit_u16(dev_info->min_mtu);
>> + rte_trace_point_emit_u16(dev_info->max_mtu);
>> + rte_trace_point_emit_u32(dev_info->min_rx_bufsize);
>> + rte_trace_point_emit_u32(dev_info->max_rx_pktlen);
>> + rte_trace_point_emit_u16(dev_info->max_rx_queues);
>> + rte_trace_point_emit_u16(dev_info->max_tx_queues);
>> + rte_trace_point_emit_u32(dev_info->max_mac_addrs);
>> + rte_trace_point_emit_u64(dev_info->rx_offload_capa);
>> + rte_trace_point_emit_u64(dev_info->tx_offload_capa);
>> + rte_trace_point_emit_u64(dev_info->rx_queue_offload_capa);
>> + rte_trace_point_emit_u64(dev_info->tx_queue_offload_capa);
>> + rte_trace_point_emit_u16(dev_info->reta_size);
>> + rte_trace_point_emit_u8(dev_info->hash_key_size);
>> + rte_trace_point_emit_u64(dev_info->flow_type_rss_offloads);
>> + rte_trace_point_emit_u16(dev_info->rx_desc_lim.nb_max);
>> + rte_trace_point_emit_u16(dev_info->rx_desc_lim.nb_min);
>> + rte_trace_point_emit_u16(dev_info->rx_desc_lim.nb_align);
>> + rte_trace_point_emit_u16(dev_info->tx_desc_lim.nb_max);
>> + rte_trace_point_emit_u16(dev_info->tx_desc_lim.nb_min);
>> + rte_trace_point_emit_u16(dev_info->tx_desc_lim.nb_align);
>> + rte_trace_point_emit_u32(dev_info->speed_capa);
>> + rte_trace_point_emit_u16(dev_info->nb_rx_queues);
>> + rte_trace_point_emit_u16(dev_info->nb_tx_queues);
>> + rte_trace_point_emit_u64(dev_info->dev_capa);
>> +)
>> +
>> +RTE_TRACE_POINT(
>> + rte_ethdev_trace_conf_get,
>> + RTE_TRACE_POINT_ARGS(uint16_t port_id, struct rte_eth_conf
>> *dev_conf),
>> + rte_trace_point_emit_u16(port_id);
>> + rte_trace_point_emit_ptr(dev_conf);
>> + rte_trace_point_emit_u32(dev_conf->link_speeds);
>> + rte_trace_point_emit_u64(dev_conf->rxmode.offloads);
>> + rte_trace_point_emit_u64(dev_conf->txmode.offloads);
>> + rte_trace_point_emit_u32(dev_conf->lpbk_mode);
>> +)
>> +
>> +RTE_TRACE_POINT(
>> + rte_ethdev_trace_set_ptypes,
>> + RTE_TRACE_POINT_ARGS(uint16_t port_id, int supported_num,
>> + unsigned int num, uint32_t set_ptypes),
>> + rte_trace_point_emit_u16(port_id);
>> + rte_trace_point_emit_int(supported_num);
>> + rte_trace_point_emit_u32(num);
>> + rte_trace_point_emit_u32(set_ptypes);
>> +)
>> +
>> +RTE_TRACE_POINT(
>> + rte_ethdev_trace_set_mtu,
>> + RTE_TRACE_POINT_ARGS(uint16_t port_id, uint16_t mtu, int ret),
>> + rte_trace_point_emit_u16(port_id);
>> + rte_trace_point_emit_u16(mtu);
>> + rte_trace_point_emit_int(ret);
>> +)
>> +
>> +RTE_TRACE_POINT(
>> + rte_ethdev_trace_vlan_filter,
>> + RTE_TRACE_POINT_ARGS(uint16_t port_id, uint16_t vlan_id, int on,
>> int ret),
>> + rte_trace_point_emit_u16(port_id);
>> + rte_trace_point_emit_u16(vlan_id);
>> + rte_trace_point_emit_int(on);
>> + rte_trace_point_emit_int(ret);
>> +)
>> +
>> +RTE_TRACE_POINT(
>> + rte_ethdev_trace_set_vlan_strip_on_queue,
>> + RTE_TRACE_POINT_ARGS(uint16_t port_id, uint16_t rx_queue_id,
>> + int on),
>> + rte_trace_point_emit_u16(port_id);
>> + rte_trace_point_emit_u16(rx_queue_id);
>> + rte_trace_point_emit_int(on);
>> +)
>> +
>> +RTE_TRACE_POINT(
>> + rte_ethdev_trace_set_vlan_ether_type,
>> + RTE_TRACE_POINT_ARGS(uint16_t port_id, enum rte_vlan_type
>> vlan_type,
>> + uint16_t tag_type, int ret),
>> + rte_trace_point_emit_u16(port_id);
>> + rte_trace_point_emit_int(vlan_type);
>> + rte_trace_point_emit_u16(tag_type);
>> + rte_trace_point_emit_int(ret);
>> +)
>> +
>> +RTE_TRACE_POINT(
>> + rte_ethdev_trace_set_vlan_offload,
>> + RTE_TRACE_POINT_ARGS(uint16_t port_id, int offload_mask, int ret),
>> + rte_trace_point_emit_u16(port_id);
>> + rte_trace_point_emit_int(offload_mask);
>> + rte_trace_point_emit_int(ret);
>> +)
>> +
>> +RTE_TRACE_POINT(
>> + rte_ethdev_trace_set_vlan_pvid,
>> + RTE_TRACE_POINT_ARGS(uint16_t port_id, uint16_t pvid, int on, int
>> ret),
>> + rte_trace_point_emit_u16(port_id);
>> + rte_trace_point_emit_u16(pvid);
>> + rte_trace_point_emit_int(on);
>> + rte_trace_point_emit_int(ret);
>> +)
>> +
>> +RTE_TRACE_POINT(
>> + rte_ethdev_trace_flow_ctrl_set,
>> + RTE_TRACE_POINT_ARGS(uint16_t port_id, struct rte_eth_fc_conf
>> *fc_conf, int ret),
>> + rte_trace_point_emit_u16(port_id);
>> + rte_trace_point_emit_u32(fc_conf->high_water);
>> + rte_trace_point_emit_u32(fc_conf->low_water);
>> + rte_trace_point_emit_u16(fc_conf->pause_time);
>> + rte_trace_point_emit_u16(fc_conf->send_xon);
>> + rte_trace_point_emit_int(fc_conf->mode);
>> + rte_trace_point_emit_u8(fc_conf->mac_ctrl_frame_fwd);
>> + rte_trace_point_emit_u8(fc_conf->autoneg);
>> + rte_trace_point_emit_int(ret);
>> +)
>> +
>> +RTE_TRACE_POINT(
>> + rte_ethdev_trace_priority_flow_ctrl_set,
>> + RTE_TRACE_POINT_ARGS(uint16_t port_id, struct rte_eth_pfc_conf
>> *pfc_conf,
>> + int ret),
>> + rte_trace_point_emit_u16(port_id);
>> + rte_trace_point_emit_u32(pfc_conf->fc.high_water);
>> + rte_trace_point_emit_u32(pfc_conf->fc.low_water);
>> + rte_trace_point_emit_u16(pfc_conf->fc.pause_time);
>> + rte_trace_point_emit_u16(pfc_conf->fc.send_xon);
>> + rte_trace_point_emit_int(pfc_conf->fc.mode);
>> + rte_trace_point_emit_u8(pfc_conf->fc.mac_ctrl_frame_fwd);
>> + rte_trace_point_emit_u8(pfc_conf->fc.autoneg);
>> + rte_trace_point_emit_u8(pfc_conf->priority);
>> + rte_trace_point_emit_int(ret);
>> +)
>> +
>> +RTE_TRACE_POINT(
>> + rte_ethdev_trace_priority_flow_ctrl_queue_configure,
>> + RTE_TRACE_POINT_ARGS(uint16_t port_id,
>> + struct rte_eth_pfc_queue_conf *pfc_queue_conf, int ret),
>> + rte_trace_point_emit_u16(port_id);
>> + rte_trace_point_emit_ptr(pfc_queue_conf);
>> + rte_trace_point_emit_int(pfc_queue_conf->mode);
>> + rte_trace_point_emit_u16(pfc_queue_conf->rx_pause.tx_qid);
>> + rte_trace_point_emit_u16(pfc_queue_conf->tx_pause.rx_qid);
>> + rte_trace_point_emit_int(ret);
>> +)
>> +
>> +RTE_TRACE_POINT(
>> + rte_ethdev_trace_rss_reta_update,
>> + RTE_TRACE_POINT_ARGS(uint16_t port_id,
>> + struct rte_eth_rss_reta_entry64 *reta_conf, uint16_t
>> reta_size,
>> + int ret),
>> + rte_trace_point_emit_u16(port_id);
>> + rte_trace_point_emit_ptr(reta_conf);
>> + rte_trace_point_emit_u16(reta_size);
>> + rte_trace_point_emit_int(ret);
>> +)
>> +
>> +RTE_TRACE_POINT(
>> + rte_ethdev_trace_rss_hash_update,
>> + RTE_TRACE_POINT_ARGS(uint16_t port_id, struct rte_eth_rss_conf
>> *rss_conf,
>> + int ret),
>> + rte_trace_point_emit_u16(port_id);
>> + rte_trace_point_emit_ptr(rss_conf->rss_key);
>> + rte_trace_point_emit_u8(rss_conf->rss_key_len);
>> + rte_trace_point_emit_u64(rss_conf->rss_hf);
>> + rte_trace_point_emit_int(ret);
>> +)
>> +
>> +RTE_TRACE_POINT(
>> + rte_ethdev_trace_udp_tunnel_port_add,
>> + RTE_TRACE_POINT_ARGS(uint16_t port_id, struct
>> rte_eth_udp_tunnel *tunnel_udp,
>> + int ret),
>> + rte_trace_point_emit_u16(port_id);
>> + rte_trace_point_emit_u16(tunnel_udp->udp_port);
>> + rte_trace_point_emit_u8(tunnel_udp->prot_type);
>> + rte_trace_point_emit_int(ret);
>> +)
>> +
>> +RTE_TRACE_POINT(
>> + rte_ethdev_trace_udp_tunnel_port_delete,
>> + RTE_TRACE_POINT_ARGS(uint16_t port_id, struct
>> rte_eth_udp_tunnel *tunnel_udp,
>> + int ret),
>> + rte_trace_point_emit_u16(port_id);
>> + rte_trace_point_emit_u16(tunnel_udp->udp_port);
>> + rte_trace_point_emit_u8(tunnel_udp->prot_type);
>> + rte_trace_point_emit_int(ret);
>> +)
>> +
>> +RTE_TRACE_POINT(
>> + rte_eth_trace_led_on,
>> + RTE_TRACE_POINT_ARGS(uint16_t port_id, int ret),
>> + rte_trace_point_emit_u16(port_id);
>> + rte_trace_point_emit_int(ret);
>> +)
>> +
>> +RTE_TRACE_POINT(
>> + rte_eth_trace_led_off,
>> + RTE_TRACE_POINT_ARGS(uint16_t port_id, int ret),
>> + rte_trace_point_emit_u16(port_id);
>> + rte_trace_point_emit_int(ret);
>> +)
>> +
>> +RTE_TRACE_POINT(
>> + rte_eth_trace_fec_get_capability,
>> + RTE_TRACE_POINT_ARGS(uint16_t port_id,
>> + struct rte_eth_fec_capa *speed_fec_capa,
>> + unsigned int num, int ret),
>> + rte_trace_point_emit_u16(port_id);
>> + rte_trace_point_emit_ptr(speed_fec_capa);
>> + rte_trace_point_emit_u32(speed_fec_capa->speed);
>> + rte_trace_point_emit_u32(speed_fec_capa->capa);
>> + rte_trace_point_emit_u32(num);
>> + rte_trace_point_emit_int(ret);
>> +)
>> +
>> +RTE_TRACE_POINT(
>> + rte_eth_trace_fec_set,
>> + RTE_TRACE_POINT_ARGS(uint16_t port_id, uint32_t fec_capa, int
>> ret),
>> + rte_trace_point_emit_u16(port_id);
>> + rte_trace_point_emit_u32(fec_capa);
>> + rte_trace_point_emit_int(ret);
>> +)
>> +
>> +RTE_TRACE_POINT(
>> + rte_ethdev_trace_mac_addr_add,
>> + RTE_TRACE_POINT_ARGS(uint16_t port_id, struct rte_ether_addr
>> *addr,
>> + uint32_t pool, int ret),
>> + uint16_t len = RTE_ETHER_ADDR_LEN;
>> +
>> + rte_trace_point_emit_u16(port_id);
>> + rte_trace_point_emit_ptr(addr);
>> + rte_trace_point_emit_blob(addr->addr_bytes, len);
>> + rte_trace_point_emit_u32(pool);
>> + rte_trace_point_emit_int(ret);
>> +)
>> +
>> +RTE_TRACE_POINT(
>> + rte_ethdev_trace_mac_addr_remove,
>> + RTE_TRACE_POINT_ARGS(uint16_t port_id, struct rte_ether_addr
>> *addr),
>> + uint16_t len = RTE_ETHER_ADDR_LEN;
>> +
>> + rte_trace_point_emit_u16(port_id);
>> + rte_trace_point_emit_ptr(addr);
>> + rte_trace_point_emit_blob(addr->addr_bytes, len);
>> +)
>> +
>> +RTE_TRACE_POINT(
>> + rte_ethdev_trace_default_mac_addr_set,
>> + RTE_TRACE_POINT_ARGS(uint16_t port_id, struct rte_ether_addr
>> *addr),
>> + uint16_t len = RTE_ETHER_ADDR_LEN;
>> +
>> + rte_trace_point_emit_u16(port_id);
>> + rte_trace_point_emit_ptr(addr);
>> + rte_trace_point_emit_blob(addr->addr_bytes, len);
>> +)
>> +
>> +RTE_TRACE_POINT(
>> + rte_ethdev_trace_uc_hash_table_set,
>> + RTE_TRACE_POINT_ARGS(uint16_t port_id, uint8_t on, int ret),
>> + rte_trace_point_emit_u16(port_id);
>> + rte_trace_point_emit_u8(on);
>> + rte_trace_point_emit_int(ret);
>> +)
>> +
>> +RTE_TRACE_POINT(
>> + rte_ethdev_trace_uc_all_hash_table_set,
>> + RTE_TRACE_POINT_ARGS(uint16_t port_id, uint8_t on, int ret),
>> + rte_trace_point_emit_u16(port_id);
>> + rte_trace_point_emit_u8(on);
>> + rte_trace_point_emit_int(ret);
>> +)
>> +
>> +RTE_TRACE_POINT(
>> + rte_eth_trace_set_queue_rate_limit,
>> + RTE_TRACE_POINT_ARGS(uint16_t port_id, uint16_t queue_idx,
>> + uint16_t tx_rate, int ret),
>> + rte_trace_point_emit_u16(port_id);
>> + rte_trace_point_emit_u16(queue_idx);
>> + rte_trace_point_emit_u16(tx_rate);
>> + rte_trace_point_emit_int(ret);
>> +)
>> +
>> +RTE_TRACE_POINT(
>> + rte_eth_trace_rx_avail_thresh_set,
>> + RTE_TRACE_POINT_ARGS(uint16_t port_id, uint16_t queue_id,
>> + uint8_t avail_thresh, int ret),
>> + rte_trace_point_emit_u16(port_id);
>> + rte_trace_point_emit_u16(queue_id);
>> + rte_trace_point_emit_u8(avail_thresh);
>> + rte_trace_point_emit_int(ret);
>> +)
>> +
>> +RTE_TRACE_POINT(
>> + rte_eth_trace_rx_avail_thresh_query,
>> + RTE_TRACE_POINT_ARGS(uint16_t port_id, uint16_t queue_id, int
>> ret),
>> + rte_trace_point_emit_u16(port_id);
>> + rte_trace_point_emit_u16(queue_id);
>> + rte_trace_point_emit_int(ret);
>> +)
>> +
>> +RTE_TRACE_POINT(
>> + rte_ethdev_trace_callback_register,
>> + RTE_TRACE_POINT_ARGS(uint16_t port_id, enum
>> rte_eth_event_type event,
>> + rte_eth_dev_cb_fn cb_fn, void *cb_arg, uint16_t next_port,
>> + uint16_t last_port),
>> + rte_trace_point_emit_u16(port_id);
>> + rte_trace_point_emit_int(event);
>> + rte_trace_point_emit_ptr(cb_fn);
>> + rte_trace_point_emit_ptr(cb_arg);
>> + rte_trace_point_emit_u16(next_port);
>> + rte_trace_point_emit_u16(last_port);
>> +)
>> +
>> +RTE_TRACE_POINT(
>> + rte_ethdev_trace_callback_unregister,
>> + RTE_TRACE_POINT_ARGS(uint16_t port_id, enum
>> rte_eth_event_type event,
>> + rte_eth_dev_cb_fn cb_fn, void *cb_arg, int ret),
>> + rte_trace_point_emit_u16(port_id);
>> + rte_trace_point_emit_int(event);
>> + rte_trace_point_emit_ptr(cb_fn);
>> + rte_trace_point_emit_ptr(cb_arg);
>> + rte_trace_point_emit_int(ret);
>> +)
>> +
>> +RTE_TRACE_POINT(
>> + rte_ethdev_trace_rx_intr_ctl,
>> + RTE_TRACE_POINT_ARGS(uint16_t port_id, uint16_t qid, int epfd, int
>> op,
>> + void *data, int rc),
>> + rte_trace_point_emit_u16(port_id);
>> + rte_trace_point_emit_u16(qid);
>> + rte_trace_point_emit_int(epfd);
>> + rte_trace_point_emit_int(op);
>> + rte_trace_point_emit_ptr(data);
>> + rte_trace_point_emit_int(rc);
>> +)
>> +
>> +RTE_TRACE_POINT(
>> + rte_ethdev_trace_rx_intr_ctl_q,
>> + RTE_TRACE_POINT_ARGS(uint16_t port_id, uint16_t queue_id, int
>> epfd,
>> + int op, void *data, int rc),
>> + rte_trace_point_emit_u16(port_id);
>> + rte_trace_point_emit_u16(queue_id);
>> + rte_trace_point_emit_int(epfd);
>> + rte_trace_point_emit_int(op);
>> + rte_trace_point_emit_ptr(data);
>> + rte_trace_point_emit_int(rc);
>> +)
>> +
>> +RTE_TRACE_POINT(
>> + rte_ethdev_trace_rx_intr_enable,
>> + RTE_TRACE_POINT_ARGS(uint16_t port_id, uint16_t queue_id, int
>> ret),
>> + rte_trace_point_emit_u16(port_id);
>> + rte_trace_point_emit_u16(queue_id);
>> + rte_trace_point_emit_int(ret);
>> +)
>> +
>> +RTE_TRACE_POINT(
>> + rte_ethdev_trace_rx_intr_disable,
>> + RTE_TRACE_POINT_ARGS(uint16_t port_id, uint16_t queue_id, int
>> ret),
>> + rte_trace_point_emit_u16(port_id);
>> + rte_trace_point_emit_u16(queue_id);
>> + rte_trace_point_emit_int(ret);
>> +)
>> +
>> +RTE_TRACE_POINT(
>> + rte_eth_trace_add_rx_callback,
>> + RTE_TRACE_POINT_ARGS(uint16_t port_id, uint16_t queue_id,
>> + rte_rx_callback_fn fn, void *user_param,
>> + struct rte_eth_rxtx_callback *cb),
>> + rte_trace_point_emit_u16(port_id);
>> + rte_trace_point_emit_u16(queue_id);
>> + rte_trace_point_emit_ptr(fn);
>> + rte_trace_point_emit_ptr(user_param);
>> + rte_trace_point_emit_ptr(cb);
>> +)
>> +
>> +RTE_TRACE_POINT(
>> + rte_eth_trace_add_first_rx_callback,
>> + RTE_TRACE_POINT_ARGS(uint16_t port_id, uint16_t queue_id,
>> + rte_rx_callback_fn fn, void *user_param,
>> + struct rte_eth_rxtx_callback *cb),
>> + rte_trace_point_emit_u16(port_id);
>> + rte_trace_point_emit_u16(queue_id);
>> + rte_trace_point_emit_ptr(fn);
>> + rte_trace_point_emit_ptr(user_param);
>> + rte_trace_point_emit_ptr(cb);
>> +)
>> +
>> +RTE_TRACE_POINT(
>> + rte_eth_trace_add_tx_callback,
>> + RTE_TRACE_POINT_ARGS(uint16_t port_id, uint16_t queue_id,
>> + rte_tx_callback_fn fn, void *user_param,
>> + struct rte_eth_rxtx_callback *cb),
>> + rte_trace_point_emit_u16(port_id);
>> + rte_trace_point_emit_u16(queue_id);
>> + rte_trace_point_emit_ptr(fn);
>> + rte_trace_point_emit_ptr(user_param);
>> + rte_trace_point_emit_ptr(cb);
>> +)
>> +
>> +RTE_TRACE_POINT(
>> + rte_eth_trace_remove_rx_callback,
>> + RTE_TRACE_POINT_ARGS(uint16_t port_id, uint16_t queue_id,
>> + const struct rte_eth_rxtx_callback *user_cb, int ret),
>> + rte_trace_point_emit_u16(port_id);
>> + rte_trace_point_emit_u16(queue_id);
>> + rte_trace_point_emit_ptr(user_cb);
>> + rte_trace_point_emit_int(ret);
>> +)
>> +
>> +RTE_TRACE_POINT(
>> + rte_eth_trace_remove_tx_callback,
>> + RTE_TRACE_POINT_ARGS(uint16_t port_id, uint16_t queue_id,
>> + const struct rte_eth_rxtx_callback *user_cb, int ret),
>> + rte_trace_point_emit_u16(port_id);
>> + rte_trace_point_emit_u16(queue_id);
>> + rte_trace_point_emit_ptr(user_cb);
>> + rte_trace_point_emit_int(ret);
>> +)
>> +
>> +RTE_TRACE_POINT(
>> + rte_ethdev_trace_set_mc_addr_list,
>> + RTE_TRACE_POINT_ARGS(uint16_t port_id,
>> + struct rte_ether_addr *mc_addr_set,
>> + uint32_t nb_mc_addr, int ret),
>> + rte_trace_point_emit_u16(port_id);
>> + rte_trace_point_emit_ptr(mc_addr_set);
>> + rte_trace_point_emit_u32(nb_mc_addr);
>> + rte_trace_point_emit_int(ret);
>> +)
>> +
>> +RTE_TRACE_POINT(
>> + rte_eth_trace_timesync_enable,
>> + RTE_TRACE_POINT_ARGS(uint16_t port_id, int ret),
>> + rte_trace_point_emit_u16(port_id);
>> + rte_trace_point_emit_int(ret);
>> +)
>> +
>> +RTE_TRACE_POINT(
>> + rte_eth_trace_timesync_disable,
>> + RTE_TRACE_POINT_ARGS(uint16_t port_id, int ret),
>> + rte_trace_point_emit_u16(port_id);
>> + rte_trace_point_emit_int(ret);
>> +)
>> +
>> +RTE_TRACE_POINT(
>> + rte_ethdev_trace_set_eeprom,
>> + RTE_TRACE_POINT_ARGS(uint16_t port_id, struct
>> rte_dev_eeprom_info *info,
>> + int ret),
>> + rte_trace_point_emit_u16(port_id);
>> + rte_trace_point_emit_ptr(info->data);
>> + rte_trace_point_emit_u32(info->offset);
>> + rte_trace_point_emit_u32(info->length);
>> + rte_trace_point_emit_u32(info->magic);
>> + rte_trace_point_emit_int(ret);
>> +)
>> +
>> +RTE_TRACE_POINT(
>> + rte_ethdev_trace_hairpin_capability_get,
>> + RTE_TRACE_POINT_ARGS(uint16_t port_id,
>> + struct rte_eth_hairpin_cap *cap, int ret),
>> + rte_trace_point_emit_u16(port_id);
>> + rte_trace_point_emit_ptr(cap);
>> + rte_trace_point_emit_int(ret);
>> +)
>> +
>> +RTE_TRACE_POINT(
>> + rte_ethdev_trace_pool_ops_supported,
>> + RTE_TRACE_POINT_ARGS(uint16_t port_id, const char *pool, int ret),
>> + rte_trace_point_emit_u16(port_id);
>> + rte_trace_point_emit_string(pool);
>> + rte_trace_point_emit_int(ret);
>> +)
>> +
>> +RTE_TRACE_POINT(
>> + rte_eth_trace_rx_metadata_negotiate,
>> + RTE_TRACE_POINT_ARGS(uint16_t port_id, uint64_t *features,
>> + uint64_t features_val, int ret),
>> + rte_trace_point_emit_u16(port_id);
>> + rte_trace_point_emit_ptr(features);
>> + rte_trace_point_emit_u64(features_val);
>> + rte_trace_point_emit_int(ret);
>> +)
>> +
>> +RTE_TRACE_POINT(
>> + rte_eth_trace_ip_reassembly_capability_get,
>> + RTE_TRACE_POINT_ARGS(uint16_t port_id,
>> + struct rte_eth_ip_reassembly_params *capa, int ret),
>> + rte_trace_point_emit_u16(port_id);
>> + rte_trace_point_emit_ptr(capa);
>> + rte_trace_point_emit_int(ret);
>> +)
>> +
>> +RTE_TRACE_POINT(
>> + rte_eth_trace_ip_reassembly_conf_set,
>> + RTE_TRACE_POINT_ARGS(uint16_t port_id,
>> + const struct rte_eth_ip_reassembly_params *conf, int ret),
>> + rte_trace_point_emit_u16(port_id);
>> + rte_trace_point_emit_ptr(conf);
>> + rte_trace_point_emit_u32(conf->timeout_ms);
>> + rte_trace_point_emit_u16(conf->max_frags);
>> + rte_trace_point_emit_u16(conf->flags);
>> + rte_trace_point_emit_int(ret);
>> +)
>> +
>> +RTE_TRACE_POINT(
>> + rte_eth_trace_cman_config_init,
>> + RTE_TRACE_POINT_ARGS(uint16_t port_id, struct
>> rte_eth_cman_config *config,
>> + int ret),
>> + rte_trace_point_emit_u16(port_id);
>> + rte_trace_point_emit_ptr(config);
>> + rte_trace_point_emit_int(config->obj);
>> + rte_trace_point_emit_int(config->mode);
>> + rte_trace_point_emit_int(ret);
>> +)
>> +
>> +RTE_TRACE_POINT(
>> + rte_eth_trace_cman_config_set,
>> + RTE_TRACE_POINT_ARGS(uint16_t port_id,
>> + const struct rte_eth_cman_config *config, int ret),
>> + rte_trace_point_emit_u16(port_id);
>> + rte_trace_point_emit_ptr(config);
>> + rte_trace_point_emit_int(config->obj);
>> + rte_trace_point_emit_int(config->mode);
>> + rte_trace_point_emit_int(ret);
>> +)
>> +
>> #ifdef __cplusplus
>> }
>> #endif
>> diff --git a/lib/ethdev/rte_ethdev_trace_fp.h
>> b/lib/ethdev/rte_ethdev_trace_fp.h
>> index 9f1d3d5a1b..3dbcc30c0a 100644
>> --- a/lib/ethdev/rte_ethdev_trace_fp.h
>> +++ b/lib/ethdev/rte_ethdev_trace_fp.h
>> @@ -304,6 +304,348 @@ RTE_TRACE_POINT_FP(
>> rte_trace_point_emit_int(ret);
>> )
>>
>> +RTE_TRACE_POINT_FP(
>> + rte_ethdev_trace_get_supported_ptypes,
>> + RTE_TRACE_POINT_ARGS(uint16_t port_id, int supported_num,
>> + int num, uint32_t ptypes),
>> + rte_trace_point_emit_u16(port_id);
>> + rte_trace_point_emit_int(supported_num);
>> + rte_trace_point_emit_int(num);
>> + rte_trace_point_emit_u32(ptypes);
>> +)
>> +
>> +RTE_TRACE_POINT_FP(
>> + rte_eth_trace_macaddrs_get,
>> + RTE_TRACE_POINT_ARGS(uint16_t port_id, unsigned int num),
>> + rte_trace_point_emit_u16(port_id);
>> + rte_trace_point_emit_u32(num);
>> +)
>> +
>> +RTE_TRACE_POINT_FP(
>> + rte_eth_trace_macaddr_get,
>> + RTE_TRACE_POINT_ARGS(uint16_t port_id, struct rte_ether_addr
>> *mac_addr),
>> + uint16_t len = RTE_ETHER_ADDR_LEN;
>> +
>> + rte_trace_point_emit_u16(port_id);
>> + rte_trace_point_emit_ptr(mac_addr);
>> + rte_trace_point_emit_blob(mac_addr->addr_bytes, len);
>> +)
>> +
>> +RTE_TRACE_POINT_FP(
>> + rte_ethdev_trace_get_mtu,
>> + RTE_TRACE_POINT_ARGS(uint16_t port_id, uint16_t mtu),
>> + rte_trace_point_emit_u16(port_id);
>> + rte_trace_point_emit_u16(mtu);
>> +)
>> +
>> +RTE_TRACE_POINT_FP(
>> + rte_ethdev_trace_get_vlan_offload,
>> + RTE_TRACE_POINT_ARGS(uint16_t port_id, int ret),
>> + rte_trace_point_emit_u16(port_id);
>> + rte_trace_point_emit_int(ret);
>> +)
>> +
>> +RTE_TRACE_POINT_FP(
>> + rte_ethdev_trace_flow_ctrl_get,
>> + RTE_TRACE_POINT_ARGS(uint16_t port_id, struct rte_eth_fc_conf
>> *fc_conf,
>> + int ret),
>> + rte_trace_point_emit_u16(port_id);
>> + rte_trace_point_emit_ptr(fc_conf);
>> + rte_trace_point_emit_u32(fc_conf->high_water);
>> + rte_trace_point_emit_u32(fc_conf->low_water);
>> + rte_trace_point_emit_u16(fc_conf->pause_time);
>> + rte_trace_point_emit_u16(fc_conf->send_xon);
>> + rte_trace_point_emit_int(fc_conf->mode);
>> + rte_trace_point_emit_u8(fc_conf->mac_ctrl_frame_fwd);
>> + rte_trace_point_emit_u8(fc_conf->autoneg);
>> + rte_trace_point_emit_int(ret);
>> +)
>> +
>> +RTE_TRACE_POINT_FP(
>> + rte_ethdev_trace_priority_flow_ctrl_queue_info_get,
>> + RTE_TRACE_POINT_ARGS(uint16_t port_id,
>> + struct rte_eth_pfc_queue_info *pfc_queue_info, int ret),
>> + rte_trace_point_emit_u16(port_id);
>> + rte_trace_point_emit_ptr(pfc_queue_info);
>> + rte_trace_point_emit_int(ret);
>> +)
>> +
>> +RTE_TRACE_POINT_FP(
>> + rte_ethdev_trace_rss_reta_query,
>> + RTE_TRACE_POINT_ARGS(uint16_t port_id, struct
>> rte_eth_rss_reta_entry64 *reta_conf,
>> + uint16_t reta_size, int ret),
>> + rte_trace_point_emit_u16(port_id);
>> + rte_trace_point_emit_ptr(reta_conf);
>> + rte_trace_point_emit_u16(reta_size);
>> + rte_trace_point_emit_int(ret);
>> +)
>> +
>> +RTE_TRACE_POINT_FP(
>> + rte_ethdev_trace_rss_hash_conf_get,
>> + RTE_TRACE_POINT_ARGS(uint16_t port_id, struct rte_eth_rss_conf
>> *rss_conf,
>> + int ret),
>> + rte_trace_point_emit_u16(port_id);
>> + rte_trace_point_emit_ptr(rss_conf->rss_key);
>> + rte_trace_point_emit_u8(rss_conf->rss_key_len);
>> + rte_trace_point_emit_u64(rss_conf->rss_hf);
>> + rte_trace_point_emit_int(ret);
>> +)
>> +
>> +RTE_TRACE_POINT_FP(
>> + rte_eth_trace_fec_get,
>> + RTE_TRACE_POINT_ARGS(uint16_t port_id, uint32_t *fec_capa, int
>> ret),
>> + rte_trace_point_emit_u16(port_id);
>> + rte_trace_point_emit_ptr(fec_capa);
>> + rte_trace_point_emit_int(ret);
>> +)
>> +
>> +RTE_TRACE_POINT_FP(
>> + rte_ethdev_trace_rx_intr_ctl_q_get_fd,
>> + RTE_TRACE_POINT_ARGS(uint16_t port_id, uint16_t queue_id, int
>> fd),
>> + rte_trace_point_emit_u16(port_id);
>> + rte_trace_point_emit_u16(queue_id);
>> + rte_trace_point_emit_int(fd);
>> +)
>> +
>> +RTE_TRACE_POINT_FP(
>> + rte_eth_trace_rx_queue_info_get,
>> + RTE_TRACE_POINT_ARGS(uint16_t port_id, uint16_t queue_id,
>> + struct rte_eth_rxq_info *qinfo),
>> + rte_trace_point_emit_u16(port_id);
>> + rte_trace_point_emit_u16(queue_id);
>> + rte_trace_point_emit_ptr(qinfo->mp);
>> + rte_trace_point_emit_u8(qinfo->conf.rx_drop_en);
>> + rte_trace_point_emit_u64(qinfo->conf.offloads);
>> + rte_trace_point_emit_u8(qinfo->scattered_rx);
>> + rte_trace_point_emit_u8(qinfo->queue_state);
>> + rte_trace_point_emit_u16(qinfo->nb_desc);
>> + rte_trace_point_emit_u16(qinfo->rx_buf_size);
>> +)
>> +
>> +RTE_TRACE_POINT_FP(
>> + rte_eth_trace_tx_queue_info_get,
>> + RTE_TRACE_POINT_ARGS(uint16_t port_id, uint16_t queue_id,
>> + struct rte_eth_txq_info *qinfo),
>> + rte_trace_point_emit_u16(port_id);
>> + rte_trace_point_emit_u16(queue_id);
>> + rte_trace_point_emit_u16(qinfo->nb_desc);
>> + rte_trace_point_emit_u8(qinfo->queue_state);
>> +)
>> +
>> +RTE_TRACE_POINT_FP(
>> + rte_eth_trace_rx_burst_mode_get,
>> + RTE_TRACE_POINT_ARGS(uint16_t port_id, uint16_t queue_id,
>> + struct rte_eth_burst_mode *mode, int ret),
>> + rte_trace_point_emit_u16(port_id);
>> + rte_trace_point_emit_u16(queue_id);
>> + rte_trace_point_emit_ptr(mode);
>> + rte_trace_point_emit_u64(mode->flags);
>> + rte_trace_point_emit_int(ret);
>> +)
>> +
>> +RTE_TRACE_POINT_FP(
>> + rte_eth_trace_tx_burst_mode_get,
>> + RTE_TRACE_POINT_ARGS(uint16_t port_id, uint16_t queue_id,
>> + struct rte_eth_burst_mode *mode, int ret),
>> + rte_trace_point_emit_u16(port_id);
>> + rte_trace_point_emit_u16(queue_id);
>> + rte_trace_point_emit_ptr(mode);
>> + rte_trace_point_emit_int(ret);
>> +)
>> +
>> +RTE_TRACE_POINT_FP(
>> + rte_eth_trace_get_monitor_addr,
>> + RTE_TRACE_POINT_ARGS(uint16_t port_id, uint16_t queue_id,
>> + struct rte_power_monitor_cond *pmc, int ret),
>> + rte_trace_point_emit_u16(port_id);
>> + rte_trace_point_emit_u16(queue_id);
>> + rte_trace_point_emit_ptr(pmc);
>> + rte_trace_point_emit_ptr(pmc->addr);
>> + rte_trace_point_emit_u8(pmc->size);
>> + rte_trace_point_emit_int(ret);
>> +)
>> +
>> +RTE_TRACE_POINT_FP(
>> + rte_eth_trace_timesync_read_rx_timestamp,
>> + RTE_TRACE_POINT_ARGS(uint16_t port_id, struct timespec
>> *timestamp,
>> + uint32_t flags, int ret),
>> + rte_trace_point_emit_u16(port_id);
>> + rte_trace_point_emit_ptr(timestamp);
>> + rte_trace_point_emit_u64(timestamp->tv_sec);
>> + rte_trace_point_emit_u64(timestamp->tv_nsec);
>> + rte_trace_point_emit_u32(flags);
>> + rte_trace_point_emit_int(ret);
>> +)
>> +
>> +RTE_TRACE_POINT_FP(
>> + rte_eth_trace_timesync_read_tx_timestamp,
>> + RTE_TRACE_POINT_ARGS(uint16_t port_id, struct timespec
>> *timestamp, int ret),
>> + rte_trace_point_emit_u16(port_id);
>> + rte_trace_point_emit_ptr(timestamp);
>> + rte_trace_point_emit_u64(timestamp->tv_sec);
>> + rte_trace_point_emit_u64(timestamp->tv_nsec);
>> + rte_trace_point_emit_int(ret);
>> +)
>> +
>> +RTE_TRACE_POINT_FP(
>> + rte_eth_trace_timesync_read_time,
>> + RTE_TRACE_POINT_ARGS(uint16_t port_id, struct timespec *time, int
>> ret),
>> + rte_trace_point_emit_u16(port_id);
>> + rte_trace_point_emit_ptr(time);
>> + rte_trace_point_emit_u64(time->tv_sec);
>> + rte_trace_point_emit_u64(time->tv_nsec);
>> + rte_trace_point_emit_int(ret);
>> +)
>> +
>> +RTE_TRACE_POINT_FP(
>> + rte_eth_trace_timesync_adjust_time,
>> + RTE_TRACE_POINT_ARGS(uint16_t port_id, int64_t delta, int ret),
>> + rte_trace_point_emit_u16(port_id);
>> + rte_trace_point_emit_i64(delta);
>> + rte_trace_point_emit_int(ret);
>> +)
>> +
>> +RTE_TRACE_POINT_FP(
>> + rte_eth_trace_timesync_write_time,
>> + RTE_TRACE_POINT_ARGS(uint16_t port_id, const struct timespec
>> *time, int ret),
>> + rte_trace_point_emit_u16(port_id);
>> + rte_trace_point_emit_ptr(time);
>> + rte_trace_point_emit_u64(time->tv_sec);
>> + rte_trace_point_emit_u64(time->tv_nsec);
>> + rte_trace_point_emit_int(ret);
>> +)
>> +
>> +RTE_TRACE_POINT_FP(
>> + rte_eth_trace_read_clock,
>> + RTE_TRACE_POINT_ARGS(uint16_t port_id, uint64_t *clk, int ret),
>> + uint64_t clk_v = *clk;
>> + rte_trace_point_emit_u16(port_id);
>> + rte_trace_point_emit_ptr(clk);
>> + rte_trace_point_emit_u64(clk_v);
>> + rte_trace_point_emit_int(ret);
>> +)
>> +
>> +RTE_TRACE_POINT_FP(
>> + rte_ethdev_trace_get_reg_info,
>> + RTE_TRACE_POINT_ARGS(uint16_t port_id, struct rte_dev_reg_info
>> *info, int ret),
>> + rte_trace_point_emit_u16(port_id);
>> + rte_trace_point_emit_ptr(info);
>> + rte_trace_point_emit_ptr(info->data);
>> + rte_trace_point_emit_u32(info->offset);
>> + rte_trace_point_emit_u32(info->length);
>> + rte_trace_point_emit_u32(info->width);
>> + rte_trace_point_emit_u32(info->version);
>> + rte_trace_point_emit_int(ret);
>> +)
>> +
>> +RTE_TRACE_POINT_FP(
>> + rte_ethdev_trace_get_eeprom_length,
>> + RTE_TRACE_POINT_ARGS(uint16_t port_id, int ret),
>> + rte_trace_point_emit_u16(port_id);
>> + rte_trace_point_emit_int(ret);
>> +)
>> +
>> +RTE_TRACE_POINT_FP(
>> + rte_ethdev_trace_get_eeprom,
>> + RTE_TRACE_POINT_ARGS(uint16_t port_id, struct
>> rte_dev_eeprom_info *info, int ret),
>> + rte_trace_point_emit_u16(port_id);
>> + rte_trace_point_emit_ptr(info);
>> + rte_trace_point_emit_ptr(info->data);
>> + rte_trace_point_emit_u32(info->offset);
>> + rte_trace_point_emit_u32(info->length);
>> + rte_trace_point_emit_u32(info->magic);
>> + rte_trace_point_emit_int(ret);
>> +)
>> +
>> +RTE_TRACE_POINT_FP(
>> + rte_ethdev_trace_get_module_info,
>> + RTE_TRACE_POINT_ARGS(uint16_t port_id,
>> + struct rte_eth_dev_module_info *modinfo, int ret),
>> + rte_trace_point_emit_u16(port_id);
>> + rte_trace_point_emit_ptr(modinfo);
>> + rte_trace_point_emit_u32(modinfo->type);
>> + rte_trace_point_emit_u32(modinfo->eeprom_len);
>> + rte_trace_point_emit_int(ret);
>> +)
>> +
>> +RTE_TRACE_POINT_FP(
>> + rte_ethdev_trace_get_module_eeprom,
>> + RTE_TRACE_POINT_ARGS(uint16_t port_id,
>> + struct rte_dev_eeprom_info *info, int ret),
>> + rte_trace_point_emit_u16(port_id);
>> + rte_trace_point_emit_ptr(info);
>> + rte_trace_point_emit_int(ret);
>> +)
>> +
>> +RTE_TRACE_POINT_FP(
>> + rte_ethdev_trace_get_dcb_info,
>> + RTE_TRACE_POINT_ARGS(uint16_t port_id, struct rte_eth_dcb_info
>> *dcb_info,
>> + int ret),
>> + uint16_t num_user_priorities =
>> RTE_ETH_DCB_NUM_USER_PRIORITIES;
>> + uint16_t num_tcs = RTE_ETH_DCB_NUM_TCS;
>> +
>> + rte_trace_point_emit_u16(port_id);
>> + rte_trace_point_emit_ptr(dcb_info);
>> + rte_trace_point_emit_u8(dcb_info->nb_tcs);
>> + rte_trace_point_emit_blob(dcb_info->prio_tc, num_user_priorities);
>> + rte_trace_point_emit_blob(dcb_info->tc_bws, num_tcs);
>> + rte_trace_point_emit_int(ret);
>> +)
>> +
>> +RTE_TRACE_POINT_FP(
>> + rte_ethdev_trace_adjust_nb_rx_tx_desc,
>> + RTE_TRACE_POINT_ARGS(uint16_t port_id),
>> + rte_trace_point_emit_u16(port_id);
>> +)
>> +
>> +RTE_TRACE_POINT_FP(
>> + rte_eth_trace_representor_info_get,
>> + RTE_TRACE_POINT_ARGS(uint16_t port_id,
>> + struct rte_eth_representor_info *info, int ret),
>> + rte_trace_point_emit_u16(port_id);
>> + rte_trace_point_emit_ptr(info);
>> + rte_trace_point_emit_int(ret);
>> +)
>> +
>> +RTE_TRACE_POINT_FP(
>> + rte_eth_trace_ip_reassembly_conf_get,
>> + RTE_TRACE_POINT_ARGS(uint16_t port_id,
>> + struct rte_eth_ip_reassembly_params *conf, int ret),
>> + rte_trace_point_emit_u16(port_id);
>> + rte_trace_point_emit_ptr(conf);
>> + rte_trace_point_emit_int(ret);
>> +)
>> +
>> +RTE_TRACE_POINT_FP(
>> + rte_eth_trace_buffer_split_get_supported_hdr_ptypes,
>> + RTE_TRACE_POINT_ARGS(uint16_t port_id, int supported_num,
>> uint32_t ptypes),
>> + rte_trace_point_emit_u16(port_id);
>> + rte_trace_point_emit_int(supported_num);
>> + rte_trace_point_emit_u32(ptypes);
>> +)
>> +
>> +RTE_TRACE_POINT_FP(
>> + rte_eth_trace_cman_info_get,
>> + RTE_TRACE_POINT_ARGS(uint16_t port_id,
>> + struct rte_eth_cman_info *info, int ret),
>> + rte_trace_point_emit_u16(port_id);
>> + rte_trace_point_emit_ptr(info);
>> + rte_trace_point_emit_u64(info->modes_supported);
>> + rte_trace_point_emit_u64(info->objs_supported);
>> + rte_trace_point_emit_int(ret);
>> +)
>> +
>> +RTE_TRACE_POINT_FP(
>> + rte_eth_trace_cman_config_get,
>> + RTE_TRACE_POINT_ARGS(uint16_t port_id,
>> + const struct rte_eth_cman_config *config, int ret),
>> + rte_trace_point_emit_u16(port_id);
>> + rte_trace_point_emit_ptr(config);
>> + rte_trace_point_emit_int(config->obj);
>> + rte_trace_point_emit_int(config->mode);
>> + rte_trace_point_emit_int(ret);
>> +)
>> +
>> #ifdef __cplusplus
>> }
>> #endif
>> diff --git a/lib/ethdev/version.map b/lib/ethdev/version.map
>> index c7ba2e3dc8..c7c6bcd42b 100644
>> --- a/lib/ethdev/version.map
>> +++ b/lib/ethdev/version.map
>> @@ -300,36 +300,71 @@ EXPERIMENTAL {
>> rte_mtr_meter_profile_get;
>>
>> # added in 23.03
>> + __rte_eth_trace_add_first_rx_callback;
>> + __rte_eth_trace_add_rx_callback;
>> + __rte_eth_trace_add_tx_callback;
>> __rte_eth_trace_allmulticast_disable;
>> __rte_eth_trace_allmulticast_enable;
>> __rte_eth_trace_allmulticast_get;
>> __rte_eth_trace_call_rx_callbacks;
>> __rte_eth_trace_call_tx_callbacks;
>> + __rte_eth_trace_cman_config_get;
>> + __rte_eth_trace_cman_config_init;
>> + __rte_eth_trace_cman_config_set;
>> + __rte_eth_trace_cman_info_get;
>> + __rte_eth_trace_fec_get_capability;
>> + __rte_eth_trace_fec_get;
>> + __rte_eth_trace_fec_set;
>> __rte_eth_trace_find_next;
>> __rte_eth_trace_find_next_of;
>> __rte_eth_trace_find_next_owned_by;
>> __rte_eth_trace_find_next_sibling;
>> + __rte_eth_trace_get_monitor_addr;
>> __rte_eth_trace_hairpin_bind;
>> __rte_eth_trace_hairpin_get_peer_ports;
>> __rte_eth_trace_hairpin_unbind;
>> + __rte_eth_trace_ip_reassembly_capability_get;
>> + __rte_eth_trace_ip_reassembly_conf_get;
>> + __rte_eth_trace_ip_reassembly_conf_set;
>> __rte_eth_trace_iterator_cleanup;
>> __rte_eth_trace_iterator_init;
>> __rte_eth_trace_iterator_next;
>> + __rte_eth_trace_led_off;
>> + __rte_eth_trace_led_on;
>> __rte_eth_trace_link_get;
>> __rte_eth_trace_link_get_nowait;
>> __rte_eth_trace_link_speed_to_str;
>> __rte_eth_trace_link_to_str;
>> + __rte_eth_trace_macaddr_get;
>> + __rte_eth_trace_macaddrs_get;
>> __rte_eth_trace_promiscuous_disable;
>> __rte_eth_trace_promiscuous_enable;
>> __rte_eth_trace_promiscuous_get;
>> + __rte_eth_trace_remove_rx_callback;
>> + __rte_eth_trace_remove_tx_callback;
>> + __rte_eth_trace_representor_info_get;
>> + __rte_eth_trace_rx_avail_thresh_query;
>> + __rte_eth_trace_rx_avail_thresh_set;
>> + __rte_eth_trace_rx_burst_mode_get;
>> __rte_eth_trace_rx_hairpin_queue_setup;
>> + __rte_eth_trace_rx_metadata_negotiate;
>> + __rte_eth_trace_rx_queue_info_get;
>> + __rte_eth_trace_set_queue_rate_limit;
>> __rte_eth_trace_speed_bitflag;
>> __rte_eth_trace_stats_get;
>> __rte_eth_trace_stats_reset;
>> + __rte_eth_trace_timesync_adjust_time;
>> + __rte_eth_trace_timesync_disable;
>> + __rte_eth_trace_timesync_enable;
>> + __rte_eth_trace_timesync_read_rx_timestamp;
>> + __rte_eth_trace_timesync_read_time;
>> + __rte_eth_trace_timesync_read_tx_timestamp;
>> + __rte_eth_trace_timesync_write_time;
>> __rte_eth_trace_tx_buffer_count_callback;
>> __rte_eth_trace_tx_buffer_drop_callback;
>> __rte_eth_trace_tx_buffer_init;
>> __rte_eth_trace_tx_buffer_set_err_callback;
>> + __rte_eth_trace_tx_burst_mode_get;
>> __rte_eth_trace_tx_done_cleanup;
>> __rte_eth_trace_tx_hairpin_queue_setup;
>> __rte_eth_trace_xstats_get;
>> @@ -338,32 +373,78 @@ EXPERIMENTAL {
>> __rte_eth_trace_xstats_get_names;
>> __rte_eth_trace_xstats_get_names_by_id;
>> __rte_eth_trace_xstats_reset;
>> + __rte_ethdev_trace_adjust_nb_rx_tx_desc;
>> + __rte_ethdev_trace_callback_register;
>> + __rte_ethdev_trace_callback_unregister;
>> __rte_ethdev_trace_capability_name;
>> + __rte_ethdev_trace_conf_get;
>> __rte_ethdev_trace_count_avail;
>> __rte_ethdev_trace_count_total;
>> + __rte_ethdev_trace_default_mac_addr_set;
>> + __rte_ethdev_trace_flow_ctrl_get;
>> + __rte_ethdev_trace_flow_ctrl_set;
>> __rte_ethdev_trace_fw_version_get;
>> + __rte_ethdev_trace_get_dcb_info;
>> + __rte_ethdev_trace_get_eeprom;
>> + __rte_ethdev_trace_get_eeprom_length;
>> + __rte_ethdev_trace_get_module_eeprom;
>> + __rte_ethdev_trace_get_module_info;
>> + __rte_ethdev_trace_get_mtu;
>> __rte_ethdev_trace_get_name_by_port;
>> __rte_ethdev_trace_get_port_by_name;
>> + __rte_ethdev_trace_get_reg_info;
>> __rte_ethdev_trace_get_sec_ctx;
>> + __rte_ethdev_trace_get_supported_ptypes;
>> + __rte_ethdev_trace_get_vlan_offload;
>> + __rte_ethdev_trace_hairpin_capability_get;
>> + __rte_ethdev_trace_info_get;
>> __rte_ethdev_trace_is_removed;
>> __rte_ethdev_trace_is_valid_port;
>> + __rte_ethdev_trace_mac_addr_add;
>> + __rte_ethdev_trace_mac_addr_remove;
>> __rte_ethdev_trace_owner_delete;
>> __rte_ethdev_trace_owner_get;
>> __rte_ethdev_trace_owner_new;
>> __rte_ethdev_trace_owner_set;
>> __rte_ethdev_trace_owner_unset;
>> + __rte_ethdev_trace_pool_ops_supported;
>> + __rte_ethdev_trace_priority_flow_ctrl_queue_configure;
>> + __rte_ethdev_trace_priority_flow_ctrl_queue_info_get;
>> + __rte_ethdev_trace_priority_flow_ctrl_set;
>> __rte_ethdev_trace_reset;
>> + __rte_ethdev_trace_rss_hash_conf_get;
>> + __rte_ethdev_trace_rss_hash_update;
>> + __rte_ethdev_trace_rss_reta_query;
>> + __rte_ethdev_trace_rss_reta_update;
>> + __rte_ethdev_trace_rx_intr_ctl;
>> + __rte_ethdev_trace_rx_intr_ctl_q;
>> + __rte_ethdev_trace_rx_intr_ctl_q_get_fd;
>> + __rte_ethdev_trace_rx_intr_disable;
>> + __rte_ethdev_trace_rx_intr_enable;
>> __rte_ethdev_trace_rx_offload_name;
>> __rte_ethdev_trace_rx_queue_start;
>> __rte_ethdev_trace_rx_queue_stop;
>> + __rte_ethdev_trace_set_eeprom;
>> __rte_ethdev_trace_set_link_down;
>> __rte_ethdev_trace_set_link_up;
>> + __rte_ethdev_trace_set_mc_addr_list;
>> + __rte_ethdev_trace_set_mtu;
>> + __rte_ethdev_trace_set_ptypes;
>> __rte_ethdev_trace_set_rx_queue_stats_mapping;
>> __rte_ethdev_trace_set_tx_queue_stats_mapping;
>> + __rte_ethdev_trace_set_vlan_ether_type;
>> + __rte_ethdev_trace_set_vlan_offload;
>> + __rte_ethdev_trace_set_vlan_pvid;
>> + __rte_ethdev_trace_set_vlan_strip_on_queue;
>> __rte_ethdev_trace_socket_id;
>> __rte_ethdev_trace_tx_offload_name;
>> __rte_ethdev_trace_tx_queue_start;
>> __rte_ethdev_trace_tx_queue_stop;
>> + __rte_ethdev_trace_uc_all_hash_table_set;
>> + __rte_ethdev_trace_uc_hash_table_set;
>> + __rte_ethdev_trace_udp_tunnel_port_add;
>> + __rte_ethdev_trace_udp_tunnel_port_delete;
>> + __rte_ethdev_trace_vlan_filter;
>> };
>>
>> INTERNAL {
>> --
>> 2.25.1
