On Wed, 2024-07-03 at 11:27 +0000, Puranjay Mohan wrote: [...] > The correct way to do this would be to change call_csr_mask() to have: > > verifier_inlines_helper_call(env, insn->imm) || bpf_jit_inlines_helper_call(insn->imm) Hi Puranjay, I've added bpf_jit_inlines_helper_call() logic in RFC v2 [1]. If you have a riscv setup at hand, would it be possible to ask you to run test case 'verifier_nocsr/canary_arm64_riscv64' on it? I verified that it works for arm64 in [2,3] but it would be nice to have it checked on riscv, which is currently not a part of the CI. Thanks, Eduard [1] https://lore.kernel.org/bpf/20240704102402.1644916-1-eddyz87@xxxxxxxxx/ [2] https://github.com/kernel-patches/bpf/actions/runs/9792217835/job/27037905408?pr=7274 [3] https://productionresultssa19.blob.core.windows.net/actions-results/6fb742ca-e78f-420e-9f1c-66e1e23365ef/workflow-job-run-0caa83a1-3221-5ca7-0e7d-7eb42ae68938/logs/job/job-logs.txt?rsct=text%2Fplain&se=2024-07-04T10%3A32%3A00Z&sig=gb0wAZ%2FPMOfZwCCtJyTqAogIyKZ%2F3lCZCfPUpMdN8rE%3D&ske=2024-07-04T18%3A36%3A08Z&skoid=ca7593d4-ee42-46cd-af88-8b886a2f84eb&sks=b&skt=2024-07-04T06%3A36%3A08Z&sktid=398a6654-997b-47e9-b12b-9515b896b4de&skv=2023-11-03&sp=r&spr=https&sr=b&st=2024-07-04T10%3A21%3A55Z&sv=2023-11-03
