always copy packet/content_size, they are overwritten by the lib when necessary
[babeltrace.git] / plugins / ctf / fs-sink / write.c
index 2285e92a2e4dc27e7eb39cead4bed82d49281265..305638cbbd4dd0a68a361ead7fb866124f882b50 100644 (file)
 
 #include "writer.h"
 
+static
+void unref_stream_class(struct bt_ctf_stream_class *writer_stream_class)
+{
+       bt_put(writer_stream_class);
+}
+
+static
+void unref_stream(struct bt_ctf_stream_class *writer_stream)
+{
+       bt_put(writer_stream);
+}
+
+gboolean empty_ht(gpointer key, gpointer value, gpointer user_data)
+{
+       return TRUE;
+}
+
+void destroy_stream_state_key(gpointer key)
+{
+       g_free((enum fs_writer_stream_state *) key);
+}
+
 static
 void trace_is_static_listener(struct bt_ctf_trace *trace, void *data)
 {
@@ -50,11 +72,12 @@ void trace_is_static_listener(struct bt_ctf_trace *trace, void *data)
 static
 struct bt_ctf_stream_class *insert_new_stream_class(
                struct writer_component *writer_component,
-               struct bt_ctf_writer *ctf_writer,
+               struct fs_writer *fs_writer,
                struct bt_ctf_stream_class *stream_class)
 {
        struct bt_ctf_stream_class *writer_stream_class = NULL;
        struct bt_ctf_trace *trace = NULL, *writer_trace = NULL;
+       struct bt_ctf_writer *ctf_writer = fs_writer->writer;
        enum bt_component_status ret;
 
        trace = bt_ctf_stream_class_get_trace(stream_class);
@@ -91,7 +114,7 @@ struct bt_ctf_stream_class *insert_new_stream_class(
                goto error;
        }
 
-       g_hash_table_insert(writer_component->stream_class_map,
+       g_hash_table_insert(fs_writer->stream_class_map,
                        (gpointer) stream_class, writer_stream_class);
 
        goto end;
@@ -105,67 +128,26 @@ end:
 }
 
 static
-struct bt_ctf_stream *insert_new_stream(
+enum fs_writer_stream_state *insert_new_stream_state(
                struct writer_component *writer_component,
-               struct bt_ctf_writer *ctf_writer,
-               struct bt_ctf_stream_class *stream_class,
-               struct bt_ctf_stream *stream)
+               struct fs_writer *fs_writer, struct bt_ctf_stream *stream)
 {
-       struct bt_ctf_stream *writer_stream = NULL;
-       struct bt_ctf_stream_class *writer_stream_class = NULL;
+       enum fs_writer_stream_state *v = NULL;
 
-       writer_stream_class = g_hash_table_lookup(
-                       writer_component->stream_class_map,
-                       (gpointer) stream_class);
-       if (!writer_stream_class) {
-               writer_stream_class = insert_new_stream_class(
-                               writer_component, ctf_writer, stream_class);
-               if (!writer_stream_class) {
-                       fprintf(writer_component->err, "[error] %s in %s:%d\n",
-                                       __func__, __FILE__, __LINE__);
-                       goto error;
-               }
-       }
-       bt_get(writer_stream_class);
-
-       writer_stream = bt_ctf_writer_create_stream(ctf_writer,
-                       writer_stream_class);
-       if (!writer_stream) {
-               fprintf(writer_component->err, "[error] %s in %s:%d\n",
-                               __func__, __FILE__, __LINE__);
-               goto error;
+       v = g_new0(enum fs_writer_stream_state, 1);
+       if (!v) {
+               fprintf(writer_component->err,
+                               "[error] %s in %s:%d\n", __func__,
+                               __FILE__, __LINE__);
        }
+       *v = FS_WRITER_UNKNOWN_STREAM;
 
-       g_hash_table_insert(writer_component->stream_map, (gpointer) stream,
-                       writer_stream);
-
-       goto end;
-
-error:
-       BT_PUT(writer_stream);
-end:
-       bt_put(writer_stream_class);
-       return writer_stream;
-}
+       g_hash_table_insert(fs_writer->stream_states, stream, v);
 
-static
-struct bt_ctf_stream *lookup_stream(struct writer_component *writer_component,
-               struct bt_ctf_stream *stream)
-{
-       return (struct bt_ctf_stream *) g_hash_table_lookup(
-                       writer_component->stream_map,
-                       (gpointer) stream);
+       return v;
 }
 
 static
-struct bt_ctf_event_class *get_event_class(struct writer_component *writer_component,
-               struct bt_ctf_stream_class *writer_stream_class,
-               struct bt_ctf_event_class *event_class)
-{
-       return bt_ctf_stream_class_get_event_class_by_id(writer_stream_class,
-                       bt_ctf_event_class_get_id(event_class));
-}
-
 struct fs_writer *insert_new_writer(
                struct writer_component *writer_component,
                struct bt_ctf_trace *trace)
@@ -174,7 +156,9 @@ struct fs_writer *insert_new_writer(
        struct bt_ctf_trace *writer_trace = NULL;
        char trace_name[PATH_MAX];
        enum bt_component_status ret;
-       struct fs_writer *fs_writer;
+       struct bt_ctf_stream *stream = NULL;
+       struct fs_writer *fs_writer = NULL;
+       int nr_stream, i;
 
        /* FIXME: replace with trace name when it will work. */
        snprintf(trace_name, PATH_MAX, "%s/%s_%03d",
@@ -215,8 +199,31 @@ struct fs_writer *insert_new_writer(
                goto error;
        }
        fs_writer->writer = ctf_writer;
+       fs_writer->trace = trace;
        fs_writer->writer_trace = writer_trace;
        BT_PUT(writer_trace);
+       fs_writer->stream_class_map = g_hash_table_new_full(g_direct_hash,
+                       g_direct_equal, NULL, (GDestroyNotify) unref_stream_class);
+       fs_writer->stream_map = g_hash_table_new_full(g_direct_hash,
+                       g_direct_equal, NULL, (GDestroyNotify) unref_stream);
+       fs_writer->stream_states = g_hash_table_new_full(g_direct_hash,
+                       g_direct_equal, NULL, destroy_stream_state_key);
+
+       /* Set all the existing streams in the unknown state. */
+       nr_stream = bt_ctf_trace_get_stream_count(trace);
+       for (i = 0; i < nr_stream; i++) {
+               stream = bt_ctf_trace_get_stream_by_index(trace, i);
+               if (!stream) {
+                       fprintf(writer_component->err,
+                                       "[error] %s in %s:%d\n", __func__,
+                                       __FILE__, __LINE__);
+                       goto error;
+               }
+               insert_new_stream_state(writer_component, fs_writer, stream);
+               BT_PUT(stream);
+       }
+
+       /* Check if the trace is already static or register a listener. */
        if (bt_ctf_trace_is_static(trace)) {
                fs_writer->trace_static = 1;
                fs_writer->static_listener_id = -1;
@@ -227,8 +234,6 @@ struct fs_writer *insert_new_writer(
                        fprintf(writer_component->err,
                                        "[error] %s in %s:%d\n", __func__, __FILE__,
                                        __LINE__);
-                       g_free(fs_writer);
-                       fs_writer = NULL;
                        goto error;
                }
                fs_writer->static_listener_id = ret;
@@ -240,7 +245,10 @@ struct fs_writer *insert_new_writer(
        goto end;
 
 error:
+       g_free(fs_writer);
+       fs_writer = NULL;
        bt_put(writer_trace);
+       bt_put(stream);
        BT_PUT(ctf_writer);
 end:
        return fs_writer;
@@ -275,13 +283,12 @@ end:
 }
 
 static
-struct bt_ctf_stream *get_writer_stream(
+struct fs_writer *get_fs_writer_from_stream(
                struct writer_component *writer_component,
-               struct bt_ctf_packet *packet, struct bt_ctf_stream *stream)
+               struct bt_ctf_stream *stream)
 {
        struct bt_ctf_stream_class *stream_class = NULL;
-       struct bt_ctf_writer *ctf_writer = NULL;
-       struct bt_ctf_stream *writer_stream = NULL;
+       struct fs_writer *fs_writer;
 
        stream_class = bt_ctf_stream_get_class(stream);
        if (!stream_class) {
@@ -290,27 +297,74 @@ struct bt_ctf_stream *get_writer_stream(
                goto error;
        }
 
-       writer_stream = lookup_stream(writer_component, stream);
-       if (!writer_stream) {
-               struct fs_writer *fs_writer;
+       fs_writer = get_fs_writer(writer_component, stream_class);
+       goto end;
 
-               fs_writer = get_fs_writer(writer_component, stream_class);
-               if (!fs_writer) {
-                       fprintf(writer_component->err, "[error] %s in %s:%d\n",
-                                       __func__, __FILE__, __LINE__);
-                       goto error;
-               }
-               ctf_writer = bt_get(fs_writer->writer);
-               writer_stream = insert_new_stream(writer_component, ctf_writer,
-                               stream_class, stream);
-               if (!writer_stream) {
+error:
+       fs_writer = NULL;
+
+end:
+       bt_put(stream_class);
+       return fs_writer;
+}
+
+static
+struct bt_ctf_stream_class *lookup_stream_class(
+               struct writer_component *writer_component,
+               struct bt_ctf_stream_class *stream_class)
+{
+       struct fs_writer *fs_writer = get_fs_writer(
+                       writer_component, stream_class);
+       assert(fs_writer);
+       return (struct bt_ctf_stream_class *) g_hash_table_lookup(
+                       fs_writer->stream_class_map, (gpointer) stream_class);
+}
+
+static
+struct bt_ctf_stream *lookup_stream(struct writer_component *writer_component,
+               struct bt_ctf_stream *stream)
+{
+       struct fs_writer *fs_writer = get_fs_writer_from_stream(
+                       writer_component, stream);
+       assert(fs_writer);
+       return (struct bt_ctf_stream *) g_hash_table_lookup(
+                       fs_writer->stream_map, (gpointer) stream);
+}
+
+static
+struct bt_ctf_stream *insert_new_stream(
+               struct writer_component *writer_component,
+               struct fs_writer *fs_writer,
+               struct bt_ctf_stream_class *stream_class,
+               struct bt_ctf_stream *stream)
+{
+       struct bt_ctf_stream *writer_stream = NULL;
+       struct bt_ctf_stream_class *writer_stream_class = NULL;
+       struct bt_ctf_writer *ctf_writer = bt_get(fs_writer->writer);
+
+       writer_stream_class = lookup_stream_class(writer_component,
+                       stream_class);
+       if (!writer_stream_class) {
+               writer_stream_class = insert_new_stream_class(
+                               writer_component, fs_writer, stream_class);
+               if (!writer_stream_class) {
                        fprintf(writer_component->err, "[error] %s in %s:%d\n",
                                        __func__, __FILE__, __LINE__);
                        goto error;
                }
-               fs_writer->active_streams++;
        }
-       bt_get(writer_stream);
+       bt_get(writer_stream_class);
+
+       writer_stream = bt_ctf_writer_create_stream(ctf_writer,
+                       writer_stream_class);
+       if (!writer_stream) {
+               fprintf(writer_component->err, "[error] %s in %s:%d\n",
+                               __func__, __FILE__, __LINE__);
+               goto error;
+       }
+
+       g_hash_table_insert(fs_writer->stream_map, (gpointer) stream,
+                       writer_stream);
 
        goto end;
 
@@ -318,26 +372,138 @@ error:
        BT_PUT(writer_stream);
 end:
        bt_put(ctf_writer);
-       bt_put(stream_class);
+       bt_put(writer_stream_class);
        return writer_stream;
 }
 
-BT_HIDDEN
-enum bt_component_status writer_close(
+static
+struct bt_ctf_event_class *get_event_class(struct writer_component *writer_component,
+               struct bt_ctf_stream_class *writer_stream_class,
+               struct bt_ctf_event_class *event_class)
+{
+       return bt_ctf_stream_class_get_event_class_by_id(writer_stream_class,
+                       bt_ctf_event_class_get_id(event_class));
+}
+
+static
+struct bt_ctf_stream *get_writer_stream(
                struct writer_component *writer_component,
-               struct fs_writer *fs_writer,
-               struct bt_ctf_trace *trace)
+               struct bt_ctf_packet *packet, struct bt_ctf_stream *stream)
 {
-       enum bt_component_status ret = BT_COMPONENT_STATUS_OK;
+       struct bt_ctf_stream *writer_stream = NULL;
+
+       writer_stream = lookup_stream(writer_component, stream);
+       if (!writer_stream) {
+               fprintf(writer_component->err, "[error] %s in %s:%d\n",
+                               __func__, __FILE__, __LINE__);
+               goto error;
+       }
+       bt_get(writer_stream);
+
+       goto end;
+
+error:
+       BT_PUT(writer_stream);
+end:
+       return writer_stream;
+}
 
+BT_HIDDEN
+void writer_close(struct writer_component *writer_component,
+               struct fs_writer *fs_writer)
+{
        if (fs_writer->static_listener_id > 0) {
-               bt_ctf_trace_remove_is_static_listener(trace,
+               bt_ctf_trace_remove_is_static_listener(fs_writer->trace,
                                fs_writer->static_listener_id);
        }
-       g_hash_table_remove(writer_component->trace_map, trace);
+
+       /* Empty the stream class HT. */
+       g_hash_table_foreach_remove(fs_writer->stream_class_map,
+                       empty_ht, NULL);
+       g_hash_table_destroy(fs_writer->stream_class_map);
+
+       /* Empty the stream HT. */
+       g_hash_table_foreach_remove(fs_writer->stream_map,
+                       empty_ht, NULL);
+       g_hash_table_destroy(fs_writer->stream_map);
+
+       /* Empty the stream state HT. */
+       g_hash_table_foreach_remove(fs_writer->stream_states,
+                       empty_ht, NULL);
+       g_hash_table_destroy(fs_writer->stream_states);
+}
+
+BT_HIDDEN
+enum bt_component_status writer_stream_begin(
+               struct writer_component *writer_component,
+               struct bt_ctf_stream *stream)
+{
+       struct bt_ctf_stream_class *stream_class = NULL;
+       struct fs_writer *fs_writer;
+       struct bt_ctf_stream *writer_stream = NULL;
+       enum bt_component_status ret = BT_COMPONENT_STATUS_OK;
+       enum fs_writer_stream_state *state;
+
+       stream_class = bt_ctf_stream_get_class(stream);
+       if (!stream_class) {
+               fprintf(writer_component->err, "[error] %s in %s:%d\n",
+                               __func__, __FILE__, __LINE__);
+               goto error;
+       }
+
+       fs_writer = get_fs_writer(writer_component, stream_class);
+       if (!fs_writer) {
+               fprintf(writer_component->err, "[error] %s in %s:%d\n",
+                               __func__, __FILE__, __LINE__);
+               goto error;
+       }
+
+       /* Set the stream as active */
+       state = g_hash_table_lookup(fs_writer->stream_states, stream);
+       if (!state) {
+               if (fs_writer->trace_static) {
+                       fprintf(writer_component->err, "[error] Adding a new "
+                                       "stream on a static trace\n");
+                       goto error;
+               }
+               state = insert_new_stream_state(writer_component, fs_writer,
+                               stream);
+       }
+       if (*state != FS_WRITER_UNKNOWN_STREAM) {
+               fprintf(writer_component->err, "[error] Unexpected stream "
+                               "state %d\n", *state);
+               goto error;
+       }
+       *state = FS_WRITER_ACTIVE_STREAM;
+
+       writer_stream = insert_new_stream(writer_component, fs_writer,
+                       stream_class, stream);
+       if (!writer_stream) {
+               fprintf(writer_component->err, "[error] %s in %s:%d\n",
+                               __func__, __FILE__, __LINE__);
+               goto error;
+       }
+       fs_writer->active_streams++;
+
+       goto end;
+
+error:
+       ret = BT_COMPONENT_STATUS_ERROR;
+end:
+       bt_put(stream_class);
        return ret;
 }
 
+void check_completed_trace(gpointer key, gpointer value, gpointer user_data)
+{
+       enum fs_writer_stream_state *state = value;
+       int *trace_completed = user_data;
+
+       if (*state != FS_WRITER_COMPLETED_STREAM) {
+               *trace_completed = 0;
+       }
+}
+
 BT_HIDDEN
 enum bt_component_status writer_stream_end(
                struct writer_component *writer_component,
@@ -347,8 +513,7 @@ enum bt_component_status writer_stream_end(
        struct fs_writer *fs_writer;
        struct bt_ctf_trace *trace = NULL;
        enum bt_component_status ret = BT_COMPONENT_STATUS_OK;
-
-       g_hash_table_remove(writer_component->stream_map, stream);
+       enum fs_writer_stream_state *state;
 
        stream_class = bt_ctf_stream_get_class(stream);
        if (!stream_class) {
@@ -364,16 +529,26 @@ enum bt_component_status writer_stream_end(
                goto error;
        }
 
-       assert(fs_writer->active_streams > 0);
-       fs_writer->active_streams--;
-       if (fs_writer->active_streams == 0 && fs_writer->trace_static) {
-               trace = bt_ctf_stream_class_get_trace(stream_class);
-               if (!trace) {
-                       fprintf(writer_component->err, "[error] %s in %s:%d\n",
-                                       __func__, __FILE__, __LINE__);
-                       goto error;
+       state = g_hash_table_lookup(fs_writer->stream_states, stream);
+       if (*state != FS_WRITER_ACTIVE_STREAM) {
+               fprintf(writer_component->err, "[error] Unexpected stream "
+                               "state %d\n", *state);
+               goto error;
+       }
+       *state = FS_WRITER_COMPLETED_STREAM;
+
+       g_hash_table_remove(fs_writer->stream_map, stream);
+
+       if (fs_writer->trace_static) {
+               int trace_completed = 1;
+
+               g_hash_table_foreach(fs_writer->stream_states,
+                               check_completed_trace, &trace_completed);
+               if (trace_completed) {
+                       writer_close(writer_component, fs_writer);
+                       g_hash_table_remove(writer_component->trace_map,
+                                       fs_writer->trace);
                }
-               ret = writer_close(writer_component, fs_writer, trace);
        }
 
        goto end;
@@ -412,7 +587,7 @@ enum bt_component_status writer_new_packet(
        BT_PUT(stream);
 
        writer_packet_context = ctf_copy_packet_context(writer_component->err,
-                       packet, writer_stream, 1);
+                       packet, writer_stream);
        if (!writer_packet_context) {
                fprintf(writer_component->err, "[error] %s in %s:%d\n",
                                __func__, __FILE__, __LINE__);
@@ -532,9 +707,7 @@ enum bt_component_status writer_output_event(
                goto error;
        }
 
-       writer_stream_class = g_hash_table_lookup(
-                       writer_component->stream_class_map,
-                       (gpointer) stream_class);
+       writer_stream_class = lookup_stream_class(writer_component, stream_class);
        if (!writer_stream_class || !bt_get(writer_stream_class)) {
                fprintf(writer_component->err, "[error] %s in %s:%d\n", __func__,
                                __FILE__, __LINE__);
This page took 0.028593 seconds and 4 git commands to generate.