Commit bd560973 authored by Ian Rogers's avatar Ian Rogers Committed by Arnaldo Carvalho de Melo

perf expr: Tidy hashmap dependency

hashmap.h comes from libbpf but isn't installed with its
headers. Always use the header file of the code in util. Change the
hashmap.h dependency in expr.h to a forward declaration, add the
necessary header file includes in the C files.
Signed-off-by: default avatarIan Rogers <irogers@google.com>
Acked-by: default avatarNamhyung Kim <namhyung@kernel.org>
Cc: Alexander Shishkin <alexander.shishkin@linux.intel.com>
Cc: Andrii Nakryiko <andrii.nakryiko@gmail.com>
Cc: Jiri Olsa <jolsa@kernel.org>
Cc: Mark Rutland <mark.rutland@arm.com>
Cc: Masahiro Yamada <masahiroy@kernel.org>
Cc: Nick Desaulniers <ndesaulniers@google.com>
Cc: Nicolas Schier <nicolas@fjasle.eu>
Cc: Peter Zijlstra <peterz@infradead.org>
Cc: Stephane Eranian <eranian@google.com>
Cc: bpf@vger.kernel.org
Link: http://lore.kernel.org/lkml/20221109184914.1357295-12-irogers@google.comSigned-off-by: default avatarArnaldo Carvalho de Melo <acme@redhat.com>
parent 84bec6f0
...@@ -2,6 +2,7 @@ ...@@ -2,6 +2,7 @@
#include "util/cputopo.h" #include "util/cputopo.h"
#include "util/debug.h" #include "util/debug.h"
#include "util/expr.h" #include "util/expr.h"
#include "util/hashmap.h"
#include "util/header.h" #include "util/header.h"
#include "util/smt.h" #include "util/smt.h"
#include "tests.h" #include "tests.h"
......
...@@ -12,6 +12,7 @@ ...@@ -12,6 +12,7 @@
#include <perf/evlist.h> #include <perf/evlist.h>
#include "util/evlist.h" #include "util/evlist.h"
#include "util/expr.h" #include "util/expr.h"
#include "util/hashmap.h"
#include "util/parse-events.h" #include "util/parse-events.h"
#include "metricgroup.h" #include "metricgroup.h"
#include "stat.h" #include "stat.h"
......
...@@ -27,11 +27,7 @@ ...@@ -27,11 +27,7 @@
#include "util.h" #include "util.h"
#include "llvm-utils.h" #include "llvm-utils.h"
#include "c++/clang-c.h" #include "c++/clang-c.h"
#ifdef HAVE_LIBBPF_SUPPORT
#include <bpf/hashmap.h>
#else
#include "util/hashmap.h" #include "util/hashmap.h"
#endif
#include "asm/bug.h" #include "asm/bug.h"
#include <internal/xyarray.h> #include <internal/xyarray.h>
......
...@@ -46,11 +46,7 @@ ...@@ -46,11 +46,7 @@
#include "string2.h" #include "string2.h"
#include "memswap.h" #include "memswap.h"
#include "util.h" #include "util.h"
#ifdef HAVE_LIBBPF_SUPPORT
#include <bpf/hashmap.h>
#else
#include "util/hashmap.h" #include "util/hashmap.h"
#endif
#include "pmu-hybrid.h" #include "pmu-hybrid.h"
#include "off_cpu.h" #include "off_cpu.h"
#include "../perf-sys.h" #include "../perf-sys.h"
......
...@@ -11,6 +11,7 @@ ...@@ -11,6 +11,7 @@
#include "expr.h" #include "expr.h"
#include "expr-bison.h" #include "expr-bison.h"
#include "expr-flex.h" #include "expr-flex.h"
#include "util/hashmap.h"
#include "smt.h" #include "smt.h"
#include "tsc.h" #include "tsc.h"
#include <linux/err.h> #include <linux/err.h>
......
...@@ -2,12 +2,7 @@ ...@@ -2,12 +2,7 @@
#ifndef PARSE_CTX_H #ifndef PARSE_CTX_H
#define PARSE_CTX_H 1 #define PARSE_CTX_H 1
#ifdef HAVE_LIBBPF_SUPPORT struct hashmap;
#include <bpf/hashmap.h>
#else
#include "util/hashmap.h"
#endif
struct metric_ref; struct metric_ref;
struct expr_scanner_ctx { struct expr_scanner_ctx {
......
...@@ -28,6 +28,7 @@ ...@@ -28,6 +28,7 @@
#include "util.h" #include "util.h"
#include <asm/bug.h> #include <asm/bug.h>
#include "cgroup.h" #include "cgroup.h"
#include "util/hashmap.h"
struct metric_event *metricgroup__lookup(struct rblist *metric_events, struct metric_event *metricgroup__lookup(struct rblist *metric_events,
struct evsel *evsel, struct evsel *evsel,
......
...@@ -14,6 +14,7 @@ ...@@ -14,6 +14,7 @@
#include "units.h" #include "units.h"
#include <linux/zalloc.h> #include <linux/zalloc.h>
#include "iostat.h" #include "iostat.h"
#include "util/hashmap.h"
/* /*
* AGGR_GLOBAL: Use CPU 0 * AGGR_GLOBAL: Use CPU 0
......
...@@ -14,11 +14,7 @@ ...@@ -14,11 +14,7 @@
#include "evlist.h" #include "evlist.h"
#include "evsel.h" #include "evsel.h"
#include "thread_map.h" #include "thread_map.h"
#ifdef HAVE_LIBBPF_SUPPORT
#include <bpf/hashmap.h>
#else
#include "util/hashmap.h" #include "util/hashmap.h"
#endif
#include <linux/zalloc.h> #include <linux/zalloc.h>
void update_stats(struct stats *stats, u64 val) void update_stats(struct stats *stats, u64 val)
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment