From eb33df0c30a89e36d1ec992c9e251a1d018bc5c5 Mon Sep 17 00:00:00 2001
From: James Mintram <me@jamesrm.com>
Date: Wed, 1 Dec 2021 17:14:19 +0000
Subject: [PATCH] Kernel: Add an x86 include check+error in x86/PageFault.h

---
 Kernel/Arch/x86/PageFault.h           | 3 +++
 Kernel/Arch/x86/common/Interrupts.cpp | 1 +
 Kernel/Memory/MemoryManager.cpp       | 1 +
 Kernel/Memory/MemoryManager.h         | 1 -
 Kernel/Memory/Region.cpp              | 1 +
 Kernel/Memory/Region.h                | 5 ++++-
 6 files changed, 10 insertions(+), 2 deletions(-)

diff --git a/Kernel/Arch/x86/PageFault.h b/Kernel/Arch/x86/PageFault.h
index 57f38184de..a51c3e3874 100644
--- a/Kernel/Arch/x86/PageFault.h
+++ b/Kernel/Arch/x86/PageFault.h
@@ -9,6 +9,9 @@
 #include <AK/Types.h>
 #include <Kernel/VirtualAddress.h>
 
+#include <AK/Platform.h>
+VALIDATE_IS_X86()
+
 namespace Kernel {
 
 struct PageFaultFlags {
diff --git a/Kernel/Arch/x86/common/Interrupts.cpp b/Kernel/Arch/x86/common/Interrupts.cpp
index 64e41af52f..d355354ec4 100644
--- a/Kernel/Arch/x86/common/Interrupts.cpp
+++ b/Kernel/Arch/x86/common/Interrupts.cpp
@@ -24,6 +24,7 @@
 #include <Kernel/Arch/Processor.h>
 #include <Kernel/Arch/RegisterState.h>
 #include <Kernel/Arch/x86/ISRStubs.h>
+#include <Kernel/Arch/x86/PageFault.h>
 #include <Kernel/Arch/x86/TrapFrame.h>
 #include <Kernel/KSyms.h>
 
diff --git a/Kernel/Memory/MemoryManager.cpp b/Kernel/Memory/MemoryManager.cpp
index c16984e955..9d915078a1 100644
--- a/Kernel/Memory/MemoryManager.cpp
+++ b/Kernel/Memory/MemoryManager.cpp
@@ -7,6 +7,7 @@
 #include <AK/Assertions.h>
 #include <AK/Memory.h>
 #include <AK/StringView.h>
+#include <Kernel/Arch/x86/PageFault.h>
 #include <Kernel/BootInfo.h>
 #include <Kernel/CMOS.h>
 #include <Kernel/FileSystem/Inode.h>
diff --git a/Kernel/Memory/MemoryManager.h b/Kernel/Memory/MemoryManager.h
index fd75a08053..fce1bf5521 100644
--- a/Kernel/Memory/MemoryManager.h
+++ b/Kernel/Memory/MemoryManager.h
@@ -11,7 +11,6 @@
 #include <AK/NonnullOwnPtrVector.h>
 #include <AK/NonnullRefPtrVector.h>
 #include <AK/String.h>
-#include <Kernel/Arch/x86/PageFault.h>
 #include <Kernel/Arch/x86/TrapFrame.h>
 #include <Kernel/Forward.h>
 #include <Kernel/Locking/Spinlock.h>
diff --git a/Kernel/Memory/Region.cpp b/Kernel/Memory/Region.cpp
index 0e2142051a..1e89f8dd84 100644
--- a/Kernel/Memory/Region.cpp
+++ b/Kernel/Memory/Region.cpp
@@ -6,6 +6,7 @@
 
 #include <AK/Memory.h>
 #include <AK/StringView.h>
+#include <Kernel/Arch/x86/PageFault.h>
 #include <Kernel/Debug.h>
 #include <Kernel/FileSystem/Inode.h>
 #include <Kernel/Memory/AnonymousVMObject.h>
diff --git a/Kernel/Memory/Region.h b/Kernel/Memory/Region.h
index 5c31693619..2f1f8af8e0 100644
--- a/Kernel/Memory/Region.h
+++ b/Kernel/Memory/Region.h
@@ -9,7 +9,6 @@
 #include <AK/EnumBits.h>
 #include <AK/IntrusiveList.h>
 #include <AK/Weakable.h>
-#include <Kernel/Arch/x86/PageFault.h>
 #include <Kernel/Forward.h>
 #include <Kernel/Heap/SlabAllocator.h>
 #include <Kernel/KString.h>
@@ -18,6 +17,10 @@
 #include <Kernel/Sections.h>
 #include <Kernel/UnixTypes.h>
 
+namespace Kernel {
+class PageFault;
+}
+
 namespace Kernel::Memory {
 
 enum class ShouldFlushTLB {