From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-20.2 required=3.0 tests=BAYES_00, HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_CR_TRAILER,INCLUDES_PATCH, MAILING_LIST_MULTI,MENTIONS_GIT_HOSTING,NICE_REPLY_A,SPF_HELO_NONE,SPF_PASS, URIBL_BLOCKED,USER_AGENT_SANE_1 autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 0FF84C433E0 for ; Mon, 22 Feb 2021 13:34:19 +0000 (UTC) Received: from lists.gnu.org (lists.gnu.org [209.51.188.17]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id 7CF2364F00 for ; Mon, 22 Feb 2021 13:34:18 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 7CF2364F00 Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=suse.de Authentication-Results: mail.kernel.org; spf=pass smtp.mailfrom=qemu-devel-bounces+qemu-devel=archiver.kernel.org@nongnu.org Received: from localhost ([::1]:44852 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1lEBLt-0007qy-Fx for qemu-devel@archiver.kernel.org; Mon, 22 Feb 2021 08:34:17 -0500 Received: from eggs.gnu.org ([2001:470:142:3::10]:33818) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1lEBIR-0004du-S4 for qemu-devel@nongnu.org; Mon, 22 Feb 2021 08:30:43 -0500 Received: from mx2.suse.de ([195.135.220.15]:44466) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1lEBIO-0007BA-VE for qemu-devel@nongnu.org; Mon, 22 Feb 2021 08:30:43 -0500 X-Virus-Scanned: by amavisd-new at test-mx.suse.de Received: from relay2.suse.de (unknown [195.135.221.27]) by mx2.suse.de (Postfix) with ESMTP id DB853AF3E; Mon, 22 Feb 2021 13:30:37 +0000 (UTC) Subject: Re: [PATCH v20 06/17] meson: add target_user_arch To: =?UTF-8?Q?Alex_Benn=c3=a9e?= References: <20210218172156.25520-1-cfontana@suse.de> <20210218172156.25520-7-cfontana@suse.de> <87y2fg5j0s.fsf@linaro.org> From: Claudio Fontana Message-ID: <52d8bead-88c9-d4db-b144-5f717727d6bf@suse.de> Date: Mon, 22 Feb 2021 14:30:36 +0100 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.12.0 MIME-Version: 1.0 In-Reply-To: <87y2fg5j0s.fsf@linaro.org> Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: 8bit Received-SPF: pass client-ip=195.135.220.15; envelope-from=cfontana@suse.de; helo=mx2.suse.de X-Spam_score_int: -41 X-Spam_score: -4.2 X-Spam_bar: ---- X-Spam_report: (-4.2 / 5.0 requ) BAYES_00=-1.9, NICE_REPLY_A=-0.001, RCVD_IN_DNSWL_MED=-2.3, RCVD_IN_MSPIKE_H3=0.001, RCVD_IN_MSPIKE_WL=0.001, SPF_HELO_NONE=0.001, SPF_PASS=-0.001 autolearn=ham autolearn_force=no X-Spam_action: no action X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.23 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Laurent Vivier , Peter Maydell , Thomas Huth , Eduardo Habkost , Richard Henderson , qemu-devel@nongnu.org, Roman Bolshakov , Paolo Bonzini , =?UTF-8?Q?Philippe_Mathieu-Daud=c3=a9?= Errors-To: qemu-devel-bounces+qemu-devel=archiver.kernel.org@nongnu.org Sender: "Qemu-devel" On 2/22/21 2:27 PM, Alex Bennée wrote: > > Claudio Fontana writes: > >> the lack of target_user_arch makes it hard to fully leverage the >> build system in order to separate user code from softmmu code. >> >> Provide it, so that we can avoid the proliferation of #ifdef >> in target code. >> >> Signed-off-by: Claudio Fontana >> Reviewed-by: Alex Bennée > > I suspect this needs an update since hexagon got merged under your feet > (which broke some of the CI): > > https://gitlab.com/stsquad/qemu/-/pipelines/259498090/failures Hi Alex, yes, I noticed, I can respin. Would be better if by default the code in meson.build would not complain about the missing dictionary key, and just consider it as empty, but it's beyond my current meson knowledge to do that. Will respin in a jiffy, ciao, Claudio > >> --- >> meson.build | 5 +++++ >> target/alpha/meson.build | 3 +++ >> target/arm/meson.build | 2 ++ >> target/cris/meson.build | 3 +++ >> target/hppa/meson.build | 3 +++ >> target/i386/meson.build | 2 ++ >> target/m68k/meson.build | 3 +++ >> target/microblaze/meson.build | 3 +++ >> target/mips/meson.build | 3 +++ >> target/nios2/meson.build | 3 +++ >> target/openrisc/meson.build | 3 +++ >> target/ppc/meson.build | 3 +++ >> target/riscv/meson.build | 3 +++ >> target/s390x/meson.build | 3 +++ >> target/sh4/meson.build | 3 +++ >> target/sparc/meson.build | 3 +++ >> target/tilegx/meson.build | 3 +++ >> target/tricore/meson.build | 3 +++ >> target/xtensa/meson.build | 3 +++ >> 19 files changed, 57 insertions(+) >> >> diff --git a/meson.build b/meson.build >> index a923f249d8..1a355b8c1e 100644 >> --- a/meson.build >> +++ b/meson.build >> @@ -1734,6 +1734,7 @@ modules = {} >> hw_arch = {} >> target_arch = {} >> target_softmmu_arch = {} >> +target_user_arch = {} >> >> ############### >> # Trace files # >> @@ -2131,6 +2132,10 @@ foreach target : target_dirs >> abi = config_target['TARGET_ABI_DIR'] >> target_type='user' >> qemu_target_name = 'qemu-' + target_name >> + t = target_user_arch[arch].apply(config_target, strict: false) >> + arch_srcs += t.sources() >> + arch_deps += t.dependencies() >> + >> if 'CONFIG_LINUX_USER' in config_target >> base_dir = 'linux-user' >> target_inc += include_directories('linux-user/host/' / config_host['ARCH']) >> diff --git a/target/alpha/meson.build b/target/alpha/meson.build >> index 1aec55abb4..1b0555d3ee 100644 >> --- a/target/alpha/meson.build >> +++ b/target/alpha/meson.build >> @@ -14,5 +14,8 @@ alpha_ss.add(files( >> alpha_softmmu_ss = ss.source_set() >> alpha_softmmu_ss.add(files('machine.c')) >> >> +alpha_user_ss = ss.source_set() >> + >> target_arch += {'alpha': alpha_ss} >> target_softmmu_arch += {'alpha': alpha_softmmu_ss} >> +target_user_arch += {'alpha': alpha_user_ss} >> diff --git a/target/arm/meson.build b/target/arm/meson.build >> index 15b936c101..a96af5ee1b 100644 >> --- a/target/arm/meson.build >> +++ b/target/arm/meson.build >> @@ -53,6 +53,8 @@ arm_softmmu_ss.add(files( >> 'monitor.c', >> 'psci.c', >> )) >> +arm_user_ss = ss.source_set() >> >> target_arch += {'arm': arm_ss} >> target_softmmu_arch += {'arm': arm_softmmu_ss} >> +target_user_arch += {'arm': arm_user_ss} >> diff --git a/target/cris/meson.build b/target/cris/meson.build >> index 67c3793c85..7fd81e0348 100644 >> --- a/target/cris/meson.build >> +++ b/target/cris/meson.build >> @@ -10,5 +10,8 @@ cris_ss.add(files( >> cris_softmmu_ss = ss.source_set() >> cris_softmmu_ss.add(files('mmu.c', 'machine.c')) >> >> +cris_user_ss = ss.source_set() >> + >> target_arch += {'cris': cris_ss} >> target_softmmu_arch += {'cris': cris_softmmu_ss} >> +target_user_arch += {'cris': cris_user_ss} >> diff --git a/target/hppa/meson.build b/target/hppa/meson.build >> index 8a7ff82efc..85ad314671 100644 >> --- a/target/hppa/meson.build >> +++ b/target/hppa/meson.build >> @@ -15,5 +15,8 @@ hppa_ss.add(files( >> hppa_softmmu_ss = ss.source_set() >> hppa_softmmu_ss.add(files('machine.c')) >> >> +hppa_user_ss = ss.source_set() >> + >> target_arch += {'hppa': hppa_ss} >> target_softmmu_arch += {'hppa': hppa_softmmu_ss} >> +target_user_arch += {'hppa': hppa_user_ss} >> diff --git a/target/i386/meson.build b/target/i386/meson.build >> index fd24479590..cac26a4581 100644 >> --- a/target/i386/meson.build >> +++ b/target/i386/meson.build >> @@ -19,6 +19,7 @@ i386_softmmu_ss.add(files( >> 'machine.c', >> 'monitor.c', >> )) >> +i386_user_ss = ss.source_set() >> >> subdir('kvm') >> subdir('hax') >> @@ -28,3 +29,4 @@ subdir('tcg') >> >> target_arch += {'i386': i386_ss} >> target_softmmu_arch += {'i386': i386_softmmu_ss} >> +target_user_arch += {'i386': i386_user_ss} >> diff --git a/target/m68k/meson.build b/target/m68k/meson.build >> index 05cd9fbd1e..b507682684 100644 >> --- a/target/m68k/meson.build >> +++ b/target/m68k/meson.build >> @@ -13,5 +13,8 @@ m68k_ss.add(files( >> m68k_softmmu_ss = ss.source_set() >> m68k_softmmu_ss.add(files('monitor.c')) >> >> +m68k_user_ss = ss.source_set() >> + >> target_arch += {'m68k': m68k_ss} >> target_softmmu_arch += {'m68k': m68k_softmmu_ss} >> +target_user_arch += {'m68k': m68k_user_ss} >> diff --git a/target/microblaze/meson.build b/target/microblaze/meson.build >> index 05ee0ec163..52d8fcb0a3 100644 >> --- a/target/microblaze/meson.build >> +++ b/target/microblaze/meson.build >> @@ -16,5 +16,8 @@ microblaze_softmmu_ss.add(files( >> 'machine.c', >> )) >> >> +microblaze_user_ss = ss.source_set() >> + >> target_arch += {'microblaze': microblaze_ss} >> target_softmmu_arch += {'microblaze': microblaze_softmmu_ss} >> +target_user_arch += {'microblaze': microblaze_user_ss} >> diff --git a/target/mips/meson.build b/target/mips/meson.build >> index 9741545440..30843c1521 100644 >> --- a/target/mips/meson.build >> +++ b/target/mips/meson.build >> @@ -36,5 +36,8 @@ mips_softmmu_ss.add(when: 'CONFIG_TCG', if_true: files( >> 'cp0_helper.c', >> )) >> >> +mips_user_ss = ss.source_set() >> + >> target_arch += {'mips': mips_ss} >> target_softmmu_arch += {'mips': mips_softmmu_ss} >> +target_user_arch += {'mips': mips_user_ss} >> diff --git a/target/nios2/meson.build b/target/nios2/meson.build >> index e643917db1..00367056fa 100644 >> --- a/target/nios2/meson.build >> +++ b/target/nios2/meson.build >> @@ -11,5 +11,8 @@ nios2_ss.add(files( >> nios2_softmmu_ss = ss.source_set() >> nios2_softmmu_ss.add(files('monitor.c')) >> >> +nios2_user_ss = ss.source_set() >> + >> target_arch += {'nios2': nios2_ss} >> target_softmmu_arch += {'nios2': nios2_softmmu_ss} >> +target_user_arch += {'nios2': nios2_user_ss} >> diff --git a/target/openrisc/meson.build b/target/openrisc/meson.build >> index 9774a58306..794a9e8161 100644 >> --- a/target/openrisc/meson.build >> +++ b/target/openrisc/meson.build >> @@ -19,5 +19,8 @@ openrisc_ss.add(files( >> openrisc_softmmu_ss = ss.source_set() >> openrisc_softmmu_ss.add(files('machine.c')) >> >> +openrisc_user_ss = ss.source_set() >> + >> target_arch += {'openrisc': openrisc_ss} >> target_softmmu_arch += {'openrisc': openrisc_softmmu_ss} >> +target_user_arch += {'openrisc': openrisc_user_ss} >> diff --git a/target/ppc/meson.build b/target/ppc/meson.build >> index bbfef90e08..cdd69bf989 100644 >> --- a/target/ppc/meson.build >> +++ b/target/ppc/meson.build >> @@ -33,5 +33,8 @@ ppc_softmmu_ss.add(when: 'TARGET_PPC64', if_true: files( >> 'mmu-radix64.c', >> )) >> >> +ppc_user_ss = ss.source_set() >> + >> target_arch += {'ppc': ppc_ss} >> target_softmmu_arch += {'ppc': ppc_softmmu_ss} >> +target_user_arch += {'ppc': ppc_user_ss} >> diff --git a/target/riscv/meson.build b/target/riscv/meson.build >> index 14a5c62dac..a55851336b 100644 >> --- a/target/riscv/meson.build >> +++ b/target/riscv/meson.build >> @@ -31,5 +31,8 @@ riscv_softmmu_ss.add(files( >> 'machine.c' >> )) >> >> +riscv_user_ss = ss.source_set() >> + >> target_arch += {'riscv': riscv_ss} >> target_softmmu_arch += {'riscv': riscv_softmmu_ss} >> +target_user_arch += {'riscv': riscv_user_ss} >> diff --git a/target/s390x/meson.build b/target/s390x/meson.build >> index c42eadb7d2..1219f64112 100644 >> --- a/target/s390x/meson.build >> +++ b/target/s390x/meson.build >> @@ -58,5 +58,8 @@ if host_machine.cpu_family() == 's390x' and cc.has_link_argument('-Wl,--s390-pgs >> if_true: declare_dependency(link_args: ['-Wl,--s390-pgste'])) >> endif >> >> +s390x_user_ss = ss.source_set() >> + >> target_arch += {'s390x': s390x_ss} >> target_softmmu_arch += {'s390x': s390x_softmmu_ss} >> +target_user_arch += {'s390x': s390x_user_ss} >> diff --git a/target/sh4/meson.build b/target/sh4/meson.build >> index 56a57576da..5a05729bc1 100644 >> --- a/target/sh4/meson.build >> +++ b/target/sh4/meson.build >> @@ -10,5 +10,8 @@ sh4_ss.add(files( >> sh4_softmmu_ss = ss.source_set() >> sh4_softmmu_ss.add(files('monitor.c')) >> >> +sh4_user_ss = ss.source_set() >> + >> target_arch += {'sh4': sh4_ss} >> target_softmmu_arch += {'sh4': sh4_softmmu_ss} >> +target_user_arch += {'sh4': sh4_user_ss} >> diff --git a/target/sparc/meson.build b/target/sparc/meson.build >> index a3638b9503..cc77a77064 100644 >> --- a/target/sparc/meson.build >> +++ b/target/sparc/meson.build >> @@ -19,5 +19,8 @@ sparc_softmmu_ss.add(files( >> 'monitor.c', >> )) >> >> +sparc_user_ss = ss.source_set() >> + >> target_arch += {'sparc': sparc_ss} >> target_softmmu_arch += {'sparc': sparc_softmmu_ss} >> +target_user_arch += {'sparc': sparc_user_ss} >> diff --git a/target/tilegx/meson.build b/target/tilegx/meson.build >> index 678590439c..23dab8b9a1 100644 >> --- a/target/tilegx/meson.build >> +++ b/target/tilegx/meson.build >> @@ -9,5 +9,8 @@ tilegx_ss.add(zlib) >> >> tilegx_softmmu_ss = ss.source_set() >> >> +tilegx_user_ss = ss.source_set() >> + >> target_arch += {'tilegx': tilegx_ss} >> target_softmmu_arch += {'tilegx': tilegx_softmmu_ss} >> +target_user_arch += {'tilegx': tilegx_user_ss} >> diff --git a/target/tricore/meson.build b/target/tricore/meson.build >> index 0ccc829517..7086ae1a22 100644 >> --- a/target/tricore/meson.build >> +++ b/target/tricore/meson.build >> @@ -11,5 +11,8 @@ tricore_ss.add(zlib) >> >> tricore_softmmu_ss = ss.source_set() >> >> +tricore_user_ss = ss.source_set() >> + >> target_arch += {'tricore': tricore_ss} >> target_softmmu_arch += {'tricore': tricore_softmmu_ss} >> +target_user_arch += {'tricore': tricore_user_ss} >> diff --git a/target/xtensa/meson.build b/target/xtensa/meson.build >> index dd750a977e..949b2c8334 100644 >> --- a/target/xtensa/meson.build >> +++ b/target/xtensa/meson.build >> @@ -28,5 +28,8 @@ xtensa_softmmu_ss.add(files( >> 'xtensa-semi.c', >> )) >> >> +xtensa_user_ss = ss.source_set() >> + >> target_arch += {'xtensa': xtensa_ss} >> target_softmmu_arch += {'xtensa': xtensa_softmmu_ss} >> +target_user_arch += {'xtensa': xtensa_user_ss} > >