[PATCH 15/21] bpf: don't use GENMASK()
Yury Norov (NVIDIA)
yury.norov at gmail.com
Sat Oct 25 09:32:57 PDT 2025
GENMASK(high, low) notation is confusing. BITS(low, high) is more
appropriate.
Signed-off-by: Yury Norov (NVIDIA) <yury.norov at gmail.com>
---
kernel/bpf/verifier.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/kernel/bpf/verifier.c b/kernel/bpf/verifier.c
index ff40e5e65c43..a9d690d3a507 100644
--- a/kernel/bpf/verifier.c
+++ b/kernel/bpf/verifier.c
@@ -17676,7 +17676,7 @@ static void mark_fastcall_pattern_for_call(struct bpf_verifier_env *env,
* - includes R1-R5 if corresponding parameter has is described
* in the function prototype.
*/
- clobbered_regs_mask = GENMASK(cs.num_params, cs.is_void ? 1 : 0);
+ clobbered_regs_mask = BITS(cs.is_void ? 1 : 0, cs.num_params);
/* e.g. if helper call clobbers r{0,1}, expect r{2,3,4,5} in the pattern */
expected_regs_mask = ~clobbered_regs_mask & ALL_CALLER_SAVED_REGS;
@@ -24210,7 +24210,7 @@ static void compute_insn_live_regs(struct bpf_verifier_env *env,
def = ALL_CALLER_SAVED_REGS;
use = def & ~BIT(BPF_REG_0);
if (get_call_summary(env, insn, &cs))
- use = GENMASK(cs.num_params, 1);
+ use = BITS(1, cs.num_params);
break;
default:
def = 0;
--
2.43.0
More information about the linux-arm-kernel
mailing list