[bpf-next,v2,0/4] selftests: bpf: improve bpf object file rules
mbox series

Message ID 1549886481-25848-1-git-send-email-jiong.wang@netronome.com
Headers show
Series
  • selftests: bpf: improve bpf object file rules
Related show

Message

Jiong Wang Feb. 11, 2019, 12:01 p.m. UTC
This set improves bpf object file related rules in selftests Makefile.
  - tell git to ignore the build dir "alu32".
  - extend sub-register mode compilation to all bpf object files to give
    LLVM compiler bpf back-end more exercise.
  - auto-generate bpf kernel object file list.
  - relax sub-register mode compilation criteria.

v1 -> v2:
  - rename "kern_progs" to "progs". (Alexei)
  - spin a new patch to remove build server kernel requirement for
    sub-register mode compilation (Alexei)
  - rebase on top of KaFai’s latest "test_sock_fields" patch set.

Jiong Wang (4):
  selftests: bpf: add "alu32" to .gitignore
  selftests: bpf: extend sub-register mode compilation to all bpf object
    files
  selftests: bpf: centre kernel bpf objects under new subdir "progs"
  selftests: bpf: relax sub-register mode compilation criteria

 tools/testing/selftests/bpf/.gitignore             |  1 +
 tools/testing/selftests/bpf/Makefile               | 45 ++++++----------------
 tools/testing/selftests/bpf/{ => progs}/bpf_flow.c |  0
 .../selftests/bpf/{ => progs}/connect4_prog.c      |  0
 .../selftests/bpf/{ => progs}/connect6_prog.c      |  0
 .../testing/selftests/bpf/{ => progs}/dev_cgroup.c |  0
 .../selftests/bpf/{ => progs}/get_cgroup_id_kern.c |  0
 .../selftests/bpf/{ => progs}/netcnt_prog.c        |  0
 .../selftests/bpf/{ => progs}/sample_map_ret0.c    |  0
 .../selftests/bpf/{ => progs}/sample_ret0.c        |  0
 .../selftests/bpf/{ => progs}/sendmsg4_prog.c      |  0
 .../selftests/bpf/{ => progs}/sendmsg6_prog.c      |  0
 .../selftests/bpf/{ => progs}/socket_cookie_prog.c |  0
 .../selftests/bpf/{ => progs}/sockmap_parse_prog.c |  0
 .../bpf/{ => progs}/sockmap_tcp_msg_prog.c         |  0
 .../bpf/{ => progs}/sockmap_verdict_prog.c         |  0
 .../selftests/bpf/{ => progs}/test_adjust_tail.c   |  0
 .../selftests/bpf/{ => progs}/test_btf_haskv.c     |  0
 .../selftests/bpf/{ => progs}/test_btf_nokv.c      |  0
 .../bpf/{ => progs}/test_get_stack_rawtp.c         |  0
 .../testing/selftests/bpf/{ => progs}/test_l4lb.c  |  0
 .../selftests/bpf/{ => progs}/test_l4lb_noinline.c |  0
 .../bpf/{ => progs}/test_lirc_mode2_kern.c         |  0
 .../selftests/bpf/{ => progs}/test_lwt_seg6local.c |  0
 .../selftests/bpf/{ => progs}/test_map_in_map.c    |  0
 .../selftests/bpf/{ => progs}/test_map_lock.c      |  0
 .../selftests/bpf/{ => progs}/test_obj_id.c        |  0
 .../selftests/bpf/{ => progs}/test_pkt_access.c    |  0
 .../selftests/bpf/{ => progs}/test_pkt_md_access.c |  0
 .../selftests/bpf/{ => progs}/test_queue_map.c     |  0
 .../bpf/{ => progs}/test_select_reuseport_kern.c   |  0
 .../bpf/{ => progs}/test_sk_lookup_kern.c          |  0
 .../bpf/{ => progs}/test_skb_cgroup_id_kern.c      |  0
 .../bpf/{ => progs}/test_sock_fields_kern.c        |  0
 .../selftests/bpf/{ => progs}/test_sockhash_kern.c |  0
 .../selftests/bpf/{ => progs}/test_sockmap_kern.c  |  0
 .../selftests/bpf/{ => progs}/test_spin_lock.c     |  0
 .../selftests/bpf/{ => progs}/test_stack_map.c     |  0
 .../bpf/{ => progs}/test_stacktrace_build_id.c     |  0
 .../bpf/{ => progs}/test_stacktrace_map.c          |  0
 .../selftests/bpf/{ => progs}/test_tcp_estats.c    |  0
 .../selftests/bpf/{ => progs}/test_tcpbpf_kern.c   |  0
 .../bpf/{ => progs}/test_tcpnotify_kern.c          |  0
 .../selftests/bpf/{ => progs}/test_tracepoint.c    |  0
 .../selftests/bpf/{ => progs}/test_tunnel_kern.c   |  0
 tools/testing/selftests/bpf/{ => progs}/test_xdp.c |  0
 .../selftests/bpf/{ => progs}/test_xdp_meta.c      |  0
 .../selftests/bpf/{ => progs}/test_xdp_noinline.c  |  0
 .../selftests/bpf/{ => progs}/test_xdp_redirect.c  |  0
 .../selftests/bpf/{ => progs}/test_xdp_vlan.c      |  0
 .../testing/selftests/bpf/{ => progs}/xdp_dummy.c  |  0
 51 files changed, 12 insertions(+), 34 deletions(-)
 rename tools/testing/selftests/bpf/{ => progs}/bpf_flow.c (100%)
 rename tools/testing/selftests/bpf/{ => progs}/connect4_prog.c (100%)
 rename tools/testing/selftests/bpf/{ => progs}/connect6_prog.c (100%)
 rename tools/testing/selftests/bpf/{ => progs}/dev_cgroup.c (100%)
 rename tools/testing/selftests/bpf/{ => progs}/get_cgroup_id_kern.c (100%)
 rename tools/testing/selftests/bpf/{ => progs}/netcnt_prog.c (100%)
 rename tools/testing/selftests/bpf/{ => progs}/sample_map_ret0.c (100%)
 rename tools/testing/selftests/bpf/{ => progs}/sample_ret0.c (100%)
 rename tools/testing/selftests/bpf/{ => progs}/sendmsg4_prog.c (100%)
 rename tools/testing/selftests/bpf/{ => progs}/sendmsg6_prog.c (100%)
 rename tools/testing/selftests/bpf/{ => progs}/socket_cookie_prog.c (100%)
 rename tools/testing/selftests/bpf/{ => progs}/sockmap_parse_prog.c (100%)
 rename tools/testing/selftests/bpf/{ => progs}/sockmap_tcp_msg_prog.c (100%)
 rename tools/testing/selftests/bpf/{ => progs}/sockmap_verdict_prog.c (100%)
 rename tools/testing/selftests/bpf/{ => progs}/test_adjust_tail.c (100%)
 rename tools/testing/selftests/bpf/{ => progs}/test_btf_haskv.c (100%)
 rename tools/testing/selftests/bpf/{ => progs}/test_btf_nokv.c (100%)
 rename tools/testing/selftests/bpf/{ => progs}/test_get_stack_rawtp.c (100%)
 rename tools/testing/selftests/bpf/{ => progs}/test_l4lb.c (100%)
 rename tools/testing/selftests/bpf/{ => progs}/test_l4lb_noinline.c (100%)
 rename tools/testing/selftests/bpf/{ => progs}/test_lirc_mode2_kern.c (100%)
 rename tools/testing/selftests/bpf/{ => progs}/test_lwt_seg6local.c (100%)
 rename tools/testing/selftests/bpf/{ => progs}/test_map_in_map.c (100%)
 rename tools/testing/selftests/bpf/{ => progs}/test_map_lock.c (100%)
 rename tools/testing/selftests/bpf/{ => progs}/test_obj_id.c (100%)
 rename tools/testing/selftests/bpf/{ => progs}/test_pkt_access.c (100%)
 rename tools/testing/selftests/bpf/{ => progs}/test_pkt_md_access.c (100%)
 rename tools/testing/selftests/bpf/{ => progs}/test_queue_map.c (100%)
 rename tools/testing/selftests/bpf/{ => progs}/test_select_reuseport_kern.c (100%)
 rename tools/testing/selftests/bpf/{ => progs}/test_sk_lookup_kern.c (100%)
 rename tools/testing/selftests/bpf/{ => progs}/test_skb_cgroup_id_kern.c (100%)
 rename tools/testing/selftests/bpf/{ => progs}/test_sock_fields_kern.c (100%)
 rename tools/testing/selftests/bpf/{ => progs}/test_sockhash_kern.c (100%)
 rename tools/testing/selftests/bpf/{ => progs}/test_sockmap_kern.c (100%)
 rename tools/testing/selftests/bpf/{ => progs}/test_spin_lock.c (100%)
 rename tools/testing/selftests/bpf/{ => progs}/test_stack_map.c (100%)
 rename tools/testing/selftests/bpf/{ => progs}/test_stacktrace_build_id.c (100%)
 rename tools/testing/selftests/bpf/{ => progs}/test_stacktrace_map.c (100%)
 rename tools/testing/selftests/bpf/{ => progs}/test_tcp_estats.c (100%)
 rename tools/testing/selftests/bpf/{ => progs}/test_tcpbpf_kern.c (100%)
 rename tools/testing/selftests/bpf/{ => progs}/test_tcpnotify_kern.c (100%)
 rename tools/testing/selftests/bpf/{ => progs}/test_tracepoint.c (100%)
 rename tools/testing/selftests/bpf/{ => progs}/test_tunnel_kern.c (100%)
 rename tools/testing/selftests/bpf/{ => progs}/test_xdp.c (100%)
 rename tools/testing/selftests/bpf/{ => progs}/test_xdp_meta.c (100%)
 rename tools/testing/selftests/bpf/{ => progs}/test_xdp_noinline.c (100%)
 rename tools/testing/selftests/bpf/{ => progs}/test_xdp_redirect.c (100%)
 rename tools/testing/selftests/bpf/{ => progs}/test_xdp_vlan.c (100%)
 rename tools/testing/selftests/bpf/{ => progs}/xdp_dummy.c (100%)

Comments

Alexei Starovoitov Feb. 12, 2019, 4:34 a.m. UTC | #1
On Mon, Feb 11, 2019 at 12:01:17PM +0000, Jiong Wang wrote:
> This set improves bpf object file related rules in selftests Makefile.
>   - tell git to ignore the build dir "alu32".
>   - extend sub-register mode compilation to all bpf object files to give
>     LLVM compiler bpf back-end more exercise.
>   - auto-generate bpf kernel object file list.
>   - relax sub-register mode compilation criteria.
> 
> v1 -> v2:
>   - rename "kern_progs" to "progs". (Alexei)
>   - spin a new patch to remove build server kernel requirement for
>     sub-register mode compilation (Alexei)
>   - rebase on top of KaFai’s latest "test_sock_fields" patch set.

Looks great. Both test_progs and test_progs_32 are passing for me.
Applied to bpf-next. Thanks!