Re: [PATCH 1/3] LoongArch: BPF: Remove duplicated flags check

[Tiezhu Yang <yangtiezhu@xxxxxxxxxxx>]

On 2025/8/21 下午5:10, Hengqi Chen wrote:
> The check for (BPF_TRAMP_F_ORIG_STACK | BPF_TRAMP_F_SHARE_IPMODIFY)
> is duplicated in __arch_prepare_bpf_trampoline(). Remove it.
>
> Signed-off-by: Hengqi Chen <hengqi.chen@xxxxxxxxx>
> ---
>   arch/loongarch/net/bpf_jit.c | 3 ---
>   1 file changed, 3 deletions(-)
>
> diff --git a/arch/loongarch/net/bpf_jit.c b/arch/loongarch/net/bpf_jit.c
> index abfdb6bb5c38..b646c6b73014 100644
> --- a/arch/loongarch/net/bpf_jit.c
> +++ b/arch/loongarch/net/bpf_jit.c
> @@ -1462,9 +1462,6 @@ static int __arch_prepare_bpf_trampoline(struct jit_ctx *ctx, struct bpf_tramp_i
>   	struct bpf_tramp_links *fmod_ret = &tlinks[BPF_TRAMP_MODIFY_RETURN];
>   	u32 **branches = NULL;
>   
> -	if (flags & (BPF_TRAMP_F_ORIG_STACK | BPF_TRAMP_F_SHARE_IPMODIFY))
> -		return -ENOTSUPP;
> -
>   	/*
>   	 * FP + 8       [ RA to parent func ] return address to parent
>   	 *                    function

Acked-by: Tiezhu Yang <yangtiezhu@xxxxxxxxxxx>

Maybe "stack_size = 0;" in this function is redundant too, if yes,
you can remove it together.

Thanks,
Tiezhu