ASoC: SOF: ipc3-dtrace: Introduce SOF_DTRACE_INITIALIZING state

With the new state we can make sure we are not missing the first
host_offset update.

In case the dtrace is small, the DMA copy will be fast and depending on
the moonphase it might be done before we set the sdev->dtrace_state  to
SOF_DTRACE_ENABLED.

The DMA will start the copy as soon as the host starts the DMA. Set the
dtrace to enabled before we let the DMA to run in order to avoid missing
the position update.

The new state is needed to cover architectures where the host side
snd_sof_dma_trace_trigger() is a NOP and the dtrace in the firmware is
ready as soon as the IPC message has been processed.

Signed-off-by: Peter Ujfalusi <peter.ujfalusi@linux.intel.com>
Reviewed-by: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
Reviewed-by: Bard Liao <yung-chuan.liao@linux.intel.com>
Link: https://lore.kernel.org/r/20220610080119.30880-2-peter.ujfalusi@linux.intel.com
Signed-off-by: Mark Brown <broonie@kernel.org>

diff --git a/sound/soc/sof/ipc3-dtrace.c b/sound/soc/sof/ipc3-dtrace.c
index b4e1343..9292ff7 100644 (file)
--- a/sound/soc/sof/ipc3-dtrace.c
+++ b/sound/soc/sof/ipc3-dtrace.c
@@ -18,6 +18,7 @@
 enum sof_dtrace_state {
        SOF_DTRACE_DISABLED,
        SOF_DTRACE_STOPPED,
+       SOF_DTRACE_INITIALIZING,
        SOF_DTRACE_ENABLED,
 };
 
@@ -32,6 +33,15 @@ struct sof_dtrace_priv {
        enum sof_dtrace_state dtrace_state;
 };
 
+static bool trace_pos_update_expected(struct sof_dtrace_priv *priv)
+{
+       if (priv->dtrace_state == SOF_DTRACE_ENABLED ||
+           priv->dtrace_state == SOF_DTRACE_INITIALIZING)
+               return true;
+
+       return false;
+}
+
 static int trace_filter_append_elem(struct snd_sof_dev *sdev, u32 key, u32 value,
                                    struct sof_ipc_trace_filter_elem *elem_list,
                                    int capacity, int *counter)
@@ -274,7 +284,7 @@ static size_t sof_wait_dtrace_avail(struct snd_sof_dev *sdev, loff_t pos,
        if (ret)
                return ret;
 
-       if (priv->dtrace_state != SOF_DTRACE_ENABLED && priv->dtrace_draining) {
+       if (priv->dtrace_draining && !trace_pos_update_expected(priv)) {
                /*
                 * tracing has ended and all traces have been
                 * read by client, return EOF
@@ -445,6 +455,7 @@ static int ipc3_dtrace_enable(struct snd_sof_dev *sdev)
        dev_dbg(sdev->dev, "%s: stream_tag: %d\n", __func__, params.stream_tag);
 
        /* send IPC to the DSP */
+       priv->dtrace_state = SOF_DTRACE_INITIALIZING;
        ret = sof_ipc_tx_message(sdev->ipc, &params, sizeof(params), &ipc_reply, sizeof(ipc_reply));
        if (ret < 0) {
                dev_err(sdev->dev, "can't set params for DMA for trace %d\n", ret);
@@ -452,17 +463,18 @@ static int ipc3_dtrace_enable(struct snd_sof_dev *sdev)
        }
 
 start:
+       priv->dtrace_state = SOF_DTRACE_ENABLED;
+
        ret = sof_dtrace_host_trigger(sdev, SNDRV_PCM_TRIGGER_START);
        if (ret < 0) {
                dev_err(sdev->dev, "Host dtrace trigger start failed: %d\n", ret);
                goto trace_release;
        }
 
-       priv->dtrace_state = SOF_DTRACE_ENABLED;
-
        return 0;
 
 trace_release:
+       priv->dtrace_state = SOF_DTRACE_DISABLED;
        sof_dtrace_host_release(sdev);
        return ret;
 }
@@ -546,7 +558,7 @@ int ipc3_dtrace_posn_update(struct snd_sof_dev *sdev,
        if (!sdev->fw_trace_is_supported)
                return 0;
 
-       if (priv->dtrace_state == SOF_DTRACE_ENABLED &&
+       if (trace_pos_update_expected(priv) &&
            priv->host_offset != posn->host_offset) {
                priv->host_offset = posn->host_offset;
                wake_up(&priv->trace_sleep);