ALSA: firewire-motu: add tracepoints for messages for unique protocol
authorTakashi Sakamoto <o-takashi@sakamocchi.jp>
Sun, 9 Apr 2017 12:33:29 +0000 (21:33 +0900)
committerTakashi Iwai <tiwai@suse.de>
Tue, 11 Apr 2017 06:30:47 +0000 (08:30 +0200)
MOTU units transfer/receive messages in each data block of their
isochronous packet payload. A part of content in the message is cleard for
MIDI message transmission, while the rest is unknown yet. Additional
features are required to assist users and developers to reveal the
details.

This commit adds tracepoints for the purpose. The tracepoints are designed
for MOTU's protocol version 2 and 3 (Protocol version 1 is not upstreamed
yet). In the tracepoints, events are probed to gather first two 24 bit
data chunks of each data block. The chunks are formatted into elements
of 64 bit array with padding in MSB.

Signed-off-by: Takashi Sakamoto <o-takashi@sakamocchi.jp>
Signed-off-by: Takashi Iwai <tiwai@suse.de>
sound/firewire/motu/amdtp-motu-trace.h
sound/firewire/motu/amdtp-motu.c

index 5862bf930c56749eb867a70b909379ecc3ee5944..cd0cbfa9f96f3d25c96a82cc27c069fecf21fabb 100644 (file)
@@ -15,6 +15,8 @@
 
 static void copy_sph(u32 *frame, __be32 *buffer, unsigned int data_blocks,
                     unsigned int data_block_quadlets);
+static void copy_message(u64 *frames, __be32 *buffer, unsigned int data_blocks,
+                        unsigned int data_block_quadlets);
 
 TRACE_EVENT(in_data_block_sph,
        TP_PROTO(struct amdtp_stream *s, unsigned int data_blocks, __be32 *buffer),
@@ -64,6 +66,54 @@ TRACE_EVENT(out_data_block_sph,
        )
 );
 
+TRACE_EVENT(in_data_block_message,
+       TP_PROTO(struct amdtp_stream *s, unsigned int data_blocks, __be32 *buffer),
+       TP_ARGS(s, data_blocks, buffer),
+       TP_STRUCT__entry(
+               __field(int, src)
+               __field(int, dst)
+               __field(unsigned int, data_blocks)
+               __dynamic_array(u64, messages, data_blocks)
+       ),
+       TP_fast_assign(
+               __entry->src = fw_parent_device(s->unit)->node_id;
+               __entry->dst = fw_parent_device(s->unit)->card->node_id;
+               __entry->data_blocks = data_blocks;
+               copy_message(__get_dynamic_array(messages), buffer, data_blocks, s->data_block_quadlets);
+       ),
+       TP_printk(
+               "%04x %04x %u %s",
+               __entry->src,
+               __entry->dst,
+               __entry->data_blocks,
+               __print_array(__get_dynamic_array(messages), __entry->data_blocks, 8)
+       )
+);
+
+TRACE_EVENT(out_data_block_message,
+       TP_PROTO(struct amdtp_stream *s, unsigned int data_blocks, __be32 *buffer),
+       TP_ARGS(s, data_blocks, buffer),
+       TP_STRUCT__entry(
+               __field(int, src)
+               __field(int, dst)
+               __field(unsigned int, data_blocks)
+               __dynamic_array(u64, messages, data_blocks)
+       ),
+       TP_fast_assign(
+               __entry->src = fw_parent_device(s->unit)->card->node_id;
+               __entry->dst = fw_parent_device(s->unit)->node_id;
+               __entry->data_blocks = data_blocks;
+               copy_message(__get_dynamic_array(messages), buffer, data_blocks, s->data_block_quadlets);
+       ),
+       TP_printk(
+               "%04x %04x %u %s",
+               __entry->src,
+               __entry->dst,
+               __entry->data_blocks,
+               __print_array(__get_dynamic_array(messages), __entry->data_blocks, 8)
+       )
+);
+
 #endif
 
 #undef TRACE_INCLUDE_PATH
index 2c77e8bb5424c9d22a7c9f1852de2a31085c7f40..996b5f8189189dc608511d4c5174ad285833d037 100644 (file)
@@ -280,6 +280,21 @@ static void copy_sph(u32 *frames, __be32 *buffer, unsigned int data_blocks,
        }
 }
 
+/* For tracepoints. */
+static void copy_message(u64 *frames, __be32 *buffer, unsigned int data_blocks,
+                        unsigned int data_block_quadlets)
+{
+       unsigned int i;
+
+       /* This is just for v2/v3 protocol. */
+       for (i = 0; i < data_blocks; ++i) {
+               *frames = (be32_to_cpu(buffer[1]) << 16) |
+                         (be32_to_cpu(buffer[2]) >> 16);
+               buffer += data_block_quadlets;
+               frames++;
+       }
+}
+
 static unsigned int process_tx_data_blocks(struct amdtp_stream *s,
                                __be32 *buffer, unsigned int data_blocks,
                                unsigned int *syt)
@@ -288,6 +303,7 @@ static unsigned int process_tx_data_blocks(struct amdtp_stream *s,
        struct snd_pcm_substream *pcm;
 
        trace_in_data_block_sph(s, data_blocks, buffer);
+       trace_in_data_block_message(s, data_blocks, buffer);
 
        if (p->midi_ports)
                read_midi_messages(s, buffer, data_blocks);
@@ -365,6 +381,7 @@ static unsigned int process_rx_data_blocks(struct amdtp_stream *s,
        write_sph(s, buffer, data_blocks);
 
        trace_out_data_block_sph(s, data_blocks, buffer);
+       trace_out_data_block_message(s, data_blocks, buffer);
 
        return data_blocks;
 }