14 files changed, 699 insertions, 97 deletions

diff --git a/tools/lib/bpf/Build b/tools/lib/bpf/Build
index d8749756352d..64c679d67109 100644
--- a/tools/lib/bpf/Build
+++ b/tools/lib/bpf/Build
@@ -1 +1 @@
-libbpf-y := libbpf.o bpf.o
+libbpf-y := libbpf.o bpf.o nlattr.o
diff --git a/tools/lib/bpf/Makefile b/tools/lib/bpf/Makefile
index 4555304dc18e..e6d5f8d1477f 100644
--- a/tools/lib/bpf/Makefile
+++ b/tools/lib/bpf/Makefile
@@ -93,7 +93,6 @@ export prefix libdir src obj
 # Shell quotes
 libdir_SQ = $(subst ','\'',$(libdir))
 libdir_relative_SQ = $(subst ','\'',$(libdir_relative))
-plugin_dir_SQ = $(subst ','\'',$(plugin_dir))
 
 LIB_FILE = libbpf.a libbpf.so
 
@@ -150,7 +149,7 @@ CMD_TARGETS = $(LIB_FILE)
 
 TARGETS = $(CMD_TARGETS)
 
-all: fixdep $(VERSION_FILES) all_cmd
+all: fixdep all_cmd
 
 all_cmd: $(CMD_TARGETS)
 
@@ -161,6 +160,12 @@ $(BPF_IN): force elfdep bpfdep
         @(test -f ../../include/uapi/linux/bpf_common.h -a -f ../../../include/uapi/linux/bpf_common.h && ( \
         (diff -B ../../include/uapi/linux/bpf_common.h ../../../include/uapi/linux/bpf_common.h >/dev/null) || \
         echo "Warning: Kernel ABI header at 'tools/include/uapi/linux/bpf_common.h' differs from latest version at 'include/uapi/linux/bpf_common.h'" >&2 )) || true
+        @(test -f ../../include/uapi/linux/netlink.h -a -f ../../../include/uapi/linux/netlink.h && ( \
+        (diff -B ../../include/uapi/linux/netlink.h ../../../include/uapi/linux/netlink.h >/dev/null) || \
+        echo "Warning: Kernel ABI header at 'tools/include/uapi/linux/netlink.h' differs from latest version at 'include/uapi/linux/netlink.h'" >&2 )) || true
+        @(test -f ../../include/uapi/linux/if_link.h -a -f ../../../include/uapi/linux/if_link.h && ( \
+        (diff -B ../../include/uapi/linux/if_link.h ../../../include/uapi/linux/if_link.h >/dev/null) || \
+        echo "Warning: Kernel ABI header at 'tools/include/uapi/linux/if_link.h' differs from latest version at 'include/uapi/linux/if_link.h'" >&2 )) || true
         $(Q)$(MAKE) $(build)=libbpf
 
 $(OUTPUT)libbpf.so: $(BPF_IN)
@@ -169,21 +174,11 @@ $(OUTPUT)libbpf.so: $(BPF_IN)
 $(OUTPUT)libbpf.a: $(BPF_IN)
         $(QUIET_LINK)$(RM) $@; $(AR) rcs $@ $^
 
-define update_dir
-  (echo $1 > $@.tmp;                            \
-   if [ -r $@ ] && cmp -s $@ $@.tmp; then       \
-     rm -f $@.tmp;                              \
-   else                                         \
-     echo '  UPDATE                 $@';        \
-     mv -f $@.tmp $@;                           \
-   fi);
-endef
-
 define do_install
         if [ ! -d '$(DESTDIR_SQ)$2' ]; then             \
                 $(INSTALL) -d -m 755 '$(DESTDIR_SQ)$2'; \
         fi;                                             \
-        $(INSTALL) $1 '$(DESTDIR_SQ)$2'
+        $(INSTALL) $1 $(if $3,-m $3,) '$(DESTDIR_SQ)$2'
 endef
 
 install_lib: all_cmd
@@ -192,7 +187,8 @@ install_lib: all_cmd
 
 install_headers:
         $(call QUIET_INSTALL, headers) \
-                $(call do_install,bpf.h,$(prefix)/include/bpf,644)
+                $(call do_install,bpf.h,$(prefix)/include/bpf,644); \
+                $(call do_install,libbpf.h,$(prefix)/include/bpf,644);
 
 install: install_lib
 
@@ -203,7 +199,7 @@ config-clean:
         $(Q)$(MAKE) -C $(srctree)/tools/build/feature/ clean >/dev/null
 
 clean:
-        $(call QUIET_CLEAN, libbpf) $(RM) *.o *~ $(TARGETS) *.a *.so $(VERSION_FILES) .*.d .*.cmd \
+        $(call QUIET_CLEAN, libbpf) $(RM) *.o *~ $(TARGETS) *.a *.so .*.d .*.cmd \
                 $(RM) LIBBPF-CFLAGS
         $(call QUIET_CLEAN, core-gen) $(RM) $(OUTPUT)FEATURE-DUMP.libbpf
 
@@ -213,10 +209,10 @@ PHONY += force elfdep bpfdep
 force:
 
 elfdep:
-        @if [ "$(feature-libelf)" != "1" ]; then echo "No libelf found"; exit -1 ; fi
+        @if [ "$(feature-libelf)" != "1" ]; then echo "No libelf found"; exit 1 ; fi
 
 bpfdep:
-        @if [ "$(feature-bpf)" != "1" ]; then echo "BPF API too old"; exit -1 ; fi
+        @if [ "$(feature-bpf)" != "1" ]; then echo "BPF API too old"; exit 1 ; fi
 
 # Declare the contents of the .PHONY variable as phony.  We keep that
 # information in a variable so we can use it in if_changed and friends.
diff --git a/tools/lib/bpf/bpf.c b/tools/lib/bpf/bpf.c
index 5128677e4117..592a58a2b681 100644
--- a/tools/lib/bpf/bpf.c
+++ b/tools/lib/bpf/bpf.c
@@ -1,3 +1,5 @@
+// SPDX-License-Identifier: LGPL-2.1
+
 /*
  * common eBPF ELF operations.
  *
@@ -25,6 +27,16 @@
 #include <asm/unistd.h>
 #include <linux/bpf.h>
 #include "bpf.h"
+#include "libbpf.h"
+#include "nlattr.h"
+#include <linux/rtnetlink.h>
+#include <linux/if_link.h>
+#include <sys/socket.h>
+#include <errno.h>
+
+#ifndef SOL_NETLINK
+#define SOL_NETLINK 270
+#endif
 
 /*
  * When building perf, unistd.h is overridden. __NR_bpf is
@@ -46,7 +58,9 @@
 # endif
 #endif
 
+#ifndef min
 #define min(x, y) ((x) < (y) ? (x) : (y))
+#endif
 
 static inline __u64 ptr_to_u64(const void *ptr)
 {
@@ -413,3 +427,124 @@ int bpf_obj_get_info_by_fd(int prog_fd, void *info, __u32 *info_len)
 
         return err;
 }
+
+int bpf_set_link_xdp_fd(int ifindex, int fd, __u32 flags)
+{
+        struct sockaddr_nl sa;
+        int sock, seq = 0, len, ret = -1;
+        char buf[4096];
+        struct nlattr *nla, *nla_xdp;
+        struct {
+                struct nlmsghdr  nh;
+                struct ifinfomsg ifinfo;
+                char             attrbuf[64];
+        } req;
+        struct nlmsghdr *nh;
+        struct nlmsgerr *err;
+        socklen_t addrlen;
+        int one = 1;
+
+        memset(&sa, 0, sizeof(sa));
+        sa.nl_family = AF_NETLINK;
+
+        sock = socket(AF_NETLINK, SOCK_RAW, NETLINK_ROUTE);
+        if (sock < 0) {
+                return -errno;
+        }
+
+        if (setsockopt(sock, SOL_NETLINK, NETLINK_EXT_ACK,
+                       &one, sizeof(one)) < 0) {
+                fprintf(stderr, "Netlink error reporting not supported\n");
+        }
+
+        if (bind(sock, (struct sockaddr *)&sa, sizeof(sa)) < 0) {
+                ret = -errno;
+                goto cleanup;
+        }
+
+        addrlen = sizeof(sa);
+        if (getsockname(sock, (struct sockaddr *)&sa, &addrlen) < 0) {
+                ret = -errno;
+                goto cleanup;
+        }
+
+        if (addrlen != sizeof(sa)) {
+                ret = -LIBBPF_ERRNO__INTERNAL;
+                goto cleanup;
+        }
+
+        memset(&req, 0, sizeof(req));
+        req.nh.nlmsg_len = NLMSG_LENGTH(sizeof(struct ifinfomsg));
+        req.nh.nlmsg_flags = NLM_F_REQUEST | NLM_F_ACK;
+        req.nh.nlmsg_type = RTM_SETLINK;
+        req.nh.nlmsg_pid = 0;
+        req.nh.nlmsg_seq = ++seq;
+        req.ifinfo.ifi_family = AF_UNSPEC;
+        req.ifinfo.ifi_index = ifindex;
+
+        /* started nested attribute for XDP */
+        nla = (struct nlattr *)(((char *)&req)
+                                + NLMSG_ALIGN(req.nh.nlmsg_len));
+        nla->nla_type = NLA_F_NESTED | IFLA_XDP;
+        nla->nla_len = NLA_HDRLEN;
+
+        /* add XDP fd */
+        nla_xdp = (struct nlattr *)((char *)nla + nla->nla_len);
+        nla_xdp->nla_type = IFLA_XDP_FD;
+        nla_xdp->nla_len = NLA_HDRLEN + sizeof(int);
+        memcpy((char *)nla_xdp + NLA_HDRLEN, &fd, sizeof(fd));
+        nla->nla_len += nla_xdp->nla_len;
+
+        /* if user passed in any flags, add those too */
+        if (flags) {
+                nla_xdp = (struct nlattr *)((char *)nla + nla->nla_len);
+                nla_xdp->nla_type = IFLA_XDP_FLAGS;
+                nla_xdp->nla_len = NLA_HDRLEN + sizeof(flags);
+                memcpy((char *)nla_xdp + NLA_HDRLEN, &flags, sizeof(flags));
+                nla->nla_len += nla_xdp->nla_len;
+        }
+
+        req.nh.nlmsg_len += NLA_ALIGN(nla->nla_len);
+
+        if (send(sock, &req, req.nh.nlmsg_len, 0) < 0) {
+                ret = -errno;
+                goto cleanup;
+        }
+
+        len = recv(sock, buf, sizeof(buf), 0);
+        if (len < 0) {
+                ret = -errno;
+                goto cleanup;
+        }
+
+        for (nh = (struct nlmsghdr *)buf; NLMSG_OK(nh, len);
+             nh = NLMSG_NEXT(nh, len)) {
+                if (nh->nlmsg_pid != sa.nl_pid) {
+                        ret = -LIBBPF_ERRNO__WRNGPID;
+                        goto cleanup;
+                }
+                if (nh->nlmsg_seq != seq) {
+                        ret = -LIBBPF_ERRNO__INVSEQ;
+                        goto cleanup;
+                }
+                switch (nh->nlmsg_type) {
+                case NLMSG_ERROR:
+                        err = (struct nlmsgerr *)NLMSG_DATA(nh);
+                        if (!err->error)
+                                continue;
+                        ret = err->error;
+                        nla_dump_errormsg(nh);
+                        goto cleanup;
+                case NLMSG_DONE:
+                        break;
+                default:
+                        break;
+                }
+        }
+
+        ret = 0;
+
+cleanup:
+        close(sock);
+        return ret;
+}
diff --git a/tools/lib/bpf/bpf.h b/tools/lib/bpf/bpf.h
index 6534889e2b2f..8d18fb73d7fb 100644
--- a/tools/lib/bpf/bpf.h
+++ b/tools/lib/bpf/bpf.h
@@ -1,3 +1,5 @@
+/* SPDX-License-Identifier: LGPL-2.1 */
+
 /*
  * common eBPF ELF operations.
  *
@@ -40,7 +42,7 @@ int bpf_create_map_in_map(enum bpf_map_type map_type, const char *name,
                           __u32 map_flags);
 
 /* Recommend log buffer size */
-#define BPF_LOG_BUF_SIZE 65536
+#define BPF_LOG_BUF_SIZE (256 * 1024)
 int bpf_load_program_name(enum bpf_prog_type type, const char *name,
                           const struct bpf_insn *insns,
                           size_t insns_cnt, const char *license,
diff --git a/tools/lib/bpf/libbpf.c b/tools/lib/bpf/libbpf.c
index 5aa45f89da93..71ddc481f349 100644
--- a/tools/lib/bpf/libbpf.c
+++ b/tools/lib/bpf/libbpf.c
@@ -1,3 +1,5 @@
+// SPDX-License-Identifier: LGPL-2.1
+
 /*
  * Common eBPF ELF object loading operations.
  *
@@ -106,6 +108,8 @@ static const char *libbpf_strerror_table[NR_ERRNO] = {
         [ERRCODE_OFFSET(PROG2BIG)]      = "Program too big",
         [ERRCODE_OFFSET(KVER)]          = "Incorrect kernel version",
         [ERRCODE_OFFSET(PROGTYPE)]      = "Kernel doesn't support this program type",
+        [ERRCODE_OFFSET(WRNGPID)]       = "Wrong pid in netlink message",
+        [ERRCODE_OFFSET(INVSEQ)]        = "Invalid netlink sequence",
 };
 
 int libbpf_strerror(int err, char *buf, size_t size)
@@ -174,12 +178,19 @@ struct bpf_program {
         char *name;
         char *section_name;
         struct bpf_insn *insns;
-        size_t insns_cnt;
+        size_t insns_cnt, main_prog_cnt;
         enum bpf_prog_type type;
 
-        struct {
+        struct reloc_desc {
+                enum {
+                        RELO_LD64,
+                        RELO_CALL,
+                } type;
                 int insn_idx;
-                int map_idx;
+                union {
+                        int map_idx;
+                        int text_off;
+                };
         } *reloc_desc;
         int nr_reloc;
 
@@ -234,6 +245,7 @@ struct bpf_object {
                 } *reloc;
                 int nr_reloc;
                 int maps_shndx;
+                int text_shndx;
         } efile;
         /*
          * All loaded bpf_object is linked in a list, which is
@@ -375,9 +387,13 @@ bpf_object__init_prog_names(struct bpf_object *obj)
         size_t pi, si;
 
         for (pi = 0; pi < obj->nr_programs; pi++) {
-                char *name = NULL;
+                const char *name = NULL;
 
                 prog = &obj->programs[pi];
+                if (prog->idx == obj->efile.text_shndx) {
+                        name = ".text";
+                        goto skip_search;
+                }
 
                 for (si = 0; si < symbols->d_size / sizeof(GElf_Sym) && !name;
                      si++) {
@@ -387,6 +403,8 @@ bpf_object__init_prog_names(struct bpf_object *obj)
                                 continue;
                         if (sym.st_shndx != prog->idx)
                                 continue;
+                        if (GELF_ST_BIND(sym.st_info) != STB_GLOBAL)
+                                continue;
 
                         name = elf_strptr(obj->efile.elf,
                                           obj->efile.strtabidx,
@@ -403,7 +421,7 @@ bpf_object__init_prog_names(struct bpf_object *obj)
                                    prog->section_name);
                         return -EINVAL;
                 }
-
+skip_search:
                 prog->name = strdup(name);
                 if (!prog->name) {
                         pr_warning("failed to allocate memory for prog sym %s\n",
@@ -793,6 +811,8 @@ static int bpf_object__elf_collect(struct bpf_object *obj)
                 } else if ((sh.sh_type == SHT_PROGBITS) &&
                            (sh.sh_flags & SHF_EXECINSTR) &&
                            (data->d_size > 0)) {
+                        if (strcmp(name, ".text") == 0)
+                                obj->efile.text_shndx = idx;
                         err = bpf_object__add_program(obj, data->d_buf,
                                                       data->d_size, name, idx);
                         if (err) {
@@ -854,11 +874,14 @@ bpf_object__find_prog_by_idx(struct bpf_object *obj, int idx)
 }
 
 static int
-bpf_program__collect_reloc(struct bpf_program *prog,
-                           size_t nr_maps, GElf_Shdr *shdr,
-                           Elf_Data *data, Elf_Data *symbols,
-                           int maps_shndx, struct bpf_map *maps)
+bpf_program__collect_reloc(struct bpf_program *prog, GElf_Shdr *shdr,
+                           Elf_Data *data, struct bpf_object *obj)
 {
+        Elf_Data *symbols = obj->efile.symbols;
+        int text_shndx = obj->efile.text_shndx;
+        int maps_shndx = obj->efile.maps_shndx;
+        struct bpf_map *maps = obj->maps;
+        size_t nr_maps = obj->nr_maps;
         int i, nrels;
 
         pr_debug("collecting relocating info for: '%s'\n",
@@ -891,8 +914,11 @@ bpf_program__collect_reloc(struct bpf_program *prog,
                                    GELF_R_SYM(rel.r_info));
                         return -LIBBPF_ERRNO__FORMAT;
                 }
+                pr_debug("relo for %lld value %lld name %d\n",
+                         (long long) (rel.r_info >> 32),
+                         (long long) sym.st_value, sym.st_name);
 
-                if (sym.st_shndx != maps_shndx) {
+                if (sym.st_shndx != maps_shndx && sym.st_shndx != text_shndx) {
                         pr_warning("Program '%s' contains non-map related relo data pointing to section %u\n",
                                    prog->section_name, sym.st_shndx);
                         return -LIBBPF_ERRNO__RELOC;
@@ -901,6 +927,17 @@ bpf_program__collect_reloc(struct bpf_program *prog,
                 insn_idx = rel.r_offset / sizeof(struct bpf_insn);
                 pr_debug("relocation: insn_idx=%u\n", insn_idx);
 
+                if (insns[insn_idx].code == (BPF_JMP | BPF_CALL)) {
+                        if (insns[insn_idx].src_reg != BPF_PSEUDO_CALL) {
+                                pr_warning("incorrect bpf_call opcode\n");
+                                return -LIBBPF_ERRNO__RELOC;
+                        }
+                        prog->reloc_desc[i].type = RELO_CALL;
+                        prog->reloc_desc[i].insn_idx = insn_idx;
+                        prog->reloc_desc[i].text_off = sym.st_value;
+                        continue;
+                }
+
                 if (insns[insn_idx].code != (BPF_LD | BPF_IMM | BPF_DW)) {
                         pr_warning("bpf: relocation: invalid relo for insns[%d].code 0x%x\n",
                                    insn_idx, insns[insn_idx].code);
@@ -922,6 +959,7 @@ bpf_program__collect_reloc(struct bpf_program *prog,
                         return -LIBBPF_ERRNO__RELOC;
                 }
 
+                prog->reloc_desc[i].type = RELO_LD64;
                 prog->reloc_desc[i].insn_idx = insn_idx;
                 prog->reloc_desc[i].map_idx = map_idx;
         }
@@ -961,27 +999,76 @@ bpf_object__create_maps(struct bpf_object *obj)
 }
 
 static int
+bpf_program__reloc_text(struct bpf_program *prog, struct bpf_object *obj,
+                        struct reloc_desc *relo)
+{
+        struct bpf_insn *insn, *new_insn;
+        struct bpf_program *text;
+        size_t new_cnt;
+
+        if (relo->type != RELO_CALL)
+                return -LIBBPF_ERRNO__RELOC;
+
+        if (prog->idx == obj->efile.text_shndx) {
+                pr_warning("relo in .text insn %d into off %d\n",
+                           relo->insn_idx, relo->text_off);
+                return -LIBBPF_ERRNO__RELOC;
+        }
+
+        if (prog->main_prog_cnt == 0) {
+                text = bpf_object__find_prog_by_idx(obj, obj->efile.text_shndx);
+                if (!text) {
+                        pr_warning("no .text section found yet relo into text exist\n");
+                        return -LIBBPF_ERRNO__RELOC;
+                }
+                new_cnt = prog->insns_cnt + text->insns_cnt;
+                new_insn = realloc(prog->insns, new_cnt * sizeof(*insn));
+                if (!new_insn) {
+                        pr_warning("oom in prog realloc\n");
+                        return -ENOMEM;
+                }
+                memcpy(new_insn + prog->insns_cnt, text->insns,
+                       text->insns_cnt * sizeof(*insn));
+                prog->insns = new_insn;
+                prog->main_prog_cnt = prog->insns_cnt;
+                prog->insns_cnt = new_cnt;
+        }
+        insn = &prog->insns[relo->insn_idx];
+        insn->imm += prog->main_prog_cnt - relo->insn_idx;
+        pr_debug("added %zd insn from %s to prog %s\n",
+                 text->insns_cnt, text->section_name, prog->section_name);
+        return 0;
+}
+
+static int
 bpf_program__relocate(struct bpf_program *prog, struct bpf_object *obj)
 {
-        int i;
+        int i, err;
 
         if (!prog || !prog->reloc_desc)
                 return 0;
 
         for (i = 0; i < prog->nr_reloc; i++) {
-                int insn_idx, map_idx;
-                struct bpf_insn *insns = prog->insns;
+                if (prog->reloc_desc[i].type == RELO_LD64) {
+                        struct bpf_insn *insns = prog->insns;
+                        int insn_idx, map_idx;
 
-                insn_idx = prog->reloc_desc[i].insn_idx;
-                map_idx = prog->reloc_desc[i].map_idx;
+                        insn_idx = prog->reloc_desc[i].insn_idx;
+                        map_idx = prog->reloc_desc[i].map_idx;
 
-                if (insn_idx >= (int)prog->insns_cnt) {
-                        pr_warning("relocation out of range: '%s'\n",
-                                   prog->section_name);
-                        return -LIBBPF_ERRNO__RELOC;
+                        if (insn_idx >= (int)prog->insns_cnt) {
+                                pr_warning("relocation out of range: '%s'\n",
+                                           prog->section_name);
+                                return -LIBBPF_ERRNO__RELOC;
+                        }
+                        insns[insn_idx].src_reg = BPF_PSEUDO_MAP_FD;
+                        insns[insn_idx].imm = obj->maps[map_idx].fd;
+                } else {
+                        err = bpf_program__reloc_text(prog, obj,
+                                                      &prog->reloc_desc[i]);
+                        if (err)
+                                return err;
                 }
-                insns[insn_idx].src_reg = BPF_PSEUDO_MAP_FD;
-                insns[insn_idx].imm = obj->maps[map_idx].fd;
         }
 
         zfree(&prog->reloc_desc);
@@ -1024,7 +1111,6 @@ static int bpf_object__collect_reloc(struct bpf_object *obj)
                 Elf_Data *data = obj->efile.reloc[i].data;
                 int idx = shdr->sh_info;
                 struct bpf_program *prog;
-                size_t nr_maps = obj->nr_maps;
 
                 if (shdr->sh_type != SHT_REL) {
                         pr_warning("internal error at %d\n", __LINE__);
@@ -1038,11 +1124,9 @@ static int bpf_object__collect_reloc(struct bpf_object *obj)
                         return -LIBBPF_ERRNO__RELOC;
                 }
 
-                err = bpf_program__collect_reloc(prog, nr_maps,
+                err = bpf_program__collect_reloc(prog,
                                                  shdr, data,
-                                                 obj->efile.symbols,
-                                                 obj->efile.maps_shndx,
-                                                 obj->maps);
+                                                 obj);
                 if (err)
                         return err;
         }
@@ -1195,6 +1279,8 @@ bpf_object__load_progs(struct bpf_object *obj)
         int err;
 
         for (i = 0; i < obj->nr_programs; i++) {
+                if (obj->programs[i].idx == obj->efile.text_shndx)
+                        continue;
                 err = bpf_program__load(&obj->programs[i],
                                         obj->license,
                                         obj->kern_version);
@@ -1721,6 +1807,45 @@ BPF_PROG_TYPE_FNS(tracepoint, BPF_PROG_TYPE_TRACEPOINT);
 BPF_PROG_TYPE_FNS(xdp, BPF_PROG_TYPE_XDP);
 BPF_PROG_TYPE_FNS(perf_event, BPF_PROG_TYPE_PERF_EVENT);
 
+#define BPF_PROG_SEC(string, type) { string, sizeof(string) - 1, type }
+static const struct {
+        const char *sec;
+        size_t len;
+        enum bpf_prog_type prog_type;
+} section_names[] = {
+        BPF_PROG_SEC("socket",          BPF_PROG_TYPE_SOCKET_FILTER),
+        BPF_PROG_SEC("kprobe/",         BPF_PROG_TYPE_KPROBE),
+        BPF_PROG_SEC("kretprobe/",      BPF_PROG_TYPE_KPROBE),
+        BPF_PROG_SEC("tracepoint/",     BPF_PROG_TYPE_TRACEPOINT),
+        BPF_PROG_SEC("xdp",             BPF_PROG_TYPE_XDP),
+        BPF_PROG_SEC("perf_event",      BPF_PROG_TYPE_PERF_EVENT),
+        BPF_PROG_SEC("cgroup/skb",      BPF_PROG_TYPE_CGROUP_SKB),
+        BPF_PROG_SEC("cgroup/sock",     BPF_PROG_TYPE_CGROUP_SOCK),
+        BPF_PROG_SEC("cgroup/dev",      BPF_PROG_TYPE_CGROUP_DEVICE),
+        BPF_PROG_SEC("sockops",         BPF_PROG_TYPE_SOCK_OPS),
+        BPF_PROG_SEC("sk_skb",          BPF_PROG_TYPE_SK_SKB),
+};
+#undef BPF_PROG_SEC
+
+static enum bpf_prog_type bpf_program__guess_type(struct bpf_program *prog)
+{
+        int i;
+
+        if (!prog->section_name)
+                goto err;
+
+        for (i = 0; i < ARRAY_SIZE(section_names); i++)
+                if (strncmp(prog->section_name, section_names[i].sec,
+                            section_names[i].len) == 0)
+                        return section_names[i].prog_type;
+
+err:
+        pr_warning("failed to guess program type based on section name %s\n",
+                   prog->section_name);
+
+        return BPF_PROG_TYPE_UNSPEC;
+}
+
 int bpf_map__fd(struct bpf_map *map)
 {
         return map ? map->fd : -EINVAL;
@@ -1818,7 +1943,7 @@ long libbpf_get_error(const void *ptr)
 int bpf_prog_load(const char *file, enum bpf_prog_type type,
                   struct bpf_object **pobj, int *prog_fd)
 {
-        struct bpf_program *prog;
+        struct bpf_program *prog, *first_prog = NULL;
         struct bpf_object *obj;
         int err;
 
@@ -1826,13 +1951,30 @@ int bpf_prog_load(const char *file, enum bpf_prog_type type,
         if (IS_ERR(obj))
                 return -ENOENT;
 
-        prog = bpf_program__next(NULL, obj);
-        if (!prog) {
+        bpf_object__for_each_program(prog, obj) {
+                /*
+                 * If type is not specified, try to guess it based on
+                 * section name.
+                 */
+                if (type == BPF_PROG_TYPE_UNSPEC) {
+                        type = bpf_program__guess_type(prog);
+                        if (type == BPF_PROG_TYPE_UNSPEC) {
+                                bpf_object__close(obj);
+                                return -EINVAL;
+                        }
+                }
+
+                bpf_program__set_type(prog, type);
+                if (prog->idx != obj->efile.text_shndx && !first_prog)
+                        first_prog = prog;
+        }
+
+        if (!first_prog) {
+                pr_warning("object file doesn't contain bpf program\n");
                 bpf_object__close(obj);
                 return -ENOENT;
         }
 
-        bpf_program__set_type(prog, type);
         err = bpf_object__load(obj);
         if (err) {
                 bpf_object__close(obj);
@@ -1840,6 +1982,6 @@ int bpf_prog_load(const char *file, enum bpf_prog_type type,
         }
 
         *pobj = obj;
-        *prog_fd = bpf_program__fd(prog);
+        *prog_fd = bpf_program__fd(first_prog);
         return 0;
 }
diff --git a/tools/lib/bpf/libbpf.h b/tools/lib/bpf/libbpf.h
index 6e20003109e0..f85906533cdd 100644
--- a/tools/lib/bpf/libbpf.h
+++ b/tools/lib/bpf/libbpf.h
@@ -1,3 +1,5 @@
+/* SPDX-License-Identifier: LGPL-2.1 */
+
 /*
  * Common eBPF ELF object loading operations.
  *
@@ -42,6 +44,8 @@ enum libbpf_errno {
         LIBBPF_ERRNO__PROG2BIG, /* Program too big */
         LIBBPF_ERRNO__KVER,     /* Incorrect kernel version */
         LIBBPF_ERRNO__PROGTYPE, /* Kernel doesn't support this program type */
+        LIBBPF_ERRNO__WRNGPID,  /* Wrong pid in netlink message */
+        LIBBPF_ERRNO__INVSEQ,   /* Invalid netlink sequence */
         __LIBBPF_ERRNO__END,
 };
 
@@ -246,4 +250,6 @@ long libbpf_get_error(const void *ptr);
 
 int bpf_prog_load(const char *file, enum bpf_prog_type type,
                   struct bpf_object **pobj, int *prog_fd);
+
+int bpf_set_link_xdp_fd(int ifindex, int fd, __u32 flags);
 #endif
diff --git a/tools/lib/bpf/nlattr.c b/tools/lib/bpf/nlattr.c
new file mode 100644
index 000000000000..4719434278b2
--- /dev/null
+++ b/tools/lib/bpf/nlattr.c
@@ -0,0 +1,187 @@
+// SPDX-License-Identifier: LGPL-2.1
+
+/*
+ * NETLINK      Netlink attributes
+ *
+ *      This library is free software; you can redistribute it and/or
+ *      modify it under the terms of the GNU Lesser General Public
+ *      License as published by the Free Software Foundation version 2.1
+ *      of the License.
+ *
+ * Copyright (c) 2003-2013 Thomas Graf <tgraf@suug.ch>
+ */
+
+#include <errno.h>
+#include "nlattr.h"
+#include <linux/rtnetlink.h>
+#include <string.h>
+#include <stdio.h>
+
+static uint16_t nla_attr_minlen[NLA_TYPE_MAX+1] = {
+        [NLA_U8]        = sizeof(uint8_t),
+        [NLA_U16]       = sizeof(uint16_t),
+        [NLA_U32]       = sizeof(uint32_t),
+        [NLA_U64]       = sizeof(uint64_t),
+        [NLA_STRING]    = 1,
+        [NLA_FLAG]      = 0,
+};
+
+static int nla_len(const struct nlattr *nla)
+{
+        return nla->nla_len - NLA_HDRLEN;
+}
+
+static struct nlattr *nla_next(const struct nlattr *nla, int *remaining)
+{
+        int totlen = NLA_ALIGN(nla->nla_len);
+
+        *remaining -= totlen;
+        return (struct nlattr *) ((char *) nla + totlen);
+}
+
+static int nla_ok(const struct nlattr *nla, int remaining)
+{
+        return remaining >= sizeof(*nla) &&
+               nla->nla_len >= sizeof(*nla) &&
+               nla->nla_len <= remaining;
+}
+
+static void *nla_data(const struct nlattr *nla)
+{
+        return (char *) nla + NLA_HDRLEN;
+}
+
+static int nla_type(const struct nlattr *nla)
+{
+        return nla->nla_type & NLA_TYPE_MASK;
+}
+
+static int validate_nla(struct nlattr *nla, int maxtype,
+                        struct nla_policy *policy)
+{
+        struct nla_policy *pt;
+        unsigned int minlen = 0;
+        int type = nla_type(nla);
+
+        if (type < 0 || type > maxtype)
+                return 0;
+
+        pt = &policy[type];
+
+        if (pt->type > NLA_TYPE_MAX)
+                return 0;
+
+        if (pt->minlen)
+                minlen = pt->minlen;
+        else if (pt->type != NLA_UNSPEC)
+                minlen = nla_attr_minlen[pt->type];
+
+        if (nla_len(nla) < minlen)
+                return -1;
+
+        if (pt->maxlen && nla_len(nla) > pt->maxlen)
+                return -1;
+
+        if (pt->type == NLA_STRING) {
+                char *data = nla_data(nla);
+                if (data[nla_len(nla) - 1] != '\0')
+                        return -1;
+        }
+
+        return 0;
+}
+
+static inline int nlmsg_len(const struct nlmsghdr *nlh)
+{
+        return nlh->nlmsg_len - NLMSG_HDRLEN;
+}
+
+/**
+ * Create attribute index based on a stream of attributes.
+ * @arg tb              Index array to be filled (maxtype+1 elements).
+ * @arg maxtype         Maximum attribute type expected and accepted.
+ * @arg head            Head of attribute stream.
+ * @arg len             Length of attribute stream.
+ * @arg policy          Attribute validation policy.
+ *
+ * Iterates over the stream of attributes and stores a pointer to each
+ * attribute in the index array using the attribute type as index to
+ * the array. Attribute with a type greater than the maximum type
+ * specified will be silently ignored in order to maintain backwards
+ * compatibility. If \a policy is not NULL, the attribute will be
+ * validated using the specified policy.
+ *
+ * @see nla_validate
+ * @return 0 on success or a negative error code.
+ */
+static int nla_parse(struct nlattr *tb[], int maxtype, struct nlattr *head, int len,
+                     struct nla_policy *policy)
+{
+        struct nlattr *nla;
+        int rem, err;
+
+        memset(tb, 0, sizeof(struct nlattr *) * (maxtype + 1));
+
+        nla_for_each_attr(nla, head, len, rem) {
+                int type = nla_type(nla);
+
+                if (type > maxtype)
+                        continue;
+
+                if (policy) {
+                        err = validate_nla(nla, maxtype, policy);
+                        if (err < 0)
+                                goto errout;
+                }
+
+                if (tb[type])
+                        fprintf(stderr, "Attribute of type %#x found multiple times in message, "
+                                  "previous attribute is being ignored.\n", type);
+
+                tb[type] = nla;
+        }
+
+        err = 0;
+errout:
+        return err;
+}
+
+/* dump netlink extended ack error message */
+int nla_dump_errormsg(struct nlmsghdr *nlh)
+{
+        struct nla_policy extack_policy[NLMSGERR_ATTR_MAX + 1] = {
+                [NLMSGERR_ATTR_MSG]     = { .type = NLA_STRING },
+                [NLMSGERR_ATTR_OFFS]    = { .type = NLA_U32 },
+        };
+        struct nlattr *tb[NLMSGERR_ATTR_MAX + 1], *attr;
+        struct nlmsgerr *err;
+        char *errmsg = NULL;
+        int hlen, alen;
+
+        /* no TLVs, nothing to do here */
+        if (!(nlh->nlmsg_flags & NLM_F_ACK_TLVS))
+                return 0;
+
+        err = (struct nlmsgerr *)NLMSG_DATA(nlh);
+        hlen = sizeof(*err);
+
+        /* if NLM_F_CAPPED is set then the inner err msg was capped */
+        if (!(nlh->nlmsg_flags & NLM_F_CAPPED))
+                hlen += nlmsg_len(&err->msg);
+
+        attr = (struct nlattr *) ((void *) err + hlen);
+        alen = nlh->nlmsg_len - hlen;
+
+        if (nla_parse(tb, NLMSGERR_ATTR_MAX, attr, alen, extack_policy) != 0) {
+                fprintf(stderr,
+                        "Failed to parse extended error attributes\n");
+                return 0;
+        }
+
+        if (tb[NLMSGERR_ATTR_MSG])
+                errmsg = (char *) nla_data(tb[NLMSGERR_ATTR_MSG]);
+
+        fprintf(stderr, "Kernel error message: %s\n", errmsg);
+
+        return 0;
+}
diff --git a/tools/lib/bpf/nlattr.h b/tools/lib/bpf/nlattr.h
new file mode 100644
index 000000000000..931a71f68f93
--- /dev/null
+++ b/tools/lib/bpf/nlattr.h
@@ -0,0 +1,72 @@
+/* SPDX-License-Identifier: LGPL-2.1 */
+
+/*
+ * NETLINK      Netlink attributes
+ *
+ *      This library is free software; you can redistribute it and/or
+ *      modify it under the terms of the GNU Lesser General Public
+ *      License as published by the Free Software Foundation version 2.1
+ *      of the License.
+ *
+ * Copyright (c) 2003-2013 Thomas Graf <tgraf@suug.ch>
+ */
+
+#ifndef __NLATTR_H
+#define __NLATTR_H
+
+#include <stdint.h>
+#include <linux/netlink.h>
+/* avoid multiple definition of netlink features */
+#define __LINUX_NETLINK_H
+
+/**
+ * Standard attribute types to specify validation policy
+ */
+enum {
+        NLA_UNSPEC,     /**< Unspecified type, binary data chunk */
+        NLA_U8,         /**< 8 bit integer */
+        NLA_U16,        /**< 16 bit integer */
+        NLA_U32,        /**< 32 bit integer */
+        NLA_U64,        /**< 64 bit integer */
+        NLA_STRING,     /**< NUL terminated character string */
+        NLA_FLAG,       /**< Flag */
+        NLA_MSECS,      /**< Micro seconds (64bit) */
+        NLA_NESTED,     /**< Nested attributes */
+        __NLA_TYPE_MAX,
+};
+
+#define NLA_TYPE_MAX (__NLA_TYPE_MAX - 1)
+
+/**
+ * @ingroup attr
+ * Attribute validation policy.
+ *
+ * See section @core_doc{core_attr_parse,Attribute Parsing} for more details.
+ */
+struct nla_policy {
+        /** Type of attribute or NLA_UNSPEC */
+        uint16_t        type;
+
+        /** Minimal length of payload required */
+        uint16_t        minlen;
+
+        /** Maximal length of payload allowed */
+        uint16_t        maxlen;
+};
+
+/**
+ * @ingroup attr
+ * Iterate over a stream of attributes
+ * @arg pos     loop counter, set to current attribute
+ * @arg head    head of attribute stream
+ * @arg len     length of attribute stream
+ * @arg rem     initialized to len, holds bytes currently remaining in stream
+ */
+#define nla_for_each_attr(pos, head, len, rem) \
+        for (pos = head, rem = len; \
+             nla_ok(pos, rem); \
+             pos = nla_next(pos, &(rem)))
+
+int nla_dump_errormsg(struct nlmsghdr *nlh);
+
+#endif /* __NLATTR_H */
diff --git a/tools/lib/find_bit.c b/tools/lib/find_bit.c
index 42c15f906aac..a88bd507091e 100644
--- a/tools/lib/find_bit.c
+++ b/tools/lib/find_bit.c
@@ -22,22 +22,29 @@
 #include <linux/bitmap.h>
 #include <linux/kernel.h>
 
-#if !defined(find_next_bit)
+#if !defined(find_next_bit) || !defined(find_next_zero_bit) || \
+                !defined(find_next_and_bit)
 
 /*
- * This is a common helper function for find_next_bit and
- * find_next_zero_bit.  The difference is the "invert" argument, which
- * is XORed with each fetched word before searching it for one bits.
+ * This is a common helper function for find_next_bit, find_next_zero_bit, and
+ * find_next_and_bit. The differences are:
+ *  - The "invert" argument, which is XORed with each fetched word before
+ *    searching it for one bits.
+ *  - The optional "addr2", which is anded with "addr1" if present.
  */
-static unsigned long _find_next_bit(const unsigned long *addr,
-                unsigned long nbits, unsigned long start, unsigned long invert)
+static inline unsigned long _find_next_bit(const unsigned long *addr1,
+                const unsigned long *addr2, unsigned long nbits,
+                unsigned long start, unsigned long invert)
 {
         unsigned long tmp;
 
         if (unlikely(start >= nbits))
                 return nbits;
 
-        tmp = addr[start / BITS_PER_LONG] ^ invert;
+        tmp = addr1[start / BITS_PER_LONG];
+        if (addr2)
+                tmp &= addr2[start / BITS_PER_LONG];
+        tmp ^= invert;
 
         /* Handle 1st word. */
         tmp &= BITMAP_FIRST_WORD_MASK(start);
@@ -48,7 +55,10 @@ static unsigned long _find_next_bit(const unsigned long *addr,
                 if (start >= nbits)
                         return nbits;
 
-                tmp = addr[start / BITS_PER_LONG] ^ invert;
+                tmp = addr1[start / BITS_PER_LONG];
+                if (addr2)
+                        tmp &= addr2[start / BITS_PER_LONG];
+                tmp ^= invert;
         }
 
         return min(start + __ffs(tmp), nbits);
@@ -62,7 +72,7 @@ static unsigned long _find_next_bit(const unsigned long *addr,
 unsigned long find_next_bit(const unsigned long *addr, unsigned long size,
                             unsigned long offset)
 {
-        return _find_next_bit(addr, size, offset, 0UL);
+        return _find_next_bit(addr, NULL, size, offset, 0UL);
 }
 #endif
 
@@ -104,6 +114,15 @@ unsigned long find_first_zero_bit(const unsigned long *addr, unsigned long size)
 unsigned long find_next_zero_bit(const unsigned long *addr, unsigned long size,
                                  unsigned long offset)
 {
-        return _find_next_bit(addr, size, offset, ~0UL);
+        return _find_next_bit(addr, NULL, size, offset, ~0UL);
+}
+#endif
+
+#ifndef find_next_and_bit
+unsigned long find_next_and_bit(const unsigned long *addr1,
+                const unsigned long *addr2, unsigned long size,
+                unsigned long offset)
+{
+        return _find_next_bit(addr1, addr2, size, offset, 0UL);
 }
 #endif
diff --git a/tools/lib/subcmd/pager.c b/tools/lib/subcmd/pager.c
index 5ba754d17952..9997a8805a82 100644
--- a/tools/lib/subcmd/pager.c
+++ b/tools/lib/subcmd/pager.c
@@ -30,10 +30,13 @@ static void pager_preexec(void)
          * have real input
          */
         fd_set in;
+        fd_set exception;
 
         FD_ZERO(&in);
+        FD_ZERO(&exception);
         FD_SET(0, &in);
-        select(1, &in, NULL, &in, NULL);
+        FD_SET(0, &exception);
+        select(1, &in, NULL, &exception, NULL);
 
         setenv("LESS", "FRSX", 0);
 }
diff --git a/tools/lib/traceevent/event-parse.c b/tools/lib/traceevent/event-parse.c
index 7ce724fc0544..e5f2acbb70cc 100644
--- a/tools/lib/traceevent/event-parse.c
+++ b/tools/lib/traceevent/event-parse.c
@@ -1094,7 +1094,7 @@ static enum event_type __read_token(char **tok)
                 if (strcmp(*tok, "LOCAL_PR_FMT") == 0) {
                         free(*tok);
                         *tok = NULL;
-                        return force_token("\"\%s\" ", tok);
+                        return force_token("\"%s\" ", tok);
                 } else if (strcmp(*tok, "STA_PR_FMT") == 0) {
                         free(*tok);
                         *tok = NULL;
@@ -3970,6 +3970,11 @@ static void print_str_arg(struct trace_seq *s, void *data, int size,
                                 val &= ~fval;
                         }
                 }
+                if (val) {
+                        if (print && arg->flags.delim)
+                                trace_seq_puts(s, arg->flags.delim);
+                        trace_seq_printf(s, "0x%llx", val);
+                }
                 break;
         case PRINT_SYMBOL:
                 val = eval_num_arg(data, size, event, arg->symbol.field);
@@ -3980,6 +3985,8 @@ static void print_str_arg(struct trace_seq *s, void *data, int size,
                                 break;
                         }
                 }
+                if (!flag)
+                        trace_seq_printf(s, "0x%llx", val);
                 break;
         case PRINT_HEX:
         case PRINT_HEX_STR:
@@ -4293,6 +4300,26 @@ static struct print_arg *make_bprint_args(char *fmt, void *data, int size, struc
                                 goto process_again;
                         case 'p':
                                 ls = 1;
+                                if (isalnum(ptr[1])) {
+                                        ptr++;
+                                        /* Check for special pointers */
+                                        switch (*ptr) {
+                                        case 's':
+                                        case 'S':
+                                        case 'f':
+                                        case 'F':
+                                                break;
+                                        default:
+                                                /*
+                                                 * Older kernels do not process
+                                                 * dereferenced pointers.
+                                                 * Only process if the pointer
+                                                 * value is a printable.
+                                                 */
+                                                if (isprint(*(char *)bptr))
+                                                        goto process_string;
+                                        }
+                                }
                                 /* fall through */
                         case 'd':
                         case 'u':
@@ -4345,6 +4372,7 @@ static struct print_arg *make_bprint_args(char *fmt, void *data, int size, struc
 
                                 break;
                         case 's':
+ process_string:
                                 arg = alloc_arg();
                                 if (!arg) {
                                         do_warning_event(event, "%s(%d): not enough memory!",
@@ -4949,21 +4977,27 @@ static void pretty_print(struct trace_seq *s, void *data, int size, struct event
                                 else
                                         ls = 2;
 
-                                if (*(ptr+1) == 'F' || *(ptr+1) == 'f' ||
-                                    *(ptr+1) == 'S' || *(ptr+1) == 's') {
+                                if (isalnum(ptr[1]))
                                         ptr++;
+
+                                if (arg->type == PRINT_BSTRING) {
+                                        trace_seq_puts(s, arg->string.string);
+                                        break;
+                                }
+
+                                if (*ptr == 'F' || *ptr == 'f' ||
+                                    *ptr == 'S' || *ptr == 's') {
                                         show_func = *ptr;
-                                } else if (*(ptr+1) == 'M' || *(ptr+1) == 'm') {
-                                        print_mac_arg(s, *(ptr+1), data, size, event, arg);
-                                        ptr++;
+                                } else if (*ptr == 'M' || *ptr == 'm') {
+                                        print_mac_arg(s, *ptr, data, size, event, arg);
                                         arg = arg->next;
                                         break;
-                                } else if (*(ptr+1) == 'I' || *(ptr+1) == 'i') {
+                                } else if (*ptr == 'I' || *ptr == 'i') {
                                         int n;
 
-                                        n = print_ip_arg(s, ptr+1, data, size, event, arg);
+                                        n = print_ip_arg(s, ptr, data, size, event, arg);
                                         if (n > 0) {
-                                                ptr += n;
+                                                ptr += n - 1;
                                                 arg = arg->next;
                                                 break;
                                         }
@@ -5532,8 +5566,14 @@ void pevent_print_event(struct pevent *pevent, struct trace_seq *s,
 
         event = pevent_find_event_by_record(pevent, record);
         if (!event) {
-                do_warning("ug! no event found for type %d",
-                           trace_parse_common_type(pevent, record->data));
+                int i;
+                int type = trace_parse_common_type(pevent, record->data);
+
+                do_warning("ug! no event found for type %d", type);
+                trace_seq_printf(s, "[UNKNOWN TYPE %d]", type);
+                for (i = 0; i < record->size; i++)
+                        trace_seq_printf(s, " %02x",
+                                         ((unsigned char *)record->data)[i]);
                 return;
         }
 
diff --git a/tools/lib/traceevent/event-plugin.c b/tools/lib/traceevent/event-plugin.c
index a16756ae3526..d542cb60ca1a 100644
--- a/tools/lib/traceevent/event-plugin.c
+++ b/tools/lib/traceevent/event-plugin.c
@@ -120,12 +120,12 @@ char **traceevent_plugin_list_options(void)
                 for (op = reg->options; op->name; op++) {
                         char *alias = op->plugin_alias ? op->plugin_alias : op->file;
                         char **temp = list;
+                        int ret;
 
-                        name = malloc(strlen(op->name) + strlen(alias) + 2);
-                        if (!name)
+                        ret = asprintf(&name, "%s:%s", alias, op->name);
+                        if (ret < 0)
                                 goto err;
 
-                        sprintf(name, "%s:%s", alias, op->name);
                         list = realloc(list, count + 2);
                         if (!list) {
                                 list = temp;
@@ -290,17 +290,14 @@ load_plugin(struct pevent *pevent, const char *path,
         const char *alias;
         char *plugin;
         void *handle;
+        int ret;
 
-        plugin = malloc(strlen(path) + strlen(file) + 2);
-        if (!plugin) {
+        ret = asprintf(&plugin, "%s/%s", path, file);
+        if (ret < 0) {
                 warning("could not allocate plugin memory\n");
                 return;
         }
 
-        strcpy(plugin, path);
-        strcat(plugin, "/");
-        strcat(plugin, file);
-
         handle = dlopen(plugin, RTLD_NOW | RTLD_GLOBAL);
         if (!handle) {
                 warning("could not load plugin '%s'\n%s\n",
@@ -391,6 +388,7 @@ load_plugins(struct pevent *pevent, const char *suffix,
         char *home;
         char *path;
         char *envdir;
+        int ret;
 
         if (pevent->flags & PEVENT_DISABLE_PLUGINS)
                 return;
@@ -421,16 +419,12 @@ load_plugins(struct pevent *pevent, const char *suffix,
         if (!home)
                 return;
 
-        path = malloc(strlen(home) + strlen(LOCAL_PLUGIN_DIR) + 2);
-        if (!path) {
+        ret = asprintf(&path, "%s/%s", home, LOCAL_PLUGIN_DIR);
+        if (ret < 0) {
                 warning("could not allocate plugin memory\n");
                 return;
         }
 
-        strcpy(path, home);
-        strcat(path, "/");
-        strcat(path, LOCAL_PLUGIN_DIR);
-
         load_plugins_dir(pevent, suffix, path, load_plugin, data);
 
         free(path);
diff --git a/tools/lib/traceevent/kbuffer-parse.c b/tools/lib/traceevent/kbuffer-parse.c
index c94e3641b046..ca424b157e46 100644
--- a/tools/lib/traceevent/kbuffer-parse.c
+++ b/tools/lib/traceevent/kbuffer-parse.c
@@ -24,8 +24,8 @@
 
 #include "kbuffer.h"
 
-#define MISSING_EVENTS (1 << 31)
-#define MISSING_STORED (1 << 30)
+#define MISSING_EVENTS (1UL << 31)
+#define MISSING_STORED (1UL << 30)
 
 #define COMMIT_MASK ((1 << 27) - 1)
 
diff --git a/tools/lib/traceevent/parse-filter.c b/tools/lib/traceevent/parse-filter.c
index 315df0a70265..431e8b309f6e 100644
--- a/tools/lib/traceevent/parse-filter.c
+++ b/tools/lib/traceevent/parse-filter.c
@@ -287,12 +287,10 @@ find_event(struct pevent *pevent, struct event_list **events,
                 sys_name = NULL;
         }
 
-        reg = malloc(strlen(event_name) + 3);
-        if (reg == NULL)
+        ret = asprintf(&reg, "^%s$", event_name);
+        if (ret < 0)
                 return PEVENT_ERRNO__MEM_ALLOC_FAILED;
 
-        sprintf(reg, "^%s$", event_name);
-
         ret = regcomp(&ereg, reg, REG_ICASE|REG_NOSUB);
         free(reg);
 
@@ -300,13 +298,12 @@ find_event(struct pevent *pevent, struct event_list **events,
                 return PEVENT_ERRNO__INVALID_EVENT_NAME;
 
         if (sys_name) {
-                reg = malloc(strlen(sys_name) + 3);
-                if (reg == NULL) {
+                ret = asprintf(&reg, "^%s$", sys_name);
+                if (ret < 0) {
                         regfree(&ereg);
                         return PEVENT_ERRNO__MEM_ALLOC_FAILED;
                 }
 
-                sprintf(reg, "^%s$", sys_name);
                 ret = regcomp(&sreg, reg, REG_ICASE|REG_NOSUB);
                 free(reg);
                 if (ret) {
@@ -1634,6 +1631,7 @@ int pevent_filter_clear_trivial(struct event_filter *filter,
                 case FILTER_TRIVIAL_FALSE:
                         if (filter_type->filter->boolean.value)
                                 continue;
+                        break;
                 case FILTER_TRIVIAL_TRUE:
                         if (!filter_type->filter->boolean.value)
                                 continue;
@@ -1879,17 +1877,25 @@ static const char *get_field_str(struct filter_arg *arg, struct pevent_record *r
         struct pevent *pevent;
         unsigned long long addr;
         const char *val = NULL;
+        unsigned int size;
         char hex[64];
 
         /* If the field is not a string convert it */
         if (arg->str.field->flags & FIELD_IS_STRING) {
                 val = record->data + arg->str.field->offset;
+                size = arg->str.field->size;
+
+                if (arg->str.field->flags & FIELD_IS_DYNAMIC) {
+                        addr = *(unsigned int *)val;
+                        val = record->data + (addr & 0xffff);
+                        size = addr >> 16;
+                }
 
                 /*
                  * We need to copy the data since we can't be sure the field
                  * is null terminated.
                  */
-                if (*(val + arg->str.field->size - 1)) {
+                if (*(val + size - 1)) {
                         /* copy it */
                         memcpy(arg->str.buffer, val, arg->str.field->size);
                         /* the buffer is already NULL terminated */