[PATCH v3] api: pktio: remove deprecated APIs

include/odp/api/spec/packet_io_types.h

@@ -13,7 +13,6 @@
 #define ODP_API_SPEC_PACKET_IO_TYPES_H_
 #include <odp/visibility_begin.h>
 
-#include <odp/api/deprecated.h>
 #include <odp/api/packet_types.h>
 #include <odp/api/packet_io_stats.h>
 #include <odp/api/pool_types.h>
@@ -538,14 +537,6 @@ typedef union odp_pktout_config_opt_t {
 		 */
 		uint64_t aging_ena  : 1;
 
-		/**
-		 * For backwards compatibility, setting this flag is the same as setting
-		 * tx_compl.mode_event in odp_pktio_config_t. The default value is zero.
-		 *
-		 * @deprecated Use odp_pktio_config_t::mode_event instead.
-		 */
-		uint64_t ODP_DEPRECATE(tx_compl_ena) : 1;
-
 		/** Enable packet protocol stats update */
 		uint64_t proto_stats_ena : 1;
 
@@ -1126,13 +1117,6 @@ typedef struct odp_pktio_capability_t {
 		 */
 		odp_bool_t queue_type_plain;
 
-		/**
-		 * For backwards compatibility, mode_all is synonym of mode_event.
-		 *
-		 * @deprecated Use mode_event instead.
-		 */
-		uint32_t ODP_DEPRECATE(mode_all)   : 1;
-
 		/** Packet transmit completion mode ODP_PACKET_TX_COMPL_EVENT support */
 		uint32_t mode_event : 1;
 

include/odp/api/spec/packet_types.h

@@ -13,7 +13,6 @@
 #define ODP_API_SPEC_PACKET_TYPES_H_
 #include <odp/visibility_begin.h>
 
-#include <odp/api/deprecated.h>
 #include <odp/api/proto_stats_types.h>
 #include <odp/api/queue_types.h>
 
@@ -505,15 +504,6 @@ typedef enum odp_packet_tx_compl_mode_t {
 
 } odp_packet_tx_compl_mode_t;
 
-/**
- * For backwards compatibility, ODP_PACKET_TX_COMPL_ALL is synonym of ODP_PACKET_TX_COMPL_EVENT.
- *
- * @deprecated Use #ODP_PACKET_TX_COMPL_EVENT instead.
- */
-#if ODP_DEPRECATED_API
-#define ODP_PACKET_TX_COMPL_ALL ODP_PACKET_TX_COMPL_EVENT
-#endif
-
 /**
  * Packet transmit completion request options
  */

platform/linux-generic/odp_packet_io.c

@@ -7,7 +7,6 @@
 
 #include <odp/api/buffer.h>
 #include <odp/api/debug.h>
-#include <odp/api/deprecated.h>
 #include <odp/api/packet.h>
 #include <odp/api/packet_io.h>
 #include <odp/api/proto_stats.h>
@@ -652,13 +651,10 @@ int odp_pktio_config(odp_pktio_t hdl, const odp_pktio_config_t *config)
 	entry->config = *config;
 
 	entry->enabled.tx_ts = config->pktout.bit.ts_ena;
-	entry->enabled.tx_compl = (config->pktout.bit.ODP_DEPRECATE(tx_compl_ena) ||
-				   config->tx_compl.mode_event ||
-				   config->tx_compl.mode_poll);
+	entry->enabled.tx_compl = (config->tx_compl.mode_event || config->tx_compl.mode_poll);
 
 	if (entry->enabled.tx_compl) {
-		if ((config->pktout.bit.ODP_DEPRECATE(tx_compl_ena) ||
-		     config->tx_compl.mode_event) && configure_tx_event_compl(entry)) {
+		if (config->tx_compl.mode_event && configure_tx_event_compl(entry)) {
 			unlock_entry(entry);
 			_ODP_ERR("Unable to configure Tx event completion\n");
 			return -1;

platform/linux-generic/pktio/dpdk.c

@@ -10,7 +10,6 @@
 #include <odp_posix_extensions.h>
 
 #include <odp/api/cpumask.h>
-#include <odp/api/deprecated.h>
 #include <odp/api/hints.h>
 #include <odp/api/packet.h>
 #include <odp/api/packet_io.h>
@@ -1768,10 +1767,6 @@ static int dpdk_init_capability(pktio_entry_t *pktio_entry,
 	capa->config.pktout.bit.ts_ena = 1;
 
 	if (!_ODP_DPDK_ZERO_COPY) {
-#if ODP_DEPRECATED_API
-		capa->config.pktout.bit.tx_compl_ena = 1;
-		capa->tx_compl.mode_all = 1;
-#endif
 		capa->tx_compl.mode_event = 1;
 		capa->tx_compl.mode_poll = 1;
 		capa->free_ctrl.dont_free = 1;

platform/linux-generic/pktio/ipc.c

@@ -3,7 +3,6 @@
  * Copyright (c) 2019-2022 Nokia
  */
 
-#include <odp/api/deprecated.h>
 #include <odp/api/hints.h>
 #include <odp/api/pool.h>
 #include <odp/api/system_info.h>
@@ -920,10 +919,7 @@ static int ipc_capability(pktio_entry_t *pktio_entry ODP_UNUSED, odp_pktio_capab
 
 	capa->max_input_queues  = 1;
 	capa->max_output_queues = 1;
-#if ODP_DEPRECATED_API
-	capa->config.pktout.bit.tx_compl_ena = 1;
-	capa->tx_compl.mode_all = 1;
-#endif
+
 	capa->tx_compl.mode_event = 1;
 	capa->tx_compl.mode_poll = 1;
 

platform/linux-generic/pktio/loop.c

@@ -4,7 +4,6 @@
  */
 
 #include <odp/api/debug.h>
-#include <odp/api/deprecated.h>
 #include <odp/api/event.h>
 #include <odp/api/hash.h>
 #include <odp/api/hints.h>
@@ -696,10 +695,7 @@ static int loopback_init_capability(pktio_entry_t *pktio_entry)
 	capa->config.pktout.bit.udp_chksum = 1;
 	capa->config.pktout.bit.sctp_chksum = 1;
 	capa->config.pktout.bit.ts_ena = 1;
-#if ODP_DEPRECATED_API
-	capa->config.pktout.bit.tx_compl_ena = 1;
-	capa->tx_compl.mode_all = 1;
-#endif
+
 	capa->tx_compl.mode_event = 1;
 	capa->tx_compl.mode_poll = 1;
 

platform/linux-generic/pktio/null.c

@@ -4,7 +4,6 @@
  */
 
 #include <odp/api/debug.h>
-#include <odp/api/deprecated.h>
 #include <odp/api/hints.h>
 #include <odp/api/packet_io.h>
 
@@ -138,10 +137,7 @@ static int null_capability(pktio_entry_t *pktio_entry ODP_UNUSED,
 	capa->config.pktin.bit.ts_ptp = 1;
 
 	capa->config.pktout.bit.ts_ena = 1;
-#if ODP_DEPRECATED_API
-	capa->config.pktout.bit.tx_compl_ena = 1;
-	capa->tx_compl.mode_all = 1;
-#endif
+
 	capa->tx_compl.mode_event = 1;
 	capa->tx_compl.mode_poll = 1;
 

platform/linux-generic/pktio/pcap.c

@@ -37,7 +37,6 @@
 #include <odp_posix_extensions.h>
 
 #include <odp/api/debug.h>
-#include <odp/api/deprecated.h>
 #include <odp/api/hints.h>
 #include <odp/api/packet.h>
 #include <odp/api/packet_io.h>
@@ -506,10 +505,7 @@ static int pcapif_capability(pktio_entry_t *pktio_entry ODP_UNUSED,
 	capa->config.pktin.bit.ts_ptp = 1;
 
 	capa->config.pktout.bit.ts_ena = 1;
-#if ODP_DEPRECATED_API
-	capa->config.pktout.bit.tx_compl_ena = 1;
-	capa->tx_compl.mode_all = 1;
-#endif
+
 	capa->tx_compl.mode_event = 1;
 	capa->tx_compl.mode_poll = 1;
 

platform/linux-generic/pktio/socket.c

@@ -7,7 +7,6 @@
 
 #include <odp/api/align.h>
 #include <odp/api/debug.h>
-#include <odp/api/deprecated.h>
 #include <odp/api/hints.h>
 #include <odp/api/packet.h>
 #include <odp/api/packet_io.h>
@@ -586,10 +585,7 @@ static int sock_capability(pktio_entry_t *pktio_entry,
 	capa->config.pktin.bit.ts_ptp = 1;
 
 	capa->config.pktout.bit.ts_ena = 1;
-#if ODP_DEPRECATED_API
-	capa->config.pktout.bit.tx_compl_ena = 1;
-	capa->tx_compl.mode_all = 1;
-#endif
+
 	capa->tx_compl.mode_event = 1;
 	capa->tx_compl.mode_poll = 1;
 

platform/linux-generic/pktio/socket_mmap.c

@@ -6,7 +6,6 @@
 #include <odp_posix_extensions.h>
 
 #include <odp/api/debug.h>
-#include <odp/api/deprecated.h>
 #include <odp/api/hints.h>
 #include <odp/api/packet.h>
 #include <odp/api/packet_io.h>
@@ -890,10 +889,7 @@ static int sock_mmap_capability(pktio_entry_t *pktio_entry,
 	capa->config.pktin.bit.ts_ptp = 1;
 
 	capa->config.pktout.bit.ts_ena = 1;
-#if ODP_DEPRECATED_API
-	capa->config.pktout.bit.tx_compl_ena = 1;
-	capa->tx_compl.mode_all = 1;
-#endif
+
 	capa->tx_compl.mode_event = 1;
 	capa->tx_compl.mode_poll = 1;
 

platform/linux-generic/pktio/tap.c

@@ -29,7 +29,6 @@
 #include <odp_posix_extensions.h>
 
 #include <odp/api/debug.h>
-#include <odp/api/deprecated.h>
 #include <odp/api/hints.h>
 #include <odp/api/packet_io.h>
 #include <odp/api/random.h>
@@ -535,10 +534,7 @@ static int tap_capability(pktio_entry_t *pktio_entry ODP_UNUSED,
 	capa->config.pktin.bit.ts_ptp = 1;
 
 	capa->config.pktout.bit.ts_ena = 1;
-#if ODP_DEPRECATED_API
-	capa->config.pktout.bit.tx_compl_ena = 1;
-	capa->tx_compl.mode_all = 1;
-#endif
+
 	capa->tx_compl.mode_event = 1;
 	capa->tx_compl.mode_poll = 1;
 

test/validation/api/pktio/pktio.c

@@ -3846,10 +3846,7 @@ static void pktio_test_pktout_compl_event(bool use_plain_queue, uint32_t test_fl
 		/* Configure Tx completion offload for PKTIO Tx */
 		if (i == 0) {
 			CU_ASSERT_FATAL(pktio_capa.tx_compl.mode_event == 1);
-#if ODP_DEPRECATED_API
-			CU_ASSERT_FATAL(pktio_capa.tx_compl.mode_all ==
-					pktio_capa.tx_compl.mode_event);
-#endif
+
 			if (use_plain_queue) {
 				CU_ASSERT_FATAL(pktio_capa.tx_compl.queue_type_plain != 0);
 			} else {
@@ -3884,13 +3881,10 @@ static void pktio_test_pktout_compl_event(bool use_plain_queue, uint32_t test_fl
 	/* Disabled by default */
 	CU_ASSERT(odp_packet_has_tx_compl_request(pkt_tbl[0]) == 0);
 
-	/* Check that disable works. Also COMPL_ALL should be still supported. */
+	/* Check that disable works */
 	opt.queue = compl_queue[0];
-#if ODP_DEPRECATED_API
-	opt.mode = ODP_PACKET_TX_COMPL_ALL;
-#else
 	opt.mode = ODP_PACKET_TX_COMPL_EVENT;
-#endif
+
 	odp_packet_tx_compl_request(pkt_tbl[0], &opt);
 	CU_ASSERT(odp_packet_has_tx_compl_request(pkt_tbl[0]) != 0);
 	opt.mode = ODP_PACKET_TX_COMPL_DISABLED;