diff mbox series

ethdev: remove experimental flag from getting intr fd API

Message ID 20210901015054.340542-1-xiaoyun.li@intel.com (mailing list archive)
State Accepted, archived
Delegated to: Ferruh Yigit
Headers
Series ethdev: remove experimental flag from getting intr fd API |

Checks

Context Check Description
ci/checkpatch success coding style OK
ci/github-robot: build success github build: passed
ci/iol-broadcom-Performance success Performance Testing PASS
ci/iol-broadcom-Functional success Functional Testing PASS
ci/iol-intel-Functional success Functional Testing PASS
ci/iol-intel-Performance success Performance Testing PASS
ci/iol-aarch64-compile-testing success Testing PASS
ci/Intel-compilation success Compilation OK
ci/intel-Testing success Testing PASS
ci/iol-mellanox-Performance success Performance Testing PASS
ci/iol-x86_64-unit-testing success Testing PASS

Commit Message

Li, Xiaoyun Sept. 1, 2021, 1:50 a.m. UTC 
  Remove the experimental tag for rte_eth_dev_rx_intr_ctl_q_get_fd API
that was introduced in 18.11 and have been around for 11 releases.

Signed-off-by: Xiaoyun Li <xiaoyun.li@intel.com>
---
 lib/ethdev/rte_ethdev.h | 4 ----
 lib/ethdev/version.map  | 2 +-
 2 files changed, 1 insertion(+), 5 deletions(-)

Comments

Andrew Rybchenko Sept. 1, 2021, 7:08 a.m. UTC | #1 
On 9/1/21 4:50 AM, Xiaoyun Li wrote:
> Remove the experimental tag for rte_eth_dev_rx_intr_ctl_q_get_fd API
> that was introduced in 18.11 and have been around for 11 releases.
> 
> Signed-off-by: Xiaoyun Li <xiaoyun.li@intel.com>

Acked-by: Andrew Rybchenko <andrew.rybchenko@oktetlabs.ru>
Ferruh Yigit Sept. 1, 2021, 8:53 a.m. UTC | #2 
On 9/1/2021 8:08 AM, Andrew Rybchenko wrote:
> On 9/1/21 4:50 AM, Xiaoyun Li wrote:
>> Remove the experimental tag for rte_eth_dev_rx_intr_ctl_q_get_fd API
>> that was introduced in 18.11 and have been around for 11 releases.
>>
>> Signed-off-by: Xiaoyun Li <xiaoyun.li@intel.com>
> 
> Acked-by: Andrew Rybchenko <andrew.rybchenko@oktetlabs.ru>
> 

Acked-by: Ferruh Yigit <ferruh.yigit@intel.com>
Ray Kinsella Sept. 1, 2021, 11:17 a.m. UTC | #3 
On 01/09/2021 09:53, Ferruh Yigit wrote:
> On 9/1/2021 8:08 AM, Andrew Rybchenko wrote:
>> On 9/1/21 4:50 AM, Xiaoyun Li wrote:
>>> Remove the experimental tag for rte_eth_dev_rx_intr_ctl_q_get_fd API
>>> that was introduced in 18.11 and have been around for 11 releases.
>>>
>>> Signed-off-by: Xiaoyun Li <xiaoyun.li@intel.com>
>>
>> Acked-by: Andrew Rybchenko <andrew.rybchenko@oktetlabs.ru>
>>
> 
> Acked-by: Ferruh Yigit <ferruh.yigit@intel.com>
> 
Acked-by: Ray Kinsella <mdr@ashroe.eu>
David Marchand Sept. 3, 2021, 12:17 p.m. UTC | #4 
On Wed, Sep 1, 2021 at 4:03 AM Xiaoyun Li <xiaoyun.li@intel.com> wrote:
>
> Remove the experimental tag for rte_eth_dev_rx_intr_ctl_q_get_fd API
> that was introduced in 18.11 and have been around for 11 releases.
>
> Signed-off-by: Xiaoyun Li <xiaoyun.li@intel.com>

Odd that this API is used nowhere in-tree but it provides a simple
alternative to the DPDK interrupt builtin framework.
I had a go with this in OVS, and it seems to work as described (once
fixed virtio-user).

Acked-by: David Marchand <david.marchand@redhat.com>
Ray Kinsella Sept. 3, 2021, 12:19 p.m. UTC | #5 
On 03/09/2021 13:17, David Marchand wrote:
> On Wed, Sep 1, 2021 at 4:03 AM Xiaoyun Li <xiaoyun.li@intel.com> wrote:
>>
>> Remove the experimental tag for rte_eth_dev_rx_intr_ctl_q_get_fd API
>> that was introduced in 18.11 and have been around for 11 releases.
>>
>> Signed-off-by: Xiaoyun Li <xiaoyun.li@intel.com>
> 
> Odd that this API is used nowhere in-tree but it provides a simple
> alternative to the DPDK interrupt builtin framework.
> I had a go with this in OVS, and it seems to work as described (once
> fixed virtio-user).
> 
> Acked-by: David Marchand <david.marchand@redhat.com>
> 

FD.io VPP uses it, in its interrupt mgmt framework.

Acked-by: Ray Kinsella <mdr@ashroe.eu>

Ray K
Ferruh Yigit Sept. 14, 2021, 4:13 p.m. UTC | #6 
On 9/1/2021 12:17 PM, Kinsella, Ray wrote:
> 
> 
> On 01/09/2021 09:53, Ferruh Yigit wrote:
>> On 9/1/2021 8:08 AM, Andrew Rybchenko wrote:
>>> On 9/1/21 4:50 AM, Xiaoyun Li wrote:
>>>> Remove the experimental tag for rte_eth_dev_rx_intr_ctl_q_get_fd API
>>>> that was introduced in 18.11 and have been around for 11 releases.
>>>>
>>>> Signed-off-by: Xiaoyun Li <xiaoyun.li@intel.com>
>>>
>>> Acked-by: Andrew Rybchenko <andrew.rybchenko@oktetlabs.ru>
>>>
>>
>> Acked-by: Ferruh Yigit <ferruh.yigit@intel.com>
>>
> Acked-by: Ray Kinsella <mdr@ashroe.eu>
> 

Applied to dpdk-next-net/main, thanks.
diff mbox series

Patch

diff --git a/lib/ethdev/rte_ethdev.h b/lib/ethdev/rte_ethdev.h
index d2b27c351f..2ccd651490 100644
--- a/lib/ethdev/rte_ethdev.h
+++ b/lib/ethdev/rte_ethdev.h
@@ -3647,9 +3647,6 @@  int rte_eth_dev_rx_intr_ctl_q(uint16_t port_id, uint16_t queue_id,
 			      int epfd, int op, void *data);
 
 /**
- * @warning
- * @b EXPERIMENTAL: this API may change without prior notice.
- *
  * Get interrupt fd per Rx queue.
  *
  * @param port_id
@@ -3663,7 +3660,6 @@  int rte_eth_dev_rx_intr_ctl_q(uint16_t port_id, uint16_t queue_id,
  *           successful.
  *   - (-1) on error.
  */
-__rte_experimental
 int
 rte_eth_dev_rx_intr_ctl_q_get_fd(uint16_t port_id, uint16_t queue_id);
 
diff --git a/lib/ethdev/version.map b/lib/ethdev/version.map
index 3eece75b72..a0fc4826a1 100644
--- a/lib/ethdev/version.map
+++ b/lib/ethdev/version.map
@@ -42,6 +42,7 @@  DPDK_22 {
 	rte_eth_dev_rss_reta_update;
 	rte_eth_dev_rx_intr_ctl;
 	rte_eth_dev_rx_intr_ctl_q;
+	rte_eth_dev_rx_intr_ctl_q_get_fd;
 	rte_eth_dev_rx_intr_disable;
 	rte_eth_dev_rx_intr_enable;
 	rte_eth_dev_rx_offload_name;
@@ -154,7 +155,6 @@  EXPERIMENTAL {
 	rte_eth_dev_get_module_info;
 
 	# added in 18.11
-	rte_eth_dev_rx_intr_ctl_q_get_fd;
 	rte_flow_conv;
 
 	# added in 19.05