platform/upstream/bcc.git
7 years agoAdd TableStorage class for wrapping bpf map tracking
Brenden Blanco [Wed, 29 Mar 2017 16:58:31 +0000 (09:58 -0700)]
Add TableStorage class for wrapping bpf map tracking

Adds a TableStorage class for use by language frontends to store/access
references to loaded bpf maps. Includes support for shared and
namespaced maps, in a directory-like hierarchy.

Add a FileDesc helper class to automatically wrap open file descriptors.
The object prevents implicit copying of the fd (allows only
rvalue/move()), and takes care of the close() call.

Add a reference implementation of a TableStorageImpl that performs the
current default behavior expected by BPF_TABLE_PUBLIC, which is to share
maps between BPFModules in the same-process only. A stub implementation
for bpffs is started.

Update b/clang frontends to use this new class.

Also included is a framework for extracting the type information of maps
in an extensible way. Migrate BMapDeclVisitor to use this as the first
consumer.

Signed-off-by: Brenden Blanco <bblanco@gmail.com>
7 years agoMerge pull request #1090 from brendangregg/master
4ast [Mon, 3 Apr 2017 03:03:54 +0000 (20:03 -0700)]
Merge pull request #1090 from brendangregg/master

update tools diagram

7 years agoupdate tools diagram
Brendan Gregg [Mon, 3 Apr 2017 01:54:55 +0000 (18:54 -0700)]
update tools diagram

7 years agoMerge pull request #1078 from goldshtn/funcslower
Brendan Gregg [Sun, 2 Apr 2017 20:01:23 +0000 (13:01 -0700)]
Merge pull request #1078 from goldshtn/funcslower

funcslower: Trace slow kernel or user function calls

7 years agou* tools: per-language wrappers (#1086)
Paul Chaignon [Sun, 2 Apr 2017 19:57:13 +0000 (21:57 +0200)]
u* tools: per-language wrappers (#1086)

7 years agoMerge pull request #1079 from goldshtn/retire-stacksnoop
Brenden Blanco [Sun, 2 Apr 2017 17:08:12 +0000 (10:08 -0700)]
Merge pull request #1079 from goldshtn/retire-stacksnoop

stacksnoop: Retire and move to examples/tracing

7 years agofuncslower: Add to smoke tests
Sasha Goldshtein [Thu, 30 Mar 2017 09:08:09 +0000 (05:08 -0400)]
funcslower: Add to smoke tests

7 years agofuncslower: Add to README.md
Sasha Goldshtein [Thu, 30 Mar 2017 09:06:30 +0000 (05:06 -0400)]
funcslower: Add to README.md

7 years agofuncslower: Usage examples, annotated
Sasha Goldshtein [Thu, 30 Mar 2017 09:06:22 +0000 (05:06 -0400)]
funcslower: Usage examples, annotated

7 years agofuncslower: Man page
Sasha Goldshtein [Thu, 30 Mar 2017 09:06:07 +0000 (05:06 -0400)]
funcslower: Man page

7 years agofuncslower: Trace slow kernel or user function calls
Sasha Goldshtein [Thu, 30 Mar 2017 09:05:10 +0000 (05:05 -0400)]
funcslower: Trace slow kernel or user function calls

This tool attaches to entry and return points of specified
functions and traces function calls slower than a given threshold.

Resolves #915.

7 years agostacksnoop: Retire and move to examples/tracing
Sasha Goldshtein [Thu, 30 Mar 2017 10:32:23 +0000 (06:32 -0400)]
stacksnoop: Retire and move to examples/tracing

Move stacksnoop to examples/tracing. Originally we considered
leaving a script that redirects to the `trace` tool, but decided
not to. Any users of stacksnoop can migrate directly to `trace`.

Resolves #737.

7 years agomap.insert bcc helper to expose the BPF_NOEXIST flag (#1085)
Paul Chaignon [Sat, 1 Apr 2017 16:23:58 +0000 (18:23 +0200)]
map.insert bcc helper to expose the BPF_NOEXIST flag (#1085)

Inserts element in map only if it does not already exist. Throws a
warning during rewriter step if used on a BPF array.

7 years agoAdd an option to resolve address without demangling (#1084)
Teng Qin [Sat, 1 Apr 2017 15:00:23 +0000 (08:00 -0700)]
Add an option to resolve address without demangling (#1084)

* Add an option to resolve address without demangling

* Expose new no-demangling option to Python

7 years agoMerge pull request #1083 from palmtenor/fixdoc
4ast [Fri, 31 Mar 2017 04:37:41 +0000 (21:37 -0700)]
Merge pull request #1083 from palmtenor/fixdoc

Fix some documentation mistakes

7 years agofixdoc
Teng Qin [Fri, 31 Mar 2017 04:32:37 +0000 (21:32 -0700)]
fixdoc

7 years agotools: add tcptracer (#762)
Iago López Galeiras [Thu, 30 Mar 2017 17:07:47 +0000 (19:07 +0200)]
tools: add tcptracer (#762)

* tools: add tcptracer

This allows tracking TCP connections by tracking TCP connects, closes
and accepts.

This is different from existing tools like tcpconnect or tcpaccept in
that:

* It includes more information like network namespace or source ports
for tcpconnects or remote ports for tcpaccepts
* It traces tcp_close allowing to see when a connection ends
* It only shows information about established connections

* tcptracer: add to README

* tcptracer: add example

* tcptracer: add man page

7 years agou* tools: automatically detect the language (#1067)
Paul Chaignon [Thu, 30 Mar 2017 17:05:40 +0000 (19:05 +0200)]
u* tools: automatically detect the language (#1067)

* cc: bcc function to detect the language of a process

bcc_procutils_language looks into /proc/$pid/cmdline, /proc/$pid/exe,
and /proc/$pid/maps to determine the language.
Python wrapper takes a list of candidate languages; if the detected
language is not part of the list, None is returned.

* u* tools: automatically detect the language

Uses the detect_language bcc helper. -l switch can override the
detected language. In uthreads and ucalls, the language can be
overwritten to 'none' to trace pthreads and syscalls respectively.

All tools use the -l switch to set the language, for consistency.

7 years agoMerge pull request #1076 from palmtenor/helpermacro
Brenden Blanco [Thu, 30 Mar 2017 15:16:20 +0000 (08:16 -0700)]
Merge pull request #1076 from palmtenor/helpermacro

Improve helper Macros

7 years agobcc container improvements (#1051)
kmjohansen [Thu, 30 Mar 2017 07:58:31 +0000 (00:58 -0700)]
bcc container improvements (#1051)

* Bcc should look at mountns during symbol resolution.

Allow bcc to resolve symbols in processes that have mappings in a
different mount namespace.  This allows us to obtain stack traces from
the host when our target resides in a container.  With this change it's
possible to get stacks from targets that used to show up as unknown.

* When searching for perf-map files look in container, and then host.

Allow perf-map files to exist either in the container under the pid
that's specific to the container's pid namespace, or in the host
container using the pid that's specific to the initial pid namespace.
This lets us store the perf-map either in the continer or on the host,
depending upon which is easier for the person performing the debugging.

* Allow bcc to place uprobes and USDT probes in containers.

The uprobe/usdt mechanism uses the target's inode in order to determine
where to place the probe.  The inode lookup occurs at the time the file
path is written to uprobe_events.  If bpf_attach_uprobe() has been
passed a pid, and that pid is in a different mount namespace from the
caller, attempt to switch to the victim's mount namespace so that we can
select the correct inode for the probe.

* Add unit tests for the container improvements code.

7 years agoUpdate documentation for new helper Macros
Teng Qin [Wed, 29 Mar 2017 20:49:31 +0000 (13:49 -0700)]
Update documentation for new helper Macros

7 years agoUse new helper Macro across files
Teng Qin [Wed, 29 Mar 2017 20:39:17 +0000 (13:39 -0700)]
Use new helper Macro across files

7 years agoImprove BPF helper Macros
Teng Qin [Wed, 29 Mar 2017 20:23:48 +0000 (13:23 -0700)]
Improve BPF helper Macros

7 years agocc: define load_func and unload_func public (#1065)
Mauricio Vasquez B [Mon, 27 Mar 2017 18:16:26 +0000 (13:16 -0500)]
cc: define load_func and unload_func public (#1065)

With the current API it is not possible to load any kind of ebpf
program, for example BPF_PROG_TYPE_SCHED_CLS, this commit solves that by
defining load_func and unload_func as part of the public API.

Signed-off-by: Mauricio Vasquez B <mauricio.vasquez@polito.it>
7 years agoMerge pull request #1068 from goldshtn/syscount-python3
Brenden Blanco [Sun, 26 Mar 2017 19:04:05 +0000 (12:04 -0700)]
Merge pull request #1068 from goldshtn/syscount-python3

syscount: Use zip_longest for Python 3 compatibility

7 years agoMerge pull request #1069 from pchaigno/links
Brenden Blanco [Sun, 26 Mar 2017 18:01:07 +0000 (11:01 -0700)]
Merge pull request #1069 from pchaigno/links

LINKS.md

7 years agoMerge pull request #1066 from pchaigno/remove-import-netaddr
Brenden Blanco [Sun, 26 Mar 2017 18:00:12 +0000 (11:00 -0700)]
Merge pull request #1066 from pchaigno/remove-import-netaddr

solisten: replace netaddr with socket library

7 years agoCreate LINKS.md
Paul Chaignon [Sun, 26 Mar 2017 12:18:47 +0000 (14:18 +0200)]
Create LINKS.md

7 years agosyscount: Use zip_longest for Python 3 compatibility
Sasha Goldshtein [Sun, 26 Mar 2017 13:06:51 +0000 (13:06 +0000)]
syscount: Use zip_longest for Python 3 compatibility

7 years agoPython 3 compatibility fixes around string handling (#986)
Rafael F [Sun, 26 Mar 2017 12:54:25 +0000 (14:54 +0200)]
Python 3 compatibility fixes around string handling (#986)

This fixes the bcc module and all the affected tools for issues related to string handling in Python 3. Specifically, when passing Python strings to C libraries they are encoded as ASCII, and when constructing Python strings from C strings, they are decoded first.

7 years agoMerge pull request #1061 from lcp/kernel-docs
4ast [Sun, 26 Mar 2017 04:58:50 +0000 (21:58 -0700)]
Merge pull request #1061 from lcp/kernel-docs

docs: add the supplementary commits for LRU hashes

7 years agosolisten: replace netaddr with socket library
Paul Chaignon [Sat, 25 Mar 2017 12:40:46 +0000 (13:40 +0100)]
solisten: replace netaddr with socket library

7 years agodocs: add the supplementary commits for LRU hashes
Gary Lin [Fri, 24 Mar 2017 08:24:38 +0000 (16:24 +0800)]
docs: add the supplementary commits for LRU hashes

This commit addes two kernel commits for the LRU hashes to provide the
more information about LRU hashes. The original commits (29ba732acbee
and 8f8449384ec3) mark the inclusion of LRU hashes, but the commit
messages do not mention the details. The commits 3a08c2fd7634 and
961578b63474 actually implement the LRU list and provide the detailed
information about why LRU hash is created and how it works, and this can
help people to understand LRU hashes more quickly.

Signed-off-by: Gary Lin <glin@suse.com>
7 years agoMerge pull request #1060 from qmonnet/kernelfeatures
4ast [Thu, 23 Mar 2017 16:26:28 +0000 (09:26 -0700)]
Merge pull request #1060 from qmonnet/kernelfeatures

docs: update features list (new maps: array/hash of maps)

7 years agodocs: update features list (new maps: array/hash of maps)
Quentin Monnet [Thu, 23 Mar 2017 10:25:23 +0000 (11:25 +0100)]
docs: update features list (new maps: array/hash of maps)

7 years agoMerge pull request #1046 from pchaigno/check-fmt-printk
Brenden Blanco [Thu, 23 Mar 2017 03:23:05 +0000 (20:23 -0700)]
Merge pull request #1046 from pchaigno/check-fmt-printk

Verify format specifiers in bpf_trace_printk in rewriter

7 years agoMerge pull request #1059 from pchaigno/fix-readme-markdown
Brenden Blanco [Wed, 22 Mar 2017 20:55:21 +0000 (13:55 -0700)]
Merge pull request #1059 from pchaigno/fix-readme-markdown

Fix the Markdown list in the README

7 years agoMerge pull request #1058 from ColinIanKing/master
Brenden Blanco [Wed, 22 Mar 2017 13:58:00 +0000 (06:58 -0700)]
Merge pull request #1058 from ColinIanKing/master

snapcraft: add in some new tools missing from the snapcraft apps list

7 years agoFix the Markdown list in the README
Paul Chaignon [Wed, 22 Mar 2017 12:51:05 +0000 (13:51 +0100)]
Fix the Markdown list in the README

7 years agosnapcraft: add in some new tools missing from the snapcraft apps list
Colin Ian King [Wed, 22 Mar 2017 10:52:07 +0000 (10:52 +0000)]
snapcraft: add in some new tools missing from the snapcraft apps list

Add in bpflist and mysqld_qslower to apps list and re-order
dbstat in the list.

Signed-off-by: Colin Ian King <colin.king@canonical.com>
7 years agoVerify format specifiers in bpf_trace_printk in rewriter
Paul Chaignon [Sun, 12 Mar 2017 13:49:01 +0000 (14:49 +0100)]
Verify format specifiers in bpf_trace_printk in rewriter

Verifies format specifiers while rewriting calls to bpf_trace_printk
and prints a warning the printk will be rejected by the kernel at
runtime.

For tests, redirects stderr at the file descriptor level in order to
catch warnings from the C library.

7 years agoMerge pull request #1053 from palmtenor/bufferepoll
Brenden Blanco [Tue, 21 Mar 2017 14:28:49 +0000 (07:28 -0700)]
Merge pull request #1053 from palmtenor/bufferepoll

Use epoll in BPFPerfBuffer

7 years agoMerge pull request #1056 from goldshtn/bpflist-enh
Brenden Blanco [Tue, 21 Mar 2017 14:06:58 +0000 (07:06 -0700)]
Merge pull request #1056 from goldshtn/bpflist-enh

bpflist: Add to tests and use Python directory listing

7 years agoMerge pull request #1055 from goldshtn/syms-encode
Brenden Blanco [Tue, 21 Mar 2017 14:04:37 +0000 (07:04 -0700)]
Merge pull request #1055 from goldshtn/syms-encode

python: Allow module=None when resolving kernel symbols

7 years agotcplife for Lua (#1052)
Simon Liu [Tue, 21 Mar 2017 09:35:20 +0000 (04:35 -0500)]
tcplife for Lua (#1052)

7 years agoUse epoll in BPFPerfBuffer
Teng Qin [Sat, 18 Mar 2017 20:07:41 +0000 (13:07 -0700)]
Use epoll in BPFPerfBuffer

7 years agoClose all opened CPU perf buffer (not only currently online ones)
Teng Qin [Sat, 18 Mar 2017 20:13:02 +0000 (13:13 -0700)]
Close all opened CPU perf buffer (not only currently online ones)

7 years agoMerge pull request #1050 from palmtenor/exampleinstall
Brenden Blanco [Thu, 16 Mar 2017 01:33:24 +0000 (18:33 -0700)]
Merge pull request #1050 from palmtenor/exampleinstall

Add build option for installing C++ examples

7 years agoAdd build option for installing C++ examples
Teng Qin [Thu, 16 Mar 2017 00:50:11 +0000 (17:50 -0700)]
Add build option for installing C++ examples

7 years agoMerge pull request #1032 from goldshtn/tools-tests
Brenden Blanco [Sat, 11 Mar 2017 20:05:30 +0000 (12:05 -0800)]
Merge pull request #1032 from goldshtn/tools-tests

Smoke tests for the tools

7 years agotests: Add smoke tests for most tools
Sasha Goldshtein [Tue, 14 Feb 2017 01:09:05 +0000 (20:09 -0500)]
tests: Add smoke tests for most tools

This commit adds basic smoke tests for most tools in tools/ by
running the tool with either a short duration, or interrupting it
with a SIGINT after a short duration. The tests check the return
value from the tool to detect any Python exceptions or other
errors, but they do not read the standard error or standard output
and parse the tool's result.

Some tools are not covered by these smoke tests for reasons
documented in the test itself:

* btrfsdist and btrfsslower need btrfs
* cachetop doesn't like to run without a terminal
* dbslower, dbstat, and mysqld_qslower need a database engine
* deadlock_detector allocates a huge amount of memory
* softirqs doesn't work on new kernels and needs fixing (#1031)
* ugc needs a USDT-enabled runtime with GC probes
* zfsdist and zfsslower need zfs

This is a good place to start, but clearly for some tools,
especially those with a complex interface like trace and argdist,
we need more than just basic smoke tests.

7 years agotrace: Exit with nonzero return code on error
Sasha Goldshtein [Tue, 14 Feb 2017 01:25:32 +0000 (20:25 -0500)]
trace: Exit with nonzero return code on error

7 years agoMerge pull request #1044 from goldshtn/ausyscall
4ast [Sat, 11 Mar 2017 00:44:03 +0000 (16:44 -0800)]
Merge pull request #1044 from goldshtn/ausyscall

syscount: Use ausyscalls if available to get syscall list

7 years agoMerge pull request #1043 from goldshtn/bpflist
Brendan Gregg [Fri, 10 Mar 2017 20:55:04 +0000 (12:55 -0800)]
Merge pull request #1043 from goldshtn/bpflist

bpflist: Display processes with running BPF programs and maps

7 years agoMerge pull request #1039 from iovisor/tag_v0.3.0 v0.3.0
Brenden Blanco [Fri, 10 Mar 2017 02:07:20 +0000 (18:07 -0800)]
Merge pull request #1039 from iovisor/tag_v0.3.0

Prepare debian changelog for v0.3.0 tag

7 years agoPrepare debian changelog for v0.3.0 tag
Brenden Blanco [Thu, 9 Mar 2017 19:10:20 +0000 (11:10 -0800)]
Prepare debian changelog for v0.3.0 tag

Signed-off-by: Brenden Blanco <bblanco@gmail.com>
7 years agobpflist: Display processes with running BPF programs and maps
Sasha Goldshtein [Thu, 9 Mar 2017 14:37:50 +0000 (14:37 +0000)]
bpflist: Display processes with running BPF programs and maps

This tool displays processes with running BPF programs and maps,
and also optionally kprobes and uprobes. This is a poor-man's version
that snoops BPF file descriptors, as proposed by @brendangregg.

Example:

```
PID    COMM             TYPE     COUNT
4058   fileslower       prog     4
4058   fileslower       map      2
4106   bashreadline     map      1
4106   bashreadline     prog     1
```

Resolves #1036.

7 years agosyscount: Use ausyscalls if available to get syscall list
Sasha Goldshtein [Thu, 9 Mar 2017 13:43:27 +0000 (13:43 +0000)]
syscount: Use ausyscalls if available to get syscall list

If ausyscall is installed, it can provide a clean, up-to-date list of
syscall numbers for the current architecture. This is much more useful
than the default hardcoded list for x86-64, which is currently used by
syscount.

Try to run `ausyscall --dump` and parse the output before resorting to
the static list. Tested on FC/Linux 4.9 and produces 327 syscalls.

Resolves #1001.

7 years agoMerge pull request #1034 from brendangregg/master
4ast [Thu, 9 Mar 2017 00:27:38 +0000 (16:27 -0800)]
Merge pull request #1034 from brendangregg/master

funclatency: remove unnecessary include

7 years agoMerge pull request #1030 from zvonkok/zvonkok-s390x
4ast [Thu, 9 Mar 2017 00:25:05 +0000 (16:25 -0800)]
Merge pull request #1030 from zvonkok/zvonkok-s390x

Added s390x support. Needs 4.10 Kernel

7 years agofunclatency: remove unnecessary include
Brendan Gregg [Wed, 8 Mar 2017 19:17:00 +0000 (11:17 -0800)]
funclatency: remove unnecessary include

7 years agoAdded s390x support. Needs 4.10 Kernel
Zvonko Kosic [Tue, 7 Mar 2017 06:30:25 +0000 (07:30 +0100)]
Added s390x support. Needs 4.10 Kernel

7 years agoMerge pull request #1005 from pchaigno/rewrite-only-deref
Brenden Blanco [Tue, 7 Mar 2017 00:14:19 +0000 (16:14 -0800)]
Merge pull request #1005 from pchaigno/rewrite-only-deref

Restrict rewrite of unary operators to dereference operator

7 years agoRestrict rewrite of unary operators to dereference operator
Paul Chaignon [Thu, 23 Feb 2017 09:03:58 +0000 (10:03 +0100)]
Restrict rewrite of unary operators to dereference operator

Since the whole expression, unary operator included, is replaced by a
call to bpf_probe_read, the dereference operator is currently the
only unary operator properly rewritten. When rewriting an increment
expression (++val) for instance, the increment operator is lost in
translation.

Trying to rewrite all unary operators sometimes confuses bcc and
results in improper code, for instance when trying to rewrite a
logical negation.

7 years agoMerge pull request #1027 from fajs/debuild_no_parallel_tests
Brenden Blanco [Mon, 6 Mar 2017 18:58:19 +0000 (10:58 -0800)]
Merge pull request #1027 from fajs/debuild_no_parallel_tests

debuild: Do not parallelize tests

7 years agodebuild: Do not parallelize tests
Florian Schmidt [Mon, 6 Mar 2017 16:30:55 +0000 (17:30 +0100)]
debuild: Do not parallelize tests

The tests in the test suite are not parallelizable and will fail if run
in parallel. Make the test step non-parallel to fix this issue.

7 years agoMerge pull request #1006 from pchaigno/fix-bpf_dins_pkt-rewrite
Brenden Blanco [Mon, 6 Mar 2017 16:20:11 +0000 (08:20 -0800)]
Merge pull request #1006 from pchaigno/fix-bpf_dins_pkt-rewrite

Fix bpf_dins_pkt rewrite in BinaryOperator

7 years agoMerge pull request #1024 from r4f4/explicit-static
Brenden Blanco [Mon, 6 Mar 2017 16:16:11 +0000 (08:16 -0800)]
Merge pull request #1024 from r4f4/explicit-static

cmake: Explicitly mark static libraries as such

7 years agoMerge pull request #1023 from dneiter/filetop_sort
4ast [Sun, 5 Mar 2017 23:34:41 +0000 (15:34 -0800)]
Merge pull request #1023 from dneiter/filetop_sort

filetop: support specifying sort column via cmdline argument

7 years agocmake: Explicitly mark static libraries as such
Rafael Fonseca [Sun, 5 Mar 2017 17:09:54 +0000 (18:09 +0100)]
cmake: Explicitly mark static libraries as such

Some distros (e.g Fedora) override the default behaviour of building
static libraries to building dynamic ones instead. By explicitly
building the correct libraries as static, we make sure BCC properly
compiles everywhere.

7 years agoFix bpf_dins_pkt rewrite in BinaryOperator
Paul Chaignon [Thu, 23 Feb 2017 19:06:03 +0000 (20:06 +0100)]
Fix bpf_dins_pkt rewrite in BinaryOperator

Binary operator expressions where the left hand-side expression is a
reference to the packet are replaced by a call to the bpf_dins_pkt
helper. When replacing text, the Clang Rewriter tries to maintain a
list of offsets between the original and the new position of tokens.

Replacing the whole binary operator expression with the call to
bpf_dins_pkt confuses the Rewriter and it is unable to track the new
position of the right hand-side expression. Rewriting the binary
operator expression in two times without rewriting the right
hand-side expression itself solves the issue.

7 years agoTravis CI build to check compliance with PEP8 (#987)
Paul Chaignon [Sat, 4 Mar 2017 19:07:56 +0000 (20:07 +0100)]
Travis CI build to check compliance with PEP8 (#987)

* Travis CI build to check compliance with PEP8

* argdist: linter cleanup

* dbslower: linter cleanup

* dbstat: linter cleanup

* memleak: linter cleanup

* syscount: linter cleanup

* tplist: linter cleanup

* trace: linter cleanup

* ucalls: linter cleanup

* uflow: linter cleanup

* ugc: linter cleanup

* uobjnew: linter cleanup

* ustat: linter cleanup

7 years agooffwaketime.py for older kernels (#977)
Brendan Gregg [Sat, 4 Mar 2017 08:31:58 +0000 (00:31 -0800)]
offwaketime.py for older kernels (#977)

7 years agoMerge pull request #1021 from markdrayton/sym-mod-fix
4ast [Sat, 4 Mar 2017 05:27:16 +0000 (21:27 -0800)]
Merge pull request #1021 from markdrayton/sym-mod-fix

python: handle null module in BPF.sym

7 years agoMerge pull request #1022 from goldshtn/syms-multiple-regions
4ast [Sat, 4 Mar 2017 05:26:11 +0000 (21:26 -0800)]
Merge pull request #1022 from goldshtn/syms-multiple-regions

Symbol resolution with multiple executable regions per module

7 years agoMerge pull request #1018 from GabrielGanne/python-xdp-return-values
Brenden Blanco [Fri, 3 Mar 2017 20:42:04 +0000 (12:42 -0800)]
Merge pull request #1018 from GabrielGanne/python-xdp-return-values

add XDP return values to python interface

7 years agocc: Symbol resolution with multiple executable regions per module
Sasha Goldshtein [Fri, 3 Mar 2017 20:13:31 +0000 (15:13 -0500)]
cc: Symbol resolution with multiple executable regions per module

The symbol resolution code used to assume for most purposes that
there is a single executable region per module. When there were
several, there was no crash, but symbols were not resolved correctly.
The reason is that the symbol offsets are relative to the first
executable region's start address, but bcc would resolve them
relative to the region in which they appeared. For example, given
the following regions and spans for a module libfoo.so loaded into
some process:

  1000-2000 r-xp libfoo.so
  2000-3000 rw-p libfoo.so
  3000-4000 r-xp libfoo.so
  4000-5000 r--- libfoo.so

Now, suppose there is a symbol bar() loaded at address 3500. In
the binary on disk, bar() is at offset 2500 from the beginning of
the module (but not the beginning of the 3000-4000 region!). When
we look at the candidate regions, we find 3000-4000, and discover
that 3500 lies within it. Then we subtract 3500-3000 to find the
offset from the beginning of the region, get 500, and now look
for a symbol that contains the relative address 500. As a result,
we might find some random symbol in the region 1000-2000, and
report that address 3500 corresponds to that random symbol rather
than to bar().

This commit fixes the situation by keeping only a single `Module`
instance for each module, even if that module spans multiple
executable regions. We remember all executable region start and
end ranges so we can determine whether an address (like 3500 in
the above example) lies within the module. But for the purpose of
finding the actual symbol, we need only the offset from the start
of the _first_ executable region, and then need to look up a symbol
based on that.

This was discovered and fixed while tracing .NET Core processes on
Linux, where libcoreclr.so (the main CLR binary) has several
executable regions. Resolving symbols from any but the first region
would produce totally bogus results.

7 years agocc: Fix assertion for debug builds
Sasha Goldshtein [Fri, 3 Mar 2017 20:13:16 +0000 (15:13 -0500)]
cc: Fix assertion for debug builds

7 years agorange Python 2 -> 3 compatibility (#983)
Rafael F [Fri, 3 Mar 2017 18:57:28 +0000 (19:57 +0100)]
range Python 2 -> 3 compatibility (#983)

7 years agousdt: fix argument passing on python3 (#984)
Rafael F [Fri, 3 Mar 2017 18:56:50 +0000 (19:56 +0100)]
usdt: fix argument passing on python3 (#984)

This fixes the following error:

$: ./tplist -v -v -l /usr/lib64/dri/i965_dri.so
argument 1: <class 'TypeError'>: wrong type

7 years agoUpdate tutorial_bcc_python_developer.md (#1017)
Alan Thompson [Fri, 3 Mar 2017 18:55:42 +0000 (12:55 -0600)]
Update tutorial_bcc_python_developer.md (#1017)

small typo

7 years agoMerge pull request #1020 from goldshtn/duplicate_modules
Brenden Blanco [Fri, 3 Mar 2017 18:54:13 +0000 (10:54 -0800)]
Merge pull request #1020 from goldshtn/duplicate_modules

cc: Don't parse the same module multiple times for USDT probes

7 years agocc: Don't parse the same module multiple times for USDT probes
Sasha Goldshtein [Fri, 3 Mar 2017 13:00:55 +0000 (08:00 -0500)]
cc: Don't parse the same module multiple times for USDT probes

If a module has more than one executable region, it is reported
multiple times by `bcc_procutils_each_module`. This is fine for
symbol resolution, but we don't need the duplicates for parsing
the ELF header looking for USDT probes: the first appearance of
that module is enough. This also prevents issues with the same
probe appearing multiple times with the same location, which
results in an invalid program when reading USDT arguments.

Fix by storing each visited module in the USDT::Context class,
and ignoring modules that were already visited.

7 years agoadd XDP return values to python interface
Gabriel Ganne [Mon, 27 Feb 2017 18:33:49 +0000 (19:33 +0100)]
add XDP return values to python interface

Signed-off-by: Gabriel Ganne <gabriel.ganne@enea.com>
Signed-off-by: Romain Ly <romain.ly@enea.com>
7 years agopython: handle null module in BPF.sym
Mark Drayton [Thu, 2 Mar 2017 11:41:31 +0000 (03:41 -0800)]
python: handle null module in BPF.sym

Check to see if `module` is None before attempting to call `os.path.basename`
on it. Before:

```
>>> BPF.sym(0x400001, 12345, show_module=True)
Traceback (most recent call last):
..
AttributeError: 'NoneType' object has no attribute 'rfind'
```

After:

```
>>> BPF.sym(0x400001, 12345, show_module=True)
'[unknown]'
```

7 years agofiletop: support specifying sort column via cmdline argument
Daniel Neiter [Thu, 2 Mar 2017 01:21:25 +0000 (17:21 -0800)]
filetop: support specifying sort column via cmdline argument

7 years agoMerge pull request #1015 from goldshtn/perfmap-retry
4ast [Thu, 2 Mar 2017 00:47:28 +0000 (16:47 -0800)]
Merge pull request #1015 from goldshtn/perfmap-retry

cc: Retry symbol resolution using perfmap

7 years agoMerge pull request #1002 from goldshtn/nested-symbols
4ast [Wed, 1 Mar 2017 18:08:45 +0000 (10:08 -0800)]
Merge pull request #1002 from goldshtn/nested-symbols

cc: Handle nested functions correctly when resolving symbols

7 years agoMerge pull request #1012 from goldshtn/buildid-fix
4ast [Tue, 28 Feb 2017 23:46:45 +0000 (15:46 -0800)]
Merge pull request #1012 from goldshtn/buildid-fix

cc: Fix SEGV when there is no build-id section

7 years agoMerge pull request #1014 from iovisor/test-debuginfo-fix
4ast [Tue, 28 Feb 2017 23:14:50 +0000 (15:14 -0800)]
Merge pull request #1014 from iovisor/test-debuginfo-fix

Fix long running test_debuginfo and python3 fix

7 years agoFix long running test_debuginfo and python3 fix
Brenden Blanco [Tue, 28 Feb 2017 20:22:42 +0000 (12:22 -0800)]
Fix long running test_debuginfo and python3 fix

Make sure subclass calls super().tearDown to clean up dummy process.
Also, fixup a python3 str.encode().

Fixes: #1013
Signed-off-by: Brenden Blanco <bblanco@gmail.com>
7 years agocc: Retry symbol resolution using perfmap
Sasha Goldshtein [Tue, 28 Feb 2017 19:16:49 +0000 (14:16 -0500)]
cc: Retry symbol resolution using perfmap

When a symbol lies within a module, and that module doesn't have
debuginfo (or doesn't even have an ELF header), the symbol will
always be resolved as [unknown]. However, the /tmp/perf-PID.map
(perf map) for that process might actually have an entry for that
symbol, if it was dynamically generated by some external tool.
This commit changes the resolution process such that if the desired
address lies in a module but that module doesn't have debuginfo,
we keep trying to resolve it in subsequent modules (including the
perf map). If we resolve it successfully using the perf map, the
reported symbol information will have the original module's name,
so we don't lose fidelity.

The motivation for this change is the way symbols work with .NET
Core on Linux. The runtime binaries are compiled ahead-of-time to
native code, but do not have debuginfo. There is an external tool,
which generates a file similar to a perf map (albeit with relative
addresses) for these binaries. This file can then be merged into
the main perf map for the process and used for symbol resolution,
but only if we keep trying to use the perf map when the symbol is
in a previously-seen module.

7 years agoMerge pull request #997 from markdrayton/perf-buffer-size
Brenden Blanco [Tue, 28 Feb 2017 16:53:41 +0000 (08:53 -0800)]
Merge pull request #997 from markdrayton/perf-buffer-size

Make perf ring buffer size configurable

7 years agoMake perf ring buffer size configurable
Mark Drayton [Mon, 20 Feb 2017 18:13:03 +0000 (18:13 +0000)]
Make perf ring buffer size configurable

As discussed in #966, this PR makes the size of the ring buffer used to send
data to userspace configurable. It changes the Python, Lua and C++ APIs to
expose this knob.

It also defaults the buffer size to a larger value (64 pages per CPU, an 8x
increase) for several tools which produce a lot of output, as well as making it
configurable in `trace` via a `-b` flag.

7 years agoMerge pull request #1003 from ColinIanKing/master
Brenden Blanco [Sun, 26 Feb 2017 22:32:25 +0000 (14:32 -0800)]
Merge pull request #1003 from ColinIanKing/master

snapcraft: add in some new tools missing from the snapcraft apps list

7 years agoMerge pull request #1008 from pchaigno/support-macro
Brenden Blanco [Sun, 26 Feb 2017 21:58:58 +0000 (13:58 -0800)]
Merge pull request #1008 from pchaigno/support-macro

Support for macros in rewriter

7 years agoSupport for macros in rewriter
Paul Chaignon [Sun, 26 Feb 2017 11:22:03 +0000 (12:22 +0100)]
Support for macros in rewriter

Extends the scope of 98b90974625's fix to support macros anywhere in the
rewriter. All SourceRange objects are replaced to use macro expanded locations.

7 years agoMerge pull request #994 from qmonnet/kernelfeatures
4ast [Thu, 23 Feb 2017 17:42:20 +0000 (09:42 -0800)]
Merge pull request #994 from qmonnet/kernelfeatures

docs: Update eBPF features list

7 years agodocs: Update eBPF features list
Quentin Monnet [Tue, 21 Feb 2017 11:58:33 +0000 (12:58 +0100)]
docs: Update eBPF features list

Update of BPF features list, following the release of kernel 4.10.

7 years agosnapcraft: add in some new tools missing from the snapcraft apps list
Colin Ian King [Thu, 23 Feb 2017 11:49:41 +0000 (11:49 +0000)]
snapcraft: add in some new tools missing from the snapcraft apps list

Add in syscount, dbstat and dbslower to apps list.

Signed-off-by: Colin Ian King <colin.king@canonical.com>