- 18 Oct, 2017 1 commit
-
-
David Xia authored
add required linux-headers package
-
- 17 Oct, 2017 1 commit
-
-
4ast authored
bpf: print out the src debug info to a temporary file
-
- 16 Oct, 2017 3 commits
-
-
4ast authored
attempt to compile with system bpf.h if default compile failed
-
Yonghong Song authored
Currently, for C++ API and for each func, the original src and the rewritten source has been stored in BCC_PROG_TAG_DIR directory. This patch allows the source debug info (bytecode embedded by source code) also stored in the same BCC_PROG_TAG_DIR directory. This feature is not turned on by default now. It requires non-zero (debug_flag & DEBUG_SOURCE). The DEBUG_SOURCE enables "-g", with which a lot of more llvm insns executed and it may increase application RSS overhead by 4M (in my test). As an example, if you modify examples/cpp/RandomRead.cc to enable DEBUG_SOURCE, as below - bpf = new ebpf::BPF(); + bpf = new ebpf::BPF(8); After running the application, you can see: -bash-4.3$ ls /var/tmp/bcc/bpf_prog_7f01346289a53cc3/ on_urandom_read.c on_urandom_read.dis.txt on_urandom_read.rewritten.c -bash-4.3$ cat /var/tmp/bcc/bpf_prog_7f01346289a53cc3/on_urandom_read.dis.txt ; int on_urandom_read(struct urandom_read_args* attr) { // Line 23 0: bf 16 00 00 00 00 00 00 r6 = r1 1: b7 01 00 00 00 00 00 00 r1 = 0 ; struct event_t event = {}; // Line 24 2: 63 1a f8 ff 00 00 00 00 *(u32 *)(r10 - 8) = r1 3: 63 1a f4 ff 00 00 00 00 *(u32 *)(r10 - 12) = r1 ..... Signed-off-by: Yonghong Song <yhs@fb.com>
-
Yonghong Song authored
Currently, bcc uses its own version of bpf.h which tries to sync with upstream header regularly. If the host bpf.h version is lower, bcc can still compile as some bcc codes may requires a higher version of bpf.h. Such an approach does have a drawback. Suppose service A, statically linked with bcc, runs on kernel version X. Now, the kernel upgrades to version Y. After kernel upgrade/reboot, service A may not be able to compile since old bcc bpf.h may not align with the new kernel headers. For such cases, new version of service A needs rollout. This patch addresses this issue by attempting a second compilation using system bpf.h instead. The feature is not on by default. To enable it, pass -DBCC_BACKUP_COMPILE=1 in cmake setup stage. Signed-off-by: Yonghong Song <yhs@fb.com>
-
- 12 Oct, 2017 5 commits
-
-
Brendan Gregg authored
execsnoop: argument to change the number of arguments parsed
-
Edward Betts authored
-
Paul Chaignon authored
New argument to change the maximum number of arguments parsed and displayed.
-
Brendan Gregg authored
Fix 'tools/syscount' from using incorrect fallback values
-
4ast authored
sync src/cc/compat/linux headers with latest net-next
-
- 11 Oct, 2017 4 commits
-
-
Brendan Gregg authored
hardirqs, softirqs: Fix distribution mode units handling
-
Yonghong Song authored
Signed-off-by: Yonghong Song <yhs@fb.com>
-
Brendan Gregg authored
execsnoop: Fix -x handling
-
yonghong-song authored
Update SDT argument constraints
-
- 10 Oct, 2017 3 commits
-
-
yonghong-song authored
Fix segfault with enumerations
-
Sandipan Das authored
This prevents 'tools/syscount' from using incorrect system call descriptions if the 'ausyscall' command is not found. In this case, it uses a lookup table as a fallback. This, however, is compliant with x86_64 only. For now, we fix this by raising an exception and exiting if the 'ausyscall' executable is not available for non-x86_64 systems instead of having additional lookup tables for other architectures. Signed-off-by: Sandipan Das <sandipan@linux.vnet.ibm.com>
-
Sandipan Das authored
This sets the default USDT argument parser for all the architectures that currently do not implement one to the parser for x86_64. This is to avoid a compilation error about the architecture being not supported yet. Signed-off-by: Sandipan Das <sandipan@linux.vnet.ibm.com>
-
- 08 Oct, 2017 2 commits
-
-
Paul Chaignon authored
When serializing map types to JSON, if it encounters an enumeration, the rewriter goes into an infinite loop until it segfaults. This fix properly serializes enumerations in the same way unions and structs are. enum a { CHOICE_A, CHOICE_B, }; BPF_HASH(m, u32, enum a); is serialized as: ["a", ["CHOICE_A","CHOICE_B"], "enum"]
-
yonghong-song authored
Support resolve vDSO symbols
-
- 07 Oct, 2017 4 commits
-
-
Teng Qin authored
-
Teng Qin authored
Parse everything with one `fscanf` instead of using an extra `fgets`
-
Prashant Bhole authored
making it consistent with rest of the code
-
Prashant Bhole authored
sll_ifindex=0 matches any interface, which is not exepected here
-
- 06 Oct, 2017 4 commits
-
-
yonghong-song authored
Improve Kernel symbols loading
-
yonghong-song authored
Avoid potential SEGFAULT when resolving Kernel symbols
-
Brendan Gregg authored
Fix 'tools/statsnoop' from failing to attach kprobes
-
Teng Qin authored
-
- 05 Oct, 2017 4 commits
-
-
Teng Qin authored
-
Sandipan Das authored
This fixes 'tools/statsnoop' from failing to attach probes when the expected entry point for a system call cannot be found. This script uses the 'stat', 'statfs' and 'newstat' system calls, all of which must be implemented to be POSIX compliant. However, the names of the actual entry points for their respective implementations in the kernel might vary across architectures. For example, a powerpc64 kernel does not define 'sys_stat' but still provides the 'stat' system call via 'sys_newstat'. This causes the script to fail if it tries to attach a probe at 'sys_stat'. We avoid this by performing some extra checks to see if these entry points exist. Signed-off-by: Sandipan Das <sandipan@linux.vnet.ibm.com>
-
Sandipan Das authored
This updates the USDT argument constraint for powerpc and powerpc64 from 'nQr' to 'nZr'. The 'Q' memory constraint will allow a memory operand only if it is an offset from a register. The 'Z' memory constraint is similar but will additionally allow a memory operand that is an indexed or indirect from a register. This offers more flexibility. Signed-off-by: Sandipan Das <sandipan@linux.vnet.ibm.com>
-
Sandipan Das authored
This sets the USDT argument constraint for all architectures rather than just restricting it to powerpc and x86 variants. However, the other architectures might still need additional argument parsing support. Signed-off-by: Sandipan Das <sandipan@linux.vnet.ibm.com>
-
- 04 Oct, 2017 3 commits
-
-
yonghong-song authored
examples:dns_matching: make it work as a DNS sniffer
-
Prashant Bhole authored
Accepts arguments from user. This change makes it slightly more interactive. usage is show with -h option, so no extra documentation required for understanding the usage.
-
Prashant Bhole authored
Reason: The intention of initial version of this example was to provide a loop-uprolling example and expected functionality was to drop DNS packets requesting the DNS name contained in the map. But the functionality doesn't work as exepected because the BPF program attached to the raw socket only filters the packets received by the python program. With these modifications, it still serves as a loop-unrolling example, with slightly different functionality. Inverted return values of bpf program. It keeps the packet if the name in DNS packet is also exists in the map. All other packets are dropped. Python program is modified to read packets from raw socket. DNS data from the packet is parsed and printed using dnslib library.
-
- 03 Oct, 2017 2 commits
-
-
yonghong-song authored
Add basic USDT support for powerpc64
-
Yonghong Song authored
The patch adds a new debug option "DEBUG_SOURCE = 8" to dump insns embedded with source. In C++ API, users can change BPF constructor "flag" value to enable debug output. In Python API, users can change "debug" value to enable debug output. For example, for python test program test_usdt.py, the debug output looks like below: ...... Disassembly of section .bpf.fn.do_trace1: do_trace1: ; int do_trace1(struct pt_regs *ctx) { // Line 110 0: bf 16 00 00 00 00 00 00 r6 = r1 1: b7 01 00 00 00 00 00 00 r1 = 0 ; struct probe_result_t1 result = {}; // Line 111 2: 7b 1a f0 ff 00 00 00 00 *(u64 *)(r10 - 16) = r1 ; switch(ctx->ip) { // Line 5 3: 79 61 80 00 00 00 00 00 r1 = *(u64 *)(r6 + 128) 4: 15 01 04 00 d7 06 40 00 if r1 == 4196055 goto 4 5: 55 01 06 00 ce 06 40 00 if r1 != 4196046 goto 6 ; case 0x4006ceULL: *((int8_t *)dest) = ctx->ax; __asm__ __volatile__("": : :"memory"); return 0; // Line 6 6: 79 61 50 00 00 00 00 00 r1 = *(u64 *)(r6 + 80) ...... For asm insns, byte code is also dumped out (similar to objdump). For source codes, only lines in the module file are printed (as expected). The line number is added at the end of source code, which is especially helpful for inlined functions. This functionality is only in llvm 6.x (the trunk version), which provides an public interface to create a dwarf context based on a set of in-memory debug sections. llvm 5.x also provides such a public interface in a different way, and this patch does not support it in bcc yet. llvm 4.x and lower do not have such a public interface and hence will not be supported in bcc. In this patch, the debug output only goes to stderr. A subsequent patch will dump the per-function output into <BCC_PROG_TAG_DIR>/bpf_prog_<tag>/ if it is available. Signed-off-by: Yonghong Song <yhs@fb.com>
-
- 02 Oct, 2017 1 commit
-
-
Prashant Bhole authored
Makes it slightly easier to add more cache entries
-
- 01 Oct, 2017 1 commit
-
-
yonghong-song authored
examples: fix dns_matching
-
- 29 Sep, 2017 1 commit
-
-
Paul Chaignon authored
Partially reverts 80667b7b, "Fix unary operator handling of probe reads with parens", keeping the test case. With 4c6ecb46, "Restrict rewrite of unary operators to dereference operator," only dereferences are rewritten, removing the need for the previous fix. Reverting 80667b7b allows bcc to rewrite more dereferences, as highlighted in the new test case.
-
- 28 Sep, 2017 1 commit
-
-
Prashant Bhole authored
The for loop was copying garbase characters after domain name. Fixed this by adding correct terminating condition. Also removed unnecessary code Signed-off-by: Prashant Bhole <bhole_prashant_q7@lab.ntt.co.jp>
-