telemetry: add IO counter for a HDA Host interface

Currently SOF can collect IO performance data only for HAD Link interface.
The corresponding counter on Host side is missing. This commit adds the
missing counter with DMA index used as the counter instance

Signed-off-by: Wojciech Jablonski <wojciech.jablonski@intel.com>

Author: wjablon1

src/audio/copier/host_copier.h

@@ -22,6 +22,10 @@
 #include <sof/lib/notifier.h>
 #include "copier.h"
 
+#ifdef CONFIG_SOF_TELEMETRY_IO_PERFORMANCE_MEASUREMENTS
+struct io_perf_data_item;
+#endif
+
 typedef void (*copy_callback_t)(struct comp_dev *dev, size_t bytes);
 
 struct host_data;
@@ -112,6 +116,9 @@ struct host_data {
 	uint64_t next_sync;
 	uint64_t period_in_cycles;
 #endif
+#ifdef CONFIG_SOF_TELEMETRY_IO_PERFORMANCE_MEASUREMENTS
+	struct io_perf_data_item *io_perf_host_byte_count;
+#endif
 };
 
 int host_common_new(struct host_data *hd, struct comp_dev *dev,

src/audio/host-legacy.c

@@ -25,6 +25,7 @@
 #include <rtos/string.h>
 #include <sof/ut.h>
 #include <sof/trace/trace.h>
+#include <sof/debug/telemetry/performance_monitor.h>
 #include <ipc/stream.h>
 #include <ipc/topology.h>
 #include <user/trace.h>
@@ -250,6 +251,10 @@ void host_common_update(struct host_data *hd, struct comp_dev *dev, uint32_t byt
 	if (ret < 0)
 		return;
 
+#ifdef CONFIG_SOF_TELEMETRY_IO_PERFORMANCE_MEASUREMENTS
+	io_perf_monitor_update_data(hd->io_perf_host_byte_count, bytes);
+#endif
+
 	hd->total_data_processed += bytes;
 
 	/* new local period, update host buffer position blks
@@ -599,6 +604,14 @@ static struct comp_dev *host_new(const struct comp_driver *drv,
 
 void host_common_free(struct host_data *hd)
 {
+#ifdef CONFIG_SOF_TELEMETRY_IO_PERFORMANCE_MEASUREMENTS
+	/* Check for NULL just in case the params() step is omitted */
+	if (hd->io_perf_host_byte_count) {
+		io_perf_monitor_release_slot(hd->io_perf_host_byte_count);
+		hd->io_perf_host_byte_count = NULL;
+	}
+#endif
+
 	dma_put(hd->dma);
 
 	ipc_msg_free(hd->msg);
@@ -829,6 +842,25 @@ int host_common_params(struct host_data *hd, struct comp_dev *dev,
 		pcm_get_conversion_function(audio_stream_get_frm_fmt(&hd->local_buffer->stream),
 					    audio_stream_get_frm_fmt(&hd->local_buffer->stream));
 
+#ifdef CONFIG_SOF_TELEMETRY_IO_PERFORMANCE_MEASUREMENTS
+	if (!hd->io_perf_host_byte_count) {
+		/* On the Host side, unlike the DAI side, an IO port direction enum values match
+		 * stream direction enum values (CAPTURE/PLAYBACK), so we can directly use
+		 * params->direction here.
+		 */
+		struct io_perf_data_item init_data = {
+			IO_PERF_HDA_ID,
+			hd->chan->index,
+			params->direction,
+			IO_PERF_POWERED_UP_ENABLED,
+			IO_PERF_D0IX_POWER_MODE,
+			0, 0, 0
+		};
+
+		io_perf_monitor_init_data(&hd->io_perf_host_byte_count, &init_data);
+	}
+#endif
+
 out:
 
 	hd->cb_dev = dev;

src/audio/host-zephyr.c

@@ -24,6 +24,7 @@
 #include <rtos/string.h>
 #include <sof/ut.h>
 #include <sof/trace/trace.h>
+#include <sof/debug/telemetry/performance_monitor.h>
 #include <ipc/stream.h>
 #include <ipc/topology.h>
 #include <user/trace.h>
@@ -262,6 +263,10 @@ void host_common_update(struct host_data *hd, struct comp_dev *dev, uint32_t byt
 		return;
 	}
 
+#ifdef CONFIG_SOF_TELEMETRY_IO_PERFORMANCE_MEASUREMENTS
+	io_perf_monitor_update_data(hd->io_perf_host_byte_count, bytes);
+#endif
+
 	hd->total_data_processed += bytes;
 
 	/* new local period, update host buffer position blks
@@ -771,6 +776,14 @@ __cold void host_common_free(struct host_data *hd)
 {
 	assert_can_be_cold();
 
+#ifdef CONFIG_SOF_TELEMETRY_IO_PERFORMANCE_MEASUREMENTS
+	/* Check for NULL just in case the params() step is omitted */
+	if (hd->io_perf_host_byte_count) {
+		io_perf_monitor_release_slot(hd->io_perf_host_byte_count);
+		hd->io_perf_host_byte_count = NULL;
+	}
+#endif
+
 	sof_dma_put(hd->dma);
 
 	ipc_msg_free(hd->msg);
@@ -1074,6 +1087,25 @@ int host_common_params(struct host_data *hd, struct comp_dev *dev,
 	hd->copy = hd->copy_type == COMP_COPY_ONE_SHOT ? host_copy_one_shot :
 		host_copy_normal;
 
+#ifdef CONFIG_SOF_TELEMETRY_IO_PERFORMANCE_MEASUREMENTS
+	if (!hd->io_perf_host_byte_count) {
+		/* On the Host side, unlike the DAI side, an IO port direction enum values match
+		 * stream direction enum values (CAPTURE/PLAYBACK), so we can directly use
+		 * params->direction here.
+		 */
+		struct io_perf_data_item init_data = {
+			IO_PERF_HDA_ID,
+			hd->chan->index,
+			params->direction,
+			IO_PERF_POWERED_UP_ENABLED,
+			IO_PERF_D0IX_POWER_MODE,
+			0, 0, 0
+		};
+
+		io_perf_monitor_init_data(&hd->io_perf_host_byte_count, &init_data);
+	}
+#endif
+
 	return 0;
 
 err_free_block_cfg: