On 06/23/2017 01:31 AM, Oliver O'Halloran wrote:
Currently ZONE_DEVICE depends on X86_64 and this will get unwieldly as
new architectures (and platforms) get ZONE_DEVICE support. Move to an
arch selected Kconfig option to save us the trouble.

Cc: linux...@kvack.org
Acked-by: Ingo Molnar <mi...@kernel.org>
Acked-by: Balbir Singh <bsinghar...@gmail.com>
Signed-off-by: Oliver O'Halloran <ooh...@gmail.com>
---
v2: Added missing hunk.
v3: No changes
---
  arch/x86/Kconfig | 1 +
  mm/Kconfig       | 6 +++++-
  2 files changed, 6 insertions(+), 1 deletion(-)

diff --git a/arch/x86/Kconfig b/arch/x86/Kconfig
index 0efb4c9497bc..325429a3f32f 100644
--- a/arch/x86/Kconfig
+++ b/arch/x86/Kconfig
@@ -59,6 +59,7 @@ config X86
        select ARCH_HAS_STRICT_KERNEL_RWX
        select ARCH_HAS_STRICT_MODULE_RWX
        select ARCH_HAS_UBSAN_SANITIZE_ALL
+       select ARCH_HAS_ZONE_DEVICE             if X86_64
        select ARCH_HAVE_NMI_SAFE_CMPXCHG
        select ARCH_MIGHT_HAVE_ACPI_PDC         if ACPI
        select ARCH_MIGHT_HAVE_PC_PARPORT
diff --git a/mm/Kconfig b/mm/Kconfig
index beb7a455915d..790e52a8a486 100644
--- a/mm/Kconfig
+++ b/mm/Kconfig
@@ -683,12 +683,16 @@ config IDLE_PAGE_TRACKING
See Documentation/vm/idle_page_tracking.txt for more details. +# arch_add_memory() comprehends device memory
+config ARCH_HAS_ZONE_DEVICE
+       bool
+
  config ZONE_DEVICE
        bool "Device memory (pmem, etc...) hotplug support"
        depends on MEMORY_HOTPLUG
        depends on MEMORY_HOTREMOVE
        depends on SPARSEMEM_VMEMMAP
-       depends on X86_64 #arch_add_memory() comprehends device memory
+       depends on ARCH_HAS_ZONE_DEVICE
help
          Device memory hotplug support allows for establishing pmem,


Hi Oliver,

+1, this is nice to have, and it behaves as expected on x86_64 with and without HMM, at least with the small bit of Kconfig dependency testing I did here.

thanks,
john h

Reply via email to