mirror of
https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git
synced 2024-09-12 13:55:32 +00:00
![Takashi Sakamoto](/assets/img/avatar_default.png)
An initial commit to add tracepoints for packets without CIP headers
introduces a wrong assignment to 'data_blocks' value of
'out_packet_without_header' tracepoint.
This commit fixes the bug.
Cc: <stable@vger.kernel.org> # v4.12+
Fixes: b164d2fd6e
('ALSA: firewire_lib: add tracepoints for packets without CIP headers')
Signed-off-by: Takashi Sakamoto <o-takashi@sakamocchi.jp>
Signed-off-by: Takashi Iwai <tiwai@suse.de>
198 lines
6 KiB
C
198 lines
6 KiB
C
/*
|
|
* amdtp-stream-trace.h - tracepoint definitions to dump a part of packet data
|
|
*
|
|
* Copyright (c) 2016 Takashi Sakamoto
|
|
* Licensed under the terms of the GNU General Public License, version 2.
|
|
*/
|
|
|
|
#undef TRACE_SYSTEM
|
|
#define TRACE_SYSTEM snd_firewire_lib
|
|
|
|
#if !defined(_AMDTP_STREAM_TRACE_H) || defined(TRACE_HEADER_MULTI_READ)
|
|
#define _AMDTP_STREAM_TRACE_H
|
|
|
|
#include <linux/tracepoint.h>
|
|
|
|
TRACE_EVENT(in_packet,
|
|
TP_PROTO(const struct amdtp_stream *s, u32 cycles, u32 *cip_header, unsigned int payload_length, unsigned int index),
|
|
TP_ARGS(s, cycles, cip_header, payload_length, index),
|
|
TP_STRUCT__entry(
|
|
__field(unsigned int, second)
|
|
__field(unsigned int, cycle)
|
|
__field(int, channel)
|
|
__field(int, src)
|
|
__field(int, dest)
|
|
__field(u32, cip_header0)
|
|
__field(u32, cip_header1)
|
|
__field(unsigned int, payload_quadlets)
|
|
__field(unsigned int, packet_index)
|
|
__field(unsigned int, irq)
|
|
__field(unsigned int, index)
|
|
),
|
|
TP_fast_assign(
|
|
__entry->second = cycles / CYCLES_PER_SECOND;
|
|
__entry->cycle = cycles % CYCLES_PER_SECOND;
|
|
__entry->channel = s->context->channel;
|
|
__entry->src = fw_parent_device(s->unit)->node_id;
|
|
__entry->dest = fw_parent_device(s->unit)->card->node_id;
|
|
__entry->cip_header0 = cip_header[0];
|
|
__entry->cip_header1 = cip_header[1];
|
|
__entry->payload_quadlets = payload_length / 4;
|
|
__entry->packet_index = s->packet_index;
|
|
__entry->irq = !!in_interrupt();
|
|
__entry->index = index;
|
|
),
|
|
TP_printk(
|
|
"%02u %04u %04x %04x %02d %08x %08x %03u %02u %01u %02u",
|
|
__entry->second,
|
|
__entry->cycle,
|
|
__entry->src,
|
|
__entry->dest,
|
|
__entry->channel,
|
|
__entry->cip_header0,
|
|
__entry->cip_header1,
|
|
__entry->payload_quadlets,
|
|
__entry->packet_index,
|
|
__entry->irq,
|
|
__entry->index)
|
|
);
|
|
|
|
TRACE_EVENT(out_packet,
|
|
TP_PROTO(const struct amdtp_stream *s, u32 cycles, __be32 *cip_header, unsigned int payload_length, unsigned int index),
|
|
TP_ARGS(s, cycles, cip_header, payload_length, index),
|
|
TP_STRUCT__entry(
|
|
__field(unsigned int, second)
|
|
__field(unsigned int, cycle)
|
|
__field(int, channel)
|
|
__field(int, src)
|
|
__field(int, dest)
|
|
__field(u32, cip_header0)
|
|
__field(u32, cip_header1)
|
|
__field(unsigned int, payload_quadlets)
|
|
__field(unsigned int, packet_index)
|
|
__field(unsigned int, irq)
|
|
__field(unsigned int, index)
|
|
),
|
|
TP_fast_assign(
|
|
__entry->second = cycles / CYCLES_PER_SECOND;
|
|
__entry->cycle = cycles % CYCLES_PER_SECOND;
|
|
__entry->channel = s->context->channel;
|
|
__entry->src = fw_parent_device(s->unit)->card->node_id;
|
|
__entry->dest = fw_parent_device(s->unit)->node_id;
|
|
__entry->cip_header0 = be32_to_cpu(cip_header[0]);
|
|
__entry->cip_header1 = be32_to_cpu(cip_header[1]);
|
|
__entry->payload_quadlets = payload_length / 4;
|
|
__entry->packet_index = s->packet_index;
|
|
__entry->irq = !!in_interrupt();
|
|
__entry->index = index;
|
|
),
|
|
TP_printk(
|
|
"%02u %04u %04x %04x %02d %08x %08x %03u %02u %01u %02u",
|
|
__entry->second,
|
|
__entry->cycle,
|
|
__entry->src,
|
|
__entry->dest,
|
|
__entry->channel,
|
|
__entry->cip_header0,
|
|
__entry->cip_header1,
|
|
__entry->payload_quadlets,
|
|
__entry->packet_index,
|
|
__entry->irq,
|
|
__entry->index)
|
|
);
|
|
|
|
TRACE_EVENT(in_packet_without_header,
|
|
TP_PROTO(const struct amdtp_stream *s, u32 cycles, unsigned int payload_quadlets, unsigned int data_blocks, unsigned int index),
|
|
TP_ARGS(s, cycles, payload_quadlets, data_blocks, index),
|
|
TP_STRUCT__entry(
|
|
__field(unsigned int, second)
|
|
__field(unsigned int, cycle)
|
|
__field(int, channel)
|
|
__field(int, src)
|
|
__field(int, dest)
|
|
__field(unsigned int, payload_quadlets)
|
|
__field(unsigned int, data_blocks)
|
|
__field(unsigned int, data_block_counter)
|
|
__field(unsigned int, packet_index)
|
|
__field(unsigned int, irq)
|
|
__field(unsigned int, index)
|
|
),
|
|
TP_fast_assign(
|
|
__entry->second = cycles / CYCLES_PER_SECOND;
|
|
__entry->cycle = cycles % CYCLES_PER_SECOND;
|
|
__entry->channel = s->context->channel;
|
|
__entry->src = fw_parent_device(s->unit)->node_id;
|
|
__entry->dest = fw_parent_device(s->unit)->card->node_id;
|
|
__entry->payload_quadlets = payload_quadlets;
|
|
__entry->data_blocks = data_blocks,
|
|
__entry->data_block_counter = s->data_block_counter,
|
|
__entry->packet_index = s->packet_index;
|
|
__entry->irq = !!in_interrupt();
|
|
__entry->index = index;
|
|
),
|
|
TP_printk(
|
|
"%02u %04u %04x %04x %02d %03u %3u %3u %02u %01u %02u",
|
|
__entry->second,
|
|
__entry->cycle,
|
|
__entry->src,
|
|
__entry->dest,
|
|
__entry->channel,
|
|
__entry->payload_quadlets,
|
|
__entry->data_blocks,
|
|
__entry->data_block_counter,
|
|
__entry->packet_index,
|
|
__entry->irq,
|
|
__entry->index)
|
|
);
|
|
|
|
TRACE_EVENT(out_packet_without_header,
|
|
TP_PROTO(const struct amdtp_stream *s, u32 cycles, unsigned int payload_length, unsigned int data_blocks, unsigned int index),
|
|
TP_ARGS(s, cycles, payload_length, data_blocks, index),
|
|
TP_STRUCT__entry(
|
|
__field(unsigned int, second)
|
|
__field(unsigned int, cycle)
|
|
__field(int, channel)
|
|
__field(int, src)
|
|
__field(int, dest)
|
|
__field(unsigned int, payload_quadlets)
|
|
__field(unsigned int, data_blocks)
|
|
__field(unsigned int, data_block_counter)
|
|
__field(unsigned int, packet_index)
|
|
__field(unsigned int, irq)
|
|
__field(unsigned int, index)
|
|
),
|
|
TP_fast_assign(
|
|
__entry->second = cycles / CYCLES_PER_SECOND;
|
|
__entry->cycle = cycles % CYCLES_PER_SECOND;
|
|
__entry->channel = s->context->channel;
|
|
__entry->src = fw_parent_device(s->unit)->card->node_id;
|
|
__entry->dest = fw_parent_device(s->unit)->node_id;
|
|
__entry->payload_quadlets = payload_length / 4;
|
|
__entry->data_blocks = data_blocks,
|
|
__entry->data_block_counter = s->data_block_counter,
|
|
__entry->packet_index = s->packet_index;
|
|
__entry->irq = !!in_interrupt();
|
|
__entry->index = index;
|
|
),
|
|
TP_printk(
|
|
"%02u %04u %04x %04x %02d %03u %02u %03u %02u %01u %02u",
|
|
__entry->second,
|
|
__entry->cycle,
|
|
__entry->src,
|
|
__entry->dest,
|
|
__entry->channel,
|
|
__entry->payload_quadlets,
|
|
__entry->data_blocks,
|
|
__entry->data_block_counter,
|
|
__entry->packet_index,
|
|
__entry->irq,
|
|
__entry->index)
|
|
);
|
|
|
|
#endif
|
|
|
|
#undef TRACE_INCLUDE_PATH
|
|
#define TRACE_INCLUDE_PATH .
|
|
#undef TRACE_INCLUDE_FILE
|
|
#define TRACE_INCLUDE_FILE amdtp-stream-trace
|
|
#include <trace/define_trace.h>
|