]> www.infradead.org Git - users/willy/pagecache.git/commitdiff
sh: add support for suppressing warning backtraces
authorGuenter Roeck <linux@roeck-us.net>
Thu, 13 Mar 2025 11:43:26 +0000 (11:43 +0000)
committerAndrew Morton <akpm@linux-foundation.org>
Fri, 14 Mar 2025 22:56:17 +0000 (15:56 -0700)
Add name of functions triggering warning backtraces to the __bug_table
object section to enable support for suppressing WARNING backtraces.

To limit image size impact, the pointer to the function name is only added
to the __bug_table section if both CONFIG_KUNIT_SUPPRESS_BACKTRACE and
CONFIG_DEBUG_BUGVERBOSE are enabled.  Otherwise, the __func__ assembly
parameter is replaced with a (dummy) NULL parameter to avoid an image size
increase due to unused __func__ entries (this is necessary because
__func__ is not a define but a virtual variable).

Link: https://lkml.kernel.org/r/20250313114329.284104-12-acarmina@redhat.com
Signed-off-by: Guenter Roeck <linux@roeck-us.net>
Signed-off-by: Alessandro Carminati <acarmina@redhat.com>
Tested-by: Linux Kernel Functional Testing <lkft@linaro.org>
Acked-by: Dan Carpenter <dan.carpenter@linaro.org>
Cc: Yoshinori Sato <ysato@users.sourceforge.jp>
Cc: Rich Felker <dalias@libc.org>
Cc: John Paul Adrian Glaubitz <glaubitz@physik.fu-berlin.de>
Cc: Albert Ou <aou@eecs.berkeley.edu>
Cc: Alexander Gordeev <agordeev@linux.ibm.com>
Cc: Arnd Bergmann <arnd@arndb.de>
Cc: Arthur Grillo <arthurgrillo@riseup.net>
Cc: Borislav Petkov <bp@alien8.de>
Cc: Brendan Higgins <brendan.higgins@linux.dev>
Cc: Catalin Marinas <catalin.marinas@arm.com>
Cc: Charlie Jenkins <charlie@rivosinc.com>
Cc: Daniel Diaz <daniel.diaz@linaro.org>
Cc: Daniel Vetter <daniel@ffwll.ch>
Cc: Dave Airlie <airlied@gmail.com>
Cc: Dave Hansen <dave.hansen@linux.intel.com>
Cc: David Gow <davidgow@google.com>
Cc: Heiko Carstens <hca@linux.ibm.com>
Cc: Helge Deller <deller@gmx.de>
Cc: Huacai Chen <chenhuacai@kernel.org>
Cc: Ingo Molnar <mingo@redhat.com>
Cc: Jani Nikula <jani.nikula@intel.com>
Cc: Kees Cook <keescook@chromium.org>
Cc: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
Cc: MaĆ­ra Canal <mcanal@igalia.com>
Cc: Maxime Ripard <mripard@kernel.org>
Cc: Michael Ellerman <mpe@ellerman.id.au>
Cc: Naresh Kamboju <naresh.kamboju@linaro.org>
Cc: Palmer Dabbelt <palmer@dabbelt.com>
Cc: Paul Walmsley <paul.walmsley@sifive.com>
Cc: Rae Moar <rmoar@google.com>
Cc: Simon Horman <horms@kernel.org>
Cc: Thomas Gleixner <tglx@linutronix.de>
Cc: Thomas Zimemrmann <tzimmermann@suse.de>
Cc: Vasily Gorbik <gor@linux.ibm.com>
Cc: Ville Syrjala <ville.syrjala@linux.intel.com>
Cc: Will Deacon <will@kernel.org>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
arch/sh/include/asm/bug.h

index 05a485c4fabcfce04b14f2077afd8cf0f4be278a..470ce6567d20938149f2f403b5611ca1b9801f8e 100644 (file)
  * The offending file and line are encoded in the __bug_table section.
  */
 #ifdef CONFIG_DEBUG_BUGVERBOSE
+
+#ifdef CONFIG_KUNIT_SUPPRESS_BACKTRACE
+# define HAVE_BUG_FUNCTION
+# define __BUG_FUNC_PTR        "\t.long %O2\n"
+#else
+# define __BUG_FUNC_PTR
+#endif /* CONFIG_KUNIT_SUPPRESS_BACKTRACE */
+
 #define _EMIT_BUG_ENTRY                                \
        "\t.pushsection __bug_table,\"aw\"\n"   \
        "2:\t.long 1b, %O1\n"                   \
-       "\t.short %O2, %O3\n"                   \
-       "\t.org 2b+%O4\n"                       \
+       __BUG_FUNC_PTR                          \
+       "\t.short %O3, %O4\n"                   \
+       "\t.org 2b+%O5\n"                       \
        "\t.popsection\n"
 #else
 #define _EMIT_BUG_ENTRY                                \
        "\t.pushsection __bug_table,\"aw\"\n"   \
        "2:\t.long 1b\n"                        \
-       "\t.short %O3\n"                        \
-       "\t.org 2b+%O4\n"                       \
+       "\t.short %O4\n"                        \
+       "\t.org 2b+%O5\n"                       \
        "\t.popsection\n"
 #endif
 
+#ifdef HAVE_BUG_FUNCTION
+# define __BUG_FUNC    __func__
+#else
+# define __BUG_FUNC    NULL
+#endif
+
 #define BUG()                                          \
 do {                                                   \
        __asm__ __volatile__ (                          \
@@ -47,6 +62,7 @@ do {                                                  \
                 :                                      \
                 : "n" (TRAPA_BUG_OPCODE),              \
                   "i" (__FILE__),                      \
+                  "i" (__BUG_FUNC),                    \
                   "i" (__LINE__), "i" (0),             \
                   "i" (sizeof(struct bug_entry)));     \
        unreachable();                                  \
@@ -60,6 +76,7 @@ do {                                                  \
                 :                                      \
                 : "n" (TRAPA_BUG_OPCODE),              \
                   "i" (__FILE__),                      \
+                  "i" (__BUG_FUNC),                    \
                   "i" (__LINE__),                      \
                   "i" (BUGFLAG_WARNING|(flags)),       \
                   "i" (sizeof(struct bug_entry)));     \
@@ -85,6 +102,7 @@ do {                                                 \
                 :                                      \
                 : "n" (TRAPA_BUG_OPCODE),              \
                   "i" (__FILE__),                      \
+                  "i" (__BUG_FUNC),                    \
                   "i" (__LINE__),                      \
                   "i" (BUGFLAG_UNWINDER),              \
                   "i" (sizeof(struct bug_entry)));     \