]> git.droids-corp.org - dpdk.git/commitdiff
kni: remove deprecated functions
authorStephen Hemminger <shemming@brocade.com>
Mon, 15 Jun 2015 16:51:12 +0000 (09:51 -0700)
committerThomas Monjalon <thomas.monjalon@6wind.com>
Thu, 3 Sep 2015 17:22:48 +0000 (19:22 +0200)
These functions were tagged as deprecated in 2.0 so they can be
removed in 2.2.
The library version is incremented.

Signed-off-by: Stephen Hemminger <stephen@networkplumber.org>
Acked-by: Helin Zhang <helin.zhang@intel.com>
[Thomas: update doc and version]
Signed-off-by: Thomas Monjalon <thomas.monjalon@6wind.com>
Acked-by: Neil Horman <nhorman@tuxdriver.com>
app/test/Makefile
app/test/test_kni.c
doc/guides/prog_guide/kernel_nic_interface.rst
doc/guides/rel_notes/deprecation.rst
doc/guides/rel_notes/release_2_2.rst
doc/guides/sample_app_ug/kernel_nic_interface.rst
lib/librte_kni/Makefile
lib/librte_kni/rte_kni.c
lib/librte_kni/rte_kni.h
lib/librte_kni/rte_kni_version.map

index e7f148f6b369f2f3d149566aa6b88b8330c07068..7778e1c9f7d7a631df5f7b3f75aa64a1147d8e84 100644 (file)
@@ -146,12 +146,6 @@ SRCS-$(CONFIG_RTE_LIBRTE_KVARGS) += test_kvargs.c
 CFLAGS += -O3
 CFLAGS += $(WERROR_FLAGS)
 
-# Disable warnings of deprecated-declarations in test_kni.c
-ifeq ($(CC), icc)
-CFLAGS_test_kni.o += -wd1478
-else
-CFLAGS_test_kni.o += -Wno-deprecated-declarations
-endif
 CFLAGS += -D_GNU_SOURCE
 
 # Disable VTA for memcpy test
index 506b543a68792e8acc701f4d97644e0434cb34de..9dad98898899acc1fe8a6bce848e14ee1b507a72 100644 (file)
@@ -398,17 +398,6 @@ test_kni_processing(uint8_t port_id, struct rte_mempool *mp)
                printf("fail to create kni\n");
                return -1;
        }
-       if (rte_kni_get_port_id(kni) != port_id) {
-               printf("fail to get port id\n");
-               ret = -1;
-               goto fail_kni;
-       }
-
-       if (rte_kni_info_get(RTE_MAX_ETHPORTS)) {
-               printf("Unexpectedly get a KNI successfully\n");
-               ret = -1;
-               goto fail_kni;
-       }
 
        test_kni_ctx = kni;
        test_kni_processing_flag = 0;
@@ -591,14 +580,6 @@ test_kni(void)
                goto fail;
        }
 
-       /* test of getting port id according to NULL kni context */
-       if (rte_kni_get_port_id(NULL) < RTE_MAX_ETHPORTS) {
-               ret = -1;
-               printf("unexpectedly get port id successfully by NULL kni "
-                                                               "pointer\n");
-               goto fail;
-       }
-
        /* test of releasing NULL kni context */
        ret = rte_kni_release(NULL);
        if (ret == 0) {
@@ -645,23 +626,6 @@ test_kni(void)
                goto fail;
        }
 
-       /* test the interface of creating a KNI, for backward compatibility */
-       memset(&ops, 0, sizeof(ops));
-       ops = kni_ops;
-       kni = rte_kni_create(port_id, MAX_PACKET_SZ, mp, &ops);
-       if (!kni) {
-               ret = -1;
-               printf("Fail to create a KNI device for port %d\n", port_id);
-               goto fail;
-       }
-
-       ret = rte_kni_release(kni);
-       if (ret < 0) {
-               printf("Fail to release a KNI device\n");
-               goto fail;
-       }
-
-       ret = 0;
 
 fail:
        rte_eth_dev_stop(port_id);
index 713d30b13ef379562c2ec112fadb838eddae17a2..0d91476198872455230f6f1c097d38f8fbac0ad0 100644 (file)
@@ -100,8 +100,6 @@ Refer to rte_kni_common.h in the DPDK source code for more details.
 
 The physical addresses will be re-mapped into the kernel address space and stored in separate KNI contexts.
 
-Once KNI interfaces are created, the KNI context information can be queried by calling the rte_kni_info_get() function.
-
 The KNI interfaces can be deleted by a DPDK application dynamically after being created.
 Furthermore, all those KNI interfaces not deleted will be deleted on the release operation
 of the miscellaneous device (when the DPDK application is closed).
index e7e213cb0fb78a41d26e023d988b871bb76a7f3d..04819fa2746f5e387cac9e10436ee737162d1bab 100644 (file)
@@ -40,9 +40,6 @@ Deprecation Notices
   the tunnel type, TNI/VNI, inner MAC and inner VLAN are monitored.
   The release 2.2 will contain these changes without backwards compatibility.
 
-* librte_kni: Functions based on port id are deprecated for a long time and
-  should be removed (rte_kni_create, rte_kni_get_port_id and rte_kni_info_get).
-
 * librte_pmd_ring: The deprecated functions rte_eth_ring_pair_create and
   rte_eth_ring_pair_attach should be removed.
 
index 6e73092085b427e27a56e30c9c00e7dc37a577d1..6dcfe8872848d7f40b369877e56ecd865a8c7ea9 100644 (file)
@@ -21,6 +21,9 @@ API Changes
 
 * The deprecated ACL API ipv4vlan is removed.
 
+* The deprecated KNI functions are removed:
+  rte_kni_create(), rte_kni_get_port_id() and rte_kni_info_get().
+
 
 ABI Changes
 -----------
@@ -56,7 +59,7 @@ The libraries prepended with a plus sign were incremented in this version.
      librte_ip_frag.so.1
      librte_ivshmem.so.1
      librte_jobstats.so.1
-     librte_kni.so.1
+   + librte_kni.so.2
      librte_kvargs.so.1
    + librte_lpm.so.2
    + librte_mbuf.so.2
index 02dde59499f3025d8fac006da11de372eed77ada..f1deca996983db88a36c6a63a55b80df3fd247ee 100644 (file)
@@ -242,15 +242,6 @@ Setup of mbuf pool, driver and queues is similar to the setup done in the L2 For
 In addition, one or more kernel NIC interfaces are allocated for each
 of the configured ports according to the command line parameters.
 
-The code for creating the kernel NIC interface for a specific port is as follows:
-
-.. code-block:: c
-
-    kni = rte_kni_create(port, MAX_PACKET_SZ, pktmbuf_pool, &kni_ops);
-    if (kni == NULL)
-        rte_exit(EXIT_FAILURE, "Fail to create kni dev "
-           "for port: %d\n", port);
-
 The code for allocating the kernel NIC interfaces for a specific port is as follows:
 
 .. code-block:: c
index 7107832f6f80452851dbe47167755fe0a4b17805..1398164d44b920e9104ef76fa17c573f8e670117 100644 (file)
@@ -38,7 +38,7 @@ CFLAGS += $(WERROR_FLAGS) -I$(SRCDIR) -O3 -fno-strict-aliasing
 
 EXPORT_MAP := rte_kni_version.map
 
-LIBABIVER := 1
+LIBABIVER := 2
 
 # all source are stored in SRCS-y
 SRCS-$(CONFIG_RTE_LIBRTE_KNI) := rte_kni.c
index 08155dbdfa166f14712d28802c37ae3587a9d6cd..ea9baf4caa652f26e71f10541fe47f8625b671c7 100644 (file)
@@ -311,31 +311,6 @@ kni_fail:
                         max_kni_ifaces);
 }
 
-/* It is deprecated and just for backward compatibility */
-struct rte_kni *
-rte_kni_create(uint8_t port_id,
-              unsigned mbuf_size,
-              struct rte_mempool *pktmbuf_pool,
-              struct rte_kni_ops *ops)
-{
-       struct rte_kni_conf conf;
-       struct rte_eth_dev_info info;
-
-       memset(&info, 0, sizeof(info));
-       memset(&conf, 0, sizeof(conf));
-       rte_eth_dev_info_get(port_id, &info);
-
-       snprintf(conf.name, sizeof(conf.name), "vEth%u", port_id);
-       conf.addr = info.pci_dev->addr;
-       conf.id = info.pci_dev->id;
-       conf.group_id = (uint16_t)port_id;
-       conf.mbuf_size = mbuf_size;
-
-       /* Save the port id for request handling */
-       ops->port_id = port_id;
-
-       return rte_kni_alloc(pktmbuf_pool, &conf, ops);
-}
 
 struct rte_kni *
 rte_kni_alloc(struct rte_mempool *pktmbuf_pool,
@@ -650,16 +625,6 @@ kni_allocate_mbufs(struct rte_kni *kni)
        }
 }
 
-/* It is deprecated and just for backward compatibility */
-uint8_t
-rte_kni_get_port_id(struct rte_kni *kni)
-{
-       if (!kni)
-               return ~0x0;
-
-       return kni->ops.port_id;
-}
-
 struct rte_kni *
 rte_kni_get(const char *name)
 {
@@ -686,22 +651,6 @@ rte_kni_get_name(const struct rte_kni *kni)
        return kni->name;
 }
 
-/*
- * It is deprecated and just for backward compatibility.
- */
-struct rte_kni *
-rte_kni_info_get(uint8_t port_id)
-{
-       char name[RTE_MEMZONE_NAMESIZE];
-
-       if (port_id >= RTE_MAX_ETHPORTS)
-               return NULL;
-
-       snprintf(name, RTE_MEMZONE_NAMESIZE, "vEth%u", port_id);
-
-       return rte_kni_get(name);
-}
-
 static enum kni_ops_status
 kni_check_request_register(struct rte_kni_ops *ops)
 {
index 52ffdb73778a719263b6d3e462716d6f39b63760..ef9faa9f158e78a28eca5f116c900916aeee922b 100644 (file)
@@ -128,30 +128,6 @@ extern struct rte_kni *rte_kni_alloc(struct rte_mempool *pktmbuf_pool,
                                     const struct rte_kni_conf *conf,
                                     struct rte_kni_ops *ops);
 
-/**
- * It create a KNI device for specific port.
- *
- * Note: It is deprecated and just for backward compatibility.
- *
- * @param port_id
- *  Port ID.
- * @param mbuf_size
- *  mbuf size.
- * @param pktmbuf_pool
- *  The mempool for allocting mbufs for packets.
- * @param ops
- *  The pointer to the callbacks for the KNI kernel requests.
- *
- * @return
- *  - The pointer to the context of a KNI interface.
- *  - NULL indicate error.
- */
-extern struct rte_kni *rte_kni_create(uint8_t port_id,
-                                     unsigned mbuf_size,
-                                     struct rte_mempool *pktmbuf_pool,
-                                     struct rte_kni_ops *ops) \
-                                     __attribute__ ((deprecated));
-
 /**
  * Release KNI interface according to the context. It will also release the
  * paired KNI interface in kernel space. All processing on the specific KNI
@@ -220,21 +196,6 @@ extern unsigned rte_kni_rx_burst(struct rte_kni *kni,
 extern unsigned rte_kni_tx_burst(struct rte_kni *kni,
                struct rte_mbuf **mbufs, unsigned num);
 
-/**
- * Get the port id from KNI interface.
- *
- * Note: It is deprecated and just for backward compatibility.
- *
- * @param kni
- *  The KNI interface context.
- *
- * @return
- *  On success: The port id.
- *  On failure: ~0x0
- */
-extern uint8_t rte_kni_get_port_id(struct rte_kni *kni) \
-                               __attribute__ ((deprecated));
-
 /**
  * Get the KNI context of its name.
  *
@@ -257,21 +218,6 @@ extern struct rte_kni *rte_kni_get(const char *name);
  */
 extern const char *rte_kni_get_name(const struct rte_kni *kni);
 
-/**
- * Get the KNI context of the specific port.
- *
- * Note: It is deprecated and just for backward compatibility.
- *
- * @param port_id
- *  the port id.
- *
- * @return
- *  On success: Pointer to KNI interface.
- *  On failure: NULL
- */
-extern struct rte_kni *rte_kni_info_get(uint8_t port_id) \
-                               __attribute__ ((deprecated));
-
 /**
  * Register KNI request handling for a specified port,and it can
  * be called by master process or slave process.
index a987d31f31b43371bf22c6abfa73eded7f5b44ed..acd515eb0d2b00c1a0ead9fe4479c8842a83d55d 100644 (file)
@@ -3,12 +3,9 @@ DPDK_2.0 {
 
        rte_kni_alloc;
        rte_kni_close;
-       rte_kni_create;
        rte_kni_get;
        rte_kni_get_name;
-       rte_kni_get_port_id;
        rte_kni_handle_request;
-       rte_kni_info_get;
        rte_kni_init;
        rte_kni_register_handlers;
        rte_kni_release;