Re: [PATCH] ALSA: hda: trace PCM open only after assigning a stream

[Takashi Iwai]

On Sun, 13 Sep 2026 14:51:54 +0200,
Slavin Liu wrote:
> 
> Stream assignment can fail when hardware streams are exhausted.
> Move the tracepoint after the NULL check because its payload accesses
> the assigned stream tag.
> 
> Detected by static analysis and reviewed with AI-assisted source auditing.
> 
> Fixes: 184865085b88 ("ALSA: hda - rename hda_intel_trace.h to hda_controller_trace.h")
> Assisted-by: LLM
> Signed-off-by: Slavin Liu <bolin.liu@xxxxxxxxxx>

Applied now.

I think I wanted to track the error state, too, and that's likely the
reason the code was put there.  But, judging from the current code,
better to move as in this patch in the end.


thanks,

Takashi