The definition of the mask gets hidden from the user. This way we will
be able to modify this definition in the future, without breaking the
API. Such a modification will be compulsory, if too many new tracing
options are added in the future. Note that encapsulating the mask
definition, requires two API methods to be eliminated, however those
methods have no particular use-cases for the moment.
Signed-off-by: Yordan Karadzhov (VMware) <y.karadz@xxxxxxxxx>
---
include/tracefs.h | 6 +-----
src/tracefs-tools.c | 33 +++++++++++----------------------
2 files changed, 12 insertions(+), 27 deletions(-)
diff --git a/include/tracefs.h b/include/tracefs.h
index 5c4e50d..0665e8d 100644
--- a/include/tracefs.h
+++ b/include/tracefs.h
@@ -131,11 +131,7 @@ enum tracefs_option_id {
};
#define TRACEFS_OPTION_MAX (TRACEFS_OPTION_VERBOSE + 1)
-struct tracefs_options_mask {
- unsigned long long mask;
-};
-void tracefs_option_set(struct tracefs_options_mask *options, enum tracefs_option_id id);
-void tracefs_option_clear(struct tracefs_options_mask *options, enum tracefs_option_id id);
+struct tracefs_options_mask;
bool tracefs_option_is_set(struct tracefs_options_mask *options,
enum tracefs_option_id id);
struct tracefs_options_mask *tracefs_options_get_supported(struct tracefs_instance *instance);
diff --git a/src/tracefs-tools.c b/src/tracefs-tools.c
index 3b5f773..11a4c8c 100644
--- a/src/tracefs-tools.c
+++ b/src/tracefs-tools.c
@@ -19,6 +19,10 @@
#define TRACE_CTRL "tracing_on"
+struct tracefs_options_mask {
+ unsigned long long mask;
+};
+
static const char * const options_map[] = {
"unknown",
"annotate",
@@ -197,6 +201,13 @@ enum tracefs_option_id tracefs_option_id(const char *name)
return TRACEFS_OPTION_INVALID;
}
+static void tracefs_option_set(struct tracefs_options_mask *options,
+ enum tracefs_option_id id)
+{
+ if (options && id > TRACEFS_OPTION_INVALID)
+ options->mask |= (1ULL << (id - 1));
+}
+
static struct tracefs_options_mask *trace_get_options(struct tracefs_instance *instance,
bool enabled)
{
@@ -366,25 +377,3 @@ bool tracefs_option_is_set(struct tracefs_options_mask *options,
return options->mask & (1ULL << (id - 1));
return false;
}
-
-/**
- * tracefs_option_set - Set option in options bitmask
- * @options: Pointer to a bitmask with options
- * @id: trace option id
- */
-void tracefs_option_set(struct tracefs_options_mask *options, enum tracefs_option_id id)
-{
- if (options && id > TRACEFS_OPTION_INVALID)
- options->mask |= (1ULL << (id - 1));
-}
-
-/**
- * tracefs_option_clear - Clear option from options bitmask
- * @options: Pointer to a bitmask with options
- * @id: trace option id
- */
-void tracefs_option_clear(struct tracefs_options_mask *options, enum tracefs_option_id id)
-{
- if (options && id > TRACEFS_OPTION_INVALID)
- options->mask &= ~(1ULL << (id - 1));
-}
--
2.27.0
|