#include "lib/logging.h"
#include "lib/assert-pre.h"
-#include "compat/uuid.h"
+#include "common/uuid.h"
#include "clock-class.h"
#include "clock-snapshot.h"
#include <babeltrace2/trace-ir/clock-snapshot-const.h>
BT_HIDDEN
void bt_clock_snapshot_destroy(struct bt_clock_snapshot *clock_snapshot)
{
+ BT_ASSERT(clock_snapshot);
BT_LIB_LOGD("Destroying clock snapshot: %!+k", clock_snapshot);
BT_OBJECT_PUT_REF_AND_RESET(clock_snapshot->clock_class);
g_free(clock_snapshot);
bt_object_init_unique(&ret->base);
ret->clock_class = clock_class;
- bt_object_get_no_null_check(clock_class);
+ bt_object_get_ref_no_null_check(clock_class);
bt_clock_class_freeze(clock_class);
BT_LIB_LOGD("Created clock snapshot object: %!+k", ret);
{
struct bt_clock_snapshot *clock_snapshot = NULL;
- BT_ASSERT(clock_class);
+ BT_ASSERT_DBG(clock_class);
clock_snapshot = bt_object_pool_create_object(&clock_class->cs_pool);
if (!clock_snapshot) {
BT_LIB_LOGE_APPEND_CAUSE(
"Cannot allocate one clock snapshot from clock class's clock snapshot pool: "
"%![cc-]+K", clock_class);
- goto error;
+ goto end;
}
if (G_LIKELY(!clock_snapshot->clock_class)) {
clock_snapshot->clock_class = clock_class;
- bt_object_get_no_null_check(clock_class);
- }
-
- goto end;
-
-error:
- if (clock_snapshot) {
- bt_clock_snapshot_recycle(clock_snapshot);
- clock_snapshot = NULL;
+ bt_object_get_ref_no_null_check(clock_class);
}
end:
{
struct bt_clock_class *clock_class;
- BT_ASSERT(clock_snapshot);
+ BT_ASSERT_DBG(clock_snapshot);
BT_LIB_LOGD("Recycling clock snapshot: %!+k", clock_snapshot);
/*
*/
bt_clock_snapshot_reset(clock_snapshot);
clock_class = clock_snapshot->clock_class;
- BT_ASSERT(clock_class);
+ BT_ASSERT_DBG(clock_class);
clock_snapshot->clock_class = NULL;
bt_object_pool_recycle_object(&clock_class->cs_pool, clock_snapshot);
bt_object_put_ref(clock_class);
uint64_t bt_clock_snapshot_get_value(
const struct bt_clock_snapshot *clock_snapshot)
{
- BT_ASSERT_PRE_NON_NULL(clock_snapshot, "Clock snapshot");
- BT_ASSERT_PRE(clock_snapshot->is_set,
+ BT_ASSERT_PRE_DEV_NON_NULL(clock_snapshot, "Clock snapshot");
+ BT_ASSERT_PRE_DEV(clock_snapshot->is_set,
"Clock snapshot is not set: %!+k", clock_snapshot);
return clock_snapshot->value_cycles;
}
{
int ret = BT_FUNC_STATUS_OK;
- BT_ASSERT_PRE_NON_NULL(clock_snapshot, "Clock snapshot");
- BT_ASSERT_PRE_NON_NULL(ret_value_ns, "Value (ns) (output)");
- BT_ASSERT_PRE(clock_snapshot->is_set,
+ BT_ASSERT_PRE_DEV_NON_NULL(clock_snapshot, "Clock snapshot");
+ BT_ASSERT_PRE_DEV_NON_NULL(ret_value_ns, "Value (ns) (output)");
+ BT_ASSERT_PRE_DEV(clock_snapshot->is_set,
"Clock snapshot is not set: %!+k", clock_snapshot);
if (clock_snapshot->ns_from_origin_overflows) {
- BT_LIB_LOGD("Clock snapshot, once converted to nanoseconds from origin, "
+ BT_LIB_LOGE_APPEND_CAUSE(
+ "Clock snapshot, once converted to nanoseconds from origin, "
"overflows the signed 64-bit integer range: "
"%![cs-]+k", clock_snapshot);
- ret = BT_FUNC_STATUS_OVERFLOW;
+ ret = BT_FUNC_STATUS_OVERFLOW_ERROR;
goto end;
}
const struct bt_clock_class *bt_clock_snapshot_borrow_clock_class_const(
const struct bt_clock_snapshot *clock_snapshot)
{
- BT_ASSERT_PRE_NON_NULL(clock_snapshot, "Clock snapshot");
+ BT_ASSERT_PRE_DEV_NON_NULL(clock_snapshot, "Clock snapshot");
return clock_snapshot->clock_class;
}