[PATCH stable v6.6] riscv: kprobes: Fix wrong lengths passed to patch_text_nosync()
Greg Kroah-Hartman
gregkh at linuxfoundation.org
Tue Apr 29 09:31:09 PDT 2025
On Tue, Apr 29, 2025 at 06:14:18PM +0200, Nam Cao wrote:
> Unlike patch_text(), patch_text_nosync() takes the length in bytes, not
> number of instructions. It is therefore wrong for arch_prepare_ss_slot() to
> pass length=1 while patching one instruction.
>
> This bug was introduced by commit b1756750a397 ("riscv: kprobes: Use
> patch_text_nosync() for insn slots"). It has been fixed upstream by commit
> 51781ce8f448 ("riscv: Pass patch_text() the length in bytes"). However,
> beside fixing this bug, this commit does many other things, making it
> unsuitable for backporting.
We would almost always want the original commit, why not just send that
instead? What is wrong with it being in here as-is?
thanks,
greg k-h
More information about the linux-riscv
mailing list