ebpf: improve trace event coverage to all key operations
The existing error trace event is renamed to have a name prefix matching its source file & to remove the redundant first arg that adds no useful information. Signed-off-by: Daniel P. Berrangé <berrange@redhat.com> Signed-off-by: Jason Wang <jasowang@redhat.com>
This commit is contained in:
parent
b5900dff14
commit
f5cae19d10
@ -74,6 +74,10 @@ static bool ebpf_rss_mmap(struct EBPFRSSContext *ctx, Error **errp)
|
||||
goto indirection_fail;
|
||||
}
|
||||
|
||||
trace_ebpf_rss_mmap(ctx,
|
||||
ctx->mmap_configuration,
|
||||
ctx->mmap_toeplitz_key,
|
||||
ctx->mmap_indirections_table);
|
||||
return true;
|
||||
|
||||
indirection_fail:
|
||||
@ -131,6 +135,11 @@ bool ebpf_rss_load(struct EBPFRSSContext *ctx, Error **errp)
|
||||
ctx->map_toeplitz_key = bpf_map__fd(
|
||||
rss_bpf_ctx->maps.tap_rss_map_toeplitz_key);
|
||||
|
||||
trace_ebpf_rss_load(ctx,
|
||||
ctx->program_fd,
|
||||
ctx->map_configuration,
|
||||
ctx->map_indirections_table,
|
||||
ctx->map_toeplitz_key);
|
||||
if (!ebpf_rss_mmap(ctx, errp)) {
|
||||
goto error;
|
||||
}
|
||||
@ -178,6 +187,12 @@ bool ebpf_rss_load_fds(struct EBPFRSSContext *ctx, int program_fd,
|
||||
ctx->map_toeplitz_key = toeplitz_fd;
|
||||
ctx->map_indirections_table = table_fd;
|
||||
|
||||
trace_ebpf_rss_load(ctx,
|
||||
ctx->program_fd,
|
||||
ctx->map_configuration,
|
||||
ctx->map_indirections_table,
|
||||
ctx->map_toeplitz_key);
|
||||
|
||||
if (!ebpf_rss_mmap(ctx, errp)) {
|
||||
ctx->program_fd = -1;
|
||||
ctx->map_configuration = -1;
|
||||
@ -259,6 +274,8 @@ bool ebpf_rss_set_all(struct EBPFRSSContext *ctx, struct EBPFRSSConfig *config,
|
||||
|
||||
ebpf_rss_set_toepliz_key(ctx, toeplitz_key);
|
||||
|
||||
trace_ebpf_rss_set_data(ctx, config, indirections_table, toeplitz_key);
|
||||
|
||||
return true;
|
||||
}
|
||||
|
||||
@ -268,6 +285,8 @@ void ebpf_rss_unload(struct EBPFRSSContext *ctx)
|
||||
return;
|
||||
}
|
||||
|
||||
trace_ebpf_rss_unload(ctx);
|
||||
|
||||
ebpf_rss_munmap(ctx);
|
||||
|
||||
if (ctx->obj) {
|
||||
|
@ -1,6 +1,10 @@
|
||||
# See docs/devel/tracing.rst for syntax documentation.
|
||||
|
||||
# ebpf-rss.c
|
||||
ebpf_rss_load(void *ctx, int progfd, int cfgfd, int toepfd, int indirfd) "ctx=%p program-fd=%d config-fd=%d toeplitz-fd=%d indirection-fd=%d"
|
||||
ebpf_rss_load_error(void *ctx) "ctx=%p"
|
||||
ebpf_rss_mmap(void *ctx, void *cfgptr, void *toepptr, void *indirptr) "ctx=%p config-ptr=%p toeplitz-ptr=%p indirection-ptr=%p"
|
||||
ebpf_rss_mmap_error(void *ctx, const char *object) "ctx=%p object=%s"
|
||||
ebpf_rss_open_error(void *ctx) "ctx=%p"
|
||||
ebpf_rss_set_data(void *ctx, void *cfgptr, void *toepptr, void *indirptr) "ctx=%p config-ptr=%p toeplitz-ptr=%p indirection-ptr=%p"
|
||||
ebpf_rss_unload(void *ctx) "rss unload ctx=%p"
|
||||
|
Loading…
Reference in New Issue
Block a user