Thread (27 messages) 27 messages, 12 authors, 2024-03-05

Re: [PATCH 3/4] arch: define CONFIG_PAGE_SIZE_*KB on all architectures

From: Geert Uytterhoeven <geert@linux-m68k.org>
Date: 2024-02-27 08:55:02
Also in: linux-alpha, linux-arm-kernel, linux-m68k, linux-mips, linux-riscv, linux-s390, linux-sh, linux-um, lkml, loongarch, sparclinux

Hi Arnd,

On Mon, Feb 26, 2024 at 5:15 PM Arnd Bergmann [off-list ref] wrote:
From: Arnd Bergmann <arnd@arndb.de>

Most architectures only support a single hardcoded page size. In order
to ensure that each one of these sets the corresponding Kconfig symbols,
change over the PAGE_SHIFT definition to the common one and allow
only the hardware page size to be selected.

Signed-off-by: Arnd Bergmann <arnd@arndb.de>
Thanks for your patch!
quoted hunk ↗ jump to hunk
--- a/arch/m68k/Kconfig
+++ b/arch/m68k/Kconfig
@@ -84,12 +84,15 @@ config MMU

 config MMU_MOTOROLA
        bool
+       select HAVE_PAGE_SIZE_4KB

 config MMU_COLDFIRE
+       select HAVE_PAGE_SIZE_8KB
I think you can do without this...
quoted hunk ↗ jump to hunk
        bool

 config MMU_SUN3
        bool
+       select HAVE_PAGE_SIZE_8KB
        depends on MMU && !MMU_MOTOROLA && !MMU_COLDFIRE

 config ARCH_SUPPORTS_KEXEC
diff --git a/arch/m68k/Kconfig.cpu b/arch/m68k/Kconfig.cpu
index 9dcf245c9cbf..c777a129768a 100644
--- a/arch/m68k/Kconfig.cpu
+++ b/arch/m68k/Kconfig.cpu
@@ -30,6 +30,7 @@ config COLDFIRE
        select GENERIC_CSUM
        select GPIOLIB
        select HAVE_LEGACY_CLK
+       select HAVE_PAGE_SIZE_8KB if !MMU
.... if you would drop the !MMU-dependency here.
quoted hunk ↗ jump to hunk
 endchoice
@@ -45,6 +46,7 @@ config M68000
        select GENERIC_CSUM
        select CPU_NO_EFFICIENT_FFS
        select HAVE_ARCH_HASH
+       select HAVE_PAGE_SIZE_4KB
Perhaps replace this by

    config M68KCLASSIC
            bool "Classic M68K CPU family support"
            select HAVE_ARCH_PFN_VALID
  +         select HAVE_PAGE_SIZE_4KB if !MMU

so it covers all 680x0 CPUs without MMU?
        select LEGACY_TIMER_TICK
        help
          The Freescale (was Motorola) 68000 CPU is the first generation of
Gr{oetje,eeting}s,

                        Geert

-- 
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
                                -- Linus Torvalds
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help