[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-Id: <44f910bf-ac2c-4b2f-8e50-5cfc7dd0761a@app.fastmail.com>
Date: Mon, 22 Sep 2025 08:58:57 +0200
From: "Arnd Bergmann" <arnd@...db.de>
To: "Jason Gunthorpe" <jgg@...dia.com>
Cc: "Geert Uytterhoeven" <geert@...ux-m68k.org>, ksummit@...ts.linux.dev,
linux-kernel@...r.kernel.org, linux-arm-kernel@...ts.infradead.org,
linuxppc-dev@...ts.ozlabs.org, linux-mips@...r.kernel.org,
linux-mm@...ck.org, imx@...ts.linux.dev,
"Christophe Leroy" <christophe.leroy@...roup.eu>,
"Richard Weinberger" <richard@....at>,
"Lucas Stach" <l.stach@...gutronix.de>,
"Linus Walleij" <linus.walleij@...aro.org>,
"Geert Uytterhoeven" <geert+renesas@...der.be>,
"Ankur Arora" <ankur.a.arora@...cle.com>,
"David Hildenbrand" <david@...hat.com>,
"Mike Rapoport" <rppt@...nel.org>,
"Lorenzo Stoakes" <lorenzo.stoakes@...cle.com>,
"Matthew Wilcox" <willy@...radead.org>,
"Andrew Morton" <akpm@...ux-foundation.org>,
"Liam R. Howlett" <Liam.Howlett@...cle.com>,
"Vlastimil Babka" <vbabka@...e.cz>,
"Suren Baghdasaryan" <surenb@...gle.com>,
"Ira Weiny" <ira.weiny@...el.com>, "Nishanth Menon" <nm@...com>,
Heiko Stübner <heiko@...ech.de>,
"Alexander Sverdlin" <alexander.sverdlin@...il.com>,
"Chester A. Unal" <chester.a.unal@...nc9.com>,
"Sergio Paracuellos" <sergio.paracuellos@...il.com>,
"Andreas Larsson" <andreas@...sler.com>,
"Nicolas Schichan" <nschichan@...ebox.fr>
Subject: Re: [TECH TOPIC] Reaching consensus on CONFIG_HIGHMEM phaseout
On Fri, Sep 19, 2025, at 16:34, Jason Gunthorpe wrote:
> On Fri, Sep 19, 2025 at 04:22:20PM +0200, Arnd Bergmann wrote:
>
>> SAM9X7 in 2023 added DDR3 mode with up to 256MB according to
>> the datasheet, no idea why they chose not to support more,
>
> Quite possible the internal SOC wiring only has 28 address lines. :)
>
> I am right it sounds like we can drop vivt cache support with highmem?
> vivt is alot easier to understand if kmap isn't running around
> creating virtual addresses aliases!
I think so, yes. I've prototyped a patch for it to add a dependency
on !VIVT and the drop the code paths that are clearly no longer
needed without it. This obviously needs proper testing, but the
diffstat is promising:
diff --git a/arch/arm/Kconfig b/arch/arm/Kconfig
index f14bac913d50..0a31cfaf3818 100644
--- a/arch/arm/Kconfig
+++ b/arch/arm/Kconfig
@@ -1214,6 +1214,7 @@ config ARCH_SPARSEMEM_ENABLE
config HIGHMEM
bool "High Memory Support"
depends on MMU
+ depends on !CPU_CACHE_VIVT
select KMAP_LOCAL
select KMAP_LOCAL_NON_LINEAR_PTE_ARRAY
help
arch/arm/Kconfig | 1 +
arch/arm/configs/gemini_defconfig | 1 -
arch/arm/configs/multi_v5_defconfig | 1 -
arch/arm/configs/mvebu_v5_defconfig | 1 -
arch/arm/include/asm/highmem.h | 56 +-----------------------
arch/arm/mm/cache-feroceon-l2.c | 31 +------------
arch/arm/mm/cache-xsc3l2.c | 47 +++-----------------
arch/arm/mm/dma-mapping.c | 14 ++----
arch/arm/mm/flush.c | 19 ++------
mm/highmem.c | 86 +++++--------------------------------
10 files changed, 27 insertions(+), 230 deletions(-)
I did get an email from Nicolas Schichan (added to Cc here),
and he is still supporting a widely deployed Kirkwood based
platform that uses 1GB RAM configurations. He should get
a chance to test that with CONFIG_VMSPLIT_3G_OPT, but I
would expect that to continue working, possibly with minor
bugfixes.
For the timeline, the coming merge window is for an LTS
release, so if we remove VIVT highmem support in 6.19,
any remaining users we missed at least still be able to
use 6.18 for a couple of years and get security updates.
Arnd
Powered by blists - more mailing lists