]> www.infradead.org Git - users/jedix/linux-maple.git/commit
Merge patch series "riscv: Rework the arch_kgdb_breakpoint() implementation"
authorPalmer Dabbelt <palmer@rivosinc.com>
Wed, 16 Apr 2025 14:29:36 +0000 (07:29 -0700)
committerPalmer Dabbelt <palmer@rivosinc.com>
Wed, 16 Apr 2025 14:31:23 +0000 (07:31 -0700)
commitdc3e30b4992336007c9798e5785b47819c7ef58e
tree29ef890e9288b4b83d402394376eabb101710de9
parentadf53771a3123df99ca26e38818760fbcf5c05d0
parent550c2aa787d1b06efcb11de1877354502a1237f2
Merge patch series "riscv: Rework the arch_kgdb_breakpoint() implementation"

WangYuli <wangyuli@uniontech.com> says:

1. The arch_kgdb_breakpoint() function defines the kgdb_compiled_break
   symbol using inline assembly.

   There's a potential issue where the compiler might inline
   arch_kgdb_breakpoint(), which would then define the kgdb_compiled_break
   symbol multiple times, leading to fail to link vmlinux.o.

   This isn't merely a potential compilation problem. The intent here
   is to determine the global symbol address of kgdb_compiled_break,
   and if this function is inlined multiple times, it would logically
   be a grave error.

2. Remove ".option norvc/.option rvc" to fix a bug that the C extension
   would unconditionally enable even if the kernel is being built with
   CONFIG_RISCV_ISA_C=n.

* b4-shazam-merge:
  riscv: KGDB: Remove ".option norvc/.option rvc" for kgdb_compiled_break
  riscv: KGDB: Do not inline arch_kgdb_breakpoint()

Link: https://lore.kernel.org/r/D5A83DF3A06E1DF9+20250411072905.55134-1-wangyuli@uniontech.com
Signed-off-by: Palmer Dabbelt <palmer@rivosinc.com>