linux-kernel - Re: [PATCH v5 9/9] riscv/kprobe: Search free registers from unused caller-saved ones

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <87pmbwvmmr.fsf@all.your.base.are.belong.to.us>
Date:   Mon, 02 Jan 2023 19:04:28 +0100
From:   Björn Töpel <bjorn@...nel.org>
To:     Chen Guokai <chenguokai17@...ls.ucas.ac.cn>,
        paul.walmsley@...ive.com, palmer@...belt.com,
        aou@...s.berkeley.edu, rostedt@...dmis.org, mingo@...hat.com,
        sfr@...b.auug.org.au
Cc:     linux-riscv@...ts.infradead.org, linux-kernel@...r.kernel.org,
        liaochang1@...wei.com, Chen Guokai <chenguokai17@...ls.ucas.ac.cn>
Subject: Re: [PATCH v5 9/9] riscv/kprobe: Search free registers from unused
 caller-saved ones

Chen Guokai <chenguokai17@...ls.ucas.ac.cn> writes:

> diff --git a/arch/riscv/kernel/probes/opt.c b/arch/riscv/kernel/probes/opt.c
> index 1c0e9d218f6f..884e77d2df4c 100644
> --- a/arch/riscv/kernel/probes/opt.c
> +++ b/arch/riscv/kernel/probes/opt.c
> @@ -12,6 +12,7 @@
>  #include <asm/kprobes.h>
>  #include <asm/patch.h>
>  #include <asm/asm-offsets.h>
> +#include <linux/extable.h>
>  
>  #include "simulate-insn.h"
>  #include "decode-insn.h"
> @@ -130,7 +131,7 @@ static void prepare_detour_buffer(kprobe_opcode_t *code, kprobe_opcode_t *slot,
>   * as a destination register before any branch or jump instruction.
>   */
>  static void find_register(unsigned long start, unsigned long end,
> -			       unsigned long *write, unsigned long *read)
> +			  unsigned long *write, unsigned long *read)

Probably a patch messup; This makes the series no apply fully.


Björn