diff options
author | Zachary Leaf <zachary.leaf@arm.com> | 2022-05-23 06:22:27 -0500 |
---|---|---|
committer | Damjan Marion <dmarion@0xa5.net> | 2022-07-12 15:29:23 +0000 |
commit | c7d43a5eb19f2acab900274432cfd0e136d6cb44 (patch) | |
tree | 3ad3bd5191a1199430db40cec0b286ab531993be /src/plugins/perfmon/perfmon.h | |
parent | 20ac58e5c5ef59bc860270037aa7a3b0966a4ec2 (diff) |
perfmon: make less arch dependent
In preparation for enabling perfmon on Arm platforms, move some Intel
/arch specific logic into the /intel directory and update the CMake to
split the common code from arch specific files.
Since the dispatch_wrapper code is very different on Arm/Intel,
each arch can provide their own implementation + conduct any additional
arch specific config e.g. on Intel, all indexes from the mmap pages are
cached. The new method intel_config_dispatch_wrapper conducts this
config and returns a pointer to the dispatch wrapper to use.
Similarly, is_bundle_supported() looks very different on Arm/Intel, so
each implementation is to provide their own arch specific checks.
Two new callbacks/function ptrs are added in PERFMON_REGISTER_SOURCE to
support this - .bundle_support and .config_dispatch_wrapper.
Type: refactor
Signed-off-by: Zachary Leaf <zachary.leaf@arm.com>
Change-Id: Idd121ddcfd1cc80a57c949cecd64eb2db0ac8be3
Diffstat (limited to 'src/plugins/perfmon/perfmon.h')
-rw-r--r-- | src/plugins/perfmon/perfmon.h | 11 |
1 files changed, 8 insertions, 3 deletions
diff --git a/src/plugins/perfmon/perfmon.h b/src/plugins/perfmon/perfmon.h index fa15ef63aca..f08cc7a5dcb 100644 --- a/src/plugins/perfmon/perfmon.h +++ b/src/plugins/perfmon/perfmon.h @@ -23,7 +23,9 @@ #include <vppinfra/cpu.h> #include <vlib/vlib.h> +#if defined(__x86_64__) #define PERF_MAX_EVENTS 12 /* 4 fixed and 8 programable on ICX */ +#endif typedef enum { @@ -86,11 +88,14 @@ typedef struct } perfmon_instance_type_t; struct perfmon_source; -extern vlib_node_function_t *perfmon_dispatch_wrappers[PERF_MAX_EVENTS + 1]; +typedef struct perfmon_bundle perfmon_bundle_t; typedef clib_error_t *(perfmon_source_init_fn_t) (vlib_main_t *vm, struct perfmon_source *); typedef perfmon_event_type_t (perfmon_source_get_event_type) (u32 event); +typedef u8 (perfmon_source_bundle_support_t) (perfmon_bundle_t *); +typedef clib_error_t *(perfmon_source_config_dispatch_wrapper_t) ( + perfmon_bundle_t *b, vlib_node_function_t **dispatch_wrapper); typedef struct perfmon_source { @@ -103,10 +108,10 @@ typedef struct perfmon_source format_function_t *format_config; perfmon_source_get_event_type *get_event_type; perfmon_source_init_fn_t *init_fn; + perfmon_source_bundle_support_t *bundle_support; + perfmon_source_config_dispatch_wrapper_t *config_dispatch_wrapper; } perfmon_source_t; -struct perfmon_bundle; - typedef clib_error_t *(perfmon_bundle_init_fn_t) (vlib_main_t *vm, struct perfmon_bundle *); |