Trace IR and notification APIs: split into private and public APIs
[babeltrace.git] / plugins / ctf / fs-src / query.c
index 696d3d43e297b4c2f6eeb31ff18577c103972760..d7c9824d468bcb94eb5d68821742e97d1ae4b1b6 100644 (file)
 
 #include "query.h"
 #include <stdbool.h>
-#include <assert.h>
+#include <babeltrace/assert-internal.h>
 #include "metadata.h"
 #include "../common/metadata/decoder.h"
 #include <babeltrace/common-internal.h>
 #include <babeltrace/babeltrace-internal.h>
-#include <babeltrace/ctf-ir/stream.h>
+#include <babeltrace/babeltrace.h>
 #include "fs.h"
 
 #define BT_LOG_TAG "PLUGIN-CTF-FS-QUERY-SRC"
@@ -46,15 +46,16 @@ struct range {
 };
 
 BT_HIDDEN
-struct bt_component_class_query_return metadata_info_query(
+struct bt_component_class_query_method_return metadata_info_query(
                struct bt_component_class *comp_class,
                struct bt_value *params)
 {
-       struct bt_component_class_query_return query_ret = {
+       struct bt_component_class_query_method_return query_ret = {
                .result = NULL,
                .status = BT_QUERY_STATUS_OK,
        };
 
+       struct bt_private_value *result = NULL;
        struct bt_value *path_value = NULL;
        char *metadata_text = NULL;
        FILE *metadata_fp = NULL;
@@ -64,27 +65,25 @@ struct bt_component_class_query_return metadata_info_query(
        const char *path;
        bool is_packetized;
 
-       query_ret.result = bt_value_map_create();
-       if (!query_ret.result) {
+       result = bt_private_value_map_create();
+       if (!result) {
                query_ret.status = BT_QUERY_STATUS_NOMEM;
                goto error;
        }
 
+       query_ret.result = bt_value_borrow_from_private(result);
+       BT_ASSERT(params);
+
        if (!bt_value_is_map(params)) {
                BT_LOGE_STR("Query parameters is not a map value object.");
                query_ret.status = BT_QUERY_STATUS_INVALID_PARAMS;
                goto error;
        }
 
-       path_value = bt_value_map_get(params, "path");
-       ret = bt_value_string_get(path_value, &path);
-       if (ret) {
-               BT_LOGE_STR("Cannot get `path` string parameter.");
-               query_ret.status = BT_QUERY_STATUS_INVALID_PARAMS;
-               goto error;
-       }
+       path_value = bt_value_map_borrow_entry_value(params, "path");
+       path = bt_value_string_get(path_value);
 
-       assert(path);
+       BT_ASSERT(path);
        metadata_fp = ctf_fs_metadata_open_file(path);
        if (!metadata_fp) {
                BT_LOGE("Cannot open trace metadata: path=\"%s\".", path);
@@ -146,14 +145,14 @@ struct bt_component_class_query_return metadata_info_query(
 
        g_string_append(g_metadata_text, metadata_text);
 
-       ret = bt_value_map_insert_string(query_ret.result, "text",
+       ret = bt_private_value_map_insert_string_entry(result, "text",
                g_metadata_text->str);
        if (ret) {
                BT_LOGE_STR("Cannot insert metadata text into query result.");
                goto error;
        }
 
-       ret = bt_value_map_insert_bool(query_ret.result, "is-packetized",
+       ret = bt_private_value_map_insert_bool_entry(result, "is-packetized",
                is_packetized);
        if (ret) {
                BT_LOGE_STR("Cannot insert \"is-packetized\" attribute into query result.");
@@ -163,14 +162,14 @@ struct bt_component_class_query_return metadata_info_query(
        goto end;
 
 error:
-       BT_PUT(query_ret.result);
+       BT_OBJECT_PUT_REF_AND_RESET(result);
+       query_ret.result = NULL;
 
        if (query_ret.status >= 0) {
                query_ret.status = BT_QUERY_STATUS_ERROR;
        }
 
 end:
-       bt_put(path_value);
        free(metadata_text);
 
        if (g_metadata_text) {
@@ -185,59 +184,62 @@ end:
 }
 
 static
-int add_range(struct bt_value *info, struct range *range,
+int add_range(struct bt_private_value *info, struct range *range,
                const char *range_name)
 {
        int ret = 0;
        enum bt_value_status status;
-       struct bt_value *range_map = NULL;
+       struct bt_private_value *range_map = NULL;
 
        if (!range->set) {
                /* Not an error. */
                goto end;
        }
 
-       range_map = bt_value_map_create();
+       range_map = bt_private_value_map_create();
        if (!range_map) {
                ret = -1;
                goto end;
        }
 
-       status = bt_value_map_insert_integer(range_map, "begin",
+       status = bt_private_value_map_insert_integer_entry(range_map, "begin",
                        range->begin_ns);
        if (status != BT_VALUE_STATUS_OK) {
                ret = -1;
                goto end;
        }
 
-       status = bt_value_map_insert_integer(range_map, "end",
+       status = bt_private_value_map_insert_integer_entry(range_map, "end",
                        range->end_ns);
        if (status != BT_VALUE_STATUS_OK) {
                ret = -1;
                goto end;
        }
 
-       status = bt_value_map_insert(info, range_name, range_map);
+       status = bt_private_value_map_insert_entry(info, range_name,
+               bt_value_borrow_from_private(range_map));
        if (status != BT_VALUE_STATUS_OK) {
                ret = -1;
                goto end;
        }
+
 end:
-       bt_put(range_map);
+       bt_object_put_ref(range_map);
        return ret;
 }
 
 static
-int add_stream_ids(struct bt_value *info, struct bt_ctf_stream *stream)
+int add_stream_ids(struct bt_private_value *info,
+               struct bt_stream *stream)
 {
        int ret = 0;
        int64_t stream_class_id, stream_instance_id;
        enum bt_value_status status;
-       struct bt_ctf_stream_class *stream_class = NULL;
+       struct bt_stream_class *stream_class = NULL;
 
-       stream_instance_id = bt_ctf_stream_get_id(stream);
+       stream_instance_id = bt_stream_get_id(stream);
        if (stream_instance_id != -1) {
-               status = bt_value_map_insert_integer(info, "id",
+               status = bt_private_value_map_insert_integer_entry(info, "id",
                                stream_instance_id);
                if (status != BT_VALUE_STATUS_OK) {
                        ret = -1;
@@ -245,42 +247,42 @@ int add_stream_ids(struct bt_value *info, struct bt_ctf_stream *stream)
                }
        }
 
-       stream_class = bt_ctf_stream_get_class(stream);
+       stream_class = bt_stream_borrow_class(stream);
        if (!stream_class) {
                ret = -1;
                goto end;
        }
 
-       stream_class_id = bt_ctf_stream_class_get_id(stream_class);
+       stream_class_id = bt_stream_class_get_id(stream_class);
        if (stream_class_id == -1) {
                ret = -1;
                goto end;
        }
 
-       status = bt_value_map_insert_integer(info, "class-id", stream_class_id);
+       status = bt_private_value_map_insert_integer_entry(info, "class-id", stream_class_id);
        if (status != BT_VALUE_STATUS_OK) {
                ret = -1;
                goto end;
        }
+
 end:
-       bt_put(stream_class);
        return ret;
 }
 
 static
 int populate_stream_info(struct ctf_fs_ds_file_group *group,
-               struct bt_value *group_info,
+               struct bt_private_value *group_info,
                struct range *stream_range)
 {
        int ret = 0;
        size_t file_idx;
        enum bt_value_status status;
-       struct bt_value *file_paths;
+       struct bt_private_value *file_paths;
 
        stream_range->begin_ns = INT64_MAX;
        stream_range->end_ns = 0;
 
-       file_paths = bt_value_array_create();
+       file_paths = bt_private_value_array_create();
        if (!file_paths) {
                ret = -1;
                goto end;
@@ -299,7 +301,7 @@ int populate_stream_info(struct ctf_fs_ds_file_group *group,
                        goto end;
                }
 
-               status = bt_value_array_append_string(file_paths,
+               status = bt_private_value_array_append_string_element(file_paths,
                                info->path->str);
                if (status != BT_VALUE_STATUS_OK) {
                        ret = -1;
@@ -327,30 +329,32 @@ int populate_stream_info(struct ctf_fs_ds_file_group *group,
                }
        }
 
-       status = bt_value_map_insert(group_info, "paths", file_paths);
+       status = bt_private_value_map_insert_entry(group_info, "paths",
+               bt_value_borrow_from_private(file_paths));
        if (status != BT_VALUE_STATUS_OK) {
                ret = -1;
                goto end;
        }
 
-       ret = add_stream_ids(group_info, group->stream);
+       ret = add_stream_ids(group_info,
+               bt_stream_borrow_from_private(group->stream));
        if (ret) {
                goto end;
        }
 end:
-       bt_put(file_paths);
+       bt_object_put_ref(file_paths);
        return ret;
 }
 
 static
 int populate_trace_info(const char *trace_path, const char *trace_name,
-               struct bt_value *trace_info)
+               struct bt_private_value *trace_info)
 {
        int ret = 0;
        size_t group_idx;
        struct ctf_fs_trace *trace = NULL;
        enum bt_value_status status;
-       struct bt_value *file_groups;
+       struct bt_private_value *file_groups;
        struct range trace_range = {
                .begin_ns = INT64_MAX,
                .end_ns = 0,
@@ -362,18 +366,18 @@ int populate_trace_info(const char *trace_path, const char *trace_name,
                .set = false,
        };
 
-       file_groups = bt_value_array_create();
+       file_groups = bt_private_value_array_create();
        if (!file_groups) {
                goto end;
        }
 
-       status = bt_value_map_insert_string(trace_info, "name",
-                       trace_name);
+       status = bt_private_value_map_insert_string_entry(trace_info, "name",
+               trace_name);
        if (status != BT_VALUE_STATUS_OK) {
                ret = -1;
                goto end;
        }
-       status = bt_value_map_insert_string(trace_info, "path",
+       status = bt_private_value_map_insert_string_entry(trace_info, "path",
                        trace_path);
        if (status != BT_VALUE_STATUS_OK) {
                ret = -1;
@@ -387,7 +391,7 @@ int populate_trace_info(const char *trace_path, const char *trace_name,
                goto end;
        }
 
-       assert(trace->ds_file_groups);
+       BT_ASSERT(trace->ds_file_groups);
        /* Add trace range info only if it contains streams. */
        if (trace->ds_file_groups->len == 0) {
                ret = -1;
@@ -397,12 +401,12 @@ int populate_trace_info(const char *trace_path, const char *trace_name,
        /* Find range of all stream groups, and of the trace. */
        for (group_idx = 0; group_idx < trace->ds_file_groups->len;
                        group_idx++) {
-               struct bt_value *group_info;
+               struct bt_private_value *group_info;
                struct range group_range = { .set = false };
                struct ctf_fs_ds_file_group *group = g_ptr_array_index(
                                trace->ds_file_groups, group_idx);
 
-               group_info = bt_value_map_create();
+               group_info = bt_private_value_map_create();
                if (!group_info) {
                        ret = -1;
                        goto end;
@@ -410,7 +414,7 @@ int populate_trace_info(const char *trace_path, const char *trace_name,
 
                ret = populate_stream_info(group, group_info, &group_range);
                if (ret) {
-                       bt_put(group_info);
+                       bt_object_put_ref(group_info);
                        goto end;
                }
 
@@ -426,8 +430,10 @@ int populate_trace_info(const char *trace_path, const char *trace_name,
                        trace_intersection.end_ns = min(trace_intersection.end_ns,
                                        group_range.end_ns);
                        trace_intersection.set = true;
-                       status = bt_value_array_append(file_groups, group_info);
-                       bt_put(group_info);
+                       status = bt_private_value_array_append_element(
+                               file_groups,
+                               bt_value_borrow_from_private(group_info));
+                       bt_object_put_ref(group_info);
                        if (status != BT_VALUE_STATUS_OK) {
                                goto end;
                        }
@@ -438,35 +444,40 @@ int populate_trace_info(const char *trace_path, const char *trace_name,
        if (ret) {
                goto end;
        }
-       ret = add_range(trace_info, &trace_intersection,
-                       "intersection-range-ns");
-       if (ret) {
-               goto end;
+
+       if (trace_intersection.begin_ns < trace_intersection.end_ns) {
+               ret = add_range(trace_info, &trace_intersection,
+                               "intersection-range-ns");
+               if (ret) {
+                       goto end;
+               }
        }
 
-       status = bt_value_map_insert(trace_info, "streams", file_groups);
-       BT_PUT(file_groups);
+       status = bt_private_value_map_insert_entry(trace_info, "streams",
+               bt_value_borrow_from_private(file_groups));
+       BT_OBJECT_PUT_REF_AND_RESET(file_groups);
        if (status != BT_VALUE_STATUS_OK) {
                ret = -1;
                goto end;
        }
 
 end:
-       bt_put(file_groups);
+       bt_object_put_ref(file_groups);
        ctf_fs_trace_destroy(trace);
        return ret;
 }
 
 BT_HIDDEN
-struct bt_component_class_query_return trace_info_query(
+struct bt_component_class_query_method_return trace_info_query(
                struct bt_component_class *comp_class,
                struct bt_value *params)
 {
-       struct bt_component_class_query_return query_ret = {
+       struct bt_component_class_query_method_return query_ret = {
                .result = NULL,
                .status = BT_QUERY_STATUS_OK,
        };
 
+       struct bt_private_value *result = NULL;
        struct bt_value *path_value = NULL;
        int ret = 0;
        const char *path = NULL;
@@ -476,26 +487,23 @@ struct bt_component_class_query_return trace_info_query(
        GList *tn_node = NULL;
        GString *normalized_path = NULL;
 
+       BT_ASSERT(params);
+
        if (!bt_value_is_map(params)) {
                BT_LOGE("Query parameters is not a map value object.");
                query_ret.status = BT_QUERY_STATUS_INVALID_PARAMS;
                goto error;
        }
 
-       path_value = bt_value_map_get(params, "path");
-       ret = bt_value_string_get(path_value, &path);
-       if (ret) {
-               BT_LOGE("Cannot get `path` string parameter.");
-               query_ret.status = BT_QUERY_STATUS_INVALID_PARAMS;
-               goto error;
-       }
+       path_value = bt_value_map_borrow_entry_value(params, "path");
+       path = bt_value_string_get(path_value);
 
        normalized_path = bt_common_normalize_path(path, NULL);
        if (!normalized_path) {
                BT_LOGE("Failed to normalize path: `%s`.", path);
                goto error;
        }
-       assert(path);
+       BT_ASSERT(path);
 
        ret = ctf_fs_find_traces(&trace_paths, normalized_path->str);
        if (ret) {
@@ -509,12 +517,14 @@ struct bt_component_class_query_return trace_info_query(
                goto error;
        }
 
-       query_ret.result = bt_value_array_create();
-       if (!query_ret.result) {
+       result = bt_private_value_array_create();
+       if (!result) {
                query_ret.status = BT_QUERY_STATUS_NOMEM;
                goto error;
        }
 
+       query_ret.result = bt_value_borrow_from_private(result);
+
        /* Iterates over both trace paths and names simultaneously. */
        for (tp_node = trace_paths, tn_node = trace_names; tp_node;
                        tp_node = g_list_next(tp_node),
@@ -522,9 +532,9 @@ struct bt_component_class_query_return trace_info_query(
                GString *trace_path = tp_node->data;
                GString *trace_name = tn_node->data;
                enum bt_value_status status;
-               struct bt_value *trace_info;
+               struct bt_private_value *trace_info;
 
-               trace_info = bt_value_map_create();
+               trace_info = bt_private_value_map_create();
                if (!trace_info) {
                        BT_LOGE("Failed to create trace info map.");
                        goto error;
@@ -533,12 +543,13 @@ struct bt_component_class_query_return trace_info_query(
                ret = populate_trace_info(trace_path->str, trace_name->str,
                        trace_info);
                if (ret) {
-                       bt_put(trace_info);
+                       bt_object_put_ref(trace_info);
                        goto error;
                }
 
-               status = bt_value_array_append(query_ret.result, trace_info);
-               bt_put(trace_info);
+               status = bt_private_value_array_append_element(result,
+                       bt_value_borrow_from_private(trace_info));
+               bt_object_put_ref(trace_info);
                if (status != BT_VALUE_STATUS_OK) {
                        goto error;
                }
@@ -547,7 +558,8 @@ struct bt_component_class_query_return trace_info_query(
        goto end;
 
 error:
-       BT_PUT(query_ret.result);
+       BT_OBJECT_PUT_REF_AND_RESET(result);
+       query_ret.result = NULL;
 
        if (query_ret.status >= 0) {
                query_ret.status = BT_QUERY_STATUS_ERROR;
@@ -574,6 +586,5 @@ end:
                g_list_free(trace_names);
        }
        /* "path" becomes invalid with the release of path_value. */
-       bt_put(path_value);
        return query_ret;
 }
This page took 0.029485 seconds and 4 git commands to generate.