From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by inbox.dpdk.org (Postfix) with ESMTP id 76EF043E7C; Mon, 15 Apr 2024 18:46:00 +0200 (CEST) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 7C17B40684; Mon, 15 Apr 2024 18:45:50 +0200 (CEST) Received: from linux.microsoft.com (linux.microsoft.com [13.77.154.182]) by mails.dpdk.org (Postfix) with ESMTP id 5283B4026C for ; Mon, 15 Apr 2024 18:45:46 +0200 (CEST) Received: by linux.microsoft.com (Postfix, from userid 1086) id 577FB20FC5F7; Mon, 15 Apr 2024 09:45:45 -0700 (PDT) DKIM-Filter: OpenDKIM Filter v2.11.0 linux.microsoft.com 577FB20FC5F7 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linux.microsoft.com; s=default; t=1713199545; bh=3V5I5oCKrMuxJ2g1wzw/YCHS+IW31uPtFvHrSYYw+L8=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=rgIsmL8gs+nIJx0WVHqKkmStemRVyzGMupuHbz7o1fwufjQTAenXv41Vm/GrU6Y5B vkSwhX7P9HM7ssUOVrndN+5G/apEJDNU+34OZiG4eZ3BjAlgBM2QHV2OjsB/W8jt7j VjovWuVo5dTwxJa9Vdhl0E+IcAASBxUoBZGgydDc= From: Tyler Retzlaff To: dev@dpdk.org Cc: Bruce Richardson , Harman Kalra , Tyler Retzlaff Subject: [PATCH v4 1/2] build: build only one library type on Windows Date: Mon, 15 Apr 2024 09:45:42 -0700 Message-Id: <1713199543-30038-2-git-send-email-roretzla@linux.microsoft.com> X-Mailer: git-send-email 1.8.3.1 In-Reply-To: <1713199543-30038-1-git-send-email-roretzla@linux.microsoft.com> References: <1710445477-23848-1-git-send-email-roretzla@linux.microsoft.com> <1713199543-30038-1-git-send-email-roretzla@linux.microsoft.com> X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org MSVC is the only compiler that can produce usable shared libraries for DPDK on Windows because of the use of exported TLS variables. Disable building of shared libraries with LLVM and MinGW so that remaining __declspec macros needed for the functional libraries built by MSVC can be used without triggering errors in LLVM and MinGW builds. For Windows only install the default_library type to avoid confusion. Windows builds cannot build both shared and static in a single pass so install only the functional variant. Signed-off-by: Tyler Retzlaff --- app/meson.build | 4 +++ config/meson.build | 21 ++++++++++++++++ drivers/meson.build | 62 +++++++++++++++++++++------------------------ examples/meson.build | 4 +++ lib/meson.build | 71 ++++++++++++++++++++++------------------------------ 5 files changed, 87 insertions(+), 75 deletions(-) diff --git a/app/meson.build b/app/meson.build index 21b6da2..d94378e 100644 --- a/app/meson.build +++ b/app/meson.build @@ -104,6 +104,10 @@ foreach app:apps link_libs = dpdk_static_libraries + dpdk_drivers endif + if is_windows and is_shared_enabled + cflags += '-DRTE_BUILD_SHARED_LIB' + endif + exec = executable('dpdk-' + name, sources, c_args: cflags, diff --git a/config/meson.build b/config/meson.build index 8c8b019..c74899e 100644 --- a/config/meson.build +++ b/config/meson.build @@ -516,4 +516,25 @@ if get_option('default_library') == 'both' NOTE: DPDK always builds both shared and static libraries. Please set "default_library" to either "static" or "shared" to select default linkage for apps and any examples.''') +elif get_option('default_library') == 'shared' and is_windows and not is_ms_compiler + error( ''' + Unsupported value "shared" for "default_library" option. + + NOTE: DPDK Windows shared is only supported when building with MSVC. Please set + "default_library" to either "static" or use MSVC.''') +endif + +is_shared_enabled=true +install_static = true +install_shared = true +if is_windows + install_static = get_option('default_library') == 'static' + install_shared = get_option('default_library') == 'shared' + if not is_ms_compiler or not install_shared + is_shared_enabled = false + else + # enable export / import of thread_local variables + add_project_arguments('/experimental:tlsDllInterface', language: 'c') + add_project_link_arguments('/experimental:tlsDllInterface', language: 'c') + endif endif diff --git a/drivers/meson.build b/drivers/meson.build index 66931d4..92a85f8 100644 --- a/drivers/meson.build +++ b/drivers/meson.build @@ -208,6 +208,9 @@ foreach subpath:subdirs enabled_drivers += name lib_name = '_'.join(['rte', class, name]) cflags += '-DRTE_LOG_DEFAULT_LOGTYPE=' + '.'.join([log_prefix, name]) + if is_windows and is_shared_enabled + cflags += '-DRTE_BUILD_SHARED_LIB' + endif if annotate_locks and cc.get_id() == 'clang' and cc.version().version_compare('>=3.5.0') cflags += '-DRTE_ANNOTATE_LOCKS' cflags += '-Wthread-safety' @@ -247,7 +250,7 @@ foreach subpath:subdirs include_directories: includes, dependencies: static_deps, c_args: cflags, - install: true) + install: install_static) # now build the shared driver version_map = '@0@/@1@/version.map'.format(meson.current_source_dir(), drv_path) @@ -271,48 +274,39 @@ foreach subpath:subdirs endif endif - if is_windows - if is_ms_linker + if is_shared_enabled + if is_ms_compiler 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()] - if meson.version().version_compare('<0.54.0') - lk_args += ['-Wl,/implib:drivers\\lib' + lib_name + '.dll.a'] - endif + lk_args = ['/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()] + lk_args = ['-Wl,--version-script=' + version_map] endif - else - lk_args = ['-Wl,--version-script=' + version_map] endif - shared_lib = shared_library(lib_name, sources, - objects: objs, - include_directories: includes, - dependencies: shared_deps, - c_args: cflags, - link_args: lk_args, - link_depends: lk_deps, - version: abi_version, - soversion: so_version, - install: true, - install_dir: driver_install_path) - - # create a dependency object and add it to the global dictionary so - # testpmd or other built-in apps can find it if necessary - shared_dep = declare_dependency(link_with: shared_lib, - include_directories: includes, - dependencies: shared_deps) + shared_dep = {} + if is_shared_enabled + shared_lib = shared_library(lib_name, sources, + objects: objs, + include_directories: includes, + dependencies: shared_deps, + c_args: cflags, + link_args: lk_args, + link_depends: lk_deps, + version: abi_version, + soversion: so_version, + install: install_shared, + install_dir: driver_install_path) + + # create a dependency object and add it to the global dictionary so + # testpmd or other built-in apps can find it if necessary + shared_dep = declare_dependency(link_with: shared_lib, + include_directories: includes, + dependencies: shared_deps) + endif static_dep = declare_dependency( include_directories: includes, dependencies: static_deps) diff --git a/examples/meson.build b/examples/meson.build index 8e8968a..9ca2cb9 100644 --- a/examples/meson.build +++ b/examples/meson.build @@ -121,6 +121,10 @@ foreach example: examples continue endif + if is_windows and is_shared_enabled + cflags += '-DRTE_BUILD_SHARED_LIB' + endif + if allow_experimental_apis cflags += '-DALLOW_EXPERIMENTAL_API' endif diff --git a/lib/meson.build b/lib/meson.build index 179a272..f2894c6 100644 --- a/lib/meson.build +++ b/lib/meson.build @@ -235,6 +235,10 @@ foreach l:libraries cflags += '-Wthread-safety' endif + if is_windows and is_shared_enabled + cflags += '-DRTE_BUILD_SHARED_LIB' + endif + # first build static lib static_lib = static_library(libname, sources, @@ -242,11 +246,13 @@ foreach l:libraries c_args: cflags, dependencies: static_deps, include_directories: includes, - install: true) + install: install_static) static_dep = declare_dependency( include_directories: includes, dependencies: static_deps) + dpdk_static_libraries = [static_lib] + dpdk_static_libraries + if not use_function_versioning or is_windows # use pre-build objects to build shared lib sources = [] @@ -260,33 +266,14 @@ foreach l:libraries 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 is_shared_enabled if is_ms_compiler - lk_args = ['/def:' + def_file.full_path()] - if meson.version().version_compare('<0.54.0') - lk_args += ['/implib:lib\\librte_' + l + '.dll.a'] - endif - else - lk_args = ['-Wl,/def:' + def_file.full_path()] - if meson.version().version_compare('<0.54.0') - lk_args += ['-Wl,/implib:lib\\librte_' + l + '.dll.a'] - endif - endif - else - if is_windows - mingw_map = custom_target(libname + '_mingw', + 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] - - lk_args = ['-Wl,--version-script=' + mingw_map.full_path()] + output: '@0@_exports.def'.format(libname)) + lk_deps += [def_file] + lk_args = ['/def:' + def_file.full_path()] else lk_args = ['-Wl,--version-script=' + version_map] endif @@ -304,23 +291,25 @@ foreach l:libraries output: name + '.sym_chk') endif - shared_lib = shared_library(libname, - sources, - objects: objs, - c_args: cflags, - dependencies: shared_deps, - include_directories: includes, - link_args: lk_args, - link_depends: lk_deps, - version: abi_version, - soversion: so_version, - install: true) - shared_dep = declare_dependency(link_with: shared_lib, - include_directories: includes, - dependencies: shared_deps) + shared_dep = {} + if is_shared_enabled + shared_lib = shared_library(libname, + sources, + objects: objs, + c_args: cflags, + dependencies: shared_deps, + include_directories: includes, + link_args: lk_args, + link_depends: lk_deps, + version: abi_version, + soversion: so_version, + install: install_shared) + shared_dep = declare_dependency(link_with: shared_lib, + include_directories: includes, + dependencies: shared_deps) - dpdk_libraries = [shared_lib] + dpdk_libraries - dpdk_static_libraries = [static_lib] + dpdk_static_libraries + dpdk_libraries = [shared_lib] + dpdk_libraries + endif set_variable('shared_rte_' + name, shared_dep) set_variable('static_rte_' + name, static_dep) -- 1.8.3.1