@@ -95,6 +95,7 @@ F: devtools/check-maintainers.sh
F: devtools/check-forbidden-tokens.awk
F: devtools/check-git-log.sh
F: devtools/check-spdx-tag.sh
+F: devtools/check-symbol-change.py
F: devtools/check-symbol-change.sh
F: devtools/check-symbol-maps.sh
F: devtools/checkpatches.sh
@@ -127,6 +128,7 @@ F: config/
F: buildtools/check-symbols.sh
F: buildtools/chkincs/
F: buildtools/call-sphinx-build.py
+F: buildtools/gen-version-map.py
F: buildtools/get-cpu-count.py
F: buildtools/get-numa-count.py
F: buildtools/list-dir-globs.py
new file mode 100755
@@ -0,0 +1,106 @@
+#!/usr/bin/env python3
+# SPDX-License-Identifier: BSD-3-Clause
+# Copyright (c) 2025 Red Hat, Inc.
+
+"""Generate a version map file used by GNU or MSVC linker."""
+
+import re
+import sys
+
+scriptname, link_mode, abi_version_file, output, *files = sys.argv
+
+# From rte_export.h
+export_exp_sym_regexp = re.compile(r"^RTE_EXPORT_EXPERIMENTAL_SYMBOL\(([^,]+), ([0-9]+.[0-9]+)\)")
+export_int_sym_regexp = re.compile(r"^RTE_EXPORT_INTERNAL_SYMBOL\(([^)]+)\)")
+export_sym_regexp = re.compile(r"^RTE_EXPORT_SYMBOL\(([^)]+)\)")
+# From rte_function_versioning.h
+ver_sym_regexp = re.compile(r"^RTE_VERSION_SYMBOL\(([^,]+), [^,]+, ([^,]+),")
+ver_exp_sym_regexp = re.compile(r"^RTE_VERSION_EXPERIMENTAL_SYMBOL\([^,]+, ([^,]+),")
+default_sym_regexp = re.compile(r"^RTE_DEFAULT_SYMBOL\(([^,]+), [^,]+, ([^,]+),")
+
+with open(abi_version_file) as f:
+ abi = 'DPDK_{}'.format(re.match("([0-9]+).[0-9]", f.readline()).group(1))
+
+symbols = {}
+
+for file in files:
+ with open(file, encoding="utf-8") as f:
+ for ln in f.readlines():
+ node = None
+ symbol = None
+ comment = ''
+ if export_exp_sym_regexp.match(ln):
+ node = 'EXPERIMENTAL'
+ symbol = export_exp_sym_regexp.match(ln).group(1)
+ comment = ' # added in {}'.format(export_exp_sym_regexp.match(ln).group(2))
+ elif export_int_sym_regexp.match(ln):
+ node = 'INTERNAL'
+ symbol = export_int_sym_regexp.match(ln).group(1)
+ elif export_sym_regexp.match(ln):
+ node = abi
+ symbol = export_sym_regexp.match(ln).group(1)
+ elif ver_sym_regexp.match(ln):
+ node = 'DPDK_{}'.format(ver_sym_regexp.match(ln).group(1))
+ symbol = ver_sym_regexp.match(ln).group(2)
+ elif ver_exp_sym_regexp.match(ln):
+ node = 'EXPERIMENTAL'
+ symbol = ver_exp_sym_regexp.match(ln).group(1)
+ elif default_sym_regexp.match(ln):
+ node = 'DPDK_{}'.format(default_sym_regexp.match(ln).group(1))
+ symbol = default_sym_regexp.match(ln).group(2)
+
+ if not symbol:
+ continue
+
+ if node not in symbols:
+ symbols[node] = {}
+ symbols[node][symbol] = comment
+
+if link_mode == 'msvc':
+ with open(output, "w") as outfile:
+ print(f"EXPORTS", file=outfile)
+ for key in (abi, 'EXPERIMENTAL', 'INTERNAL'):
+ if key not in symbols:
+ continue
+ for symbol in sorted(symbols[key].keys()):
+ print(f"\t{symbol}", file=outfile)
+ del symbols[key]
+else:
+ with open(output, "w") as outfile:
+ local_token = False
+ for key in (abi, 'EXPERIMENTAL', 'INTERNAL'):
+ if key not in symbols:
+ continue
+ print(f"{key} {{\n\tglobal:\n", file=outfile)
+ for symbol in sorted(symbols[key].keys()):
+ if link_mode == 'mingw' and symbol.startswith('per_lcore'):
+ prefix = '__emutls_v.'
+ else:
+ prefix = ''
+ comment = symbols[key][symbol]
+ print(f"\t{prefix}{symbol};{comment}", file=outfile)
+ if not local_token:
+ print("\n\tlocal: *;", file=outfile)
+ local_token = True
+ print("};", file=outfile)
+ del symbols[key]
+ for key in sorted(symbols.keys()):
+ print(f"{key} {{\n\tglobal:\n", file=outfile)
+ for symbol in sorted(symbols[key].keys()):
+ if link_mode == 'mingw' and symbol.startswith('per_lcore'):
+ prefix = '__emutls_v.'
+ else:
+ prefix = ''
+ comment = symbols[key][symbol]
+ print(f"\t{prefix}{symbol};{comment}", file=outfile)
+ print(f"}} {abi};", file=outfile)
+ if not local_token:
+ print("\n\tlocal: *;", file=outfile)
+ local_token = True
+ del symbols[key]
+ # No exported symbol, add a catch all
+ if not local_token:
+ print(f"{abi} {{", file=outfile)
+ print("\n\tlocal: *;", file=outfile)
+ local_token = True
+ print("};", file=outfile)
@@ -62,10 +62,14 @@ for file in $@; do
if (current_section == "") {
next;
}
+ symbol_version = current_version
+ if (/^[^}].*[^:*]; # added in /) {
+ symbol_version = $5
+ }
if ("'$version'" != "") {
- if ("'$version'" == "unset" && current_version != "") {
+ if ("'$version'" == "unset" && symbol_version != "") {
next;
- } else if ("'$version'" != "unset" && "'$version'" != current_version) {
+ } else if ("'$version'" != "unset" && "'$version'" != symbol_version) {
next;
}
}
@@ -73,7 +77,7 @@ for file in $@; do
if ("'$symbol'" == "all" || $1 == "'$symbol'") {
ret = 0;
if ("'$quiet'" == "") {
- print "'$file' "current_section" "$1" "current_version;
+ print "'$file' "current_section" "$1" "symbol_version;
}
if ("'$symbol'" != "all") {
exit 0;
@@ -16,6 +16,7 @@ else
py3 = ['meson', 'runpython']
endif
echo = py3 + ['-c', 'import sys; print(*sys.argv[1:])']
+gen_version_map = py3 + files('gen-version-map.py')
list_dir_globs = py3 + files('list-dir-globs.py')
map_to_win_cmd = py3 + files('map_to_win.py')
sphinx_wrapper = py3 + files('call-sphinx-build.py')
@@ -300,11 +300,13 @@ if cc.get_id() == 'clang' and dpdk_conf.get('RTE_ARCH_64') == false
dpdk_extra_ldflags += '-latomic'
endif
-# add -include rte_config to cflags
+# add -include some headers to cflags
if is_ms_compiler
add_project_arguments('/FI', 'rte_config.h', language: 'c')
+ add_project_arguments('/FI', 'rte_export.h', language: 'c')
else
add_project_arguments('-include', 'rte_config.h', language: 'c')
+ add_project_arguments('-include', 'rte_export.h', language: 'c')
endif
# enable extra warnings and disable any unwanted warnings
new file mode 100644
@@ -0,0 +1,16 @@
+/* SPDX-License-Identifier: BSD-3-Clause
+ * Copyright (c) 2025 Red Hat, Inc.
+ */
+
+#ifndef RTE_EXPORT_H
+#define RTE_EXPORT_H
+
+/* *Internal* macros for exporting symbols, used by the build system.
+ * For RTE_EXPORT_EXPERIMENTAL_SYMBOL, ver indicates the
+ * version this symbol was introduced in.
+ */
+#define RTE_EXPORT_EXPERIMENTAL_SYMBOL(a, ver)
+#define RTE_EXPORT_INTERNAL_SYMBOL(a)
+#define RTE_EXPORT_SYMBOL(a)
+
+#endif /* RTE_EXPORT_H */
new file mode 100755
@@ -0,0 +1,90 @@
+#!/usr/bin/env python3
+# SPDX-License-Identifier: BSD-3-Clause
+# Copyright (c) 2025 Red Hat, Inc.
+
+"""Check exported symbols change in a patch."""
+
+import re
+import sys
+
+file_header_regexp = re.compile(r"^(\-\-\-|\+\+\+) [ab]/(lib|drivers)/([^/]+)/([^/]+)")
+# From rte_export.h
+export_exp_sym_regexp = re.compile(r"^.RTE_EXPORT_EXPERIMENTAL_SYMBOL\(([^,]+),")
+export_int_sym_regexp = re.compile(r"^.RTE_EXPORT_INTERNAL_SYMBOL\(([^)]+)\)")
+export_sym_regexp = re.compile(r"^.RTE_EXPORT_SYMBOL\(([^)]+)\)")
+# TODO, handle versioned symbols from rte_function_versioning.h
+# ver_sym_regexp = re.compile(r"^.RTE_VERSION_SYMBOL\(([^,]+), [^,]+, ([^,]+),")
+# ver_exp_sym_regexp = re.compile(r"^.RTE_VERSION_EXPERIMENTAL_SYMBOL\([^,]+, ([^,]+),")
+# default_sym_regexp = re.compile(r"^.RTE_DEFAULT_SYMBOL\(([^,]+), [^,]+, ([^,]+),")
+
+symbols = {}
+
+for file in sys.argv[1:]:
+ with open(file, encoding="utf-8") as f:
+ for ln in f.readlines():
+ if file_header_regexp.match(ln):
+ if file_header_regexp.match(ln).group(2) == "lib":
+ lib = '/'.join(file_header_regexp.match(ln).group(2, 3))
+ elif file_header_regexp.match(ln).group(3) == "intel":
+ lib = '/'.join(file_header_regexp.match(ln).group(2, 3, 4))
+ else:
+ lib = '/'.join(file_header_regexp.match(ln).group(2, 3))
+
+ if lib not in symbols:
+ symbols[lib] = {}
+ continue
+
+ if export_exp_sym_regexp.match(ln):
+ symbol = export_exp_sym_regexp.match(ln).group(1)
+ node = 'EXPERIMENTAL'
+ elif export_int_sym_regexp.match(ln):
+ node = 'INTERNAL'
+ symbol = export_int_sym_regexp.match(ln).group(1)
+ elif export_sym_regexp.match(ln):
+ symbol = export_sym_regexp.match(ln).group(1)
+ node = 'stable'
+ else:
+ continue
+
+ if symbol not in symbols[lib]:
+ symbols[lib][symbol] = {}
+ added = ln[0] == '+'
+ if added and 'added' in symbols[lib][symbol] and node != symbols[lib][symbol]['added']:
+ print(f"{symbol} in {lib} was found in multiple ABI, please check.")
+ if not added and 'removed' in symbols[lib][symbol] and node != symbols[lib][symbol]['removed']:
+ print(f"{symbol} in {lib} was found in multiple ABI, please check.")
+ if added:
+ symbols[lib][symbol]['added'] = node
+ else:
+ symbols[lib][symbol]['removed'] = node
+
+ for lib in sorted(symbols.keys()):
+ error = False
+ for symbol in sorted(symbols[lib].keys()):
+ if 'removed' not in symbols[lib][symbol]:
+ # Symbol addition
+ node = symbols[lib][symbol]['added']
+ if node == 'stable':
+ print(f"ERROR: {symbol} in {lib} has been added directly to stable ABI.")
+ error = True
+ else:
+ print(f"INFO: {symbol} in {lib} has been added to {node} ABI.")
+ continue
+
+ if 'added' not in symbols[lib][symbol]:
+ # Symbol removal
+ node = symbols[lib][symbol]['added']
+ if node == 'stable':
+ print(f"INFO: {symbol} in {lib} has been removed from stable ABI.")
+ print(f"Please check it has gone though the deprecation process.")
+ continue
+
+ if symbols[lib][symbol]['added'] == symbols[lib][symbol]['removed']:
+ # Symbol was moved around
+ continue
+
+ # Symbol modifications
+ added = symbols[lib][symbol]['added']
+ removed = symbols[lib][symbol]['removed']
+ print(f"INFO: {symbol} in {lib} is moving from {removed} to {added}")
+ print(f"Please check it has gone though the deprecation process.")
@@ -60,20 +60,6 @@ if [ -n "$local_miss_maps" ] ; then
ret=1
fi
-find_empty_maps ()
-{
- for map in $@ ; do
- [ $(buildtools/map-list-symbol.sh $map | wc -l) != '0' ] || echo $map
- done
-}
-
-empty_maps=$(find_empty_maps $@)
-if [ -n "$empty_maps" ] ; then
- echo "Found empty maps:"
- echo "$empty_maps"
- ret=1
-fi
-
find_bad_format_maps ()
{
abi_version=$(cut -d'.' -f 1 ABI_VERSION)
@@ -33,7 +33,7 @@ VOLATILE,PREFER_PACKED,PREFER_ALIGNED,PREFER_PRINTF,STRLCPY,\
PREFER_KERNEL_TYPES,PREFER_FALLTHROUGH,BIT_MACRO,CONST_STRUCT,\
SPLIT_STRING,LONG_LINE_STRING,C99_COMMENT_TOLERANCE,\
LINE_SPACING,PARENTHESIS_ALIGNMENT,NETWORKING_BLOCK_COMMENT_STYLE,\
-NEW_TYPEDEFS,COMPARISON_TO_NULL,AVOID_BUG"
+NEW_TYPEDEFS,COMPARISON_TO_NULL,AVOID_BUG,EXPORT_SYMBOL"
options="$options $DPDK_CHECKPATCH_OPTIONS"
print_usage () {
@@ -58,12 +58,12 @@ persists over multiple releases.
.. code-block:: none
- $ head ./lib/acl/version.map
+ $ head ./build/lib/acl_exports.map
DPDK_21 {
global:
...
- $ head ./lib/eal/version.map
+ $ head ./build/lib/eal_exports.map
DPDK_21 {
global:
...
@@ -77,7 +77,7 @@ that library.
.. code-block:: none
- $ head ./lib/acl/version.map
+ $ head ./build/lib/acl_exports.map
DPDK_21 {
global:
...
@@ -88,7 +88,7 @@ that library.
} DPDK_21;
...
- $ head ./lib/eal/version.map
+ $ head ./build/lib/eal_exports.map
DPDK_21 {
global:
...
@@ -100,12 +100,12 @@ how this may be done.
.. code-block:: none
- $ head ./lib/acl/version.map
+ $ head ./build/lib/acl_exports.map
DPDK_22 {
global:
...
- $ head ./lib/eal/version.map
+ $ head ./build/lib/eal_exports.map
DPDK_22 {
global:
...
@@ -134,8 +134,7 @@ linked to the DPDK.
To support backward compatibility the ``rte_function_versioning.h``
header file provides macros to use when updating exported functions. These
-macros are used in conjunction with the ``version.map`` file for
-a given library to allow multiple versions of a symbol to exist in a shared
+macros allow multiple versions of a symbol to exist in a shared
library so that older binaries need not be immediately recompiled.
The macros are:
@@ -169,6 +168,7 @@ Assume we have a function as follows
* Create an acl context object for apps to
* manipulate
*/
+ RTE_EXPORT_SYMBOL(rte_acl_create)
int
rte_acl_create(struct rte_acl_param *param)
{
@@ -187,6 +187,7 @@ private, is safe), but it also requires modifying the code as follows
* Create an acl context object for apps to
* manipulate
*/
+ RTE_EXPORT_SYMBOL(rte_acl_create)
int
rte_acl_create(struct rte_acl_param *param, int debug)
{
@@ -203,78 +204,16 @@ The addition of a parameter to the function is ABI breaking as the function is
public, and existing application may use it in its current form. However, the
compatibility macros in DPDK allow a developer to use symbol versioning so that
multiple functions can be mapped to the same public symbol based on when an
-application was linked to it. To see how this is done, we start with the
-requisite libraries version map file. Initially the version map file for the acl
-library looks like this
+application was linked to it.
-.. code-block:: none
-
- DPDK_21 {
- global:
-
- rte_acl_add_rules;
- rte_acl_build;
- rte_acl_classify;
- rte_acl_classify_alg;
- rte_acl_classify_scalar;
- rte_acl_create;
- rte_acl_dump;
- rte_acl_find_existing;
- rte_acl_free;
- rte_acl_ipv4vlan_add_rules;
- rte_acl_ipv4vlan_build;
- rte_acl_list_dump;
- rte_acl_reset;
- rte_acl_reset_rules;
- rte_acl_set_ctx_classify;
-
- local: *;
- };
-
-This file needs to be modified as follows
-
-.. code-block:: none
-
- DPDK_21 {
- global:
-
- rte_acl_add_rules;
- rte_acl_build;
- rte_acl_classify;
- rte_acl_classify_alg;
- rte_acl_classify_scalar;
- rte_acl_create;
- rte_acl_dump;
- rte_acl_find_existing;
- rte_acl_free;
- rte_acl_ipv4vlan_add_rules;
- rte_acl_ipv4vlan_build;
- rte_acl_list_dump;
- rte_acl_reset;
- rte_acl_reset_rules;
- rte_acl_set_ctx_classify;
-
- local: *;
- };
-
- DPDK_22 {
- global:
- rte_acl_create;
-
- } DPDK_21;
-
-The addition of the new block tells the linker that a new version node
-``DPDK_22`` is available, which contains the symbol rte_acl_create, and inherits
-the symbols from the DPDK_21 node. This list is directly translated into a
-list of exported symbols when DPDK is compiled as a shared library.
-
-Next, we need to specify in the code which function maps to the rte_acl_create
+We need to specify in the code which function maps to the rte_acl_create
symbol at which versions. First, at the site of the initial symbol definition,
we wrap the function with ``RTE_VERSION_SYMBOL``, passing the current ABI version,
the function return type, the function name and its arguments.
.. code-block:: c
+ -RTE_EXPORT_SYMBOL(rte_acl_create)
-int
-rte_acl_create(struct rte_acl_param *param)
+RTE_VERSION_SYMBOL(21, int, rte_acl_create, (struct rte_acl_param *param))
@@ -314,9 +253,9 @@ The macro instructs the linker to create the new default symbol
``rte_acl_create@DPDK_22``, which points to the function named ``rte_acl_create_v22``
(declared by the macro).
-And that's it, on the next shared library rebuild, there will be two versions of
-rte_acl_create, an old DPDK_21 version, used by previously built applications,
-and a new DPDK_22 version, used by future built applications.
+And that's it. On the next shared library rebuild, there will be two versions of rte_acl_create,
+an old DPDK_21 version, used by previously built applications, and a new DPDK_22 version,
+used by newly built applications.
.. note::
@@ -366,6 +305,7 @@ Assume we have an experimental function ``rte_acl_create`` as follows:
* Create an acl context object for apps to
* manipulate
*/
+ RTE_EXPORT_EXPERIMENTAL_SYMBOL(rte_acl_create)
__rte_experimental
int
rte_acl_create(struct rte_acl_param *param)
@@ -373,27 +313,8 @@ Assume we have an experimental function ``rte_acl_create`` as follows:
...
}
-In the map file, experimental symbols are listed as part of the ``EXPERIMENTAL``
-version node.
-
-.. code-block:: none
-
- DPDK_21 {
- global:
- ...
-
- local: *;
- };
-
- EXPERIMENTAL {
- global:
-
- rte_acl_create;
- };
-
When we promote the symbol to the stable ABI, we simply strip the
-``__rte_experimental`` annotation from the function and move the symbol from the
-``EXPERIMENTAL`` node, to the node of the next major ABI version as follow.
+``__rte_experimental`` annotation from the function.
.. code-block:: c
@@ -401,31 +322,13 @@ When we promote the symbol to the stable ABI, we simply strip the
* Create an acl context object for apps to
* manipulate
*/
+ RTE_EXPORT_SYMBOL(rte_acl_create)
int
rte_acl_create(struct rte_acl_param *param)
{
...
}
-We then update the map file, adding the symbol ``rte_acl_create``
-to the ``DPDK_22`` version node.
-
-.. code-block:: none
-
- DPDK_21 {
- global:
- ...
-
- local: *;
- };
-
- DPDK_22 {
- global:
-
- rte_acl_create;
- } DPDK_21;
-
-
Although there are strictly no guarantees or commitments associated with
:ref:`experimental symbols <experimental_apis>`, a maintainer may wish to offer
an alias to experimental. The process to add an alias to experimental,
@@ -452,30 +355,6 @@ and ``DPDK_22`` version nodes.
return rte_acl_create(param);
}
-In the map file, we map the symbol to both the ``EXPERIMENTAL``
-and ``DPDK_22`` version nodes.
-
-.. code-block:: none
-
- DPDK_21 {
- global:
- ...
-
- local: *;
- };
-
- DPDK_22 {
- global:
-
- rte_acl_create;
- } DPDK_21;
-
- EXPERIMENTAL {
- global:
-
- rte_acl_create;
- };
-
.. _abi_deprecation:
Deprecating part of a public API
@@ -484,38 +363,7 @@ ________________________________
Lets assume that you've done the above updates, and in preparation for the next
major ABI version you decide you would like to retire the old version of the
function. After having gone through the ABI deprecation announcement process,
-removal is easy. Start by removing the symbol from the requisite version map
-file:
-
-.. code-block:: none
-
- DPDK_21 {
- global:
-
- rte_acl_add_rules;
- rte_acl_build;
- rte_acl_classify;
- rte_acl_classify_alg;
- rte_acl_classify_scalar;
- rte_acl_dump;
- - rte_acl_create
- rte_acl_find_existing;
- rte_acl_free;
- rte_acl_ipv4vlan_add_rules;
- rte_acl_ipv4vlan_build;
- rte_acl_list_dump;
- rte_acl_reset;
- rte_acl_reset_rules;
- rte_acl_set_ctx_classify;
-
- local: *;
- };
-
- DPDK_22 {
- global:
- rte_acl_create;
- } DPDK_21;
-
+removal is easy.
Next remove the corresponding versioned export.
@@ -539,36 +387,7 @@ of a major ABI version. If a version node completely specifies an API, then
removing part of it, typically makes it incomplete. In those cases it is better
to remove the entire node.
-To do this, start by modifying the version map file, such that all symbols from
-the node to be removed are merged into the next node in the map.
-
-In the case of our map above, it would transform to look as follows
-
-.. code-block:: none
-
- DPDK_22 {
- global:
-
- rte_acl_add_rules;
- rte_acl_build;
- rte_acl_classify;
- rte_acl_classify_alg;
- rte_acl_classify_scalar;
- rte_acl_dump;
- rte_acl_create
- rte_acl_find_existing;
- rte_acl_free;
- rte_acl_ipv4vlan_add_rules;
- rte_acl_ipv4vlan_build;
- rte_acl_list_dump;
- rte_acl_reset;
- rte_acl_reset_rules;
- rte_acl_set_ctx_classify;
-
- local: *;
- };
-
-Then any uses of RTE_DEFAULT_SYMBOL that pointed to the old node should be
+Any uses of RTE_DEFAULT_SYMBOL that pointed to the old node should be
updated to point to the new version node in any header files for all affected
symbols.
@@ -275,14 +275,14 @@ foreach subpath:subdirs
dependencies: static_deps,
c_args: cflags)
objs += tmp_lib.extract_all_objects(recursive: true)
- sources = custom_target(out_filename,
+ sources_pmd_info = custom_target(out_filename,
command: [pmdinfo, tmp_lib.full_path(), '@OUTPUT@', pmdinfogen],
output: out_filename,
depends: [tmp_lib])
# now build the static driver
static_lib = static_library(lib_name,
- sources,
+ sources_pmd_info,
objects: objs,
include_directories: includes,
dependencies: static_deps,
@@ -292,48 +292,70 @@ foreach subpath:subdirs
# now build the shared driver
version_map = '@0@/@1@/version.map'.format(meson.current_source_dir(), drv_path)
- lk_deps = []
- lk_args = []
if not fs.is_file(version_map)
- version_map = '@0@/version.map'.format(meson.current_source_dir())
- lk_deps += [version_map]
- else
- lk_deps += [version_map]
- if not is_windows and developer_mode
- # on unix systems check the output of the
- # check-symbols.sh script, using it as a
- # dependency of the .so build
- lk_deps += custom_target(lib_name + '.sym_chk',
- command: [check_symbols, version_map, '@INPUT@'],
- capture: true,
- input: static_lib,
- output: lib_name + '.sym_chk')
- endif
- endif
-
- if is_windows
if is_ms_linker
- def_file = custom_target(lib_name + '_def',
- command: [map_to_win_cmd, '@INPUT@', '@OUTPUT@'],
- input: version_map,
- output: '@0@_exports.def'.format(lib_name))
- lk_deps += [def_file]
-
- lk_args = ['-Wl,/def:' + def_file.full_path()]
+ link_mode = 'mslinker'
+ elif is_windows
+ link_mode = 'mingw'
else
- mingw_map = custom_target(lib_name + '_mingw',
- command: [map_to_win_cmd, '@INPUT@', '@OUTPUT@'],
- input: version_map,
- output: '@0@_mingw.map'.format(lib_name))
- lk_deps += [mingw_map]
-
- lk_args = ['-Wl,--version-script=' + mingw_map.full_path()]
+ link_mode = 'gnu'
+ endif
+ version_map = custom_target(lib_name + '_map',
+ command: [gen_version_map, link_mode, abi_version_file, '@OUTPUT@', '@INPUT@'],
+ input: sources + sources_avx2 + sources_avx512,
+ output: '_'.join(class, name, 'exports.map'))
+ version_map_path = version_map.full_path()
+ version_map_dep = [version_map]
+ lk_deps = [version_map]
+
+ if is_ms_linker and is_ms_compiler
+ lk_args = ['/def:' + version_map.full_path()]
+ elif is_ms_linker
+ lk_args = ['-Wl,/def:' + version_map.full_path()]
+ else
+ lk_args = ['-Wl,--version-script=' + version_map.full_path()]
endif
else
- lk_args = ['-Wl,--version-script=' + version_map]
+ version_map_path = version_map
+ version_map_dep = []
+ lk_deps = [version_map]
+
+ if is_windows
+ if is_ms_linker
+ def_file = custom_target(lib_name + '_def',
+ command: [map_to_win_cmd, '@INPUT@', '@OUTPUT@'],
+ input: version_map,
+ output: '@0@_exports.def'.format(lib_name))
+ lk_deps += [def_file]
+
+ lk_args = ['-Wl,/def:' + def_file.full_path()]
+ else
+ mingw_map = custom_target(lib_name + '_mingw',
+ command: [map_to_win_cmd, '@INPUT@', '@OUTPUT@'],
+ input: version_map,
+ output: '@0@_mingw.map'.format(lib_name))
+ lk_deps += [mingw_map]
+
+ lk_args = ['-Wl,--version-script=' + mingw_map.full_path()]
+ endif
+ else
+ lk_args = ['-Wl,--version-script=' + version_map]
+ endif
+ endif
+
+ if not is_windows and developer_mode
+ # on unix systems check the output of the
+ # check-symbols.sh script, using it as a
+ # dependency of the .so build
+ lk_deps += custom_target(lib_name + '.sym_chk',
+ command: [check_symbols, version_map_path, '@INPUT@'],
+ capture: true,
+ input: static_lib,
+ output: lib_name + '.sym_chk',
+ depends: version_map_dep)
endif
- shared_lib = shared_library(lib_name, sources,
+ shared_lib = shared_library(lib_name, sources_pmd_info,
objects: objs,
include_directories: includes,
dependencies: shared_deps,
deleted file mode 100644
@@ -1,3 +0,0 @@
-DPDK_25 {
- local: *;
-};
@@ -1,6 +1,7 @@
# SPDX-License-Identifier: BSD-3-Clause
# Copyright(c) 2017-2019 Intel Corporation
+fs = import('fs')
# process all libraries equally, as far as possible
# "core" libs first, then others alphabetically as far as possible
@@ -254,42 +255,58 @@ foreach l:libraries
include_directories: includes,
dependencies: static_deps)
- if not use_function_versioning or is_windows
- # use pre-build objects to build shared lib
- sources = []
- objs += static_lib.extract_all_objects(recursive: false)
- else
- # for compat we need to rebuild with
- # RTE_BUILD_SHARED_LIB defined
- cflags += '-DRTE_BUILD_SHARED_LIB'
- endif
-
- version_map = '@0@/@1@/version.map'.format(meson.current_source_dir(), l)
- lk_deps = [version_map]
-
- if is_ms_linker
- def_file = custom_target(libname + '_def',
- command: [map_to_win_cmd, '@INPUT@', '@OUTPUT@'],
- input: version_map,
- output: '@0@_exports.def'.format(libname))
- lk_deps += [def_file]
+ if not fs.is_file('@0@/@1@/version.map'.format(meson.current_source_dir(), l))
+ if is_ms_linker
+ link_mode = 'mslinker'
+ elif is_windows
+ link_mode = 'mingw'
+ else
+ link_mode = 'gnu'
+ endif
+ version_map = custom_target(libname + '_map',
+ command: [gen_version_map, link_mode, abi_version_file, '@OUTPUT@', '@INPUT@'],
+ input: sources,
+ output: '_'.join(name, 'exports.map'))
+ version_map_path = version_map.full_path()
+ version_map_dep = [version_map]
+ lk_deps = [version_map]
- if is_ms_compiler
- lk_args = ['/def:' + def_file.full_path()]
+ if is_ms_linker and is_ms_compiler
+ lk_args = ['/def:' + version_map.full_path()]
+ elif is_ms_linker
+ lk_args = ['-Wl,/def:' + version_map.full_path()]
else
- lk_args = ['-Wl,/def:' + def_file.full_path()]
+ lk_args = ['-Wl,--version-script=' + version_map.full_path()]
endif
else
- if is_windows
- mingw_map = custom_target(libname + '_mingw',
+ version_map = '@0@/@1@/version.map'.format(meson.current_source_dir(), l)
+ version_map_path = version_map
+ version_map_dep = []
+ lk_deps = [version_map]
+ if is_ms_linker
+ def_file = custom_target(libname + '_def',
command: [map_to_win_cmd, '@INPUT@', '@OUTPUT@'],
input: version_map,
- output: '@0@_mingw.map'.format(libname))
- lk_deps += [mingw_map]
+ output: '@0@_exports.def'.format(libname))
+ lk_deps += [def_file]
- lk_args = ['-Wl,--version-script=' + mingw_map.full_path()]
+ if is_ms_compiler
+ lk_args = ['/def:' + def_file.full_path()]
+ else
+ lk_args = ['-Wl,/def:' + def_file.full_path()]
+ endif
else
- lk_args = ['-Wl,--version-script=' + version_map]
+ if is_windows
+ mingw_map = custom_target(libname + '_mingw',
+ command: [map_to_win_cmd, '@INPUT@', '@OUTPUT@'],
+ input: version_map,
+ output: '@0@_mingw.map'.format(libname))
+ lk_deps += [mingw_map]
+
+ lk_args = ['-Wl,--version-script=' + mingw_map.full_path()]
+ else
+ lk_args = ['-Wl,--version-script=' + version_map]
+ endif
endif
endif
@@ -298,11 +315,21 @@ foreach l:libraries
# check-symbols.sh script, using it as a
# dependency of the .so build
lk_deps += custom_target(name + '.sym_chk',
- command: [check_symbols,
- version_map, '@INPUT@'],
+ command: [check_symbols, version_map_path, '@INPUT@'],
capture: true,
input: static_lib,
- output: name + '.sym_chk')
+ output: name + '.sym_chk',
+ depends: version_map_dep)
+ endif
+
+ if not use_function_versioning or is_windows
+ # use pre-build objects to build shared lib
+ sources = []
+ objs += static_lib.extract_all_objects(recursive: false)
+ else
+ # for compat we need to rebuild with
+ # RTE_BUILD_SHARED_LIB defined
+ cflags += '-DRTE_BUILD_SHARED_LIB'
endif
shared_lib = shared_library(libname,