allow multiple ctf streams to be open
[babeltrace.git] / plugins / ctf / fs / data-stream.c
index 6fba603af2fb72c01bd22fc7f25c21dc8190460a..4c3ab29a33e94bc56a6903dd6226efe541c0181e 100644 (file)
 #include <inttypes.h>
 #include <sys/mman.h>
 #include <babeltrace/ctf-ir/stream.h>
+#include <babeltrace/plugin/notification/iterator.h>
+#include "file.h"
+#include "metadata.h"
+#include "../common/notif-iter/notif-iter.h"
+#include <assert.h>
 
 #define PRINT_ERR_STREAM       ctf_fs->error_fp
 #define PRINT_PREFIX           "ctf-fs-data-stream"
 #include "print.h"
 
-#include "file.h"
-#include "metadata.h"
-#include "../common/notif-iter/notif-iter.h"
-
-static void ctf_fs_stream_destroy(struct ctf_fs_stream *stream)
+BT_HIDDEN
+void ctf_fs_stream_destroy(struct ctf_fs_stream *stream)
 {
        if (stream->file) {
                ctf_fs_file_destroy(stream->file);
@@ -95,22 +97,20 @@ static int mmap_next(struct ctf_fs_stream *stream)
 
        /* Map new region */
        stream->mmap_addr = mmap((void *) 0, stream->mmap_len,
-               PROT_READ, MAP_PRIVATE, fileno(stream->file->fp),
-               stream->mmap_offset);
+                       PROT_READ, MAP_PRIVATE, fileno(stream->file->fp),
+                       stream->mmap_offset);
        if (stream->mmap_addr == MAP_FAILED) {
                PERR("Cannot memory-map address (size %zu) of file \"%s\" (%p) at offset %zu: %s\n",
-                       stream->mmap_len, stream->file->path->str,
-                       stream->file->fp, stream->mmap_offset,
-                       strerror(errno));
+                               stream->mmap_len, stream->file->path->str,
+                               stream->file->fp, stream->mmap_offset,
+                               strerror(errno));
                goto error;
        }
 
        goto end;
-
 error:
        stream_munmap(stream);
        ret = -1;
-
 end:
        return ret;
 }
@@ -193,7 +193,8 @@ static struct bt_ctf_notif_iter_medium_ops medops = {
        .get_stream = medop_get_stream,
 };
 
-static struct ctf_fs_stream *ctf_fs_stream_create(
+BT_HIDDEN
+struct ctf_fs_stream *ctf_fs_stream_create(
                struct ctf_fs_component *ctf_fs, struct ctf_fs_file *file)
 {
        struct ctf_fs_stream *stream = g_new0(struct ctf_fs_stream, 1);
@@ -204,158 +205,73 @@ static struct ctf_fs_stream *ctf_fs_stream_create(
 
        stream->file = file;
        stream->notif_iter = bt_ctf_notif_iter_create(ctf_fs->metadata.trace,
-               12, medops, stream, ctf_fs->error_fp);
+                       ctf_fs->page_size, medops, stream, ctf_fs->error_fp);
        if (!stream->notif_iter) {
                goto error;
        }
-       stream->mmap_len = ctf_fs->page_size;
-
+       stream->mmap_len = ctf_fs->page_size * 2048;
        goto end;
-
 error:
-       /* Do not touch borrowed file */
+       /* Do not touch "borrowed" file. */
        stream->file = NULL;
        ctf_fs_stream_destroy(stream);
        stream = NULL;
-
 end:
        return stream;
 }
 
-int ctf_fs_data_stream_open_streams(struct ctf_fs_component *ctf_fs)
-{
-       int ret = 0;
-       GError *error = NULL;
-       GDir *dir = g_dir_open(ctf_fs->trace_path->str, 0, &error);
-       const char *name;
-
-       if (!dir) {
-               PERR("Cannot open directory \"%s\": %s (code %d)\n",
-                       ctf_fs->trace_path->str, error->message,
-                       error->code);
-               goto error;
-       }
-
-       while ((name = g_dir_read_name(dir))) {
-               struct ctf_fs_file *file = NULL;
-               struct ctf_fs_stream *stream = NULL;
-
-               if (strcmp(name, CTF_FS_METADATA_FILENAME) == 0) {
-                       /* Ignore the metadata stream */
-                       PDBG("Ignoring metadata file \"%s\"\n",
-                               name);
-                       continue;
-               }
-
-               if (name[0] == '.') {
-                       PDBG("Ignoring hidden file \"%s\"\n",
-                               name);
-                       continue;
-               }
-
-               /* Create the file */
-               file = ctf_fs_file_create(ctf_fs);
-               if (!file) {
-                       PERR("Cannot create stream file object\n");
-                       goto error;
-               }
-
-               /* Create full path string */
-               g_string_append(file->path, ctf_fs->trace_path->str);
-               g_string_append(file->path, "/");
-               g_string_append(file->path, name);
-
-               if (!g_file_test(file->path->str, G_FILE_TEST_IS_REGULAR)) {
-                       PDBG("Ignoring non-regular file \"%s\"\n", name);
-                       ctf_fs_file_destroy(file);
-                       continue;
-               }
-
-               /* Open the file */
-               if (ctf_fs_file_open(ctf_fs, file, "rb")) {
-                       ctf_fs_file_destroy(file);
-                       goto error;
-               }
-
-               /* Create a private stream */
-               stream = ctf_fs_stream_create(ctf_fs, file);
-               if (!stream) {
-                       ctf_fs_file_destroy(file);
-                       goto error;
-               }
-
-               /* Append file to the array of files */
-               g_ptr_array_add(ctf_fs->data_stream.streams, stream);
-       }
-
-       goto end;
-
-error:
-       ret = -1;
-
-end:
-       if (dir) {
-               g_dir_close(dir);
-               dir = NULL;
-       }
-
-       if (error) {
-               g_error_free(error);
-       }
-
-       return ret;
-}
-
-int ctf_fs_data_stream_init(struct ctf_fs_component *ctf_fs,
-               struct ctf_fs_data_stream *data_stream)
-{
-       int ret = 0;
-
-       data_stream->streams = g_ptr_array_new_with_free_func(
-               (GDestroyNotify) ctf_fs_stream_destroy);
-       if (!data_stream->streams) {
-               PERR("Cannot allocate array of streams\n");
-               goto error;
-       }
-
-       goto end;
-
-error:
-       ret = -1;
-
-end:
-       return ret;
-}
-
-void ctf_fs_data_stream_deinit(struct ctf_fs_data_stream *data_stream)
-{
-       g_ptr_array_free(data_stream->streams, TRUE);
-}
-
-int ctf_fs_data_stream_get_next_notification(
+enum bt_notification_iterator_status ctf_fs_data_stream_get_next_notification(
                struct ctf_fs_component *ctf_fs,
-               struct bt_ctf_notif_iter_notif **notification)
+               struct bt_notification **notification)
 {
-       int ret = 0;
-       struct ctf_fs_stream *stream = g_ptr_array_index(
-               ctf_fs->data_stream.streams, 0);
        enum bt_ctf_notif_iter_status status;
+       enum bt_notification_iterator_status ret;
+       /* FIXME, only iterating on one stream for the moment. */
+       struct ctf_fs_stream *stream = g_ptr_array_index(ctf_fs->streams, 0);
 
-       status = bt_ctf_notif_iter_get_next_notification(
-               stream->notif_iter, notification);
+       if (stream->end_reached) {
+               status = BT_CTF_NOTIF_ITER_STATUS_EOF;
+               goto end;
+       }
+
+       status = bt_ctf_notif_iter_get_next_notification(stream->notif_iter,
+                       notification);
        if (status != BT_CTF_NOTIF_ITER_STATUS_OK &&
                        status != BT_CTF_NOTIF_ITER_STATUS_EOF) {
-               goto error;
+               goto end;
        }
+
+       /* Should be handled in bt_ctf_notif_iter_get_next_notification. */
        if (status == BT_CTF_NOTIF_ITER_STATUS_EOF) {
-               *notification = NULL;
+               *notification = bt_notification_stream_end_create(
+                               stream->stream);
+               if (!*notification) {
+                       status = BT_CTF_NOTIF_ITER_STATUS_ERROR;
+               }
+               status = BT_CTF_NOTIF_ITER_STATUS_OK;
+               stream->end_reached = true;
        }
-
-       goto end;
-
-error:
-       ret = -1;
-
 end:
+       switch (status) {
+       case BT_CTF_NOTIF_ITER_STATUS_EOF:
+               ret = BT_NOTIFICATION_ITERATOR_STATUS_END;
+               break;
+       case BT_CTF_NOTIF_ITER_STATUS_OK:
+               ret = BT_NOTIFICATION_ITERATOR_STATUS_OK;
+               break;
+       case BT_CTF_NOTIF_ITER_STATUS_AGAIN:
+               /*
+                * Should not make it this far as this is medium-specific;
+                * there is nothing for the user to do and it should have been
+                * handled upstream.
+                */
+               assert(0);
+       case BT_CTF_NOTIF_ITER_STATUS_INVAL:
+               /* No argument provided by the user, so don't return INVAL. */
+       case BT_CTF_NOTIF_ITER_STATUS_ERROR:
+       default:
+               ret = BT_NOTIFICATION_ITERATOR_STATUS_ERROR;
+               break;
+       }
        return ret;
 }
This page took 0.028425 seconds and 4 git commands to generate.