Re: [PATCH RESEND bpf-next v3 0/6] Zbb support and code simplification for RV64 JIT

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 





On 2024/1/23 0:30, Björn Töpel wrote:
Pu Lehui <pulehui@xxxxxxxxxxxxxxx> writes:

On 2024/1/22 23:07, Björn Töpel wrote:
Björn Töpel <bjorn@xxxxxxxxxx> writes:

Pu Lehui <pulehui@xxxxxxxxxxxxxxx> writes:

On 2024/1/22 22:33, Björn Töpel wrote:
Pu Lehui <pulehui@xxxxxxxxxxxxxxx> writes:

Add Zbb support [0] to optimize code size and performance of RV64 JIT.
Meanwhile, adjust the code for unification and simplification. Tests
test_bpf.ko and test_verifier have passed, as well as the relative
testcases of test_progs*.

Link: https://github.com/riscv/riscv-bitmanip/releases/download/1.0.0/bitmanip-1.0.0-38-g865e7a7.pdf [0]

v3 resend:
- resend for mail be treated as spam.

v3:
- Change to early-exit code style and make code more explicit.

Lehui,

Sorry for the delay. I'm chasing a struct_ops RISC-V BPF regression in
6.8-rc1, I will need to wrap my head around that prior reviewing
properly.


Oh, I also found the problem with struct ops and fixed it

Pu, with your patch bpf_iter_setsockopt, bpf_tcp_ca, and dummy_st_ops
passes!

Please spin a proper fixes patch, and feel free to add:

Tested-by: Björn Töpel <bjorn@xxxxxxxxxxxx>
Acked-by: Björn Töpel <bjorn@xxxxxxxxxx>


Is that in a hurry? If not, I would like to send it with the upcoming
patchset.

This is a separate fix, right? What patchset are you referring to where
the fix would be in?


Yes, you are right! I'll send a bug fix patch as soon as possible. Coming soon is the bpf_prog_pack for RV64 Trampoline, which currently does not seem to be related to the bugfix, will populate the commit message and send it later.😁

As of now 6.8-rc1 is broken! It would be a great with a fix asap...


Cheers,
Björn





[Index of Archives]     [Linux Samsung SoC]     [Linux Rockchip SoC]     [Linux Actions SoC]     [Linux for Synopsys ARC Processors]     [Linux NFS]     [Linux NILFS]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]


  Powered by Linux