mbox series

[bpf-next,0/2] Remove bpf_dump_raw_ok() check for func_info and line_info

Message ID 20181212181820.779776-1-kafai@fb.com
Headers show
Series Remove bpf_dump_raw_ok() check for func_info and line_info | expand

Message

Martin KaFai Lau Dec. 12, 2018, 6:18 p.m. UTC 
This patch set removes the bpf_dump_raw_ok() guard for the
func_info and line_info during bpf_prog_get_info_by_fd().

Please see individual patch for details.
    
Martin KaFai Lau (2):
  bpf: Remove bpf_dump_raw_ok() check for func_info and line_info
  bpf: Remove !func_info and !line_info check from test_btf and bpftool

 kernel/bpf/syscall.c                   | 32 ++++++++++----------------
 tools/bpf/bpftool/prog.c               |  7 ------
 tools/lib/bpf/bpf_prog_linfo.c         |  6 +----
 tools/testing/selftests/bpf/test_btf.c |  7 ------
 4 files changed, 13 insertions(+), 39 deletions(-)

Comments

Yonghong Song Dec. 12, 2018, 6:25 p.m. UTC | #1 
On 12/12/18 10:18 AM, Martin KaFai Lau wrote:
> This patch set removes the bpf_dump_raw_ok() guard for the
> func_info and line_info during bpf_prog_get_info_by_fd().
> 
> Please see individual patch for details.
>      
> Martin KaFai Lau (2):
>    bpf: Remove bpf_dump_raw_ok() check for func_info and line_info
>    bpf: Remove !func_info and !line_info check from test_btf and bpftool
> 
>   kernel/bpf/syscall.c                   | 32 ++++++++++----------------
>   tools/bpf/bpftool/prog.c               |  7 ------
>   tools/lib/bpf/bpf_prog_linfo.c         |  6 +----
>   tools/testing/selftests/bpf/test_btf.c |  7 ------
>   4 files changed, 13 insertions(+), 39 deletions(-)
> 
Thanks for the fix! For the whole series,
Acked-by: Yonghong Song <yhs@fb.com>
Daniel Borkmann Dec. 13, 2018, 11:19 a.m. UTC | #2 
On 12/12/2018 07:18 PM, Martin KaFai Lau wrote:
> This patch set removes the bpf_dump_raw_ok() guard for the
> func_info and line_info during bpf_prog_get_info_by_fd().
> 
> Please see individual patch for details.
>     
> Martin KaFai Lau (2):
>   bpf: Remove bpf_dump_raw_ok() check for func_info and line_info
>   bpf: Remove !func_info and !line_info check from test_btf and bpftool
> 
>  kernel/bpf/syscall.c                   | 32 ++++++++++----------------
>  tools/bpf/bpftool/prog.c               |  7 ------
>  tools/lib/bpf/bpf_prog_linfo.c         |  6 +----
>  tools/testing/selftests/bpf/test_btf.c |  7 ------
>  4 files changed, 13 insertions(+), 39 deletions(-)
> 

Applied, thanks!