From patchwork Fri Mar 15 13:08:48 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: =?utf-8?q?Philippe_Mathieu-Daud=C3=A9?= X-Patchwork-Id: 780495 Delivered-To: patch@linaro.org Received: by 2002:a5d:46c1:0:b0:33e:7753:30bd with SMTP id g1csp193028wrs; Fri, 15 Mar 2024 06:17:11 -0700 (PDT) X-Forwarded-Encrypted: i=2; AJvYcCUrhLz9cDimWA+28PIEAuL2zMZPuUoS7GpkqbkId/Iac304SN+suohiHgOA9TrKSIiKUpwfdbmZhRRETSmA0iDy X-Google-Smtp-Source: AGHT+IHQF1OX8TOC0JD3hphBng0Uf3g93guDjQgQ8VPcGG8Xb8WdaltZXx1E560j6fBhvqzvKRLh X-Received: by 2002:a05:6808:2211:b0:3c3:6571:3775 with SMTP id bd17-20020a056808221100b003c365713775mr4314394oib.19.1710508631448; Fri, 15 Mar 2024 06:17:11 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1710508631; cv=none; d=google.com; s=arc-20160816; b=WWgIuSW+8jCYPztzhUb4THXEDs01bwz+Tpwhcy6jYCCl4p8FnO2sD2NyoeHRmChduF BsXEYUFhaVU6rgM+pc1GlrzmIRSpzaEQOUaqtoQLC+YqDwyL6LoAOtnvnJCT3Qrwh5Vu n0iq4PA/dpxK1tXXET+EMKqb9zmo76B1CRWkn26E2qcozCQ6qseIAq5I4OqfISAGZ8Cq BgDyL4jhSIyuMLQFkzKkakXUYyAN4CPiIC+qBwJT+Edn+h0aXDYS3+4ehJkrh37OdIjE 00rR6/idnRAhCtgRlZIiHpHmZPv4/Qh9dnFYMb0subPviSfnPGzWCTmsXvRM7rVCWVCD eQPQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=sender:errors-to:list-subscribe:list-help:list-post:list-archive :list-unsubscribe:list-id:precedence:content-transfer-encoding :mime-version:message-id:date:subject:cc:to:from:dkim-signature; bh=7a422lMoFqhS7qWnbusg1+rx0QiF01UmLh0mI1ZQEY0=; fh=Ol3tAPX/VLbajEqrc9A2NaG0RWh2tFO21EWtQ1skWSs=; b=mdI/2j8S/lSl6EDJdI+J5TRcdKjHhQBELUtIaq58TO3ON2TNhObCQNlOLinaelxWHm A3c0SYBx72MQHGPbVNNAEPy8Gzr47OOJq2ZpWGyKCK3HeRQF4WQsIr4E4kmR0nT1szI0 D7hvDymq1emc2VpWDQPGjs9Teg15GBv8vioMFv++K7C51eOyrWUZRnm+DeaN03MkkFHO dDyvjIgUaNm5+Ckjjcy/SdRw97bUZefRfG9f6cHKwCGe7Obt7qz6SjvsUacM7kV4VY3x t7EFc+zSYD+TDtFfTFdOL/rPbnEHJeIP9P1XhoPXnb/iNf0S5XmNLtCiRx2fECifIfbW CbEw==; dara=google.com ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@linaro.org header.s=google header.b=HN3TCHDX; spf=pass (google.com: domain of qemu-devel-bounces+patch=linaro.org@nongnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom="qemu-devel-bounces+patch=linaro.org@nongnu.org"; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linaro.org Return-Path: Received: from lists.gnu.org (lists.gnu.org. [209.51.188.17]) by mx.google.com with ESMTPS id p15-20020a05622a048f00b00430b82a1124si252022qtx.155.2024.03.15.06.17.11 for (version=TLS1_2 cipher=ECDHE-ECDSA-CHACHA20-POLY1305 bits=256/256); Fri, 15 Mar 2024 06:17:11 -0700 (PDT) Received-SPF: pass (google.com: domain of qemu-devel-bounces+patch=linaro.org@nongnu.org designates 209.51.188.17 as permitted sender) client-ip=209.51.188.17; Authentication-Results: mx.google.com; dkim=pass header.i=@linaro.org header.s=google header.b=HN3TCHDX; spf=pass (google.com: domain of qemu-devel-bounces+patch=linaro.org@nongnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom="qemu-devel-bounces+patch=linaro.org@nongnu.org"; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linaro.org Received: from localhost ([::1] helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1rl7Jc-0005eS-Jb; Fri, 15 Mar 2024 09:09:40 -0400 Received: from eggs.gnu.org ([2001:470:142:3::10]) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1rl7JM-0005cD-NB for qemu-devel@nongnu.org; Fri, 15 Mar 2024 09:09:27 -0400 Received: from mail-lf1-x12d.google.com ([2a00:1450:4864:20::12d]) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_128_GCM_SHA256:128) (Exim 4.90_1) (envelope-from ) id 1rl7JF-0007X9-KR for qemu-devel@nongnu.org; Fri, 15 Mar 2024 09:09:24 -0400 Received: by mail-lf1-x12d.google.com with SMTP id 2adb3069b0e04-51381021af1so2897782e87.0 for ; Fri, 15 Mar 2024 06:09:16 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1710508155; x=1711112955; darn=nongnu.org; h=content-transfer-encoding:mime-version:message-id:date:subject:cc :to:from:from:to:cc:subject:date:message-id:reply-to; bh=7a422lMoFqhS7qWnbusg1+rx0QiF01UmLh0mI1ZQEY0=; b=HN3TCHDXYLS6LMy4hk6gXVR2LxyUJq8I/ly2Tw+zZODMIcsNzMEm/hawiCIY7p4bqe ndisUQ1GJhjvNrmQ8bHmX/zpnH+EnHDrGYBKa2ZL/FHwse0p6/lE5QUyL9EVbarLKgY1 y/LeN2eA+L2zuwfFA6zb6ye0DT7R5MLynwEy+rns3Mp+eH4uQ+aGNJiGzWBcFyKSyl+Z mXhwSUmzzmU/WttLdzgV4EvAO6IQuR2epZwQhN1W3vKOIlsD1V1tq4GOu25AwBQOp863 R716Pof5cPYoJooM9Lm0WQfetMqpIjF56hfyRpRLtrScKZmJFoJDHFg1UN/9fyPwsaJ3 vLwA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1710508155; x=1711112955; h=content-transfer-encoding:mime-version:message-id:date:subject:cc :to:from:x-gm-message-state:from:to:cc:subject:date:message-id :reply-to; bh=7a422lMoFqhS7qWnbusg1+rx0QiF01UmLh0mI1ZQEY0=; b=qvZZQJ6wd1e8kC+6hbtbeb/kT8hjQq6WX84AG0yaIIjHkj3TPTGe3n/rYNWG6Iv3/d 8UY97TXpmNAHUo9bjimKlPIZ7tZi8wpp7f/muo4IjmbOJrUig7Wi4z/IAX5Tuw/virq4 fAhxB8SSajW6ucI5T05YwkijpAKsYUgSU9C0RaISjv9cqhvrfOpbkERTp6jYC/CguV+O l1PwV6ZGKIbFVHzECPK3i0Ox6DOlb8jMsA1HzCJXjpOclPPt+EQipyhbDILA3CI03fhj B5pAJPdrR3cWd+7WLDeC87Oyy4/c4KdMJQF0HNZGKs51VuBPult1i7i1vkfosv4Lfb3l MbrQ== X-Gm-Message-State: AOJu0Yznn1F0M7HHBwCZM7LeOMKVzipCjYZd1H9xvJWlc28dtHLWLIBi dn1yuW0IZiKKyZvaVPiSiaz8K8tXlEuJEKIwJuHPUTbIqFoDVT4Ma5zkOqyfReL4M1CF4vNwx4+ z X-Received: by 2002:a05:6512:468:b0:512:e02f:9fa7 with SMTP id x8-20020a056512046800b00512e02f9fa7mr2636463lfd.1.1710508154576; Fri, 15 Mar 2024 06:09:14 -0700 (PDT) Received: from m1x-phil.lan ([176.176.145.26]) by smtp.gmail.com with ESMTPSA id gx27-20020a1709068a5b00b00a3d11feb32esm1690762ejc.186.2024.03.15.06.09.12 (version=TLS1_3 cipher=TLS_CHACHA20_POLY1305_SHA256 bits=256/256); Fri, 15 Mar 2024 06:09:14 -0700 (PDT) From: =?utf-8?q?Philippe_Mathieu-Daud=C3=A9?= To: qemu-devel@nongnu.org, Markus Armbruster Cc: qemu-riscv@nongnu.org, Anton Johansson , qemu-s390x@nongnu.org, qemu-ppc@nongnu.org, =?utf-8?q?Daniel_P_=2E_Berrang?= =?utf-8?q?=C3=A9?= , Paolo Bonzini , Eduardo Habkost , Claudio Fontana , Richard Henderson , Peter Maydell , =?utf-8?q?Alex_Benn=C3=A9e?= , Manos Pitsidianakis , Zhao Liu , qemu-arm@nongnu.org, Thomas Huth , =?utf-8?q?Philippe_Mathieu-Daud=C3=A9?= Subject: [RFC PATCH-for-9.1 00/21] qapi: Make @query-cpu-definitions command target-agnostic Date: Fri, 15 Mar 2024 14:08:48 +0100 Message-ID: <20240315130910.15750-1-philmd@linaro.org> X-Mailer: git-send-email 2.41.0 MIME-Version: 1.0 Received-SPF: pass client-ip=2a00:1450:4864:20::12d; envelope-from=philmd@linaro.org; helo=mail-lf1-x12d.google.com X-Spam_score_int: -20 X-Spam_score: -2.1 X-Spam_bar: -- X-Spam_report: (-2.1 / 5.0 requ) BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, RCVD_IN_DNSWL_NONE=-0.0001, SPF_HELO_NONE=0.001, SPF_PASS=-0.001, T_SCC_BODY_TEXT_LINE=-0.01 autolearn=unavailable autolearn_force=no X-Spam_action: no action X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: qemu-devel-bounces+patch=linaro.org@nongnu.org Sender: qemu-devel-bounces+patch=linaro.org@nongnu.org Hi Alex, Markus, Markus mentioned QAPI problems with the heterogeneous emulation binary. My understanding is, while QAPI can use host-specific conditional (OS, library available, configure option), it shouldn't use target-specific ones. This series is an example on how to remove target specific bits from the @query-cpu-definitions command. Target specific code is registered as CPUClass handlers, then a generic method is used, iterating over all targets built in. The first set of patches were already posted / reviewed last year. The PPC and S390X targets still need work (help welcomed), however the code is useful enough to be tested and see if this is a good approach. The only drawback is a change in QAPI introspection, because targets not implementing @query-cpu-definitions were returning "CommandNotFound". My view is this was an incomplete implementation, rather than a feature. Regards, Phil. Philippe Mathieu-Daudé (21): target/i386: Declare CPU QOM types using DEFINE_TYPES() macro target/mips: Declare CPU QOM types using DEFINE_TYPES() macro target/ppc: Declare CPU QOM types using DEFINE_TYPES() macro target/sparc: Declare CPU QOM types using DEFINE_TYPES() macro cpus: Open code OBJECT_DECLARE_TYPE() in OBJECT_DECLARE_CPU_TYPE() target/i386: Make X86_CPU common to new I386_CPU / X86_64_CPU types target/mips: Make MIPS_CPU common to new MIPS32_CPU / MIPS64_CPU types target/sparc: Make SPARC_CPU common to new SPARC32_CPU/SPARC64_CPU types qapi: Merge machine-common.json with qapi/machine.json qapi: Make CpuModel* definitions target agnostic qapi: Make CpuDefinitionInfo target agnostic system: Introduce QemuArchBit enum system: Introduce cpu_typename_by_arch_bit() system: Introduce QMP generic_query_cpu_definitions() target/arm: Use QMP generic_query_cpu_definitions() target/loongarch: Use QMP generic_query_cpu_definitions() target/riscv: Use QMP generic_query_cpu_definitions() target/i386: Use QMP generic_query_cpu_definitions() target/ppc: Factor ppc_add_alias_definitions() out target/ppc: Use QMP generic_query_cpu_definitions() qapi: Make @query-cpu-definitions target-agnostic MAINTAINERS | 3 +- qapi/machine-common.json | 21 ---- qapi/machine-target.json | 167 +------------------------- qapi/machine.json | 166 ++++++++++++++++++++++++- qapi/qapi-schema.json | 1 - include/hw/core/cpu.h | 7 +- include/hw/core/sysemu-cpu-ops.h | 14 +++ include/sysemu/arch_init.h | 71 +++++++---- target/i386/cpu-qom.h | 16 ++- target/mips/cpu-qom.h | 13 +- target/ppc/cpu-models.h | 4 + target/riscv/cpu.h | 2 + target/s390x/cpu.h | 2 +- target/sparc/cpu-qom.h | 9 +- system/cpu-qmp-cmds.c | 71 +++++++++++ system/cpu-qom-helpers.c | 58 +++++++++ target/arm/arm-qmp-cmds.c | 27 ----- target/i386/cpu.c | 77 ++++++------ target/loongarch/loongarch-qmp-cmds.c | 25 ---- target/mips/cpu.c | 34 ++++-- target/mips/sysemu/mips-qmp-cmds.c | 31 ----- target/ppc/cpu_init.c | 53 ++++---- target/ppc/ppc-qmp-cmds.c | 31 +---- target/riscv/cpu.c | 1 + target/riscv/riscv-qmp-cmds.c | 13 +- target/sparc/cpu.c | 35 ++++-- tests/qtest/cpu-plug-test.c | 2 +- qapi/meson.build | 1 - system/meson.build | 2 + 29 files changed, 515 insertions(+), 442 deletions(-) delete mode 100644 qapi/machine-common.json create mode 100644 system/cpu-qmp-cmds.c create mode 100644 system/cpu-qom-helpers.c