diff options
author | Jakub Kicinski <jakub.kicinski@netronome.com> | 2018-03-28 20:48:28 -0400 |
---|---|---|
committer | Alexei Starovoitov <ast@kernel.org> | 2018-03-28 22:36:13 -0400 |
commit | fc4484970e7e4bf40ba958f7bc48474ddadfdbfc (patch) | |
tree | 25792acf4952851b76d992e92f59a1e56b6da2bb | |
parent | 87b10ecdced224dc0def123e1f57bc6c5ac4ac5c (diff) |
nfp: bpf: rename map_lookup_stack() to map_call_stack_common()
We will reuse most of map call code gen for other map calls.
Rename the lookup gen function and use meta->func_id instead
of hard-coding lookup.
Signed-off-by: Jakub Kicinski <jakub.kicinski@netronome.com>
Reviewed-by: Quentin Monnet <quentin.monnet@netronome.com>
Reviewed-by: Jiong Wang <jiong.wang@netronome.com>
Signed-off-by: Alexei Starovoitov <ast@kernel.org>
-rw-r--r-- | drivers/net/ethernet/netronome/nfp/bpf/jit.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/drivers/net/ethernet/netronome/nfp/bpf/jit.c b/drivers/net/ethernet/netronome/nfp/bpf/jit.c index 0701590e8302..b37c9f86a18f 100644 --- a/drivers/net/ethernet/netronome/nfp/bpf/jit.c +++ b/drivers/net/ethernet/netronome/nfp/bpf/jit.c | |||
@@ -1352,7 +1352,7 @@ static int adjust_head(struct nfp_prog *nfp_prog, struct nfp_insn_meta *meta) | |||
1352 | } | 1352 | } |
1353 | 1353 | ||
1354 | static int | 1354 | static int |
1355 | map_lookup_stack(struct nfp_prog *nfp_prog, struct nfp_insn_meta *meta) | 1355 | map_call_stack_common(struct nfp_prog *nfp_prog, struct nfp_insn_meta *meta) |
1356 | { | 1356 | { |
1357 | struct bpf_offloaded_map *offmap; | 1357 | struct bpf_offloaded_map *offmap; |
1358 | struct nfp_bpf_map *nfp_map; | 1358 | struct nfp_bpf_map *nfp_map; |
@@ -1378,7 +1378,7 @@ map_lookup_stack(struct nfp_prog *nfp_prog, struct nfp_insn_meta *meta) | |||
1378 | */ | 1378 | */ |
1379 | tid = ur_load_imm_any(nfp_prog, nfp_map->tid, imm_a(nfp_prog)); | 1379 | tid = ur_load_imm_any(nfp_prog, nfp_map->tid, imm_a(nfp_prog)); |
1380 | 1380 | ||
1381 | emit_br_relo(nfp_prog, BR_UNC, BR_OFF_RELO + BPF_FUNC_map_lookup_elem, | 1381 | emit_br_relo(nfp_prog, BR_UNC, BR_OFF_RELO + meta->func_id, |
1382 | 2, RELO_BR_HELPER); | 1382 | 2, RELO_BR_HELPER); |
1383 | ret_tgt = nfp_prog_current_offset(nfp_prog) + 2; | 1383 | ret_tgt = nfp_prog_current_offset(nfp_prog) + 2; |
1384 | 1384 | ||
@@ -2326,7 +2326,7 @@ static int call(struct nfp_prog *nfp_prog, struct nfp_insn_meta *meta) | |||
2326 | case BPF_FUNC_xdp_adjust_head: | 2326 | case BPF_FUNC_xdp_adjust_head: |
2327 | return adjust_head(nfp_prog, meta); | 2327 | return adjust_head(nfp_prog, meta); |
2328 | case BPF_FUNC_map_lookup_elem: | 2328 | case BPF_FUNC_map_lookup_elem: |
2329 | return map_lookup_stack(nfp_prog, meta); | 2329 | return map_call_stack_common(nfp_prog, meta); |
2330 | default: | 2330 | default: |
2331 | WARN_ONCE(1, "verifier allowed unsupported function\n"); | 2331 | WARN_ONCE(1, "verifier allowed unsupported function\n"); |
2332 | return -EOPNOTSUPP; | 2332 | return -EOPNOTSUPP; |