On 7/24/20 11:38 AM, Philippe Mathieu-Daudé wrote: > On 7/24/20 9:56 AM, Thomas Huth wrote: >> On 24/07/2020 09.16, Philippe Mathieu-Daudé wrote: >>> At least one of softmmu or user mode has to be enabled to use >>> capstone. If not, don't clone/built it. >>> >>> This save CI time for the tools/documentation-only build jobs. >>> >>> Signed-off-by: Philippe Mathieu-Daudé <f4...@amsat.org> >>> --- >>> configure | 4 ++++ >>> 1 file changed, 4 insertions(+) >>> >>> diff --git a/configure b/configure >>> index 4bd80ed507..bc5757159a 100755 >>> --- a/configure >>> +++ b/configure >>> @@ -5381,6 +5381,10 @@ fi >>> ########################################## >>> # capstone >>> >>> +if test -z "$capstone" && test $tcg = 'no' ; then # !tcg implies !softmmu >>> + capstone="no" >>> +fi >> >> I don't think this is right. You could have a KVM-only build where you >> still want to use the disassembler for the human monitor. > > I had the same question with KVM, I agree this is unclear, this is why > I added RFC. > > Don't we have !softmmu implies !kvm?
It works because it falls back to the old disas.c (if capstone is here, use it, else fall-back). Does this means we can directly remove the capstone experiment & submodule without waiting for the libllvm integration? > >> >> But maybe it could be disabled if softmmu="no", linux_user="no" and >> bsd_user="no" ? >> >> Thomas >> >> >