struct lttng_channel: split protocol ABI from instrumentation ABI

[lttng-ust.git] / liblttng-ust / lttng-ring-buffer-client.h

diff --git a/liblttng-ust/lttng-ring-buffer-client.h b/liblttng-ust/lttng-ring-buffer-client.h
index 31934b674637c71df83351a65c05fc07ea1bcfeb..f791cd1eed12ee405f8bfb1aad712dfd680bb680 100644 (file)
--- a/liblttng-ust/lttng-ring-buffer-client.h
+++ b/liblttng-ust/lttng-ring-buffer-client.h
@@ -649,33 +649,43 @@ struct lttng_channel *_channel_create(const char *name,
                                const int *stream_fds, int nr_stream_fds,
                                int64_t blocking_timeout)
 {
-       struct lttng_channel chan_priv_init;
+       struct lttng_ust_abi_channel_config chan_priv_init;
        struct lttng_ust_shm_handle *handle;
        struct lttng_channel *lttng_chan;
-       void *priv;
+
+       lttng_chan = zmalloc(sizeof(struct lttng_channel));
+       if (!lttng_chan)
+               return NULL;
+       memcpy(lttng_chan->uuid, uuid, LTTNG_UST_UUID_LEN);
+       lttng_chan->id = chan_id;
 
        memset(&chan_priv_init, 0, sizeof(chan_priv_init));
        memcpy(chan_priv_init.uuid, uuid, LTTNG_UST_UUID_LEN);
        chan_priv_init.id = chan_id;
+
        handle = channel_create(&client_config, name,
-                       &priv, __alignof__(struct lttng_channel),
-                       sizeof(struct lttng_channel),
+                       __alignof__(struct lttng_ust_abi_channel_config),
+                       sizeof(struct lttng_ust_abi_channel_config),
                        &chan_priv_init,
-                       buf_addr, subbuf_size, num_subbuf,
+                       lttng_chan, buf_addr, subbuf_size, num_subbuf,
                        switch_timer_interval, read_timer_interval,
                        stream_fds, nr_stream_fds, blocking_timeout);
        if (!handle)
-               return NULL;
-       lttng_chan = priv;
+               goto error;
        lttng_chan->handle = handle;
        lttng_chan->chan = shmp(handle, handle->chan);
        return lttng_chan;
+
+error:
+       free(lttng_chan);
+       return NULL;
 }
 
 static
 void lttng_channel_destroy(struct lttng_channel *chan)
 {
        channel_destroy(chan->chan, chan->handle, 1);
+       free(chan);
 }
 
 static
@@ -797,15 +807,18 @@ static struct lttng_transport lttng_relay_transport = {
        .name = "relay-" RING_BUFFER_MODE_TEMPLATE_STRING "-mmap",
        .ops = {
                .struct_size = sizeof(struct lttng_ust_channel_ops),
-               .channel_create = _channel_create,
-               .channel_destroy = lttng_channel_destroy,
+               .priv = __LTTNG_COMPOUND_LITERAL(struct lttng_ust_channel_ops_private, {
+                       .pub = &lttng_relay_transport.ops,
+                       .channel_create = _channel_create,
+                       .channel_destroy = lttng_channel_destroy,
+                       .packet_avail_size = NULL,      /* Would be racy anyway */
+                       .is_finalized = lttng_is_finalized,
+                       .is_disabled = lttng_is_disabled,
+                       .flush_buffer = lttng_flush_buffer,
+               }),
                .event_reserve = lttng_event_reserve,
                .event_commit = lttng_event_commit,
                .event_write = lttng_event_write,
-               .packet_avail_size = NULL,      /* Would be racy anyway */
-               .is_finalized = lttng_is_finalized,
-               .is_disabled = lttng_is_disabled,
-               .flush_buffer = lttng_flush_buffer,
                .event_strcpy = lttng_event_strcpy,
        },
        .client_config = &client_config,