Glibc added the strlcpy and strlcat functions to version 2.38, released in 2023, meaning they are natively available in modern linux distros. At this point, the value of having the libbsd provided versions of these functions is reduced, so let's simplify the code options here by providing just two options for strlcpy rather than three:
1. native implementation for BSD and recent Linux 2. DPDK-specific fallbacks using snprintf Since the strlcpy and strlcat functions are the only two items used from libbsd, we can then drop completely any DPDK dependency on libbsd. Signed-off-by: Bruce Richardson <[email protected]> --- V2: * took the work further than v1, dropping libbsd dependency entirely. Now DPDK just supports native strlcpy or it's own fallback version. --- app/test/test_string_fns.c | 4 ++-- buildtools/pkg-config/meson.build | 3 +-- config/meson.build | 7 +++---- lib/eal/include/rte_string_fns.h | 21 +++++---------------- lib/eal/meson.build | 4 +--- lib/telemetry/telemetry.c | 2 -- lib/telemetry/telemetry_data.c | 1 - lib/telemetry/telemetry_legacy.c | 2 -- 8 files changed, 12 insertions(+), 32 deletions(-) diff --git a/app/test/test_string_fns.c b/app/test/test_string_fns.c index 697cb7ed15..213a9312ea 100644 --- a/app/test/test_string_fns.c +++ b/app/test/test_string_fns.c @@ -134,7 +134,7 @@ static int test_rte_strlcat(void) { /* only run actual unit tests if we have system-provided strlcat */ -#if defined(__BSD_VISIBLE) || defined(RTE_USE_LIBBSD) +#ifdef RTE_HAS_STRLCPY #define BUF_LEN 32 const char dst[BUF_LEN] = "Test string"; const char src[] = " appended"; @@ -168,7 +168,7 @@ test_rte_strlcat(void) } LOG("Checked %zu combinations\n", i); #undef BUF_LEN -#endif /* defined(__BSD_VISIBLE) || defined(RTE_USE_LIBBSD) */ +#endif /* RTE_HAS_STRLCPY */ return 0; } diff --git a/buildtools/pkg-config/meson.build b/buildtools/pkg-config/meson.build index b36add17e3..a0a265ad92 100644 --- a/buildtools/pkg-config/meson.build +++ b/buildtools/pkg-config/meson.build @@ -47,8 +47,7 @@ pkg.generate(name: 'DPDK', # main DPDK pkgconfig file description: '''The Data Plane Development Kit (DPDK). Note that CFLAGS might contain an -march flag higher than typical baseline. This is required for a number of static inline functions in the public headers.''', - requires: ['libdpdk-libs', libbsd], # may need libbsd for string funcs - # if libbsd is not enabled, then this is blank + requires: ['libdpdk-libs'], libraries_private: ['-Wl,--whole-archive'] + dpdk_drivers + dpdk_static_libraries + ['-Wl,--no-whole-archive'] + platform_flags diff --git a/config/meson.build b/config/meson.build index d7f5e55c18..237e747eec 100644 --- a/config/meson.build +++ b/config/meson.build @@ -266,10 +266,9 @@ if libarchive.found() dpdk_conf.set('RTE_HAS_LIBARCHIVE', 1) endif -# check for libbsd -libbsd = dependency('libbsd', required: false, method: 'pkg-config') -if libbsd.found() - dpdk_conf.set('RTE_USE_LIBBSD', 1) +# check for strlcpy/strlcat in native libc, otherwise use DPDK fallback +if cc.has_function('strlcpy', prefix: '#include <string.h>') + dpdk_conf.set('RTE_HAS_STRLCPY', 1) endif jansson_dep = dependency('jansson', required: false, method: 'pkg-config') diff --git a/lib/eal/include/rte_string_fns.h b/lib/eal/include/rte_string_fns.h index 3713c94acb..4b2323fc34 100644 --- a/lib/eal/include/rte_string_fns.h +++ b/lib/eal/include/rte_string_fns.h @@ -55,7 +55,7 @@ rte_strsplit(char *string, int stringlen, /** * @internal * DPDK-specific version of strlcpy for systems without - * libc or libbsd copies of the function + * a native libc copy of the function */ static inline size_t rte_strlcpy(char *dst, const char *src, size_t size) @@ -66,7 +66,7 @@ rte_strlcpy(char *dst, const char *src, size_t size) /** * @internal * DPDK-specific version of strlcat for systems without - * libc or libbsd copies of the function + * a native libc copy of the function */ static inline size_t rte_strlcat(char *dst, const char *src, size_t size) @@ -81,24 +81,13 @@ rte_strlcat(char *dst, const char *src, size_t size) } #endif -/* pull in a strlcpy function */ -#ifdef RTE_EXEC_ENV_FREEBSD -#ifndef __BSD_VISIBLE /* non-standard functions are hidden */ +/* provide strlcpy/strlcat aliases where not natively available */ +#if !defined(RTE_HAS_STRLCPY) || \ + (defined(RTE_EXEC_ENV_FREEBSD) && !defined(__BSD_VISIBLE)) #define strlcpy(dst, src, size) rte_strlcpy(dst, src, size) #define strlcat(dst, src, size) rte_strlcat(dst, src, size) #endif -#else /* non-BSD platforms */ -#ifdef RTE_USE_LIBBSD -#include <bsd/string.h> - -#else /* no BSD header files, create own */ -#define strlcpy(dst, src, size) rte_strlcpy(dst, src, size) -#define strlcat(dst, src, size) rte_strlcat(dst, src, size) - -#endif /* RTE_USE_LIBBSD */ -#endif /* FREEBSD */ - #ifdef __cplusplus extern "C" { #endif diff --git a/lib/eal/meson.build b/lib/eal/meson.build index f9fcee24ee..092f5c5261 100644 --- a/lib/eal/meson.build +++ b/lib/eal/meson.build @@ -18,9 +18,7 @@ deps += ['argparse', 'kvargs'] if not is_windows deps += ['telemetry'] endif -if dpdk_conf.has('RTE_USE_LIBBSD') - ext_deps += libbsd -endif + if dpdk_conf.has('RTE_HAS_LIBARCHIVE') ext_deps += libarchive endif diff --git a/lib/telemetry/telemetry.c b/lib/telemetry/telemetry.c index e591c1e283..f863445798 100644 --- a/lib/telemetry/telemetry.c +++ b/lib/telemetry/telemetry.c @@ -13,8 +13,6 @@ #include <sys/stat.h> #endif /* !RTE_EXEC_ENV_WINDOWS */ -/* we won't link against libbsd, so just always use DPDKs-specific strlcpy */ -#undef RTE_USE_LIBBSD #include <eal_export.h> #include <rte_string_fns.h> #include <rte_common.h> diff --git a/lib/telemetry/telemetry_data.c b/lib/telemetry/telemetry_data.c index 0a006559ab..08bdc4ea36 100644 --- a/lib/telemetry/telemetry_data.c +++ b/lib/telemetry/telemetry_data.c @@ -7,7 +7,6 @@ #include <stdlib.h> #include <inttypes.h> -#undef RTE_USE_LIBBSD #include <stdbool.h> #include <eal_export.h> diff --git a/lib/telemetry/telemetry_legacy.c b/lib/telemetry/telemetry_legacy.c index 1d73282ba8..af497a594a 100644 --- a/lib/telemetry/telemetry_legacy.c +++ b/lib/telemetry/telemetry_legacy.c @@ -10,8 +10,6 @@ #include <pthread.h> #endif /* !RTE_EXEC_ENV_WINDOWS */ -/* we won't link against libbsd, so just always use DPDKs-specific strlcpy */ -#undef RTE_USE_LIBBSD #include <eal_export.h> #include <rte_string_fns.h> #include <rte_common.h> -- 2.53.0
