X-Git-Url: http://git.efficios.com/?p=babeltrace.git;a=blobdiff_plain;f=lib%2Fcontext.c;h=87901b3c61027aadea02e0ddf0f9aa042b4aaac1;hp=d6b1ec8b6ba71f48e5ffe9428baf3a89f8cc42df;hb=d0150567cbd3c467ca94968c16d8f7dad05e935b;hpb=ca71827554747d1a4dabf9d82a7c4eacb6f908d5 diff --git a/lib/context.c b/lib/context.c index d6b1ec8b..87901b3c 100644 --- a/lib/context.c +++ b/lib/context.c @@ -17,6 +17,14 @@ * * The above copyright notice and this permission notice shall be included in * all copies or substantial portions of the Software. + * + * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR + * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY, + * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE + * AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER + * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, + * OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE + * SOFTWARE. */ #include @@ -26,16 +34,20 @@ #include #include #include +#include #include #include #include #include +#include -#include #include /* For O_RDONLY */ #include +static +void remove_trace_handle(struct bt_trace_handle *handle); + struct bt_context *bt_context_create(void) { struct bt_context *ctx; @@ -48,26 +60,29 @@ struct bt_context *bt_context_create(void) /* Instanciate the trace handle container */ ctx->trace_handles = g_hash_table_new_full(g_direct_hash, g_direct_equal, NULL, - (GDestroyNotify) bt_trace_handle_destroy); + (GDestroyNotify) remove_trace_handle); ctx->current_iterator = NULL; ctx->tc = g_new0(struct trace_collection, 1); - init_trace_collection(ctx->tc); + bt_init_trace_collection(ctx->tc); return ctx; } int bt_context_add_trace(struct bt_context *ctx, const char *path, const char *format_name, - void (*packet_seek)(struct stream_pos *pos, size_t index, + void (*packet_seek)(struct bt_stream_pos *pos, size_t index, int whence), - struct mmap_stream_list *stream_list, + struct bt_mmap_stream_list *stream_list, FILE *metadata) { - struct trace_descriptor *td; - struct format *fmt; + struct bt_trace_descriptor *td; + struct bt_format *fmt; struct bt_trace_handle *handle; - int ret; + int ret, closeret; + + if (!ctx || !format_name || (!path && !stream_list)) + return -EINVAL; fmt = bt_lookup_format(g_quark_from_string(format_name)); if (!fmt) { @@ -79,16 +94,16 @@ int bt_context_add_trace(struct bt_context *ctx, const char *path, if (path) { td = fmt->open_trace(path, O_RDONLY, packet_seek, NULL); if (!td) { - fprintf(stderr, "[warning] [Context] Cannot open_trace of the format %s .\n\n", - path); + fprintf(stderr, "[warning] [Context] Cannot open_trace of format %s at path %s.\n", + format_name, path); ret = -1; goto end; } } else { td = fmt->open_mmap_trace(stream_list, packet_seek, metadata); if (!td) { - fprintf(stderr, "[error] [Context] Cannot open_trace of the format %s .\n\n", - path); + fprintf(stderr, "[error] [Context] Cannot open_mmap_trace of format %s.\n\n", + format_name); ret = -1; goto end; } @@ -96,65 +111,103 @@ int bt_context_add_trace(struct bt_context *ctx, const char *path, /* Create an handle for the trace */ handle = bt_trace_handle_create(ctx); - if (handle < 0) { + if (!handle) { fprintf(stderr, "[error] [Context] Creating trace handle %s .\n\n", path); ret = -1; - goto end; + goto error_close; } handle->format = fmt; handle->td = td; - handle->timestamp_begin = fmt->timestamp_begin(td, handle); - handle->timestamp_end = fmt->timestamp_end(td, handle); - strncpy(handle->path, path, PATH_MAX); - handle->path[PATH_MAX - 1] = '\0'; + if (path) { + strncpy(handle->path, path, PATH_MAX); + handle->path[PATH_MAX - 1] = '\0'; + } + + ret = bt_trace_collection_add(ctx->tc, td); + if (ret != 0) + goto error_destroy_handle; if (fmt->set_handle) fmt->set_handle(td, handle); if (fmt->set_context) fmt->set_context(td, ctx); + if (fmt->convert_index_timestamp) { + ret = fmt->convert_index_timestamp(td); + if (ret < 0) + goto error_collection_del; + } + + if (fmt->timestamp_begin) + handle->real_timestamp_begin = fmt->timestamp_begin(td, + handle, BT_CLOCK_REAL); + if (fmt->timestamp_end) + handle->real_timestamp_end = fmt->timestamp_end(td, handle, + BT_CLOCK_REAL); + if (fmt->timestamp_begin) + handle->cycles_timestamp_begin = fmt->timestamp_begin(td, + handle, BT_CLOCK_CYCLES); + if (fmt->timestamp_end) + handle->cycles_timestamp_end = fmt->timestamp_end(td, handle, + BT_CLOCK_CYCLES); + /* Add new handle to container */ g_hash_table_insert(ctx->trace_handles, (gpointer) (unsigned long) handle->id, handle); - ret = trace_collection_add(ctx->tc, td); - if (ret == 0) - return handle->id; + + return handle->id; + +error_collection_del: + /* Remove from containers */ + bt_trace_collection_remove(handle->td->ctx->tc, handle->td); +error_destroy_handle: + bt_trace_handle_destroy(handle); +error_close: + closeret = fmt->close_trace(td); + if (closeret) { + fprintf(stderr, "Error in close_trace callback\n"); + } end: return ret; } -void bt_context_remove_trace(struct bt_context *ctx, int handle_id) +int bt_context_remove_trace(struct bt_context *ctx, int handle_id) { - struct bt_trace_handle *handle; - - handle = g_hash_table_lookup(ctx->trace_handles, - (gpointer) (unsigned long) handle_id); - assert(handle != NULL); - - /* Remove from containers */ - trace_collection_remove(ctx->tc, handle->td); - g_hash_table_remove(ctx->trace_handles, - (gpointer) (unsigned long) handle_id); + int ret = 0; - /* Close the trace */ - handle->format->close_trace(handle->td); + if (!ctx) { + ret = -EINVAL; + goto end; + } - /* Destory the handle */ - bt_trace_handle_destroy(handle); + /* + * Remove the handle. remove_trace_handle will be called + * automatically. + */ + if (!g_hash_table_remove(ctx->trace_handles, + (gpointer) (unsigned long) handle_id)) { + ret = -ENOENT; + goto end; + } +end: + return ret; } static void bt_context_destroy(struct bt_context *ctx) { - finalize_trace_collection(ctx->tc); + assert(ctx); - /* Remote all traces. The g_hash_table_destroy will call - * bt_trace_handle_destroy on each elements. + /* + * Remove all traces. The g_hash_table_destroy will call + * remove_trace_handle on each element. */ g_hash_table_destroy(ctx->trace_handles); + bt_finalize_trace_collection(ctx->tc); + /* ctx->tc should always be valid */ assert(ctx->tc != NULL); g_free(ctx->tc); @@ -163,12 +216,32 @@ void bt_context_destroy(struct bt_context *ctx) void bt_context_get(struct bt_context *ctx) { + assert(ctx); ctx->refcount++; } void bt_context_put(struct bt_context *ctx) { + assert(ctx); ctx->refcount--; if (ctx->refcount == 0) bt_context_destroy(ctx); } + +static +void remove_trace_handle(struct bt_trace_handle *handle) +{ + int ret; + + if (!handle->td->ctx) + return; + /* Remove from containers */ + bt_trace_collection_remove(handle->td->ctx->tc, handle->td); + /* Close the trace */ + ret = handle->format->close_trace(handle->td); + if (ret) { + fprintf(stderr, "Error in close_trace callback\n"); + } + + bt_trace_handle_destroy(handle); +}