The following commit has been merged into the x86/debug branch of tip:

Commit-ID:     06aa9378df017ea7482b1bfdcd750104c8b3c407
Gitweb:        
https://git.kernel.org/tip/06aa9378df017ea7482b1bfdcd750104c8b3c407
Author:        Nam Cao <nam...@linutronix.de>
AuthorDate:    Mon, 12 May 2025 12:50:59 +02:00
Committer:     Ingo Molnar <mi...@kernel.org>
CommitterDate: Fri, 16 May 2025 10:13:59 +02:00

x86/tracing, x86/mm: Move page fault tracepoints to generic

Page fault tracepoints are interesting for other architectures as well.
Move them to be generic.

Signed-off-by: Nam Cao <nam...@linutronix.de>
Signed-off-by: Ingo Molnar <mi...@kernel.org>
Cc: Dave Hansen <dave.han...@linux.intel.com>
Cc: Gabriele Monaco <gmon...@redhat.com>
Cc: H. Peter Anvin <h...@zytor.com>
Cc: John Ogness <john.ogn...@linutronix.de>
Cc: Linus Torvalds <torva...@linux-foundation.org>
Cc: Peter Zijlstra <pet...@infradead.org>
Cc: Steven Rostedt <rost...@goodmis.org>
Cc: linux-trace-kernel@vger.kernel.org
Link: 
https://lore.kernel.org/r/89c2f284adf9b4c933f0e65811c50cef900a5a95.1747046848.git.nam...@linutronix.de
---
 arch/x86/include/asm/trace/exceptions.h | 48 +------------------------
 arch/x86/mm/Makefile                    |  2 +-
 arch/x86/mm/fault.c                     |  2 +-
 include/trace/events/exceptions.h       | 43 ++++++++++++++++++++++-
 4 files changed, 44 insertions(+), 51 deletions(-)
 delete mode 100644 arch/x86/include/asm/trace/exceptions.h
 create mode 100644 include/trace/events/exceptions.h

diff --git a/arch/x86/include/asm/trace/exceptions.h 
b/arch/x86/include/asm/trace/exceptions.h
deleted file mode 100644
index 34bc821..0000000
--- a/arch/x86/include/asm/trace/exceptions.h
+++ /dev/null
@@ -1,48 +0,0 @@
-/* SPDX-License-Identifier: GPL-2.0 */
-#undef TRACE_SYSTEM
-#define TRACE_SYSTEM exceptions
-
-#if !defined(_TRACE_PAGE_FAULT_H) || defined(TRACE_HEADER_MULTI_READ)
-#define _TRACE_PAGE_FAULT_H
-
-#include <linux/tracepoint.h>
-
-DECLARE_EVENT_CLASS(x86_exceptions,
-
-       TP_PROTO(unsigned long address, struct pt_regs *regs,
-                unsigned long error_code),
-
-       TP_ARGS(address, regs, error_code),
-
-       TP_STRUCT__entry(
-               __field(                unsigned long, address  )
-               __field(                unsigned long, ip       )
-               __field(                unsigned long, error_code )
-       ),
-
-       TP_fast_assign(
-               __entry->address = address;
-               __entry->ip = regs->ip;
-               __entry->error_code = error_code;
-       ),
-
-       TP_printk("address=%ps ip=%ps error_code=0x%lx",
-                 (void *)__entry->address, (void *)__entry->ip,
-                 __entry->error_code) );
-
-DEFINE_EVENT(x86_exceptions, page_fault_user,
-       TP_PROTO(unsigned long address, struct pt_regs *regs, unsigned long 
error_code),
-       TP_ARGS(address, regs, error_code));
-
-DEFINE_EVENT(x86_exceptions, page_fault_kernel,
-       TP_PROTO(unsigned long address, struct pt_regs *regs, unsigned long 
error_code),
-       TP_ARGS(address, regs, error_code));
-
-#undef TRACE_INCLUDE_PATH
-#undef TRACE_INCLUDE_FILE
-#define TRACE_INCLUDE_PATH .
-#define TRACE_INCLUDE_FILE exceptions
-#endif /*  _TRACE_PAGE_FAULT_H */
-
-/* This part must be outside protection */
-#include <trace/define_trace.h>
diff --git a/arch/x86/mm/Makefile b/arch/x86/mm/Makefile
index 32035d5..629a8bf 100644
--- a/arch/x86/mm/Makefile
+++ b/arch/x86/mm/Makefile
@@ -34,8 +34,6 @@ obj-y                         += pat/
 CFLAGS_physaddr.o              := -fno-stack-protector
 CFLAGS_mem_encrypt_identity.o  := -fno-stack-protector
 
-CFLAGS_fault.o := -I $(src)/../include/asm/trace
-
 obj-$(CONFIG_X86_32)           += pgtable_32.o iomap_32.o
 
 obj-$(CONFIG_HUGETLB_PAGE)     += hugetlbpage.o
diff --git a/arch/x86/mm/fault.c b/arch/x86/mm/fault.c
index 7e3e51f..ad4cb15 100644
--- a/arch/x86/mm/fault.c
+++ b/arch/x86/mm/fault.c
@@ -38,7 +38,7 @@
 #include <asm/sev.h>                   /* snp_dump_hva_rmpentry()      */
 
 #define CREATE_TRACE_POINTS
-#include <asm/trace/exceptions.h>
+#include <trace/events/exceptions.h>
 
 /*
  * Returns 0 if mmiotrace is disabled, or if the fault is not
diff --git a/include/trace/events/exceptions.h 
b/include/trace/events/exceptions.h
new file mode 100644
index 0000000..a631f8d
--- /dev/null
+++ b/include/trace/events/exceptions.h
@@ -0,0 +1,43 @@
+/* SPDX-License-Identifier: GPL-2.0 */
+#undef TRACE_SYSTEM
+#define TRACE_SYSTEM exceptions
+
+#if !defined(_TRACE_PAGE_FAULT_H) || defined(TRACE_HEADER_MULTI_READ)
+#define _TRACE_PAGE_FAULT_H
+
+#include <linux/tracepoint.h>
+
+DECLARE_EVENT_CLASS(exceptions,
+
+       TP_PROTO(unsigned long address, struct pt_regs *regs,
+                unsigned long error_code),
+
+       TP_ARGS(address, regs, error_code),
+
+       TP_STRUCT__entry(
+               __field(                unsigned long, address  )
+               __field(                unsigned long, ip       )
+               __field(                unsigned long, error_code )
+       ),
+
+       TP_fast_assign(
+               __entry->address = address;
+               __entry->ip = instruction_pointer(regs);
+               __entry->error_code = error_code;
+       ),
+
+       TP_printk("address=%ps ip=%ps error_code=0x%lx",
+                 (void *)__entry->address, (void *)__entry->ip,
+                 __entry->error_code) );
+
+DEFINE_EVENT(exceptions, page_fault_user,
+       TP_PROTO(unsigned long address, struct pt_regs *regs, unsigned long 
error_code),
+       TP_ARGS(address, regs, error_code));
+DEFINE_EVENT(exceptions, page_fault_kernel,
+       TP_PROTO(unsigned long address, struct pt_regs *regs, unsigned long 
error_code),
+       TP_ARGS(address, regs, error_code));
+
+#endif /*  _TRACE_PAGE_FAULT_H */
+
+/* This part must be outside protection */
+#include <trace/define_trace.h>

Reply via email to