Use integer extraction functions
[babeltrace.git] / formats / ctf / ctf.c
index 95a581fa4300f55c65cb9677f90c8196852bdef3..7767c3478ca0e2b005f732e4b858f602d58310f4 100644 (file)
@@ -55,7 +55,9 @@
 extern int yydebug;
 
 static
-struct trace_descriptor *ctf_open_trace(const char *path, int flags);
+struct trace_descriptor *ctf_open_trace(const char *path, int flags,
+               void (*move_pos_slow)(struct ctf_stream_pos *pos, size_t offset,
+                       int whence), FILE *metadata_fp);
 static
 void ctf_close_trace(struct trace_descriptor *descriptor);
 
@@ -97,7 +99,7 @@ void ctf_update_timestamp(struct ctf_stream *stream,
                integer_definition->declaration;
        uint64_t oldval, newval, updateval;
 
-       if (integer_declaration->len == 64) {
+       if (unlikely(integer_declaration->len == 64)) {
                stream->timestamp = integer_definition->value._unsigned;
                return;
        }
@@ -125,19 +127,27 @@ int ctf_read_event(struct stream_pos *ppos, struct ctf_stream *stream)
        uint64_t id = 0;
        int ret;
 
+       /* We need to check for EOF here for empty files. */
+       if (unlikely(pos->offset == EOF))
+               return EOF;
+
        ctf_pos_get_event(pos);
 
-       if (pos->offset == EOF)
+       /*
+        * This is the EOF check after we've advanced the position in
+        * ctf_pos_get_event.
+        */
+       if (unlikely(pos->offset == EOF))
                return EOF;
        assert(pos->offset < pos->content_size);
 
        /* Read event header */
-       if (stream->stream_event_header) {
+       if (likely(stream->stream_event_header)) {
                struct definition_integer *integer_definition;
                struct definition *variant;
 
                ret = generic_rw(ppos, &stream->stream_event_header->p);
-               if (ret)
+               if (unlikely(ret))
                        goto error;
                /* lookup event id */
                integer_definition = lookup_integer(&stream->stream_event_header->p, "id", FALSE);
@@ -185,12 +195,12 @@ int ctf_read_event(struct stream_pos *ppos, struct ctf_stream *stream)
                        goto error;
        }
 
-       if (id >= stream_class->events_by_id->len) {
+       if (unlikely(id >= stream_class->events_by_id->len)) {
                fprintf(stdout, "[error] Event id %" PRIu64 " is outside range.\n", id);
                return -EINVAL;
        }
        event = g_ptr_array_index(stream->events_by_id, id);
-       if (!event) {
+       if (unlikely(!event)) {
                fprintf(stdout, "[error] Event id %" PRIu64 " is unknown.\n", id);
                return -EINVAL;
        }
@@ -203,7 +213,7 @@ int ctf_read_event(struct stream_pos *ppos, struct ctf_stream *stream)
        }
 
        /* Read event payload */
-       if (event->event_fields) {
+       if (likely(event->event_fields)) {
                ret = generic_rw(ppos, &event->event_fields->p);
                if (ret)
                        goto error;
@@ -227,7 +237,7 @@ int ctf_write_event(struct stream_pos *pos, struct ctf_stream *stream)
        id = stream->event_id;
 
        /* print event header */
-       if (stream->stream_event_header) {
+       if (likely(stream->stream_event_header)) {
                ret = generic_rw(pos, &stream->stream_event_header->p);
                if (ret)
                        goto error;
@@ -240,12 +250,12 @@ int ctf_write_event(struct stream_pos *pos, struct ctf_stream *stream)
                        goto error;
        }
 
-       if (id >= stream_class->events_by_id->len) {
+       if (unlikely(id >= stream_class->events_by_id->len)) {
                fprintf(stdout, "[error] Event id %" PRIu64 " is outside range.\n", id);
                return -EINVAL;
        }
        event = g_ptr_array_index(stream->events_by_id, id);
-       if (!event) {
+       if (unlikely(!event)) {
                fprintf(stdout, "[error] Event id %" PRIu64 " is unknown.\n", id);
                return -EINVAL;
        }
@@ -258,7 +268,7 @@ int ctf_write_event(struct stream_pos *pos, struct ctf_stream *stream)
        }
 
        /* Read and print event payload */
-       if (event->event_fields) {
+       if (likely(event->event_fields)) {
                ret = generic_rw(pos, &event->event_fields->p);
                if (ret)
                        goto error;
@@ -406,6 +416,7 @@ void ctf_move_pos_slow(struct ctf_stream_pos *pos, size_t offset, int whence)
                        /* empty packet */
                        pos->offset = index->data_offset;
                        offset = 0;
+                       whence = SEEK_CUR;
                        goto read_next_packet;
                } else {
                        pos->offset = EOF;
@@ -609,7 +620,9 @@ int ctf_open_trace_metadata_stream_read(struct ctf_trace *td, FILE **fp,
 }
 
 static
-int ctf_open_trace_metadata_read(struct ctf_trace *td)
+int ctf_open_trace_metadata_read(struct ctf_trace *td,
+               void (*move_pos_slow)(struct ctf_stream_pos *pos, size_t offset,
+                       int whence), FILE *metadata_fp)
 {
        struct ctf_scanner *scanner;
        struct ctf_file_stream *metadata_stream;
@@ -618,22 +631,34 @@ int ctf_open_trace_metadata_read(struct ctf_trace *td)
        int ret = 0;
 
        metadata_stream = g_new0(struct ctf_file_stream, 1);
-       td->metadata = &metadata_stream->parent;
-       metadata_stream->pos.fd = openat(td->dirfd, "metadata", O_RDONLY);
-       if (metadata_stream->pos.fd < 0) {
-               fprintf(stdout, "Unable to open metadata.\n");
-               return metadata_stream->pos.fd;
+
+       if (move_pos_slow) {
+               metadata_stream->pos.move_pos_slow = move_pos_slow;
+       } else {
+               fprintf(stderr, "[error] move_pos_slow function undefined.\n");
+               ret = -1;
+               goto end_stream;
        }
 
-       if (babeltrace_debug)
-               yydebug = 1;
+       if (metadata_fp) {
+               fp = metadata_fp;
+       } else {
+               td->metadata = &metadata_stream->parent;
+               metadata_stream->pos.fd = openat(td->dirfd, "metadata", O_RDONLY);
+               if (metadata_stream->pos.fd < 0) {
+                       fprintf(stdout, "Unable to open metadata.\n");
+                       return metadata_stream->pos.fd;
+               }
 
-       fp = fdopen(metadata_stream->pos.fd, "r");
-       if (!fp) {
-               fprintf(stdout, "[error] Unable to open metadata stream.\n");
-               ret = -errno;
-               goto end_stream;
+               fp = fdopen(metadata_stream->pos.fd, "r");
+               if (!fp) {
+                       fprintf(stdout, "[error] Unable to open metadata stream.\n");
+                       ret = -errno;
+                       goto end_stream;
+               }
        }
+       if (babeltrace_debug)
+               yydebug = 1;
 
        if (packet_metadata(td, fp)) {
                ret = ctf_open_trace_metadata_stream_read(td, &fp, &buf);
@@ -875,16 +900,14 @@ int create_stream_packet_index(struct ctf_trace *td,
                                return ret;
                        len_index = struct_declaration_lookup_field_index(file_stream->parent.trace_packet_header->declaration, g_quark_from_static_string("magic"));
                        if (len_index >= 0) {
-                               struct definition_integer *defint;
                                struct definition *field;
+                               uint64_t magic;
 
                                field = struct_definition_get_field_from_index(file_stream->parent.trace_packet_header, len_index);
-                               assert(field->declaration->id == CTF_TYPE_INTEGER);
-                               defint = container_of(field, struct definition_integer, p);
-                               assert(defint->declaration->signedness == FALSE);
-                               if (defint->value._unsigned != CTF_MAGIC) {
+                               magic = get_unsigned_int(field);
+                               if (magic != CTF_MAGIC) {
                                        fprintf(stdout, "[error] Invalid magic number 0x%" PRIX64 " at packet %u (file offset %zd).\n",
-                                                       defint->value._unsigned,
+                                                       magic,
                                                        file_stream->pos.packet_index->len,
                                                        (ssize_t) pos->mmap_offset);
                                        return -EINVAL;
@@ -903,16 +926,12 @@ int create_stream_packet_index(struct ctf_trace *td,
                                assert(field->declaration->id == CTF_TYPE_ARRAY);
                                defarray = container_of(field, struct definition_array, p);
                                assert(array_len(defarray) == UUID_LEN);
-                               assert(defarray->declaration->elem->id == CTF_TYPE_INTEGER);
 
                                for (i = 0; i < UUID_LEN; i++) {
                                        struct definition *elem;
-                                       struct definition_integer *defint;
 
                                        elem = array_index(defarray, i);
-                                       assert(elem);
-                                       defint = container_of(elem, struct definition_integer, p);
-                                       uuidval[i] = defint->value._unsigned;
+                                       uuidval[i] = get_unsigned_int(elem);
                                }
                                ret = uuid_compare(td->uuid, uuidval);
                                if (ret) {
@@ -924,14 +943,10 @@ int create_stream_packet_index(struct ctf_trace *td,
 
                        len_index = struct_declaration_lookup_field_index(file_stream->parent.trace_packet_header->declaration, g_quark_from_static_string("stream_id"));
                        if (len_index >= 0) {
-                               struct definition_integer *defint;
                                struct definition *field;
 
                                field = struct_definition_get_field_from_index(file_stream->parent.trace_packet_header, len_index);
-                               assert(field->declaration->id == CTF_TYPE_INTEGER);
-                               defint = container_of(field, struct definition_integer, p);
-                               assert(defint->declaration->signedness == FALSE);
-                               stream_id = defint->value._unsigned;
+                               stream_id = get_unsigned_int(field);
                        }
                }
 
@@ -965,14 +980,10 @@ int create_stream_packet_index(struct ctf_trace *td,
                        /* read content size from header */
                        len_index = struct_declaration_lookup_field_index(file_stream->parent.stream_packet_context->declaration, g_quark_from_static_string("content_size"));
                        if (len_index >= 0) {
-                               struct definition_integer *defint;
                                struct definition *field;
 
                                field = struct_definition_get_field_from_index(file_stream->parent.stream_packet_context, len_index);
-                               assert(field->declaration->id == CTF_TYPE_INTEGER);
-                               defint = container_of(field, struct definition_integer, p);
-                               assert(defint->declaration->signedness == FALSE);
-                               packet_index.content_size = defint->value._unsigned;
+                               packet_index.content_size = get_unsigned_int(field);
                        } else {
                                /* Use file size for packet size */
                                packet_index.content_size = filestats.st_size * CHAR_BIT;
@@ -981,14 +992,10 @@ int create_stream_packet_index(struct ctf_trace *td,
                        /* read packet size from header */
                        len_index = struct_declaration_lookup_field_index(file_stream->parent.stream_packet_context->declaration, g_quark_from_static_string("packet_size"));
                        if (len_index >= 0) {
-                               struct definition_integer *defint;
                                struct definition *field;
 
                                field = struct_definition_get_field_from_index(file_stream->parent.stream_packet_context, len_index);
-                               assert(field->declaration->id == CTF_TYPE_INTEGER);
-                               defint = container_of(field, struct definition_integer, p);
-                               assert(defint->declaration->signedness == FALSE);
-                               packet_index.packet_size = defint->value._unsigned;
+                               packet_index.packet_size = get_unsigned_int(field);
                        } else {
                                /* Use content size if non-zero, else file size */
                                packet_index.packet_size = packet_index.content_size ? : filestats.st_size * CHAR_BIT;
@@ -997,27 +1004,19 @@ int create_stream_packet_index(struct ctf_trace *td,
                        /* read timestamp begin from header */
                        len_index = struct_declaration_lookup_field_index(file_stream->parent.stream_packet_context->declaration, g_quark_from_static_string("timestamp_begin"));
                        if (len_index >= 0) {
-                               struct definition_integer *defint;
                                struct definition *field;
 
                                field = struct_definition_get_field_from_index(file_stream->parent.stream_packet_context, len_index);
-                               assert(field->declaration->id == CTF_TYPE_INTEGER);
-                               defint = container_of(field, struct definition_integer, p);
-                               assert(defint->declaration->signedness == FALSE);
-                               packet_index.timestamp_begin = defint->value._unsigned;
+                               packet_index.timestamp_begin = get_unsigned_int(field);
                        }
 
                        /* read timestamp end from header */
                        len_index = struct_declaration_lookup_field_index(file_stream->parent.stream_packet_context->declaration, g_quark_from_static_string("timestamp_end"));
                        if (len_index >= 0) {
-                               struct definition_integer *defint;
                                struct definition *field;
 
                                field = struct_definition_get_field_from_index(file_stream->parent.stream_packet_context, len_index);
-                               assert(field->declaration->id == CTF_TYPE_INTEGER);
-                               defint = container_of(field, struct definition_integer, p);
-                               assert(defint->declaration->signedness == FALSE);
-                               packet_index.timestamp_end = defint->value._unsigned;
+                               packet_index.timestamp_end = get_unsigned_int(field);
                        }
                } else {
                        /* Use file size for packet size */
@@ -1083,7 +1082,9 @@ error:
  * description (metadata).
  */
 static
-int ctf_open_file_stream_read(struct ctf_trace *td, const char *path, int flags)
+int ctf_open_file_stream_read(struct ctf_trace *td, const char *path, int flags,
+               void (*move_pos_slow)(struct ctf_stream_pos *pos, size_t offset,
+                       int whence))
 {
        int ret;
        struct ctf_file_stream *file_stream;
@@ -1092,6 +1093,15 @@ int ctf_open_file_stream_read(struct ctf_trace *td, const char *path, int flags)
        if (ret < 0)
                goto error;
        file_stream = g_new0(struct ctf_file_stream, 1);
+
+       if (move_pos_slow) {
+               file_stream->pos.move_pos_slow = move_pos_slow;
+       } else {
+               fprintf(stderr, "[error] move_pos_slow function undefined.\n");
+               ret = -1;
+               goto error_def;
+       }
+
        ctf_init_pos(&file_stream->pos, ret, flags);
        ret = create_trace_definitions(td, &file_stream->parent);
        if (ret)
@@ -1116,7 +1126,9 @@ error:
 }
 
 static
-int ctf_open_trace_read(struct ctf_trace *td, const char *path, int flags)
+int ctf_open_trace_read(struct ctf_trace *td, const char *path, int flags,
+               void (*move_pos_slow)(struct ctf_stream_pos *pos, size_t offset,
+                       int whence), FILE *metadata_fp)
 {
        int ret;
        struct dirent *dirent;
@@ -1144,7 +1156,7 @@ int ctf_open_trace_read(struct ctf_trace *td, const char *path, int flags)
         * Keep the metadata file separate.
         */
 
-       ret = ctf_open_trace_metadata_read(td);
+       ret = ctf_open_trace_metadata_read(td, move_pos_slow, metadata_fp);
        if (ret) {
                goto error_metadata;
        }
@@ -1173,7 +1185,7 @@ int ctf_open_trace_read(struct ctf_trace *td, const char *path, int flags)
                                || !strcmp(diriter->d_name, "..")
                                || !strcmp(diriter->d_name, "metadata"))
                        continue;
-               ret = ctf_open_file_stream_read(td, diriter->d_name, flags);
+               ret = ctf_open_file_stream_read(td, diriter->d_name, flags, move_pos_slow);
                if (ret) {
                        fprintf(stdout, "[error] Open file stream error.\n");
                        goto readdir_error;
@@ -1194,7 +1206,9 @@ error:
 }
 
 static
-struct trace_descriptor *ctf_open_trace(const char *path, int flags)
+struct trace_descriptor *ctf_open_trace(const char *path, int flags,
+               void (*move_pos_slow)(struct ctf_stream_pos *pos, size_t offset,
+                       int whence), FILE *metadata_fp)
 {
        struct ctf_trace *td;
        int ret;
@@ -1207,7 +1221,7 @@ struct trace_descriptor *ctf_open_trace(const char *path, int flags)
                        fprintf(stdout, "[error] Path missing for input CTF trace.\n");
                        goto error;
                }
-               ret = ctf_open_trace_read(td, path, flags);
+               ret = ctf_open_trace_read(td, path, flags, move_pos_slow, metadata_fp);
                if (ret)
                        goto error;
                break;
This page took 0.029715 seconds and 4 git commands to generate.