#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"
};
BT_HIDDEN
-struct bt_value *metadata_info_query(struct bt_component_class *comp_class,
+struct bt_component_class_query_method_return metadata_info_query(
+ struct bt_component_class *comp_class,
struct bt_value *params)
{
- struct bt_value *result = NULL;
+ 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;
const char *path;
bool is_packetized;
- result = bt_value_map_create();
+ 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)) {
- fprintf(stderr,
- "Query parameters is not a map value object\n");
+ 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");
+ path_value = bt_value_map_borrow_entry_value(params, "path");
ret = bt_value_string_get(path_value, &path);
if (ret) {
- fprintf(stderr,
- "Cannot get `path` string parameter\n");
+ BT_LOGE_STR("Cannot get `path` string parameter.");
+ query_ret.status = BT_QUERY_STATUS_INVALID_PARAMS;
goto error;
}
- assert(path);
+ BT_ASSERT(path);
metadata_fp = ctf_fs_metadata_open_file(path);
if (!metadata_fp) {
- fprintf(stderr,
- "Cannot open trace at path `%s`\n", path);
+ BT_LOGE("Cannot open trace metadata: path=\"%s\".", path);
goto error;
}
ret = ctf_metadata_decoder_packetized_file_stream_to_buf(
metadata_fp, &metadata_text, bo);
if (ret) {
- fprintf(stderr,
- "Cannot decode packetized metadata file\n");
+ BT_LOGE("Cannot decode packetized metadata file: path=\"%s\"",
+ path);
goto error;
}
} else {
ret = fseek(metadata_fp, 0, SEEK_END);
if (ret) {
- fprintf(stderr, "Error in fseek: %s", strerror(errno));
+ BT_LOGE_ERRNO("Failed to seek to the end of the metadata file",
+ ": path=\"%s\"", path);
goto error;
}
filesize = ftell(metadata_fp);
if (filesize < 0) {
- fprintf(stderr, "Error in ftell: %s", strerror(errno));
+ BT_LOGE_ERRNO("Failed to get the current position in the metadata file",
+ ": path=\"%s\"", path);
goto error;
}
rewind(metadata_fp);
metadata_text = malloc(filesize + 1);
if (!metadata_text) {
- fprintf(stderr,
- "Cannot allocate buffer for metadata text\n");
+ BT_LOGE_STR("Cannot allocate buffer for metadata text.");
goto error;
}
if (fread(metadata_text, filesize, 1, metadata_fp) != 1) {
- fprintf(stderr,
- "Cannot read metadata file\n");
+ BT_LOGE_ERRNO("Cannot read metadata file", ": path=\"%s\"",
+ path);
goto error;
}
g_string_append(g_metadata_text, metadata_text);
- ret = bt_value_map_insert_string(result, "text",
+ ret = bt_private_value_map_insert_string_entry(result, "text",
g_metadata_text->str);
if (ret) {
- fprintf(stderr, "Cannot insert metadata text into result\n");
+ BT_LOGE_STR("Cannot insert metadata text into query result.");
goto error;
}
- ret = bt_value_map_insert_bool(result, "is-packetized",
+ ret = bt_private_value_map_insert_bool_entry(result, "is-packetized",
is_packetized);
if (ret) {
- fprintf(stderr, "Cannot insert is packetized into result\n");
+ BT_LOGE_STR("Cannot insert \"is-packetized\" attribute into query result.");
goto error;
}
goto end;
error:
- BT_PUT(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) {
if (metadata_fp) {
fclose(metadata_fp);
}
- return result;
+
+ return query_ret;
}
+
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;
}
}
- 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;
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;
}
}
- 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;
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,
.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;
goto end;
}
- trace = ctf_fs_trace_create(trace_path, trace_name, NULL);
+ trace = ctf_fs_trace_create(trace_path, trace_name, NULL, NULL);
if (!trace) {
BT_LOGE("Failed to create fs trace at \'%s\'", trace_path);
ret = -1;
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;
/* 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;
ret = populate_stream_info(group, group_info, &group_range);
if (ret) {
- bt_put(group_info);
+ bt_object_put_ref(group_info);
goto end;
}
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;
}
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_value *trace_info_query(struct bt_component_class *comp_class,
+struct bt_component_class_query_method_return trace_info_query(
+ struct bt_component_class *comp_class,
struct bt_value *params)
{
- struct bt_value *trace_infos = NULL;
+ 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;
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");
+ path_value = bt_value_map_borrow_entry_value(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;
}
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) {
goto error;
}
- trace_infos = bt_value_array_create();
- if (!trace_infos) {
+ 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),
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;
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(trace_infos, 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;
}
goto end;
error:
- BT_PUT(trace_infos);
+ BT_OBJECT_PUT_REF_AND_RESET(result);
+ query_ret.result = NULL;
+
+ if (query_ret.status >= 0) {
+ query_ret.status = BT_QUERY_STATUS_ERROR;
+ }
+
end:
if (normalized_path) {
g_string_free(normalized_path, TRUE);
g_list_free(trace_names);
}
/* "path" becomes invalid with the release of path_value. */
- bt_put(path_value);
- return trace_infos;
+ return query_ret;
}