X-Git-Url: http://git.efficios.com/?p=babeltrace.git;a=blobdiff_plain;f=lib%2Fcontext.c;h=87901b3c61027aadea02e0ddf0f9aa042b4aaac1;hp=0f727a180a53c5834c07ff2830544b61b7c73f97;hb=4a6f6ed3aeb9845dec4b8ec9ca8573ed1c064187;hpb=91f361c1757256c11988ab16376419a72592e1a9 diff --git a/lib/context.c b/lib/context.c index 0f727a18..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,6 +34,7 @@ #include #include #include +#include #include #include #include @@ -36,6 +45,9 @@ #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,26 @@ 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; @@ -82,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; } @@ -99,11 +111,11 @@ 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; @@ -112,67 +124,90 @@ int bt_context_add_trace(struct bt_context *ctx, const char *path, 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) - goto end; - - ret = fmt->convert_index_timestamp(td); - if (ret < 0) - goto end; - - handle->real_timestamp_begin = fmt->timestamp_begin(td, handle, BT_CLOCK_REAL); - handle->real_timestamp_end = fmt->timestamp_end(td, handle, BT_CLOCK_REAL); - handle->cycles_timestamp_begin = fmt->timestamp_begin(td, handle, BT_CLOCK_CYCLES); - handle->cycles_timestamp_end = fmt->timestamp_end(td, handle, BT_CLOCK_CYCLES); 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; } int bt_context_remove_trace(struct bt_context *ctx, int handle_id) { - struct bt_trace_handle *handle; - - if (!ctx) - return -EINVAL; + int ret = 0; - handle = g_hash_table_lookup(ctx->trace_handles, - (gpointer) (unsigned long) handle_id); - if (!handle) - return -ENOENT; - - /* Remove from containers */ - trace_collection_remove(ctx->tc, handle->td); - /* Close the trace */ - handle->format->close_trace(handle->td); + if (!ctx) { + ret = -EINVAL; + goto end; + } - /* Remove and free the handle */ - g_hash_table_remove(ctx->trace_handles, - (gpointer) (unsigned long) handle_id); - return 0; + /* + * 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) { assert(ctx); - finalize_trace_collection(ctx->tc); - /* 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); @@ -192,3 +227,21 @@ void bt_context_put(struct bt_context *ctx) 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); +}