| Message ID | 20200917235720.2685978-2-richard.henderson@linaro.org |
|---|---|
| State | New |
| Headers | show |
| Series | capstone + disassembler patches | expand |
On Fri, Sep 18, 2020 at 7:59 AM Richard Henderson < richard.henderson@linaro.org> wrote: > > There are better ways to do this, e.g. meson cmake subproject, > but that requires cmake 3.7 and some of our CI environments > only provide cmake 3.5. > > Nor can we add a meson.build file to capstone/, because the git > submodule would then always report "untracked files". Fixing that > would require creating our own branch on the qemu git mirror, at > which point we could just as easily create a native meson subproject. > > Instead, build the library via the main meson.build. > > This improves the current state of affairs in that we will re-link > the qemu executables against a changed libcapstone.a, which we wouldn't > do before-hand. In addition, the use of the configuration header file > instead of command-line -DEFINES means that we will rebuild the > capstone objects with changes to meson.build. > > Signed-off-by: Richard Henderson <richard.henderson@linaro.org> > --- > Cc: Paolo Bonzini <pbonzini@redhat.com> > v2: Further reduce probing in configure (paolo), > Drop state 'internal' and use 'git' even when it isn't git. > Move CONFIG_CAPSTONE to config_host_data. > v3: Add Submodules separator; fix default in meson_options.txt. > --- > configure | 61 +++---------------------- > Makefile | 16 ------- > meson.build | 111 +++++++++++++++++++++++++++++++++++++++++++--- > meson_options.txt | 4 ++ > 4 files changed, 115 insertions(+), 77 deletions(-) > > diff --git a/configure b/configure > index 7564479008..76636c430d 100755 > --- a/configure > +++ b/configure > @@ -478,7 +478,7 @@ opengl="" > opengl_dmabuf="no" > cpuid_h="no" > avx2_opt="" > -capstone="" > +capstone="auto" > lzo="" > snappy="" > bzip2="" > @@ -1580,7 +1580,7 @@ for opt do > ;; > --enable-capstone) capstone="yes" > ;; > - --enable-capstone=git) capstone="git" > + --enable-capstone=git) capstone="internal" > ;; > --enable-capstone=system) capstone="system" > ;; > @@ -5128,51 +5128,11 @@ fi > # capstone > > case "$capstone" in > - "" | yes) > - if $pkg_config capstone; then > - capstone=system > - elif test -e "${source_path}/.git" && test $git_update = 'yes' ; then > - capstone=git > - elif test -e "${source_path}/capstone/Makefile" ; then > - capstone=internal > - elif test -z "$capstone" ; then > - capstone=no > - else > - feature_not_found "capstone" "Install capstone devel or git submodule" > - fi > - ;; > - > - system) > - if ! $pkg_config capstone; then > - feature_not_found "capstone" "Install capstone devel" > - fi > - ;; > -esac > - > -case "$capstone" in > - git | internal) > - if test "$capstone" = git; then > + auto | yes | internal) > + # Simpler to always update submodule, even if not needed. > + if test -e "${source_path}/.git" && test $git_update = 'yes' ; then > git_submodules="${git_submodules} capstone" > fi > - mkdir -p capstone > - if test "$mingw32" = "yes"; then > - LIBCAPSTONE=capstone.lib > - else > - LIBCAPSTONE=libcapstone.a > - fi > - capstone_libs="-Lcapstone -lcapstone" > - capstone_cflags="-I${source_path}/capstone/include" > - ;; > - > - system) > - capstone_libs="$($pkg_config --libs capstone)" > - capstone_cflags="$($pkg_config --cflags capstone)" > - ;; > - > - no) > - ;; > - *) > - error_exit "Unknown state for capstone: $capstone" > ;; > esac > > @@ -7292,11 +7252,6 @@ fi > if test "$ivshmem" = "yes" ; then > echo "CONFIG_IVSHMEM=y" >> $config_host_mak > fi > -if test "$capstone" != "no" ; then > - echo "CONFIG_CAPSTONE=y" >> $config_host_mak > - echo "CAPSTONE_CFLAGS=$capstone_cflags" >> $config_host_mak > - echo "CAPSTONE_LIBS=$capstone_libs" >> $config_host_mak > -fi > if test "$debug_mutex" = "yes" ; then > echo "CONFIG_DEBUG_MUTEX=y" >> $config_host_mak > fi > @@ -7819,9 +7774,6 @@ done # for target in $targets > if [ "$fdt" = "git" ]; then > subdirs="$subdirs dtc" > fi > -if [ "$capstone" = "git" -o "$capstone" = "internal" ]; then > - subdirs="$subdirs capstone" > -fi > echo "SUBDIRS=$subdirs" >> $config_host_mak > if test -n "$LIBCAPSTONE"; then > echo "LIBCAPSTONE=$LIBCAPSTONE" >> $config_host_mak These LIBCAPSTONE should be removed, as they are empty and have no meaning now > @@ -8008,7 +7960,8 @@ NINJA=${ninja:-$PWD/ninjatool} $meson setup \ > -Db_coverage=$(if test "$gcov" = yes; then echo true; else echo false; fi) \ > -Dsdl=$sdl -Dsdl_image=$sdl_image \ > -Dvnc=$vnc -Dvnc_sasl=$vnc_sasl -Dvnc_jpeg=$vnc_jpeg -Dvnc_png=$vnc_png \ > - -Dgettext=$gettext -Dxkbcommon=$xkbcommon -Du2f=$u2f\ > + -Dgettext=$gettext -Dxkbcommon=$xkbcommon -Du2f=$u2f \ > + -Dcapstone=$capstone \ These lines should ident with 8 space instead of two tab. There is four lines of them -Dsdl=$sdl -Dsdl_image=$sdl_image \ -Dvnc=$vnc -Dvnc_sasl=$vnc_sasl -Dvnc_jpeg=$vnc_jpeg -Dvnc_png=$vnc_png \ -Dgettext=$gettext -Dxkbcommon=$xkbcommon -Du2f=$u2f \ -Dcapstone=$capstone \ > $cross_arg \ > "$PWD" "$source_path" > > diff --git a/Makefile b/Makefile > index 7c60b9dcb8..f3da1760ad 100644 > --- a/Makefile > +++ b/Makefile > @@ -156,22 +156,6 @@ dtc/all: .git-submodule-status dtc/libfdt > dtc/%: .git-submodule-status > @mkdir -p $@ > > -# Overriding CFLAGS causes us to lose defines added in the sub-makefile. > -# Not overriding CFLAGS leads to mis-matches between compilation modes. > -# Therefore we replicate some of the logic in the sub-makefile. > -# Remove all the extra -Warning flags that QEMU uses that Capstone doesn't; > -# no need to annoy QEMU developers with such things. > -CAP_CFLAGS = $(patsubst -W%,,$(CFLAGS) $(QEMU_CFLAGS)) $(CAPSTONE_CFLAGS) > -CAP_CFLAGS += -DCAPSTONE_USE_SYS_DYN_MEM > -CAP_CFLAGS += -DCAPSTONE_HAS_ARM > -CAP_CFLAGS += -DCAPSTONE_HAS_ARM64 > -CAP_CFLAGS += -DCAPSTONE_HAS_POWERPC > -CAP_CFLAGS += -DCAPSTONE_HAS_X86 > - > -.PHONY: capstone/all > -capstone/all: .git-submodule-status > - $(call quiet-command,$(MAKE) -C $(SRC_PATH)/capstone CAPSTONE_SHARED=no BUILDDIR="$(BUILD_DIR)/capstone" CC="$(CC)" AR="$(AR)" LD="$(LD)" RANLIB="$(RANLIB)" CFLAGS="$(CAP_CFLAGS)" $(SUBDIR_MAKEFLAGS) $(BUILD_DIR)/capstone/$(LIBCAPSTONE)) > - > .PHONY: slirp/all > slirp/all: .git-submodule-status > $(call quiet-command,$(MAKE) -C $(SRC_PATH)/slirp \ > diff --git a/meson.build b/meson.build > index f4d1ab1096..f23273693d 100644 > --- a/meson.build > +++ b/meson.build > @@ -10,6 +10,7 @@ else > keyval = import('unstable-keyval') > endif > ss = import('sourceset') > +fs = import('fs') > > sh = find_program('sh') > cc = meson.get_compiler('c') > @@ -409,11 +410,6 @@ if 'CONFIG_USB_LIBUSB' in config_host > libusb = declare_dependency(compile_args: config_host['LIBUSB_CFLAGS'].split(), > link_args: config_host['LIBUSB_LIBS'].split()) > endif > -capstone = not_found > -if 'CONFIG_CAPSTONE' in config_host > - capstone = declare_dependency(compile_args: config_host['CAPSTONE_CFLAGS'].split(), > - link_args: config_host['CAPSTONE_LIBS'].split()) > -endif > libpmem = not_found > if 'CONFIG_LIBPMEM' in config_host > libpmem = declare_dependency(compile_args: config_host['LIBPMEM_CFLAGS'].split(), > @@ -470,7 +466,6 @@ foreach k, v: config_host > config_host_data.set(k, v == 'y' ? 1 : v) > endif > endforeach > -genh += configure_file(output: 'config-host.h', configuration: config_host_data) > > minikconf = find_program('scripts/minikconf.py') > config_all_devices = {} > @@ -610,6 +605,108 @@ config_all += { > 'CONFIG_ALL': true, > } > > +# Submodules > + > +capstone = not_found > +capstone_opt = get_option('capstone') > +if capstone_opt == 'no' > + capstone_opt = false > +elif capstone_opt in ['yes', 'auto', 'system'] > + have_internal = fs.exists('capstone/Makefile') > + capstone = dependency('capstone', static: enable_static, > + required: capstone_opt == 'system' or > + capstone_opt == 'yes' and not have_internal) > + if capstone.found() > + capstone_opt = 'system' > + elif have_internal > + capstone_opt = 'internal' > + else > + capstone_opt = false > + endif > +endif > +if capstone_opt == 'internal' > + capstone_data = configuration_data() > + capstone_data.set('CAPSTONE_USE_SYS_DYN_MEM', '1') > + > + capstone_files = files( > + 'capstone/cs.c', > + 'capstone/MCInst.c', > + 'capstone/MCInstrDesc.c', > + 'capstone/MCRegisterInfo.c', > + 'capstone/SStream.c', > + 'capstone/utils.c' > + ) > + > + if 'CONFIG_ARM_DIS' in config_all_disas > + capstone_data.set('CAPSTONE_HAS_ARM', '1') > + capstone_files += files( > + 'capstone/arch/ARM/ARMDisassembler.c', > + 'capstone/arch/ARM/ARMInstPrinter.c', > + 'capstone/arch/ARM/ARMMapping.c', > + 'capstone/arch/ARM/ARMModule.c' > + ) > + endif > + > + # FIXME: This config entry currently depends on a c++ compiler. > + # Which is needed for building libvixl, but not for capstone. > + if 'CONFIG_ARM_A64_DIS' in config_all_disas > + capstone_data.set('CAPSTONE_HAS_ARM64', '1') > + capstone_files += files( > + 'capstone/arch/AArch64/AArch64BaseInfo.c', > + 'capstone/arch/AArch64/AArch64Disassembler.c', > + 'capstone/arch/AArch64/AArch64InstPrinter.c', > + 'capstone/arch/AArch64/AArch64Mapping.c', > + 'capstone/arch/AArch64/AArch64Module.c' > + ) > + endif > + > + if 'CONFIG_PPC_DIS' in config_all_disas > + capstone_data.set('CAPSTONE_HAS_POWERPC', '1') > + capstone_files += files( > + 'capstone/arch/PowerPC/PPCDisassembler.c', > + 'capstone/arch/PowerPC/PPCInstPrinter.c', > + 'capstone/arch/PowerPC/PPCMapping.c', > + 'capstone/arch/PowerPC/PPCModule.c' > + ) > + endif > + > + if 'CONFIG_I386_DIS' in config_all_disas > + capstone_data.set('CAPSTONE_HAS_X86', 1) > + capstone_files += files( > + 'capstone/arch/X86/X86Disassembler.c', > + 'capstone/arch/X86/X86DisassemblerDecoder.c', > + 'capstone/arch/X86/X86ATTInstPrinter.c', > + 'capstone/arch/X86/X86IntelInstPrinter.c', > + 'capstone/arch/X86/X86Mapping.c', > + 'capstone/arch/X86/X86Module.c' > + ) > + endif > + > + configure_file(output: 'capstone-defs.h', configuration: capstone_data) > + > + capstone_cargs = [ > + # FIXME: There does not seem to be a way to completely replace the c_args > + # that come from add_project_arguments() -- we can only add to them. > + # So: disable all warnings with a big hammer. > + '-Wno-error', '-w', > + > + # Include all configuration defines via a header file, which will wind up > + # as a dependency on the object file, and thus changes here will result > + # in a rebuild. > + '-include', 'capstone-defs.h' > + ] > + > + libcapstone = static_library('capstone', > + sources: capstone_files, > + c_args: capstone_cargs, > + include_directories: 'capstone/include') > + capstone = declare_dependency(link_with: libcapstone, > + include_directories: 'capstone/include') > +endif > +config_host_data.set('CONFIG_CAPSTONE', capstone.found()) > + > +genh += configure_file(output: 'config-host.h', configuration: config_host_data) > + > # Generators > > hxtool = find_program('scripts/hxtool') > @@ -1512,7 +1609,7 @@ summary_info += {'vvfat support': config_host.has_key('CONFIG_VVFAT')} > summary_info += {'qed support': config_host.has_key('CONFIG_QED')} > summary_info += {'parallels support': config_host.has_key('CONFIG_PARALLELS')} > summary_info += {'sheepdog support': config_host.has_key('CONFIG_SHEEPDOG')} > -summary_info += {'capstone': config_host.has_key('CONFIG_CAPSTONE')} > +summary_info += {'capstone': capstone_opt} > summary_info += {'libpmem support': config_host.has_key('CONFIG_LIBPMEM')} > summary_info += {'libdaxctl support': config_host.has_key('CONFIG_LIBDAXCTL')} > summary_info += {'libudev': config_host.has_key('CONFIG_LIBUDEV')} > diff --git a/meson_options.txt b/meson_options.txt > index 543cf70043..e650a937e7 100644 > --- a/meson_options.txt > +++ b/meson_options.txt > @@ -22,3 +22,7 @@ option('vnc_sasl', type : 'feature', value : 'auto', > description: 'SASL authentication for VNC server') > option('xkbcommon', type : 'feature', value : 'auto', > description: 'xkbcommon support') > + > +option('capstone', type: 'combo', value: 'auto', > + choices: ['no', 'yes', 'auto', 'system', 'internal'], > + description: 'Whether and how to find the capstone library') > -- > 2.25.1 > > -- 此致 礼 罗勇刚 Yours sincerely, Yonggang Luo <div dir="ltr"><br><br>On Fri, Sep 18, 2020 at 7:59 AM Richard Henderson <<a href="mailto:richard.henderson@linaro.org">richard.henderson@linaro.org</a>> wrote:<br>><br>> There are better ways to do this, e.g. meson cmake subproject,<br>> but that requires cmake 3.7 and some of our CI environments<br>> only provide cmake 3.5.<br>><br>> Nor can we add a meson.build file to capstone/, because the git<br>> submodule would then always report "untracked files". Fixing that<br>> would require creating our own branch on the qemu git mirror, at<br>> which point we could just as easily create a native meson subproject.<br>><br>> Instead, build the library via the main meson.build.<br>><br>> This improves the current state of affairs in that we will re-link<br>> the qemu executables against a changed libcapstone.a, which we wouldn't<br>> do before-hand. In addition, the use of the configuration header file<br>> instead of command-line -DEFINES means that we will rebuild the<br>> capstone objects with changes to meson.build.<br>><br>> Signed-off-by: Richard Henderson <<a href="mailto:richard.henderson@linaro.org">richard.henderson@linaro.org</a>><br>> ---<br>> Cc: Paolo Bonzini <<a href="mailto:pbonzini@redhat.com">pbonzini@redhat.com</a>><br>> v2: Further reduce probing in configure (paolo),<br>> Drop state 'internal' and use 'git' even when it isn't git.<br>> Move CONFIG_CAPSTONE to config_host_data.<br>> v3: Add Submodules separator; fix default in meson_options.txt.<br>> ---<br>> configure | 61 +++----------------------<br>> Makefile | 16 -------<br>> meson.build | 111 +++++++++++++++++++++++++++++++++++++++++++---<br>> meson_options.txt | 4 ++<br>> 4 files changed, 115 insertions(+), 77 deletions(-)<br>><br>> diff --git a/configure b/configure<br>> index 7564479008..76636c430d 100755<br>> --- a/configure<br>> +++ b/configure<br>> @@ -478,7 +478,7 @@ opengl=""<br>> opengl_dmabuf="no"<br>> cpuid_h="no"<br>> avx2_opt=""<br>> -capstone=""<br>> +capstone="auto"<br>> lzo=""<br>> snappy=""<br>> bzip2=""<br>> @@ -1580,7 +1580,7 @@ for opt do<br>> ;;<br>> --enable-capstone) capstone="yes"<br>> ;;<br>> - --enable-capstone=git) capstone="git"<br>> + --enable-capstone=git) capstone="internal"<br>> ;;<br>> --enable-capstone=system) capstone="system"<br>> ;;<br>> @@ -5128,51 +5128,11 @@ fi<br>> # capstone<br>><br>> case "$capstone" in<br>> - "" | yes)<br>> - if $pkg_config capstone; then<br>> - capstone=system<br>> - elif test -e "${source_path}/.git" && test $git_update = 'yes' ; then<br>> - capstone=git<br>> - elif test -e "${source_path}/capstone/Makefile" ; then<br>> - capstone=internal<br>> - elif test -z "$capstone" ; then<br>> - capstone=no<br>> - else<br>> - feature_not_found "capstone" "Install capstone devel or git submodule"<br>> - fi<br>> - ;;<br>> -<br>> - system)<br>> - if ! $pkg_config capstone; then<br>> - feature_not_found "capstone" "Install capstone devel"<br>> - fi<br>> - ;;<br>> -esac<br>> -<br>> -case "$capstone" in<br>> - git | internal)<br>> - if test "$capstone" = git; then<br>> + auto | yes | internal)<br>> + # Simpler to always update submodule, even if not needed.<br>> + if test -e "${source_path}/.git" && test $git_update = 'yes' ; then<br>> git_submodules="${git_submodules} capstone"<br>> fi<br>> - mkdir -p capstone<br>> - if test "$mingw32" = "yes"; then<br>> - LIBCAPSTONE=capstone.lib<br>> - else<br>> - LIBCAPSTONE=libcapstone.a<br>> - fi<br>> - capstone_libs="-Lcapstone -lcapstone"<br>> - capstone_cflags="-I${source_path}/capstone/include"<br>> - ;;<br>> -<br>> - system)<br>> - capstone_libs="$($pkg_config --libs capstone)"<br>> - capstone_cflags="$($pkg_config --cflags capstone)"<br>> - ;;<br>> -<br>> - no)<br>> - ;;<br>> - *)<br>> - error_exit "Unknown state for capstone: $capstone"<br>> ;;<br>> esac<br>><br>> @@ -7292,11 +7252,6 @@ fi<br>> if test "$ivshmem" = "yes" ; then<br>> echo "CONFIG_IVSHMEM=y" >> $config_host_mak<br>> fi<br>> -if test "$capstone" != "no" ; then<br>> - echo "CONFIG_CAPSTONE=y" >> $config_host_mak<br>> - echo "CAPSTONE_CFLAGS=$capstone_cflags" >> $config_host_mak<br>> - echo "CAPSTONE_LIBS=$capstone_libs" >> $config_host_mak<br>> -fi<br>> if test "$debug_mutex" = "yes" ; then<br>> echo "CONFIG_DEBUG_MUTEX=y" >> $config_host_mak<br>> fi<br>> @@ -7819,9 +7774,6 @@ done # for target in $targets<br>> if [ "$fdt" = "git" ]; then<br>> subdirs="$subdirs dtc"<br>> fi<br>> -if [ "$capstone" = "git" -o "$capstone" = "internal" ]; then<br>> - subdirs="$subdirs capstone"<br>> -fi<br>> echo "SUBDIRS=$subdirs" >> $config_host_mak<br>> if test -n "$LIBCAPSTONE"; then<br>> echo "LIBCAPSTONE=$LIBCAPSTONE" >> $config_host_mak<div>These LIBCAPSTONE should be removed, as they are empty and have no meaning now<br>> @@ -8008,7 +7960,8 @@ NINJA=${ninja:-$PWD/ninjatool} $meson setup \<br>> -Db_coverage=$(if test "$gcov" = yes; then echo true; else echo false; fi) \<br>> -Dsdl=$sdl -Dsdl_image=$sdl_image \<br>> -Dvnc=$vnc -Dvnc_sasl=$vnc_sasl -Dvnc_jpeg=$vnc_jpeg -Dvnc_png=$vnc_png \<br>> - -Dgettext=$gettext -Dxkbcommon=$xkbcommon -Du2f=$u2f\<br>> + -Dgettext=$gettext -Dxkbcommon=$xkbcommon -Du2f=$u2f \<br>> + -Dcapstone=$capstone \</div><div>These lines should ident with 8 space instead of two tab.</div><div>There is four lines of them</div><div> -Dsdl=$sdl -Dsdl_image=$sdl_image \<br> -Dvnc=$vnc -Dvnc_sasl=$vnc_sasl -Dvnc_jpeg=$vnc_jpeg -Dvnc_png=$vnc_png \<br> -Dgettext=$gettext -Dxkbcommon=$xkbcommon -Du2f=$u2f \<br> -Dcapstone=$capstone \</div><div><br>> $cross_arg \<br>> "$PWD" "$source_path"<br>><br>> diff --git a/Makefile b/Makefile<br>> index 7c60b9dcb8..f3da1760ad 100644<br>> --- a/Makefile<br>> +++ b/Makefile<br>> @@ -156,22 +156,6 @@ dtc/all: .git-submodule-status dtc/libfdt<br>> dtc/%: .git-submodule-status<br>> @mkdir -p $@<br>><br>> -# Overriding CFLAGS causes us to lose defines added in the sub-makefile.<br>> -# Not overriding CFLAGS leads to mis-matches between compilation modes.<br>> -# Therefore we replicate some of the logic in the sub-makefile.<br>> -# Remove all the extra -Warning flags that QEMU uses that Capstone doesn't;<br>> -# no need to annoy QEMU developers with such things.<br>> -CAP_CFLAGS = $(patsubst -W%,,$(CFLAGS) $(QEMU_CFLAGS)) $(CAPSTONE_CFLAGS)<br>> -CAP_CFLAGS += -DCAPSTONE_USE_SYS_DYN_MEM<br>> -CAP_CFLAGS += -DCAPSTONE_HAS_ARM<br>> -CAP_CFLAGS += -DCAPSTONE_HAS_ARM64<br>> -CAP_CFLAGS += -DCAPSTONE_HAS_POWERPC<br>> -CAP_CFLAGS += -DCAPSTONE_HAS_X86<br>> -<br>> -.PHONY: capstone/all<br>> -capstone/all: .git-submodule-status<br>> - $(call quiet-command,$(MAKE) -C $(SRC_PATH)/capstone CAPSTONE_SHARED=no BUILDDIR="$(BUILD_DIR)/capstone" CC="$(CC)" AR="$(AR)" LD="$(LD)" RANLIB="$(RANLIB)" CFLAGS="$(CAP_CFLAGS)" $(SUBDIR_MAKEFLAGS) $(BUILD_DIR)/capstone/$(LIBCAPSTONE))<br>> -<br>> .PHONY: slirp/all<br>> slirp/all: .git-submodule-status<br>> $(call quiet-command,$(MAKE) -C $(SRC_PATH)/slirp \<br>> diff --git a/meson.build b/meson.build<br>> index f4d1ab1096..f23273693d 100644<br>> --- a/meson.build<br>> +++ b/meson.build<br>> @@ -10,6 +10,7 @@ else<br>> keyval = import('unstable-keyval')<br>> endif<br>> ss = import('sourceset')<br>> +fs = import('fs')<br>><br>> sh = find_program('sh')<br>> cc = meson.get_compiler('c')<br>> @@ -409,11 +410,6 @@ if 'CONFIG_USB_LIBUSB' in config_host<br>> libusb = declare_dependency(compile_args: config_host['LIBUSB_CFLAGS'].split(),<br>> link_args: config_host['LIBUSB_LIBS'].split())<br>> endif<br>> -capstone = not_found<br>> -if 'CONFIG_CAPSTONE' in config_host<br>> - capstone = declare_dependency(compile_args: config_host['CAPSTONE_CFLAGS'].split(),<br>> - link_args: config_host['CAPSTONE_LIBS'].split())<br>> -endif<br>> libpmem = not_found<br>> if 'CONFIG_LIBPMEM' in config_host<br>> libpmem = declare_dependency(compile_args: config_host['LIBPMEM_CFLAGS'].split(),<br>> @@ -470,7 +466,6 @@ foreach k, v: config_host<br>> config_host_data.set(k, v == 'y' ? 1 : v)<br>> endif<br>> endforeach<br>> -genh += configure_file(output: 'config-host.h', configuration: config_host_data)<br>><br>> minikconf = find_program('scripts/minikconf.py')<br>> config_all_devices = {}<br>> @@ -610,6 +605,108 @@ config_all += {<br>> 'CONFIG_ALL': true,<br>> }<br>><br>> +# Submodules<br>> +<br>> +capstone = not_found<br>> +capstone_opt = get_option('capstone')<br>> +if capstone_opt == 'no'<br>> + capstone_opt = false<br>> +elif capstone_opt in ['yes', 'auto', 'system']<br>> + have_internal = fs.exists('capstone/Makefile')<br>> + capstone = dependency('capstone', static: enable_static,<br>> + required: capstone_opt == 'system' or<br>> + capstone_opt == 'yes' and not have_internal)<br>> + if capstone.found()<br>> + capstone_opt = 'system'<br>> + elif have_internal<br>> + capstone_opt = 'internal'<br>> + else<br>> + capstone_opt = false<br>> + endif<br>> +endif<br>> +if capstone_opt == 'internal'<br>> + capstone_data = configuration_data()<br>> + capstone_data.set('CAPSTONE_USE_SYS_DYN_MEM', '1')<br>> +<br>> + capstone_files = files(<br>> + 'capstone/cs.c',<br>> + 'capstone/MCInst.c',<br>> + 'capstone/MCInstrDesc.c',<br>> + 'capstone/MCRegisterInfo.c',<br>> + 'capstone/SStream.c',<br>> + 'capstone/utils.c'<br>> + )<br>> +<br>> + if 'CONFIG_ARM_DIS' in config_all_disas<br>> + capstone_data.set('CAPSTONE_HAS_ARM', '1')<br>> + capstone_files += files(<br>> + 'capstone/arch/ARM/ARMDisassembler.c',<br>> + 'capstone/arch/ARM/ARMInstPrinter.c',<br>> + 'capstone/arch/ARM/ARMMapping.c',<br>> + 'capstone/arch/ARM/ARMModule.c'<br>> + )<br>> + endif<br>> +<br>> + # FIXME: This config entry currently depends on a c++ compiler.<br>> + # Which is needed for building libvixl, but not for capstone.<br>> + if 'CONFIG_ARM_A64_DIS' in config_all_disas<br>> + capstone_data.set('CAPSTONE_HAS_ARM64', '1')<br>> + capstone_files += files(<br>> + 'capstone/arch/AArch64/AArch64BaseInfo.c',<br>> + 'capstone/arch/AArch64/AArch64Disassembler.c',<br>> + 'capstone/arch/AArch64/AArch64InstPrinter.c',<br>> + 'capstone/arch/AArch64/AArch64Mapping.c',<br>> + 'capstone/arch/AArch64/AArch64Module.c'<br>> + )<br>> + endif<br>> +<br>> + if 'CONFIG_PPC_DIS' in config_all_disas<br>> + capstone_data.set('CAPSTONE_HAS_POWERPC', '1')<br>> + capstone_files += files(<br>> + 'capstone/arch/PowerPC/PPCDisassembler.c',<br>> + 'capstone/arch/PowerPC/PPCInstPrinter.c',<br>> + 'capstone/arch/PowerPC/PPCMapping.c',<br>> + 'capstone/arch/PowerPC/PPCModule.c'<br>> + )<br>> + endif<br>> +<br>> + if 'CONFIG_I386_DIS' in config_all_disas<br>> + capstone_data.set('CAPSTONE_HAS_X86', 1)<br>> + capstone_files += files(<br>> + 'capstone/arch/X86/X86Disassembler.c',<br>> + 'capstone/arch/X86/X86DisassemblerDecoder.c',<br>> + 'capstone/arch/X86/X86ATTInstPrinter.c',<br>> + 'capstone/arch/X86/X86IntelInstPrinter.c',<br>> + 'capstone/arch/X86/X86Mapping.c',<br>> + 'capstone/arch/X86/X86Module.c'<br>> + )<br>> + endif<br>> +<br>> + configure_file(output: 'capstone-defs.h', configuration: capstone_data)<br>> +<br>> + capstone_cargs = [<br>> + # FIXME: There does not seem to be a way to completely replace the c_args<br>> + # that come from add_project_arguments() -- we can only add to them.<br>> + # So: disable all warnings with a big hammer.<br>> + '-Wno-error', '-w',<br>> +<br>> + # Include all configuration defines via a header file, which will wind up<br>> + # as a dependency on the object file, and thus changes here will result<br>> + # in a rebuild.<br>> + '-include', 'capstone-defs.h'<br>> + ]<br>> +<br>> + libcapstone = static_library('capstone',<br>> + sources: capstone_files,<br>> + c_args: capstone_cargs,<br>> + include_directories: 'capstone/include')<br>> + capstone = declare_dependency(link_with: libcapstone,<br>> + include_directories: 'capstone/include')<br>> +endif<br>> +config_host_data.set('CONFIG_CAPSTONE', capstone.found())<br>> +<br>> +genh += configure_file(output: 'config-host.h', configuration: config_host_data)<br>> +<br>> # Generators<br>><br>> hxtool = find_program('scripts/hxtool')<br>> @@ -1512,7 +1609,7 @@ summary_info += {'vvfat support': config_host.has_key('CONFIG_VVFAT')}<br>> summary_info += {'qed support': config_host.has_key('CONFIG_QED')}<br>> summary_info += {'parallels support': config_host.has_key('CONFIG_PARALLELS')}<br>> summary_info += {'sheepdog support': config_host.has_key('CONFIG_SHEEPDOG')}<br>> -summary_info += {'capstone': config_host.has_key('CONFIG_CAPSTONE')}<br>> +summary_info += {'capstone': capstone_opt}<br>> summary_info += {'libpmem support': config_host.has_key('CONFIG_LIBPMEM')}<br>> summary_info += {'libdaxctl support': config_host.has_key('CONFIG_LIBDAXCTL')}<br>> summary_info += {'libudev': config_host.has_key('CONFIG_LIBUDEV')}<br>> diff --git a/meson_options.txt b/meson_options.txt<br>> index 543cf70043..e650a937e7 100644<br>> --- a/meson_options.txt<br>> +++ b/meson_options.txt<br>> @@ -22,3 +22,7 @@ option('vnc_sasl', type : 'feature', value : 'auto',<br>> description: 'SASL authentication for VNC server')<br>> option('xkbcommon', type : 'feature', value : 'auto',<br>> description: 'xkbcommon support')<br>> +<br>> +option('capstone', type: 'combo', value: 'auto',<br>> + choices: ['no', 'yes', 'auto', 'system', 'internal'],<br>> + description: 'Whether and how to find the capstone library')<br>> --<br>> 2.25.1<br>><br>><br><br><br>--<br> 此致<br>礼<br>罗勇刚<br>Yours<br> sincerely,<br>Yonggang Luo</div></div>
On 18/09/20 01:57, Richard Henderson wrote: > There are better ways to do this, e.g. meson cmake subproject, > but that requires cmake 3.7 and some of our CI environments > only provide cmake 3.5. > > Nor can we add a meson.build file to capstone/, because the git > submodule would then always report "untracked files". Fixing that > would require creating our own branch on the qemu git mirror, at > which point we could just as easily create a native meson subproject. > > Instead, build the library via the main meson.build. > > This improves the current state of affairs in that we will re-link > the qemu executables against a changed libcapstone.a, which we wouldn't > do before-hand. In addition, the use of the configuration header file > instead of command-line -DEFINES means that we will rebuild the > capstone objects with changes to meson.build. > > Signed-off-by: Richard Henderson <richard.henderson@linaro.org> > --- > Cc: Paolo Bonzini <pbonzini@redhat.com> > v2: Further reduce probing in configure (paolo), > Drop state 'internal' and use 'git' even when it isn't git. > Move CONFIG_CAPSTONE to config_host_data. > v3: Add Submodules separator; fix default in meson_options.txt. > --- Acked-by: Paolo Bonzini <pbonzini@redhat.com> Thanks! That's also a nice blueprint if anyone wants to do the same on libfdt. Paolo > configure | 61 +++---------------------- > Makefile | 16 ------- > meson.build | 111 +++++++++++++++++++++++++++++++++++++++++++--- > meson_options.txt | 4 ++ > 4 files changed, 115 insertions(+), 77 deletions(-) > > diff --git a/configure b/configure > index 7564479008..76636c430d 100755 > --- a/configure > +++ b/configure > @@ -478,7 +478,7 @@ opengl="" > opengl_dmabuf="no" > cpuid_h="no" > avx2_opt="" > -capstone="" > +capstone="auto" > lzo="" > snappy="" > bzip2="" > @@ -1580,7 +1580,7 @@ for opt do > ;; > --enable-capstone) capstone="yes" > ;; > - --enable-capstone=git) capstone="git" > + --enable-capstone=git) capstone="internal" > ;; > --enable-capstone=system) capstone="system" > ;; > @@ -5128,51 +5128,11 @@ fi > # capstone > > case "$capstone" in > - "" | yes) > - if $pkg_config capstone; then > - capstone=system > - elif test -e "${source_path}/.git" && test $git_update = 'yes' ; then > - capstone=git > - elif test -e "${source_path}/capstone/Makefile" ; then > - capstone=internal > - elif test -z "$capstone" ; then > - capstone=no > - else > - feature_not_found "capstone" "Install capstone devel or git submodule" > - fi > - ;; > - > - system) > - if ! $pkg_config capstone; then > - feature_not_found "capstone" "Install capstone devel" > - fi > - ;; > -esac > - > -case "$capstone" in > - git | internal) > - if test "$capstone" = git; then > + auto | yes | internal) > + # Simpler to always update submodule, even if not needed. > + if test -e "${source_path}/.git" && test $git_update = 'yes' ; then > git_submodules="${git_submodules} capstone" > fi > - mkdir -p capstone > - if test "$mingw32" = "yes"; then > - LIBCAPSTONE=capstone.lib > - else > - LIBCAPSTONE=libcapstone.a > - fi > - capstone_libs="-Lcapstone -lcapstone" > - capstone_cflags="-I${source_path}/capstone/include" > - ;; > - > - system) > - capstone_libs="$($pkg_config --libs capstone)" > - capstone_cflags="$($pkg_config --cflags capstone)" > - ;; > - > - no) > - ;; > - *) > - error_exit "Unknown state for capstone: $capstone" > ;; > esac > > @@ -7292,11 +7252,6 @@ fi > if test "$ivshmem" = "yes" ; then > echo "CONFIG_IVSHMEM=y" >> $config_host_mak > fi > -if test "$capstone" != "no" ; then > - echo "CONFIG_CAPSTONE=y" >> $config_host_mak > - echo "CAPSTONE_CFLAGS=$capstone_cflags" >> $config_host_mak > - echo "CAPSTONE_LIBS=$capstone_libs" >> $config_host_mak > -fi > if test "$debug_mutex" = "yes" ; then > echo "CONFIG_DEBUG_MUTEX=y" >> $config_host_mak > fi > @@ -7819,9 +7774,6 @@ done # for target in $targets > if [ "$fdt" = "git" ]; then > subdirs="$subdirs dtc" > fi > -if [ "$capstone" = "git" -o "$capstone" = "internal" ]; then > - subdirs="$subdirs capstone" > -fi > echo "SUBDIRS=$subdirs" >> $config_host_mak > if test -n "$LIBCAPSTONE"; then > echo "LIBCAPSTONE=$LIBCAPSTONE" >> $config_host_mak > @@ -8008,7 +7960,8 @@ NINJA=${ninja:-$PWD/ninjatool} $meson setup \ > -Db_coverage=$(if test "$gcov" = yes; then echo true; else echo false; fi) \ > -Dsdl=$sdl -Dsdl_image=$sdl_image \ > -Dvnc=$vnc -Dvnc_sasl=$vnc_sasl -Dvnc_jpeg=$vnc_jpeg -Dvnc_png=$vnc_png \ > - -Dgettext=$gettext -Dxkbcommon=$xkbcommon -Du2f=$u2f\ > + -Dgettext=$gettext -Dxkbcommon=$xkbcommon -Du2f=$u2f \ > + -Dcapstone=$capstone \ > $cross_arg \ > "$PWD" "$source_path" > > diff --git a/Makefile b/Makefile > index 7c60b9dcb8..f3da1760ad 100644 > --- a/Makefile > +++ b/Makefile > @@ -156,22 +156,6 @@ dtc/all: .git-submodule-status dtc/libfdt > dtc/%: .git-submodule-status > @mkdir -p $@ > > -# Overriding CFLAGS causes us to lose defines added in the sub-makefile. > -# Not overriding CFLAGS leads to mis-matches between compilation modes. > -# Therefore we replicate some of the logic in the sub-makefile. > -# Remove all the extra -Warning flags that QEMU uses that Capstone doesn't; > -# no need to annoy QEMU developers with such things. > -CAP_CFLAGS = $(patsubst -W%,,$(CFLAGS) $(QEMU_CFLAGS)) $(CAPSTONE_CFLAGS) > -CAP_CFLAGS += -DCAPSTONE_USE_SYS_DYN_MEM > -CAP_CFLAGS += -DCAPSTONE_HAS_ARM > -CAP_CFLAGS += -DCAPSTONE_HAS_ARM64 > -CAP_CFLAGS += -DCAPSTONE_HAS_POWERPC > -CAP_CFLAGS += -DCAPSTONE_HAS_X86 > - > -.PHONY: capstone/all > -capstone/all: .git-submodule-status > - $(call quiet-command,$(MAKE) -C $(SRC_PATH)/capstone CAPSTONE_SHARED=no BUILDDIR="$(BUILD_DIR)/capstone" CC="$(CC)" AR="$(AR)" LD="$(LD)" RANLIB="$(RANLIB)" CFLAGS="$(CAP_CFLAGS)" $(SUBDIR_MAKEFLAGS) $(BUILD_DIR)/capstone/$(LIBCAPSTONE)) > - > .PHONY: slirp/all > slirp/all: .git-submodule-status > $(call quiet-command,$(MAKE) -C $(SRC_PATH)/slirp \ > diff --git a/meson.build b/meson.build > index f4d1ab1096..f23273693d 100644 > --- a/meson.build > +++ b/meson.build > @@ -10,6 +10,7 @@ else > keyval = import('unstable-keyval') > endif > ss = import('sourceset') > +fs = import('fs') > > sh = find_program('sh') > cc = meson.get_compiler('c') > @@ -409,11 +410,6 @@ if 'CONFIG_USB_LIBUSB' in config_host > libusb = declare_dependency(compile_args: config_host['LIBUSB_CFLAGS'].split(), > link_args: config_host['LIBUSB_LIBS'].split()) > endif > -capstone = not_found > -if 'CONFIG_CAPSTONE' in config_host > - capstone = declare_dependency(compile_args: config_host['CAPSTONE_CFLAGS'].split(), > - link_args: config_host['CAPSTONE_LIBS'].split()) > -endif > libpmem = not_found > if 'CONFIG_LIBPMEM' in config_host > libpmem = declare_dependency(compile_args: config_host['LIBPMEM_CFLAGS'].split(), > @@ -470,7 +466,6 @@ foreach k, v: config_host > config_host_data.set(k, v == 'y' ? 1 : v) > endif > endforeach > -genh += configure_file(output: 'config-host.h', configuration: config_host_data) > > minikconf = find_program('scripts/minikconf.py') > config_all_devices = {} > @@ -610,6 +605,108 @@ config_all += { > 'CONFIG_ALL': true, > } > > +# Submodules > + > +capstone = not_found > +capstone_opt = get_option('capstone') > +if capstone_opt == 'no' > + capstone_opt = false > +elif capstone_opt in ['yes', 'auto', 'system'] > + have_internal = fs.exists('capstone/Makefile') > + capstone = dependency('capstone', static: enable_static, > + required: capstone_opt == 'system' or > + capstone_opt == 'yes' and not have_internal) > + if capstone.found() > + capstone_opt = 'system' > + elif have_internal > + capstone_opt = 'internal' > + else > + capstone_opt = false > + endif > +endif > +if capstone_opt == 'internal' > + capstone_data = configuration_data() > + capstone_data.set('CAPSTONE_USE_SYS_DYN_MEM', '1') > + > + capstone_files = files( > + 'capstone/cs.c', > + 'capstone/MCInst.c', > + 'capstone/MCInstrDesc.c', > + 'capstone/MCRegisterInfo.c', > + 'capstone/SStream.c', > + 'capstone/utils.c' > + ) > + > + if 'CONFIG_ARM_DIS' in config_all_disas > + capstone_data.set('CAPSTONE_HAS_ARM', '1') > + capstone_files += files( > + 'capstone/arch/ARM/ARMDisassembler.c', > + 'capstone/arch/ARM/ARMInstPrinter.c', > + 'capstone/arch/ARM/ARMMapping.c', > + 'capstone/arch/ARM/ARMModule.c' > + ) > + endif > + > + # FIXME: This config entry currently depends on a c++ compiler. > + # Which is needed for building libvixl, but not for capstone. > + if 'CONFIG_ARM_A64_DIS' in config_all_disas > + capstone_data.set('CAPSTONE_HAS_ARM64', '1') > + capstone_files += files( > + 'capstone/arch/AArch64/AArch64BaseInfo.c', > + 'capstone/arch/AArch64/AArch64Disassembler.c', > + 'capstone/arch/AArch64/AArch64InstPrinter.c', > + 'capstone/arch/AArch64/AArch64Mapping.c', > + 'capstone/arch/AArch64/AArch64Module.c' > + ) > + endif > + > + if 'CONFIG_PPC_DIS' in config_all_disas > + capstone_data.set('CAPSTONE_HAS_POWERPC', '1') > + capstone_files += files( > + 'capstone/arch/PowerPC/PPCDisassembler.c', > + 'capstone/arch/PowerPC/PPCInstPrinter.c', > + 'capstone/arch/PowerPC/PPCMapping.c', > + 'capstone/arch/PowerPC/PPCModule.c' > + ) > + endif > + > + if 'CONFIG_I386_DIS' in config_all_disas > + capstone_data.set('CAPSTONE_HAS_X86', 1) > + capstone_files += files( > + 'capstone/arch/X86/X86Disassembler.c', > + 'capstone/arch/X86/X86DisassemblerDecoder.c', > + 'capstone/arch/X86/X86ATTInstPrinter.c', > + 'capstone/arch/X86/X86IntelInstPrinter.c', > + 'capstone/arch/X86/X86Mapping.c', > + 'capstone/arch/X86/X86Module.c' > + ) > + endif > + > + configure_file(output: 'capstone-defs.h', configuration: capstone_data) > + > + capstone_cargs = [ > + # FIXME: There does not seem to be a way to completely replace the c_args > + # that come from add_project_arguments() -- we can only add to them. > + # So: disable all warnings with a big hammer. > + '-Wno-error', '-w', > + > + # Include all configuration defines via a header file, which will wind up > + # as a dependency on the object file, and thus changes here will result > + # in a rebuild. > + '-include', 'capstone-defs.h' > + ] > + > + libcapstone = static_library('capstone', > + sources: capstone_files, > + c_args: capstone_cargs, > + include_directories: 'capstone/include') > + capstone = declare_dependency(link_with: libcapstone, > + include_directories: 'capstone/include') > +endif > +config_host_data.set('CONFIG_CAPSTONE', capstone.found()) > + > +genh += configure_file(output: 'config-host.h', configuration: config_host_data) > + > # Generators > > hxtool = find_program('scripts/hxtool') > @@ -1512,7 +1609,7 @@ summary_info += {'vvfat support': config_host.has_key('CONFIG_VVFAT')} > summary_info += {'qed support': config_host.has_key('CONFIG_QED')} > summary_info += {'parallels support': config_host.has_key('CONFIG_PARALLELS')} > summary_info += {'sheepdog support': config_host.has_key('CONFIG_SHEEPDOG')} > -summary_info += {'capstone': config_host.has_key('CONFIG_CAPSTONE')} > +summary_info += {'capstone': capstone_opt} > summary_info += {'libpmem support': config_host.has_key('CONFIG_LIBPMEM')} > summary_info += {'libdaxctl support': config_host.has_key('CONFIG_LIBDAXCTL')} > summary_info += {'libudev': config_host.has_key('CONFIG_LIBUDEV')} > diff --git a/meson_options.txt b/meson_options.txt > index 543cf70043..e650a937e7 100644 > --- a/meson_options.txt > +++ b/meson_options.txt > @@ -22,3 +22,7 @@ option('vnc_sasl', type : 'feature', value : 'auto', > description: 'SASL authentication for VNC server') > option('xkbcommon', type : 'feature', value : 'auto', > description: 'xkbcommon support') > + > +option('capstone', type: 'combo', value: 'auto', > + choices: ['no', 'yes', 'auto', 'system', 'internal'], > + description: 'Whether and how to find the capstone library') >
On Fri, Sep 18, 2020 at 4:19 PM Paolo Bonzini <pbonzini@redhat.com> wrote: > > On 18/09/20 01:57, Richard Henderson wrote: > > There are better ways to do this, e.g. meson cmake subproject, > > but that requires cmake 3.7 and some of our CI environments > > only provide cmake 3.5. > > > > Nor can we add a meson.build file to capstone/, because the git > > submodule would then always report "untracked files". Fixing that > > would require creating our own branch on the qemu git mirror, at > > which point we could just as easily create a native meson subproject. > > > > Instead, build the library via the main meson.build. > > > > This improves the current state of affairs in that we will re-link > > the qemu executables against a changed libcapstone.a, which we wouldn't > > do before-hand. In addition, the use of the configuration header file > > instead of command-line -DEFINES means that we will rebuild the > > capstone objects with changes to meson.build. > > > > Signed-off-by: Richard Henderson <richard.henderson@linaro.org> > > --- > > Cc: Paolo Bonzini <pbonzini@redhat.com> > > v2: Further reduce probing in configure (paolo), > > Drop state 'internal' and use 'git' even when it isn't git. > > Move CONFIG_CAPSTONE to config_host_data. > > v3: Add Submodules separator; fix default in meson_options.txt. > > --- > > Acked-by: Paolo Bonzini <pbonzini@redhat.com> > > Thanks! That's also a nice blueprint if anyone wants to do the same on > libfdt. > > Paolo > > > > configure | 61 +++---------------------- > > Makefile | 16 ------- > > meson.build | 111 +++++++++++++++++++++++++++++++++++++++++++--- > > meson_options.txt | 4 ++ > > 4 files changed, 115 insertions(+), 77 deletions(-) > > > > diff --git a/configure b/configure > > index 7564479008..76636c430d 100755 > > --- a/configure > > +++ b/configure > > @@ -478,7 +478,7 @@ opengl="" > > opengl_dmabuf="no" > > cpuid_h="no" > > avx2_opt="" > > -capstone="" > > +capstone="auto" > > lzo="" > > snappy="" > > bzip2="" > > @@ -1580,7 +1580,7 @@ for opt do > > ;; > > --enable-capstone) capstone="yes" > > ;; > > - --enable-capstone=git) capstone="git" > > + --enable-capstone=git) capstone="internal" > > ;; > > --enable-capstone=system) capstone="system" > > ;; > > @@ -5128,51 +5128,11 @@ fi > > # capstone > > > > case "$capstone" in > > - "" | yes) > > - if $pkg_config capstone; then > > - capstone=system > > - elif test -e "${source_path}/.git" && test $git_update = 'yes' ; then > > - capstone=git > > - elif test -e "${source_path}/capstone/Makefile" ; then > > - capstone=internal > > - elif test -z "$capstone" ; then > > - capstone=no > > - else > > - feature_not_found "capstone" "Install capstone devel or git submodule" > > - fi > > - ;; > > - > > - system) > > - if ! $pkg_config capstone; then > > - feature_not_found "capstone" "Install capstone devel" > > - fi > > - ;; > > -esac > > - > > -case "$capstone" in > > - git | internal) > > - if test "$capstone" = git; then > > + auto | yes | internal) > > + # Simpler to always update submodule, even if not needed. > > + if test -e "${source_path}/.git" && test $git_update = 'yes' ; then > > git_submodules="${git_submodules} capstone" > > fi > > - mkdir -p capstone > > - if test "$mingw32" = "yes"; then > > - LIBCAPSTONE=capstone.lib > > - else > > - LIBCAPSTONE=libcapstone.a > > - fi > > - capstone_libs="-Lcapstone -lcapstone" > > - capstone_cflags="-I${source_path}/capstone/include" > > - ;; > > - > > - system) > > - capstone_libs="$($pkg_config --libs capstone)" > > - capstone_cflags="$($pkg_config --cflags capstone)" > > - ;; > > - > > - no) > > - ;; > > - *) > > - error_exit "Unknown state for capstone: $capstone" > > ;; > > esac > > > > @@ -7292,11 +7252,6 @@ fi > > if test "$ivshmem" = "yes" ; then > > echo "CONFIG_IVSHMEM=y" >> $config_host_mak > > fi > > -if test "$capstone" != "no" ; then > > - echo "CONFIG_CAPSTONE=y" >> $config_host_mak > > - echo "CAPSTONE_CFLAGS=$capstone_cflags" >> $config_host_mak > > - echo "CAPSTONE_LIBS=$capstone_libs" >> $config_host_mak > > -fi > > if test "$debug_mutex" = "yes" ; then > > echo "CONFIG_DEBUG_MUTEX=y" >> $config_host_mak > > fi > > @@ -7819,9 +7774,6 @@ done # for target in $targets > > if [ "$fdt" = "git" ]; then > > subdirs="$subdirs dtc" > > fi > > -if [ "$capstone" = "git" -o "$capstone" = "internal" ]; then > > - subdirs="$subdirs capstone" > > -fi > > echo "SUBDIRS=$subdirs" >> $config_host_mak > > if test -n "$LIBCAPSTONE"; then > > echo "LIBCAPSTONE=$LIBCAPSTONE" >> $config_host_mak > > @@ -8008,7 +7960,8 @@ NINJA=${ninja:-$PWD/ninjatool} $meson setup \ > > -Db_coverage=$(if test "$gcov" = yes; then echo true; else echo false; fi) \ > > -Dsdl=$sdl -Dsdl_image=$sdl_image \ > > -Dvnc=$vnc -Dvnc_sasl=$vnc_sasl -Dvnc_jpeg=$vnc_jpeg -Dvnc_png=$vnc_png \ > > - -Dgettext=$gettext -Dxkbcommon=$xkbcommon -Du2f=$u2f\ > > + -Dgettext=$gettext -Dxkbcommon=$xkbcommon -Du2f=$u2f \ > > + -Dcapstone=$capstone \ > > $cross_arg \ > > "$PWD" "$source_path" > > > > diff --git a/Makefile b/Makefile > > index 7c60b9dcb8..f3da1760ad 100644 > > --- a/Makefile > > +++ b/Makefile > > @@ -156,22 +156,6 @@ dtc/all: .git-submodule-status dtc/libfdt > > dtc/%: .git-submodule-status > > @mkdir -p $@ > > > > -# Overriding CFLAGS causes us to lose defines added in the sub-makefile. > > -# Not overriding CFLAGS leads to mis-matches between compilation modes. > > -# Therefore we replicate some of the logic in the sub-makefile. > > -# Remove all the extra -Warning flags that QEMU uses that Capstone doesn't; > > -# no need to annoy QEMU developers with such things. > > -CAP_CFLAGS = $(patsubst -W%,,$(CFLAGS) $(QEMU_CFLAGS)) $(CAPSTONE_CFLAGS) > > -CAP_CFLAGS += -DCAPSTONE_USE_SYS_DYN_MEM > > -CAP_CFLAGS += -DCAPSTONE_HAS_ARM > > -CAP_CFLAGS += -DCAPSTONE_HAS_ARM64 > > -CAP_CFLAGS += -DCAPSTONE_HAS_POWERPC > > -CAP_CFLAGS += -DCAPSTONE_HAS_X86 > > - > > -.PHONY: capstone/all > > -capstone/all: .git-submodule-status > > - $(call quiet-command,$(MAKE) -C $(SRC_PATH)/capstone CAPSTONE_SHARED=no BUILDDIR="$(BUILD_DIR)/capstone" CC="$(CC)" AR="$(AR)" LD="$(LD)" RANLIB="$(RANLIB)" CFLAGS="$(CAP_CFLAGS)" $(SUBDIR_MAKEFLAGS) $(BUILD_DIR)/capstone/$(LIBCAPSTONE)) > > - > > .PHONY: slirp/all > > slirp/all: .git-submodule-status > > $(call quiet-command,$(MAKE) -C $(SRC_PATH)/slirp \ > > diff --git a/meson.build b/meson.build > > index f4d1ab1096..f23273693d 100644 > > --- a/meson.build > > +++ b/meson.build > > @@ -10,6 +10,7 @@ else > > keyval = import('unstable-keyval') > > endif > > ss = import('sourceset') > > +fs = import('fs') > > > > sh = find_program('sh') > > cc = meson.get_compiler('c') > > @@ -409,11 +410,6 @@ if 'CONFIG_USB_LIBUSB' in config_host > > libusb = declare_dependency(compile_args: config_host['LIBUSB_CFLAGS'].split(), > > link_args: config_host['LIBUSB_LIBS'].split()) > > endif > > -capstone = not_found > > -if 'CONFIG_CAPSTONE' in config_host > > - capstone = declare_dependency(compile_args: config_host['CAPSTONE_CFLAGS'].split(), > > - link_args: config_host['CAPSTONE_LIBS'].split()) > > -endif > > libpmem = not_found > > if 'CONFIG_LIBPMEM' in config_host > > libpmem = declare_dependency(compile_args: config_host['LIBPMEM_CFLAGS'].split(), > > @@ -470,7 +466,6 @@ foreach k, v: config_host > > config_host_data.set(k, v == 'y' ? 1 : v) > > endif > > endforeach > > -genh += configure_file(output: 'config-host.h', configuration: config_host_data) > > > > minikconf = find_program('scripts/minikconf.py') > > config_all_devices = {} > > @@ -610,6 +605,108 @@ config_all += { > > 'CONFIG_ALL': true, > > } > > > > +# Submodules > > + > > +capstone = not_found > > +capstone_opt = get_option('capstone') > > +if capstone_opt == 'no' > > + capstone_opt = false > > +elif capstone_opt in ['yes', 'auto', 'system'] > > + have_internal = fs.exists('capstone/Makefile') > > + capstone = dependency('capstone', static: enable_static, > > + required: capstone_opt == 'system' or > > + capstone_opt == 'yes' and not have_internal) > > + if capstone.found() > > + capstone_opt = 'system' > > + elif have_internal > > + capstone_opt = 'internal' > > + else > > + capstone_opt = false > > + endif > > +endif > > +if capstone_opt == 'internal' > > + capstone_data = configuration_data() > > + capstone_data.set('CAPSTONE_USE_SYS_DYN_MEM', '1') > > + > > + capstone_files = files( > > + 'capstone/cs.c', > > + 'capstone/MCInst.c', > > + 'capstone/MCInstrDesc.c', > > + 'capstone/MCRegisterInfo.c', > > + 'capstone/SStream.c', > > + 'capstone/utils.c' > > + ) > > + > > + if 'CONFIG_ARM_DIS' in config_all_disas > > + capstone_data.set('CAPSTONE_HAS_ARM', '1') > > + capstone_files += files( > > + 'capstone/arch/ARM/ARMDisassembler.c', > > + 'capstone/arch/ARM/ARMInstPrinter.c', > > + 'capstone/arch/ARM/ARMMapping.c', > > + 'capstone/arch/ARM/ARMModule.c' > > + ) > > + endif > > + > > + # FIXME: This config entry currently depends on a c++ compiler. > > + # Which is needed for building libvixl, but not for capstone. > > + if 'CONFIG_ARM_A64_DIS' in config_all_disas > > + capstone_data.set('CAPSTONE_HAS_ARM64', '1') > > + capstone_files += files( > > + 'capstone/arch/AArch64/AArch64BaseInfo.c', > > + 'capstone/arch/AArch64/AArch64Disassembler.c', > > + 'capstone/arch/AArch64/AArch64InstPrinter.c', > > + 'capstone/arch/AArch64/AArch64Mapping.c', > > + 'capstone/arch/AArch64/AArch64Module.c' > > + ) > > + endif > > + > > + if 'CONFIG_PPC_DIS' in config_all_disas > > + capstone_data.set('CAPSTONE_HAS_POWERPC', '1') > > + capstone_files += files( > > + 'capstone/arch/PowerPC/PPCDisassembler.c', > > + 'capstone/arch/PowerPC/PPCInstPrinter.c', > > + 'capstone/arch/PowerPC/PPCMapping.c', > > + 'capstone/arch/PowerPC/PPCModule.c' > > + ) > > + endif > > + > > + if 'CONFIG_I386_DIS' in config_all_disas > > + capstone_data.set('CAPSTONE_HAS_X86', 1) > > + capstone_files += files( > > + 'capstone/arch/X86/X86Disassembler.c', > > + 'capstone/arch/X86/X86DisassemblerDecoder.c', > > + 'capstone/arch/X86/X86ATTInstPrinter.c', > > + 'capstone/arch/X86/X86IntelInstPrinter.c', > > + 'capstone/arch/X86/X86Mapping.c', > > + 'capstone/arch/X86/X86Module.c' > > + ) > > + endif > > + > > + configure_file(output: 'capstone-defs.h', configuration: capstone_data) > > + > > + capstone_cargs = [ > > + # FIXME: There does not seem to be a way to completely replace the c_args > > + # that come from add_project_arguments() -- we can only add to them. > > + # So: disable all warnings with a big hammer. > > + '-Wno-error', '-w', > > + > > + # Include all configuration defines via a header file, which will wind up > > + # as a dependency on the object file, and thus changes here will result > > + # in a rebuild. > > + '-include', 'capstone-defs.h' > > + ] > > + > > + libcapstone = static_library('capstone', > > + sources: capstone_files, > > + c_args: capstone_cargs, > > + include_directories: 'capstone/include') > > + capstone = declare_dependency(link_with: libcapstone, > > + include_directories: 'capstone/include') > > +endif > > +config_host_data.set('CONFIG_CAPSTONE', capstone.found()) > > + > > +genh += configure_file(output: 'config-host.h', configuration: config_host_data) > > + > > # Generators > > > > hxtool = find_program('scripts/hxtool') > > @@ -1512,7 +1609,7 @@ summary_info += {'vvfat support': config_host.has_key('CONFIG_VVFAT')} > > summary_info += {'qed support': config_host.has_key('CONFIG_QED')} > > summary_info += {'parallels support': config_host.has_key('CONFIG_PARALLELS')} > > summary_info += {'sheepdog support': config_host.has_key('CONFIG_SHEEPDOG')} > > -summary_info += {'capstone': config_host.has_key('CONFIG_CAPSTONE')} > > +summary_info += {'capstone': capstone_opt} > > summary_info += {'libpmem support': config_host.has_key('CONFIG_LIBPMEM')} > > summary_info += {'libdaxctl support': config_host.has_key('CONFIG_LIBDAXCTL')} > > summary_info += {'libudev': config_host.has_key('CONFIG_LIBUDEV')} > > diff --git a/meson_options.txt b/meson_options.txt > > index 543cf70043..e650a937e7 100644 > > --- a/meson_options.txt > > +++ b/meson_options.txt > > @@ -22,3 +22,7 @@ option('vnc_sasl', type : 'feature', value : 'auto', > > description: 'SASL authentication for VNC server') > > option('xkbcommon', type : 'feature', value : 'auto', > > description: 'xkbcommon support') > > + > > +option('capstone', type: 'combo', value: 'auto', > > + choices: ['no', 'yes', 'auto', 'system', 'internal'], > > + description: 'Whether and how to find the capstone library') > > > > I also have a question that how about convert --disable-capstone) capstone="no" ;; --enable-capstone) capstone="yes" ;; to --disable-capstone) capstone="disabled" ;; --enable-capstone) capstone="enabled" ;; for consistence with meson -- 此致 礼 罗勇刚 Yours sincerely, Yonggang Luo <div dir="ltr"><br><br>On Fri, Sep 18, 2020 at 4:19 PM Paolo Bonzini <<a href="mailto:pbonzini@redhat.com">pbonzini@redhat.com</a>> wrote:<br>><br>> On 18/09/20 01:57, Richard Henderson wrote:<br>> > There are better ways to do this, e.g. meson cmake subproject,<br>> > but that requires cmake 3.7 and some of our CI environments<br>> > only provide cmake 3.5.<br>> ><br>> > Nor can we add a meson.build file to capstone/, because the git<br>> > submodule would then always report "untracked files". Fixing that<br>> > would require creating our own branch on the qemu git mirror, at<br>> > which point we could just as easily create a native meson subproject.<br>> ><br>> > Instead, build the library via the main meson.build.<br>> ><br>> > This improves the current state of affairs in that we will re-link<br>> > the qemu executables against a changed libcapstone.a, which we wouldn't<br>> > do before-hand. In addition, the use of the configuration header file<br>> > instead of command-line -DEFINES means that we will rebuild the<br>> > capstone objects with changes to meson.build.<br>> ><br>> > Signed-off-by: Richard Henderson <<a href="mailto:richard.henderson@linaro.org">richard.henderson@linaro.org</a>><br>> > ---<br>> > Cc: Paolo Bonzini <<a href="mailto:pbonzini@redhat.com">pbonzini@redhat.com</a>><br>> > v2: Further reduce probing in configure (paolo),<br>> > Drop state 'internal' and use 'git' even when it isn't git.<br>> > Move CONFIG_CAPSTONE to config_host_data.<br>> > v3: Add Submodules separator; fix default in meson_options.txt.<br>> > ---<br>><br>> Acked-by: Paolo Bonzini <<a href="mailto:pbonzini@redhat.com">pbonzini@redhat.com</a>><br>><br>> Thanks! That's also a nice blueprint if anyone wants to do the same on<br>> libfdt.<br>><br>> Paolo<br>><br>><br>> > configure | 61 +++----------------------<br>> > Makefile | 16 -------<br>> > meson.build | 111 +++++++++++++++++++++++++++++++++++++++++++---<br>> > meson_options.txt | 4 ++<br>> > 4 files changed, 115 insertions(+), 77 deletions(-)<br>> ><br>> > diff --git a/configure b/configure<br>> > index 7564479008..76636c430d 100755<br>> > --- a/configure<br>> > +++ b/configure<br>> > @@ -478,7 +478,7 @@ opengl=""<br>> > opengl_dmabuf="no"<br>> > cpuid_h="no"<br>> > avx2_opt=""<br>> > -capstone=""<br>> > +capstone="auto"<br>> > lzo=""<br>> > snappy=""<br>> > bzip2=""<br>> > @@ -1580,7 +1580,7 @@ for opt do<br>> > ;;<br>> > --enable-capstone) capstone="yes"<br>> > ;;<br>> > - --enable-capstone=git) capstone="git"<br>> > + --enable-capstone=git) capstone="internal"<br>> > ;;<br>> > --enable-capstone=system) capstone="system"<br>> > ;;<br>> > @@ -5128,51 +5128,11 @@ fi<br>> > # capstone<br>> > <br>> > case "$capstone" in<br>> > - "" | yes)<br>> > - if $pkg_config capstone; then<br>> > - capstone=system<br>> > - elif test -e "${source_path}/.git" && test $git_update = 'yes' ; then<br>> > - capstone=git<br>> > - elif test -e "${source_path}/capstone/Makefile" ; then<br>> > - capstone=internal<br>> > - elif test -z "$capstone" ; then<br>> > - capstone=no<br>> > - else<br>> > - feature_not_found "capstone" "Install capstone devel or git submodule"<br>> > - fi<br>> > - ;;<br>> > -<br>> > - system)<br>> > - if ! $pkg_config capstone; then<br>> > - feature_not_found "capstone" "Install capstone devel"<br>> > - fi<br>> > - ;;<br>> > -esac<br>> > -<br>> > -case "$capstone" in<br>> > - git | internal)<br>> > - if test "$capstone" = git; then<br>> > + auto | yes | internal)<br>> > + # Simpler to always update submodule, even if not needed.<br>> > + if test -e "${source_path}/.git" && test $git_update = 'yes' ; then<br>> > git_submodules="${git_submodules} capstone"<br>> > fi<br>> > - mkdir -p capstone<br>> > - if test "$mingw32" = "yes"; then<br>> > - LIBCAPSTONE=capstone.lib<br>> > - else<br>> > - LIBCAPSTONE=libcapstone.a<br>> > - fi<br>> > - capstone_libs="-Lcapstone -lcapstone"<br>> > - capstone_cflags="-I${source_path}/capstone/include"<br>> > - ;;<br>> > -<br>> > - system)<br>> > - capstone_libs="$($pkg_config --libs capstone)"<br>> > - capstone_cflags="$($pkg_config --cflags capstone)"<br>> > - ;;<br>> > -<br>> > - no)<br>> > - ;;<br>> > - *)<br>> > - error_exit "Unknown state for capstone: $capstone"<br>> > ;;<br>> > esac<br>> > <br>> > @@ -7292,11 +7252,6 @@ fi<br>> > if test "$ivshmem" = "yes" ; then<br>> > echo "CONFIG_IVSHMEM=y" >> $config_host_mak<br>> > fi<br>> > -if test "$capstone" != "no" ; then<br>> > - echo "CONFIG_CAPSTONE=y" >> $config_host_mak<br>> > - echo "CAPSTONE_CFLAGS=$capstone_cflags" >> $config_host_mak<br>> > - echo "CAPSTONE_LIBS=$capstone_libs" >> $config_host_mak<br>> > -fi<br>> > if test "$debug_mutex" = "yes" ; then<br>> > echo "CONFIG_DEBUG_MUTEX=y" >> $config_host_mak<br>> > fi<br>> > @@ -7819,9 +7774,6 @@ done # for target in $targets<br>> > if [ "$fdt" = "git" ]; then<br>> > subdirs="$subdirs dtc"<br>> > fi<br>> > -if [ "$capstone" = "git" -o "$capstone" = "internal" ]; then<br>> > - subdirs="$subdirs capstone"<br>> > -fi<br>> > echo "SUBDIRS=$subdirs" >> $config_host_mak<br>> > if test -n "$LIBCAPSTONE"; then<br>> > echo "LIBCAPSTONE=$LIBCAPSTONE" >> $config_host_mak<br>> > @@ -8008,7 +7960,8 @@ NINJA=${ninja:-$PWD/ninjatool} $meson setup \<br>> > -Db_coverage=$(if test "$gcov" = yes; then echo true; else echo false; fi) \<br>> > -Dsdl=$sdl -Dsdl_image=$sdl_image \<br>> > -Dvnc=$vnc -Dvnc_sasl=$vnc_sasl -Dvnc_jpeg=$vnc_jpeg -Dvnc_png=$vnc_png \<br>> > - -Dgettext=$gettext -Dxkbcommon=$xkbcommon -Du2f=$u2f\<br>> > + -Dgettext=$gettext -Dxkbcommon=$xkbcommon -Du2f=$u2f \<br>> > + -Dcapstone=$capstone \<br>> > $cross_arg \<br>> > "$PWD" "$source_path"<br>> > <br>> > diff --git a/Makefile b/Makefile<br>> > index 7c60b9dcb8..f3da1760ad 100644<br>> > --- a/Makefile<br>> > +++ b/Makefile<br>> > @@ -156,22 +156,6 @@ dtc/all: .git-submodule-status dtc/libfdt<br>> > dtc/%: .git-submodule-status<br>> > @mkdir -p $@<br>> > <br>> > -# Overriding CFLAGS causes us to lose defines added in the sub-makefile.<br>> > -# Not overriding CFLAGS leads to mis-matches between compilation modes.<br>> > -# Therefore we replicate some of the logic in the sub-makefile.<br>> > -# Remove all the extra -Warning flags that QEMU uses that Capstone doesn't;<br>> > -# no need to annoy QEMU developers with such things.<br>> > -CAP_CFLAGS = $(patsubst -W%,,$(CFLAGS) $(QEMU_CFLAGS)) $(CAPSTONE_CFLAGS)<br>> > -CAP_CFLAGS += -DCAPSTONE_USE_SYS_DYN_MEM<br>> > -CAP_CFLAGS += -DCAPSTONE_HAS_ARM<br>> > -CAP_CFLAGS += -DCAPSTONE_HAS_ARM64<br>> > -CAP_CFLAGS += -DCAPSTONE_HAS_POWERPC<br>> > -CAP_CFLAGS += -DCAPSTONE_HAS_X86<br>> > -<br>> > -.PHONY: capstone/all<br>> > -capstone/all: .git-submodule-status<br>> > - $(call quiet-command,$(MAKE) -C $(SRC_PATH)/capstone CAPSTONE_SHARED=no BUILDDIR="$(BUILD_DIR)/capstone" CC="$(CC)" AR="$(AR)" LD="$(LD)" RANLIB="$(RANLIB)" CFLAGS="$(CAP_CFLAGS)" $(SUBDIR_MAKEFLAGS) $(BUILD_DIR)/capstone/$(LIBCAPSTONE))<br>> > -<br>> > .PHONY: slirp/all<br>> > slirp/all: .git-submodule-status<br>> > $(call quiet-command,$(MAKE) -C $(SRC_PATH)/slirp \<br>> > diff --git a/meson.build b/meson.build<br>> > index f4d1ab1096..f23273693d 100644<br>> > --- a/meson.build<br>> > +++ b/meson.build<br>> > @@ -10,6 +10,7 @@ else<br>> > keyval = import('unstable-keyval')<br>> > endif<br>> > ss = import('sourceset')<br>> > +fs = import('fs')<br>> > <br>> > sh = find_program('sh')<br>> > cc = meson.get_compiler('c')<br>> > @@ -409,11 +410,6 @@ if 'CONFIG_USB_LIBUSB' in config_host<br>> > libusb = declare_dependency(compile_args: config_host['LIBUSB_CFLAGS'].split(),<br>> > link_args: config_host['LIBUSB_LIBS'].split())<br>> > endif<br>> > -capstone = not_found<br>> > -if 'CONFIG_CAPSTONE' in config_host<br>> > - capstone = declare_dependency(compile_args: config_host['CAPSTONE_CFLAGS'].split(),<br>> > - link_args: config_host['CAPSTONE_LIBS'].split())<br>> > -endif<br>> > libpmem = not_found<br>> > if 'CONFIG_LIBPMEM' in config_host<br>> > libpmem = declare_dependency(compile_args: config_host['LIBPMEM_CFLAGS'].split(),<br>> > @@ -470,7 +466,6 @@ foreach k, v: config_host<br>> > config_host_data.set(k, v == 'y' ? 1 : v)<br>> > endif<br>> > endforeach<br>> > -genh += configure_file(output: 'config-host.h', configuration: config_host_data)<br>> > <br>> > minikconf = find_program('scripts/minikconf.py')<br>> > config_all_devices = {}<br>> > @@ -610,6 +605,108 @@ config_all += {<br>> > 'CONFIG_ALL': true,<br>> > }<br>> > <br>> > +# Submodules<br>> > +<br>> > +capstone = not_found<br>> > +capstone_opt = get_option('capstone')<br>> > +if capstone_opt == 'no'<br>> > + capstone_opt = false<br>> > +elif capstone_opt in ['yes', 'auto', 'system']<br>> > + have_internal = fs.exists('capstone/Makefile')<br>> > + capstone = dependency('capstone', static: enable_static,<br>> > + required: capstone_opt == 'system' or<br>> > + capstone_opt == 'yes' and not have_internal)<br>> > + if capstone.found()<br>> > + capstone_opt = 'system'<br>> > + elif have_internal<br>> > + capstone_opt = 'internal'<br>> > + else<br>> > + capstone_opt = false<br>> > + endif<br>> > +endif<br>> > +if capstone_opt == 'internal'<br>> > + capstone_data = configuration_data()<br>> > + capstone_data.set('CAPSTONE_USE_SYS_DYN_MEM', '1')<br>> > +<br>> > + capstone_files = files(<br>> > + 'capstone/cs.c',<br>> > + 'capstone/MCInst.c',<br>> > + 'capstone/MCInstrDesc.c',<br>> > + 'capstone/MCRegisterInfo.c',<br>> > + 'capstone/SStream.c',<br>> > + 'capstone/utils.c'<br>> > + )<br>> > +<br>> > + if 'CONFIG_ARM_DIS' in config_all_disas<br>> > + capstone_data.set('CAPSTONE_HAS_ARM', '1')<br>> > + capstone_files += files(<br>> > + 'capstone/arch/ARM/ARMDisassembler.c',<br>> > + 'capstone/arch/ARM/ARMInstPrinter.c',<br>> > + 'capstone/arch/ARM/ARMMapping.c',<br>> > + 'capstone/arch/ARM/ARMModule.c'<br>> > + )<br>> > + endif<br>> > +<br>> > + # FIXME: This config entry currently depends on a c++ compiler.<br>> > + # Which is needed for building libvixl, but not for capstone.<br>> > + if 'CONFIG_ARM_A64_DIS' in config_all_disas<br>> > + capstone_data.set('CAPSTONE_HAS_ARM64', '1')<br>> > + capstone_files += files(<br>> > + 'capstone/arch/AArch64/AArch64BaseInfo.c',<br>> > + 'capstone/arch/AArch64/AArch64Disassembler.c',<br>> > + 'capstone/arch/AArch64/AArch64InstPrinter.c',<br>> > + 'capstone/arch/AArch64/AArch64Mapping.c',<br>> > + 'capstone/arch/AArch64/AArch64Module.c'<br>> > + )<br>> > + endif<br>> > +<br>> > + if 'CONFIG_PPC_DIS' in config_all_disas<br>> > + capstone_data.set('CAPSTONE_HAS_POWERPC', '1')<br>> > + capstone_files += files(<br>> > + 'capstone/arch/PowerPC/PPCDisassembler.c',<br>> > + 'capstone/arch/PowerPC/PPCInstPrinter.c',<br>> > + 'capstone/arch/PowerPC/PPCMapping.c',<br>> > + 'capstone/arch/PowerPC/PPCModule.c'<br>> > + )<br>> > + endif<br>> > +<br>> > + if 'CONFIG_I386_DIS' in config_all_disas<br>> > + capstone_data.set('CAPSTONE_HAS_X86', 1)<br>> > + capstone_files += files(<br>> > + 'capstone/arch/X86/X86Disassembler.c',<br>> > + 'capstone/arch/X86/X86DisassemblerDecoder.c',<br>> > + 'capstone/arch/X86/X86ATTInstPrinter.c',<br>> > + 'capstone/arch/X86/X86IntelInstPrinter.c',<br>> > + 'capstone/arch/X86/X86Mapping.c',<br>> > + 'capstone/arch/X86/X86Module.c'<br>> > + )<br>> > + endif<br>> > +<br>> > + configure_file(output: 'capstone-defs.h', configuration: capstone_data)<br>> > +<br>> > + capstone_cargs = [<br>> > + # FIXME: There does not seem to be a way to completely replace the c_args<br>> > + # that come from add_project_arguments() -- we can only add to them.<br>> > + # So: disable all warnings with a big hammer.<br>> > + '-Wno-error', '-w',<br>> > +<br>> > + # Include all configuration defines via a header file, which will wind up<br>> > + # as a dependency on the object file, and thus changes here will result<br>> > + # in a rebuild.<br>> > + '-include', 'capstone-defs.h'<br>> > + ]<br>> > +<br>> > + libcapstone = static_library('capstone',<br>> > + sources: capstone_files,<br>> > + c_args: capstone_cargs,<br>> > + include_directories: 'capstone/include')<br>> > + capstone = declare_dependency(link_with: libcapstone,<br>> > + include_directories: 'capstone/include')<br>> > +endif<br>> > +config_host_data.set('CONFIG_CAPSTONE', capstone.found())<br>> > +<br>> > +genh += configure_file(output: 'config-host.h', configuration: config_host_data)<br>> > +<br>> > # Generators<br>> > <br>> > hxtool = find_program('scripts/hxtool')<br>> > @@ -1512,7 +1609,7 @@ summary_info += {'vvfat support': config_host.has_key('CONFIG_VVFAT')}<br>> > summary_info += {'qed support': config_host.has_key('CONFIG_QED')}<br>> > summary_info += {'parallels support': config_host.has_key('CONFIG_PARALLELS')}<br>> > summary_info += {'sheepdog support': config_host.has_key('CONFIG_SHEEPDOG')}<br>> > -summary_info += {'capstone': config_host.has_key('CONFIG_CAPSTONE')}<br>> > +summary_info += {'capstone': capstone_opt}<br>> > summary_info += {'libpmem support': config_host.has_key('CONFIG_LIBPMEM')}<br>> > summary_info += {'libdaxctl support': config_host.has_key('CONFIG_LIBDAXCTL')}<br>> > summary_info += {'libudev': config_host.has_key('CONFIG_LIBUDEV')}<br>> > diff --git a/meson_options.txt b/meson_options.txt<br>> > index 543cf70043..e650a937e7 100644<br>> > --- a/meson_options.txt<br>> > +++ b/meson_options.txt<br>> > @@ -22,3 +22,7 @@ option('vnc_sasl', type : 'feature', value : 'auto',<br>> > description: 'SASL authentication for VNC server')<br>> > option('xkbcommon', type : 'feature', value : 'auto',<br>> > description: 'xkbcommon support')<br>> > +<br>> > +option('capstone', type: 'combo', value: 'auto',<br>> > + choices: ['no', 'yes', 'auto', 'system', 'internal'],<br>> > + description: 'Whether and how to find the capstone library')<br>> ><br>><br>><div>I also have a question that how about convert</div><div> --disable-capstone) capstone="no"<br> ;;<br> --enable-capstone) capstone="yes"<br> ;;</div><div>to </div><div> --disable-capstone) capstone="disabled"<br> ;;<br> --enable-capstone) capstone="enabled"<br> ;;</div><div><br></div><div>for consistence with meson<br>--<br> 此致<br>礼<br>罗勇刚<br>Yours<br> sincerely,<br>Yonggang Luo</div></div>
On 18/09/20 10:42, 罗勇刚(Yonggang Luo) wrote: > I also have a question that how about convert > --disable-capstone) capstone="no" > ;; > --enable-capstone) capstone="yes" > ;; > to > --disable-capstone) capstone="disabled" > ;; > --enable-capstone) capstone="enabled" > ;; > > for consistence with meson We can do it but it need not hold this series IMHO. Paolo
On Fri, Sep 18, 2020 at 8:58 PM Paolo Bonzini <pbonzini@redhat.com> wrote: > > On 18/09/20 10:42, 罗勇刚(Yonggang Luo) wrote: > > I also have a question that how about convert > > --disable-capstone) capstone="no" > > ;; > > --enable-capstone) capstone="yes" > > ;; > > to > > --disable-capstone) capstone="disabled" > > ;; > > --enable-capstone) capstone="enabled" > > ;; > > > > for consistence with meson > > We can do it but it need not hold this series IMHO. Fine with that, I am trying to convert iconv and curses from configure to meson waiting for this series. > > Paolo > -- 此致 礼 罗勇刚 Yours sincerely, Yonggang Luo <div dir="ltr"><br><br>On Fri, Sep 18, 2020 at 8:58 PM Paolo Bonzini <<a href="mailto:pbonzini@redhat.com">pbonzini@redhat.com</a>> wrote:<br>><br>> On 18/09/20 10:42, 罗勇刚(Yonggang Luo) wrote:<br>> > I also have a question that how about convert<br>> > --disable-capstone) capstone="no"<br>> > ;;<br>> > --enable-capstone) capstone="yes"<br>> > ;;<br>> > to <br>> > --disable-capstone) capstone="disabled"<br>> > ;;<br>> > --enable-capstone) capstone="enabled"<br>> > ;;<br>> ><br>> > for consistence with meson<br>><br>> We can do it but it need not hold this series IMHO.<div>Fine with that, I am trying to convert iconv and curses from configure to meson</div><div>waiting for this series.<br>><br>> Paolo<br>><br><br><br>--<br> 此致<br>礼<br>罗勇刚<br>Yours<br> sincerely,<br>Yonggang Luo</div></div>
Richard Henderson <richard.henderson@linaro.org> writes: > There are better ways to do this, e.g. meson cmake subproject, > but that requires cmake 3.7 and some of our CI environments > only provide cmake 3.5. > > Nor can we add a meson.build file to capstone/, because the git > submodule would then always report "untracked files". Fixing that > would require creating our own branch on the qemu git mirror, at > which point we could just as easily create a native meson subproject. > > Instead, build the library via the main meson.build. > > This improves the current state of affairs in that we will re-link > the qemu executables against a changed libcapstone.a, which we wouldn't > do before-hand. In addition, the use of the configuration header file > instead of command-line -DEFINES means that we will rebuild the > capstone objects with changes to meson.build. Something is breaking when switching to a branch with this on from current master: Linking target qemu-hppa /usr/bin/ld: libcommon.fa.p/disas_alpha.c.o: in function `print_insn_alpha': /home/alex/lsrc/qemu.git/builds/all/../../disas/alpha.c:1818: undefined reference to `bfd_getl32' collect2: error: ld returned 1 exit status make: *** [Makefile.ninja:5965: qemu-alpha] Error 1 make: *** Waiting for unfinished jobs.... /usr/bin/ld: libcommon.fa.p/disas_hppa.c.o: in function `print_insn_hppa': /home/alex/lsrc/qemu.git/builds/all/../../disas/hppa.c:1969: undefined reference to `bfd_getb32' collect2: error: ld returned 1 exit status make: *** [Makefile.ninja:6259: qemu-hppa] Error 1 Aggressively wiping out the submodule and doing a fresh configure in a empty build directory and I still see a failure: ../../disas/capstone.c:25:1: error: expected ‘=’, ‘,’, ‘;’, ‘asm’ or ‘__attribute__’ before ‘cap_skipdata_s390x_cb’ cap_skipdata_s390x_cb(const uint8_t *code, size_t code_size, ^~~~~~~~~~~~~~~~~~~~~ ../../disas/capstone.c:49:17: error: ‘cap_skipdata_s390x_cb’ undeclared here (not in a function); did you mean ‘cap_skipdata_s390x’? .callback = cap_skipdata_s390x_cb ^~~~~~~~~~~~~~~~~~~~~ cap_skipdata_s390x Makefile.ninja:1424: recipe for target 'libcommon.fa.p/disas_capstone.c.o' failed make: *** [libcommon.fa.p/disas_capstone.c.o] Error 1 make: *** Waiting for unfinished jobs.... > > Signed-off-by: Richard Henderson <richard.henderson@linaro.org> > --- > Cc: Paolo Bonzini <pbonzini@redhat.com> > v2: Further reduce probing in configure (paolo), > Drop state 'internal' and use 'git' even when it isn't git. > Move CONFIG_CAPSTONE to config_host_data. > v3: Add Submodules separator; fix default in meson_options.txt. > --- > configure | 61 +++---------------------- > Makefile | 16 ------- > meson.build | 111 +++++++++++++++++++++++++++++++++++++++++++--- > meson_options.txt | 4 ++ > 4 files changed, 115 insertions(+), 77 deletions(-) > > diff --git a/configure b/configure > index 7564479008..76636c430d 100755 > --- a/configure > +++ b/configure > @@ -478,7 +478,7 @@ opengl="" > opengl_dmabuf="no" > cpuid_h="no" > avx2_opt="" > -capstone="" > +capstone="auto" > lzo="" > snappy="" > bzip2="" > @@ -1580,7 +1580,7 @@ for opt do > ;; > --enable-capstone) capstone="yes" > ;; > - --enable-capstone=git) capstone="git" > + --enable-capstone=git) capstone="internal" > ;; > --enable-capstone=system) capstone="system" > ;; > @@ -5128,51 +5128,11 @@ fi > # capstone > > case "$capstone" in > - "" | yes) > - if $pkg_config capstone; then > - capstone=system > - elif test -e "${source_path}/.git" && test $git_update = 'yes' ; then > - capstone=git > - elif test -e "${source_path}/capstone/Makefile" ; then > - capstone=internal > - elif test -z "$capstone" ; then > - capstone=no > - else > - feature_not_found "capstone" "Install capstone devel or git submodule" > - fi > - ;; > - > - system) > - if ! $pkg_config capstone; then > - feature_not_found "capstone" "Install capstone devel" > - fi > - ;; > -esac > - > -case "$capstone" in > - git | internal) > - if test "$capstone" = git; then > + auto | yes | internal) > + # Simpler to always update submodule, even if not needed. > + if test -e "${source_path}/.git" && test $git_update = 'yes' ; then > git_submodules="${git_submodules} capstone" > fi > - mkdir -p capstone > - if test "$mingw32" = "yes"; then > - LIBCAPSTONE=capstone.lib > - else > - LIBCAPSTONE=libcapstone.a > - fi > - capstone_libs="-Lcapstone -lcapstone" > - capstone_cflags="-I${source_path}/capstone/include" > - ;; > - > - system) > - capstone_libs="$($pkg_config --libs capstone)" > - capstone_cflags="$($pkg_config --cflags capstone)" > - ;; > - > - no) > - ;; > - *) > - error_exit "Unknown state for capstone: $capstone" > ;; > esac > > @@ -7292,11 +7252,6 @@ fi > if test "$ivshmem" = "yes" ; then > echo "CONFIG_IVSHMEM=y" >> $config_host_mak > fi > -if test "$capstone" != "no" ; then > - echo "CONFIG_CAPSTONE=y" >> $config_host_mak > - echo "CAPSTONE_CFLAGS=$capstone_cflags" >> $config_host_mak > - echo "CAPSTONE_LIBS=$capstone_libs" >> $config_host_mak > -fi > if test "$debug_mutex" = "yes" ; then > echo "CONFIG_DEBUG_MUTEX=y" >> $config_host_mak > fi > @@ -7819,9 +7774,6 @@ done # for target in $targets > if [ "$fdt" = "git" ]; then > subdirs="$subdirs dtc" > fi > -if [ "$capstone" = "git" -o "$capstone" = "internal" ]; then > - subdirs="$subdirs capstone" > -fi > echo "SUBDIRS=$subdirs" >> $config_host_mak > if test -n "$LIBCAPSTONE"; then > echo "LIBCAPSTONE=$LIBCAPSTONE" >> $config_host_mak > @@ -8008,7 +7960,8 @@ NINJA=${ninja:-$PWD/ninjatool} $meson setup \ > -Db_coverage=$(if test "$gcov" = yes; then echo true; else echo false; fi) \ > -Dsdl=$sdl -Dsdl_image=$sdl_image \ > -Dvnc=$vnc -Dvnc_sasl=$vnc_sasl -Dvnc_jpeg=$vnc_jpeg -Dvnc_png=$vnc_png \ > - -Dgettext=$gettext -Dxkbcommon=$xkbcommon -Du2f=$u2f\ > + -Dgettext=$gettext -Dxkbcommon=$xkbcommon -Du2f=$u2f \ > + -Dcapstone=$capstone \ > $cross_arg \ > "$PWD" "$source_path" > > diff --git a/Makefile b/Makefile > index 7c60b9dcb8..f3da1760ad 100644 > --- a/Makefile > +++ b/Makefile > @@ -156,22 +156,6 @@ dtc/all: .git-submodule-status dtc/libfdt > dtc/%: .git-submodule-status > @mkdir -p $@ > > -# Overriding CFLAGS causes us to lose defines added in the sub-makefile. > -# Not overriding CFLAGS leads to mis-matches between compilation modes. > -# Therefore we replicate some of the logic in the sub-makefile. > -# Remove all the extra -Warning flags that QEMU uses that Capstone doesn't; > -# no need to annoy QEMU developers with such things. > -CAP_CFLAGS = $(patsubst -W%,,$(CFLAGS) $(QEMU_CFLAGS)) $(CAPSTONE_CFLAGS) > -CAP_CFLAGS += -DCAPSTONE_USE_SYS_DYN_MEM > -CAP_CFLAGS += -DCAPSTONE_HAS_ARM > -CAP_CFLAGS += -DCAPSTONE_HAS_ARM64 > -CAP_CFLAGS += -DCAPSTONE_HAS_POWERPC > -CAP_CFLAGS += -DCAPSTONE_HAS_X86 > - > -.PHONY: capstone/all > -capstone/all: .git-submodule-status > - $(call quiet-command,$(MAKE) -C $(SRC_PATH)/capstone CAPSTONE_SHARED=no BUILDDIR="$(BUILD_DIR)/capstone" CC="$(CC)" AR="$(AR)" LD="$(LD)" RANLIB="$(RANLIB)" CFLAGS="$(CAP_CFLAGS)" $(SUBDIR_MAKEFLAGS) $(BUILD_DIR)/capstone/$(LIBCAPSTONE)) > - > .PHONY: slirp/all > slirp/all: .git-submodule-status > $(call quiet-command,$(MAKE) -C $(SRC_PATH)/slirp \ > diff --git a/meson.build b/meson.build > index f4d1ab1096..f23273693d 100644 > --- a/meson.build > +++ b/meson.build > @@ -10,6 +10,7 @@ else > keyval = import('unstable-keyval') > endif > ss = import('sourceset') > +fs = import('fs') > > sh = find_program('sh') > cc = meson.get_compiler('c') > @@ -409,11 +410,6 @@ if 'CONFIG_USB_LIBUSB' in config_host > libusb = declare_dependency(compile_args: config_host['LIBUSB_CFLAGS'].split(), > link_args: config_host['LIBUSB_LIBS'].split()) > endif > -capstone = not_found > -if 'CONFIG_CAPSTONE' in config_host > - capstone = declare_dependency(compile_args: config_host['CAPSTONE_CFLAGS'].split(), > - link_args: config_host['CAPSTONE_LIBS'].split()) > -endif > libpmem = not_found > if 'CONFIG_LIBPMEM' in config_host > libpmem = declare_dependency(compile_args: config_host['LIBPMEM_CFLAGS'].split(), > @@ -470,7 +466,6 @@ foreach k, v: config_host > config_host_data.set(k, v == 'y' ? 1 : v) > endif > endforeach > -genh += configure_file(output: 'config-host.h', configuration: config_host_data) > > minikconf = find_program('scripts/minikconf.py') > config_all_devices = {} > @@ -610,6 +605,108 @@ config_all += { > 'CONFIG_ALL': true, > } > > +# Submodules > + > +capstone = not_found > +capstone_opt = get_option('capstone') > +if capstone_opt == 'no' > + capstone_opt = false > +elif capstone_opt in ['yes', 'auto', 'system'] > + have_internal = fs.exists('capstone/Makefile') > + capstone = dependency('capstone', static: enable_static, > + required: capstone_opt == 'system' or > + capstone_opt == 'yes' and not have_internal) > + if capstone.found() > + capstone_opt = 'system' > + elif have_internal > + capstone_opt = 'internal' > + else > + capstone_opt = false > + endif > +endif > +if capstone_opt == 'internal' > + capstone_data = configuration_data() > + capstone_data.set('CAPSTONE_USE_SYS_DYN_MEM', '1') > + > + capstone_files = files( > + 'capstone/cs.c', > + 'capstone/MCInst.c', > + 'capstone/MCInstrDesc.c', > + 'capstone/MCRegisterInfo.c', > + 'capstone/SStream.c', > + 'capstone/utils.c' > + ) > + > + if 'CONFIG_ARM_DIS' in config_all_disas > + capstone_data.set('CAPSTONE_HAS_ARM', '1') > + capstone_files += files( > + 'capstone/arch/ARM/ARMDisassembler.c', > + 'capstone/arch/ARM/ARMInstPrinter.c', > + 'capstone/arch/ARM/ARMMapping.c', > + 'capstone/arch/ARM/ARMModule.c' > + ) > + endif > + > + # FIXME: This config entry currently depends on a c++ compiler. > + # Which is needed for building libvixl, but not for capstone. > + if 'CONFIG_ARM_A64_DIS' in config_all_disas > + capstone_data.set('CAPSTONE_HAS_ARM64', '1') > + capstone_files += files( > + 'capstone/arch/AArch64/AArch64BaseInfo.c', > + 'capstone/arch/AArch64/AArch64Disassembler.c', > + 'capstone/arch/AArch64/AArch64InstPrinter.c', > + 'capstone/arch/AArch64/AArch64Mapping.c', > + 'capstone/arch/AArch64/AArch64Module.c' > + ) > + endif > + > + if 'CONFIG_PPC_DIS' in config_all_disas > + capstone_data.set('CAPSTONE_HAS_POWERPC', '1') > + capstone_files += files( > + 'capstone/arch/PowerPC/PPCDisassembler.c', > + 'capstone/arch/PowerPC/PPCInstPrinter.c', > + 'capstone/arch/PowerPC/PPCMapping.c', > + 'capstone/arch/PowerPC/PPCModule.c' > + ) > + endif > + > + if 'CONFIG_I386_DIS' in config_all_disas > + capstone_data.set('CAPSTONE_HAS_X86', 1) > + capstone_files += files( > + 'capstone/arch/X86/X86Disassembler.c', > + 'capstone/arch/X86/X86DisassemblerDecoder.c', > + 'capstone/arch/X86/X86ATTInstPrinter.c', > + 'capstone/arch/X86/X86IntelInstPrinter.c', > + 'capstone/arch/X86/X86Mapping.c', > + 'capstone/arch/X86/X86Module.c' > + ) > + endif > + > + configure_file(output: 'capstone-defs.h', configuration: capstone_data) > + > + capstone_cargs = [ > + # FIXME: There does not seem to be a way to completely replace the c_args > + # that come from add_project_arguments() -- we can only add to them. > + # So: disable all warnings with a big hammer. > + '-Wno-error', '-w', > + > + # Include all configuration defines via a header file, which will wind up > + # as a dependency on the object file, and thus changes here will result > + # in a rebuild. > + '-include', 'capstone-defs.h' > + ] > + > + libcapstone = static_library('capstone', > + sources: capstone_files, > + c_args: capstone_cargs, > + include_directories: 'capstone/include') > + capstone = declare_dependency(link_with: libcapstone, > + include_directories: 'capstone/include') > +endif > +config_host_data.set('CONFIG_CAPSTONE', capstone.found()) > + > +genh += configure_file(output: 'config-host.h', configuration: config_host_data) > + > # Generators > > hxtool = find_program('scripts/hxtool') > @@ -1512,7 +1609,7 @@ summary_info += {'vvfat support': config_host.has_key('CONFIG_VVFAT')} > summary_info += {'qed support': config_host.has_key('CONFIG_QED')} > summary_info += {'parallels support': config_host.has_key('CONFIG_PARALLELS')} > summary_info += {'sheepdog support': config_host.has_key('CONFIG_SHEEPDOG')} > -summary_info += {'capstone': config_host.has_key('CONFIG_CAPSTONE')} > +summary_info += {'capstone': capstone_opt} > summary_info += {'libpmem support': config_host.has_key('CONFIG_LIBPMEM')} > summary_info += {'libdaxctl support': config_host.has_key('CONFIG_LIBDAXCTL')} > summary_info += {'libudev': config_host.has_key('CONFIG_LIBUDEV')} > diff --git a/meson_options.txt b/meson_options.txt > index 543cf70043..e650a937e7 100644 > --- a/meson_options.txt > +++ b/meson_options.txt > @@ -22,3 +22,7 @@ option('vnc_sasl', type : 'feature', value : 'auto', > description: 'SASL authentication for VNC server') > option('xkbcommon', type : 'feature', value : 'auto', > description: 'xkbcommon support') > + > +option('capstone', type: 'combo', value: 'auto', > + choices: ['no', 'yes', 'auto', 'system', 'internal'], > + description: 'Whether and how to find the capstone library') -- Alex Bennée
Alex Bennée <alex.bennee@linaro.org> writes: > Richard Henderson <richard.henderson@linaro.org> writes: > >> There are better ways to do this, e.g. meson cmake subproject, >> but that requires cmake 3.7 and some of our CI environments >> only provide cmake 3.5. >> >> Nor can we add a meson.build file to capstone/, because the git >> submodule would then always report "untracked files". Fixing that >> would require creating our own branch on the qemu git mirror, at >> which point we could just as easily create a native meson subproject. >> >> Instead, build the library via the main meson.build. >> >> This improves the current state of affairs in that we will re-link >> the qemu executables against a changed libcapstone.a, which we wouldn't >> do before-hand. In addition, the use of the configuration header file >> instead of command-line -DEFINES means that we will rebuild the >> capstone objects with changes to meson.build. > > Something is breaking when switching to a branch with this on from > current master: > > Linking target qemu-hppa > /usr/bin/ld: libcommon.fa.p/disas_alpha.c.o: in function `print_insn_alpha': > /home/alex/lsrc/qemu.git/builds/all/../../disas/alpha.c:1818: undefined reference to `bfd_getl32' > collect2: error: ld returned 1 exit status > make: *** [Makefile.ninja:5965: qemu-alpha] Error 1 > make: *** Waiting for unfinished jobs.... > /usr/bin/ld: libcommon.fa.p/disas_hppa.c.o: in function `print_insn_hppa': > /home/alex/lsrc/qemu.git/builds/all/../../disas/hppa.c:1969: undefined reference to `bfd_getb32' > collect2: error: ld returned 1 exit status > make: *** [Makefile.ninja:6259: qemu-hppa] Error 1 > > Aggressively wiping out the submodule and doing a fresh configure in a > empty build directory and I still see a failure: > > ../../disas/capstone.c:25:1: error: expected ‘=’, ‘,’, ‘;’, ‘asm’ or ‘__attribute__’ before ‘cap_skipdata_s390x_cb’ > cap_skipdata_s390x_cb(const uint8_t *code, size_t code_size, > ^~~~~~~~~~~~~~~~~~~~~ > ../../disas/capstone.c:49:17: error: ‘cap_skipdata_s390x_cb’ undeclared here (not in a function); did you mean ‘cap_skipdata_s390x’? > .callback = cap_skipdata_s390x_cb > ^~~~~~~~~~~~~~~~~~~~~ > cap_skipdata_s390x > Makefile.ninja:1424: recipe for target 'libcommon.fa.p/disas_capstone.c.o' failed > make: *** [libcommon.fa.p/disas_capstone.c.o] Error 1 > make: *** Waiting for unfinished jobs.... > >> >> Signed-off-by: Richard Henderson <richard.henderson@linaro.org> >> --- >> Cc: Paolo Bonzini <pbonzini@redhat.com> >> v2: Further reduce probing in configure (paolo), >> Drop state 'internal' and use 'git' even when it isn't git. >> Move CONFIG_CAPSTONE to config_host_data. >> v3: Add Submodules separator; fix default in meson_options.txt. >> --- >> configure | 61 +++---------------------- >> Makefile | 16 ------- >> meson.build | 111 +++++++++++++++++++++++++++++++++++++++++++--- >> meson_options.txt | 4 ++ >> 4 files changed, 115 insertions(+), 77 deletions(-) >> >> diff --git a/configure b/configure >> index 7564479008..76636c430d 100755 >> --- a/configure >> +++ b/configure >> @@ -478,7 +478,7 @@ opengl="" >> opengl_dmabuf="no" >> cpuid_h="no" >> avx2_opt="" >> -capstone="" >> +capstone="auto" >> lzo="" >> snappy="" >> bzip2="" >> @@ -1580,7 +1580,7 @@ for opt do >> ;; >> --enable-capstone) capstone="yes" >> ;; >> - --enable-capstone=git) capstone="git" >> + --enable-capstone=git) capstone="internal" >> ;; >> --enable-capstone=system) capstone="system" >> ;; >> @@ -5128,51 +5128,11 @@ fi >> # capstone >> >> case "$capstone" in >> - "" | yes) >> - if $pkg_config capstone; then >> - capstone=system >> - elif test -e "${source_path}/.git" && test $git_update = 'yes' ; then >> - capstone=git >> - elif test -e "${source_path}/capstone/Makefile" ; then >> - capstone=internal >> - elif test -z "$capstone" ; then >> - capstone=no >> - else >> - feature_not_found "capstone" "Install capstone devel or git submodule" >> - fi >> - ;; >> - >> - system) >> - if ! $pkg_config capstone; then >> - feature_not_found "capstone" "Install capstone devel" >> - fi >> - ;; >> -esac >> - >> -case "$capstone" in >> - git | internal) >> - if test "$capstone" = git; then >> + auto | yes | internal) >> + # Simpler to always update submodule, even if not needed. >> + if test -e "${source_path}/.git" && test $git_update = 'yes' ; then >> git_submodules="${git_submodules} capstone" >> fi >> - mkdir -p capstone >> - if test "$mingw32" = "yes"; then >> - LIBCAPSTONE=capstone.lib >> - else >> - LIBCAPSTONE=libcapstone.a >> - fi >> - capstone_libs="-Lcapstone -lcapstone" >> - capstone_cflags="-I${source_path}/capstone/include" >> - ;; >> - >> - system) >> - capstone_libs="$($pkg_config --libs capstone)" >> - capstone_cflags="$($pkg_config --cflags capstone)" >> - ;; >> - >> - no) >> - ;; >> - *) >> - error_exit "Unknown state for capstone: $capstone" >> ;; >> esac >> >> @@ -7292,11 +7252,6 @@ fi >> if test "$ivshmem" = "yes" ; then >> echo "CONFIG_IVSHMEM=y" >> $config_host_mak >> fi >> -if test "$capstone" != "no" ; then >> - echo "CONFIG_CAPSTONE=y" >> $config_host_mak >> - echo "CAPSTONE_CFLAGS=$capstone_cflags" >> $config_host_mak >> - echo "CAPSTONE_LIBS=$capstone_libs" >> $config_host_mak >> -fi >> if test "$debug_mutex" = "yes" ; then >> echo "CONFIG_DEBUG_MUTEX=y" >> $config_host_mak >> fi >> @@ -7819,9 +7774,6 @@ done # for target in $targets >> if [ "$fdt" = "git" ]; then >> subdirs="$subdirs dtc" >> fi >> -if [ "$capstone" = "git" -o "$capstone" = "internal" ]; then >> - subdirs="$subdirs capstone" >> -fi >> echo "SUBDIRS=$subdirs" >> $config_host_mak >> if test -n "$LIBCAPSTONE"; then >> echo "LIBCAPSTONE=$LIBCAPSTONE" >> $config_host_mak >> @@ -8008,7 +7960,8 @@ NINJA=${ninja:-$PWD/ninjatool} $meson setup \ >> -Db_coverage=$(if test "$gcov" = yes; then echo true; else echo false; fi) \ >> -Dsdl=$sdl -Dsdl_image=$sdl_image \ >> -Dvnc=$vnc -Dvnc_sasl=$vnc_sasl -Dvnc_jpeg=$vnc_jpeg -Dvnc_png=$vnc_png \ >> - -Dgettext=$gettext -Dxkbcommon=$xkbcommon -Du2f=$u2f\ >> + -Dgettext=$gettext -Dxkbcommon=$xkbcommon -Du2f=$u2f \ >> + -Dcapstone=$capstone \ >> $cross_arg \ >> "$PWD" "$source_path" >> >> diff --git a/Makefile b/Makefile >> index 7c60b9dcb8..f3da1760ad 100644 >> --- a/Makefile >> +++ b/Makefile >> @@ -156,22 +156,6 @@ dtc/all: .git-submodule-status dtc/libfdt >> dtc/%: .git-submodule-status >> @mkdir -p $@ >> >> -# Overriding CFLAGS causes us to lose defines added in the sub-makefile. >> -# Not overriding CFLAGS leads to mis-matches between compilation modes. >> -# Therefore we replicate some of the logic in the sub-makefile. >> -# Remove all the extra -Warning flags that QEMU uses that Capstone doesn't; >> -# no need to annoy QEMU developers with such things. >> -CAP_CFLAGS = $(patsubst -W%,,$(CFLAGS) $(QEMU_CFLAGS)) $(CAPSTONE_CFLAGS) >> -CAP_CFLAGS += -DCAPSTONE_USE_SYS_DYN_MEM >> -CAP_CFLAGS += -DCAPSTONE_HAS_ARM >> -CAP_CFLAGS += -DCAPSTONE_HAS_ARM64 >> -CAP_CFLAGS += -DCAPSTONE_HAS_POWERPC >> -CAP_CFLAGS += -DCAPSTONE_HAS_X86 >> - >> -.PHONY: capstone/all >> -capstone/all: .git-submodule-status >> - $(call quiet-command,$(MAKE) -C $(SRC_PATH)/capstone CAPSTONE_SHARED=no BUILDDIR="$(BUILD_DIR)/capstone" CC="$(CC)" AR="$(AR)" LD="$(LD)" RANLIB="$(RANLIB)" CFLAGS="$(CAP_CFLAGS)" $(SUBDIR_MAKEFLAGS) $(BUILD_DIR)/capstone/$(LIBCAPSTONE)) >> - >> .PHONY: slirp/all >> slirp/all: .git-submodule-status >> $(call quiet-command,$(MAKE) -C $(SRC_PATH)/slirp \ >> diff --git a/meson.build b/meson.build >> index f4d1ab1096..f23273693d 100644 >> --- a/meson.build >> +++ b/meson.build >> @@ -10,6 +10,7 @@ else >> keyval = import('unstable-keyval') >> endif >> ss = import('sourceset') >> +fs = import('fs') >> >> sh = find_program('sh') >> cc = meson.get_compiler('c') >> @@ -409,11 +410,6 @@ if 'CONFIG_USB_LIBUSB' in config_host >> libusb = declare_dependency(compile_args: config_host['LIBUSB_CFLAGS'].split(), >> link_args: config_host['LIBUSB_LIBS'].split()) >> endif >> -capstone = not_found >> -if 'CONFIG_CAPSTONE' in config_host >> - capstone = declare_dependency(compile_args: config_host['CAPSTONE_CFLAGS'].split(), >> - link_args: config_host['CAPSTONE_LIBS'].split()) >> -endif >> libpmem = not_found >> if 'CONFIG_LIBPMEM' in config_host >> libpmem = declare_dependency(compile_args: config_host['LIBPMEM_CFLAGS'].split(), >> @@ -470,7 +466,6 @@ foreach k, v: config_host >> config_host_data.set(k, v == 'y' ? 1 : v) >> endif >> endforeach >> -genh += configure_file(output: 'config-host.h', configuration: config_host_data) >> >> minikconf = find_program('scripts/minikconf.py') >> config_all_devices = {} >> @@ -610,6 +605,108 @@ config_all += { >> 'CONFIG_ALL': true, >> } >> >> +# Submodules >> + >> +capstone = not_found >> +capstone_opt = get_option('capstone') >> +if capstone_opt == 'no' >> + capstone_opt = false >> +elif capstone_opt in ['yes', 'auto', 'system'] >> + have_internal = fs.exists('capstone/Makefile') >> + capstone = dependency('capstone', static: enable_static, >> + required: capstone_opt == 'system' or >> + capstone_opt == 'yes' and not have_internal) >> + if capstone.found() >> + capstone_opt = 'system' >> + elif have_internal >> + capstone_opt = 'internal' >> + else >> + capstone_opt = false >> + endif >> +endif >> +if capstone_opt == 'internal' >> + capstone_data = configuration_data() >> + capstone_data.set('CAPSTONE_USE_SYS_DYN_MEM', '1') >> + >> + capstone_files = files( >> + 'capstone/cs.c', >> + 'capstone/MCInst.c', >> + 'capstone/MCInstrDesc.c', >> + 'capstone/MCRegisterInfo.c', >> + 'capstone/SStream.c', >> + 'capstone/utils.c' >> + ) >> + >> + if 'CONFIG_ARM_DIS' in config_all_disas >> + capstone_data.set('CAPSTONE_HAS_ARM', '1') >> + capstone_files += files( >> + 'capstone/arch/ARM/ARMDisassembler.c', >> + 'capstone/arch/ARM/ARMInstPrinter.c', >> + 'capstone/arch/ARM/ARMMapping.c', >> + 'capstone/arch/ARM/ARMModule.c' >> + ) >> + endif >> + >> + # FIXME: This config entry currently depends on a c++ compiler. >> + # Which is needed for building libvixl, but not for capstone. >> + if 'CONFIG_ARM_A64_DIS' in config_all_disas >> + capstone_data.set('CAPSTONE_HAS_ARM64', '1') >> + capstone_files += files( >> + 'capstone/arch/AArch64/AArch64BaseInfo.c', >> + 'capstone/arch/AArch64/AArch64Disassembler.c', >> + 'capstone/arch/AArch64/AArch64InstPrinter.c', >> + 'capstone/arch/AArch64/AArch64Mapping.c', >> + 'capstone/arch/AArch64/AArch64Module.c' >> + ) >> + endif >> + >> + if 'CONFIG_PPC_DIS' in config_all_disas >> + capstone_data.set('CAPSTONE_HAS_POWERPC', '1') >> + capstone_files += files( >> + 'capstone/arch/PowerPC/PPCDisassembler.c', >> + 'capstone/arch/PowerPC/PPCInstPrinter.c', >> + 'capstone/arch/PowerPC/PPCMapping.c', >> + 'capstone/arch/PowerPC/PPCModule.c' >> + ) >> + endif >> + >> + if 'CONFIG_I386_DIS' in config_all_disas >> + capstone_data.set('CAPSTONE_HAS_X86', 1) >> + capstone_files += files( >> + 'capstone/arch/X86/X86Disassembler.c', >> + 'capstone/arch/X86/X86DisassemblerDecoder.c', >> + 'capstone/arch/X86/X86ATTInstPrinter.c', >> + 'capstone/arch/X86/X86IntelInstPrinter.c', >> + 'capstone/arch/X86/X86Mapping.c', >> + 'capstone/arch/X86/X86Module.c' >> + ) >> + endif >> + >> + configure_file(output: 'capstone-defs.h', configuration: capstone_data) >> + >> + capstone_cargs = [ >> + # FIXME: There does not seem to be a way to completely replace the c_args >> + # that come from add_project_arguments() -- we can only add to them. >> + # So: disable all warnings with a big hammer. >> + '-Wno-error', '-w', >> + >> + # Include all configuration defines via a header file, which will wind up >> + # as a dependency on the object file, and thus changes here will result >> + # in a rebuild. >> + '-include', 'capstone-defs.h' >> + ] >> + >> + libcapstone = static_library('capstone', >> + sources: capstone_files, >> + c_args: capstone_cargs, >> + include_directories: 'capstone/include') >> + capstone = declare_dependency(link_with: libcapstone, >> + include_directories: 'capstone/include') >> +endif >> +config_host_data.set('CONFIG_CAPSTONE', capstone.found()) >> + >> +genh += configure_file(output: 'config-host.h', configuration: config_host_data) >> + >> # Generators >> >> hxtool = find_program('scripts/hxtool') >> @@ -1512,7 +1609,7 @@ summary_info += {'vvfat support': config_host.has_key('CONFIG_VVFAT')} >> summary_info += {'qed support': config_host.has_key('CONFIG_QED')} >> summary_info += {'parallels support': config_host.has_key('CONFIG_PARALLELS')} >> summary_info += {'sheepdog support': config_host.has_key('CONFIG_SHEEPDOG')} >> -summary_info += {'capstone': config_host.has_key('CONFIG_CAPSTONE')} >> +summary_info += {'capstone': capstone_opt} >> summary_info += {'libpmem support': config_host.has_key('CONFIG_LIBPMEM')} >> summary_info += {'libdaxctl support': config_host.has_key('CONFIG_LIBDAXCTL')} >> summary_info += {'libudev': config_host.has_key('CONFIG_LIBUDEV')} >> diff --git a/meson_options.txt b/meson_options.txt >> index 543cf70043..e650a937e7 100644 >> --- a/meson_options.txt >> +++ b/meson_options.txt >> @@ -22,3 +22,7 @@ option('vnc_sasl', type : 'feature', value : 'auto', >> description: 'SASL authentication for VNC server') >> option('xkbcommon', type : 'feature', value : 'auto', >> description: 'xkbcommon support') >> + >> +option('capstone', type: 'combo', value: 'auto', >> + choices: ['no', 'yes', 'auto', 'system', 'internal'], >> + description: 'Whether and how to find the capstone library') Hmm even more confusing is configure does: GIT submodules: ui/keycodemapdb tests/fp/berkeley-testfloat-3 tests/fp/berkeley-softfloat-3 meson dtc capstone slirp but also: capstone: system which I can't quite help but feel is going to be confusing. -- Alex Bennée
On Mon, Sep 21, 2020 at 7:06 PM Alex Bennée <alex.bennee@linaro.org> wrote: > > > Alex Bennée <alex.bennee@linaro.org> writes: > > > Richard Henderson <richard.henderson@linaro.org> writes: > > > >> There are better ways to do this, e.g. meson cmake subproject, > >> but that requires cmake 3.7 and some of our CI environments > >> only provide cmake 3.5. > >> > >> Nor can we add a meson.build file to capstone/, because the git > >> submodule would then always report "untracked files". Fixing that > >> would require creating our own branch on the qemu git mirror, at > >> which point we could just as easily create a native meson subproject. > >> > >> Instead, build the library via the main meson.build. > >> > >> This improves the current state of affairs in that we will re-link > >> the qemu executables against a changed libcapstone.a, which we wouldn't > >> do before-hand. In addition, the use of the configuration header file > >> instead of command-line -DEFINES means that we will rebuild the > >> capstone objects with changes to meson.build. > > > > Something is breaking when switching to a branch with this on from > > current master: > > > > Linking target qemu-hppa > > /usr/bin/ld: libcommon.fa.p/disas_alpha.c.o: in function `print_insn_alpha': > > /home/alex/lsrc/qemu.git/builds/all/../../disas/alpha.c:1818: undefined reference to `bfd_getl32' > > collect2: error: ld returned 1 exit status > > make: *** [Makefile.ninja:5965: qemu-alpha] Error 1 > > make: *** Waiting for unfinished jobs.... > > /usr/bin/ld: libcommon.fa.p/disas_hppa.c.o: in function `print_insn_hppa': > > /home/alex/lsrc/qemu.git/builds/all/../../disas/hppa.c:1969: undefined reference to `bfd_getb32' > > collect2: error: ld returned 1 exit status > > make: *** [Makefile.ninja:6259: qemu-hppa] Error 1 > > > > Aggressively wiping out the submodule and doing a fresh configure in a > > empty build directory and I still see a failure: > > > > ../../disas/capstone.c:25:1: error: expected ‘=’, ‘,’, ‘;’, ‘asm’ or ‘__attribute__’ before ‘cap_skipdata_s390x_cb’ > > cap_skipdata_s390x_cb(const uint8_t *code, size_t code_size, > > ^~~~~~~~~~~~~~~~~~~~~ > > ../../disas/capstone.c:49:17: error: ‘cap_skipdata_s390x_cb’ undeclared here (not in a function); did you mean ‘cap_skipdata_s390x’? > > .callback = cap_skipdata_s390x_cb > > ^~~~~~~~~~~~~~~~~~~~~ > > cap_skipdata_s390x > > Makefile.ninja:1424: recipe for target 'libcommon.fa.p/disas_capstone.c.o' failed > > make: *** [libcommon.fa.p/disas_capstone.c.o] Error 1 > > make: *** Waiting for unfinished jobs.... > > > >> > >> Signed-off-by: Richard Henderson <richard.henderson@linaro.org> > >> --- > >> Cc: Paolo Bonzini <pbonzini@redhat.com> > >> v2: Further reduce probing in configure (paolo), > >> Drop state 'internal' and use 'git' even when it isn't git. > >> Move CONFIG_CAPSTONE to config_host_data. > >> v3: Add Submodules separator; fix default in meson_options.txt. > >> --- > >> configure | 61 +++---------------------- > >> Makefile | 16 ------- > >> meson.build | 111 +++++++++++++++++++++++++++++++++++++++++++--- > >> meson_options.txt | 4 ++ > >> 4 files changed, 115 insertions(+), 77 deletions(-) > >> > >> diff --git a/configure b/configure > >> index 7564479008..76636c430d 100755 > >> --- a/configure > >> +++ b/configure > >> @@ -478,7 +478,7 @@ opengl="" > >> opengl_dmabuf="no" > >> cpuid_h="no" > >> avx2_opt="" > >> -capstone="" > >> +capstone="auto" > >> lzo="" > >> snappy="" > >> bzip2="" > >> @@ -1580,7 +1580,7 @@ for opt do > >> ;; > >> --enable-capstone) capstone="yes" > >> ;; > >> - --enable-capstone=git) capstone="git" > >> + --enable-capstone=git) capstone="internal" > >> ;; > >> --enable-capstone=system) capstone="system" > >> ;; > >> @@ -5128,51 +5128,11 @@ fi > >> # capstone > >> > >> case "$capstone" in > >> - "" | yes) > >> - if $pkg_config capstone; then > >> - capstone=system > >> - elif test -e "${source_path}/.git" && test $git_update = 'yes' ; then > >> - capstone=git > >> - elif test -e "${source_path}/capstone/Makefile" ; then > >> - capstone=internal > >> - elif test -z "$capstone" ; then > >> - capstone=no > >> - else > >> - feature_not_found "capstone" "Install capstone devel or git submodule" > >> - fi > >> - ;; > >> - > >> - system) > >> - if ! $pkg_config capstone; then > >> - feature_not_found "capstone" "Install capstone devel" > >> - fi > >> - ;; > >> -esac > >> - > >> -case "$capstone" in > >> - git | internal) > >> - if test "$capstone" = git; then > >> + auto | yes | internal) > >> + # Simpler to always update submodule, even if not needed. > >> + if test -e "${source_path}/.git" && test $git_update = 'yes' ; then > >> git_submodules="${git_submodules} capstone" > >> fi > >> - mkdir -p capstone > >> - if test "$mingw32" = "yes"; then > >> - LIBCAPSTONE=capstone.lib > >> - else > >> - LIBCAPSTONE=libcapstone.a > >> - fi > >> - capstone_libs="-Lcapstone -lcapstone" > >> - capstone_cflags="-I${source_path}/capstone/include" > >> - ;; > >> - > >> - system) > >> - capstone_libs="$($pkg_config --libs capstone)" > >> - capstone_cflags="$($pkg_config --cflags capstone)" > >> - ;; > >> - > >> - no) > >> - ;; > >> - *) > >> - error_exit "Unknown state for capstone: $capstone" > >> ;; > >> esac > >> > >> @@ -7292,11 +7252,6 @@ fi > >> if test "$ivshmem" = "yes" ; then > >> echo "CONFIG_IVSHMEM=y" >> $config_host_mak > >> fi > >> -if test "$capstone" != "no" ; then > >> - echo "CONFIG_CAPSTONE=y" >> $config_host_mak > >> - echo "CAPSTONE_CFLAGS=$capstone_cflags" >> $config_host_mak > >> - echo "CAPSTONE_LIBS=$capstone_libs" >> $config_host_mak > >> -fi > >> if test "$debug_mutex" = "yes" ; then > >> echo "CONFIG_DEBUG_MUTEX=y" >> $config_host_mak > >> fi > >> @@ -7819,9 +7774,6 @@ done # for target in $targets > >> if [ "$fdt" = "git" ]; then > >> subdirs="$subdirs dtc" > >> fi > >> -if [ "$capstone" = "git" -o "$capstone" = "internal" ]; then > >> - subdirs="$subdirs capstone" > >> -fi > >> echo "SUBDIRS=$subdirs" >> $config_host_mak > >> if test -n "$LIBCAPSTONE"; then > >> echo "LIBCAPSTONE=$LIBCAPSTONE" >> $config_host_mak > >> @@ -8008,7 +7960,8 @@ NINJA=${ninja:-$PWD/ninjatool} $meson setup \ > >> -Db_coverage=$(if test "$gcov" = yes; then echo true; else echo false; fi) \ > >> -Dsdl=$sdl -Dsdl_image=$sdl_image \ > >> -Dvnc=$vnc -Dvnc_sasl=$vnc_sasl -Dvnc_jpeg=$vnc_jpeg -Dvnc_png=$vnc_png \ > >> - -Dgettext=$gettext -Dxkbcommon=$xkbcommon -Du2f=$u2f\ > >> + -Dgettext=$gettext -Dxkbcommon=$xkbcommon -Du2f=$u2f \ > >> + -Dcapstone=$capstone \ > >> $cross_arg \ > >> "$PWD" "$source_path" > >> > >> diff --git a/Makefile b/Makefile > >> index 7c60b9dcb8..f3da1760ad 100644 > >> --- a/Makefile > >> +++ b/Makefile > >> @@ -156,22 +156,6 @@ dtc/all: .git-submodule-status dtc/libfdt > >> dtc/%: .git-submodule-status > >> @mkdir -p $@ > >> > >> -# Overriding CFLAGS causes us to lose defines added in the sub-makefile. > >> -# Not overriding CFLAGS leads to mis-matches between compilation modes. > >> -# Therefore we replicate some of the logic in the sub-makefile. > >> -# Remove all the extra -Warning flags that QEMU uses that Capstone doesn't; > >> -# no need to annoy QEMU developers with such things. > >> -CAP_CFLAGS = $(patsubst -W%,,$(CFLAGS) $(QEMU_CFLAGS)) $(CAPSTONE_CFLAGS) > >> -CAP_CFLAGS += -DCAPSTONE_USE_SYS_DYN_MEM > >> -CAP_CFLAGS += -DCAPSTONE_HAS_ARM > >> -CAP_CFLAGS += -DCAPSTONE_HAS_ARM64 > >> -CAP_CFLAGS += -DCAPSTONE_HAS_POWERPC > >> -CAP_CFLAGS += -DCAPSTONE_HAS_X86 > >> - > >> -.PHONY: capstone/all > >> -capstone/all: .git-submodule-status > >> - $(call quiet-command,$(MAKE) -C $(SRC_PATH)/capstone CAPSTONE_SHARED=no BUILDDIR="$(BUILD_DIR)/capstone" CC="$(CC)" AR="$(AR)" LD="$(LD)" RANLIB="$(RANLIB)" CFLAGS="$(CAP_CFLAGS)" $(SUBDIR_MAKEFLAGS) $(BUILD_DIR)/capstone/$(LIBCAPSTONE)) > >> - > >> .PHONY: slirp/all > >> slirp/all: .git-submodule-status > >> $(call quiet-command,$(MAKE) -C $(SRC_PATH)/slirp \ > >> diff --git a/meson.build b/meson.build > >> index f4d1ab1096..f23273693d 100644 > >> --- a/meson.build > >> +++ b/meson.build > >> @@ -10,6 +10,7 @@ else > >> keyval = import('unstable-keyval') > >> endif > >> ss = import('sourceset') > >> +fs = import('fs') > >> > >> sh = find_program('sh') > >> cc = meson.get_compiler('c') > >> @@ -409,11 +410,6 @@ if 'CONFIG_USB_LIBUSB' in config_host > >> libusb = declare_dependency(compile_args: config_host['LIBUSB_CFLAGS'].split(), > >> link_args: config_host['LIBUSB_LIBS'].split()) > >> endif > >> -capstone = not_found > >> -if 'CONFIG_CAPSTONE' in config_host > >> - capstone = declare_dependency(compile_args: config_host['CAPSTONE_CFLAGS'].split(), > >> - link_args: config_host['CAPSTONE_LIBS'].split()) > >> -endif > >> libpmem = not_found > >> if 'CONFIG_LIBPMEM' in config_host > >> libpmem = declare_dependency(compile_args: config_host['LIBPMEM_CFLAGS'].split(), > >> @@ -470,7 +466,6 @@ foreach k, v: config_host > >> config_host_data.set(k, v == 'y' ? 1 : v) > >> endif > >> endforeach > >> -genh += configure_file(output: 'config-host.h', configuration: config_host_data) > >> > >> minikconf = find_program('scripts/minikconf.py') > >> config_all_devices = {} > >> @@ -610,6 +605,108 @@ config_all += { > >> 'CONFIG_ALL': true, > >> } > >> > >> +# Submodules > >> + > >> +capstone = not_found > >> +capstone_opt = get_option('capstone') > >> +if capstone_opt == 'no' > >> + capstone_opt = false > >> +elif capstone_opt in ['yes', 'auto', 'system'] > >> + have_internal = fs.exists('capstone/Makefile') > >> + capstone = dependency('capstone', static: enable_static, > >> + required: capstone_opt == 'system' or > >> + capstone_opt == 'yes' and not have_internal) > >> + if capstone.found() > >> + capstone_opt = 'system' > >> + elif have_internal > >> + capstone_opt = 'internal' > >> + else > >> + capstone_opt = false > >> + endif > >> +endif > >> +if capstone_opt == 'internal' > >> + capstone_data = configuration_data() > >> + capstone_data.set('CAPSTONE_USE_SYS_DYN_MEM', '1') > >> + > >> + capstone_files = files( > >> + 'capstone/cs.c', > >> + 'capstone/MCInst.c', > >> + 'capstone/MCInstrDesc.c', > >> + 'capstone/MCRegisterInfo.c', > >> + 'capstone/SStream.c', > >> + 'capstone/utils.c' > >> + ) > >> + > >> + if 'CONFIG_ARM_DIS' in config_all_disas > >> + capstone_data.set('CAPSTONE_HAS_ARM', '1') > >> + capstone_files += files( > >> + 'capstone/arch/ARM/ARMDisassembler.c', > >> + 'capstone/arch/ARM/ARMInstPrinter.c', > >> + 'capstone/arch/ARM/ARMMapping.c', > >> + 'capstone/arch/ARM/ARMModule.c' > >> + ) > >> + endif > >> + > >> + # FIXME: This config entry currently depends on a c++ compiler. > >> + # Which is needed for building libvixl, but not for capstone. > >> + if 'CONFIG_ARM_A64_DIS' in config_all_disas > >> + capstone_data.set('CAPSTONE_HAS_ARM64', '1') > >> + capstone_files += files( > >> + 'capstone/arch/AArch64/AArch64BaseInfo.c', > >> + 'capstone/arch/AArch64/AArch64Disassembler.c', > >> + 'capstone/arch/AArch64/AArch64InstPrinter.c', > >> + 'capstone/arch/AArch64/AArch64Mapping.c', > >> + 'capstone/arch/AArch64/AArch64Module.c' > >> + ) > >> + endif > >> + > >> + if 'CONFIG_PPC_DIS' in config_all_disas > >> + capstone_data.set('CAPSTONE_HAS_POWERPC', '1') > >> + capstone_files += files( > >> + 'capstone/arch/PowerPC/PPCDisassembler.c', > >> + 'capstone/arch/PowerPC/PPCInstPrinter.c', > >> + 'capstone/arch/PowerPC/PPCMapping.c', > >> + 'capstone/arch/PowerPC/PPCModule.c' > >> + ) > >> + endif > >> + > >> + if 'CONFIG_I386_DIS' in config_all_disas > >> + capstone_data.set('CAPSTONE_HAS_X86', 1) > >> + capstone_files += files( > >> + 'capstone/arch/X86/X86Disassembler.c', > >> + 'capstone/arch/X86/X86DisassemblerDecoder.c', > >> + 'capstone/arch/X86/X86ATTInstPrinter.c', > >> + 'capstone/arch/X86/X86IntelInstPrinter.c', > >> + 'capstone/arch/X86/X86Mapping.c', > >> + 'capstone/arch/X86/X86Module.c' > >> + ) > >> + endif > >> + > >> + configure_file(output: 'capstone-defs.h', configuration: capstone_data) > >> + > >> + capstone_cargs = [ > >> + # FIXME: There does not seem to be a way to completely replace the c_args > >> + # that come from add_project_arguments() -- we can only add to them. > >> + # So: disable all warnings with a big hammer. > >> + '-Wno-error', '-w', > >> + > >> + # Include all configuration defines via a header file, which will wind up > >> + # as a dependency on the object file, and thus changes here will result > >> + # in a rebuild. > >> + '-include', 'capstone-defs.h' > >> + ] > >> + > >> + libcapstone = static_library('capstone', > >> + sources: capstone_files, > >> + c_args: capstone_cargs, > >> + include_directories: 'capstone/include') > >> + capstone = declare_dependency(link_with: libcapstone, > >> + include_directories: 'capstone/include') > >> +endif > >> +config_host_data.set('CONFIG_CAPSTONE', capstone.found()) > >> + > >> +genh += configure_file(output: 'config-host.h', configuration: config_host_data) > >> + > >> # Generators > >> > >> hxtool = find_program('scripts/hxtool') > >> @@ -1512,7 +1609,7 @@ summary_info += {'vvfat support': config_host.has_key('CONFIG_VVFAT')} > >> summary_info += {'qed support': config_host.has_key('CONFIG_QED')} > >> summary_info += {'parallels support': config_host.has_key('CONFIG_PARALLELS')} > >> summary_info += {'sheepdog support': config_host.has_key('CONFIG_SHEEPDOG')} > >> -summary_info += {'capstone': config_host.has_key('CONFIG_CAPSTONE')} > >> +summary_info += {'capstone': capstone_opt} > >> summary_info += {'libpmem support': config_host.has_key('CONFIG_LIBPMEM')} > >> summary_info += {'libdaxctl support': config_host.has_key('CONFIG_LIBDAXCTL')} > >> summary_info += {'libudev': config_host.has_key('CONFIG_LIBUDEV')} > >> diff --git a/meson_options.txt b/meson_options.txt > >> index 543cf70043..e650a937e7 100644 > >> --- a/meson_options.txt > >> +++ b/meson_options.txt > >> @@ -22,3 +22,7 @@ option('vnc_sasl', type : 'feature', value : 'auto', > >> description: 'SASL authentication for VNC server') > >> option('xkbcommon', type : 'feature', value : 'auto', > >> description: 'xkbcommon support') > >> + > >> +option('capstone', type: 'combo', value: 'auto', > >> + choices: ['no', 'yes', 'auto', 'system', 'internal'], > >> + description: 'Whether and how to find the capstone library') > > > Hmm even more confusing is configure does: > > GIT submodules: ui/keycodemapdb tests/fp/berkeley-testfloat-3 tests/fp/berkeley-softfloat-3 meson dtc capstone slirp > > but also: > > capstone: system > > which I can't quite help but feel is going to be confusing. maybe remove the capstone system support is a good idea after-all. > > -- > Alex Bennée > -- 此致 礼 罗勇刚 Yours sincerely, Yonggang Luo <div dir="ltr"><br><br>On Mon, Sep 21, 2020 at 7:06 PM Alex Bennée <<a href="mailto:alex.bennee@linaro.org">alex.bennee@linaro.org</a>> wrote:<br>><br>><br>> Alex Bennée <<a href="mailto:alex.bennee@linaro.org">alex.bennee@linaro.org</a>> writes:<br>><br>> > Richard Henderson <<a href="mailto:richard.henderson@linaro.org">richard.henderson@linaro.org</a>> writes:<br>> ><br>> >> There are better ways to do this, e.g. meson cmake subproject,<br>> >> but that requires cmake 3.7 and some of our CI environments<br>> >> only provide cmake 3.5.<br>> >><br>> >> Nor can we add a meson.build file to capstone/, because the git<br>> >> submodule would then always report "untracked files". Fixing that<br>> >> would require creating our own branch on the qemu git mirror, at<br>> >> which point we could just as easily create a native meson subproject.<br>> >><br>> >> Instead, build the library via the main meson.build.<br>> >><br>> >> This improves the current state of affairs in that we will re-link<br>> >> the qemu executables against a changed libcapstone.a, which we wouldn't<br>> >> do before-hand. In addition, the use of the configuration header file<br>> >> instead of command-line -DEFINES means that we will rebuild the<br>> >> capstone objects with changes to meson.build.<br>> ><br>> > Something is breaking when switching to a branch with this on from<br>> > current master:<br>> ><br>> > Linking target qemu-hppa<br>> > /usr/bin/ld: libcommon.fa.p/disas_alpha.c.o: in function `print_insn_alpha':<br>> > /home/alex/lsrc/qemu.git/builds/all/../../disas/alpha.c:1818: undefined reference to `bfd_getl32'<br>> > collect2: error: ld returned 1 exit status<br>> > make: *** [Makefile.ninja:5965: qemu-alpha] Error 1<br>> > make: *** Waiting for unfinished jobs....<br>> > /usr/bin/ld: libcommon.fa.p/disas_hppa.c.o: in function `print_insn_hppa':<br>> > /home/alex/lsrc/qemu.git/builds/all/../../disas/hppa.c:1969: undefined reference to `bfd_getb32'<br>> > collect2: error: ld returned 1 exit status<br>> > make: *** [Makefile.ninja:6259: qemu-hppa] Error 1<br>> ><br>> > Aggressively wiping out the submodule and doing a fresh configure in a<br>> > empty build directory and I still see a failure:<br>> ><br>> > ../../disas/capstone.c:25:1: error: expected ‘=’, ‘,’, ‘;’, ‘asm’ or ‘__attribute__’ before ‘cap_skipdata_s390x_cb’<br>> > cap_skipdata_s390x_cb(const uint8_t *code, size_t code_size,<br>> > ^~~~~~~~~~~~~~~~~~~~~<br>> > ../../disas/capstone.c:49:17: error: ‘cap_skipdata_s390x_cb’ undeclared here (not in a function); did you mean ‘cap_skipdata_s390x’?<br>> > .callback = cap_skipdata_s390x_cb<br>> > ^~~~~~~~~~~~~~~~~~~~~<br>> > cap_skipdata_s390x<br>> > Makefile.ninja:1424: recipe for target 'libcommon.fa.p/disas_capstone.c.o' failed<br>> > make: *** [libcommon.fa.p/disas_capstone.c.o] Error 1<br>> > make: *** Waiting for unfinished jobs....<br>> ><br>> >><br>> >> Signed-off-by: Richard Henderson <<a href="mailto:richard.henderson@linaro.org">richard.henderson@linaro.org</a>><br>> >> ---<br>> >> Cc: Paolo Bonzini <<a href="mailto:pbonzini@redhat.com">pbonzini@redhat.com</a>><br>> >> v2: Further reduce probing in configure (paolo),<br>> >> Drop state 'internal' and use 'git' even when it isn't git.<br>> >> Move CONFIG_CAPSTONE to config_host_data.<br>> >> v3: Add Submodules separator; fix default in meson_options.txt.<br>> >> ---<br>> >> configure | 61 +++----------------------<br>> >> Makefile | 16 -------<br>> >> meson.build | 111 +++++++++++++++++++++++++++++++++++++++++++---<br>> >> meson_options.txt | 4 ++<br>> >> 4 files changed, 115 insertions(+), 77 deletions(-)<br>> >><br>> >> diff --git a/configure b/configure<br>> >> index 7564479008..76636c430d 100755<br>> >> --- a/configure<br>> >> +++ b/configure<br>> >> @@ -478,7 +478,7 @@ opengl=""<br>> >> opengl_dmabuf="no"<br>> >> cpuid_h="no"<br>> >> avx2_opt=""<br>> >> -capstone=""<br>> >> +capstone="auto"<br>> >> lzo=""<br>> >> snappy=""<br>> >> bzip2=""<br>> >> @@ -1580,7 +1580,7 @@ for opt do<br>> >> ;;<br>> >> --enable-capstone) capstone="yes"<br>> >> ;;<br>> >> - --enable-capstone=git) capstone="git"<br>> >> + --enable-capstone=git) capstone="internal"<br>> >> ;;<br>> >> --enable-capstone=system) capstone="system"<br>> >> ;;<br>> >> @@ -5128,51 +5128,11 @@ fi<br>> >> # capstone<br>> >> <br>> >> case "$capstone" in<br>> >> - "" | yes)<br>> >> - if $pkg_config capstone; then<br>> >> - capstone=system<br>> >> - elif test -e "${source_path}/.git" && test $git_update = 'yes' ; then<br>> >> - capstone=git<br>> >> - elif test -e "${source_path}/capstone/Makefile" ; then<br>> >> - capstone=internal<br>> >> - elif test -z "$capstone" ; then<br>> >> - capstone=no<br>> >> - else<br>> >> - feature_not_found "capstone" "Install capstone devel or git submodule"<br>> >> - fi<br>> >> - ;;<br>> >> -<br>> >> - system)<br>> >> - if ! $pkg_config capstone; then<br>> >> - feature_not_found "capstone" "Install capstone devel"<br>> >> - fi<br>> >> - ;;<br>> >> -esac<br>> >> -<br>> >> -case "$capstone" in<br>> >> - git | internal)<br>> >> - if test "$capstone" = git; then<br>> >> + auto | yes | internal)<br>> >> + # Simpler to always update submodule, even if not needed.<br>> >> + if test -e "${source_path}/.git" && test $git_update = 'yes' ; then<br>> >> git_submodules="${git_submodules} capstone"<br>> >> fi<br>> >> - mkdir -p capstone<br>> >> - if test "$mingw32" = "yes"; then<br>> >> - LIBCAPSTONE=capstone.lib<br>> >> - else<br>> >> - LIBCAPSTONE=libcapstone.a<br>> >> - fi<br>> >> - capstone_libs="-Lcapstone -lcapstone"<br>> >> - capstone_cflags="-I${source_path}/capstone/include"<br>> >> - ;;<br>> >> -<br>> >> - system)<br>> >> - capstone_libs="$($pkg_config --libs capstone)"<br>> >> - capstone_cflags="$($pkg_config --cflags capstone)"<br>> >> - ;;<br>> >> -<br>> >> - no)<br>> >> - ;;<br>> >> - *)<br>> >> - error_exit "Unknown state for capstone: $capstone"<br>> >> ;;<br>> >> esac<br>> >> <br>> >> @@ -7292,11 +7252,6 @@ fi<br>> >> if test "$ivshmem" = "yes" ; then<br>> >> echo "CONFIG_IVSHMEM=y" >> $config_host_mak<br>> >> fi<br>> >> -if test "$capstone" != "no" ; then<br>> >> - echo "CONFIG_CAPSTONE=y" >> $config_host_mak<br>> >> - echo "CAPSTONE_CFLAGS=$capstone_cflags" >> $config_host_mak<br>> >> - echo "CAPSTONE_LIBS=$capstone_libs" >> $config_host_mak<br>> >> -fi<br>> >> if test "$debug_mutex" = "yes" ; then<br>> >> echo "CONFIG_DEBUG_MUTEX=y" >> $config_host_mak<br>> >> fi<br>> >> @@ -7819,9 +7774,6 @@ done # for target in $targets<br>> >> if [ "$fdt" = "git" ]; then<br>> >> subdirs="$subdirs dtc"<br>> >> fi<br>> >> -if [ "$capstone" = "git" -o "$capstone" = "internal" ]; then<br>> >> - subdirs="$subdirs capstone"<br>> >> -fi<br>> >> echo "SUBDIRS=$subdirs" >> $config_host_mak<br>> >> if test -n "$LIBCAPSTONE"; then<br>> >> echo "LIBCAPSTONE=$LIBCAPSTONE" >> $config_host_mak<br>> >> @@ -8008,7 +7960,8 @@ NINJA=${ninja:-$PWD/ninjatool} $meson setup \<br>> >> -Db_coverage=$(if test "$gcov" = yes; then echo true; else echo false; fi) \<br>> >> -Dsdl=$sdl -Dsdl_image=$sdl_image \<br>> >> -Dvnc=$vnc -Dvnc_sasl=$vnc_sasl -Dvnc_jpeg=$vnc_jpeg -Dvnc_png=$vnc_png \<br>> >> - -Dgettext=$gettext -Dxkbcommon=$xkbcommon -Du2f=$u2f\<br>> >> + -Dgettext=$gettext -Dxkbcommon=$xkbcommon -Du2f=$u2f \<br>> >> + -Dcapstone=$capstone \<br>> >> $cross_arg \<br>> >> "$PWD" "$source_path"<br>> >> <br>> >> diff --git a/Makefile b/Makefile<br>> >> index 7c60b9dcb8..f3da1760ad 100644<br>> >> --- a/Makefile<br>> >> +++ b/Makefile<br>> >> @@ -156,22 +156,6 @@ dtc/all: .git-submodule-status dtc/libfdt<br>> >> dtc/%: .git-submodule-status<br>> >> @mkdir -p $@<br>> >> <br>> >> -# Overriding CFLAGS causes us to lose defines added in the sub-makefile.<br>> >> -# Not overriding CFLAGS leads to mis-matches between compilation modes.<br>> >> -# Therefore we replicate some of the logic in the sub-makefile.<br>> >> -# Remove all the extra -Warning flags that QEMU uses that Capstone doesn't;<br>> >> -# no need to annoy QEMU developers with such things.<br>> >> -CAP_CFLAGS = $(patsubst -W%,,$(CFLAGS) $(QEMU_CFLAGS)) $(CAPSTONE_CFLAGS)<br>> >> -CAP_CFLAGS += -DCAPSTONE_USE_SYS_DYN_MEM<br>> >> -CAP_CFLAGS += -DCAPSTONE_HAS_ARM<br>> >> -CAP_CFLAGS += -DCAPSTONE_HAS_ARM64<br>> >> -CAP_CFLAGS += -DCAPSTONE_HAS_POWERPC<br>> >> -CAP_CFLAGS += -DCAPSTONE_HAS_X86<br>> >> -<br>> >> -.PHONY: capstone/all<br>> >> -capstone/all: .git-submodule-status<br>> >> - $(call quiet-command,$(MAKE) -C $(SRC_PATH)/capstone CAPSTONE_SHARED=no BUILDDIR="$(BUILD_DIR)/capstone" CC="$(CC)" AR="$(AR)" LD="$(LD)" RANLIB="$(RANLIB)" CFLAGS="$(CAP_CFLAGS)" $(SUBDIR_MAKEFLAGS) $(BUILD_DIR)/capstone/$(LIBCAPSTONE))<br>> >> -<br>> >> .PHONY: slirp/all<br>> >> slirp/all: .git-submodule-status<br>> >> $(call quiet-command,$(MAKE) -C $(SRC_PATH)/slirp \<br>> >> diff --git a/meson.build b/meson.build<br>> >> index f4d1ab1096..f23273693d 100644<br>> >> --- a/meson.build<br>> >> +++ b/meson.build<br>> >> @@ -10,6 +10,7 @@ else<br>> >> keyval = import('unstable-keyval')<br>> >> endif<br>> >> ss = import('sourceset')<br>> >> +fs = import('fs')<br>> >> <br>> >> sh = find_program('sh')<br>> >> cc = meson.get_compiler('c')<br>> >> @@ -409,11 +410,6 @@ if 'CONFIG_USB_LIBUSB' in config_host<br>> >> libusb = declare_dependency(compile_args: config_host['LIBUSB_CFLAGS'].split(),<br>> >> link_args: config_host['LIBUSB_LIBS'].split())<br>> >> endif<br>> >> -capstone = not_found<br>> >> -if 'CONFIG_CAPSTONE' in config_host<br>> >> - capstone = declare_dependency(compile_args: config_host['CAPSTONE_CFLAGS'].split(),<br>> >> - link_args: config_host['CAPSTONE_LIBS'].split())<br>> >> -endif<br>> >> libpmem = not_found<br>> >> if 'CONFIG_LIBPMEM' in config_host<br>> >> libpmem = declare_dependency(compile_args: config_host['LIBPMEM_CFLAGS'].split(),<br>> >> @@ -470,7 +466,6 @@ foreach k, v: config_host<br>> >> config_host_data.set(k, v == 'y' ? 1 : v)<br>> >> endif<br>> >> endforeach<br>> >> -genh += configure_file(output: 'config-host.h', configuration: config_host_data)<br>> >> <br>> >> minikconf = find_program('scripts/minikconf.py')<br>> >> config_all_devices = {}<br>> >> @@ -610,6 +605,108 @@ config_all += {<br>> >> 'CONFIG_ALL': true,<br>> >> }<br>> >> <br>> >> +# Submodules<br>> >> +<br>> >> +capstone = not_found<br>> >> +capstone_opt = get_option('capstone')<br>> >> +if capstone_opt == 'no'<br>> >> + capstone_opt = false<br>> >> +elif capstone_opt in ['yes', 'auto', 'system']<br>> >> + have_internal = fs.exists('capstone/Makefile')<br>> >> + capstone = dependency('capstone', static: enable_static,<br>> >> + required: capstone_opt == 'system' or<br>> >> + capstone_opt == 'yes' and not have_internal)<br>> >> + if capstone.found()<br>> >> + capstone_opt = 'system'<br>> >> + elif have_internal<br>> >> + capstone_opt = 'internal'<br>> >> + else<br>> >> + capstone_opt = false<br>> >> + endif<br>> >> +endif<br>> >> +if capstone_opt == 'internal'<br>> >> + capstone_data = configuration_data()<br>> >> + capstone_data.set('CAPSTONE_USE_SYS_DYN_MEM', '1')<br>> >> +<br>> >> + capstone_files = files(<br>> >> + 'capstone/cs.c',<br>> >> + 'capstone/MCInst.c',<br>> >> + 'capstone/MCInstrDesc.c',<br>> >> + 'capstone/MCRegisterInfo.c',<br>> >> + 'capstone/SStream.c',<br>> >> + 'capstone/utils.c'<br>> >> + )<br>> >> +<br>> >> + if 'CONFIG_ARM_DIS' in config_all_disas<br>> >> + capstone_data.set('CAPSTONE_HAS_ARM', '1')<br>> >> + capstone_files += files(<br>> >> + 'capstone/arch/ARM/ARMDisassembler.c',<br>> >> + 'capstone/arch/ARM/ARMInstPrinter.c',<br>> >> + 'capstone/arch/ARM/ARMMapping.c',<br>> >> + 'capstone/arch/ARM/ARMModule.c'<br>> >> + )<br>> >> + endif<br>> >> +<br>> >> + # FIXME: This config entry currently depends on a c++ compiler.<br>> >> + # Which is needed for building libvixl, but not for capstone.<br>> >> + if 'CONFIG_ARM_A64_DIS' in config_all_disas<br>> >> + capstone_data.set('CAPSTONE_HAS_ARM64', '1')<br>> >> + capstone_files += files(<br>> >> + 'capstone/arch/AArch64/AArch64BaseInfo.c',<br>> >> + 'capstone/arch/AArch64/AArch64Disassembler.c',<br>> >> + 'capstone/arch/AArch64/AArch64InstPrinter.c',<br>> >> + 'capstone/arch/AArch64/AArch64Mapping.c',<br>> >> + 'capstone/arch/AArch64/AArch64Module.c'<br>> >> + )<br>> >> + endif<br>> >> +<br>> >> + if 'CONFIG_PPC_DIS' in config_all_disas<br>> >> + capstone_data.set('CAPSTONE_HAS_POWERPC', '1')<br>> >> + capstone_files += files(<br>> >> + 'capstone/arch/PowerPC/PPCDisassembler.c',<br>> >> + 'capstone/arch/PowerPC/PPCInstPrinter.c',<br>> >> + 'capstone/arch/PowerPC/PPCMapping.c',<br>> >> + 'capstone/arch/PowerPC/PPCModule.c'<br>> >> + )<br>> >> + endif<br>> >> +<br>> >> + if 'CONFIG_I386_DIS' in config_all_disas<br>> >> + capstone_data.set('CAPSTONE_HAS_X86', 1)<br>> >> + capstone_files += files(<br>> >> + 'capstone/arch/X86/X86Disassembler.c',<br>> >> + 'capstone/arch/X86/X86DisassemblerDecoder.c',<br>> >> + 'capstone/arch/X86/X86ATTInstPrinter.c',<br>> >> + 'capstone/arch/X86/X86IntelInstPrinter.c',<br>> >> + 'capstone/arch/X86/X86Mapping.c',<br>> >> + 'capstone/arch/X86/X86Module.c'<br>> >> + )<br>> >> + endif<br>> >> +<br>> >> + configure_file(output: 'capstone-defs.h', configuration: capstone_data)<br>> >> +<br>> >> + capstone_cargs = [<br>> >> + # FIXME: There does not seem to be a way to completely replace the c_args<br>> >> + # that come from add_project_arguments() -- we can only add to them.<br>> >> + # So: disable all warnings with a big hammer.<br>> >> + '-Wno-error', '-w',<br>> >> +<br>> >> + # Include all configuration defines via a header file, which will wind up<br>> >> + # as a dependency on the object file, and thus changes here will result<br>> >> + # in a rebuild.<br>> >> + '-include', 'capstone-defs.h'<br>> >> + ]<br>> >> +<br>> >> + libcapstone = static_library('capstone',<br>> >> + sources: capstone_files,<br>> >> + c_args: capstone_cargs,<br>> >> + include_directories: 'capstone/include')<br>> >> + capstone = declare_dependency(link_with: libcapstone,<br>> >> + include_directories: 'capstone/include')<br>> >> +endif<br>> >> +config_host_data.set('CONFIG_CAPSTONE', capstone.found())<br>> >> +<br>> >> +genh += configure_file(output: 'config-host.h', configuration: config_host_data)<br>> >> +<br>> >> # Generators<br>> >> <br>> >> hxtool = find_program('scripts/hxtool')<br>> >> @@ -1512,7 +1609,7 @@ summary_info += {'vvfat support': config_host.has_key('CONFIG_VVFAT')}<br>> >> summary_info += {'qed support': config_host.has_key('CONFIG_QED')}<br>> >> summary_info += {'parallels support': config_host.has_key('CONFIG_PARALLELS')}<br>> >> summary_info += {'sheepdog support': config_host.has_key('CONFIG_SHEEPDOG')}<br>> >> -summary_info += {'capstone': config_host.has_key('CONFIG_CAPSTONE')}<br>> >> +summary_info += {'capstone': capstone_opt}<br>> >> summary_info += {'libpmem support': config_host.has_key('CONFIG_LIBPMEM')}<br>> >> summary_info += {'libdaxctl support': config_host.has_key('CONFIG_LIBDAXCTL')}<br>> >> summary_info += {'libudev': config_host.has_key('CONFIG_LIBUDEV')}<br>> >> diff --git a/meson_options.txt b/meson_options.txt<br>> >> index 543cf70043..e650a937e7 100644<br>> >> --- a/meson_options.txt<br>> >> +++ b/meson_options.txt<br>> >> @@ -22,3 +22,7 @@ option('vnc_sasl', type : 'feature', value : 'auto',<br>> >> description: 'SASL authentication for VNC server')<br>> >> option('xkbcommon', type : 'feature', value : 'auto',<br>> >> description: 'xkbcommon support')<br>> >> +<br>> >> +option('capstone', type: 'combo', value: 'auto',<br>> >> + choices: ['no', 'yes', 'auto', 'system', 'internal'],<br>> >> + description: 'Whether and how to find the capstone library')<br>><br>><br>> Hmm even more confusing is configure does:<br>><br>> GIT submodules: ui/keycodemapdb tests/fp/berkeley-testfloat-3 tests/fp/berkeley-softfloat-3 meson dtc capstone slirp<br>><br>> but also:<br>><br>> capstone: system<br>><br>> which I can't quite help but feel is going to be confusing.<div>maybe remove the capstone system support is a good idea after-all.<br>><br>> --<br>> Alex Bennée<br>><br><br><br>--<br> 此致<br>礼<br>罗勇刚<br>Yours<br> sincerely,<br>Yonggang Luo</div></div>
On 21/09/20 12:54, Alex Bennée wrote: > Something is breaking when switching to a branch with this on from > current master: > > Linking target qemu-hppa > /usr/bin/ld: libcommon.fa.p/disas_alpha.c.o: in function `print_insn_alpha': > /home/alex/lsrc/qemu.git/builds/all/../../disas/alpha.c:1818: undefined reference to `bfd_getl32' > collect2: error: ld returned 1 exit status > make: *** [Makefile.ninja:5965: qemu-alpha] Error 1 > make: *** Waiting for unfinished jobs.... > /usr/bin/ld: libcommon.fa.p/disas_hppa.c.o: in function `print_insn_hppa': > /home/alex/lsrc/qemu.git/builds/all/../../disas/hppa.c:1969: undefined reference to `bfd_getb32' > collect2: error: ld returned 1 exit status > make: *** [Makefile.ninja:6259: qemu-hppa] Error 1 From "https://wiki.qemu.org/Features/Meson": Known issues ============ There is one Make issue that might happen when switching branches: if the command line changes, the files are not rebuilt. This is always a problem with Make, but it may become more common now due to the use of statically-linked libraries. The simplest fix here would be to use ninja to build QEMU, i.e. do the main QEMU build recursively just like we do for submodules, keeping Make purely to orchestrate the build. Paolo
diff --git a/configure b/configure index 7564479008..76636c430d 100755 --- a/configure +++ b/configure @@ -478,7 +478,7 @@ opengl="" opengl_dmabuf="no" cpuid_h="no" avx2_opt="" -capstone="" +capstone="auto" lzo="" snappy="" bzip2="" @@ -1580,7 +1580,7 @@ for opt do ;; --enable-capstone) capstone="yes" ;; - --enable-capstone=git) capstone="git" + --enable-capstone=git) capstone="internal" ;; --enable-capstone=system) capstone="system" ;; @@ -5128,51 +5128,11 @@ fi # capstone case "$capstone" in - "" | yes) - if $pkg_config capstone; then - capstone=system - elif test -e "${source_path}/.git" && test $git_update = 'yes' ; then - capstone=git - elif test -e "${source_path}/capstone/Makefile" ; then - capstone=internal - elif test -z "$capstone" ; then - capstone=no - else - feature_not_found "capstone" "Install capstone devel or git submodule" - fi - ;; - - system) - if ! $pkg_config capstone; then - feature_not_found "capstone" "Install capstone devel" - fi - ;; -esac - -case "$capstone" in - git | internal) - if test "$capstone" = git; then + auto | yes | internal) + # Simpler to always update submodule, even if not needed. + if test -e "${source_path}/.git" && test $git_update = 'yes' ; then git_submodules="${git_submodules} capstone" fi - mkdir -p capstone - if test "$mingw32" = "yes"; then - LIBCAPSTONE=capstone.lib - else - LIBCAPSTONE=libcapstone.a - fi - capstone_libs="-Lcapstone -lcapstone" - capstone_cflags="-I${source_path}/capstone/include" - ;; - - system) - capstone_libs="$($pkg_config --libs capstone)" - capstone_cflags="$($pkg_config --cflags capstone)" - ;; - - no) - ;; - *) - error_exit "Unknown state for capstone: $capstone" ;; esac @@ -7292,11 +7252,6 @@ fi if test "$ivshmem" = "yes" ; then echo "CONFIG_IVSHMEM=y" >> $config_host_mak fi -if test "$capstone" != "no" ; then - echo "CONFIG_CAPSTONE=y" >> $config_host_mak - echo "CAPSTONE_CFLAGS=$capstone_cflags" >> $config_host_mak - echo "CAPSTONE_LIBS=$capstone_libs" >> $config_host_mak -fi if test "$debug_mutex" = "yes" ; then echo "CONFIG_DEBUG_MUTEX=y" >> $config_host_mak fi @@ -7819,9 +7774,6 @@ done # for target in $targets if [ "$fdt" = "git" ]; then subdirs="$subdirs dtc" fi -if [ "$capstone" = "git" -o "$capstone" = "internal" ]; then - subdirs="$subdirs capstone" -fi echo "SUBDIRS=$subdirs" >> $config_host_mak if test -n "$LIBCAPSTONE"; then echo "LIBCAPSTONE=$LIBCAPSTONE" >> $config_host_mak @@ -8008,7 +7960,8 @@ NINJA=${ninja:-$PWD/ninjatool} $meson setup \ -Db_coverage=$(if test "$gcov" = yes; then echo true; else echo false; fi) \ -Dsdl=$sdl -Dsdl_image=$sdl_image \ -Dvnc=$vnc -Dvnc_sasl=$vnc_sasl -Dvnc_jpeg=$vnc_jpeg -Dvnc_png=$vnc_png \ - -Dgettext=$gettext -Dxkbcommon=$xkbcommon -Du2f=$u2f\ + -Dgettext=$gettext -Dxkbcommon=$xkbcommon -Du2f=$u2f \ + -Dcapstone=$capstone \ $cross_arg \ "$PWD" "$source_path" diff --git a/Makefile b/Makefile index 7c60b9dcb8..f3da1760ad 100644 --- a/Makefile +++ b/Makefile @@ -156,22 +156,6 @@ dtc/all: .git-submodule-status dtc/libfdt dtc/%: .git-submodule-status @mkdir -p $@ -# Overriding CFLAGS causes us to lose defines added in the sub-makefile. -# Not overriding CFLAGS leads to mis-matches between compilation modes. -# Therefore we replicate some of the logic in the sub-makefile. -# Remove all the extra -Warning flags that QEMU uses that Capstone doesn't; -# no need to annoy QEMU developers with such things. -CAP_CFLAGS = $(patsubst -W%,,$(CFLAGS) $(QEMU_CFLAGS)) $(CAPSTONE_CFLAGS) -CAP_CFLAGS += -DCAPSTONE_USE_SYS_DYN_MEM -CAP_CFLAGS += -DCAPSTONE_HAS_ARM -CAP_CFLAGS += -DCAPSTONE_HAS_ARM64 -CAP_CFLAGS += -DCAPSTONE_HAS_POWERPC -CAP_CFLAGS += -DCAPSTONE_HAS_X86 - -.PHONY: capstone/all -capstone/all: .git-submodule-status - $(call quiet-command,$(MAKE) -C $(SRC_PATH)/capstone CAPSTONE_SHARED=no BUILDDIR="$(BUILD_DIR)/capstone" CC="$(CC)" AR="$(AR)" LD="$(LD)" RANLIB="$(RANLIB)" CFLAGS="$(CAP_CFLAGS)" $(SUBDIR_MAKEFLAGS) $(BUILD_DIR)/capstone/$(LIBCAPSTONE)) - .PHONY: slirp/all slirp/all: .git-submodule-status $(call quiet-command,$(MAKE) -C $(SRC_PATH)/slirp \ diff --git a/meson.build b/meson.build index f4d1ab1096..f23273693d 100644 --- a/meson.build +++ b/meson.build @@ -10,6 +10,7 @@ else keyval = import('unstable-keyval') endif ss = import('sourceset') +fs = import('fs') sh = find_program('sh') cc = meson.get_compiler('c') @@ -409,11 +410,6 @@ if 'CONFIG_USB_LIBUSB' in config_host libusb = declare_dependency(compile_args: config_host['LIBUSB_CFLAGS'].split(), link_args: config_host['LIBUSB_LIBS'].split()) endif -capstone = not_found -if 'CONFIG_CAPSTONE' in config_host - capstone = declare_dependency(compile_args: config_host['CAPSTONE_CFLAGS'].split(), - link_args: config_host['CAPSTONE_LIBS'].split()) -endif libpmem = not_found if 'CONFIG_LIBPMEM' in config_host libpmem = declare_dependency(compile_args: config_host['LIBPMEM_CFLAGS'].split(), @@ -470,7 +466,6 @@ foreach k, v: config_host config_host_data.set(k, v == 'y' ? 1 : v) endif endforeach -genh += configure_file(output: 'config-host.h', configuration: config_host_data) minikconf = find_program('scripts/minikconf.py') config_all_devices = {} @@ -610,6 +605,108 @@ config_all += { 'CONFIG_ALL': true, } +# Submodules + +capstone = not_found +capstone_opt = get_option('capstone') +if capstone_opt == 'no' + capstone_opt = false +elif capstone_opt in ['yes', 'auto', 'system'] + have_internal = fs.exists('capstone/Makefile') + capstone = dependency('capstone', static: enable_static, + required: capstone_opt == 'system' or + capstone_opt == 'yes' and not have_internal) + if capstone.found() + capstone_opt = 'system' + elif have_internal + capstone_opt = 'internal' + else + capstone_opt = false + endif +endif +if capstone_opt == 'internal' + capstone_data = configuration_data() + capstone_data.set('CAPSTONE_USE_SYS_DYN_MEM', '1') + + capstone_files = files( + 'capstone/cs.c', + 'capstone/MCInst.c', + 'capstone/MCInstrDesc.c', + 'capstone/MCRegisterInfo.c', + 'capstone/SStream.c', + 'capstone/utils.c' + ) + + if 'CONFIG_ARM_DIS' in config_all_disas + capstone_data.set('CAPSTONE_HAS_ARM', '1') + capstone_files += files( + 'capstone/arch/ARM/ARMDisassembler.c', + 'capstone/arch/ARM/ARMInstPrinter.c', + 'capstone/arch/ARM/ARMMapping.c', + 'capstone/arch/ARM/ARMModule.c' + ) + endif + + # FIXME: This config entry currently depends on a c++ compiler. + # Which is needed for building libvixl, but not for capstone. + if 'CONFIG_ARM_A64_DIS' in config_all_disas + capstone_data.set('CAPSTONE_HAS_ARM64', '1') + capstone_files += files( + 'capstone/arch/AArch64/AArch64BaseInfo.c', + 'capstone/arch/AArch64/AArch64Disassembler.c', + 'capstone/arch/AArch64/AArch64InstPrinter.c', + 'capstone/arch/AArch64/AArch64Mapping.c', + 'capstone/arch/AArch64/AArch64Module.c' + ) + endif + + if 'CONFIG_PPC_DIS' in config_all_disas + capstone_data.set('CAPSTONE_HAS_POWERPC', '1') + capstone_files += files( + 'capstone/arch/PowerPC/PPCDisassembler.c', + 'capstone/arch/PowerPC/PPCInstPrinter.c', + 'capstone/arch/PowerPC/PPCMapping.c', + 'capstone/arch/PowerPC/PPCModule.c' + ) + endif + + if 'CONFIG_I386_DIS' in config_all_disas + capstone_data.set('CAPSTONE_HAS_X86', 1) + capstone_files += files( + 'capstone/arch/X86/X86Disassembler.c', + 'capstone/arch/X86/X86DisassemblerDecoder.c', + 'capstone/arch/X86/X86ATTInstPrinter.c', + 'capstone/arch/X86/X86IntelInstPrinter.c', + 'capstone/arch/X86/X86Mapping.c', + 'capstone/arch/X86/X86Module.c' + ) + endif + + configure_file(output: 'capstone-defs.h', configuration: capstone_data) + + capstone_cargs = [ + # FIXME: There does not seem to be a way to completely replace the c_args + # that come from add_project_arguments() -- we can only add to them. + # So: disable all warnings with a big hammer. + '-Wno-error', '-w', + + # Include all configuration defines via a header file, which will wind up + # as a dependency on the object file, and thus changes here will result + # in a rebuild. + '-include', 'capstone-defs.h' + ] + + libcapstone = static_library('capstone', + sources: capstone_files, + c_args: capstone_cargs, + include_directories: 'capstone/include') + capstone = declare_dependency(link_with: libcapstone, + include_directories: 'capstone/include') +endif +config_host_data.set('CONFIG_CAPSTONE', capstone.found()) + +genh += configure_file(output: 'config-host.h', configuration: config_host_data) + # Generators hxtool = find_program('scripts/hxtool') @@ -1512,7 +1609,7 @@ summary_info += {'vvfat support': config_host.has_key('CONFIG_VVFAT')} summary_info += {'qed support': config_host.has_key('CONFIG_QED')} summary_info += {'parallels support': config_host.has_key('CONFIG_PARALLELS')} summary_info += {'sheepdog support': config_host.has_key('CONFIG_SHEEPDOG')} -summary_info += {'capstone': config_host.has_key('CONFIG_CAPSTONE')} +summary_info += {'capstone': capstone_opt} summary_info += {'libpmem support': config_host.has_key('CONFIG_LIBPMEM')} summary_info += {'libdaxctl support': config_host.has_key('CONFIG_LIBDAXCTL')} summary_info += {'libudev': config_host.has_key('CONFIG_LIBUDEV')} diff --git a/meson_options.txt b/meson_options.txt index 543cf70043..e650a937e7 100644 --- a/meson_options.txt +++ b/meson_options.txt @@ -22,3 +22,7 @@ option('vnc_sasl', type : 'feature', value : 'auto', description: 'SASL authentication for VNC server') option('xkbcommon', type : 'feature', value : 'auto', description: 'xkbcommon support') + +option('capstone', type: 'combo', value: 'auto', + choices: ['no', 'yes', 'auto', 'system', 'internal'], + description: 'Whether and how to find the capstone library')
There are better ways to do this, e.g. meson cmake subproject, but that requires cmake 3.7 and some of our CI environments only provide cmake 3.5. Nor can we add a meson.build file to capstone/, because the git submodule would then always report "untracked files". Fixing that would require creating our own branch on the qemu git mirror, at which point we could just as easily create a native meson subproject. Instead, build the library via the main meson.build. This improves the current state of affairs in that we will re-link the qemu executables against a changed libcapstone.a, which we wouldn't do before-hand. In addition, the use of the configuration header file instead of command-line -DEFINES means that we will rebuild the capstone objects with changes to meson.build. Signed-off-by: Richard Henderson <richard.henderson@linaro.org> --- Cc: Paolo Bonzini <pbonzini@redhat.com> v2: Further reduce probing in configure (paolo), Drop state 'internal' and use 'git' even when it isn't git. Move CONFIG_CAPSTONE to config_host_data. v3: Add Submodules separator; fix default in meson_options.txt. --- configure | 61 +++---------------------- Makefile | 16 ------- meson.build | 111 +++++++++++++++++++++++++++++++++++++++++++--- meson_options.txt | 4 ++ 4 files changed, 115 insertions(+), 77 deletions(-)