Re: [PATCH v3 0/5] LoongArch: BPF: Support more atomic instructions

[Tiezhu Yang]

On 4/21/26 21:25, Tiezhu Yang wrote:
> On 4/21/26 20:38, Huacai Chen wrote:
> > Hi, Tiezhu,
> >
> > Maybe we need V4? If so, please submit as soon as possible.
>
> For now, TBH, I do not want to do that.
>
> Patch #1 is OK.
>
> Patch #2 is OK.
>
> Patch #3 is OK according to Wang Rui's reply.
>
> Patch #4 has no big problems, the current testcases passed,
> I am not sure whether it has problem, I can fix it if there
> is a real problem, maybe it needs to update the testcase first.
>
> patch #5 is OK, hope you can replace "usable" with "enable" in
> the patch subject and commit message when applying.

For patch #5, correct myself, enable is verb, usable is adjective,
so just keep it as is, or modify the log as following:

---
selftests/bpf: Enable CAN_USE_LOAD_ACQ_STORE_REL for LoongArch

In order to do the following load-acquire and store-release tests
on LoongArch:

  sudo ./test_progs -t verifier_load_acquire
  sudo ./test_progs -t verifier_store_release
  sudo ./test_progs -t verifier_precision/bpf_load_acquire
  sudo ./test_progs -t verifier_precision/bpf_store_release
  sudo ./test_progs -t compute_live_registers/atomic_load_acq_store_rel

it needs to enable CAN_USE_LOAD_ACQ_STORE_REL for LoongArch.
---

For short, replace "make ... usable" with "enable ...".

Thanks,
Tiezhu