plugins: do not limit exported symbols if modules are active
On Mac --enable-modules and --enable-plugins are currently incompatible, because the
Apple -Wl,-exported_symbols_list command line options prevents the export of any
symbols needed by the modules. On x86 -Wl,--dynamic-list does not have this effect,
but only because the -Wl,--export-dynamic option provided by gmodule-2.0.pc overrides
it. On Apple there is no -Wl,--export-dynamic, because it is the default, and thus
no override.
Either way, when modules are active there is no reason to include the plugin_ldflags.
While at it, avoid the useless -Wl,--export-dynamic when --enable-plugins is
specified but --enable-modules is not; this way, the GNU and Apple configurations
are more similar.
Resolves: https://gitlab.com/qemu-project/qemu/-/issues/516
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
[AJB: fix noexport to no-export]
Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
Message-Id: <20210811100550.54714-1-pbonzini@redhat.com>
Cc: qemu-stable@nongnu.org
2021-08-11 13:05:50 +03:00
|
|
|
# Modules need more symbols than just those in plugins/qemu-plugins.symbols
|
|
|
|
if not enable_modules
|
2023-11-03 11:17:48 +03:00
|
|
|
if host_os == 'darwin'
|
2022-07-14 11:43:57 +03:00
|
|
|
configure_file(
|
2021-12-10 11:55:15 +03:00
|
|
|
input: files('qemu-plugins.symbols'),
|
|
|
|
output: 'qemu-plugins-ld64.symbols',
|
|
|
|
capture: true,
|
|
|
|
command: ['sed', '-ne', 's/^[[:space:]]*\\(qemu_.*\\);/_\\1/p', '@INPUT@'])
|
2024-06-06 16:07:23 +03:00
|
|
|
emulator_link_args += ['-Wl,-exported_symbols_list,plugins/qemu-plugins-ld64.symbols']
|
2021-12-10 11:55:15 +03:00
|
|
|
else
|
2024-06-06 16:07:23 +03:00
|
|
|
emulator_link_args += ['-Xlinker', '--dynamic-list=' + (meson.project_source_root() / 'plugins/qemu-plugins.symbols')]
|
plugins: do not limit exported symbols if modules are active
On Mac --enable-modules and --enable-plugins are currently incompatible, because the
Apple -Wl,-exported_symbols_list command line options prevents the export of any
symbols needed by the modules. On x86 -Wl,--dynamic-list does not have this effect,
but only because the -Wl,--export-dynamic option provided by gmodule-2.0.pc overrides
it. On Apple there is no -Wl,--export-dynamic, because it is the default, and thus
no override.
Either way, when modules are active there is no reason to include the plugin_ldflags.
While at it, avoid the useless -Wl,--export-dynamic when --enable-plugins is
specified but --enable-modules is not; this way, the GNU and Apple configurations
are more similar.
Resolves: https://gitlab.com/qemu-project/qemu/-/issues/516
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
[AJB: fix noexport to no-export]
Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
Message-Id: <20210811100550.54714-1-pbonzini@redhat.com>
Cc: qemu-stable@nongnu.org
2021-08-11 13:05:50 +03:00
|
|
|
endif
|
2020-08-03 18:04:25 +03:00
|
|
|
endif
|
|
|
|
|
2023-08-30 13:20:53 +03:00
|
|
|
if get_option('plugins')
|
2023-11-03 11:17:48 +03:00
|
|
|
if host_os == 'windows'
|
2023-11-06 21:51:04 +03:00
|
|
|
dlltool = find_program('dlltool', required: true)
|
|
|
|
|
|
|
|
# Generate a .lib file for plugins to link against.
|
|
|
|
# First, create a .def file listing all the symbols a plugin should expect to have
|
|
|
|
# available in qemu
|
|
|
|
win32_plugin_def = configure_file(
|
|
|
|
input: files('qemu-plugins.symbols'),
|
|
|
|
output: 'qemu_plugin_api.def',
|
|
|
|
capture: true,
|
|
|
|
command: ['sed', '-e', '0,/^/s//EXPORTS/; s/[{};]//g', '@INPUT@'])
|
|
|
|
# then use dlltool to assemble a delaylib.
|
|
|
|
win32_qemu_plugin_api_lib = configure_file(
|
|
|
|
input: win32_plugin_def,
|
2023-11-20 18:08:23 +03:00
|
|
|
output: 'libqemu_plugin_api.a',
|
2023-11-06 21:51:04 +03:00
|
|
|
command: [dlltool, '--input-def', '@INPUT@',
|
|
|
|
'--output-delaylib', '@OUTPUT@', '--dllname', 'qemu.exe']
|
|
|
|
)
|
|
|
|
endif
|
2023-08-30 13:20:53 +03:00
|
|
|
specific_ss.add(files(
|
|
|
|
'loader.c',
|
|
|
|
'core.c',
|
|
|
|
'api.c',
|
2024-06-06 16:07:23 +03:00
|
|
|
))
|
2023-08-30 13:20:53 +03:00
|
|
|
endif
|