[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PULL 05/18] configure: simplify creation of plugin symbol list
From: |
Paolo Bonzini |
Subject: |
[PULL 05/18] configure: simplify creation of plugin symbol list |
Date: |
Mon, 10 Jan 2022 15:40:21 +0100 |
--dynamic-list is present on all supported ELF (not Windows or Darwin)
platforms, since it dates back to 2006; -exported_symbols_list is
likewise present on all supported versions of macOS. Do not bother
doing a functional test in configure.
Remove the file creation from configure as well: for Darwin, move the
the creation of the Darwin-formatted symbols to meson; for ELF, use the
file in the source path directly and switch from -Wl, to -Xlinker to
not break weird paths that include a comma.
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
---
configure | 81 +--------------------------------------------
plugins/meson.build | 13 +++++---
2 files changed, 10 insertions(+), 84 deletions(-)
diff --git a/configure b/configure
index 3a523a3d14..1b5fa5ae93 100755
--- a/configure
+++ b/configure
@@ -78,7 +78,6 @@ TMPC="${TMPDIR1}/${TMPB}.c"
TMPO="${TMPDIR1}/${TMPB}.o"
TMPCXX="${TMPDIR1}/${TMPB}.cxx"
TMPE="${TMPDIR1}/${TMPB}.exe"
-TMPTXT="${TMPDIR1}/${TMPB}.txt"
rm -f config.log
@@ -1696,6 +1695,7 @@ if test "$static" = "yes" ; then
plugins="no"
fi
fi
+test "$plugins" = "" && plugins=yes
cat > $TMPC << EOF
@@ -2317,69 +2317,6 @@ EOF
fi
fi
-##########################################
-# plugin linker support probe
-
-if test "$plugins" != "no"; then
-
- #########################################
- # See if --dynamic-list is supported by the linker
-
- ld_dynamic_list="no"
- cat > $TMPTXT <<EOF
-{
- foo;
-};
-EOF
-
- cat > $TMPC <<EOF
-#include <stdio.h>
-void foo(void);
-
-void foo(void)
-{
- printf("foo\n");
-}
-
-int main(void)
-{
- foo();
- return 0;
-}
-EOF
-
- if compile_prog "" "-Wl,--dynamic-list=$TMPTXT" ; then
- ld_dynamic_list="yes"
- fi
-
- #########################################
- # See if -exported_symbols_list is supported by the linker
-
- ld_exported_symbols_list="no"
- cat > $TMPTXT <<EOF
- _foo
-EOF
-
- if compile_prog "" "-Wl,-exported_symbols_list,$TMPTXT" ; then
- ld_exported_symbols_list="yes"
- fi
-
- if test "$ld_dynamic_list" = "no" &&
- test "$ld_exported_symbols_list" = "no" ; then
- if test "$plugins" = "yes"; then
- error_exit \
- "Plugin support requires dynamic linking and specifying a set
of symbols " \
- "that are exported to plugins. Unfortunately your linker
doesn't " \
- "support the flag (--dynamic-list or -exported_symbols_list)
used " \
- "for this purpose."
- else
- plugins="no"
- fi
- else
- plugins="yes"
- fi
-fi
-
##########################################
# glib support probe
@@ -3614,22 +3551,6 @@ fi
if test "$plugins" = "yes" ; then
echo "CONFIG_PLUGIN=y" >> $config_host_mak
- # Copy the export object list to the build dir
- if test "$ld_dynamic_list" = "yes" ; then
- echo "CONFIG_HAS_LD_DYNAMIC_LIST=yes" >> $config_host_mak
- ld_symbols=qemu-plugins-ld.symbols
- cp "$source_path/plugins/qemu-plugins.symbols" $ld_symbols
- elif test "$ld_exported_symbols_list" = "yes" ; then
- echo "CONFIG_HAS_LD_EXPORTED_SYMBOLS_LIST=yes" >> $config_host_mak
- ld64_symbols=qemu-plugins-ld64.symbols
- echo "# Automatically generated by configure - do not modify" >
$ld64_symbols
- grep 'qemu_' "$source_path/plugins/qemu-plugins.symbols" | sed 's/;//g'
| \
- sed -E 's/^[[:space:]]*(.*)/_\1/' >> $ld64_symbols
- else
- error_exit \
- "If \$plugins=yes, either \$ld_dynamic_list or " \
- "\$ld_exported_symbols_list should have been set to 'yes'."
- fi
fi
if test -n "$gdb_bin"; then
diff --git a/plugins/meson.build b/plugins/meson.build
index b3de57853b..fa12047327 100644
--- a/plugins/meson.build
+++ b/plugins/meson.build
@@ -1,10 +1,15 @@
plugin_ldflags = []
# Modules need more symbols than just those in plugins/qemu-plugins.symbols
if not enable_modules
- if 'CONFIG_HAS_LD_DYNAMIC_LIST' in config_host
- plugin_ldflags = ['-Wl,--dynamic-list=qemu-plugins-ld.symbols']
- elif 'CONFIG_HAS_LD_EXPORTED_SYMBOLS_LIST' in config_host
- plugin_ldflags = ['-Wl,-exported_symbols_list,qemu-plugins-ld64.symbols']
+ if targetos == 'darwin'
+ qemu_plugins_symbols_list = configure_file(
+ input: files('qemu-plugins.symbols'),
+ output: 'qemu-plugins-ld64.symbols',
+ capture: true,
+ command: ['sed', '-ne', 's/^[[:space:]]*\\(qemu_.*\\);/_\\1/p',
'@INPUT@'])
+ plugin_ldflags =
['-Wl,-exported_symbols_list,plugins/qemu-plugins-ld64.symbols']
+ else
+ plugin_ldflags = ['-Xlinker', '--dynamic-list=' +
(meson.project_source_root() / 'plugins/qemu-plugins.symbols')]
endif
endif
--
2.33.1
- [PULL v4 00/18] Build system and KVM changes for 2021-12-23, Paolo Bonzini, 2022/01/10
- [PULL 01/18] meson: reuse common_user_inc when building files specific to user-mode emulators, Paolo Bonzini, 2022/01/10
- [PULL 02/18] user: move common-user includes to a subdirectory of {bsd, linux}-user/, Paolo Bonzini, 2022/01/10
- [PULL 03/18] meson: cleanup common-user/ build, Paolo Bonzini, 2022/01/10
- [PULL 04/18] block/file-posix: Simplify the XFS_IOC_DIOINFO handling, Paolo Bonzini, 2022/01/10
- [PULL 07/18] configure, makefile: remove traces of really old files, Paolo Bonzini, 2022/01/10
- [PULL 05/18] configure: simplify creation of plugin symbol list,
Paolo Bonzini <=
- [PULL 06/18] configure: do not set bsd_user/linux_user early, Paolo Bonzini, 2022/01/10
- [PULL 09/18] configure: move non-command-line variables away from command-line parsing section, Paolo Bonzini, 2022/01/10
- [PULL 08/18] configure: parse --enable/--disable-strip automatically, flip default, Paolo Bonzini, 2022/01/10
- [PULL 10/18] meson: build contrib/ executables after generated headers, Paolo Bonzini, 2022/01/10
- [PULL 11/18] configure, meson: move config-poison.h to meson, Paolo Bonzini, 2022/01/10
- [PULL 12/18] meson: add comments in the target-specific flags section, Paolo Bonzini, 2022/01/10
- [PULL 13/18] KVM: use KVM_{GET|SET}_SREGS2 when supported., Paolo Bonzini, 2022/01/10
- [PULL 14/18] KVM: x86: ignore interrupt_bitmap field of KVM_GET/SET_SREGS, Paolo Bonzini, 2022/01/10
- [PULL 15/18] tests/tcg: Fix target-specific Makefile variables path for user-mode, Paolo Bonzini, 2022/01/10
- [PULL 16/18] configure: do not create roms/seabios/config.mak if SeaBIOS not present, Paolo Bonzini, 2022/01/10