RE: [PATCH] riscv, bpf: Use STACK_ALIGN macro for size rounding up

From: Wang, Xiao W
Date: Wed May 22 2024 - 23:12:24 EST




> -----Original Message-----
> From: Pu Lehui <pulehui@xxxxxxxxxx>
> Sent: Thursday, May 23, 2024 9:43 AM
> To: Wang, Xiao W <xiao.w.wang@xxxxxxxxx>
> Cc: paul.walmsley@xxxxxxxxxx; palmer@xxxxxxxxxxx;
> aou@xxxxxxxxxxxxxxxxx; luke.r.nels@xxxxxxxxx; xi.wang@xxxxxxxxx;
> bjorn@xxxxxxxxxx; ast@xxxxxxxxxx; daniel@xxxxxxxxxxxxx; andrii@xxxxxxxxxx;
> martin.lau@xxxxxxxxx; eddyz87@xxxxxxxxx; song@xxxxxxxxxx;
> yonghong.song@xxxxxxxxx; john.fastabend@xxxxxxxxx; kpsingh@xxxxxxxxxx;
> sdf@xxxxxxxxxx; haoluo@xxxxxxxxxx; jolsa@xxxxxxxxxx; linux-
> riscv@xxxxxxxxxxxxxxxxxxx; linux-kernel@xxxxxxxxxxxxxxx; bpf@xxxxxxxxxxxxxxx;
> Li, Haicheng <haicheng.li@xxxxxxxxx>
> Subject: Re: [PATCH] riscv, bpf: Use STACK_ALIGN macro for size rounding up
>
>
> On 2024/5/22 13:45, Xiao Wang wrote:
> > Use the macro STACK_ALIGN that is defined in asm/processor.h for stack size
> > rounding up, just like bpf_jit_comp32.c does.
> >
> > Signed-off-by: Xiao Wang <xiao.w.wang@xxxxxxxxx>
> > ---
> > arch/riscv/net/bpf_jit_comp64.c | 6 +++---
> > 1 file changed, 3 insertions(+), 3 deletions(-)
>
> It met a patching conflict. I think you should target for the bpf-next tree.
> https://github.com/kernel-patches/bpf/pull/7080

OK, I would make v2 based on bpf-next.

BRs,
Xiao