From patchwork Tue Oct 3 22:12:37 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Steven Rostedt X-Patchwork-Id: 13408020 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 0570F3E48A for ; Tue, 3 Oct 2023 22:11:32 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 2693AC433C8; Tue, 3 Oct 2023 22:11:32 +0000 (UTC) Date: Tue, 3 Oct 2023 18:12:37 -0400 From: Steven Rostedt To: Linux Trace Devel Cc: Ross Zwisler Subject: [PATCH] libtraceeval: Use name for type look up for traceeval_stat Message-ID: <20231003181237.616cc8db@gandalf.local.home> X-Mailer: Claws Mail 3.19.1 (GTK+ 2.24.33; x86_64-pc-linux-gnu) Precedence: bulk X-Mailing-List: linux-trace-devel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 From: "Steven Rostedt (Google)" The functions that retrieve the traceeval_stat currently requires passing in the struct traceeval_type. But this is fragile, as it may not be set up properly. It was originally done that way as a "fast" way to look up stats, but in reality it will likely just cause more issues with wrong types passed in. That decision was based more on defining policy which the library should not do. Instead, just pass in the string name of the value type to find and retrieve the traceeval_data value from the traceeval that way. Signed-off-by: Steven Rostedt (Google) --- include/traceeval.h | 4 ++-- src/histograms.c | 39 ++++++++++++++++++++++++++++++--------- 2 files changed, 32 insertions(+), 11 deletions(-) diff --git a/include/traceeval.h b/include/traceeval.h index 48c28dfc2f99..32996ba17e5f 100644 --- a/include/traceeval.h +++ b/include/traceeval.h @@ -219,7 +219,7 @@ size_t traceeval_count(struct traceeval *teval); struct traceeval_stat *traceeval_stat_size(struct traceeval *teval, const struct traceeval_data *keys, size_t nr_keys, - struct traceeval_type *type); + const char *val_name); unsigned long long traceeval_stat_max(struct traceeval_stat *stat); unsigned long long traceeval_stat_min(struct traceeval_stat *stat); @@ -239,7 +239,7 @@ int traceeval_iterator_query(struct traceeval_iterator *iter, void traceeval_iterator_results_release(struct traceeval_iterator *iter, const struct traceeval_data *results); struct traceeval_stat *traceeval_iterator_stat(struct traceeval_iterator *iter, - struct traceeval_type *type); + const char *val_name); int traceeval_iterator_remove(struct traceeval_iterator *iter); #endif /* __LIBTRACEEVAL_HIST_H__ */ diff --git a/src/histograms.c b/src/histograms.c index 3ba7687af9bb..2be95e6a8d1f 100644 --- a/src/histograms.c +++ b/src/histograms.c @@ -849,17 +849,36 @@ static int update_entry(struct traceeval *teval, struct entry *entry, return -1; } +static struct traceeval_type *find_val_type(struct traceeval *teval, const char *name) +{ + struct traceeval_type *type; + int i; + + for (i = 0; i < teval->nr_val_types; i++) { + type = &teval->val_types[i]; + + if (strcmp(type->name, name) == 0) + return type; + } + return NULL; +} + struct traceeval_stat *traceeval_stat_size(struct traceeval *teval, const struct traceeval_data *keys, size_t nr_keys, - struct traceeval_type *type) + const char *val_name) { + struct traceeval_type *type; struct entry *entry; int ret; if (teval->nr_key_types != nr_keys) return NULL; + type = find_val_type(teval, val_name); + if (!type) + return NULL; + if (!is_stat_type(type)) return NULL; @@ -1116,12 +1135,9 @@ static struct traceeval_type *find_sort_type(struct traceeval *teval, int i; /* Check values first, and then keys */ - for (i = 0; i < teval->nr_val_types; i++) { - type = &teval->val_types[i]; - - if (strcmp(type->name, name) == 0) - return type; - } + type = find_val_type(teval, name); + if (type) + return type; for (i = 0; i < teval->nr_key_types; i++) { type = &teval->key_types[i]; @@ -1426,16 +1442,21 @@ void traceeval_iterator_results_release(struct traceeval_iterator *iter, /** * traceeval_iterator_stat - return the stats from the last iterator entry * @iter: The iterator to retrieve the stats from - * @type: The value type to get the stat from + * @val_name: The name of the value to get the stat from * * Returns the stats of the @type for the current iterator entry on success, * or NULL if not found or an error occurred. */ struct traceeval_stat *traceeval_iterator_stat(struct traceeval_iterator *iter, - struct traceeval_type *type) + const char *val_name) { + struct traceeval_type *type; struct entry *entry; + type = find_val_type(iter->teval, val_name); + if (!type) + return NULL; + if (!is_stat_type(type)) return NULL;