]> git.droids-corp.org - dpdk.git/commitdiff
config: remove compatibility build defines
authorBruce Richardson <bruce.richardson@intel.com>
Wed, 9 Dec 2020 17:14:20 +0000 (17:14 +0000)
committerThomas Monjalon <thomas@monjalon.net>
Wed, 20 Jan 2021 00:43:25 +0000 (01:43 +0100)
As announced in the deprecation note, remove all compatibility build
defines from previous make/meson versions and use only the standardized
ones - RTE_LIB_<name> for libraries, and RTE_<CLASS>_<NAME> for drivers.

Signed-off-by: Bruce Richardson <bruce.richardson@intel.com>
Signed-off-by: Thomas Monjalon <thomas@monjalon.net>
29 files changed:
config/meson.build
config/rte_compatibility_defines.h [deleted file]
config/rte_config.h
doc/guides/rel_notes/deprecation.rst
doc/guides/rel_notes/release_21_02.rst
drivers/baseband/meson.build
drivers/bus/meson.build
drivers/common/meson.build
drivers/common/mlx5/meson.build
drivers/common/qat/meson.build
drivers/compress/meson.build
drivers/compress/octeontx/meson.build
drivers/crypto/meson.build
drivers/crypto/null/meson.build
drivers/crypto/octeontx/meson.build
drivers/crypto/octeontx2/meson.build
drivers/crypto/scheduler/meson.build
drivers/crypto/virtio/meson.build
drivers/event/meson.build
drivers/mempool/meson.build
drivers/meson.build
drivers/net/meson.build
drivers/raw/meson.build
drivers/regex/meson.build
drivers/regex/mlx5/meson.build
drivers/regex/octeontx2/meson.build
drivers/vdpa/meson.build
drivers/vdpa/mlx5/meson.build
lib/meson.build

index f3d61b170bd471025e42c448145418879693511f..1e6dfe0753b73010226e9dabd307492e8d3d7596 100644 (file)
@@ -278,7 +278,7 @@ endif
 # set the install path for the drivers
 dpdk_conf.set_quoted('RTE_EAL_PMD_PATH', eal_pmd_path)
 
-install_headers(['rte_config.h', 'rte_compatibility_defines.h'],
+install_headers(['rte_config.h'],
                subdir: get_option('include_subdir_arch'))
 
 # enable VFIO only if it is linux OS
diff --git a/config/rte_compatibility_defines.h b/config/rte_compatibility_defines.h
deleted file mode 100644 (file)
index 4760005..0000000
+++ /dev/null
@@ -1,129 +0,0 @@
-/* SPDX-License-Identifier: BSD-3-Clause
- * Copyright(c) 2017 Intel Corporation
- */
-
-#ifndef _RTE_CONFIG_H_
-#error "This file should only be included via rte_config.h"
-#endif
-
-/*
- * NOTE: these defines are for compatibility only and will be removed in a
- * future DPDK release.
- */
-
-#ifdef RTE_LIBRTE_BITRATESTATS
-#define RTE_LIBRTE_BITRATE
-#endif
-
-#ifdef RTE_LIBRTE_LATENCYSTATS
-#define RTE_LIBRTE_LATENCY_STATS
-#endif
-
-#ifdef RTE_LIBRTE_DPAAX_COMMON
-#define RTE_LIBRTE_COMMON_DPAAX
-#endif
-
-#ifdef RTE_LIBRTE_VMBUS_BUS
-#define RTE_LIBRTE_VMBUS
-#endif
-
-#ifdef RTE_LIBRTE_BUCKET_MEMPOOL
-#define RTE_DRIVER_MEMPOOL_BUCKET
-#endif
-
-#ifdef RTE_LIBRTE_RING_MEMPOOL
-#define RTE_DRIVER_MEMPOOL_RING
-#endif
-
-#ifdef RTE_LIBRTE_STACK_MEMPOOL
-#define RTE_DRIVER_MEMPOOL_STACK
-#endif
-
-#ifdef RTE_LIBRTE_AF_PACKET_PMD
-#define RTE_LIBRTE_PMD_AF_PACKET
-#endif
-
-#ifdef RTE_LIBRTE_AF_XDP_PMD
-#define RTE_LIBRTE_PMD_AF_XDP
-#endif
-
-#ifdef RTE_LIBRTE_BOND_PMD
-#define RTE_LIBRTE_PMD_BOND
-#endif
-
-#ifdef RTE_LIBRTE_E1000_PMD
-#define RTE_LIBRTE_EM_PMD
-#endif
-
-#ifdef RTE_LIBRTE_E1000_PMD
-#define RTE_LIBRTE_IGB_PMD
-#endif
-
-#ifdef RTE_LIBRTE_FAILSAFE_PMD
-#define RTE_LIBRTE_PMD_FAILSAFE
-#endif
-
-#ifdef RTE_LIBRTE_KNI_PMD
-#define RTE_LIBRTE_PMD_KNI
-#endif
-
-#ifdef RTE_LIBRTE_LIQUIDIO_PMD
-#define RTE_LIBRTE_LIO_PMD
-#endif
-
-#ifdef RTE_LIBRTE_MEMIF_PMD
-#define RTE_LIBRTE_PMD_MEMIF
-#endif
-
-#ifdef RTE_LIBRTE_NULL_PMD
-#define RTE_LIBRTE_PMD_NULL
-#endif
-
-#ifdef RTE_LIBRTE_PCAP_PMD
-#define RTE_LIBRTE_PMD_PCAP
-#endif
-
-#ifdef RTE_LIBRTE_RING_PMD
-#define RTE_LIBRTE_PMD_RING
-#endif
-
-#ifdef RTE_LIBRTE_SFC_PMD
-#define RTE_LIBRTE_SFC_EFX_PMD
-#endif
-
-#ifdef RTE_LIBRTE_SOFTNIC_PMD
-#define RTE_LIBRTE_PMD_SOFTNIC
-#endif
-
-#ifdef RTE_LIBRTE_SZEDATA2_PMD
-#define RTE_LIBRTE_PMD_SZEDATA2
-#endif
-
-#ifdef RTE_LIBRTE_TAP_PMD
-#define RTE_LIBRTE_PMD_TAP
-#endif
-
-#ifdef RTE_LIBRTE_THUNDERX_PMD
-#define RTE_LIBRTE_THUNDERX_NICVF_PMD
-#endif
-
-#ifdef RTE_LIBRTE_VHOST_PMD
-#define RTE_LIBRTE_PMD_VHOST
-#endif
-
-#ifdef RTE_LIBRTE_PMD_ARMV8
-#define RTE_LIBRTE_PMD_ARMV8_CRYPTO
-#endif
-
-#ifdef RTE_LIBRTE_PMD_MVSAM
-#define RTE_LIBRTE_PMD_MVSAM_CRYPTO
-#endif
-
-#ifdef RTE_LIBRTE_PMD_OCTEONTX_COMPRESS
-#define RTE_LIBRTE_PMD_OCTEONTX_ZIPVF
-#endif
-
-#ifdef RTE_LIBRTE_PMD_OCTEONTX_EVENTDEV
-#define RTE_LIBRTE_PMD_OCTEONTX_SSOVF
-#endif
-
index 87f9786d74223e0651ffb7658e699928c8d71a54..55a2fc50eda46f44fb95dc2dde3d22c3d38062cf 100644 (file)
@@ -13,7 +13,6 @@
 #define _RTE_CONFIG_H_
 
 #include <rte_build_config.h>
-#include "rte_compatibility_defines.h"
 
 /* legacy defines */
 #ifdef RTE_EXEC_ENV_LINUX
index 0fdaf37ed18aff978b548d2619e8113e0987c127..82c1a90a373dbe59ebf4ffcb0c517972bd3ac207 100644 (file)
@@ -11,14 +11,6 @@ here.
 Deprecation Notices
 -------------------
 
-* build: The macros defined to indicate which DPDK libraries and drivers
-  are included in the meson build are changing to a standardized format of
-  ``RTE_LIB_<NAME>`` and ``RTE_<CLASS>_<NAME>``, where ``NAME`` is the
-  upper-case component name, e.g. EAL, ETHDEV, IXGBE, and ``CLASS`` is the
-  upper-case name of the device class to which a driver belongs e.g.
-  ``NET``, ``CRYPTO``, ``VDPA``. The old macros are deprecated and will be
-  removed in a future release.
-
 * kvargs: The function ``rte_kvargs_process`` will get a new parameter
   for returning key match count. It will ease handling of no-match case.
 
index 0ee4f051862945770baaa6d844c80634ad511a05..ae36b6a3fafd79897d345dc59b6c90680d5049b3 100644 (file)
@@ -155,6 +155,13 @@ API Changes
    Also, make sure to start the actual text at the margin.
    =======================================================
 
+* config: Removed the old macros, included in ``rte_config.h``,
+  to indicate which DPDK libraries and drivers are built.
+  The new macros are generated by meson in a standardized format:
+  ``RTE_LIB_<NAME>`` and ``RTE_<CLASS>_<NAME>``, where ``NAME`` is
+  the upper-case component name, e.g. ``EAL``, ``ETHDEV``, ``VIRTIO``,
+  and ``CLASS`` is the upper-case driver class, e.g. ``NET``, ``CRYPTO``.
+
 * cryptodev: The structure ``rte_cryptodev`` has been updated with pointers
   for adding enqueue and dequeue callbacks.
 
index 2c7efe394c9c2296fd9b7c3d350e231b41fe5fdb..920e3b02ee9a3910407cdd4124a4cb631dc7eefc 100644 (file)
@@ -6,5 +6,3 @@ if is_windows
 endif
 
 drivers = ['null', 'turbo_sw', 'fpga_lte_fec', 'fpga_5gnr_fec', 'acc100']
-
-config_flag_fmt = 'RTE_LIBRTE_PMD_BBDEV_@0@'
index bd73efd1871b81e563d6711cfbbb8de558e2e065..2e7727af081f2199621d8ee402283f4b10d216b3 100644 (file)
@@ -3,4 +3,3 @@
 
 drivers = ['dpaa', 'fslmc', 'ifpga', 'pci', 'vdev', 'vmbus']
 std_deps = ['eal']
-config_flag_fmt = 'RTE_LIBRTE_@0@_BUS'
index d78882f05e0b0df9c959b2b857546707e9c0facc..ba6325adf3b56389e6a915eb9118ae4c5644d222 100644 (file)
@@ -7,4 +7,3 @@ endif
 
 std_deps = ['eal']
 drivers = ['cpt', 'dpaax', 'iavf', 'mvep', 'octeontx', 'octeontx2', 'sfc_efx']
-config_flag_fmt = 'RTE_LIBRTE_@0@_COMMON'
index 03e3e7b4a01f46fdadb0b227456787dbf251f6e6..b2efea4432dbbc475772fa18bf3c63f20cfa7a6b 100644 (file)
@@ -7,7 +7,6 @@ if not (is_linux or (is_windows and is_ms_linker))
        subdir_done()
 endif
 
-config_flag_fmt = 'RTE_LIBRTE_@0@_COMMON'
 deps += ['hash', 'pci', 'bus_pci', 'net', 'eal', 'kvargs']
 sources += files(
        'mlx5_devx_cmds.c',
index 29e1299f20b6fe0b324e40c9ac4a6d014519828f..7dd80cc53f2c26eb4308e0991c55cdec1e724176 100644 (file)
@@ -1,8 +1,6 @@
 # SPDX-License-Identifier: BSD-3-Clause
 # Copyright(c) 2017-2018 Intel Corporation
 
-config_flag_fmt = 'RTE_LIBRTE_@0@_COMMON'
-
 qat_crypto = true
 qat_crypto_path = 'crypto/qat'
 qat_crypto_relpath = '../../' + qat_crypto_path
index 33f5e33dde4a1500067de232661e51516b4376f8..cd96def086d53ec6f37f7d1c4c66ae0eb17bf898 100644 (file)
@@ -8,4 +8,3 @@ endif
 drivers = ['isal', 'octeontx', 'zlib']
 
 std_deps = ['compressdev'] # compressdev pulls in all other needed deps
-config_flag_fmt = 'RTE_LIBRTE_PMD_@0@'
index 2d71b4da219cbf25f18d3b97c40f3c61926fc68c..e1b7bed422361153350f51adc7ce18c4b34bc6c4 100644 (file)
@@ -1,7 +1,6 @@
 # SPDX-License-Identifier: BSD-3-Clause
 # Copyright(c) 2018 Cavium, Inc
 
-fmt_name = 'octeontx_compress'
 sources = files('otx_zip.c', 'otx_zip_pmd.c')
 includes += include_directories('include')
 deps += ['mempool_octeontx', 'bus_pci']
index 34f2d99c09650e9a2dfc8fb2d3fd308c043aca17..c927c5f8d3cddf22b8aada70ae00a657236c18d2 100644 (file)
@@ -26,4 +26,3 @@ drivers = ['aesni_gcm',
           'zuc']
 
 std_deps = ['cryptodev'] # cryptodev pulls in all other needed deps
-config_flag_fmt = 'RTE_LIBRTE_PMD_@0@'
index 421414916cbbac3b96a60eb7ebe0146ee4aeb3a8..1f7d644de180e4ca230a14e8841a203c44cbeca6 100644 (file)
@@ -2,5 +2,4 @@
 # Copyright(c) 2017 Intel Corporation
 
 deps += 'bus_vdev'
-fmt_name = 'null_crypto'
 sources = files('null_crypto_pmd.c', 'null_crypto_pmd_ops.c')
index 88c812d3faac81343c2f26ad5768763cf52424f9..a353d37a1050e221f51862432de4ce1b0457d6a1 100644 (file)
@@ -7,7 +7,6 @@ endif
 
 deps += ['bus_pci']
 deps += ['common_cpt']
-fmt_name = 'octeontx_crypto'
 
 sources = files('otx_cryptodev.c',
                'otx_cryptodev_capabilities.c',
index 0aad4e9a16897962f337371e5e28b10839cebe9e..283818e5dab4e5ecac3af511a0e7089e2fee8f84 100644 (file)
@@ -13,7 +13,6 @@ deps += ['common_octeontx2']
 deps += ['ethdev']
 deps += ['eventdev']
 deps += ['security']
-fmt_name = 'octeontx2_crypto'
 
 sources = files('otx2_cryptodev.c',
                'otx2_cryptodev_capabilities.c',
index 2209c502329181ccc7e03bb9d10c4bd1a6745543..b67d9306e068b3cd4e5d744ae1b3936680ac40c4 100644 (file)
@@ -2,7 +2,6 @@
 # Copyright(c) 2018 Luca Boccassi <bluca@debian.org>
 
 deps += ['bus_vdev', 'reorder']
-fmt_name = 'crypto_scheduler'
 sources = files(
        'rte_cryptodev_scheduler.c',
        'scheduler_failover.c',
index 6cea782debdd3af55bd1afa19bd60b9e4127c292..20e9e9551e87f21cf9962b970debc68ea1f474f6 100644 (file)
@@ -3,6 +3,5 @@
 
 includes += include_directories('../../../lib/librte_vhost')
 deps += 'bus_pci'
-fmt_name = 'virtio_crypto'
 sources = files('virtio_cryptodev.c', 'virtio_pci.c',
                'virtio_rxtx.c', 'virtqueue.c')
index c1a6990a0437fa956146bca116f5fb825bbf4736..a49288a5d4d8bfc9e3df5ea2c9889407893df677 100644 (file)
@@ -12,4 +12,3 @@ if not (toolchain == 'gcc' and cc.version().version_compare('<4.8.6') and
        drivers += 'octeontx'
 endif
 std_deps = ['eventdev', 'kvargs']
-config_flag_fmt = 'RTE_LIBRTE_PMD_@0@_EVENTDEV'
index a6ddd35b2bbf8c05fd4f02b0851e98f0ad78d9dd..4428813dae43c3dce363c72ed1a2dbed051d993b 100644 (file)
@@ -3,4 +3,3 @@
 
 drivers = ['bucket', 'dpaa', 'dpaa2', 'octeontx', 'octeontx2', 'ring', 'stack']
 std_deps = ['mempool']
-config_flag_fmt = 'RTE_LIBRTE_@0@_MEMPOOL'
index f9febc579e007e35e83d9d343725e3714ab936da..77f65fa9090eb9d76317dd6ded7bad6813f28a7b 100644 (file)
@@ -32,7 +32,6 @@ endif
 foreach subpath:subdirs
        drivers = []
        std_deps = []
-       config_flag_fmt = '' # format string used to set the value in dpdk_conf
 
        # subpath can be either "class" or "class/driver"
        if subpath.contains('/')
@@ -58,7 +57,6 @@ foreach subpath:subdirs
                build = true # set to false to disable, e.g. missing deps
                reason = '<unknown reason>' # set if build == false to explain
                name = drv
-               fmt_name = ''
                sources = []
                headers = []
                objs = []
@@ -114,27 +112,6 @@ foreach subpath:subdirs
                        lib_name = '_'.join(['rte', class, name])
                        dpdk_conf.set(lib_name.to_upper(), 1)
 
-                       if fmt_name == ''
-                               fmt_name = name
-                       endif
-
-                       dpdk_conf.set(config_flag_fmt.format(fmt_name.to_upper()),1) #old-style macro
-                       # for driver compatibility, since we changed the
-                       # default to match that of make. Remove in future release
-                       # after following deprecation process
-                       if config_flag_fmt.contains('_PMD_@0@') and (class == 'crypto'
-                                       or class == 'compress'
-                                       or class == 'event')
-                               alt_flag_fmt = '_@0@_PMD'.join(
-                                               config_flag_fmt.split('_PMD_@0@'))
-                               if config_flag_fmt.contains('EVENTDEV')
-                                       alt_flag_fmt = '_@0@_EVENTDEV_PMD'.join(
-                                               config_flag_fmt.split('_PMD_@0@_EVENTDEV'))
-                               endif
-                               dpdk_conf.set(alt_flag_fmt.format(
-                                               fmt_name.to_upper()), 1)
-                       endif
-
                        dpdk_extra_ldflags += pkgconfig_extra_libs
 
                        install_headers(headers)
index 4948500aef4a9b0c9dcb48060ea400bbcef190cd..4cbca96414fa55e6fdf0a58c0eec52a2582b423f 100644 (file)
@@ -57,4 +57,3 @@ drivers = ['af_packet',
 std_deps = ['ethdev', 'kvargs'] # 'ethdev' also pulls in mbuf, net, eal etc
 std_deps += ['bus_pci']         # very many PMDs depend on PCI, so make std
 std_deps += ['bus_vdev']        # same with vdev bus
-config_flag_fmt = 'RTE_LIBRTE_@0@_PMD'
index d8d6cf58241034b6204c91ead117aed4dbf6c280..8c3a036dfd19c9b0c0773177e3fce0f16d21433f 100644 (file)
@@ -11,4 +11,3 @@ drivers = ['dpaa2_cmdif', 'dpaa2_qdma',
        'octeontx2_ep',
        'skeleton']
 std_deps = ['rawdev']
-config_flag_fmt = 'RTE_LIBRTE_PMD_@0@_RAWDEV'
index 880a1b4a3b329da4fa1ae875d2272b3e371206c5..2d05d5af121d5a19f02ecd945626920e869f9fab 100644 (file)
@@ -3,4 +3,3 @@
 
 drivers = ['mlx5', 'octeontx2']
 std_deps = ['ethdev', 'kvargs'] # 'ethdev' also pulls in mbuf, net, eal etc
-config_flag_fmt = 'RTE_LIBRTE_@0@_PMD'
index d7cb2c572162920f51c326ce77488c83d971b33d..87e961181248990e5202356f88a18994564f0d47 100644 (file)
@@ -7,7 +7,6 @@ if not is_linux
        subdir_done()
 endif
 
-fmt_name = 'mlx5_regex'
 deps += ['common_mlx5', 'eal', 'regexdev']
 sources = files(
        'mlx5_regex.c',
index 34e51728c23d4f61a66967ab352e421af52d017e..6ccc8589a7c91f98313cd33d65ff1248a3d97b1b 100644 (file)
@@ -22,7 +22,6 @@ sources = files('otx2_regexdev.c',
                'otx2_regexdev_compiler.c'
                )
 
-fmt_name = 'octeontx2_regex'
 deps += ['bus_pci', 'common_octeontx2', 'regexdev']
 
 includes += include_directories('../../common/octeontx2')
index b346e43086f13c86e96a04e14a51ad62a75fda4a..4929be4c035dfd78baa0b814f66d541c425a8603 100644 (file)
@@ -9,4 +9,3 @@ drivers = ['ifc',
           'mlx5',]
 std_deps = ['bus_pci', 'kvargs']
 std_deps += ['vhost']
-config_flag_fmt = 'RTE_LIBRTE_@0@_PMD'
index 254a8ec14b450fe8383c4e4a5bd1cd011e83e339..a83ca86b3f6ae9f7768502a0834460e819fd0213 100644 (file)
@@ -7,7 +7,6 @@ if not is_linux
        subdir_done()
 endif
 
-fmt_name = 'mlx5_vdpa'
 deps += ['hash', 'common_mlx5', 'vhost', 'pci', 'eal', 'sched']
 sources = files(
        'mlx5_vdpa.c',
index d6492f7a091af4b4b90ade6118a0c1548d9b3c8d..44f0a6214240fbef95d0a925798c3d5a34bad662 100644 (file)
@@ -101,8 +101,7 @@ foreach l:libraries
                set_variable(name.underscorify() + '_disable_reason', reason)
        else
                enabled_libs += name
-               dpdk_conf.set('RTE_LIBRTE_' + name.to_upper(), 1) #old macro
-               dpdk_conf.set('RTE_LIB_' + name.to_upper(), 1) # new macro
+               dpdk_conf.set('RTE_LIB_' + name.to_upper(), 1)
                install_headers(headers)
 
                libname = 'rte_' + name