relayd: Extend relayd get configuration command to check for trace format support
[lttng-tools.git] / src / bin / lttng-relayd / main.cpp
index e1c74e358ebff36c9426cba3c2cc82a1ece4b3c8..e2d947cc693aa96920c2f0f0cbb6d39ffbdea6ce 100644 (file)
@@ -1329,7 +1329,7 @@ static void *relay_thread_dispatcher(void *data __attribute__((unused)))
                                /* Continue thread execution */
                                break;
                        }
-                       new_conn = caa_container_of(node, struct relay_connection, qnode);
+                       new_conn = lttng::utils::container_of(node, &relay_connection::qnode);
 
                        DBG("Dispatching request waiting on sock %d", new_conn->sock->fd);
 
@@ -2539,12 +2539,14 @@ static ssize_t relay_unpack_rotate_streams_header(
         * We start by "unpacking" `stream_count` to figure out the padding length
         * emited by our peer.
         */
-       memcpy(&rotate_streams.stream_count, payload->data,
-                       sizeof(rotate_streams.stream_count));
-       rotate_streams = (typeof(rotate_streams)) {
-               .stream_count = be32toh(rotate_streams.stream_count),
-               .new_chunk_id = LTTNG_OPTIONAL_INIT_UNSET,
-       };
+       {
+               decltype(rotate_streams.stream_count) stream_count;
+
+               memcpy(&stream_count, payload->data, sizeof(stream_count));
+               rotate_streams.stream_count = be32toh(stream_count);
+       }
+
+       rotate_streams.new_chunk_id = LTTNG_OPTIONAL_INIT_UNSET;
 
        /*
         * Payload size expected given the possible padding lengths in
@@ -2574,13 +2576,11 @@ static ssize_t relay_unpack_rotate_streams_header(
                memcpy(&packed_rotate_streams, payload->data, header_len);
 
                /* Unpack the packed structure to the natively-packed version. */
-               *_rotate_streams = (typeof(*_rotate_streams)) {
-                       .stream_count = be32toh(packed_rotate_streams.stream_count),
-                       .new_chunk_id = (typeof(_rotate_streams->new_chunk_id)) {
-                               .is_set = !!packed_rotate_streams.new_chunk_id.is_set,
-                               .value = be64toh(packed_rotate_streams.new_chunk_id.value),
-                       }
+               _rotate_streams->new_chunk_id = (typeof(_rotate_streams->new_chunk_id)){
+                       .is_set = !!packed_rotate_streams.new_chunk_id.is_set,
+                       .value = be64toh(packed_rotate_streams.new_chunk_id.value),
                };
+               _rotate_streams->stream_count = be32toh(packed_rotate_streams.stream_count);
        } else if (payload->size == expected_payload_size_3_bytes_padding) {
                struct lttcomm_relayd_rotate_streams_3_bytes_padding padded_rotate_streams;
 
@@ -2590,13 +2590,11 @@ static ssize_t relay_unpack_rotate_streams_header(
                memcpy(&padded_rotate_streams, payload->data, header_len);
 
                /* Unpack the 3-byte padded structure to the natively-packed version. */
-               *_rotate_streams = (typeof(*_rotate_streams)) {
-                       .stream_count = be32toh(padded_rotate_streams.stream_count),
-                       .new_chunk_id = (typeof(_rotate_streams->new_chunk_id)) {
-                               .is_set = !!padded_rotate_streams.new_chunk_id.is_set,
-                               .value = be64toh(padded_rotate_streams.new_chunk_id.value),
-                       }
+               _rotate_streams->new_chunk_id = (typeof(_rotate_streams->new_chunk_id)){
+                       .is_set = !!padded_rotate_streams.new_chunk_id.is_set,
+                       .value = be64toh(padded_rotate_streams.new_chunk_id.value),
                };
+               _rotate_streams->stream_count = be32toh(padded_rotate_streams.stream_count);
        } else if (payload->size == expected_payload_size_7_bytes_padding) {
                struct lttcomm_relayd_rotate_streams_7_bytes_padding padded_rotate_streams;
 
@@ -2606,13 +2604,11 @@ static ssize_t relay_unpack_rotate_streams_header(
                memcpy(&padded_rotate_streams, payload->data, header_len);
 
                /* Unpack the 7-byte padded structure to the natively-packed version. */
-               *_rotate_streams = (typeof(*_rotate_streams)) {
-                       .stream_count = be32toh(padded_rotate_streams.stream_count),
-                       .new_chunk_id = (typeof(_rotate_streams->new_chunk_id)) {
-                               .is_set = !!padded_rotate_streams.new_chunk_id.is_set,
-                               .value = be64toh(padded_rotate_streams.new_chunk_id.value),
-                       }
+               _rotate_streams->new_chunk_id = (typeof(_rotate_streams->new_chunk_id)){
+                       .is_set = !!padded_rotate_streams.new_chunk_id.is_set,
+                       .value = be64toh(padded_rotate_streams.new_chunk_id.value),
                };
+               _rotate_streams->stream_count = be32toh(padded_rotate_streams.stream_count);
 
                header_len = sizeof(padded_rotate_streams);
        } else {
@@ -3290,9 +3286,14 @@ static int relay_get_configuration(
        struct lttcomm_relayd_get_configuration *msg;
        struct lttcomm_relayd_get_configuration_reply reply = {};
        struct lttng_buffer_view header_view;
+       struct lttng_dynamic_buffer buffer;
+       struct lttng_dynamic_buffer extra_payload;
        uint64_t query_flags = 0;
        uint64_t result_flags = 0;
 
+       lttng_dynamic_buffer_init(&buffer);
+       lttng_dynamic_buffer_init(&extra_payload);
+
        header_view = lttng_buffer_view_from_view(payload, 0, sizeof(*msg));
        if (!lttng_buffer_view_is_valid(&header_view)) {
                ERR("Failed to receive payload of chunk close command");
@@ -3304,26 +3305,47 @@ static int relay_get_configuration(
        msg = (typeof(msg)) header_view.data;
        query_flags = be64toh(msg->query_flags);
 
-       if (query_flags) {
+       if (query_flags & ~LTTCOMM_RELAYD_CONFIGURATION_QUERY_FLAG_MASK) {
                ret = LTTNG_ERR_INVALID_PROTOCOL;
                goto reply;
        }
+
        if (opt_allow_clear) {
                result_flags |= LTTCOMM_RELAYD_CONFIGURATION_FLAG_CLEAR_ALLOWED;
        }
-       ret = 0;
+
+       if (query_flags & LTTCOMM_RELAYD_CONFIGURATION_QUERY_FLAG_SUPPORTED_TRACE_FORMAT) {
+               uint64_t supported_trace_format = 0;
+
+               supported_trace_format |= LTTCOMM_RELAYD_CONFIGURATION_TRACE_FORMAT_SUPPORTED_CTF1;
+
+               supported_trace_format = htobe64(supported_trace_format);
+
+               lttcomm_relayd_get_configuration_specialized_query_reply s_reply = {};
+               s_reply.query_flag = htobe64(
+                               LTTCOMM_RELAYD_CONFIGURATION_QUERY_FLAG_SUPPORTED_TRACE_FORMAT);
+               s_reply.payload_len = htobe64((uint64_t) sizeof(supported_trace_format));
+               lttng_dynamic_buffer_append(&extra_payload, &s_reply, sizeof(s_reply));
+               lttng_dynamic_buffer_append(&extra_payload, &supported_trace_format,
+                               sizeof(supported_trace_format));
+       }
+
 reply:
-       reply.generic.ret_code = htobe32((uint32_t) (ret == 0 ? LTTNG_OK : LTTNG_ERR_INVALID_PROTOCOL));
+       reply.generic.ret_code =
+                       htobe32((uint32_t) (ret == 0 ? LTTNG_OK : LTTNG_ERR_INVALID_PROTOCOL));
        reply.relayd_configuration_flags = htobe64(result_flags);
 
-       send_ret = conn->sock->ops->sendmsg(
-                       conn->sock, &reply, sizeof(reply), 0);
-       if (send_ret < (ssize_t) sizeof(reply)) {
-               ERR("Failed to send \"get configuration\" command reply (ret = %zd)",
-                               send_ret);
+       lttng_dynamic_buffer_append(&buffer, &reply, sizeof(reply));
+       lttng_dynamic_buffer_append_buffer(&buffer, &extra_payload);
+
+       send_ret = conn->sock->ops->sendmsg(conn->sock, buffer.data, buffer.size, 0);
+       if (send_ret < (ssize_t) buffer.size) {
+               ERR("Failed to send \"get configuration\" command reply (ret = %zd)", send_ret);
                ret = -1;
        }
 end_no_reply:
+       lttng_dynamic_buffer_reset(&extra_payload);
+       lttng_dynamic_buffer_reset(&buffer);
        return ret;
 }
 
This page took 0.026674 seconds and 5 git commands to generate.