[RFC] ethdev: make offload name API non-experimental
Checks
Commit Message
The offload name functions are useful, but since they are
marked experimental they can not be used by upstream projects.
For example, VPP duplicates the same table in its code.
Signed-off-by: Stephen Hemminger <stephen@networkplumber.org>
---
lib/librte_ethdev/rte_ethdev.c | 4 ++--
lib/librte_ethdev/rte_ethdev.h | 10 ++--------
lib/librte_ethdev/rte_ethdev_version.map | 9 +++++++--
3 files changed, 11 insertions(+), 12 deletions(-)
Comments
On 10/19/2018 6:35 PM, Stephen Hemminger wrote:
> The offload name functions are useful, but since they are
> marked experimental they can not be used by upstream projects.
> For example, VPP duplicates the same table in its code.
>
> Signed-off-by: Stephen Hemminger <stephen@networkplumber.org>
+1, they have been added on 18.02, and I think we can remove experimental tag
for them now.
On 10/22/2018 3:39 PM, Ferruh Yigit wrote:
> On 10/19/2018 6:35 PM, Stephen Hemminger wrote:
>> The offload name functions are useful, but since they are
>> marked experimental they can not be used by upstream projects.
>> For example, VPP duplicates the same table in its code.
>>
>> Signed-off-by: Stephen Hemminger <stephen@networkplumber.org>
>
> +1, they have been added on 18.02, and I think we can remove experimental tag
> for them now.
Is there any objection to this, if not I am for getting change for rc1?
On 10/19/2018 6:35 PM, Stephen Hemminger wrote:
> The offload name functions are useful, but since they are
> marked experimental they can not be used by upstream projects.
> For example, VPP duplicates the same table in its code.
>
> Signed-off-by: Stephen Hemminger <stephen@networkplumber.org>
Reviewed-by: Ferruh Yigit <ferruh.yigit@intel.com>
> +DPDK_18.11 {
> + global:
> +
> + rte_eth_dev_rx_offload_name;
> + rte_eth_dev_tx_offload_name;
> +} DPDK_18.11;
DPDK_18.08, can be fixed while merging.
On 10/26/2018 3:47 PM, Ferruh Yigit wrote:
> On 10/19/2018 6:35 PM, Stephen Hemminger wrote:
>> The offload name functions are useful, but since they are
>> marked experimental they can not be used by upstream projects.
>> For example, VPP duplicates the same table in its code.
>>
>> Signed-off-by: Stephen Hemminger <stephen@networkplumber.org>
>
> Reviewed-by: Ferruh Yigit <ferruh.yigit@intel.com>
Applied to dpdk-next-net/master, thanks.
@@ -983,7 +983,7 @@ rte_eth_speed_bitflag(uint32_t speed, int duplex)
}
}
-const char * __rte_experimental
+const char *
rte_eth_dev_rx_offload_name(uint64_t offload)
{
const char *name = "UNKNOWN";
@@ -999,7 +999,7 @@ rte_eth_dev_rx_offload_name(uint64_t offload)
return name;
}
-const char * __rte_experimental
+const char *
rte_eth_dev_tx_offload_name(uint64_t offload)
{
const char *name = "UNKNOWN";
@@ -1465,9 +1465,6 @@ int rte_eth_dev_detach(uint16_t port_id, char *devname);
uint32_t rte_eth_speed_bitflag(uint32_t speed, int duplex);
/**
- * @warning
- * @b EXPERIMENTAL: this API may change without prior notice
- *
* Get DEV_RX_OFFLOAD_* flag name.
*
* @param offload
@@ -1475,12 +1472,9 @@ uint32_t rte_eth_speed_bitflag(uint32_t speed, int duplex);
* @return
* Offload name or 'UNKNOWN' if the flag cannot be recognised.
*/
-const char * __rte_experimental rte_eth_dev_rx_offload_name(uint64_t offload);
+const char *rte_eth_dev_rx_offload_name(uint64_t offload);
/**
- * @warning
- * @b EXPERIMENTAL: this API may change without prior notice
- *
* Get DEV_TX_OFFLOAD_* flag name.
*
* @param offload
@@ -1488,7 +1482,7 @@ const char * __rte_experimental rte_eth_dev_rx_offload_name(uint64_t offload);
* @return
* Offload name or 'UNKNOWN' if the flag cannot be recognised.
*/
-const char * __rte_experimental rte_eth_dev_tx_offload_name(uint64_t offload);
+const char *rte_eth_dev_tx_offload_name(uint64_t offload);
/**
* Configure an Ethernet device.
@@ -220,6 +220,13 @@ DPDK_18.08 {
} DPDK_18.05;
+DPDK_18.11 {
+ global:
+
+ rte_eth_dev_rx_offload_name;
+ rte_eth_dev_tx_offload_name;
+} DPDK_18.11;
+
EXPERIMENTAL {
global:
@@ -236,8 +243,6 @@ EXPERIMENTAL {
rte_eth_dev_owner_set;
rte_eth_dev_owner_unset;
rte_eth_dev_rx_intr_ctl_q_get_fd;
- rte_eth_dev_rx_offload_name;
- rte_eth_dev_tx_offload_name;
rte_eth_switch_domain_alloc;
rte_eth_switch_domain_free;
rte_flow_conv;