ethdev: hide internal structures
authorKonstantin Ananyev <konstantin.ananyev@intel.com>
Wed, 13 Oct 2021 13:37:04 +0000 (14:37 +0100)
committerFerruh Yigit <ferruh.yigit@intel.com>
Wed, 13 Oct 2021 20:14:59 +0000 (22:14 +0200)
Move rte_eth_dev, rte_eth_dev_data, rte_eth_rxtx_callback and related
data into private header (ethdev_driver.h).
Few minor changes to keep DPDK building after that.

Signed-off-by: Konstantin Ananyev <konstantin.ananyev@intel.com>
Acked-by: Andrew Rybchenko <andrew.rybchenko@oktetlabs.ru>
Reviewed-by: Ferruh Yigit <ferruh.yigit@intel.com>
Tested-by: Feifei Wang <feifei.wang2@arm.com>
14 files changed:
doc/guides/rel_notes/release_21_11.rst
drivers/common/octeontx2/otx2_sec_idev.c
drivers/crypto/octeontx2/otx2_cryptodev_ops.c
drivers/net/cxgbe/base/adapter.h
drivers/net/dpaa2/dpaa2_ptp.c
drivers/net/mvneta/mvneta_ethdev.h
drivers/net/netvsc/hn_var.h
lib/ethdev/ethdev_driver.h
lib/ethdev/rte_ethdev_core.h
lib/ethdev/version.map
lib/eventdev/rte_event_eth_rx_adapter.c
lib/eventdev/rte_event_eth_tx_adapter.c
lib/eventdev/rte_eventdev.c
lib/metrics/rte_metrics_telemetry.c

index fd2f8eb..d434e33 100644 (file)
@@ -340,6 +340,12 @@ ABI Changes
   to user, it still counts as an ABI change, as ``eth_rx_queue_count_t``
   is used by  public inline function ``rte_eth_rx_queue_count``.
 
+* ethdev: Made ``rte_eth_dev``, ``rte_eth_dev_data``, ``rte_eth_rxtx_callback``
+  private data structures. ``rte_eth_devices[]`` can't be accessed directly
+  by user any more. While it is an ABI breakage, this change is intended
+  to be transparent for both users (no changes in user app is required) and
+  PMD developers (no changes in PMD is required).
+
 * security: ``rte_security_set_pkt_metadata`` and ``rte_security_get_userdata``
   routines used by inline outbound and inline inbound security processing were
   made inline and enhanced to do simple 64-bit set/get for PMDs that do not
index 6e9643c..b561b67 100644 (file)
@@ -4,7 +4,7 @@
 
 #include <rte_atomic.h>
 #include <rte_bus_pci.h>
-#include <rte_ethdev.h>
+#include <ethdev_driver.h>
 #include <rte_spinlock.h>
 
 #include "otx2_common.h"
index 37fad11..f0b72e0 100644 (file)
@@ -6,7 +6,7 @@
 
 #include <cryptodev_pmd.h>
 #include <rte_errno.h>
-#include <rte_ethdev.h>
+#include <ethdev_driver.h>
 #include <rte_event_crypto_adapter.h>
 
 #include "otx2_cryptodev.h"
index 01a2a9d..1c7c8af 100644 (file)
@@ -12,7 +12,7 @@
 #include <rte_mbuf.h>
 #include <rte_io.h>
 #include <rte_rwlock.h>
-#include <rte_ethdev.h>
+#include <ethdev_driver.h>
 
 #include "../cxgbe_compat.h"
 #include "../cxgbe_ofld.h"
index 899dd5d..8d79e39 100644 (file)
@@ -10,7 +10,7 @@
 #include <unistd.h>
 #include <stdarg.h>
 
-#include <rte_ethdev.h>
+#include <ethdev_driver.h>
 #include <rte_log.h>
 #include <rte_eth_ctrl.h>
 #include <rte_malloc.h>
index ef80677..6585a71 100644 (file)
@@ -7,7 +7,7 @@
 #ifndef _MVNETA_ETHDEV_H_
 #define _MVNETA_ETHDEV_H_
 
-#include <rte_ethdev.h>
+#include <ethdev_driver.h>
 #include <rte_malloc.h>
 #include <rte_log.h>
 
index 18703f9..fbb3995 100644 (file)
@@ -7,6 +7,7 @@
  */
 
 #include <rte_eal_paging.h>
+#include <ethdev_driver.h>
 
 /*
  * Tunable ethdev params
index 56db53d..0174ba0 100644 (file)
 
 #include <rte_ethdev.h>
 
+/**
+ * @internal
+ * Structure used to hold information about the callbacks to be called for a
+ * queue on RX and TX.
+ */
+struct rte_eth_rxtx_callback {
+       struct rte_eth_rxtx_callback *next;
+       union{
+               rte_rx_callback_fn rx;
+               rte_tx_callback_fn tx;
+       } fn;
+       void *param;
+};
+
+/**
+ * @internal
+ * The generic data structure associated with each ethernet device.
+ *
+ * Pointers to burst-oriented packet receive and transmit functions are
+ * located at the beginning of the structure, along with the pointer to
+ * where all the data elements for the particular device are stored in shared
+ * memory. This split allows the function pointer and driver data to be per-
+ * process, while the actual configuration data for the device is shared.
+ */
+struct rte_eth_dev {
+       eth_rx_burst_t rx_pkt_burst; /**< Pointer to PMD receive function. */
+       eth_tx_burst_t tx_pkt_burst; /**< Pointer to PMD transmit function. */
+       eth_tx_prep_t tx_pkt_prepare;
+       /**< Pointer to PMD transmit prepare function. */
+       eth_rx_queue_count_t rx_queue_count;
+       /**< Get the number of used RX descriptors. */
+       eth_rx_descriptor_status_t rx_descriptor_status;
+       /**< Check the status of a Rx descriptor. */
+       eth_tx_descriptor_status_t tx_descriptor_status;
+       /**< Check the status of a Tx descriptor. */
+
+       /**
+        * points to device data that is shared between
+        * primary and secondary processes.
+        */
+       struct rte_eth_dev_data *data;
+       void *process_private; /**< Pointer to per-process device data. */
+       const struct eth_dev_ops *dev_ops; /**< Functions exported by PMD */
+       struct rte_device *device; /**< Backing device */
+       struct rte_intr_handle *intr_handle; /**< Device interrupt handle */
+       /** User application callbacks for NIC interrupts */
+       struct rte_eth_dev_cb_list link_intr_cbs;
+       /**
+        * User-supplied functions called from rx_burst to post-process
+        * received packets before passing them to the user
+        */
+       struct rte_eth_rxtx_callback *post_rx_burst_cbs[RTE_MAX_QUEUES_PER_PORT];
+       /**
+        * User-supplied functions called from tx_burst to pre-process
+        * received packets before passing them to the driver for transmission.
+        */
+       struct rte_eth_rxtx_callback *pre_tx_burst_cbs[RTE_MAX_QUEUES_PER_PORT];
+       enum rte_eth_dev_state state; /**< Flag indicating the port state */
+       void *security_ctx; /**< Context for security ops */
+
+       uint64_t reserved_64s[4]; /**< Reserved for future fields */
+       void *reserved_ptrs[4];   /**< Reserved for future fields */
+} __rte_cache_aligned;
+
+struct rte_eth_dev_sriov;
+struct rte_eth_dev_owner;
+
+/**
+ * @internal
+ * The data part, with no function pointers, associated with each ethernet
+ * device. This structure is safe to place in shared memory to be common
+ * among different processes in a multi-process configuration.
+ */
+struct rte_eth_dev_data {
+       char name[RTE_ETH_NAME_MAX_LEN]; /**< Unique identifier name */
+
+       void **rx_queues; /**< Array of pointers to RX queues. */
+       void **tx_queues; /**< Array of pointers to TX queues. */
+       uint16_t nb_rx_queues; /**< Number of RX queues. */
+       uint16_t nb_tx_queues; /**< Number of TX queues. */
+
+       struct rte_eth_dev_sriov sriov;    /**< SRIOV data */
+
+       void *dev_private;
+                       /**< PMD-specific private data.
+                        *   @see rte_eth_dev_release_port()
+                        */
+
+       struct rte_eth_link dev_link;   /**< Link-level information & status. */
+       struct rte_eth_conf dev_conf;   /**< Configuration applied to device. */
+       uint16_t mtu;                   /**< Maximum Transmission Unit. */
+       uint32_t min_rx_buf_size;
+                       /**< Common RX buffer size handled by all queues. */
+
+       uint64_t rx_mbuf_alloc_failed; /**< RX ring mbuf allocation failures. */
+       struct rte_ether_addr *mac_addrs;
+                       /**< Device Ethernet link address.
+                        *   @see rte_eth_dev_release_port()
+                        */
+       uint64_t mac_pool_sel[ETH_NUM_RECEIVE_MAC_ADDR];
+                       /**< Bitmap associating MAC addresses to pools. */
+       struct rte_ether_addr *hash_mac_addrs;
+                       /**< Device Ethernet MAC addresses of hash filtering.
+                        *   @see rte_eth_dev_release_port()
+                        */
+       uint16_t port_id;           /**< Device [external] port identifier. */
+
+       __extension__
+       uint8_t promiscuous   : 1,
+               /**< RX promiscuous mode ON(1) / OFF(0). */
+               scattered_rx : 1,
+               /**< RX of scattered packets is ON(1) / OFF(0) */
+               all_multicast : 1,
+               /**< RX all multicast mode ON(1) / OFF(0). */
+               dev_started : 1,
+               /**< Device state: STARTED(1) / STOPPED(0). */
+               lro         : 1,
+               /**< RX LRO is ON(1) / OFF(0) */
+               dev_configured : 1;
+               /**< Indicates whether the device is configured.
+                *   CONFIGURED(1) / NOT CONFIGURED(0).
+                */
+       uint8_t rx_queue_state[RTE_MAX_QUEUES_PER_PORT];
+               /**< Queues state: HAIRPIN(2) / STARTED(1) / STOPPED(0). */
+       uint8_t tx_queue_state[RTE_MAX_QUEUES_PER_PORT];
+               /**< Queues state: HAIRPIN(2) / STARTED(1) / STOPPED(0). */
+       uint32_t dev_flags;             /**< Capabilities. */
+       int numa_node;                  /**< NUMA node connection. */
+       struct rte_vlan_filter_conf vlan_filter_conf;
+                       /**< VLAN filter configuration. */
+       struct rte_eth_dev_owner owner; /**< The port owner. */
+       uint16_t representor_id;
+                       /**< Switch-specific identifier.
+                        *   Valid if RTE_ETH_DEV_REPRESENTOR in dev_flags.
+                        */
+       uint16_t backer_port_id;
+                       /**< Port ID of the backing device.
+                        *   This device will be used to query representor
+                        *   info and calculate representor IDs.
+                        *   Valid if RTE_ETH_DEV_REPRESENTOR in dev_flags.
+                        */
+
+       pthread_mutex_t flow_ops_mutex; /**< rte_flow ops mutex. */
+       uint64_t reserved_64s[4]; /**< Reserved for future fields */
+       void *reserved_ptrs[4];   /**< Reserved for future fields */
+} __rte_cache_aligned;
+
+/**
+ * @internal
+ * The pool of *rte_eth_dev* structures. The size of the pool
+ * is configured at compile-time in the <rte_ethdev.c> file.
+ */
+extern struct rte_eth_dev rte_eth_devices[];
+
 /**< @internal Declaration of the hairpin peer queue information structure. */
 struct rte_hairpin_peer_info;
 
index df7168c..2b8660c 100644 (file)
@@ -109,153 +109,4 @@ struct rte_eth_fp_ops {
 
 extern struct rte_eth_fp_ops rte_eth_fp_ops[RTE_MAX_ETHPORTS];
 
-
-/**
- * @internal
- * Structure used to hold information about the callbacks to be called for a
- * queue on RX and TX.
- */
-struct rte_eth_rxtx_callback {
-       struct rte_eth_rxtx_callback *next;
-       union{
-               rte_rx_callback_fn rx;
-               rte_tx_callback_fn tx;
-       } fn;
-       void *param;
-};
-
-/**
- * @internal
- * The generic data structure associated with each ethernet device.
- *
- * Pointers to burst-oriented packet receive and transmit functions are
- * located at the beginning of the structure, along with the pointer to
- * where all the data elements for the particular device are stored in shared
- * memory. This split allows the function pointer and driver data to be per-
- * process, while the actual configuration data for the device is shared.
- */
-struct rte_eth_dev {
-       eth_rx_burst_t rx_pkt_burst; /**< Pointer to PMD receive function. */
-       eth_tx_burst_t tx_pkt_burst; /**< Pointer to PMD transmit function. */
-       eth_tx_prep_t tx_pkt_prepare; /**< Pointer to PMD transmit prepare function. */
-
-       eth_rx_queue_count_t       rx_queue_count; /**< Get the number of used RX descriptors. */
-       eth_rx_descriptor_status_t rx_descriptor_status; /**< Check the status of a Rx descriptor. */
-       eth_tx_descriptor_status_t tx_descriptor_status; /**< Check the status of a Tx descriptor. */
-
-       /**
-        * Next two fields are per-device data but *data is shared between
-        * primary and secondary processes and *process_private is per-process
-        * private. The second one is managed by PMDs if necessary.
-        */
-       struct rte_eth_dev_data *data;  /**< Pointer to device data. */
-       void *process_private; /**< Pointer to per-process device data. */
-       const struct eth_dev_ops *dev_ops; /**< Functions exported by PMD */
-       struct rte_device *device; /**< Backing device */
-       struct rte_intr_handle *intr_handle; /**< Device interrupt handle */
-       /** User application callbacks for NIC interrupts */
-       struct rte_eth_dev_cb_list link_intr_cbs;
-       /**
-        * User-supplied functions called from rx_burst to post-process
-        * received packets before passing them to the user
-        */
-       struct rte_eth_rxtx_callback *post_rx_burst_cbs[RTE_MAX_QUEUES_PER_PORT];
-       /**
-        * User-supplied functions called from tx_burst to pre-process
-        * received packets before passing them to the driver for transmission.
-        */
-       struct rte_eth_rxtx_callback *pre_tx_burst_cbs[RTE_MAX_QUEUES_PER_PORT];
-       enum rte_eth_dev_state state; /**< Flag indicating the port state */
-       void *security_ctx; /**< Context for security ops */
-
-       uint64_t reserved_64s[4]; /**< Reserved for future fields */
-       void *reserved_ptrs[4];   /**< Reserved for future fields */
-} __rte_cache_aligned;
-
-struct rte_eth_dev_sriov;
-struct rte_eth_dev_owner;
-
-/**
- * @internal
- * The data part, with no function pointers, associated with each ethernet device.
- *
- * This structure is safe to place in shared memory to be common among different
- * processes in a multi-process configuration.
- */
-struct rte_eth_dev_data {
-       char name[RTE_ETH_NAME_MAX_LEN]; /**< Unique identifier name */
-
-       void **rx_queues; /**< Array of pointers to RX queues. */
-       void **tx_queues; /**< Array of pointers to TX queues. */
-       uint16_t nb_rx_queues; /**< Number of RX queues. */
-       uint16_t nb_tx_queues; /**< Number of TX queues. */
-
-       struct rte_eth_dev_sriov sriov;    /**< SRIOV data */
-
-       void *dev_private;
-                       /**< PMD-specific private data.
-                        *   @see rte_eth_dev_release_port()
-                        */
-
-       struct rte_eth_link dev_link;   /**< Link-level information & status. */
-       struct rte_eth_conf dev_conf;   /**< Configuration applied to device. */
-       uint16_t mtu;                   /**< Maximum Transmission Unit. */
-       uint32_t min_rx_buf_size;
-                       /**< Common RX buffer size handled by all queues. */
-
-       uint64_t rx_mbuf_alloc_failed; /**< RX ring mbuf allocation failures. */
-       struct rte_ether_addr *mac_addrs;
-                       /**< Device Ethernet link address.
-                        *   @see rte_eth_dev_release_port()
-                        */
-       uint64_t mac_pool_sel[ETH_NUM_RECEIVE_MAC_ADDR];
-                       /**< Bitmap associating MAC addresses to pools. */
-       struct rte_ether_addr *hash_mac_addrs;
-                       /**< Device Ethernet MAC addresses of hash filtering.
-                        *   @see rte_eth_dev_release_port()
-                        */
-       uint16_t port_id;           /**< Device [external] port identifier. */
-
-       __extension__
-       uint8_t promiscuous   : 1, /**< RX promiscuous mode ON(1) / OFF(0). */
-               scattered_rx : 1,  /**< RX of scattered packets is ON(1) / OFF(0) */
-               all_multicast : 1, /**< RX all multicast mode ON(1) / OFF(0). */
-               dev_started : 1,   /**< Device state: STARTED(1) / STOPPED(0). */
-               lro         : 1,   /**< RX LRO is ON(1) / OFF(0) */
-               dev_configured : 1;
-               /**< Indicates whether the device is configured.
-                *   CONFIGURED(1) / NOT CONFIGURED(0).
-                */
-       uint8_t rx_queue_state[RTE_MAX_QUEUES_PER_PORT];
-               /**< Queues state: HAIRPIN(2) / STARTED(1) / STOPPED(0). */
-       uint8_t tx_queue_state[RTE_MAX_QUEUES_PER_PORT];
-               /**< Queues state: HAIRPIN(2) / STARTED(1) / STOPPED(0). */
-       uint32_t dev_flags;             /**< Capabilities. */
-       int numa_node;                  /**< NUMA node connection. */
-       struct rte_vlan_filter_conf vlan_filter_conf;
-                       /**< VLAN filter configuration. */
-       struct rte_eth_dev_owner owner; /**< The port owner. */
-       uint16_t representor_id;
-                       /**< Switch-specific identifier.
-                        *   Valid if RTE_ETH_DEV_REPRESENTOR in dev_flags.
-                        */
-       uint16_t backer_port_id;
-                       /**< Port ID of the backing device.
-                        *   This device will be used to query representor
-                        *   info and calculate representor IDs.
-                        *   Valid if RTE_ETH_DEV_REPRESENTOR in dev_flags.
-                        */
-
-       pthread_mutex_t flow_ops_mutex; /**< rte_flow ops mutex. */
-       uint64_t reserved_64s[4]; /**< Reserved for future fields */
-       void *reserved_ptrs[4];   /**< Reserved for future fields */
-} __rte_cache_aligned;
-
-/**
- * @internal
- * The pool of *rte_eth_dev* structures. The size of the pool
- * is configured at compile-time in the <rte_ethdev.c> file.
- */
-extern struct rte_eth_dev rte_eth_devices[];
-
 #endif /* _RTE_ETHDEV_CORE_H_ */
index ca81f5d..96ac8ab 100644 (file)
@@ -73,7 +73,6 @@ DPDK_22 {
        rte_eth_dev_udp_tunnel_port_add;
        rte_eth_dev_udp_tunnel_port_delete;
        rte_eth_dev_vlan_filter;
-       rte_eth_devices;
        rte_eth_find_next;
        rte_eth_find_next_of;
        rte_eth_find_next_owned_by;
@@ -269,6 +268,7 @@ INTERNAL {
        rte_eth_dev_release_port;
        rte_eth_dev_internal_reset;
        rte_eth_devargs_parse;
+       rte_eth_devices;
        rte_eth_dma_zone_free;
        rte_eth_dma_zone_reserve;
        rte_eth_hairpin_queue_peer_bind;
index 13dfb28..89c4ca5 100644 (file)
@@ -11,7 +11,7 @@
 #include <rte_common.h>
 #include <rte_dev.h>
 #include <rte_errno.h>
-#include <rte_ethdev.h>
+#include <ethdev_driver.h>
 #include <rte_log.h>
 #include <rte_malloc.h>
 #include <rte_service_component.h>
index 18c0359..1c06c87 100644 (file)
@@ -3,7 +3,7 @@
  */
 #include <rte_spinlock.h>
 #include <rte_service_component.h>
-#include <rte_ethdev.h>
+#include <ethdev_driver.h>
 
 #include "eventdev_pmd.h"
 #include "rte_eventdev_trace.h"
index e347d6d..ebef5f0 100644 (file)
@@ -29,7 +29,7 @@
 #include <rte_common.h>
 #include <rte_malloc.h>
 #include <rte_errno.h>
-#include <rte_ethdev.h>
+#include <ethdev_driver.h>
 #include <rte_cryptodev.h>
 #include <cryptodev_pmd.h>
 #include <rte_telemetry.h>
index 269f8ef..5be21b2 100644 (file)
@@ -2,7 +2,7 @@
  * Copyright(c) 2020 Intel Corporation
  */
 
-#include <rte_ethdev.h>
+#include <ethdev_driver.h>
 #include <rte_string_fns.h>
 #ifdef RTE_LIB_TELEMETRY
 #include <telemetry_internal.h>