On 8/6/15 12:02 AM, Kaixu Xia wrote:
This is a simple example and shows how to use the new ability
to get the selected Hardware PMU counter value.

Signed-off-by: Kaixu Xia <xiaka...@huawei.com>
---
  samples/bpf/Makefile       |  4 +++
  samples/bpf/bpf_helpers.h  |  2 ++
  samples/bpf/tracex6_kern.c | 26 ++++++++++++++++++
  samples/bpf/tracex6_user.c | 68 ++++++++++++++++++++++++++++++++++++++++++++++

1.
I see a bunch of warnings building it:
  HOSTCC  samples/bpf/tracex6_user.o
../samples/bpf/tracex6_user.c: In function ‘test_bpf_perf_event’:
../samples/bpf/tracex6_user.c:49:2: warning: passing argument 1 of ‘close’ makes integer from pointer without a cast [enabled by default]
In file included from ../samples/bpf/tracex6_user.c:2:0:
/usr/include/unistd.h:354:12: note: expected ‘int’ but argument is of type ‘int *’ ../samples/bpf/tracex6_user.c:20:16: warning: unused variable ‘value’ [-Wunused-variable] ../samples/bpf/tracex6_user.c:42:8: warning: ignoring return value of ‘system’, declared with attribute warn_unused_result [-Wunused-result] ../samples/bpf/tracex6_user.c:43:8: warning: ignoring return value of ‘system’, declared with attribute warn_unused_result [-Wunused-result] ../samples/bpf/tracex6_user.c:44:8: warning: ignoring return value of ‘system’, declared with attribute warn_unused_result [-Wunused-result]
  HOSTLD  samples/bpf/tracex6
  HOSTCC  samples/bpf/lathist_user.o
  HOSTLD  samples/bpf/lathist
clang -nostdinc -isystem /usr/lib/gcc/x86_64-linux-gnu/4.7/include -I../arch/x86/include -Iarch/x86/include/generated/uapi -Iarch/x86/include/generated -I../include -Iinclude -I../arch/x86/include/uapi -Iarch/x86/include/generated/uapi -I../include/uapi -Iinclude/generated/uapi -include ../include/linux/kconfig.h \
                -D__KERNEL__ -Wno-unused-value -Wno-pointer-sign \
-O2 -emit-llvm -c ../samples/bpf/tracex6_kern.c -o -| ../tools/bpf/llvm/bld/Debug+Asserts/bin/llc -march=bpf -filetype=obj -o samples/bpf/tracex6_kern.o ../samples/bpf/tracex6_kern.c:13:22: warning: declaration of 'struct pt_regs' will not be visible outside of this function [-Wvisibility]
int bpf_prog1(struct pt_regs *ctx)

Please fix.

2.
the example is incomplete.
Please add read_trace_pipe() otherwise it exits without printing
anything useful.

3.
please replace ls and pwd with ls > /dev/null
the spam on the screen is unnecessary.


--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Reply via email to