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 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 0F5FEC05027 for ; Mon, 23 Jan 2023 21:48:11 +0000 (UTC) Received: from list by lists.xenproject.org with outflank-mailman.483260.749308 (Exim 4.92) (envelope-from ) id 1pK4fX-0006ce-KW; Mon, 23 Jan 2023 21:47:59 +0000 X-Outflank-Mailman: Message body and most headers restored to incoming version Received: by outflank-mailman (output) from mailman id 483260.749308; Mon, 23 Jan 2023 21:47:59 +0000 Received: from localhost ([127.0.0.1] helo=lists.xenproject.org) by lists.xenproject.org with esmtp (Exim 4.92) (envelope-from ) id 1pK4fX-0006cX-Hl; Mon, 23 Jan 2023 21:47:59 +0000 Received: by outflank-mailman (input) for mailman id 483260; Mon, 23 Jan 2023 21:47:58 +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 1pK4fW-0006cP-GU for xen-devel@lists.xenproject.org; Mon, 23 Jan 2023 21:47:58 +0000 Received: from dfw.source.kernel.org (dfw.source.kernel.org [139.178.84.217]) by se1-gles-sth1.inumbo.com (Halon) with ESMTPS id 98ab82b1-9b67-11ed-91b6-6bf2151ebd3b; Mon, 23 Jan 2023 22:47:57 +0100 (CET) Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by dfw.source.kernel.org (Postfix) with ESMTPS id 11B51610F4; Mon, 23 Jan 2023 21:47:56 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 2F3A4C433D2; Mon, 23 Jan 2023 21:47:54 +0000 (UTC) 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: 98ab82b1-9b67-11ed-91b6-6bf2151ebd3b DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1674510475; bh=R42fdma7M0xrNIB0RtL9NSlfB6xWahp+yeTnjQSPPmM=; h=Date:From:To:cc:Subject:In-Reply-To:References:From; b=ozF0Cstm5EJeRXweNi9D51a4wmbbFI7JWY2eAGk9Zb7rnIS5F5HWBIYQxsF2aLH/t fc9FBm9xEJfVQ9/4uVOEXw8ODI7ovrdQ/RLxjFWvZUTw3kcIGW/E/mxhK5o0SG6ilz NcHJM2RVWfeLHbTkQEBgSBdPYzXUIw16lOdFhq9K9a+7d6gRh53+0lTHB6QKn30A5G X4DW3lEzZscwg9AJ69XxeG+BZqiRk7NfrqahexdO5sAh7vq3F1KirwSJeODXMMGxIx qU8IOGffkwc6q0jVAUmhM5QAJGHqkzpYhjgXPhcqp4Hce1K/olP06XXvjRhM4aad6T AZa8TEhHNldvQ== Date: Mon, 23 Jan 2023 13:47:52 -0800 (PST) From: Stefano Stabellini X-X-Sender: sstabellini@ubuntu-linux-20-04-desktop To: Julien Grall cc: xen-devel@lists.xenproject.org, Julien Grall , Stefano Stabellini , Bertrand Marquis , Volodymyr Babchuk , Andrew Cooper , George Dunlap , Jan Beulich , Wei Liu Subject: Re: [PATCH 12/22] xen/arm: fixmap: Rename the fixmap slots to follow the x86 convention In-Reply-To: <20221216114853.8227-13-julien@xen.org> Message-ID: References: <20221216114853.8227-1-julien@xen.org> <20221216114853.8227-13-julien@xen.org> User-Agent: Alpine 2.22 (DEB 394 2020-01-19) MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII On Fri, 16 Dec 2022, Julien Grall wrote: > From: Julien Grall > > At the moment the fixmap slots are prefixed differently between arm and > x86. > > Some of them (e.g. the PMAP slots) are used in common code. So it would > be better if they are named the same way to avoid having to create > aliases. > > I have decided to use the x86 naming because they are less change. So > all the Arm fixmap slots will now be prefixed with FIX rather than > FIXMAP. > > Signed-off-by: Julien Grall Reviewed-by: Stefano Stabellini > ---- > > Note that potentially more renaming that could be done to share > more code in future. I have decided to not do that to avoid going > down a rabbit hole. > --- > xen/arch/arm/acpi/lib.c | 18 +++++++++--------- > xen/arch/arm/include/asm/early_printk.h | 2 +- > xen/arch/arm/include/asm/fixmap.h | 16 ++++++++-------- > xen/arch/arm/kernel.c | 6 +++--- > xen/common/pmap.c | 8 ++++---- > 5 files changed, 25 insertions(+), 25 deletions(-) > > diff --git a/xen/arch/arm/acpi/lib.c b/xen/arch/arm/acpi/lib.c > index 41d521f720ac..736cf09ecaa8 100644 > --- a/xen/arch/arm/acpi/lib.c > +++ b/xen/arch/arm/acpi/lib.c > @@ -40,10 +40,10 @@ char *__acpi_map_table(paddr_t phys, unsigned long size) > return NULL; > > offset = phys & (PAGE_SIZE - 1); > - base = FIXMAP_ADDR(FIXMAP_ACPI_BEGIN) + offset; > + base = FIXMAP_ADDR(FIX_ACPI_BEGIN) + offset; > > /* Check the fixmap is big enough to map the region */ > - if ( (FIXMAP_ADDR(FIXMAP_ACPI_END) + PAGE_SIZE - base) < size ) > + if ( (FIXMAP_ADDR(FIX_ACPI_END) + PAGE_SIZE - base) < size ) > return NULL; > > /* With the fixmap, we can only map one region at the time */ > @@ -54,7 +54,7 @@ char *__acpi_map_table(paddr_t phys, unsigned long size) > > size += offset; > mfn = maddr_to_mfn(phys); > - idx = FIXMAP_ACPI_BEGIN; > + idx = FIX_ACPI_BEGIN; > > do { > set_fixmap(idx, mfn, PAGE_HYPERVISOR); > @@ -72,8 +72,8 @@ bool __acpi_unmap_table(const void *ptr, unsigned long size) > unsigned int idx; > > /* We are only handling fixmap address in the arch code */ > - if ( (vaddr < FIXMAP_ADDR(FIXMAP_ACPI_BEGIN)) || > - (vaddr >= (FIXMAP_ADDR(FIXMAP_ACPI_END) + PAGE_SIZE)) ) > + if ( (vaddr < FIXMAP_ADDR(FIX_ACPI_BEGIN)) || > + (vaddr >= (FIXMAP_ADDR(FIX_ACPI_END) + PAGE_SIZE)) ) > return false; > > /* > @@ -81,16 +81,16 @@ bool __acpi_unmap_table(const void *ptr, unsigned long size) > * for the ACPI fixmap region. The caller is expected to free with > * the same address. > */ > - ASSERT((vaddr & PAGE_MASK) == FIXMAP_ADDR(FIXMAP_ACPI_BEGIN)); > + ASSERT((vaddr & PAGE_MASK) == FIXMAP_ADDR(FIX_ACPI_BEGIN)); > > /* The region allocated fit in the ACPI fixmap region. */ > - ASSERT(size < (FIXMAP_ADDR(FIXMAP_ACPI_END) + PAGE_SIZE - vaddr)); > + ASSERT(size < (FIXMAP_ADDR(FIX_ACPI_END) + PAGE_SIZE - vaddr)); > ASSERT(fixmap_inuse); > > fixmap_inuse = false; > > - size += vaddr - FIXMAP_ADDR(FIXMAP_ACPI_BEGIN); > - idx = FIXMAP_ACPI_BEGIN; > + size += vaddr - FIXMAP_ADDR(FIX_ACPI_BEGIN); > + idx = FIX_ACPI_BEGIN; > > do > { > diff --git a/xen/arch/arm/include/asm/early_printk.h b/xen/arch/arm/include/asm/early_printk.h > index c5149b2976da..a5f48801f476 100644 > --- a/xen/arch/arm/include/asm/early_printk.h > +++ b/xen/arch/arm/include/asm/early_printk.h > @@ -17,7 +17,7 @@ > > /* need to add the uart address offset in page to the fixmap address */ > #define EARLY_UART_VIRTUAL_ADDRESS \ > - (FIXMAP_ADDR(FIXMAP_CONSOLE) + (CONFIG_EARLY_UART_BASE_ADDRESS & ~PAGE_MASK)) > + (FIXMAP_ADDR(FIX_CONSOLE) + (CONFIG_EARLY_UART_BASE_ADDRESS & ~PAGE_MASK)) > > #endif /* !CONFIG_EARLY_PRINTK */ > > diff --git a/xen/arch/arm/include/asm/fixmap.h b/xen/arch/arm/include/asm/fixmap.h > index d0c9a52c8c28..154db85686c2 100644 > --- a/xen/arch/arm/include/asm/fixmap.h > +++ b/xen/arch/arm/include/asm/fixmap.h > @@ -8,17 +8,17 @@ > #include > > /* Fixmap slots */ > -#define FIXMAP_CONSOLE 0 /* The primary UART */ > -#define FIXMAP_MISC 1 /* Ephemeral mappings of hardware */ > -#define FIXMAP_ACPI_BEGIN 2 /* Start mappings of ACPI tables */ > -#define FIXMAP_ACPI_END (FIXMAP_ACPI_BEGIN + NUM_FIXMAP_ACPI_PAGES - 1) /* End mappings of ACPI tables */ > -#define FIXMAP_PMAP_BEGIN (FIXMAP_ACPI_END + 1) /* Start of PMAP */ > -#define FIXMAP_PMAP_END (FIXMAP_PMAP_BEGIN + NUM_FIX_PMAP - 1) /* End of PMAP */ > +#define FIX_CONSOLE 0 /* The primary UART */ > +#define FIX_MISC 1 /* Ephemeral mappings of hardware */ > +#define FIX_ACPI_BEGIN 2 /* Start mappings of ACPI tables */ > +#define FIX_ACPI_END (FIX_ACPI_BEGIN + NUM_FIXMAP_ACPI_PAGES - 1) /* End mappings of ACPI tables */ > +#define FIX_PMAP_BEGIN (FIX_ACPI_END + 1) /* Start of PMAP */ > +#define FIX_PMAP_END (FIX_PMAP_BEGIN + NUM_FIX_PMAP - 1) /* End of PMAP */ > > -#define FIXMAP_LAST FIXMAP_PMAP_END > +#define FIX_LAST FIX_PMAP_END > > #define FIXADDR_START FIXMAP_ADDR(0) > -#define FIXADDR_TOP FIXMAP_ADDR(FIXMAP_LAST) > +#define FIXADDR_TOP FIXMAP_ADDR(FIX_LAST) > > #ifndef __ASSEMBLY__ > > diff --git a/xen/arch/arm/kernel.c b/xen/arch/arm/kernel.c > index 23b840ea9ea8..56800750fd9c 100644 > --- a/xen/arch/arm/kernel.c > +++ b/xen/arch/arm/kernel.c > @@ -49,7 +49,7 @@ struct minimal_dtb_header { > */ > void __init copy_from_paddr(void *dst, paddr_t paddr, unsigned long len) > { > - void *src = (void *)FIXMAP_ADDR(FIXMAP_MISC); > + void *src = (void *)FIXMAP_ADDR(FIX_MISC); > > while (len) { > unsigned long l, s; > @@ -57,10 +57,10 @@ void __init copy_from_paddr(void *dst, paddr_t paddr, unsigned long len) > s = paddr & (PAGE_SIZE-1); > l = min(PAGE_SIZE - s, len); > > - set_fixmap(FIXMAP_MISC, maddr_to_mfn(paddr), PAGE_HYPERVISOR_WC); > + set_fixmap(FIX_MISC, maddr_to_mfn(paddr), PAGE_HYPERVISOR_WC); > memcpy(dst, src + s, l); > clean_dcache_va_range(dst, l); > - clear_fixmap(FIXMAP_MISC); > + clear_fixmap(FIX_MISC); > > paddr += l; > dst += l; > diff --git a/xen/common/pmap.c b/xen/common/pmap.c > index 14517198aae3..6e3ba9298df4 100644 > --- a/xen/common/pmap.c > +++ b/xen/common/pmap.c > @@ -32,8 +32,8 @@ void *__init pmap_map(mfn_t mfn) > > __set_bit(idx, inuse); > > - slot = idx + FIXMAP_PMAP_BEGIN; > - ASSERT(slot >= FIXMAP_PMAP_BEGIN && slot <= FIXMAP_PMAP_END); > + slot = idx + FIX_PMAP_BEGIN; > + ASSERT(slot >= FIX_PMAP_BEGIN && slot <= FIX_PMAP_END); > > /* > * We cannot use set_fixmap() here. We use PMAP when the domain map > @@ -53,10 +53,10 @@ void __init pmap_unmap(const void *p) > unsigned int slot = virt_to_fix((unsigned long)p); > > ASSERT(system_state < SYS_STATE_smp_boot); > - ASSERT(slot >= FIXMAP_PMAP_BEGIN && slot <= FIXMAP_PMAP_END); > + ASSERT(slot >= FIX_PMAP_BEGIN && slot <= FIX_PMAP_END); > ASSERT(!in_irq()); > > - idx = slot - FIXMAP_PMAP_BEGIN; > + idx = slot - FIX_PMAP_BEGIN; > > __clear_bit(idx, inuse); > arch_pmap_unmap(slot); > -- > 2.38.1 >