From patchwork Wed Aug 24 15:29:45 2022 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Bertrand Marquis X-Patchwork-Id: 12953731 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from lists.xenproject.org (lists.xenproject.org [192.237.175.120]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id E0F91C00140 for ; Wed, 24 Aug 2022 15:30:16 +0000 (UTC) Received: from list by lists.xenproject.org with outflank-mailman.392841.631441 (Exim 4.92) (envelope-from ) id 1oQsKQ-0003wO-70; Wed, 24 Aug 2022 15:30:02 +0000 X-Outflank-Mailman: Message body and most headers restored to incoming version Received: by outflank-mailman (output) from mailman id 392841.631441; Wed, 24 Aug 2022 15:30:02 +0000 Received: from localhost ([127.0.0.1] helo=lists.xenproject.org) by lists.xenproject.org with esmtp (Exim 4.92) (envelope-from ) id 1oQsKQ-0003vo-42; Wed, 24 Aug 2022 15:30:02 +0000 Received: by outflank-mailman (input) for mailman id 392841; Wed, 24 Aug 2022 15:30:00 +0000 Received: from se1-gles-sth1-in.inumbo.com ([159.253.27.254] helo=se1-gles-sth1.inumbo.com) by lists.xenproject.org with esmtp (Exim 4.92) (envelope-from ) id 1oQsKO-0003uF-Pa for xen-devel@lists.xenproject.org; Wed, 24 Aug 2022 15:30:00 +0000 Received: from foss.arm.com (foss.arm.com [217.140.110.172]) by se1-gles-sth1.inumbo.com (Halon) with ESMTP id 9d7a1889-23c1-11ed-bd2e-47488cf2e6aa; Wed, 24 Aug 2022 17:29:59 +0200 (CEST) Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.121.207.14]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id 5A6B6106F; Wed, 24 Aug 2022 08:30:03 -0700 (PDT) Received: from e109506.cambridge.arm.com (e109506.cambridge.arm.com [10.1.199.62]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id 4EE3E3F67D; Wed, 24 Aug 2022 08:29:58 -0700 (PDT) X-BeenThere: xen-devel@lists.xenproject.org List-Id: Xen developer discussion List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Errors-To: xen-devel-bounces@lists.xenproject.org Precedence: list Sender: "Xen-devel" X-Inumbo-ID: 9d7a1889-23c1-11ed-bd2e-47488cf2e6aa From: Bertrand Marquis To: xen-devel@lists.xenproject.org Cc: Jan Beulich , Wei Liu , Anthony PERARD , Andrew Cooper , =?utf-8?q?Roger_Pau_Monn=C3=A9?= Subject: [PATCH v1] x86/public: move XEN_ACPI_ in a new header Date: Wed, 24 Aug 2022 16:29:45 +0100 Message-Id: X-Mailer: git-send-email 2.25.1 MIME-Version: 1.0 When Xen is compiled for x86 on an arm machine, libacpi build is failing due to a wrong include path: - arch-x86/xen.h includes xen.h - xen.h includes arch-arm.h (as __i386__ and __x86_64__ are not defined but arm ones are). To solve this issue move XEN_ACPI_ definitions in a new header guest-acpi.h that can be included cleanly by mk_dsdt.c Previous users needing any of the XEN_ACPI_ definitions will now need to include arch-x86/guest-acpi.h instead of arch-x86/xen.h Fixes: d6ac8e22c7c5 ("acpi/x86: define ACPI IO registers for PVH guests") Signed-off-by: Bertrand Marquis --- The x86 header is including ../xen.h before the ifndef/define so that it gets included back by xen.h. This is wrongly making the assumption that we are using an x86 compiler which is not the case when building the tools for x86 on an arm host. Moving the definitions to an independent header is making things cleaner but some might need to include a new header but the risk is low. For the release manager: - risk: very low, the definitions moved are only used in mk_dsdt and external users would just have to include the new header. - advantage: we can now compile xen for x86 on arm build machines --- Changes in v1: - was "libacpi: Fix cross building x86 on arm" - move XEN_ACPI_ definitions in a new header guest-acpi.h - adapt mk_dsdt.c - remove todo in public header --- tools/libacpi/mk_dsdt.c | 2 +- xen/include/public/arch-x86/guest-acpi.h | 50 ++++++++++++++++++++++++ xen/include/public/arch-x86/xen.h | 6 --- 3 files changed, 51 insertions(+), 7 deletions(-) create mode 100644 xen/include/public/arch-x86/guest-acpi.h diff --git a/tools/libacpi/mk_dsdt.c b/tools/libacpi/mk_dsdt.c index c5ba4c0b2fd3..1176da80ef44 100644 --- a/tools/libacpi/mk_dsdt.c +++ b/tools/libacpi/mk_dsdt.c @@ -18,7 +18,7 @@ #include #include #if defined(CONFIG_X86) -#include +#include #include #elif defined(CONFIG_ARM_64) #include diff --git a/xen/include/public/arch-x86/guest-acpi.h b/xen/include/public/arch-x86/guest-acpi.h new file mode 100644 index 000000000000..eb288faf7bba --- /dev/null +++ b/xen/include/public/arch-x86/guest-acpi.h @@ -0,0 +1,50 @@ +/****************************************************************************** + * arch-x86/xen-acpi.h + * + * XEN ACPI interface to x86 Xen. + * + * Permission is hereby granted, free of charge, to any person obtaining a copy + * of this software and associated documentation files (the "Software"), to + * deal in the Software without restriction, including without limitation the + * rights to use, copy, modify, merge, publish, distribute, sublicense, and/or + * sell copies of the Software, and to permit persons to whom the Software is + * furnished to do so, subject to the following conditions: + * + * The above copyright notice and this permission notice shall be included in + * all copies or substantial portions of the Software. + * + * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR + * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY, + * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE + * AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER + * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING + * FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER + * DEALINGS IN THE SOFTWARE. + * + */ + +#ifndef __XEN_PUBLIC_ARCH_X86_XEN_ACPI_H__ +#define __XEN_PUBLIC_ARCH_X86_XEN_ACPI_H__ + +#if defined(__XEN__) || defined(__XEN_TOOLS__) + +/* Location of online VCPU bitmap. */ +#define XEN_ACPI_CPU_MAP 0xaf00 +#define XEN_ACPI_CPU_MAP_LEN ((HVM_MAX_VCPUS + 7) / 8) + +/* GPE0 bit set during CPU hotplug */ +#define XEN_ACPI_GPE0_CPUHP_BIT 2 + +#endif + +#endif /* __XEN_PUBLIC_ARCH_X86_XEN_ACPI_H__ */ + +/* + * Local variables: + * mode: C + * c-file-style: "BSD" + * c-basic-offset: 4 + * tab-width: 4 + * indent-tabs-mode: nil + * End: + */ diff --git a/xen/include/public/arch-x86/xen.h b/xen/include/public/arch-x86/xen.h index 58a1e87ee971..546dd4496ac6 100644 --- a/xen/include/public/arch-x86/xen.h +++ b/xen/include/public/arch-x86/xen.h @@ -325,12 +325,6 @@ struct xen_arch_domainconfig { /* Max XEN_X86_* constant. Used for ABI checking. */ #define XEN_X86_MISC_FLAGS_MAX XEN_X86_ASSISTED_X2APIC -/* Location of online VCPU bitmap. */ -#define XEN_ACPI_CPU_MAP 0xaf00 -#define XEN_ACPI_CPU_MAP_LEN ((HVM_MAX_VCPUS + 7) / 8) - -/* GPE0 bit set during CPU hotplug */ -#define XEN_ACPI_GPE0_CPUHP_BIT 2 #endif /*