aboutsummaryrefslogtreecommitdiffstats
path: root/net/core/dev_api.c
diff options
context:
space:
mode:
authorWangYuli <[email protected]>2025-04-11 07:32:22 +0000
committerPalmer Dabbelt <[email protected]>2025-04-16 14:29:34 +0000
commit550c2aa787d1b06efcb11de1877354502a1237f2 (patch)
tree1c5c503a15755a2297d88316b0fda812d729b057 /net/core/dev_api.c
parentriscv: KGDB: Do not inline arch_kgdb_breakpoint() (diff)
downloadkernel-550c2aa787d1b06efcb11de1877354502a1237f2.tar.gz
kernel-550c2aa787d1b06efcb11de1877354502a1237f2.zip
riscv: KGDB: Remove ".option norvc/.option rvc" for kgdb_compiled_break
[ Quoting Samuel Holland: ] This is a separate issue, but using ".option rvc" here is a bug. It will unconditionally enable the C extension for the rest of the file, even if the kernel is being built with CONFIG_RISCV_ISA_C=n. [ Quoting Palmer Dabbelt: ] We're just looking at the address of kgdb_compiled_break, so it's fine if it ends up as a c.ebreak. [ Quoting Alexandre Ghiti: ] .option norvc is used to prevent the assembler from using compressed instructions, but it's generally used when we need to ensure the size of the instructions that are used, which is not the case here as noted by Palmer since we only care about the address. So yes it will work fine with C enabled :) So let's just remove them all. Link: https://lore.kernel.org/all/[email protected]/ Link: https://lore.kernel.org/all/mhng-69513841-5068-441d-be8f-2aeebdc56a08@palmer-ri-x1c9a/ Link: https://lore.kernel.org/all/[email protected]/ Fixes: fe89bd2be866 ("riscv: Add KGDB support") Cc: Samuel Holland <[email protected]> Cc: Palmer Dabbelt <[email protected]> Cc: Alexandre Ghiti <[email protected]> Signed-off-by: WangYuli <[email protected]> Link: https://lore.kernel.org/r/[email protected] Signed-off-by: Palmer Dabbelt <[email protected]>
Diffstat (limited to 'net/core/dev_api.c')
0 files changed, 0 insertions, 0 deletions