Merge pull request #198 from iovisor/bblanco_dev

Autoload kprobes for all types of trace_* functions

Typo in retprobe case of _trace_autoload

Signed-off-by: Brenden Blanco <bblanco@plumgrid.com>

Autoload kprobes for all types of trace_* functions

The previous patch #195 for autoloading of kprobes only did it for
trace_print. Turn this feature on for all trace_* functions. This
requires that these functions are also no longer staticmethods.

Enable the feature in examples/disksnoop.py

Signed-off-by: Brenden Blanco <bblanco@plumgrid.com>

Merge pull request #197 from iovisor/bblanco_dev

Fix breakage in bpf_probe_read from #196

Fix breakage in bpf_probe_read from #196

Argument needs to be cast to u64, otherwise it is adding a whole pointer
stride.

Signed-off-by: Brenden Blanco <bblanco@plumgrid.com>

Merge pull request #196 from iovisor/bblanco_dev2

Add debug and fix the inline replace of kprobe args

Fix the inline replace of kprobe args

The way in which args 1+ were being replaced in the C file was
fragile. Instead, assign the registers from ptregs into the function
arguments as the first statement(s) in the body of the function.
e.g.:
int sys_clone(struct ptregs *ctx, struct request *req) {
  // do something with req
}
becomes:
int sys_clone(struct ptregs *ctx, struct request *req) {
  req = ctx->di;
  // do something with req

Fixes: #192
Signed-off-by: Brenden Blanco <bblanco@plumgrid.com>

Add debug flag for printing rewritten C text

* Many times it is useful to print out the C file after the
  BFrontendAction has run.
  e.g.: BPF("file.c", debug=0x4)

Signed-off-by: Brenden Blanco <bblanco@plumgrid.com>

Merge pull request #195 from iovisor/bblanco_dev

Change auto-loading behavior of trace_print

Denote auto-loading with k[ret]probe__ prefix

Since kprobe functions will have a different prototype than the kernel
symbols they are attaching to, require that the user prefix the trace
function with a kprobe__ name to denote intent. kretprobe__ prefix is
also supported.

Signed-off-by: Brenden Blanco <bblanco@plumgrid.com>

Change auto-loading behavior of trace_print

* As @brendangregg pointed out, users will probably assume that
  handily-named C functions that can be auto-loaded will be all the
  time, rather than just in the singleton case. This is pretty easy to
  implement, so changing the behavior.

Signed-off-by: Brenden Blanco <bblanco@plumgrid.com>

Merge pull request #194 from iovisor/bblanco_dev

Support automatic kprobe event detection in common case

Support automatic kprobe event detection in common case

* In the simple case, a user only creates 1 C function to be used with
  kprobes. Detect this common case and don't require the user to repeat
  themselves by passing the fn_name to attach_kprobe().
  e.g.: BPF(text='int sys_clone(void *ctx) {/*do stuff*/}').trace_print()

Signed-off-by: Brenden Blanco <bblanco@plumgrid.com>

Merge pull request #189 from iovisor/bblanco_dev

Suppress None return when trace_pipe drops lines

Suppress None return when trace_pipe drops lines

* When a "CPU: X Lost N events" line came on the trace_pipe,
  trace_fields would return None and cause exceptions in callers that do
  (a, b, ...) = b.trace_fields() type of calls. Instead, keep reading
  from trace_pipe when such messages come.

Fixes: #187
Signed-off-by: Brenden Blanco <bblanco@plumgrid.com>

Merge pull request #184 from iovisor/bblanco_dev

Add BPF_HASH macro with variadic arguments

Merge pull request #185 from brendangregg/master

print_log2_hist() for #143

shorten hello world example

no longer need the clear()

make bitehist a simpler example of log2 histograms

add print_log2_hist() from #143

Merge remote-tracking branch 'upstream/master'

Shorten trace_readline_fields to trace_fields

* Per suggestion in #149

Signed-off-by: Brenden Blanco <bblanco@plumgrid.com>

Add BPF_HASH macro with variadic arguments

* Usage: BPF_HASH(tablename, key_type=u64, leaf_type=u64)
  2nd and 3rd arguments are optional in the C++ default argument style

Fixes: #135
Signed-off-by: Brenden Blanco <bblanco@plumgrid.com>

Merge pull request #183 from iovisor/bblanco_dev

Shorten trace_readline_fields to trace_fields

Shorten trace_readline_fields to trace_fields

* Per suggestion in #149

Signed-off-by: Brenden Blanco <bblanco@plumgrid.com>

Merge pull request #182 from brendangregg/master

simplify code using new features

simplify code using new features

Merge pull request #181 from iovisor/bblanco_dev

Reorganize cmake, some cleanups and test fixes.

The use of schedule+<offset> was unstable, remove it

* After a kernel update, the binary of schedule function changed offset,
  and the attachment point updated. Since this is just a test case, pick
  a different function to test.

Signed-off-by: Brenden Blanco <bblanco@plumgrid.com>

Remove std::exception code from exception.h

* The c++ exception code was unused, now it is just c macro style
  "exceptions"

Signed-off-by: Brenden Blanco <bblanco@plumgrid.com>

Reorganize cmake files into dedicated cmake directory

Signed-off-by: Brenden Blanco <bblanco@plumgrid.com>

Merge pull request #180 from iovisor/bblanco_dev

Update README.md and INSTALL.md, specfile dependencies

Some more minor updates to INSTALL and README

Signed-off-by: Brenden Blanco <bblanco@plumgrid.com>

Add spec and build script used in fedora buildbot

* These files are required to build the test rpm

Signed-off-by: Brenden Blanco <bblanco@plumgrid.com>

Minor updates to install doc and add dep to spec files

* Add some runtime dependencies in INSTALL.md
* make and gcc are required at runtime, so add them to libbcc Requires:

Signed-off-by: Brenden Blanco <bblanco@plumgrid.com>

Updates to README and INSTALL documents

* Some changes to python API examples
* Reflect non-rc kernel availability in install doc
* Add binary rpm section to install doc
* Remove docker build instructions

Signed-off-by: Brenden Blanco <bblanco@plumgrid.com>

Merge pull request #178 from iovisor/llvm_fixes

Fixes for when using clang/llvm as CC/CXX

Merge pull request #177 from iovisor/bblanco_dev

Enhance check for presence of static-libstdc++

Workaround for uninitialized union clang bug

* Clang does not generate code to initialize the entire union when
  initializing a nested struct. This is the case for union bpf_attr.
  As a workaround, call memset explicitly.

Signed-off-by: Brenden Blanco <bblanco@plumgrid.com>

Fix compiler warnings when using clang[++]

Signed-off-by: Brenden Blanco <bblanco@plumgrid.com>

Enhance check for presence of static-libstdc++

* Fixes a link error on Ubuntu when static-libstdc++ is not installed.

Reported-by: Denis V. Lunev <den@openvz.org>
Signed-off-by: Brenden Blanco <bblanco@plumgrid.com>

Merge pull request #176 from iovisor/bblanco_dev

Include missed spec files in s/bpf/bcc/ rename

Include missed spec files in s/bpf/bcc/ rename

Signed-off-by: Brenden Blanco <bblanco@plumgrid.com>

Merge pull request #175 from iovisor/bblanco_dev

Rename python module name to 'bcc' from 'bpf'

Rename python module name to 'bcc' from 'bpf'

* Rename python module to bcc
* Rename python-bpf (deb,rpm) package to python-bcc
* Pending this change, I will likely re-tag 0.1.6, and upload the bcc
  package to pypi.python.org

Signed-off-by: Brenden Blanco <bblanco@plumgrid.com>

Merge pull request #174 from iovisor/bblanco_dev

Update tag to v0.1.6, include finall llvm 3.7.0 release

Add debian builder script that is versioned

* This script can be run by buildbot to generate the debian files
e.g.: PARALLEL=4 scripts/build-deb.sh

Signed-off-by: Brenden Blanco <bblanco@plumgrid.com>

Fix debian changelog formatting

Signed-off-by: Brenden Blanco <bblanco@plumgrid.com>

Update tag to v0.1.6, include finall llvm 3.7.0 release

* Update llvm to 3.7.0 release

Signed-off-by: Brenden Blanco <bblanco@plumgrid.com>

Merge pull request #173 from iovisor/bblanco_dev

Add RPM packaging for el6,el7,fedora22

Add RPM packaging for el6,el7,fedora22

Signed-off-by: Brenden Blanco <bblanco@plumgrid.com>

Merge pull request #171 from iovisor/bblanco_dev

Minor change to task_switch example syntax

Merge pull request #172 from brendangregg/master

initial ksym() and ksymaddr()

Merge remote-tracking branch 'upstream/master'

Minor change to task_switch example syntax

Signed-off-by: Brenden Blanco <bblanco@plumgrid.com>

Merge pull request #170 from iovisor/bblanco_dev

Add beta RPM spec and docker script for centos

Add another spec dependency

Testing buildbot

Signed-off-by: Brenden Blanco <bblanco@plumgrid.com>

Add one dependency to spec

Also, testing buildbot with this tiny change

Signed-off-by: Brenden Blanco <bblanco@plumgrid.com>

make use of BPF.ksym()

provide ksym() and ksymaddr()

use new event_re option to improve vfscount

Add beta RPM spec and docker script for centos

Signed-off-by: Brenden Blanco <bblanco@plumgrid.com>

Merge pull request #168 from iovisor/weichunc_dev

Add gretap support for full mesh tunnel

Add gretap support for full mesh tunnel

Signed-off-by: Wei-Chun Chao <weichunc@plumgrid.com>

Merge pull request #166 from iovisor/bblanco_dev

Add proper debian build support, rename libbcc.so

Merge pull request #167 from brendangregg/master

improve examples using new features

improve code by utilizing trace_readline_fields()

improve code by utilizing .clear()

Support versioning out of git tree in debuild

This adds support for properly tagging the build when cmake is run not
in a git tree, which is the case when building from src-deb.

Signed-off-by: Brenden Blanco <bblanco@plumgrid.com>

Add proper debian build support

The cmake based build system is nice, but not quite production ready.
Move to the more painful but featureful debuild style. This is one step
towards an upstreamable package.

Rename libbpfprog (ugh) to libbcc.

Split out python-bpf and libbcc-examples into separate packages that
depend on libbcc.

Signed-off-by: Brenden Blanco <bblanco@plumgrid.com>

Merge pull request #165 from iovisor/bblanco_dev

Fix print() breakage in python 2.7 environments

Fix print() breakage in python 2.7 environments

Fixes: #164
Signed-off-by: Brenden Blanco <bblanco@plumgrid.com>

Merge pull request #163 from iovisor/bblanco_dev

Add debug flag to enable prints from kernel verifier

test_trace4 is unreliable, use a different test regex

Signed-off-by: Brenden Blanco <bblanco@plumgrid.com>

Add debug flag to enable prints from kernel verifier

Add flag to BPF() to enable debug prints from the verifier. Use by
passing 0x2 to debug= in the constructor. Add docstring for BPF
constructor.

Also, optimize bpf_prog_load so that by default the kernel is not
asked to print the verbose output. The program will be "loaded" a second
time to capture the verifier failure message.

Fixes: #147
Signed-off-by: Brenden Blanco <bblanco@plumgrid.com>

Merge pull request #162 from iovisor/bblanco_dev

Add regex support to attach_k[ret]probe

Add regex support to attach_k[ret]probe

Add new event_re parameter to kprobe functions. This searches through
the list of functions/symbols in <tracefs>/available_filter_functions
for all matching functions. Every one is attached.

Add a test case for attach_k[ret]probe.

Fixes: #141
Signed-off-by: Brenden Blanco <bblanco@plumgrid.com>

Merge pull request #161 from svinota/master

examples: use IPDB `wait_ip()` feature

examples: use IPDB `wait_ip()` feature

Merge pull request #160 from iovisor/bblanco_dev

Add field support to trace_print and friends

Add field support to trace_print and friends

Add trace_readline_fields helper to parse the output of trace_pipe
Add field parsing support to trace_print. Addresses #149.
Fix typo in trace_open s/trace/tracefile/
Make nonblocking=False the default in trace_readline
Use IOError vs BlockingIOError for greater compatibility (untested)

Fixes: #149
Signed-off-by: Brenden Blanco <bblanco@plumgrid.com>

Merge pull request #158 from iovisor/bblanco_dev

Fix map.clear() usage for array type maps

Fix map.clear() usage for array type maps

Calling delete on an array type map entry does not have an effect.
Instead, the entry needs to be zeroed out, since the array slot always
exists. To avoid unnecessary calls to update(), only call update() when
the map type is array-like. The type was not exposed to python up until
now, so add it.

Signed-off-by: Brenden Blanco <bblanco@plumgrid.com>

Merge pull request #157 from iovisor/bblanco_dev

Add test case for clearing of maps from python

Add test case for clearing of maps from python

Fixes: #142
Signed-off-by: Brenden Blanco <bblanco@plumgrid.com>

Merge pull request #156 from iovisor/bblanco_dev

Add BPF python helpers for reading trace_pipe

Add BPF python helpers for reading trace_pipe

Signed-off-by: Brenden Blanco <bblanco@plumgrid.com>

Merge pull request #153 from brendangregg/master

more examples of b[] syntax, fix func arg passing

tidier example

more examples of b[] syntax, fix func arg passing

Merge pull request #152 from iovisor/bblanco_dev

Add getitem api to BPF object to reference tables

Add getitem api to BPF object to reference tables

To simplify some common usage patterns, add a getitem api to the BPF
class. This wraps the get_table() api and allows the api user to skip
keeping their own get_table handle.

Fixes: #137
Signed-off-by: Brenden Blanco <bblanco@plumgrid.com>

Merge pull request #151 from iovisor/bblanco_dev

Change API of attach_kprobe to take a name argument

Change API of attach_kprobe to take a name argument

Per feedback on the attach_kprobe api, change up the arguments to remove
the load_func that typically preceeds the call. Instead, move this
inside the attach_kprobe implementation. Also, this makes attach_kprobe
need to be non-static. The same applies to attach_kretprobe.

Old:
fn = b.load_func("hello", BPF.KPROBE)
BPF.attach_kprobe(fn, "sys_clone")

New:
b.attach_kprobe(event="sys_clone", fn_name="hello")

Note that the kwarg style is not required, but I fixed up the current
usages to provide readability.

Signed-off-by: Brenden Blanco <bblanco@plumgrid.com>

Merge pull request #150 from brendangregg/master

some bcc examples and tools

save some syscalls

python 2 & 3 compatibility

support odd Linux distros

cleaner get_table calls