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

include paths

some bcc examples and tools

Merge pull request #134 from iovisor/bblanco_dev

Add format string argument to bpf-run

Add format string argument to bpf-run

Usage:
bpf-run -f "time={3} output={5}"

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

Merge pull request #133 from iovisor/bblanco_dev

Add bpf-run command line tool

Add bpf-run command line tool

Example usage:
sudo bpf-run -p sys_clone -c 'bpf_trace_printk("Hello, World!\n");' -t

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

Merge pull request #132 from iovisor/yhs_dev

add dhcp support for tunnel-mesh

add dhcp support for tunnel-mesh

Signed-off-by: Yonghong Song <yhs@plumgrid.com>

Merge pull request #130 from iovisor/yhs_dev

add vlan_learning and distributed_bridge in libbcc package

Merge pull request #131 from iovisor/weichunc_dev

distributed_bridge: check ip address before validating traffic

distributed_bridge: check ip address before validating traffic

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

add vlan_learning and distributed_bridge in libbcc package

Signed-off-by: Yonghong Song <yhs@plumgrid.com>

Merge pull request #129 from iovisor/weichunc_dev

distributed_bridge: wait for ns comes up before validating traffic

distributed_bridge: wait for ns comes up before validating traffic

Sometimes it takes longer for ns to complete configuration.

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

Merge pull request #128 from iovisor/weichunc_dev

test_brb2: only account for IP and ARP packets

Merge pull request #127 from iovisor/bblanco_dev

Changes to support map entry read/write in fuse

test_brb2: only account for IP and ARP packets

Sometimes background noise fails the test

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

Add testing for scanf/printf in python, deprecate table_update

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

Add sscanf C api for parsing key/leaf ascii to binary

Expose an individual API of the what is done in table_update, that lets
the caller use the module to parse keys for it.

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

Merge pull request #126 from iovisor/yhs_dev

clean up potential conflict veth before creating new one in create_ns

close ipr to avoid fd leak

Signed-off-by: Yonghong Song <yhs@plumgrid.com>

use pyroute2 IProute to remove the stale veth

Signed-off-by: Yonghong Song <yhs@plumgrid.com>

Merge pull request #125 from iovisor/bblanco_dev

Add printf for key/leaf and fix trace_printk bug

Support nested rewrite of trace arguments in bpf_trace_printk

There was an issue where the rewrite of bpf_trace_printk combined with
conversion of function argument to ctx->$reg was mangling the text. Fix
up this case and add a test.

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

Add printf key/leaf writer functionality

This extends upon the sscanf reader functionality, with the intent of
providing key/leaf pretty printing

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

Merge pull request #123 from iovisor/bblanco_dev

Fix recursive loop in parsing pointer to self struct

Fix recursive loop in parsing pointer to self struct

Issue occurs in the description visitor class, when the struct used in a
map key is a pointer to self, as in:

struct node;
struct node {
  struct node *next;
};

Avoid this in the desc by using "unsigned long long" for all pointers.

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

clean up potential conflict veth before creating new one in create_ns

Signed-off-by: Yonghong Song <yhs@plumgrid.com>

Merge pull request #122 from iovisor/yhs_dev2

Fixing more ctrl-c problem

Merge pull request #121 from iovisor/weichunc_dev

add connectivity validation in distributed bridge examples

Merge pull request #120 from iovisor/yhs_dev

Add one line desc output for simple_tc

add connectivity validation in distributed bridge examples

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

Fixing more ctrl-c problem

Signed-off-by: Yonghong Song <yhs@plumgrid.com>

print out simple_tc success at the end of test

Signed-off-by: Yonghong Song <yhs@plumgrid.com>

Add one line desc output for simple_tc

  o Also suggest task_switch.py to try with libbcc package

Signed-off-by: Yonghong Song <yhs@plumgrid.com>

Merge pull request #119 from iovisor/bblanco_dev

Patch series: reorganize bpf_module and add sscanf feature for fuse

Cleanup some flags left on during development

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

Add update_table API: accepts sscanf style strings and populates a map

This is the culmination of the previous patches. It adds an api that can
input map data in a string format, but validating the numbers and
locations of data fields. The use case is for fuse file input/output. A
printf api may follow.

Take the table with key/leaf of:
  struct Key { int a; int b; };
  struct Leaf { int a; int b; int c; struct SubLeaf { int x; int y; } s; };

One would input to this table using:
  update_table(table_name, "{1 2}", "{1 2 -3 {9 0xa}}");

The implementation uses a JITed function for each unique type, that is
invoked to run sscanf on behalf of the caller. The input must have the
exact right number of arguments. Bit fields are supported, but the
caller must be aware of the collapse of those bitfields into an aligned
field, as well as endianness.

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

Convert bpf_module structures to use vector instead of map for tables

It was bothering me that a whole bunch of accesses to the tables were
based on a string hash lookup, in addition to the vectors that were kept
to convert ids to names. Switch those around, as well as the api
internally.

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

Remove some deprecated checks in bpf_module

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

Add LLVM IR based sscanf routine, to be run by the JIT

After the modules have been created, create a helper function for each
table leaf/key type.

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

Rename BPFTable to TableDesc

This whole project is about bpf, prefixing everything with the same
acronym is redundant.

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

Extract table types from B programs as well

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

Add framework to support map string reader (fuse feature)

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

Merge pull request #118 from iovisor/bblanco_dev

Add cmake checks for required test programs

Change wording to "Recommended"

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

Add cmake checks for required test programs

Set as warning, since the build will still work fine and be able to
create a binary package.

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

Merge pull request #117 from iovisor/bblanco_dev

Move helper.h prototype for tail call into 4.2 section

Move helper.h prototype for tail call into 4.2 section

With #115, bpf_tail_call was incorrectly disabled in 4.2 builds.

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

Merge pull request #116 from iovisor/bblanco_dev

Add INSTALL.md instructions for using Ubuntu binary build

Add INSTALL.md instructions for using Ubuntu binary build

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

Merge pull request #115 from iovisor/bblanco_dev

Flags for 4.3 were accidentally committed, revert

Flags for 4.3 were accidentally committed, revert

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

Merge pull request #114 from iovisor/weichunc_dev

release IPDB in host ns early

release IPDB in host ns early

We need it for setting vxlan flow based flags currently. Release
it once the interfaces are set up. This does not help Ctrl-C on
main.py though.

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

Merge pull request #113 from iovisor/bblanco_dev

Suggest Release and add X86 target in readme

Suggest Release and add X86 target in readme

The default CMAKE_BUILD_TYPE is Debug when cloning from git, suggest
Release instead.

Also, there are some worrisome warnings during cmake when X86 target is
omitted. Additionally, I may want X86 target support for a future
feature, so lets suggest it here.

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

Merge pull request #112 from iovisor/weichunc_dev

llvm 3.7+ deb is broken

llvm 3.7+ deb is broken

https://llvm.org/bugs/show_bug.cgi?id=24154

Build from source until it's resolved. Another plan is to provide
binary package.

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

Merge pull request #110 from iovisor/yhs_dev

better clean up of created namespace/interfaces if ctrl-c is pressed

Merge pull request #111 from iovisor/weichunc_dev

fix tunnel_mesh.py

fix tunnel_mesh.py

The IP assignment was wrong.

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

better clean up of created namespace/interfaces if ctrl-c is pressed

  o There are still some cases when ctrl-c is pressed, the program does not exit
    gracefully, with a however lower probability. In my random manual experiments,
    one in ten cases it still hangs and needs further ctrl-z.

Signed-off-by: Yonghong Song <yhs@plumgrid.com>

Merge pull request #107 from iovisor/bblanco_dev

Reorganize bpf_module into multiple frontend implementations

Merge pull request #106 from tuxology/master

Add project logo and source files

Move rest of frontend implementation into subdirs

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

Add project logo and source files

The logo contains elements which are in public domain. The gear
has been modified from illustration submitted by 'hank0071' on
OpenClipart and the font used is Open Sans. There seems to be no
restriction on commercial use.

Signed-off-by: Suchakra Sharma <suchakrapani.sharma@polymtl.ca>

Move (some) frontend specific logic to the respective directories

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

Move frontends into individual directories

Other commits to follow with cleanup of the code itself.

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

Merge pull request #105 from iovisor/bblanco_dev

Enable memory cleanup in Clang, otherwise 500KB lost per compile

Enable memory cleanup in Clang, otherwise 500KB lost per compile

Since it is meant for the command line, clang ships with no memory
cleanup. We intend to run as a long-lived process, so enable that
cleanup. Valgrind now reports only trivial (and non-increasing over
time) memory leaks.

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

Merge pull request #102 from iovisor/bblanco_dev

Add version check for functions only in >= 4.3 kernel

Merge pull request #104 from iovisor/bcc_export

Add pkg-config support and include compat/linux* in packaging

Add pkg-config support and include compat/linux* in packaging

Different kernel header versions has been a continuing source of pain.
Though I don't prefer this approach, it seems the most reliable way to
get a build (for the time being) is to bundle the linux/bpf*.h in a
compat directory in the target install location.

To get visibility into that compat directory from another library, one
will need to rely on pkg-config --cflags argument to pull in that compat
directory.

Future work will be to avoid bundling the compat directory if the
version of linux-libc-dev is high enough, or if the presence of the
result of 'make headers_install' from the running kernel can be
detected.

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

Add version check for functions only in >= 4.3 kernel

Addresses issue #100

Note to developers working off of net-next: you will need to locally
disable this patch to make use of the new features.

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

Merge pull request #99 from iovisor/weichunc_dev

Assign ip address to br

Assign ip address to br

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

Merge pull request #98 from iovisor/ast_dev

fix incorrect DataLayout

fix incorrect DataLayout

Signed-off-by: Alexei Starovoitov <ast@plumgrid.com>

Merge pull request #97 from iovisor/ast_dev

update tunnel_key prototypes

update tunnel_key prototypes

Signed-off-by: Alexei Starovoitov <ast@plumgrid.com>

Merge pull request #96 from iovisor/bblanco_dev

Add option for custom log string to bpf_prog_load

Add option for custom log string to bpf_prog_load

bpf_prog_load was unconditionally printing the error from the kernel to
stderr. If libbpf is used in a library, that library may want to collect
the error to a custom location (a logfile). So, provide an option to
store the string elsewhere than stderr.

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